Skip to content

Commit 4044c37

Browse files
committed
8372348: Adjust some UL / JFR string deduplication output messages
Reviewed-by: asteiner, mdoerr Backport-of: 3f20eb943532c5c76e55b14292139749bd704ce4
1 parent 83cca73 commit 4044c37

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

src/hotspot/share/gc/shared/stringdedup/stringDedupStat.cpp

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -100,13 +100,17 @@ void StringDedup::Stat::log_summary(const Stat* last_stat, const Stat* total_sta
100100

101101
log_info(stringdedup)(
102102
"Concurrent String Deduplication "
103-
"%zu/" STRDEDUP_BYTES_FORMAT_NS " (new), "
103+
"%zu (inspected), "
104+
"%zu/" STRDEDUP_BYTES_FORMAT_NS " (new unknown), "
104105
"%zu/" STRDEDUP_BYTES_FORMAT_NS " (deduped), "
105-
"avg " STRDEDUP_PERCENT_FORMAT_NS ", "
106+
"total avg deduped/new unknown bytes " STRDEDUP_PERCENT_FORMAT_NS ", "
107+
STRDEDUP_BYTES_FORMAT_NS " (total deduped)," STRDEDUP_BYTES_FORMAT_NS " (total new unknown), "
106108
STRDEDUP_ELAPSED_FORMAT_MS " of " STRDEDUP_ELAPSED_FORMAT_MS,
109+
last_stat->_inspected,
107110
last_stat->_new, STRDEDUP_BYTES_PARAM(last_stat->_new_bytes),
108111
last_stat->_deduped, STRDEDUP_BYTES_PARAM(last_stat->_deduped_bytes),
109112
total_deduped_bytes_percent,
113+
STRDEDUP_BYTES_PARAM(total_stat->_deduped_bytes), STRDEDUP_BYTES_PARAM(total_stat->_new_bytes),
110114
strdedup_elapsed_param_ms(last_stat->_process_elapsed),
111115
strdedup_elapsed_param_ms(last_stat->_active_elapsed));
112116
}
@@ -217,14 +221,14 @@ void StringDedup::Stat::log_statistics(bool total) const {
217221
double replaced_percent = percent_of(_replaced, _new);
218222
double deleted_percent = percent_of(_deleted, _new);
219223
log_times(total ? "Total" : "Last");
220-
log_debug(stringdedup)(" Inspected: %12zu", _inspected);
221-
log_debug(stringdedup)(" Known: %12zu(%5.1f%%)", _known, known_percent);
222-
log_debug(stringdedup)(" Shared: %12zu(%5.1f%%)", _known_shared, known_shared_percent);
223-
log_debug(stringdedup)(" New: %12zu(%5.1f%%)" STRDEDUP_BYTES_FORMAT,
224+
log_debug(stringdedup)(" Inspected: %12zu", _inspected);
225+
log_debug(stringdedup)(" Known: %12zu(%5.1f%%)", _known, known_percent);
226+
log_debug(stringdedup)(" Shared: %12zu(%5.1f%%)", _known_shared, known_shared_percent);
227+
log_debug(stringdedup)(" New unknown: %12zu(%5.1f%%)" STRDEDUP_BYTES_FORMAT,
224228
_new, new_percent, STRDEDUP_BYTES_PARAM(_new_bytes));
225-
log_debug(stringdedup)(" Replaced: %12zu(%5.1f%%)", _replaced, replaced_percent);
226-
log_debug(stringdedup)(" Deleted: %12zu(%5.1f%%)", _deleted, deleted_percent);
227-
log_debug(stringdedup)(" Deduplicated: %12zu(%5.1f%%)" STRDEDUP_BYTES_FORMAT "(%5.1f%%)",
229+
log_debug(stringdedup)(" Replaced: %12zu(%5.1f%%)", _replaced, replaced_percent);
230+
log_debug(stringdedup)(" Deleted: %12zu(%5.1f%%)", _deleted, deleted_percent);
231+
log_debug(stringdedup)(" Deduplicated: %12zu(%5.1f%%)" STRDEDUP_BYTES_FORMAT "(%5.1f%%)",
228232
_deduped, deduped_percent, STRDEDUP_BYTES_PARAM(_deduped_bytes), deduped_bytes_percent);
229233
log_debug(stringdedup)(" Skipped: %zu (dead), %zu (incomplete), %zu (shared)",
230234
_skipped_dead, _skipped_incomplete, _skipped_shared);

0 commit comments

Comments
 (0)