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

[Bengt Rutisson]

Thanks Vitaly and Thomas for discussing this!

It seems like the proposed patch is fine for now. I'll remove the three 
words from the comment that Thomas suggested and push this.

Bengt

On 14/05/15 00:37, Vitaly Davidovich wrote:
>
> Right, makes sense; presumably Alpha would've tickled quite a few more 
> concurrency issues by now had it been supported :).
>
> sent from my phone
>
> On May 13, 2015 6:34 PM, "Thomas Schatzl" <thomas.schatzl at oracle.com 
> <mailto:thomas.schatzl at oracle.com>> wrote:
>
>     Hi,
>
>     On Wed, 2015-05-13 at 18:27 -0400, Vitaly Davidovich wrote:
>     > You're right Thomas, I was mistakenly looking at
>     > release_store_ptr_*fence* - all good then.
>     >
>     > Yes, it sounds like it would break on alpha (based on your
>     > description, I haven't looked at the code), but that's not a
>     supported
>     > platform anyway right? Otherwise, you'd probably have to introduce a
>     > new read barrier type for data dependence that's a nop for all but
>     > alpha so as to not penalize the others.
>
>     No, we do not support Alpha :)
>
>     I do not think there ever has been a Hotspot for Alpha.
>
>     Thanks,
>       Thomas
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20150514/23cef72b/attachment.htm>