RFR: 8328278: Do not print the tenuring threshold in AgeTable::print_on [v4]

Y. Srinivas Ramakrishna ysr at openjdk.org
Wed Apr 3 07:08:12 UTC 2024


On Tue, 2 Apr 2024 10:23:18 GMT, Guoxiong Li <gli at openjdk.org> wrote:

>> Hi all,
>> 
>> This patch places the SerialGC code that computes tenuring threshold before per GC instead of after per GC, so that the method `AgeTable::print_on` prints the right tenuring threshold. Just like the current G1 implementation.
>> 
>> Thanks for taking the time to review.
>> 
>> Best Regards,
>> -- Guoxiong
>
> Guoxiong Li 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 six additional commits since the last revision:
> 
>  - Merge branch 'master' into SERIAL_GC_AGE_TABLE
>  - Fix test TestAgeOutput.
>  - Do not print threshold.
>  - Merge branch 'master' into SERIAL_GC_AGE_TABLE
>  - Revert.
>  - JDK-8328278

Thanks for the before / after logs. This looks good to me. Reviewed.

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

Marked as reviewed by ysr (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/18329#pullrequestreview-1975726700


More information about the hotspot-gc-dev mailing list