RFR: 8366659: ObjectMonitor::wait() can deadlock with a suspension request [v8]

[Daniel D. Daugherty]

On Fri, 14 Nov 2025 17:54:05 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 22 commits:
>> 
>>  - Merge remote-tracking branch 'origin/master' into JDK-8366659-OM-wait-suspend-deadlock
>>  - 8366659: Fixed the test.
>>  - Merge remote-tracking branch 'origin/master' into JDK-8366659-OM-wait-suspend-deadlock
>>  - 8366659: Fixed lines in tests.
>>  - 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
>>  - ... and 12 more: https://git.openjdk.org/jdk/compare/ff851de8...2ac18b94
>
> test/hotspot/jtreg/serviceability/jvmti/SuspendWithObjectMonitorWait/SuspendWithObjectMonitorWait.java line 212:
> 
>> 210:                 argsLeft--;
>> 211:             }
>> 212:             if (argsLeft == 0) {
> 
> Because of the addition of a mandatory argument (test case), argument parsing will
> be more complicated. I need to mull on this a bit and I'll post another suggestion
> after this round.

I emailed you an updated version of the current test (v07) with a better
parser. Feel free to merge it in or not.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27040#discussion_r2528792374