diff options
author | Namhyung Kim <namhyung@kernel.org> | 2014-04-16 04:04:51 +0200 |
---|---|---|
committer | Jiri Olsa <jolsa@kernel.org> | 2014-05-21 11:45:35 +0200 |
commit | 22af969e8cfc6ea46d3e1a774a16d7e19b8cf4db (patch) | |
tree | e8c2e0b30db28e500d31b5ef6ec55a0aaa26ae3e /tools/perf/builtin-report.c | |
parent | perf tools: Consolidate management of default sort orders (diff) | |
download | linux-22af969e8cfc6ea46d3e1a774a16d7e19b8cf4db.tar.xz linux-22af969e8cfc6ea46d3e1a774a16d7e19b8cf4db.zip |
perf tools: Call perf_hpp__init() before setting up GUI browsers
So that it can be set properly prior to set up output fields. That
makes easy to handle/warn errors during the setup since it doesn't
need to be bothered with the GUI.
Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1400480762-22852-11-git-send-email-namhyung@kernel.org
Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Diffstat (limited to 'tools/perf/builtin-report.c')
-rw-r--r-- | tools/perf/builtin-report.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index f4d640cfdf16..c4dab7acbdbb 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -823,16 +823,16 @@ repeat: goto error; } + perf_hpp__init(); + /* Force tty output for header output. */ if (report.header || report.header_only) use_browser = 0; if (strcmp(input_name, "-") != 0) setup_browser(true); - else { + else use_browser = 0; - perf_hpp__init(); - } if (report.header || report.header_only) { perf_session__fprintf_info(session, stdout, |