Skip to content
This repository was archived by the owner on Sep 19, 2023. It is now read-only.

Commit 9997c71

Browse files
olivergillespiePaul Hohensee
authored andcommitted
8307346: Add missing gc+phases logging for ObjectCount(AfterGC) JFR event collection code
Backport-of: 3f1927a7f3a2914402a25335c47a5a8bdd5511a6
1 parent 68fb0aa commit 9997c71

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
@@ -311,7 +311,10 @@ void G1FullCollector::phase1_mark_live_objects() {
311311
_heap->complete_cleaning(purged_class);
312312
}
313313

314-
scope()->tracer()->report_object_count_after_gc(&_is_alive);
314+
{
315+
GCTraceTime(Debug, gc, phases) debug("Report Object Count", scope()->timer());
316+
scope()->tracer()->report_object_count_after_gc(&_is_alive);
317+
}
315318
#if TASKQUEUE_STATS
316319
oop_queue_set()->print_and_reset_taskqueue_stats("Oop Queue");
317320
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
@@ -2070,7 +2070,10 @@ void PSParallelCompact::marking_phase(ParallelOldTracer *gc_tracer) {
20702070
JVMCI_ONLY(JVMCI::do_unloading(purged_class));
20712071
}
20722072

2073-
_gc_tracer.report_object_count_after_gc(is_alive_closure());
2073+
{
2074+
GCTraceTime(Debug, gc, phases) tm("Report Object Count", &_gc_timer);
2075+
_gc_tracer.report_object_count_after_gc(is_alive_closure());
2076+
}
20742077
#if TASKQUEUE_STATS
20752078
ParCompactionManager::oop_task_queues()->print_and_reset_taskqueue_stats("Oop Queue");
20762079
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
@@ -230,7 +230,10 @@ void GenMarkSweep::mark_sweep_phase1(bool clear_all_softrefs) {
230230
JVMCI_ONLY(JVMCI::do_unloading(purged_class));
231231
}
232232

233-
gc_tracer()->report_object_count_after_gc(&is_alive);
233+
{
234+
GCTraceTime(Debug, gc, phases) tm_m("Report Object Count", gc_timer());
235+
gc_tracer()->report_object_count_after_gc(&is_alive);
236+
}
234237
}
235238

236239

0 commit comments

Comments
 (0)