[jdk16] RFR: 8257999: Parallel GC crash in gc/parallel/TestDynShrinkHeap.java: new region is not in covered_region

Thomas Schatzl tschatzl at openjdk.java.net
Thu Dec 17 12:58:58 UTC 2020


On Thu, 17 Dec 2020 11:17:28 GMT, Stefan Johansson <sjohanss at openjdk.org> wrote:

>> Good idea.  Here's the revised comment:
>> 
>> -  // ALWAYS do this last!!
>> +  // Ensure the space bounds are updated are made visible to other
>> +  // threads after the other data structures have been resized.
>>    OrderAccess::storestore();
>
> The second "are" should be an "and", right? Otherwise looks great!

+1

-------------

PR: https://git.openjdk.java.net/jdk16/pull/35



More information about the hotspot-gc-dev mailing list