RFR: 8349906: G1: Improve initial survivor rate for newly used young regions

Ivan Walulya iwalulya at openjdk.org
Thu Feb 13 10:12:14 UTC 2025


On Wed, 12 Feb 2025 10:55:46 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   please review this change that tries to improve the survivor rate initial values for newly expanded regions.
> 
> Currently G1 uses `InitialSurvivorRate` as survivor rate for such regions, but it is typically a pretty bad choice because
> 
> * it's rather conservative, estimating that 40% of region contents will survive
> * such a conservative value is kind of bad particularly in cases for regions that are expanded late in the mutator phase because they are not frequently updated (and with our running weights changes get propagated over a very long time), i.e. this 40% sticks for a long time
> * it is a random value, i.e. not particularly specific to the application.
> 
> The suggestion is to use the survivor rate for the last region we know the survivor rate already.
> 
> Testing: gha, tier1-7 (with other changes)
> 
> Hth,
>   Thomas

Marked as reviewed by iwalulya (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/23584#pullrequestreview-2614502592


More information about the hotspot-gc-dev mailing list