On Wed, 3 Apr 2024 10:52:10 GMT, Alan Bateman <al...@openjdk.org> wrote:
> This is a test-only addition to add a test for virtual threads invoking a > synchronized native method and invoking a native method that enter/exits a > monitor with JNI MonitorEnter/MonitorExit. The test has been in the loom repo > for some time, it can move to the main line in advance of changes to the > object monitor implementation. The java side of this test looks OK to me. I see that for testing concurrent/contended access we aren't doing nested monitor enters. Does it deserve to be tested? ------------- Marked as reviewed by jpai (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/18600#pullrequestreview-2003899439