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