Integrated: 8260584: Shenandoah: simplify "Concurrent Thread Roots" logging

[Aleksey Shipilev]

On Thu, 28 Jan 2021 12:15:32 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> There are two separate counters now:
> 
>  f(conc_thread_roots, "Concurrent Stack Processing") \
>  f(conc_thread_roots_work, " Threads") \
>   SHENANDOAH_PAR_PHASE_DO(conc_thread_roots_work_, " CT: ", f) \
> ...and `_work` counter is unused, and `conc_thread_roots` is used to report worker stats. So the log says `<total>`, where `Thread Roots` should have been mentioned:
> 
> [34.169s][info][gc,stats] Concurrent Stack Processing       11341 us, parallelism: 7.93x
> [34.169s][info][gc,stats]   Threads                         89908 us
> [34.169s][info][gc,stats]     CT: <total>                   89908 us, workers (us): 11231, 11270, 11251, 11252, 11237, 11230, 11214, 11223, 
> 
> Fixed log says:
> 
> [99.797s][info][gc,stats] Concurrent Thread Roots            3929 us, parallelism: 7.45x
> [99.797s][info][gc,stats]   CTR: <total>                    29273 us
> [99.797s][info][gc,stats]   CTR: Thread Roots               29273 us, workers (us): 3652, 3643, 3622, 3623, 3623, 3676, 3606, 3829, 
> 
> Also, I believe it should be called "Concurrent Thread Roots", in symmetry with "Concurrent Update Thread Roots" later.

This pull request has now been integrated.

Changeset: 1de3c554
Author:    Aleksey Shipilev <shade at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/1de3c554
Stats:     3 lines in 1 file changed: 0 ins; 1 del; 2 mod

8260584: Shenandoah: simplify "Concurrent Thread Roots" logging

Reviewed-by: rkennke, zgu

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

PR: https://git.openjdk.java.net/jdk/pull/2287