--- old/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp 2016-03-09 21:22:06.478789417 +0900 +++ new/src/share/vm/gc/cms/concurrentMarkSweepGeneration.cpp 2016-03-09 21:22:06.175790567 +0900 @@ -627,6 +627,7 @@ NOT_PRODUCT(_overflow_counter = CMSMarkStackOverflowInterval;) _gc_counters = new CollectorCounters("CMS", 1); + _cgc_counters = new CollectorCounters("CMS stop-the-world phases", 2); _completed_initialization = true; _inter_sweep_timer.start(); // start of time } @@ -5550,18 +5551,19 @@ void CMSCollector::do_CMS_operation(CMS_op_type op, GCCause::Cause gc_cause) { GCTraceCPUTime tcpu; - TraceCollectorStats tcs(counters()); + TraceCollectorStats tcs(EnableConcGCPerfCounter ? NULL : counters()); + TraceCollectorStats tcs_cgc(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-09 21:22:08.148783078 +0900 +++ new/src/share/vm/gc/cms/concurrentMarkSweepGeneration.hpp 2016-03-09 21:22:07.867784144 +0900 @@ -555,6 +555,7 @@ // Performance Counters CollectorCounters* _gc_counters; + CollectorCounters* _cgc_counters; // Initialization Errors bool _completed_initialization; @@ -929,7 +930,8 @@ NOT_PRODUCT(bool is_cms_reachable(HeapWord* addr);) // Performance Counter Support - CollectorCounters* counters() { return _gc_counters; } + 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-09 21:22:09.352778508 +0900 +++ new/src/share/vm/gc/g1/g1ConcurrentMark.cpp 2016-03-09 21:22:09.168779206 +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-09 21:22:10.720773315 +0900 +++ new/src/share/vm/gc/g1/g1MonitoringSupport.cpp 2016-03-09 21:22:10.355774701 +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-09 21:22:11.952768639 +0900 +++ new/src/share/vm/gc/g1/g1MonitoringSupport.hpp 2016-03-09 21:22:11.596769991 +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-09 21:22:12.908765011 +0900 +++ new/src/share/vm/gc/g1/vm_operations_g1.cpp 2016-03-09 21:22:12.710765762 +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/collectorCounters.hpp 2016-03-09 21:22:13.926761147 +0900 +++ new/src/share/vm/gc/shared/collectorCounters.hpp 2016-03-09 21:22:13.726761906 +0900 @@ -71,16 +71,19 @@ public: inline TraceCollectorStats(CollectorCounters* c) : - PerfTraceTimedEvent(c->time_counter(), c->invocation_counter()), + PerfTraceTimedEvent((c == NULL) ? NULL : c->time_counter(), + (c == NULL) ? NULL : c->invocation_counter()), _c(c) { - if (UsePerfData) { + if (UsePerfData && (_c != NULL)) { _c->last_entry_counter()->set_value(os::elapsed_counter()); } } inline ~TraceCollectorStats() { - if (UsePerfData) _c->last_exit_counter()->set_value(os::elapsed_counter()); + if (UsePerfData && (_c != NULL)) { + _c->last_exit_counter()->set_value(os::elapsed_counter()); + } } }; --- old/src/share/vm/gc/shared/vmGCOperations.hpp 2016-03-09 21:22:14.728758103 +0900 +++ new/src/share/vm/gc/shared/vmGCOperations.hpp 2016-03-09 21:22:14.491759002 +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); --- old/src/share/vm/runtime/globals.hpp 2016-03-09 21:22:15.742754254 +0900 +++ new/src/share/vm/runtime/globals.hpp 2016-03-09 21:22:15.450755362 +0900 @@ -3929,6 +3929,9 @@ "0=off, 1=conservative, 2=aggressive") \ range(0, 2) \ \ + product(bool, EnableConcGCPerfCounter, false, \ + "Enable PerfCounter for STW in CMS GC") \ + \ /* Serviceability Support */ \ \ product(bool, ManagementServer, false, \ --- old/src/share/vm/runtime/perfData.hpp 2016-03-09 21:22:17.200748720 +0900 +++ new/src/share/vm/runtime/perfData.hpp 2016-03-09 21:22:16.876749950 +0900 @@ -916,22 +916,22 @@ public: inline PerfTraceTime(PerfLongCounter* timerp) : _timerp(timerp), _recursion_counter(NULL) { - if (!UsePerfData) return; + if (!UsePerfData || (_timerp == NULL)) return; _t.start(); } inline PerfTraceTime(PerfLongCounter* timerp, int* recursion_counter) : _timerp(timerp), _recursion_counter(recursion_counter) { - if (!UsePerfData || (_recursion_counter != NULL && - (*_recursion_counter)++ > 0)) return; + if (!UsePerfData || (_timerp == NULL) || + (_recursion_counter != NULL && (*_recursion_counter)++ > 0)) return; _t.start(); } - inline void suspend() { if (!UsePerfData) return; _t.stop(); } - inline void resume() { if (!UsePerfData) return; _t.start(); } + inline void suspend() { if (!UsePerfData || (_timerp == NULL)) return; _t.stop(); } + inline void resume() { if (!UsePerfData || (_timerp == NULL)) return; _t.start(); } inline ~PerfTraceTime() { - if (!UsePerfData || (_recursion_counter != NULL && - --(*_recursion_counter) > 0)) return; + if (!UsePerfData || (_timerp == NULL) || + (_recursion_counter != NULL && --(*_recursion_counter) > 0)) return; _t.stop(); _timerp->inc(_t.ticks()); } @@ -963,12 +963,12 @@ public: inline PerfTraceTimedEvent(PerfLongCounter* timerp, PerfLongCounter* eventp): PerfTraceTime(timerp), _eventp(eventp) { - if (!UsePerfData) return; + if (!UsePerfData || (timerp == NULL)) return; _eventp->inc(); } inline PerfTraceTimedEvent(PerfLongCounter* timerp, PerfLongCounter* eventp, int* recursion_counter): PerfTraceTime(timerp, recursion_counter), _eventp(eventp) { - if (!UsePerfData) return; + if (!UsePerfData || (timerp == NULL)) return; _eventp->inc(); } };