RFR: 8314225: SIGSEGV in JavaThread::is_lock_owned [v2]

[Kevin Walls]

> JavaThread's _monitor_chunks member is temporary storage used by deoptimization.
> When other threads inspect it using JavaThread::monitor_chunks(), if it is non-null that means a deoptimization is in progress, and the value will be removed shortly.
> 
> There are a few places where we attempt to follow the MonitorChunk*, but that would only be valid if deopt is in progress, and only safe if we could know the deopt is not going to complete.  But that the deopt will complete, and will free the MonitorChunks and clear the value.  So this is rare but there is a race and a risk of following a MonitorChunk* as it gets freed, and crashing.

Kevin Walls has updated the pull request incrementally with one additional commit since the last revision:

  ThreadsListHandle required for Handshake

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17566/files
  - new: https://git.openjdk.org/jdk/pull/17566/files/34d815ae..00659a8f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17566&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17566&range=00-01

  Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/17566.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17566/head:pull/17566

PR: https://git.openjdk.org/jdk/pull/17566