RFR: 8287771: Remove useless G1 After GC summary refinement and sampling thread times [v2]

Kim Barrett kbarrett at openjdk.java.net
Wed Jun 15 02:56:42 UTC 2022


On Thu, 9 Jun 2022 14:43:54 GMT, tqxia <duke at openjdk.java.net> wrote:

>> In the "After GC summary" printed by gc+remset=trace and G1RemsetStatsSummaryPeriod > 1 we print refinement and sampling thread times. However during GC neither thread types are active, so it always prints zeros.
>> 
>> Remove the printing of these unnecessary values in the after gc summary.
>
> tqxia has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
> 
>   8287771: Remove useless G1 After GC summary refinement and sampling thread times

Looks good, apart from the parameter name, where I agree with @tschatzl .

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

Marked as reviewed by kbarrett (Reviewer).

PR: https://git.openjdk.org/jdk/pull/9106



More information about the hotspot-gc-dev mailing list