RFR: 8348400: GenShen: assert(ShenandoahHeap::heap()->is_full_gc_in_progress() || (used_regions_size() <= _max_capacity)) failed: Cannot use more than capacity # [v2]

[William Kemper]

On Wed, 12 Mar 2025 22:39:45 GMT, William Kemper <wkemper at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/shenandoahGenerationSizer.cpp line 127:
>> 
>>> 125:   }
>>> 126: 
>>> 127:   if (dst->max_capacity() + bytes_to_transfer > max_size_for(dst)) {
>> 
>> Do we need to edit the descriptions of ShenandoahMinYoungPercentage and ShenandoahMaxYoungPercentage?  Do we need to remove these options entirely from shenandoah_globals?
>
> Yes, we should probably remove one of them and name the other `ShenandoahInitYoungPercentage`. I think I will back out this change to `shGenerationSizer` in this PR, and open a different PR for removing this constraint and renaming the options. It's a bit outside the scope of this bug fix.

Filed: https://bugs.openjdk.org/browse/JDK-8351892

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23998#discussion_r1992416245