RFR: 8322853: Should use ConditionalMutexLocker in NativeHeapTrimmerThread::print_state [v2]

Denghui Dong ddong at openjdk.org
Wed Jan 3 07:06:57 UTC 2024


> Hi all,
> 
> Please help review this small fix.
> 
> MutexLocker requires that mutex != nullptr, so we should use ConditionalMutexLocker instead.
> 
> ~~Also, this patch made '_stop' volatile, since it will be read twice in run() but may be changed.~~

Denghui Dong has updated the pull request incrementally with one additional commit since the last revision:

  update

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/17226/files
  - new: https://git.openjdk.org/jdk/pull/17226/files/552593d7..85c4c46f

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=17226&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=17226&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/17226.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/17226/head:pull/17226

PR: https://git.openjdk.org/jdk/pull/17226


More information about the hotspot-runtime-dev mailing list