On Mon, 27 Nov 2023 09:44:23 GMT, Stefan Karlsson <stef...@openjdk.org> wrote:
>> test/hotspot/jtreg/serviceability/jvmti/GetOwnedMonitorInfo/GetOwnedMonitorInfoTest.java
>> line 53:
>>
>>> 51: private static native boolean hasEventPosted();
>>> 52:
>>> 53: private static void jniMonitorEnterAndLetObjectDie() {
>>
>> I can see it is convenient to just inject this test case in an existing
>> test, but I'm not sure it is necessarily the right thing to do.
>> Serviceability folk may have a stronger opinion.
>
> Yeah, I was thinking the same. Maybe @sspitsyn or @plummercj could give
> guidance here?
This looks okay. I see no problem with it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16783#discussion_r1406615663
