--- old/src/share/vm/gc/g1/g1CollectedHeap.cpp 2015-09-08 13:23:20.127225812 +0200 +++ new/src/share/vm/gc/g1/g1CollectedHeap.cpp 2015-09-08 13:23:20.019225816 +0200 @@ -53,6 +53,7 @@ #include "gc/g1/suspendibleThreadSet.hpp" #include "gc/g1/vm_operations_g1.hpp" #include "gc/shared/gcHeapSummary.hpp" +#include "gc/shared/gcId.hpp" #include "gc/shared/gcLocker.inline.hpp" #include "gc/shared/gcTimer.hpp" #include "gc/shared/gcTrace.hpp" @@ -1457,6 +1458,7 @@ gc_timer->register_gc_start(); SerialOldTracer* gc_tracer = G1MarkSweep::gc_tracer(); + GCIdMark gc_id_mark; gc_tracer->report_gc_start(gc_cause(), gc_timer->gc_start()); SvcGCMarker sgcm(SvcGCMarker::FULL); @@ -1483,7 +1485,7 @@ TraceCPUTime tcpu(G1Log::finer(), true, gclog_or_tty); { - GCTraceTime t(GCCauseString("Full GC", gc_cause()), G1Log::fine(), true, NULL, gc_tracer->gc_id()); + GCTraceTime t(GCCauseString("Full GC", gc_cause()), G1Log::fine(), true, NULL); TraceCollectorStats tcs(g1mm()->full_collection_counters()); TraceMemoryManagerStats tms(true /* fullGC */, gc_cause()); @@ -3928,7 +3930,7 @@ return; } - gclog_or_tty->gclog_stamp(_gc_tracer_stw->gc_id()); + gclog_or_tty->gclog_stamp(); GCCauseString gc_cause_str = GCCauseString("GC pause", gc_cause()) .append(collector_state()->gcs_are_young() ? "(young)" : "(mixed)") @@ -3986,6 +3988,7 @@ _gc_timer_stw->register_gc_start(); + GCIdMark gc_id_mark; _gc_tracer_stw->report_gc_start(gc_cause(), _gc_timer_stw->gc_start()); SvcGCMarker sgcm(SvcGCMarker::MINOR); @@ -5560,8 +5563,7 @@ &keep_alive, &drain_queue, NULL, - _gc_timer_stw, - _gc_tracer_stw->gc_id()); + _gc_timer_stw); } else { // Parallel reference processing assert(rp->num_q() == no_of_gc_workers, "sanity"); @@ -5572,8 +5574,7 @@ &keep_alive, &drain_queue, &par_task_executor, - _gc_timer_stw, - _gc_tracer_stw->gc_id()); + _gc_timer_stw); } _gc_tracer_stw->report_gc_reference_stats(stats);