On Thu, 13 Nov 2025 19:20:40 GMT, Patricio Chilano Mateo <[email protected]> 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 20 commits: >> >> - 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 >> - 8366659: Removed incorrect assert, >> - 8366659: Fixed merge conflict >> - ... and 10 more: https://git.openjdk.org/jdk/compare/400a83da...702880c6 > > test/hotspot/jtreg/serviceability/jvmti/SuspendWithObjectMonitorWait/SuspendWithObjectMonitorWait.java > line 31: > >> 29: * @library /test/lib >> 30: * @compile SuspendWithObjectMonitorWait.java >> 31: * @run main/othervm/native -agentlib:SuspendWithObjectMonitorWait >> SuspendWithObjectMonitorWait 1 > > Note that these passed values are overriding timeMax. The addition of the test number should have been done as a required parameter. That would push the optional timeMax parameter to be the optional second parameter. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/27040#discussion_r2524856106
