stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8326209: G1: Remove G1ConcurrentMark::_total_cleanup_time

Browse Source 
Reviewed-by: kbarrett, tschatzl
This commit is contained in:
Albert Mingkun Yang 2024-02-20 09:38:13 +00:00
parent 56073cd1ea
commit beed4145ec
2 changed files with 2 additions and 6 deletions

7
src/hotspot/share/gc/g1/g1ConcurrentMark.cpp

@ -485,7 +485,6 @@ G1ConcurrentMark::G1ConcurrentMark(G1CollectedHeap* g1h,
_remark_mark_times(),
_remark_weak_ref_times(),
_cleanup_times(),
_total_cleanup_time(0.0),
_accum_task_vtime(nullptr),
@ -1549,9 +1548,7 @@ void G1ConcurrentMark::cleanup() {
verify_during_pause(G1HeapVerifier::G1VerifyCleanup, VerifyLocation::CleanupAfter);
// Local statistics
double recent_cleanup_time = (os::elapsedTime() - start);
_total_cleanup_time += recent_cleanup_time;
_cleanup_times.add(recent_cleanup_time);
_cleanup_times.add((os::elapsedTime() - start) * 1000.0);
{
GCTraceTime(Debug, gc, phases) debug("Finalize Concurrent Mark Cleanup", _gc_timer_cm);
@ -2126,7 +2123,7 @@ void G1ConcurrentMark::print_summary_info() {
}
print_ms_time_info(" ", "cleanups", _cleanup_times);
log.trace(" Finalize live data total time = %8.2f s (avg = %8.2f ms).",
_total_cleanup_time, (_cleanup_times.num() > 0 ? _total_cleanup_time * 1000.0 / (double)_cleanup_times.num() : 0.0));
_cleanup_times.sum() / 1000.0, _cleanup_times.avg());
log.trace(" Total stop_world time = %8.2f s.",
(_init_times.sum() + _remark_times.sum() + _cleanup_times.sum())/1000.0);
log.trace(" Total concurrent time = %8.2f s (%8.2f s marking).",

1
src/hotspot/share/gc/g1/g1ConcurrentMark.hpp

@ -440,7 +440,6 @@ class G1ConcurrentMark : public CHeapObj<mtGC> {
NumberSeq _remark_mark_times;
NumberSeq _remark_weak_ref_times;
NumberSeq _cleanup_times;
double _total_cleanup_time;
double* _accum_task_vtime; // Accumulated task vtime