RFR(s): 8198369: Clean up GCId and GCIdMark
Per Liden
per.liden at oracle.com
Mon Feb 19 13:46:50 UTC 2018
Thanks Stefan!
/Per
On 02/19/2018 02:42 PM, Stefan Karlsson wrote:
> Looks good.
>
> StefanK
>
> On 2018-02-19 14:38, Per Liden wrote:
>> After discussing "8198267: Allow GCId::current_raw() calls from
>> non-NamedThreads", we noticed the need to do some additional cleanups
>> of GCId and GCIdMark. So here's a patch for that.
>>
>> In short, GCIdMarkAndRestore is not really needed, we can let the
>> normal GCIdMark have the restore semantics and get rid of
>> GCIdMarkAndRestore. GCId and GCIdMark can also be cleaned up a bit,
>> for example. the GCIdMark::_gc_id member is unnecessary, some
>> functions return "const uint" instead of "uint", etc.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8198369
>> Webrev: http://cr.openjdk.java.net/~pliden/8198369/webrev.0/
>>
>> /Per
>>
More information about the hotspot-gc-dev
mailing list