RFR(XXS): 8165018: Missing memory barrier for PPC64 in Unsafe_GetObjectVolatile
kim.barrett at oracle.com
Sat Sep 3 00:04:17 UTC 2016
> On Sep 2, 2016, at 1:03 AM, David Holmes <david.holmes at oracle.com> wrote:
> Hi Martin,
> Adding GC folk
> On 30/08/2016 8:51 PM, Doerr, Martin wrote:
>> we found that a memory barrier for PPC64 is missing in the current Unsafe implementation. get_volatile already contains the memory barrier for "support_IRIW_for_not_multiple_copy_atomic_cpu". The same is needed in Unsafe_GetObjectVolatile.
>> Here's my webrev:
> That looks fine to me.
Looks good to me too.
>> And while looking at it I wonder why Unsafe_GetObjectVolatile does not contain a G1 barrier like Unsafe_GetObject. Is it not possible to use the Volatile version to access the referent field of a Reference?
> That looks like a bug to me. :)
Looks like a bug to me too.
>> Please review. As it is shared code, I will need a sponsor, please.
> I can sponsor.
>> Best regards,
More information about the hotspot-gc-dev