On Fri, 13 Dec 2024 16:26:00 GMT, Patricio Chilano Mateo
<pchilanom...@openjdk.org> wrote:
>> Alan Bateman has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> testObjectWait2 test already owns lock
>
> test/jdk/java/lang/Thread/virtual/MonitorPinnedEvents.java line 345:
>
>> 343: vthread.interrupt();
>> 344: } else {
>> 345: synchronized (lock) {
>
> Not sure if this was intentional but we already own the monitor.
Well spotted, it was copied down from the other test other test that can also
interrupt or notify, this main thread in this case already owns the monitor so
no need to enter again. Thanks!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22718#discussion_r1884271603
