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:02:14 UTC 2025
On Tue, 4 Feb 2025 16:04:34 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/shenandoahHeuristics.cpp line 261:
>
>> 259:
>> 260: void ShenandoahHeuristics::record_success_concurrent() {
>> 261: _start_gc_is_pending = false;
>
> The name _start_gc_is_pending implies that it should be set false as soon as a gc cycle starts, not when it finishes. Maybe _gc_pending? Or maybe setting it false at the end of a gc cycle is a bug? :)
You make a good point. I'll change the control flow to cancel the trigger as soon as we start up the GC.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23305#discussion_r1949798178
More information about the hotspot-gc-dev
mailing list