RFR: 8241081: Shenandoah: Do not modify update-watermark concurrently

Aleksey Shipilev shade at redhat.com
Tue Mar 17 11:01:15 UTC 2020


On 3/17/20 11:55 AM, Roman Kennke wrote:
> Webrev:
> http://cr.openjdk.java.net/~rkennke/JDK-8241081/webrev.02/

Looks good.

One tiny thing: can you please read _update_watermark into local field, assert that value and return
the same one? Instead of doing effectively three memory accesses, which might disagree.

-- 
Thanks,
-Aleksey



More information about the shenandoah-dev mailing list