RFR(S) 8224932: Shenandoah: Rename ShenandoahHeapLock, make it general purpose lock

Aleksey Shipilev shade at redhat.com
Wed May 29 11:26:01 UTC 2019


On 5/29/19 1:13 PM, Zhengyu Gu wrote:
> Please review this refactor ShenandoahHeapLock to general purpose spin lock (I kept
> ShenandoahSpinLock, cause ShenandoahLock was taken in another changeset. Any other suggestions for
> the name?)

I don't understand. What does it mean "cause ShenandoahLock was taken in another changeset"? There
are no conflicts in current jdk/jdk I can see.

> Bug: https://bugs.openjdk.java.net/browse/JDK-8224932
> Webrev: http://cr.openjdk.java.net/~zgu/JDK-8224932/webrev.00/

Otherwise looks good.

-Aleksey



More information about the shenandoah-dev mailing list