src/hotspot/share/gc/g1/g1GCPhaseTimes.cpp

rev 49826 : imported patch 6672778-partial-queue-trimming
rev 49827 : imported patch 6672778-refactoring
rev 49828 : imported patch 6672778-stefanj-review
rev 49829 : imported patch 6672778-inconsistent-time-fixes
rev 49830 : [mq]: 6672778-satb-timing-fix

*** 191,216 ****
      if (worker_start != uninitialized) {
        assert(_gc_par_phases[GCWorkerEnd]->get(i) != uninitialized, "Worker started but not ended.");
        double total_worker_time = _gc_par_phases[GCWorkerEnd]->get(i) - _gc_par_phases[GCWorkerStart]->get(i);
        record_time_secs(GCWorkerTotal, i , total_worker_time);
  
!       double worker_known_time =
!           worker_time(ExtRootScan, i)
!           + worker_time(SATBFiltering, i)
!           + worker_time(ScanHCC, i)
!           + worker_time(UpdateRS, i)
!           + worker_time(ScanRS, i)
!           + worker_time(CodeRoots, i)
!           + worker_time(ObjCopy, i)
!           + worker_time(Termination, i);
  
        record_time_secs(Other, i, total_worker_time - worker_known_time);
      } else {
        // Make sure all slots are uninitialized since this thread did not seem to have been started
        ASSERT_PHASE_UNINITIALIZED(GCWorkerEnd);
        ASSERT_PHASE_UNINITIALIZED(ExtRootScan);
-       ASSERT_PHASE_UNINITIALIZED(SATBFiltering);
        ASSERT_PHASE_UNINITIALIZED(ScanHCC);
        ASSERT_PHASE_UNINITIALIZED(UpdateRS);
        ASSERT_PHASE_UNINITIALIZED(ScanRS);
        ASSERT_PHASE_UNINITIALIZED(CodeRoots);
        ASSERT_PHASE_UNINITIALIZED(ObjCopy);
--- 191,213 ----
      if (worker_start != uninitialized) {
        assert(_gc_par_phases[GCWorkerEnd]->get(i) != uninitialized, "Worker started but not ended.");
        double total_worker_time = _gc_par_phases[GCWorkerEnd]->get(i) - _gc_par_phases[GCWorkerStart]->get(i);
        record_time_secs(GCWorkerTotal, i , total_worker_time);
  
!       double worker_known_time = worker_time(ExtRootScan, i) +
!                                  worker_time(ScanHCC, i) +
!                                  worker_time(UpdateRS, i) +
!                                  worker_time(ScanRS, i) +
!                                  worker_time(CodeRoots, i) +
!                                  worker_time(ObjCopy, i) +
!                                  worker_time(Termination, i);
  
        record_time_secs(Other, i, total_worker_time - worker_known_time);
      } else {
        // Make sure all slots are uninitialized since this thread did not seem to have been started
        ASSERT_PHASE_UNINITIALIZED(GCWorkerEnd);
        ASSERT_PHASE_UNINITIALIZED(ExtRootScan);
        ASSERT_PHASE_UNINITIALIZED(ScanHCC);
        ASSERT_PHASE_UNINITIALIZED(UpdateRS);
        ASSERT_PHASE_UNINITIALIZED(ScanRS);
        ASSERT_PHASE_UNINITIALIZED(CodeRoots);
        ASSERT_PHASE_UNINITIALIZED(ObjCopy);