(S) RFR: 8157904: Atomic::cmpxchg for jbyte is missing a fence on initial failure
David Holmes
david.holmes at oracle.com
Wed Aug 24 22:19:08 UTC 2016
On 25/08/2016 8:07 AM, Kim Barrett wrote:
>> On Aug 24, 2016, at 5:51 PM, David Holmes <david.holmes at oracle.com> wrote:
>>
>> Thanks Volker!
>>
>> Just waiting for Kim to give the all clear.
>
> Sorry, I missed that you’d done an in-place update of the webrev.
>
> I would have left align_ptr_down alone and cast away the volatile in cmpxchg,
But that would have restored a cast the Volker and Andrew objected to.
> but have no objection to the approach you’ve taken.
Great! Thanks for the assistance with this.
David
> Looks good.
>
More information about the hotspot-runtime-dev
mailing list