RFR: 8376126: G1: Convert remaining volatiles in G1ConcurrentMark to Atomic<T> [v4]
Ivan Walulya
iwalulya at openjdk.org
Mon Feb 2 16:51:39 UTC 2026
On Wed, 28 Jan 2026 12:54:52 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Hi all,
>>
>> please convert remaining volatile declarations in `G1ConcurrentMark` to use `Atomic<T>`. These volatiles are used to indicate concurrent phased access (like changing the variable in thread A, then reading the variable in thread B while A is dormant, or only ever updating it in one direction) where concurrency safety is provided by barriers between these phases. But there are also other cases)
>>
>> The exception is `G1ConcurrentMark::_finger` that has apparently been overlooked in an earlier changes.
>>
>> Testing: gha
>>
>> Thanks,
>> Thomas
>
> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>
> * try to fix non-gcc build failures
Marked as reviewed by iwalulya (Reviewer).
-------------
PR Review: https://git.openjdk.org/jdk/pull/29368#pullrequestreview-3740705028
More information about the hotspot-gc-dev
mailing list