RFR: 8244551: Shenandoah: Fix racy update of update_watermark

Roman Kennke rkennke at redhat.com
Thu May 7 11:30:26 UTC 2020


>> http://cr.openjdk.java.net/~rkennke/JDK-8244551/webrev.01/
> 
> Nice.
> 
> *) What's the point of this cast?
> 
>  133   *const_cast<HeapWord**>(&_update_watermark) =  w;

Make access non-volatile. You think it's too much?

> *) Nit: message can be shorter "Should be at Shenandoah safepoint"
> 
>  131   assert(SafepointSynchronize::is_at_safepoint(),
>  132          "only call set_update_watermark_at_safepoint at safepoint");

Ok, will change before push.

Roman




More information about the shenandoah-dev mailing list