RFR: 8320318: ObjectMonitor Responsible thread

Fredrik Bredberg fbredberg at openjdk.org
Wed Sep 11 13:25:11 UTC 2024


On Wed, 11 Sep 2024 12:08:25 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> src/hotspot/share/runtime/objectMonitor.cpp line 310:
>> 
>>> 308: 
>>> 309: bool ObjectMonitor::enterI_with_contention_mark(JavaThread* locking_thread, ObjectMonitorContentionMark& contention_mark) {
>>> 310:   // Used by ObjectSynchronizer::enter_for() to enter for another thread.
>> 
>> This renaming is confusing for me. The `enter_for` methods were made explicit because normally locking is always done by the current thread for the current thread - but deopt breaks that. And now it seems we have an `EnterI` that is really an `EnterI_for` ??
>
> Me too. So many functions that are sort of the same.

As I wrote above, the confusing renaming has to be fixed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19454#discussion_r1754518155


More information about the hotspot-dev mailing list