RFR: 8272797: Mutex with rank safepoint_check_never imply allow_vm_block [v5]

Coleen Phillimore coleenp at openjdk.java.net
Thu Aug 26 15:42:50 UTC 2021


> See CR for details.  This essentially makes Mutex that safepoint_check_never imply a NoSafepointVerifier, which is a stronger and simpler invariant.
> Tested with tier1-6.

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

  Reverse the order of safepoint_check_required and allow_vm_block arguments so that the first can imply the value of the second.  This is good because if we decide to rename the second to something like _allow_safepoint_use instead there's less to change. Also remove default arguments as they are evil.

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/5218/files
  - new: https://git.openjdk.java.net/jdk/pull/5218/files/27179b77..a3b7d89e

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=5218&range=04
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=5218&range=03-04

  Stats: 56 lines in 15 files changed: 2 ins; 3 del; 51 mod
  Patch: https://git.openjdk.java.net/jdk/pull/5218.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/5218/head:pull/5218

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


More information about the hotspot-dev mailing list