RFR(S): 8236035: refactor ObjectMonitor::set_owner() and _owner field setting

Daniel D. Daugherty daniel.daugherty at oracle.com
Tue Jan 28 21:45:22 UTC 2020


On 1/28/20 4:37 PM, Kim Barrett wrote:
>> On Jan 28, 2020, at 12:11 PM, Daniel D. Daugherty <daniel.daugherty at oracle.com> wrote:
>> In summary, I'm planning to leave these two pieces of code as they are
>> in this version of the fix.
>>
>> Kim, please confirm that you are okay with this or not.
> I really dislike mixing acquire/release with {load,store}{load,store}
> for various reason.  I think we should consider eliminating loadload
> and friends in favor of consistent acquire/release usage.  But that's
> a discussion that is clearly out of scope for this change, so I'll
> accept the change as is.
>

Thanks Kim!

Dan


More information about the hotspot-runtime-dev mailing list