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