RFR: 8373941: Epsilon: Robust counter updates in early VM phases [v2]

Aleksey Shipilev shade at openjdk.org
Fri Jan 9 09:08:08 UTC 2026


On Fri, 9 Jan 2026 08:46:05 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

> Kim would probably prefer if you changed the `volatile bool _ready` to use the new `Atomic<bool>` class.

Right. There are other cases in Epsilon that would enjoy this conversion. Tell you what, let's do this version (so that is also easily backportable), and then I'll do a cleanup that converts all Epsilon code to `Atomic<T>`: https://bugs.openjdk.org/browse/JDK-8374876

> src/hotspot/share/gc/epsilon/epsilonHeap.cpp line 158:
> 
>> 156:   // At this point, some diagnostic subsystems might not yet be initialized.
>> 157:   // We pretend the printout happened either way. This keeps allocation path
>> 158:   // from obscessively checking the subsystems status on every allocation.
> 
> Suggestion:
> 
>   // from obsessively checking the subsystem's status on every allocation.

Done, should really be `subsystems`` :)

-------------

PR Comment: https://git.openjdk.org/jdk/pull/29061#issuecomment-3727918649
PR Review Comment: https://git.openjdk.org/jdk/pull/29061#discussion_r2675379600


More information about the hotspot-gc-dev mailing list