8279386: Remove duplicate RefProcPhaseTimeTracker
Reviewed-by: iwalulya, mli
This commit is contained in:
parent
d1e6f26160
commit
99a8351bc9
src/hotspot/share/gc/shared
@ -748,8 +748,6 @@ void ReferenceProcessor::process_soft_weak_final_refs(RefProcProxyTask& proxy_ta
|
||||
maybe_balance_queues(_discoveredFinalRefs);
|
||||
}
|
||||
|
||||
RefProcPhaseTimeTracker tt(SoftWeakFinalRefsPhase, &phase_times);
|
||||
|
||||
log_reflist("SoftWeakFinalRefsPhase Soft before", _discoveredSoftRefs, _max_num_queues);
|
||||
log_reflist("SoftWeakFinalRefsPhase Weak before", _discoveredWeakRefs, _max_num_queues);
|
||||
log_reflist("SoftWeakFinalRefsPhase Final before", _discoveredFinalRefs, _max_num_queues);
|
||||
@ -780,7 +778,6 @@ void ReferenceProcessor::process_final_keep_alive(RefProcProxyTask& proxy_task,
|
||||
}
|
||||
|
||||
// Traverse referents of final references and keep them and followers alive.
|
||||
RefProcPhaseTimeTracker tt(KeepAliveFinalRefsPhase, &phase_times);
|
||||
RefProcKeepAliveFinalPhaseTask phase_task(*this, &phase_times);
|
||||
run_task(phase_task, proxy_task, true);
|
||||
|
||||
@ -804,9 +801,6 @@ void ReferenceProcessor::process_phantom_refs(RefProcProxyTask& proxy_task,
|
||||
maybe_balance_queues(_discoveredPhantomRefs);
|
||||
}
|
||||
|
||||
// Walk phantom references appropriately.
|
||||
RefProcPhaseTimeTracker tt(PhantomRefsPhase, &phase_times);
|
||||
|
||||
log_reflist("PhantomRefsPhase Phantom before", _discoveredPhantomRefs, _max_num_queues);
|
||||
|
||||
RefProcPhantomPhaseTask phase_task(*this, &phase_times);
|
||||
|
@ -143,16 +143,6 @@ RefProcBalanceQueuesTimeTracker::~RefProcBalanceQueuesTimeTracker() {
|
||||
phase_times()->set_balance_queues_time_ms(_phase_number, elapsed);
|
||||
}
|
||||
|
||||
RefProcPhaseTimeTracker::RefProcPhaseTimeTracker(ReferenceProcessor::RefProcPhases phase_number,
|
||||
ReferenceProcessorPhaseTimes* phase_times) :
|
||||
RefProcPhaseTimeBaseTracker(phase_enum_2_phase_string(phase_number), phase_number, phase_times) {
|
||||
}
|
||||
|
||||
RefProcPhaseTimeTracker::~RefProcPhaseTimeTracker() {
|
||||
double elapsed = elapsed_time();
|
||||
phase_times()->set_phase_time_ms(_phase_number, elapsed);
|
||||
}
|
||||
|
||||
RefProcTotalPhaseTimesTracker::RefProcTotalPhaseTimesTracker(ReferenceProcessor::RefProcPhases phase_number,
|
||||
ReferenceProcessorPhaseTimes* phase_times) :
|
||||
RefProcPhaseTimeBaseTracker(phase_enum_2_phase_string(phase_number), phase_number, phase_times) {
|
||||
|
@ -146,14 +146,6 @@ public:
|
||||
~RefProcBalanceQueuesTimeTracker();
|
||||
};
|
||||
|
||||
// Updates phase time at ReferenceProcessorPhaseTimes and save it into GCTimer.
|
||||
class RefProcPhaseTimeTracker : public RefProcPhaseTimeBaseTracker {
|
||||
public:
|
||||
RefProcPhaseTimeTracker(ReferenceProcessor::RefProcPhases phase_number,
|
||||
ReferenceProcessorPhaseTimes* phase_times);
|
||||
~RefProcPhaseTimeTracker();
|
||||
};
|
||||
|
||||
// Highest level time tracker.
|
||||
class RefProcTotalPhaseTimesTracker : public RefProcPhaseTimeBaseTracker {
|
||||
public:
|
||||
|
Loading…
x
Reference in New Issue
Block a user