RFR: 8366659: ObjectMonitor::wait() can deadlock with a suspension request [v4]
Anton Artemov
duke at openjdk.org
Tue Nov 11 13:53:36 UTC 2025
On Tue, 11 Nov 2025 02:31:57 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:
>> Anton Artemov has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 18 commits:
>>
>> - Merge remote-tracking branch 'origin/master' into JDK-8366659-OM-wait-suspend-deadlock
>> - 8366659: Added a comment to a boolean arg for enter()
>> - Merge remote-tracking branch 'origin/master' into JDK-8366659-OM-wait-suspend-deadlock
>> - Merge remote-tracking branch 'origin/master' into JDK-8366659-OM-wait-suspend-deadlock
>> - 8366659: Fixed new lines.
>> - Merge remote-tracking branch 'origin/master' into JDK-8366659-OM-wait-suspend-deadlock
>> - 8366659: Removed incorrect assert,
>> - 8366659: Fixed merge conflict
>> - 8366659: Fixed whitespace.
>> - 8366659: Disabled posting JVMTI events in reenter-etner path of wait. Postponed waited event.
>> - ... and 8 more: https://git.openjdk.org/jdk/compare/79fee607...31482ba4
>
> test/hotspot/jtreg/serviceability/jvmti/SuspendWithObjectMonitorWait/SuspendWithObjectMonitorWait.java line 387:
>
>> 385: }
>> 386: try { Thread.sleep(1000);
>> 387: } catch(Exception e) {}
>
> Nit: The `Thread.sleep` call should be on its own line after L386.
Addressed in the latest commit.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27040#discussion_r2514310008
More information about the serviceability-dev
mailing list