RFR: 8205559: Remove IN_CONCURRENT_ROOT Access decorator

Kim Barrett kim.barrett at oracle.com
Tue Jun 26 18:26:30 UTC 2018


> On Jun 26, 2018, at 7:50 AM, Stefan Karlsson <stefan.karlsson at oracle.com> wrote:
> 
> Looks good.
> 
> StefanK

Thanks.

> 
> On 2018-06-24 23:51, Kim Barrett wrote:
>> Please review the removal of the IN_CONCURRENT_ROOT Access decorator.
>> All non-AS_RAW IN_NATIVE accesses will be treated as potentially
>> concurrent, and so will require barriers appropriate to the selected
>> collector.
>> Renumbered the Access decorators, eliminating gaps and mis-orderings
>> that had accrued from previous removals and renamings.
>> Fixed StringDedupTable::lookup usage of the Access API.  It was using
>> IN_CONCURRENT_ROOT incorrectly, and was not using AS_NO_KEEPALIVE
>> where it should have been.  There are other places where this class is
>> not using the Access API but should be, involving the _obj field of
>> StringDedupTableEntry; addressing those will be part of a future CR,
>> since currently only ZGC would fail because of these and ZGC does not
>> yet support string deduplication.
>> CR:
>> https://bugs.openjdk.java.net/browse/JDK-8205559
>> Webrev:
>> http://cr.openjdk.java.net/~kbarrett/8205559/open.00/
>> Testing:
>> Mach5 tier1,2,3, hs-tier4,5
>> Aurora perf testing for G1 and Parallel detected no performance regressions.




More information about the hotspot-dev mailing list