RFR: 8346920: Serial: Support allocation in old generation before GC [v2]
Albert Mingkun Yang
ayang at openjdk.org
Mon Jan 6 14:40:36 UTC 2025
On Mon, 6 Jan 2025 14:05:04 GMT, Guoxiong Li <gli at openjdk.org> wrote:
>> Albert Mingkun Yang has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
>>
>> - review
>> - Merge branch 'master' into s1-gc-time-ratio
>> - s1-gc-time-ratio
>
> src/hotspot/share/gc/serial/serialHeap.cpp line 334:
>
>> 332:
>> 333: bool first_only = !should_try_older_generation_allocation(size)
>> 334: && is_long_enough_from_prev_gc_pause_end();
>
> Is the `first_only` actually `young_only`? It means that the allocation is only attempted in young generation?
That's true. I believe Serial used to support >2 generations, so some generic names are left as is. Fixing it requires updating in multiple places, such as the method name `older_generation`, arg in `attempt_allocation`, etc. How about we do that invasive but pure refactoring in its own PR?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22899#discussion_r1904230427
More information about the hotspot-gc-dev
mailing list