Integrated: 8322853: Should use ConditionalMutexLocker in NativeHeapTrimmerThread::print_state

Denghui Dong ddong at openjdk.org
Wed Jan 3 13:12:46 UTC 2024


On Tue, 2 Jan 2024 16:04:30 GMT, Denghui Dong <ddong at openjdk.org> wrote:

> 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.~~

This pull request has now been integrated.

Changeset: 7eb25ec7
Author:    Denghui Dong <ddong at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7eb25ec7b321abcd685bc761bd4c7cb110d52497
Stats:     9 lines in 1 file changed: 1 ins; 7 del; 1 mod

8322853: Should use ConditionalMutexLocker in NativeHeapTrimmerThread::print_state

Reviewed-by: dholmes, shade

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

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


More information about the hotspot-runtime-dev mailing list