RFR: 8358529: GenShen: Heuristics do not respond to changes in SoftMaxHeapSize [v2]
William Kemper
wkemper at openjdk.org
Mon Jun 30 19:15:41 UTC 2025
On Mon, 23 Jun 2025 23:04:11 GMT, Rui Li <duke at openjdk.org> wrote:
>> src/hotspot/share/gc/shenandoah/shenandoahGeneration.hpp line 129:
>>
>>> 127: virtual ShenandoahHeuristics* initialize_heuristics(ShenandoahMode* gc_mode);
>>> 128:
>>> 129: size_t soft_max_capacity() const override;
>>
>> Can we take out the `_soft_max_capacity` member from `ShenandoahGeneration` now?
>
> Yes. Good catch. Removed
I think we can cut deeper here. If `ShenandoahHeap` is now the sole source of truth for soft max heap size, can we remove the `soft_max_capacity` getter method from `ShenandoahGeneration` (and `ShenandoahSpaceInfo`)? All the callers can just get this value directly from `ShenandoahHeap` now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25943#discussion_r2175728375
More information about the hotspot-gc-dev
mailing list