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