RFR: 8333750: GenShen: Only instantiate young/old generations in generational mode [v3]

Kelvin Nilsen kdnilsen at openjdk.org
Mon Jun 10 17:12:44 UTC 2024


On Fri, 7 Jun 2024 23:39:57 GMT, William Kemper <wkemper at openjdk.org> wrote:

>> Other modes should not instantiate or touch young/old generations.
>
> William Kemper has updated the pull request incrementally with three additional commits since the last revision:
> 
>  - Make age and youth const
>  - Do not age regions after completing old mark
>  - Detect cancellation of old GC after final mark.

Thanks for this refactoring.  All much deserved and important.  I wonder if the title of this PR should be a bit more general, since the refactoring appears to be more far reaching than just not instantiating young/old generations in non-generational mode.

src/hotspot/share/gc/shenandoah/shenandoahOldGC.cpp line 119:

> 117:     return false;
> 118:   }
> 119: 

This looks like it might be a bug fix, vs the rest of this PR, which is mostly isolation refactoring.  Should the bug fix be separated out into a different PR?

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

Marked as reviewed by kdnilsen (Committer).

PR Review: https://git.openjdk.org/shenandoah/pull/444#pullrequestreview-2108335069
PR Review Comment: https://git.openjdk.org/shenandoah/pull/444#discussion_r1633573810


More information about the shenandoah-dev mailing list