Re: 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 <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