> As discussed in JBS the deadlock occurs when the call to 
> `ReleasePrimitiveArrayCritical` performs the transition from native to VM, 
> and in the process checks for special runtime exit conditions - which 
> includes the `obj_deopt_suspend` request. The simple solution is to define a 
> custom JNI ENTRY with custom `ThreadInVMfromNative` that elides the exit 
> check.
> 
> The change is limited to `ReleasePrimitiveArrayCritical` and 
> `ReleaseStringCritical`.
> 
> UPDATE: we are now employing a much simpler solution.
> 
> There is no regression test as this has only been seen in long running stress 
> tests.
> 
> Testing:
>    -tiers 1-6

David Holmes has updated the pull request with a new target base due to a merge 
or a rebase. The incremental webrev excludes the unrelated changes brought in 
by the merge/rebase. The pull request contains four additional commits since 
the last revision:

 - Greatly simplifed fix to just defer object_deopt whilst in JNI critical 
region
 - Merge branch 'master' into 8369515-jni-critical
 - Revert "8369515"
   
   This reverts commit 3beb23ccbf5adb98d8c6ad404d40c603bbf499dc.
 - 8369515

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/28779/files
  - new: https://git.openjdk.org/jdk/pull/28779/files/3beb23cc..b96a9bc2

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

  Stats: 12369 lines in 431 files changed: 7742 ins; 1537 del; 3090 mod
  Patch: https://git.openjdk.org/jdk/pull/28779.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/28779/head:pull/28779

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

Reply via email to