diff options
author | Namhyung Kim <namhyung.kim@lge.com> | 2012-09-13 00:14:30 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2012-09-17 12:11:33 -0400 |
commit | 60e5c706b3ea56f87afc2a4a3096118d28f9cc24 (patch) | |
tree | 4ae427cb17445001c99d600ebf7cd17c42fdbf2f /tools/perf | |
parent | e5f3705e62b03251797a5173024184bfc223599d (diff) |
perf report: Add missing perf_hpp__init for pipe-mode
The perf_hpp__init() function was only called from setup_browser() so
that the pipe-mode missed the initialization thus didn't respond to
related options. Fix it.
Reported-by: Robert Richter <robert.richter@amd.com>
Tested-by: Robert Richter <robert.richter@amd.com>
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Cc: H. Peter Anvin <hpa@zytor.com>
Cc: Ingo Molnar <mingo@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Robert Richter <robert.richter@amd.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-tip-commits@vger.kernel.org
Link: http://lkml.kernel.org/r/87txv28spl.fsf_-_@sejong.aot.lge.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools/perf')
-rw-r--r-- | tools/perf/builtin-report.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index b6696dd51cb0..1da243dfbc3e 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c | |||
@@ -689,8 +689,10 @@ int cmd_report(int argc, const char **argv, const char *prefix __maybe_unused) | |||
689 | 689 | ||
690 | if (strcmp(report.input_name, "-") != 0) | 690 | if (strcmp(report.input_name, "-") != 0) |
691 | setup_browser(true); | 691 | setup_browser(true); |
692 | else | 692 | else { |
693 | use_browser = 0; | 693 | use_browser = 0; |
694 | perf_hpp__init(false, false); | ||
695 | } | ||
694 | 696 | ||
695 | setup_sorting(report_usage, options); | 697 | setup_sorting(report_usage, options); |
696 | 698 | ||