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

[Coleen Phillimore]

On Fri, 24 Sep 2021 01:38:21 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> 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.

@linzang  has a different fix for this mutex ranking problem (make them check for safepoints, which is preferred anyway) and I'll file a different PR for the wait < nosafepoint rank problem.

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

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