--- old/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp 2016-03-08 23:25:25.942268869 +0900 +++ new/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp 2016-03-08 23:25:25.777269252 +0900 @@ -626,7 +626,8 @@ } NOT_PRODUCT(_overflow_counter = CMSMarkStackOverflowInterval;) - _gc_counters = new CollectorCounters("CMS", 1); + _gc_counters = new CollectorCounters("CMS stop-the-world full collections", 1); + _cgc_counters = new CollectorCounters("CMS stop-the-world phases", 2); _completed_initialization = true; _inter_sweep_timer.start(); // start of time } @@ -5550,18 +5551,18 @@ void CMSCollector::do_CMS_operation(CMS_op_type op, GCCause::Cause gc_cause) { GCTraceCPUTime tcpu; - TraceCollectorStats tcs(counters()); + TraceCollectorStats tcs(cgc_counters()); switch (op) { case CMS_op_checkpointRootsInitial: { GCTraceTime(Info, gc) t("Pause Initial Mark", NULL, GCCause::_no_gc, true); - SvcGCMarker sgcm(SvcGCMarker::OTHER); + SvcGCMarker sgcm(SvcGCMarker::CONCURRENT); checkpointRootsInitial(); break; } case CMS_op_checkpointRootsFinal: { GCTraceTime(Info, gc) t("Pause Remark", NULL, GCCause::_no_gc, true); - SvcGCMarker sgcm(SvcGCMarker::OTHER); + SvcGCMarker sgcm(SvcGCMarker::CONCURRENT); checkpointRootsFinal(); break; } --- old/src/share/vm/gc/cms/concurrentMarkSweepGeneration.hpp 2016-03-08 23:25:26.783266918 +0900 +++ new/src/share/vm/gc/cms/concurrentMarkSweepGeneration.hpp 2016-03-08 23:25:26.628267277 +0900 @@ -555,6 +555,7 @@ // Performance Counters CollectorCounters* _gc_counters; + CollectorCounters* _cgc_counters; // Initialization Errors bool _completed_initialization; @@ -930,6 +931,7 @@ // Performance Counter Support CollectorCounters* counters() { return _gc_counters; } + CollectorCounters* cgc_counters() { return _cgc_counters; } // Timer stuff void startTimer() { assert(!_timer.is_active(), "Error"); _timer.start(); } --- old/src/share/vm/gc/g1/g1ConcurrentMark.cpp 2016-03-08 23:25:27.421265437 +0900 +++ new/src/share/vm/gc/g1/g1ConcurrentMark.cpp 2016-03-08 23:25:27.273265781 +0900 @@ -1087,8 +1087,6 @@ return; } - SvcGCMarker sgcm(SvcGCMarker::OTHER); - if (VerifyDuringGC) { HandleMark hm; // handle scope g1h->prepare_for_verify(); --- old/src/share/vm/gc/g1/g1MonitoringSupport.cpp 2016-03-08 23:25:28.072263927 +0900 +++ new/src/share/vm/gc/g1/g1MonitoringSupport.cpp 2016-03-08 23:25:27.926264266 +0900 @@ -76,6 +76,7 @@ _g1h(g1h), _incremental_collection_counters(NULL), _full_collection_counters(NULL), + _conc_collection_counters(NULL), _old_collection_counters(NULL), _old_space_counters(NULL), _young_collection_counters(NULL), @@ -104,6 +105,10 @@ // old generation collection. _full_collection_counters = new CollectorCounters("G1 stop-the-world full collections", 1); + // name "collector.2". In a generational collector this would be the + // STW phases in concurrent collection. + _conc_collection_counters = + new CollectorCounters("G1 stop-the-world phases", 2); // timer sampling for all counters supporting sampling only update the // used value. See the take_sample() method. G1 requires both used and --- old/src/share/vm/gc/g1/g1MonitoringSupport.hpp 2016-03-08 23:25:28.642262604 +0900 +++ new/src/share/vm/gc/g1/g1MonitoringSupport.hpp 2016-03-08 23:25:28.498262938 +0900 @@ -122,6 +122,8 @@ CollectorCounters* _incremental_collection_counters; // full stop-the-world collections CollectorCounters* _full_collection_counters; + // stop-the-world phases in G1 + CollectorCounters* _conc_collection_counters; // young collection set counters. The _eden_counters, // _from_counters, and _to_counters are associated with // this "generational" counter. @@ -210,6 +212,9 @@ CollectorCounters* full_collection_counters() { return _full_collection_counters; } + CollectorCounters* conc_collection_counters() { + return _conc_collection_counters; + } GenerationCounters* young_collection_counters() { return _young_collection_counters; } --- old/src/share/vm/gc/g1/vm_operations_g1.cpp 2016-03-08 23:25:29.225261251 +0900 +++ new/src/share/vm/gc/g1/vm_operations_g1.cpp 2016-03-08 23:25:29.080261588 +0900 @@ -229,6 +229,8 @@ GCTraceCPUTime tcpu; G1CollectedHeap* g1h = G1CollectedHeap::heap(); GCTraceTime(Info, gc) t(_printGCMessage, g1h->gc_timer_cm(), GCCause::_no_gc, true); + TraceCollectorStats tcs(g1h->g1mm()->conc_collection_counters()); + SvcGCMarker sgcm(SvcGCMarker::CONCURRENT); IsGCActiveMark x; _cl->do_void(); } --- old/src/share/vm/gc/shared/vmGCOperations.hpp 2016-03-08 23:25:29.783259957 +0900 +++ new/src/share/vm/gc/shared/vmGCOperations.hpp 2016-03-08 23:25:29.638260293 +0900 @@ -232,7 +232,7 @@ private: JvmtiGCMarker _jgcm; public: - typedef enum { MINOR, FULL, OTHER } reason_type; + typedef enum { MINOR, FULL, CONCURRENT, OTHER } reason_type; SvcGCMarker(reason_type reason ) { VM_GC_Operation::notify_gc_begin(reason == FULL);