Serguei Spitsyn
Serguei Spitsyn
> I have to say that I don't understand how the behaviour of `RawMonitorWait` is any different to `ObjectMonitor::wait` when it comes to the use of the is_interrupted(true). ??? Is...
Added `ObjectLocker` of the `interruptLock` for sync to the `JavaThread::is_interrupted()`.
> I think you can drop the catch of InterruptedException in Object.wait now. Easy to check this by running test/jdk/java/lang/Thread/virtual/MonitorWaitNotify.java as it has tests for interrupting Object.wait. Thanks. Fixed now....
> Okay so that is where the carrier and virtual thread states get back in sync, and that is what is missing in the `RawMonitorWait` case. The proposed fix/change to...
> Can you check if vmTestbase/nsk/jvmti/scenarios/bcinstr/BI04/bi04t002/TestDescription.java is passing now? That test is a bit annoying in that it fails every time we touch j.l.Object so you might have to update...
> Can you look at it again? That test has a copy of `Object.java` so needs to be updated every time we touch `Object.java`. It's an annoying tax and I...
I've restored the `InterruptedException` catch in the `Object.wait`. I believe, all the concerns have been resolved now.
David and Alan, thank you for review!
Patricio and Leonid, thank you for prompt review!