RFR: 8318757: VM_ThreadDump asserts in interleaved ObjectMonitor::deflate_monitor calls [v10]

Tue Nov 14 14:48:41 UTC 2023

On Mon, 13 Nov 2023 09:00:31 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> A safepointed monitor deflation pass can run interleaved with a paused async monitor deflation pass. The code is not written to handle that situation and asserts when it finds a DEFLATER_MARKER in the owner field. @pchilano also found other issues with having to monitor deflation passes interleaved. More info below ...
>
> Stefan Karlsson has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Tweak test

Still good.

test/hotspot/jtreg/runtime/Monitor/ConcurrentDeflation.java line 79:

> 77:                 monitorCount++;
> 78:             }
> 79: 

Nit: extra line

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

Marked as reviewed by pchilanomate (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/16519#pullrequestreview-1729910800
PR Review Comment: https://git.openjdk.org/jdk/pull/16519#discussion_r1392704732