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