changeset 11269:14c4374d3346

8153582: Logging of ConcGCThreads is done too early Summary: ConcGCThreads is logged as 0 because it hasn't been initialized yet. Reviewed-by: tschatzl, dfazunen
author jprovino
date Thu, 19 May 2016 10:48:28 -0400
parents 67bc9d6f1d94
children e62bbe48bd66
files src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp src/share/vm/gc/g1/g1ConcurrentMark.cpp src/share/vm/runtime/arguments.cpp
diffstat 3 files changed, 4 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp	Thu May 19 11:43:21 2016 +0200
+++ b/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp	Thu May 19 10:48:28 2016 -0400
@@ -545,6 +545,8 @@
   }
   assert((_conc_workers != NULL) == (ConcGCThreads > 1),
          "Inconsistency");
+  log_debug(gc)("ConcGCThreads: %u", ConcGCThreads);
+  log_debug(gc)("ParallelGCThreads: %u", ParallelGCThreads);
 
   // Parallel task queues; these are shared for the
   // concurrent and stop-world phases of CMS, but
--- a/src/share/vm/gc/g1/g1ConcurrentMark.cpp	Thu May 19 11:43:21 2016 +0200
+++ b/src/share/vm/gc/g1/g1ConcurrentMark.cpp	Thu May 19 10:48:28 2016 -0400
@@ -438,6 +438,8 @@
   }
 
   assert(ConcGCThreads > 0, "Should have been set");
+  log_debug(gc)("ConcGCThreads: %u", ConcGCThreads);
+  log_debug(gc)("ParallelGCThreads: %u", ParallelGCThreads);
   _parallel_marking_threads = ConcGCThreads;
   _max_parallel_marking_threads = _parallel_marking_threads;
 
--- a/src/share/vm/runtime/arguments.cpp	Thu May 19 11:43:21 2016 +0200
+++ b/src/share/vm/runtime/arguments.cpp	Thu May 19 10:48:28 2016 -0400
@@ -1641,7 +1641,6 @@
   }
 
   log_trace(gc)("MarkStackSize: %uk  MarkStackSizeMax: %uk", (unsigned int) (MarkStackSize / K), (uint) (MarkStackSizeMax / K));
-  log_trace(gc)("ConcGCThreads: %u", ConcGCThreads);
 }
 #endif // INCLUDE_ALL_GCS
 
@@ -1949,7 +1948,6 @@
   }
 
   log_trace(gc)("MarkStackSize: %uk  MarkStackSizeMax: %uk", (unsigned int) (MarkStackSize / K), (uint) (MarkStackSizeMax / K));
-  log_trace(gc)("ConcGCThreads: %u", ConcGCThreads);
 }
 
 void Arguments::set_gc_specific_flags() {