RFR: 8312116: GenShen: make instantaneous allocation rate triggers more timely [v4]
Kelvin Nilsen
kdnilsen at openjdk.org
Wed Jan 14 22:48:09 UTC 2026
On Thu, 8 Jan 2026 22:06:01 GMT, William Kemper <wkemper at openjdk.org> wrote:
>> Kelvin Nilsen has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 65 commits:
>>
>> - Fix comment
>> - Use PROPERFMT macros
>> - Simplify code flow: reviewer suggestion
>> - Merge remote-tracking branch 'jdk/master' into accelerated-triggers
>> - Remove develop/debug instrumentation
>> - add another override
>> - Change type of command-line args
>> - fix white space
>> - Add override to virtual methods
>> - Fix race between allocation reporting and querying
>> - ... and 55 more: https://git.openjdk.org/jdk/compare/dd20e915...7f3a6d1e
>
> src/hotspot/share/gc/shenandoah/heuristics/shenandoahAdaptiveHeuristics.cpp line 192:
>
>> 190:
>> 191: void ShenandoahAdaptiveHeuristics::resume_idle_span() {
>> 192: size_t mutator_available = _free_set->capacity() - _free_set->used();
>
> This is a little confusing to me. Isn't `available` defined as `capacity - used`? Why do we not use `available` here?
Agree. Making this change.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29039#discussion_r2692329526
More information about the shenandoah-dev
mailing list