infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-02-16 05:15:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

8153582: Logging of ConcGCThreads is done too early

Browse Source 
ConcGCThreads is logged as 0 because it hasn't been initialized yet.

Reviewed-by: tschatzl, dfazunen
This commit is contained in:
Joseph Provino 2016-05-19 10:48:28 -04:00
parent 2f4c6f1b05
commit 5648deddee
3 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hotspot/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp
View File

@ -545,6 +545,8 @@ CMSCollector::CMSCollector(ConcurrentMarkSweepGeneration* cmsGen,
}
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

2
hotspot/src/share/vm/gc/g1/g1ConcurrentMark.cpp
View File

@ -438,6 +438,8 @@ G1ConcurrentMark::G1ConcurrentMark(G1CollectedHeap* g1h, G1RegionToSpaceMapper*
}
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;

2
hotspot/src/share/vm/runtime/arguments.cpp
View File

@ -1641,7 +1641,6 @@ void Arguments::set_cms_and_parnew_gc_flags() {
}
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 @@ void Arguments::set_g1_gc_flags() {
}
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() {