8218920: Scan HCC should be on the same level as Update RS etc. in the log
Reviewed-by: lkorinth, kbarrett
--- a/src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp Tue Feb 19 12:56:00 2019 +0100
+++ b/src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp Tue Feb 19 12:56:02 2019 +0100
@@ -383,10 +383,10 @@
for (int i = ThreadRoots; i <= SATBFiltering; i++) {
trace_phase(_gc_par_phases[i]);
}
+ if (G1HotCardCache::default_use_cache()) {
+ debug_phase(_gc_par_phases[ScanHCC]);
+ }
debug_phase(_gc_par_phases[UpdateRS]);
- if (G1HotCardCache::default_use_cache()) {
- trace_phase(_gc_par_phases[ScanHCC]);
- }
debug_phase(_gc_par_phases[ScanRS]);
debug_phase(_gc_par_phases[CodeRoots]);
debug_phase(_gc_par_phases[ObjCopy]);
--- a/test/hotspot/jtreg/gc/g1/TestGCLogMessages.java Tue Feb 19 12:56:00 2019 +0100
+++ b/test/hotspot/jtreg/gc/g1/TestGCLogMessages.java Tue Feb 19 12:56:02 2019 +0100
@@ -100,7 +100,7 @@
new LogMessageWithLevel("Processed Buffers", Level.DEBUG),
new LogMessageWithLevel("Scanned Cards", Level.DEBUG),
new LogMessageWithLevel("Skipped Cards", Level.DEBUG),
- new LogMessageWithLevel("Scan HCC", Level.TRACE),
+ new LogMessageWithLevel("Scan HCC", Level.DEBUG),
// Scan RS
new LogMessageWithLevel("Scan RS", Level.DEBUG),
new LogMessageWithLevel("Scanned Cards", Level.DEBUG),