RFR: 8315869: UseHeavyMonitors not used

Wed Sep 20 17:52:39 UTC 2023

On Wed, 20 Sep 2023 17:36:06 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

> Please review this trivial change to remove the UseHeavyMonitors develop option, in favor of the now non-experimental LockingMode=LM_MONITOR (0) option.  Tested with tier1 locally.

Changes requested by dcubed (Reviewer).

src/hotspot/share/runtime/globals.hpp line 1064:

> 1062:   develop(bool, VerifyHeavyMonitors, false,                                 \
> 1063:           "Checks that no stack locking happens when using "                \
> 1064:           "-XX:LockingMode=LM_MONITOR")                                     \

s/LM_MONITOR/0/

I don't think you can specify the `LM_MONITOR` value on the actual cmd line.

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

PR Review: https://git.openjdk.org/jdk/pull/15845#pullrequestreview-1636204262
PR Review Comment: https://git.openjdk.org/jdk/pull/15845#discussion_r1331988698