RFR: 8348594: Shenandoah: Do not penalize for degeneration when not the fault of triggering heuristic [v2]
Kelvin Nilsen
kdnilsen at openjdk.org
Mon Feb 10 20:28:54 UTC 2025
On Tue, 4 Feb 2025 16:14:49 GMT, Paul Hohensee <phh at openjdk.org> wrote:
>> Kelvin Nilsen has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Respond to reviewer feedback
>
> src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp line 318:
>
>> 316:
>> 317: if (ShenandoahHeuristics::should_start_gc()) {
>> 318: _start_gc_is_pending = true;
>
> I assume there's no race here, i.e., only one thread reads/writes _start_gc_is_pending. If there's a race, make sure it's benign. In either case, _start_gc_is_pending is made "sticky" by this code.
There is no race. A single control thread queries should_start-gc() and that is the same thread that initiates the GC.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23305#discussion_r1949828557
More information about the hotspot-gc-dev
mailing list