RFR: 8337415: Remove inappropriate Atomic access in FreeListAllocator
Aleksey Shipilev
shade at openjdk.org
Tue Jul 30 09:54:31 UTC 2024
On Tue, 30 Jul 2024 03:23:24 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:
> Please review this change to ~FreeListAllocator to replace an inappropriate
> use of Atomic::load with a plain access.
>
> Testing: mach5 tier1
Oh, okay. So `NodeList pending_list` is local, allocated in `FreeListAllocator::PendingList::take_all` and then copied. So the atomic is not really needed, cool.
-------------
Marked as reviewed by shade (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/20382#pullrequestreview-2207223015
More information about the hotspot-gc-dev
mailing list