RFR: 8240873: Shenandoah: Short-cut arraycopy barriers

Aleksey Shipilev shade at redhat.com
Thu Mar 12 11:04:42 UTC 2020


On 3/12/20 12:04 PM, Roman Kennke wrote:
>> *) Wouldn't it make more sense to do acquire/release in (get|set)_update_watermark?
> 
> Hmm ok. This requires making the field. I am not sure if the cast in
> get_update_watermark() is ok?

I don't quite understand why cast is needed. There are already _critical_pins and _live_data fields
that are atomic, why can't we do the same?

> http://cr.openjdk.java.net/~rkennke/JDK-8240873/webrev.02/

Looks fine, modulo nit above.

-- 
Thanks,
-Aleksey




More information about the hotspot-gc-dev mailing list