RFR: 8375436: G1: Convert G1CardSet classes to use Atomic<T> [v2]
Ivan Walulya
iwalulya at openjdk.org
Fri Jan 23 11:46:37 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
Marked as reviewed by iwalulya (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/29274#pullrequestreview-3697122646
More information about the hotspot-gc-dev
mailing list