RFR: 8320318: ObjectMonitor Responsible thread [v4]
Axel Boldt-Christmas
aboldtch at openjdk.org
Fri Sep 27 06:32:40 UTC 2024
On Fri, 27 Sep 2024 06:20:29 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> src/hotspot/share/runtime/objectMonitor.cpp line 312:
>>
>>> 310: // The monitor is private to or already owned by locking_thread which must be suspended.
>>> 311: // So this code may only contend with deflation.
>>> 312: assert(locking_thread == Thread::current() || locking_thread->is_obj_deopt_suspend(), "must be");
>>
>> These comments and asserts seem to belong to `enter_for_with_contention_mark`. ??
>
> And for `enter_for`
I had the same comment earlier https://github.com/openjdk/jdk/pull/19454#discussion_r1753348561 guess it got lost in the renaming of the function.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/19454#discussion_r1778075638
More information about the hotspot-dev
mailing list