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