Skip to content

Commit

Permalink
perf script: Allow extended console debug output
Browse files Browse the repository at this point in the history
The script tool isn't using a browser, yet use_browser wasn't set
explicitly to zero. This in turn lead to confusing output such as:

  ```
  $ perf script -vvv ...
  ...
  overlapping maps in /home/milian/foobar (disable tui for more info)
  ...
  ```

Explicitly set use_browser to 0 now, which gives us the extended
debug information now in perf script as expected.

Signed-off-by: Milian Wolff <milian.wolff@kdab.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Link: http://lkml.kernel.org/r/20181021191424.16183-1-milian.wolff@kdab.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  • Loading branch information
milianw authored and acmel committed Oct 22, 2018
1 parent cbb5df7 commit c1c9b96
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion tools/perf/builtin-script.c
Original file line number Diff line number Diff line change
Expand Up @@ -3417,8 +3417,10 @@ int cmd_script(int argc, const char **argv)
exit(-1);
}

if (!script_name)
if (!script_name) {
setup_pager();
use_browser = 0;
}

session = perf_session__new(&data, false, &script.tool);
if (session == NULL)
Expand Down

0 comments on commit c1c9b96

Please sign in to comment.