Skip to content
Permalink
Browse files

8241926: Shenandoah: only print heap changes for operations that dire…

…ctly affect it

Reviewed-by: rkennke
  • Loading branch information
shipilev committed Apr 1, 2020
1 parent fc806b6 commit 6af855fd977131c8c5b65621fda1bcb9f71c4b1e
Showing with 7 additions and 7 deletions.
  1. +7 −7 src/hotspot/share/gc/shenandoah/shenandoahHeap.cpp
@@ -2799,7 +2799,7 @@ void ShenandoahHeap::entry_mark() {
TraceCollectorStats tcs(monitoring_support()->concurrent_collection_counters());

const char* msg = conc_mark_event_message();
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

ShenandoahWorkerScope scope(workers(),
@@ -2815,7 +2815,7 @@ void ShenandoahHeap::entry_evac() {
TraceCollectorStats tcs(monitoring_support()->concurrent_collection_counters());

static const char* msg = "Concurrent evacuation";
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

ShenandoahWorkerScope scope(workers(),
@@ -2830,7 +2830,7 @@ void ShenandoahHeap::entry_updaterefs() {
ShenandoahGCPhase phase(ShenandoahPhaseTimings::conc_update_refs);

static const char* msg = "Concurrent update references";
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

ShenandoahWorkerScope scope(workers(),
@@ -2845,7 +2845,7 @@ void ShenandoahHeap::entry_roots() {
ShenandoahGCPhase phase(ShenandoahPhaseTimings::conc_roots);

static const char* msg = "Concurrent roots processing";
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

ShenandoahWorkerScope scope(workers(),
@@ -2873,7 +2873,7 @@ void ShenandoahHeap::entry_reset() {
ShenandoahGCPhase phase(ShenandoahPhaseTimings::conc_reset);

static const char* msg = "Concurrent reset";
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

ShenandoahWorkerScope scope(workers(),
@@ -2887,7 +2887,7 @@ void ShenandoahHeap::entry_reset() {
void ShenandoahHeap::entry_preclean() {
if (ShenandoahPreclean && process_references()) {
static const char* msg = "Concurrent precleaning";
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

ShenandoahGCPhase conc_preclean(ShenandoahPhaseTimings::conc_preclean);
@@ -2904,7 +2904,7 @@ void ShenandoahHeap::entry_preclean() {

void ShenandoahHeap::entry_traversal() {
static const char* msg = conc_traversal_event_message();
GCTraceTime(Info, gc) time(msg, NULL, GCCause::_no_gc, true);
GCTraceTime(Info, gc) time(msg);
EventMark em("%s", msg);

TraceCollectorStats tcs(monitoring_support()->concurrent_collection_counters());

0 comments on commit 6af855f

Please sign in to comment.