Messages in this thread Patch in this message | | | From | Taeung Song <> | Subject | [PATCH v3 4/9] perf annotate: Show the proper header when using --show-total-period | Date | Thu, 20 Jul 2017 06:36:59 +0900 |
| |
Currently a first column is only "Percent", so fix it to show correct column name based on given options. (e.g. if using --show-total-period, show "Event count" as a first column)
Reported-by: Milian Wolff <milian.wolff@kdab.com> Cc: Namhyung Kim <namhyung@kernel.org> Cc: Jiri Olsa <jolsa@redhat.com> Signed-off-by: Taeung Song <treeze.taeung@gmail.com> --- tools/perf/util/annotate.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/tools/perf/util/annotate.c b/tools/perf/util/annotate.c index d4f1a0a..1e1899d 100644 --- a/tools/perf/util/annotate.c +++ b/tools/perf/util/annotate.c @@ -1850,7 +1850,9 @@ int symbol__annotate_printf(struct symbol *sym, struct map *map, 4 * evsel->nr_members : 4; graph_dotted_len = printf(" %-*.*s| Source code & Disassembly of %s for %s (%" PRIu64 " %s)\n", - width, width, "Percent", d_filename, evsel_name, + width, width, + symbol_conf.show_total_period ? "Event count" : "Percent", + d_filename, evsel_name, symbol_conf.show_total_period ? h->total_period : h->total_samples, symbol_conf.show_total_period ? "event count" : "samples"); -- 2.7.4
| |