RFR: Print correct message about gross times in stats
Aleksey Shipilev
shade at redhat.com
Mon May 15 18:13:29 UTC 2017
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
More information about the shenandoah-dev
mailing list