RFR(s): 8198267: Allow GCId::current_raw() calls from non-NamedThreads

Stefan Karlsson stefan.karlsson at oracle.com
Mon Feb 19 13:37:40 UTC 2018

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.


> /Per
> 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/
>> /Per

More information about the hotspot-gc-dev mailing list