RFR: Print correct message about gross times in stats
Roman Kennke
rkennke at redhat.com
Tue May 16 07:47:55 UTC 2017
Am 15.05.2017 um 20:13 schrieb Aleksey Shipilev:
> Hi,
>
> We should make it clear gross times are not only about TTSP, but about the
> entire safepoint mechanics. Let's make it say this:
>
> $ hg diff
> diff -r 4931b832162d src/share/vm/gc/shenandoah/shenandoahCollectorPolicy.cpp
> --- a/src/share/vm/gc/shenandoah/shenandoahCollectorPolicy.cpp Fri May 12
> 17:06:04 2017 +0200
> +++ b/src/share/vm/gc/shenandoah/shenandoahCollectorPolicy.cpp Mon May 15
> 20:10:38 2017 +0200
> @@ -1018,7 +1018,9 @@
> void ShenandoahCollectorPolicy::print_tracing_info(outputStream* out) {
> out->cr();
> out->print_cr("GC STATISTICS:");
> - out->print_cr(" \"(G)\" (gross) pauses include time to safepoint. \"(N)\"
> (net) pauses are times spent in GC.");
> + out->print_cr(" \"(G)\" (gross) pauses include VM time: time to notify and
> block threads, do the pre-");
> + out->print_cr(" and post-safepoint housekeeping. Use
> -XX:+PrintSafepointStatistics to dissect.");
> + out->print_cr(" \"(N)\" (net) pauses are the times spent in the actual GC
> code.");
> out->print_cr(" \"a\" is average time for each phase, look at levels to see
> if average makes sense.");
> out->print_cr(" \"lvls\" are quantiles: 0%% (minimum), 25%%, 50%% (median),
> 75%%, 100%% (maximum).");
> out->cr();
>
> Thanks,
> -Aleksey
>
Yes
More information about the shenandoah-dev
mailing list