RFR(T): 8219232: Unit of concurrent active time logging is wrong

Tobias Hartmann tobias.hartmann at oracle.com
Mon Feb 18 14:08:44 UTC 2019


Hi,

please review this trivial fix:

diff -r 2a78b2963793 src/hotspot/share/gc/cms/concurrentMarkSweepGeneration.cpp
--- a/src/hotspot/share/gc/cms/concurrentMarkSweepGeneration.cpp	Wed Feb 13 13:27:17 2019 +0100
+++ b/src/hotspot/share/gc/cms/concurrentMarkSweepGeneration.cpp	Mon Feb 18 15:05:56 2019 +0100
@@ -2761,7 +2761,7 @@
 CMSPhaseAccounting::~CMSPhaseAccounting() {
   _collector->gc_timer_cm()->register_gc_concurrent_end();
   _collector->stopTimer();
- log_debug(gc)("Concurrent active time: %.3fms",
TimeHelper::counter_to_seconds(_collector->timerTicks()));
+ log_debug(gc)("Concurrent active time: %.3fms",
TimeHelper::counter_to_millis(_collector->timerTicks()));
   log_trace(gc)(" (CMS %s yielded %d times)", _title, _collector->yields());
 }

Thanks,
Tobias


More information about the hotspot-gc-dev mailing list