RFR (S): 8165858: heapRegionManager is missing volatile qualifier for _claims
erik.osterlund at oracle.com
Mon Sep 19 12:14:19 UTC 2016
Thank you for the review!
On 2016-09-19 14:12, Thomas Schatzl wrote:
> Hi Erik,
> On Mon, 2016-09-19 at 13:47 +0200, Erik Österlund wrote:
>> This bug is a split out of the larger JDK-8033552 for adding
>> qualifiers to lock-free code.
>> The motivation is to make our implementation more robust since these
>> kind of issues have struck us a few times too many already.
>> CR: https://bugs.openjdk.java.net/browse/JDK-8165858
>> Webrev: http://cr.openjdk.java.net/~eosterlund/8165858/webrev.00/
>> This specific CR handles the lack of volatile qualifiers for the
>> field of the HeapRegionClaimer class in the heapRegionManager files.
>> is part of a lock-free protocol for claiming reagions, and should use
>> volatile qualifier to be more robust.
>> Testing: JPRT
> looks good to me.
More information about the hotspot-gc-dev