From bd1aac2cd36496867250c4b4fa843bfac043aa8b Mon Sep 17 00:00:00 2001 From: Thomas Schatzl Date: Fri, 24 Mar 2017 10:26:58 +0100 Subject: [PATCH] 8177453: Start_new_collection_set() timing done during wrong type of GC Move timing code from full gc to young gc. Reviewed-by: sjohanss, mgerdin --- hotspot/src/share/vm/gc/g1/g1CollectedHeap.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hotspot/src/share/vm/gc/g1/g1CollectedHeap.cpp b/hotspot/src/share/vm/gc/g1/g1CollectedHeap.cpp index 1bef3fb306c..568ff00d50a 100644 --- a/hotspot/src/share/vm/gc/g1/g1CollectedHeap.cpp +++ b/hotspot/src/share/vm/gc/g1/g1CollectedHeap.cpp @@ -1378,9 +1378,7 @@ bool G1CollectedHeap::do_full_collection(bool explicit_gc, } _verifier->check_bitmaps("Full GC End"); - double start = os::elapsedTime(); start_new_collection_set(); - g1_policy()->phase_times()->record_start_new_cset_time_ms((os::elapsedTime() - start) * 1000.0); _allocator->init_mutator_alloc_region(); @@ -3212,7 +3210,9 @@ G1CollectedHeap::do_collection_pause_at_safepoint(double target_pause_time_ms) { _survivor_evac_stats.adjust_desired_plab_sz(); _old_evac_stats.adjust_desired_plab_sz(); + double start = os::elapsedTime(); start_new_collection_set(); + g1_policy()->phase_times()->record_start_new_cset_time_ms((os::elapsedTime() - start) * 1000.0); if (evacuation_failed()) { set_used(recalculate_used());