RFR: 8322853: Should use ConditionalMutexLocker in NativeHeapTrimmerThread::print_state
Denghui Dong
ddong at openjdk.org
Tue Jan 2 16:09:53 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.
-------------
Commit messages:
- remove unused method is_stopped
- only make _stop volatile
- make some fields volatile
- 8322853: Should use ConditionalMutexLocker in NativeHeapTrimmerThread::print_state
Changes: https://git.openjdk.org/jdk/pull/17226/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=17226&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8322853
Stats: 10 lines in 1 file changed: 1 ins; 7 del; 2 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