RFR: 8274245: sun/tools/jmap/BasicJMapTest.java Mutex rank failures [v2]

Coleen Phillimore coleenp at openjdk.java.net
Fri Sep 24 01:38:21 UTC 2021


> This test fails due to Mutex rank assertions that I just added.  The new mutexes in the heapDumper needs to be ranked 'nosafepoint' and 'nosafepoint-1' since they don't cause safepoint checks.
> This code also holds two no-safepoint checking locks and waits on the second.  This doesn't break rank ordering but does run afoul of an old assert that we had in the code, that is no longer has meaning.
> 
> The test still fails because of other bugs like JDK-8274196 but it doesn't get the mutex rank assertions anymore.
> 
> Testing with tier 5 and 6, where the failure was observed, in progress.

Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:

  Remove tests for special wait assert.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5669/files
  - new: https://git.openjdk.java.net/jdk/pull/5669/files/5ccfa31d..45b4776c

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5669&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5669&range=00-01

  Stats: 48 lines in 1 file changed: 0 ins; 48 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/5669.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/5669/head:pull/5669

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


More information about the hotspot-runtime-dev mailing list