RFR: Make Shenandoah SATB mutex ranks consistent with G1

Christine Flood cflood at redhat.com
Wed Sep 27 14:41:31 UTC 2017


To the best of my knowledge, this looks fine.


Christine


On Wed, Sep 27, 2017 at 10:21 AM, Roman Kennke <rkennke at redhat.com> wrote:

> Apparently, upstream changed the ranks of the SATB mutexes from special to
> access. We missed to do the same. Fix:
>
> http://cr.openjdk.java.net/~rkennke/satbmutex/webrev.00/ <
> http://cr.openjdk.java.net/%7Erkennke/satbmutex/webrev.00/>
>
> To be honest, the real fix would be to not declare those mutexes globally,
> but put it under satbQueue.cpp where they are actually used. But this would
> have to be done upstream.
>
> Ok to push?
>
> Roman
>
>


More information about the shenandoah-dev mailing list