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

rev 49680 : imported patch 6672778-partial-queue-trimming
rev 49681 : imported patch 6672778-refactoring
rev 49682 : [mq]: 6672778-stefanj-review
rev 49683 : [mq]: 6672778-inconsistent-time-fixes

*** 481,505 ****
  
    assert(_pss->trim_ticks().value() == 0, "Possibly remaining trim ticks left over from previous use");
  }
  
  G1EvacPhaseWithTrimTimeTracker::~G1EvacPhaseWithTrimTimeTracker() {
!   _total_time += (Ticks::now() - _start) + _pss->trim_ticks();
    _trim_time += _pss->trim_ticks();
    _pss->reset_trim_ticks();
  }
  
  G1GCParPhaseTimesTracker::G1GCParPhaseTimesTracker(G1GCPhaseTimes* phase_times, G1GCPhaseTimes::GCParPhases phase, uint worker_id) :
      _phase_times(phase_times), _phase(phase), _worker_id(worker_id) {
    if (_phase_times != NULL) {
!     _start_time = os::elapsedTime();
    }
  }
  
  G1GCParPhaseTimesTracker::~G1GCParPhaseTimesTracker() {
    if (_phase_times != NULL) {
!     _phase_times->record_time_secs(_phase, _worker_id, os::elapsedTime() - _start_time);
    }
  }
  
  G1EvacPhaseTimesTracker::G1EvacPhaseTimesTracker(G1GCPhaseTimes* phase_times,
                                                   G1ParScanThreadState* pss,
--- 481,505 ----
  
    assert(_pss->trim_ticks().value() == 0, "Possibly remaining trim ticks left over from previous use");
  }
  
  G1EvacPhaseWithTrimTimeTracker::~G1EvacPhaseWithTrimTimeTracker() {
!   _total_time += (Ticks::now() - _start) - _pss->trim_ticks();
    _trim_time += _pss->trim_ticks();
    _pss->reset_trim_ticks();
  }
  
  G1GCParPhaseTimesTracker::G1GCParPhaseTimesTracker(G1GCPhaseTimes* phase_times, G1GCPhaseTimes::GCParPhases phase, uint worker_id) :
      _phase_times(phase_times), _phase(phase), _worker_id(worker_id) {
    if (_phase_times != NULL) {
!     _start_time = Ticks::now();
    }
  }
  
  G1GCParPhaseTimesTracker::~G1GCParPhaseTimesTracker() {
    if (_phase_times != NULL) {
!     _phase_times->record_time_secs(_phase, _worker_id, TicksToTimeHelper::seconds(Ticks::now() - _start_time));
    }
  }
  
  G1EvacPhaseTimesTracker::G1EvacPhaseTimesTracker(G1GCPhaseTimes* phase_times,
                                                   G1ParScanThreadState* pss,
*** 511,520 ****
    _trim_tracker(pss, _total_time, _trim_time) {
  }
  
  G1EvacPhaseTimesTracker::~G1EvacPhaseTimesTracker() {
    if (_phase_times != NULL) {
!     _start_time += TicksToTimeHelper::seconds(_trim_time);
      _phase_times->record_or_add_objcopy_time_secs(_worker_id, TicksToTimeHelper::seconds(_trim_time));
    }
  }
  
--- 511,521 ----
    _trim_tracker(pss, _total_time, _trim_time) {
  }
  
  G1EvacPhaseTimesTracker::~G1EvacPhaseTimesTracker() {
    if (_phase_times != NULL) {
!     // Exclude trim time by increasing the start time.
!     _start_time += _trim_time;
      _phase_times->record_or_add_objcopy_time_secs(_worker_id, TicksToTimeHelper::seconds(_trim_time));
    }
  }