提交 9af303e2 编写于 作者: J Jiri Olsa 提交者: Arnaldo Carvalho de Melo

perf diff: Switching the base hists to be pairs head

Making the baseline hists to act as a pairs head.

So far we don't care which hists act as a pairs head, because we have
only 2 files to deal with and any of them is suitable to do the job.

But if we want to process more files, we need to pick up one hists to
act as pairs head, and the baseline hists is the most suitable.
Signed-off-by: NJiri Olsa <jolsa@redhat.com>
Reviewed-by: NNamhyung Kim <namhyung@kernel.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
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/n/tip-cklmt2o4j87i9viz900245ae@git.kernel.orgSigned-off-by: NArnaldo Carvalho de Melo <acme@redhat.com>
上级 ec308426
...@@ -167,34 +167,34 @@ double perf_diff__period_percent(struct hist_entry *he, u64 period) ...@@ -167,34 +167,34 @@ double perf_diff__period_percent(struct hist_entry *he, u64 period)
double perf_diff__compute_delta(struct hist_entry *he, struct hist_entry *pair) double perf_diff__compute_delta(struct hist_entry *he, struct hist_entry *pair)
{ {
double new_percent = perf_diff__period_percent(he, he->stat.period); double old_percent = perf_diff__period_percent(he, he->stat.period);
double old_percent = perf_diff__period_percent(pair, pair->stat.period); double new_percent = perf_diff__period_percent(pair, pair->stat.period);
he->diff.period_ratio_delta = new_percent - old_percent; pair->diff.period_ratio_delta = new_percent - old_percent;
he->diff.computed = true; pair->diff.computed = true;
return he->diff.period_ratio_delta; return pair->diff.period_ratio_delta;
} }
double perf_diff__compute_ratio(struct hist_entry *he, struct hist_entry *pair) double perf_diff__compute_ratio(struct hist_entry *he, struct hist_entry *pair)
{ {
double new_period = he->stat.period; double old_period = he->stat.period ?: 1;
double old_period = pair->stat.period; double new_period = pair->stat.period;
he->diff.computed = true; pair->diff.computed = true;
he->diff.period_ratio = new_period / old_period; pair->diff.period_ratio = new_period / old_period;
return he->diff.period_ratio; return pair->diff.period_ratio;
} }
s64 perf_diff__compute_wdiff(struct hist_entry *he, struct hist_entry *pair) s64 perf_diff__compute_wdiff(struct hist_entry *he, struct hist_entry *pair)
{ {
u64 new_period = he->stat.period; u64 old_period = he->stat.period;
u64 old_period = pair->stat.period; u64 new_period = pair->stat.period;
he->diff.computed = true; pair->diff.computed = true;
he->diff.wdiff = new_period * compute_wdiff_w2 - pair->diff.wdiff = new_period * compute_wdiff_w2 -
old_period * compute_wdiff_w1; old_period * compute_wdiff_w1;
return he->diff.wdiff; return pair->diff.wdiff;
} }
static int formula_delta(struct hist_entry *he, struct hist_entry *pair, static int formula_delta(struct hist_entry *he, struct hist_entry *pair,
...@@ -203,15 +203,15 @@ static int formula_delta(struct hist_entry *he, struct hist_entry *pair, ...@@ -203,15 +203,15 @@ static int formula_delta(struct hist_entry *he, struct hist_entry *pair,
return scnprintf(buf, size, return scnprintf(buf, size,
"(%" PRIu64 " * 100 / %" PRIu64 ") - " "(%" PRIu64 " * 100 / %" PRIu64 ") - "
"(%" PRIu64 " * 100 / %" PRIu64 ")", "(%" PRIu64 " * 100 / %" PRIu64 ")",
he->stat.period, he->hists->stats.total_period, pair->stat.period, pair->hists->stats.total_period,
pair->stat.period, pair->hists->stats.total_period); he->stat.period, he->hists->stats.total_period);
} }
static int formula_ratio(struct hist_entry *he, struct hist_entry *pair, static int formula_ratio(struct hist_entry *he, struct hist_entry *pair,
char *buf, size_t size) char *buf, size_t size)
{ {
double new_period = he->stat.period; double old_period = he->stat.period;
double old_period = pair->stat.period; double new_period = pair->stat.period;
return scnprintf(buf, size, "%.0F / %.0F", new_period, old_period); return scnprintf(buf, size, "%.0F / %.0F", new_period, old_period);
} }
...@@ -219,8 +219,8 @@ static int formula_ratio(struct hist_entry *he, struct hist_entry *pair, ...@@ -219,8 +219,8 @@ static int formula_ratio(struct hist_entry *he, struct hist_entry *pair,
static int formula_wdiff(struct hist_entry *he, struct hist_entry *pair, static int formula_wdiff(struct hist_entry *he, struct hist_entry *pair,
char *buf, size_t size) char *buf, size_t size)
{ {
u64 new_period = he->stat.period; u64 old_period = he->stat.period;
u64 old_period = pair->stat.period; u64 new_period = pair->stat.period;
return scnprintf(buf, size, return scnprintf(buf, size,
"(%" PRIu64 " * " "%" PRId64 ") - (%" PRIu64 " * " "%" PRId64 ")", "(%" PRIu64 " * " "%" PRId64 ") - (%" PRIu64 " * " "%" PRId64 ")",
...@@ -462,23 +462,23 @@ static void hists__compute_resort(struct hists *hists) ...@@ -462,23 +462,23 @@ static void hists__compute_resort(struct hists *hists)
} }
} }
static void hists__process(struct hists *old, struct hists *new) static void hists__process(struct hists *base, struct hists *new)
{ {
hists__match(new, old); hists__match(base, new);
if (show_baseline_only) if (show_baseline_only)
hists__baseline_only(new); hists__baseline_only(base);
else else
hists__link(new, old); hists__link(base, new);
if (sort_compute) { if (sort_compute) {
hists__precompute(new); hists__precompute(base);
hists__compute_resort(new); hists__compute_resort(base);
} else { } else {
hists__output_resort(new); hists__output_resort(base);
} }
hists__fprintf(new, true, 0, 0, 0, stdout); hists__fprintf(base, true, 0, 0, 0, stdout);
} }
static void data_process(void) static void data_process(void)
......
...@@ -178,18 +178,8 @@ static int hpp__width_baseline(struct perf_hpp_fmt *fmt __maybe_unused, ...@@ -178,18 +178,8 @@ static int hpp__width_baseline(struct perf_hpp_fmt *fmt __maybe_unused,
static double baseline_percent(struct hist_entry *he) static double baseline_percent(struct hist_entry *he)
{ {
struct hist_entry *pair = hist_entry__next_pair(he); struct hists *hists = he->hists;
struct hists *pair_hists = pair ? pair->hists : NULL; return 100.0 * he->stat.period / hists->stats.total_period;
double percent = 0.0;
if (pair) {
u64 total_period = pair_hists->stats.total_period;
u64 base_period = pair->stat.period;
percent = 100.0 * base_period / total_period;
}
return percent;
} }
static int hpp__color_baseline(struct perf_hpp_fmt *fmt __maybe_unused, static int hpp__color_baseline(struct perf_hpp_fmt *fmt __maybe_unused,
...@@ -197,10 +187,8 @@ static int hpp__color_baseline(struct perf_hpp_fmt *fmt __maybe_unused, ...@@ -197,10 +187,8 @@ static int hpp__color_baseline(struct perf_hpp_fmt *fmt __maybe_unused,
{ {
double percent = baseline_percent(he); double percent = baseline_percent(he);
if (hist_entry__has_pairs(he) || symbol_conf.field_sep) return percent_color_snprintf(hpp->buf, hpp->size, " %6.2f%%",
return percent_color_snprintf(hpp->buf, hpp->size, " %6.2f%%", percent); percent);
else
return scnprintf(hpp->buf, hpp->size, " ");
} }
static int hpp__entry_baseline(struct perf_hpp_fmt *_fmt __maybe_unused, static int hpp__entry_baseline(struct perf_hpp_fmt *_fmt __maybe_unused,
...@@ -209,10 +197,7 @@ static int hpp__entry_baseline(struct perf_hpp_fmt *_fmt __maybe_unused, ...@@ -209,10 +197,7 @@ static int hpp__entry_baseline(struct perf_hpp_fmt *_fmt __maybe_unused,
double percent = baseline_percent(he); double percent = baseline_percent(he);
const char *fmt = symbol_conf.field_sep ? "%.2f" : " %6.2f%%"; const char *fmt = symbol_conf.field_sep ? "%.2f" : " %6.2f%%";
if (hist_entry__has_pairs(he) || symbol_conf.field_sep) return scnprintf(hpp->buf, hpp->size, fmt, percent);
return scnprintf(hpp->buf, hpp->size, fmt, percent);
else
return scnprintf(hpp->buf, hpp->size, " ");
} }
static int hpp__header_period_baseline(struct perf_hpp_fmt *_fmt __maybe_unused, static int hpp__header_period_baseline(struct perf_hpp_fmt *_fmt __maybe_unused,
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册