RFR: 8375436: G1: Convert G1CardSet classes to use Atomic<T> [v2]
Thomas Schatzl
tschatzl at openjdk.org
Tue Jan 20 08:06:04 UTC 2026
> 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
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/29274/files
- new: https://git.openjdk.org/jdk/pull/29274/files/35b924d7..a697555b
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=29274&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=29274&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.org/jdk/pull/29274.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/29274/head:pull/29274
PR: https://git.openjdk.org/jdk/pull/29274
More information about the hotspot-gc-dev
mailing list