diff options
author | Jiri Olsa <jolsa@redhat.com> | 2012-10-05 10:44:44 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2012-10-05 13:11:47 -0400 |
commit | 61949b212e7f6f8f31891236ba24033f9b7af8c3 (patch) | |
tree | d31f37b0360e2b9bef0ccde43b1f0139ef0232e8 /tools/perf/ui/hist.c | |
parent | 81d5f95819953321a2557b0656b24ea10af9629c (diff) |
perf diff: Add -p option to display period values for hist entries
Adding -p option to show period values for both compared hist entries.
Showing hist column PERF_HPP__PERIOD and newly added hist column
PERF_HPP__PERIOD_BASELINE.
Signed-off-by: Jiri Olsa <jolsa@redhat.com>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1349448287-18919-6-git-send-email-jolsa@redhat.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf/ui/hist.c')
-rw-r--r-- | tools/perf/ui/hist.c | 21 |
1 files changed, 21 insertions, 0 deletions
diff --git a/tools/perf/ui/hist.c b/tools/perf/ui/hist.c index 522b4ec051d5..2fadaff6312f 100644 --- a/tools/perf/ui/hist.c +++ b/tools/perf/ui/hist.c | |||
@@ -228,6 +228,26 @@ static int hpp__entry_period(struct perf_hpp *hpp, struct hist_entry *he) | |||
228 | return scnprintf(hpp->buf, hpp->size, fmt, he->stat.period); | 228 | return scnprintf(hpp->buf, hpp->size, fmt, he->stat.period); |
229 | } | 229 | } |
230 | 230 | ||
231 | static int hpp__header_period_baseline(struct perf_hpp *hpp) | ||
232 | { | ||
233 | const char *fmt = symbol_conf.field_sep ? "%s" : "%12s"; | ||
234 | |||
235 | return scnprintf(hpp->buf, hpp->size, fmt, "Period Base"); | ||
236 | } | ||
237 | |||
238 | static int hpp__width_period_baseline(struct perf_hpp *hpp __maybe_unused) | ||
239 | { | ||
240 | return 12; | ||
241 | } | ||
242 | |||
243 | static int hpp__entry_period_baseline(struct perf_hpp *hpp, struct hist_entry *he) | ||
244 | { | ||
245 | struct hist_entry *pair = he->pair; | ||
246 | u64 period = pair ? pair->stat.period : 0; | ||
247 | const char *fmt = symbol_conf.field_sep ? "%" PRIu64 : "%12" PRIu64; | ||
248 | |||
249 | return scnprintf(hpp->buf, hpp->size, fmt, period); | ||
250 | } | ||
231 | static int hpp__header_delta(struct perf_hpp *hpp) | 251 | static int hpp__header_delta(struct perf_hpp *hpp) |
232 | { | 252 | { |
233 | const char *fmt = symbol_conf.field_sep ? "%s" : "%7s"; | 253 | const char *fmt = symbol_conf.field_sep ? "%s" : "%7s"; |
@@ -359,6 +379,7 @@ struct perf_hpp_fmt perf_hpp__format[] = { | |||
359 | { .cond = false, HPP__COLOR_PRINT_FNS(overhead_guest_us) }, | 379 | { .cond = false, HPP__COLOR_PRINT_FNS(overhead_guest_us) }, |
360 | { .cond = false, HPP__PRINT_FNS(samples) }, | 380 | { .cond = false, HPP__PRINT_FNS(samples) }, |
361 | { .cond = false, HPP__PRINT_FNS(period) }, | 381 | { .cond = false, HPP__PRINT_FNS(period) }, |
382 | { .cond = false, HPP__PRINT_FNS(period_baseline) }, | ||
362 | { .cond = false, HPP__PRINT_FNS(delta) }, | 383 | { .cond = false, HPP__PRINT_FNS(delta) }, |
363 | { .cond = false, HPP__PRINT_FNS(ratio) }, | 384 | { .cond = false, HPP__PRINT_FNS(ratio) }, |
364 | { .cond = false, HPP__PRINT_FNS(wdiff) }, | 385 | { .cond = false, HPP__PRINT_FNS(wdiff) }, |