RFR: 8320515: assert(monitor->object_peek() != nullptr) failed: Owned monitors should not have a dead object [v5]

[Serguei Spitsyn]

On Mon, 27 Nov 2023 09:44:23 GMT, Stefan Karlsson <stefank at 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