Skip to content

Commit 3f1927a

Browse files
olivergillespieshipilev
authored andcommitted
8307346: Add missing gc+phases logging for ObjectCount(AfterGC) JFR event collection code
Reviewed-by: tschatzl, shade, ayang
1 parent e206d57 commit 3f1927a

File tree

3 files changed

+12
-3
lines changed

3 files changed

+12
-3
lines changed

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

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -325,7 +325,10 @@ void G1FullCollector::phase1_mark_live_objects() {
325325
_heap->complete_cleaning(purged_class);
326326
}
327327

328-
scope()->tracer()->report_object_count_after_gc(&_is_alive);
328+
{
329+
GCTraceTime(Debug, gc, phases) debug("Report Object Count", scope()->timer());
330+
scope()->tracer()->report_object_count_after_gc(&_is_alive);
331+
}
329332
#if TASKQUEUE_STATS
330333
oop_queue_set()->print_and_reset_taskqueue_stats("Oop Queue");
331334
array_queue_set()->print_and_reset_taskqueue_stats("ObjArrayOop Queue");

src/hotspot/share/gc/parallel/psParallelCompact.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2067,7 +2067,10 @@ void PSParallelCompact::marking_phase(ParallelOldTracer *gc_tracer) {
20672067
JVMCI_ONLY(JVMCI::do_unloading(purged_class));
20682068
}
20692069

2070-
_gc_tracer.report_object_count_after_gc(is_alive_closure());
2070+
{
2071+
GCTraceTime(Debug, gc, phases) tm("Report Object Count", &_gc_timer);
2072+
_gc_tracer.report_object_count_after_gc(is_alive_closure());
2073+
}
20712074
#if TASKQUEUE_STATS
20722075
ParCompactionManager::oop_task_queues()->print_and_reset_taskqueue_stats("Oop Queue");
20732076
ParCompactionManager::_objarray_task_queues->print_and_reset_taskqueue_stats("ObjArrayOop Queue");

src/hotspot/share/gc/serial/genMarkSweep.cpp

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,10 @@ void GenMarkSweep::mark_sweep_phase1(bool clear_all_softrefs) {
210210
JVMCI_ONLY(JVMCI::do_unloading(purged_class));
211211
}
212212

213-
gc_tracer()->report_object_count_after_gc(&is_alive);
213+
{
214+
GCTraceTime(Debug, gc, phases) tm_m("Report Object Count", gc_timer());
215+
gc_tracer()->report_object_count_after_gc(&is_alive);
216+
}
214217
}
215218

216219

0 commit comments

Comments
 (0)