perf hists browser: Fix children overhead dump
authorNamhyung Kim <namhyung@kernel.org>
Wed, 20 Aug 2014 08:07:56 +0000 (17:07 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 20 Aug 2014 15:14:14 +0000 (12:14 -0300)
When perf report runs on TUI, 'P' key dumps current screen to a file but
it incorrectly displayed children overhead (as same of self overhead).
This was because it fetched the value from self stats.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung.kim@lge.com>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1408522080-26556-2-git-send-email-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/ui/hist.c

index b5fa7019d2e2966e16bb1a4b856ae3c30a4a89b3..75eb6ac821f82dc917ccee5d29126b0b6bda7d5b 100644 (file)
@@ -304,7 +304,7 @@ static int hpp__color_##_type(struct perf_hpp_fmt *fmt,                             \
 static int hpp__entry_##_type(struct perf_hpp_fmt *fmt,                                \
                              struct perf_hpp *hpp, struct hist_entry *he)      \
 {                                                                              \
-       return hpp__fmt_acc(fmt, hpp, he, he_get_##_field, " %*.2f%%",          \
+       return hpp__fmt_acc(fmt, hpp, he, he_get_acc_##_field, " %*.2f%%",      \
                            hpp_entry_scnprintf, true);                         \
 }