diff options
author | Jiri Olsa <jolsa@kernel.org> | 2016-09-22 11:36:29 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2016-09-22 12:08:56 -0400 |
commit | 9da44db1493a9d384ddc1bcd1553a1803ff985b6 (patch) | |
tree | 55767da5d73cd2edb78e7b17605d056880a64140 | |
parent | 3becf4525d9c0fb9cf8ff657b2aec1c733bc742f (diff) |
perf hists: Add __hist_entry__snprintf function
Add __hist_entry__snprintf() to take a perf_hpp_list as an argument
instead of using he->hists->hpp_list.
This way we can display arbitrary list of entries regardless of the
hists setup, which will be useful in the upcoming c2c patch series.
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Don Zickus <dzickus@redhat.com>
Cc: Joe Mario <jmario@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1474558645-19956-2-git-send-email-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/ui/stdio/hist.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/tools/perf/ui/stdio/hist.c b/tools/perf/ui/stdio/hist.c index c8dca34585ea..189665c315cc 100644 --- a/tools/perf/ui/stdio/hist.c +++ b/tools/perf/ui/stdio/hist.c | |||
@@ -373,7 +373,8 @@ static size_t hist_entry_callchain__fprintf(struct hist_entry *he, | |||
373 | return 0; | 373 | return 0; |
374 | } | 374 | } |
375 | 375 | ||
376 | static int hist_entry__snprintf(struct hist_entry *he, struct perf_hpp *hpp) | 376 | static int __hist_entry__snprintf(struct hist_entry *he, struct perf_hpp *hpp, |
377 | struct perf_hpp_list *hpp_list) | ||
377 | { | 378 | { |
378 | const char *sep = symbol_conf.field_sep; | 379 | const char *sep = symbol_conf.field_sep; |
379 | struct perf_hpp_fmt *fmt; | 380 | struct perf_hpp_fmt *fmt; |
@@ -384,7 +385,7 @@ static int hist_entry__snprintf(struct hist_entry *he, struct perf_hpp *hpp) | |||
384 | if (symbol_conf.exclude_other && !he->parent) | 385 | if (symbol_conf.exclude_other && !he->parent) |
385 | return 0; | 386 | return 0; |
386 | 387 | ||
387 | hists__for_each_format(he->hists, fmt) { | 388 | perf_hpp_list__for_each_format(hpp_list, fmt) { |
388 | if (perf_hpp__should_skip(fmt, he->hists)) | 389 | if (perf_hpp__should_skip(fmt, he->hists)) |
389 | continue; | 390 | continue; |
390 | 391 | ||
@@ -410,6 +411,11 @@ static int hist_entry__snprintf(struct hist_entry *he, struct perf_hpp *hpp) | |||
410 | return hpp->buf - start; | 411 | return hpp->buf - start; |
411 | } | 412 | } |
412 | 413 | ||
414 | static int hist_entry__snprintf(struct hist_entry *he, struct perf_hpp *hpp) | ||
415 | { | ||
416 | return __hist_entry__snprintf(he, hpp, he->hists->hpp_list); | ||
417 | } | ||
418 | |||
413 | static int hist_entry__hierarchy_fprintf(struct hist_entry *he, | 419 | static int hist_entry__hierarchy_fprintf(struct hist_entry *he, |
414 | struct perf_hpp *hpp, | 420 | struct perf_hpp *hpp, |
415 | struct hists *hists, | 421 | struct hists *hists, |