RFR: 8274004: Change 'nonleaf' rank name [v3]

Coleen Phillimore coleenp at openjdk.java.net
Fri Oct 8 12:26:42 UTC 2021


> Also fixes: 8273956: Add checking for rank values
> 
> This change does 3 things.  I could separate them but this has all been tested together and most of the change is mechanical.  The first is a simple rename of nonleaf => safepoint.  The second change is to add the enum class Rank which only allows subtraction from a base rank, and some additional type checking so that rank arithmetic doesn't overlap with a different rank.  Ie if you say nosafepoint-n, that doesn't overlap with oopstorage.   The third change is to remove the _safepoint_check_always/never flag.  That is now intrinsic in the ranking, as all nosafepoint ranks are lower than all safepoint ranks.
> 
> Future changes could add rank names in the middle of nosafepoint and safepoint, like handshake.  As of now, the rank subtractions are not unmanageable.  There are actually not many nested Mutex.
> 
> This is the last of the planned subtasks for Mutex ranking cleanup.  If you have other ideas or suggestions, please let me know.
> 
> Tested with tier1-8 at one point in time and just retested with tier1-6.

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

  Remove dead code in test.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5845/files
  - new: https://git.openjdk.java.net/jdk/pull/5845/files/59bfd945..e8df15de

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

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

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


More information about the shenandoah-dev mailing list