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