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


Change looks good.


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


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