RFR: 8376195: Convert ThreadLocalAllocBuffer to use Atomic<T> [v2]
Thomas Schatzl
tschatzl at openjdk.org
Wed Feb 4 11:00:56 UTC 2026
On Mon, 26 Jan 2026 11:00:10 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Hi all,
>>
>> please review the change to use `Atomic<T>` in `ThreadLocalAllocBuffer`.
>>
>> Testing: gha
>>
>> Thanks,
>> Thomas
>
> Thomas Schatzl has updated the pull request incrementally with two additional commits since the last revision:
>
> - * kbarrett review
> - * kbarrett review
Going to revert to the original change (excluding comment changes) since TLABs are thread local (as the name implies) and should not need the atomics - however there are some intentional racy reads for JFR/statistics/MemoryMXBean that somehow need to be incorporated.
However the current path of making everything an atomic is wrong (after discussion with @kimbarrett ).
-------------
PR Comment: https://git.openjdk.org/jdk/pull/29386#issuecomment-3846743720
More information about the hotspot-dev
mailing list