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

Thomas Schatzl tschatzl at openjdk.org
Mon Oct 24 14:38:49 UTC 2022


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

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

Commit messages:
 - properly report single threaded phases as such

Changes: https://git.openjdk.org/jdk/pull/10833/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=10833&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8295839
  Stats: 13 lines in 1 file changed: 0 ins; 4 del; 9 mod
  Patch: https://git.openjdk.org/jdk/pull/10833.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10833/head:pull/10833

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


More information about the hotspot-gc-dev mailing list