RFR(s): 8198267: Allow GCId::current_raw() calls from non-NamedThreads
per.liden at oracle.com
Mon Feb 19 13:46:41 UTC 2018
On 02/19/2018 02:37 PM, Stefan Karlsson wrote:
> On 2018-02-19 14:08, Per Liden wrote:
>> Got some off-line feedback from StefanK. After some discussions we
>> decided to also make the following changes:
>> 1) Renamed current_raw() to current_or_undefined()
>> 2) Replaces a few uses of current_raw() with currentNamedThread->gc_id()
>> 3) Removes the special constructor for AbstractGangTask, which is no
>> longer needed. With the new semantics of current_or_undefined() the
>> default constructor can be used by G1PretouchTask.
> For the record, I think I would have preferred to use GCId::current() in
> AbstractGangTask, and GCId::current_or_undefined() when setting up
> G1PretouchTask, but I'm not going to push for this.
>> Webrev: http://cr.openjdk.java.net/~pliden/8198267/webrev.1/
> Looks good.
>> On 02/16/2018 02:30 PM, Per Liden wrote:
>>> Patch to allow GCId::current_raw() calls from non-NamedThreads. This
>>> is needed when, for example, an Access barrier executed by a Java
>>> thread calls into GC-code.
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8198267
>>> Webrev: http://cr.openjdk.java.net/~pliden/8198267/webrev.0/
More information about the hotspot-gc-dev