RFR: 8295839: G1: Single threaded phases (within parallel phases) report as using multiple threads in logs

[Albert Mingkun Yang]

On Mon, 24 Oct 2022 12:19:12 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> Hi all,
> 
>   can I have reviews for this change that changes log output to properly report single-threaded (sub-)phases  number of threads?
> 
> E.g.
> 
>  [87.104s][debug][gc,phases] GC(1412) Ext Root Scanning (ms): Min: 0.0, Avg: 0.0, Max: 0.0, Diff: 0.0, Sum: 0.1, Workers: 6
> [...]
> [87.104s][trace][gc,phases] GC(1412) CLDG Roots (ms): Min: 0.0, Avg: 0.0, Max: 0.0, Diff: 0.0, Sum: 0.0, Workers: 1
> [87.104s][trace][gc,phases] GC(1412) CM RefProcessor Roots (ms): Min: 0.0, Avg: 0.0, Max: 0.0, Diff: 0.0, Sum: 0.0, Workers: 1
> [...] 
> 
> instead of
> 
>  [87.104s][debug][gc,phases] GC(1412) Ext Root Scanning (ms): Min: 0.0, Avg: 0.0, Max: 0.0, Diff: 0.0, Sum: 0.1, Workers: 6
> [...]
> [87.104s][trace][gc,phases] GC(1412) CLDG Roots (ms): Min: 0.0, Avg: 0.0, Max: 0.0, Diff: 0.0, Sum: 0.0, Workers: 6
> [87.104s][trace][gc,phases] GC(1412) CM RefProcessor Roots (ms): Min: 0.0, Avg: 0.0, Max: 0.0, Diff: 0.0, Sum: 0.0, Workers: 6
> [...] 
> 
> 
> Testing: local testing of some log files, gha
> 
> Thanks,
>   Thomas

Marked as reviewed by ayang (Reviewer).

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

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