Commit 5eca104e authored by Namhyung Kim's avatar Namhyung Kim Committed by Arnaldo Carvalho de Melo

perf hists browser: Pass parent_total to callchain print functions

Pass parent node's total period to callchain print functions.  This info
is needed by later patch to determine whether it can omit percent or not
correctly.

No functional change intended.
Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/r/1453909257-26015-9-git-send-email-namhyung@kernel.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 0c841c6c
...@@ -660,6 +660,7 @@ static int hist_browser__show_callchain_list(struct hist_browser *browser, ...@@ -660,6 +660,7 @@ static int hist_browser__show_callchain_list(struct hist_browser *browser,
static int hist_browser__show_callchain_flat(struct hist_browser *browser, static int hist_browser__show_callchain_flat(struct hist_browser *browser,
struct rb_root *root, struct rb_root *root,
unsigned short row, u64 total, unsigned short row, u64 total,
u64 parent_total __maybe_unused,
print_callchain_entry_fn print, print_callchain_entry_fn print,
struct callchain_print_arg *arg, struct callchain_print_arg *arg,
check_output_full_fn is_output_full) check_output_full_fn is_output_full)
...@@ -763,6 +764,7 @@ static char *hist_browser__folded_callchain_str(struct hist_browser *browser, ...@@ -763,6 +764,7 @@ static char *hist_browser__folded_callchain_str(struct hist_browser *browser,
static int hist_browser__show_callchain_folded(struct hist_browser *browser, static int hist_browser__show_callchain_folded(struct hist_browser *browser,
struct rb_root *root, struct rb_root *root,
unsigned short row, u64 total, unsigned short row, u64 total,
u64 parent_total __maybe_unused,
print_callchain_entry_fn print, print_callchain_entry_fn print,
struct callchain_print_arg *arg, struct callchain_print_arg *arg,
check_output_full_fn is_output_full) check_output_full_fn is_output_full)
...@@ -847,14 +849,18 @@ static int hist_browser__show_callchain_folded(struct hist_browser *browser, ...@@ -847,14 +849,18 @@ static int hist_browser__show_callchain_folded(struct hist_browser *browser,
static int hist_browser__show_callchain_graph(struct hist_browser *browser, static int hist_browser__show_callchain_graph(struct hist_browser *browser,
struct rb_root *root, int level, struct rb_root *root, int level,
unsigned short row, u64 total, unsigned short row, u64 total,
u64 parent_total,
print_callchain_entry_fn print, print_callchain_entry_fn print,
struct callchain_print_arg *arg, struct callchain_print_arg *arg,
check_output_full_fn is_output_full) check_output_full_fn is_output_full)
{ {
struct rb_node *node; struct rb_node *node;
int first_row = row, offset = level * LEVEL_OFFSET_STEP; int first_row = row, offset = level * LEVEL_OFFSET_STEP;
u64 new_total;
bool need_percent; bool need_percent;
u64 percent_total = total;
if (callchain_param.mode == CHAIN_GRAPH_REL)
percent_total = parent_total;
node = rb_first(root); node = rb_first(root);
need_percent = node && rb_next(node); need_percent = node && rb_next(node);
...@@ -878,7 +884,7 @@ static int hist_browser__show_callchain_graph(struct hist_browser *browser, ...@@ -878,7 +884,7 @@ static int hist_browser__show_callchain_graph(struct hist_browser *browser,
folded_sign = callchain_list__folded(chain); folded_sign = callchain_list__folded(chain);
row += hist_browser__show_callchain_list(browser, child, row += hist_browser__show_callchain_list(browser, child,
chain, row, total, chain, row, percent_total,
was_first && need_percent, was_first && need_percent,
offset + extra_offset, offset + extra_offset,
print, arg); print, arg);
...@@ -893,13 +899,9 @@ static int hist_browser__show_callchain_graph(struct hist_browser *browser, ...@@ -893,13 +899,9 @@ static int hist_browser__show_callchain_graph(struct hist_browser *browser,
if (folded_sign == '-') { if (folded_sign == '-') {
const int new_level = level + (extra_offset ? 2 : 1); const int new_level = level + (extra_offset ? 2 : 1);
if (callchain_param.mode == CHAIN_GRAPH_REL)
new_total = child->children_hit;
else
new_total = total;
row += hist_browser__show_callchain_graph(browser, &child->rb_root, row += hist_browser__show_callchain_graph(browser, &child->rb_root,
new_level, row, new_total, new_level, row, total,
child->children_hit,
print, arg, is_output_full); print, arg, is_output_full);
} }
if (is_output_full(browser, row)) if (is_output_full(browser, row))
...@@ -918,27 +920,29 @@ static int hist_browser__show_callchain(struct hist_browser *browser, ...@@ -918,27 +920,29 @@ static int hist_browser__show_callchain(struct hist_browser *browser,
check_output_full_fn is_output_full) check_output_full_fn is_output_full)
{ {
u64 total = hists__total_period(entry->hists); u64 total = hists__total_period(entry->hists);
u64 parent_total;
int printed; int printed;
if (callchain_param.mode == CHAIN_GRAPH_REL) {
if (symbol_conf.cumulate_callchain) if (symbol_conf.cumulate_callchain)
total = entry->stat_acc->period; parent_total = entry->stat_acc->period;
else else
total = entry->stat.period; parent_total = entry->stat.period;
}
if (callchain_param.mode == CHAIN_FLAT) { if (callchain_param.mode == CHAIN_FLAT) {
printed = hist_browser__show_callchain_flat(browser, printed = hist_browser__show_callchain_flat(browser,
&entry->sorted_chain, row, total, &entry->sorted_chain, row,
print, arg, is_output_full); total, parent_total, print, arg,
is_output_full);
} else if (callchain_param.mode == CHAIN_FOLDED) { } else if (callchain_param.mode == CHAIN_FOLDED) {
printed = hist_browser__show_callchain_folded(browser, printed = hist_browser__show_callchain_folded(browser,
&entry->sorted_chain, row, total, &entry->sorted_chain, row,
print, arg, is_output_full); total, parent_total, print, arg,
is_output_full);
} else { } else {
printed = hist_browser__show_callchain_graph(browser, printed = hist_browser__show_callchain_graph(browser,
&entry->sorted_chain, level, row, total, &entry->sorted_chain, level, row,
print, arg, is_output_full); total, parent_total, print, arg,
is_output_full);
} }
if (arg->is_current_entry) if (arg->is_current_entry)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment