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

[Thomas Schatzl]

Hi,

On Wed, 2015-05-13 at 09:42 -0700, Jon Masamitsu wrote:
> 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.

  it's fine, except maybe for the comment about non-TSO architectures.
It just means that the last write publishes the data structure to the
public, so everything written before that should be guaranteed to be
written first.

As for the comment about non-TSO architectures, TSO architectures also
need that barrier because the compiler might otherwise reorder the
writes which then definitely gets visible in the wrong order.

Maybe Bengt can fix that (just remove the "On non-TSO systems," part)
before pushing?

Thanks,
  Thomas