RFR: 8260586: Shenandoah: simplify "Concurrent Weak References" logging
Roman Kennke
rkennke at openjdk.java.net
Thu Jan 28 12:51:41 UTC 2021
On Thu, 28 Jan 2021 12:36:58 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> Concurrent Weak References always does parallel worker operation. Therefore "Process" counter is redundant, and we might as well make the root counter the per-worker one. This simplifies GC logging.
>
> Old log:
>
> [95.220s][info][gc,stats] Concurrent Weak References 1709 us
> [95.220s][info][gc,stats] Process 1588 us, parallelism: 1.30x
> [95.220s][info][gc,stats] CWRF: <total> 2056 us
> [95.220s][info][gc,stats] CWRF: Weak References 2056 us, workers (us): 454, 1450, 2, 145, 4, 1, 0, 0,
>
> New log:
>
> [39.583s][info][gc,stats] Concurrent Weak References 651 us, parallelism: 1.52x
> [39.583s][info][gc,stats] CWRF: <total> 986 us
> [39.583s][info][gc,stats] CWRF: Weak References 986 us, workers (us): 183, 29, 145, 627, 1, 0, 0, 0,
Looks good! Thanks!
-------------
Marked as reviewed by rkennke (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/2288
More information about the shenandoah-dev
mailing list