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