Re: RFR: 8315869: UseHeavyMonitors not used [v2]

2023-09-20 Thread Coleen Phillimore
On Wed, 20 Sep 2023 18:00:40 GMT, Coleen Phillimore 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. > > Coleen Phillimore has updated the pull req

Re: RFR: 8315869: UseHeavyMonitors not used [v2]

2023-09-20 Thread Daniel D . Daugherty
On Wed, 20 Sep 2023 18:00:40 GMT, Coleen Phillimore 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. > > Coleen Phillimore has updated the pull req

Re: RFR: 8315869: UseHeavyMonitors not used [v2]

2023-09-20 Thread Alan Bateman
On Wed, 20 Sep 2023 18:00:40 GMT, Coleen Phillimore 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. > > Coleen Phillimore has updated the pull req

Re: RFR: 8315869: UseHeavyMonitors not used [v2]

2023-09-20 Thread Coleen Phillimore
On Wed, 20 Sep 2023 18:00:40 GMT, Coleen Phillimore 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. > > Coleen Phillimore has updated the pull req

Re: RFR: 8315869: UseHeavyMonitors not used [v2]

2023-09-20 Thread Daniel D . Daugherty
On Wed, 20 Sep 2023 18:00:40 GMT, Coleen Phillimore 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. > > Coleen Phillimore has updated the pull req

Re: RFR: 8315869: UseHeavyMonitors not used [v2]

2023-09-20 Thread Coleen Phillimore
> 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. Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revis

Re: RFR: 8315869: UseHeavyMonitors not used

2023-09-20 Thread Daniel D . Daugherty
On Wed, 20 Sep 2023 17:36:06 GMT, Coleen Phillimore 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/hot

RFR: 8315869: UseHeavyMonitors not used

2023-09-20 Thread Coleen Phillimore
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. - Commit messages: - 8315869: UseHeavyMonitors not used Changes: https://git.openjdk.org/jdk/pull/15