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

[David Holmes]

On Mon, 27 Nov 2023 22:50:46 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fix indentation
>
> test/hotspot/jtreg/runtime/Monitor/libMonitorWithDeadObjectTest.c line 110:
> 
>> 108: 
>> 109:   // Let the GC clear the weak reference to the object.
>> 110:   system_gc(env);
> 
> A single GC may not be enough...

@dcubed-ojdk there's some earlier discussion on this. Apparently a single GC is sufficient to clear an oopStorage WeakHandle, even though it may not be enough for Java level reference processing actions to be observed.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/16783#discussion_r1407199130