RFR: JDK-8035496: G1 ARM: missing remset entry noticed by VerifyAfterGC for vm/gc/concurrent/lp50yp10rp70mr30st0
bengt.rutisson at oracle.com
Wed May 13 18:41:22 UTC 2015
Hi Jon and Thomas,
On 13/05/15 19:48, Thomas Schatzl wrote:
> On Wed, 2015-05-13 at 09:42 -0700, Jon Masamitsu wrote:
>> Change looks good.
Thanks for the review, Jon!
>> 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
> 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?
Sounds good. I'll remove that sentence before I push.
More information about the hotspot-gc-dev