RFR: 8244551: Shenandoah: Fix racy update of update_watermark

Aleksey Shipilev shade at redhat.com
Thu May 7 11:31:27 UTC 2020


On 5/7/20 1:30 PM, Roman Kennke wrote:
>> *) What's the point of this cast?
>>
>>  133   *const_cast<HeapWord**>(&_update_watermark) =  w;
> 
> Make access non-volatile. You think it's too much?

I don't think that is relevant. As long as it is not the atomic store (with fences), we are fine.

-- 
Thanks,
-Aleksey



More information about the shenandoah-dev mailing list