RFR: 8263191: Consolidate ThreadInVMfromJavaNoAsyncException and ThreadBlockInVMWithDeadlockCheck with existing wrappers

Patricio Chilano Mateo pchilanomate at openjdk.java.net
Mon Mar 8 20:56:14 UTC 2021


Hi,

Please review the following small patch. ThreadInVMfromJavaNoAsyncException is exactly the same as ThreadInVMfromJava except that in the destructor we avoid checking for asynchronous exceptions. Similarly ThreadBlockInVMWithDeadlockCheck is equal to ThreadBlockInVM except that in the destructor we might need to release a Mutex in case we need to stop for a safepoint or handshake.
We can consolidate these two wrappers with the existing ones to avoid code duplication and minimize the number of transition wrappers.

Tested in mach5 tiers 1-3.

Thanks,
Patricio

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

Commit messages:
 - v1

Changes: https://git.openjdk.java.net/jdk/pull/2880/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2880&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8263191
  Stats: 90 lines in 3 files changed: 10 ins; 58 del; 22 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2880.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2880/head:pull/2880

PR: https://git.openjdk.java.net/jdk/pull/2880


More information about the hotspot-runtime-dev mailing list