RFR (L, but tedious) 8188220: Remove Atomic::*_ptr() uses and overloads from hotspot

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Thu Oct 12 11:54:09 UTC 2017

Kim, I have this change as an mq patchset.  If you teach me more mq 
commands, I'll post webrevs for each. :)


On 10/12/17 3:29 AM, Kim Barrett wrote:
>> On Oct 11, 2017, at 7:07 AM, coleen.phillimore at oracle.com wrote:
>> On 10/11/17 4:12 AM, Robbin Ehn wrote:
>>> On 10/11/2017 10:09 AM, David Holmes wrote:
>>>> On 11/10/2017 5:45 PM, Erik Österlund wrote:
>>>> Removing the operation is a different argument to renaming it. Most of the above argues for removing it. :)
>>> +1 on removing
>> Thank you for all your feedback.  Erik best described what I was thinking.  I will remove it then.  There were not that many instances and one instance that people thought would be useful, needed the old return value.
> I’ve already registered my objection to removal.  I disagree with several of Erik’s points, which don’t
> address or miss the issues brought up in the original discussion that led to its introduction, as quoted
> by David.
> I’m still slogging my way through the review, maybe about 3/4 of the way through.
> I’ve found a number of real problems, some pre-existing and discovered by looking at the code
> around your changes; I think there are a couple of ABA bugs, for example.  I’m worried that I’m
> missing some too, because I’m getting burned out from reading reams of lock-free code.  This
> is *really* hard, and I very much wish it had been broken up into more easily digestible chunks.

More information about the hotspot-dev mailing list