RFR: 8375436: G1: Convert G1CardSet classes to use Atomic<T> [v2]

Kim Barrett kbarrett at openjdk.org
Tue Jan 20 18:04:22 UTC 2026


On Tue, 20 Jan 2026 08:06:04 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

>> Hi all,
>> 
>>   please review this removal of use of the AtomicAccess class in G1 card set related classes, replacing with use of Atomic<T>.
>> 
>> Although a bit on the large side, it would be hard to do the card set changes separately per class. Other than that I think it is fairly straightforward, except for the addition of a copy constructor for `G1CardSetHashTableValue` because the `ConcurrentHashTable` requires it.
>> 
>> Testing: gha
>> 
>> Thanks,
>>   Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
> 
>   * kbarrett review

Looks good.

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

Marked as reviewed by kbarrett (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/29274#pullrequestreview-3683567966


More information about the hotspot-gc-dev mailing list