RFR: JDK-8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0

Jon Masamitsu jon.masamitsu at oracle.com
Wed May 13 16:42:59 UTC 2015


Bengt,

Change looks good.

Thomas,

Is the comment added as part of this change understandable to
you?  I don't know this code so definitely lack the greater
understanding of what's going on.   But if you think the
comment is clear (and since you know the context), it's
fine.

Jon

On 5/13/2015 2:15 AM, Bengt Rutisson wrote:
>
> Hi everyone,
>
> Could I have a couple of reviews for this change to make G1 work 
> correctly on non-TSO platforms?
>
> https://bugs.openjdk.java.net/browse/JDK-8035496
> http://cr.openjdk.java.net/~brutisso/8035496/webrev.00/
>
> The change was contributed by Bertrand Delsart. I'm shepherding it 
> into the JDK 9 repo.
>
> Thanks,
> Bengt




More information about the hotspot-gc-dev mailing list