RFR: 8375436: G1: Convert G1CardSet classes to use Atomic<T>
Thomas Schatzl
tschatzl at openjdk.org
Fri Jan 16 19:00:19 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
-------------
Commit messages:
- * fix whitespace
- 8375436
Changes: https://git.openjdk.org/jdk/pull/29274/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=29274&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8375436
Stats: 119 lines in 6 files changed: 9 ins; 4 del; 106 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