RFR: 8363929: G1: Add G1 prefix to various G1 specific global locks
Thomas Schatzl
tschatzl at openjdk.org
Thu Jul 24 10:49:53 UTC 2025
On Thu, 24 Jul 2025 08:32:34 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Hi all,
>>
>> please review this renaming of a few G1 specific locks to have a G1 prefix to make it clear that they should not be used elsewhere.
>>
>> Testing: local compilation, gha
>>
>> Thanks,
>> Thomas
>
> src/hotspot/share/runtime/mutexLocker.hpp line 73:
>
>> 71: extern Mutex* NonJavaThreadsList_lock; // a lock on the NonJavaThreads list
>> 72: extern Mutex* NonJavaThreadsListSync_lock; // a lock for NonJavaThreads list synchronization
>> 73: extern Monitor* G1CGC_lock; // used for coordination between
>
> In all hunks in this file: comment indenting. May even add "G1" to comments, where appropriate.
I remember fixing the indentation, maybe it got lost somewhere. Thanks for pointing this out.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26444#discussion_r2228162737
More information about the hotspot-gc-dev
mailing list