8154955: Negative Other Time in gc logs due to 'Wait for Root Region Scan' not included

Move the note_gc_start() call before the wait_for_root_region_scanning() one.

Reviewed-by: kbarrett, tschatzl
This commit is contained in:
Yu Zhang 2016-05-02 17:45:19 +02:00 committed by Thomas Schatzl
parent d4bfb42264
commit 547e732790

View File

@ -3106,6 +3106,8 @@ G1CollectedHeap::do_collection_pause_at_safepoint(double target_pause_time_ms) {
SvcGCMarker sgcm(SvcGCMarker::MINOR);
ResourceMark rm;
g1_policy()->note_gc_start();
wait_for_root_region_scanning();
print_heap_before_gc();
@ -3165,8 +3167,6 @@ G1CollectedHeap::do_collection_pause_at_safepoint(double target_pause_time_ms) {
Threads::number_of_non_daemon_threads());
workers()->set_active_workers(active_workers);
g1_policy()->note_gc_start();
TraceCollectorStats tcs(g1mm()->incremental_collection_counters());
TraceMemoryManagerStats tms(false /* fullGC */, gc_cause());