Messages in this thread Patch in this message | | | Date | Thu, 14 May 2015 23:45:03 -0700 | From | tip-bot for Namhyung Kim <> | Subject | [tip:perf/core] perf report: Do not restrict -T option by other options |
| |
Commit-ID: 021162cf02fcfa80cbae5f3b7304e9cb392962eb Gitweb: http://git.kernel.org/tip/021162cf02fcfa80cbae5f3b7304e9cb392962eb Author: Namhyung Kim <namhyung@kernel.org> AuthorDate: Mon, 11 May 2015 22:44:39 +0900 Committer: Arnaldo Carvalho de Melo <acme@redhat.com> CommitDate: Tue, 12 May 2015 09:59:51 -0300
perf report: Do not restrict -T option by other options
It seems there's no reason to suppress per-thread event stat by -T option when -s or -p option is used. Make it work with those options.
Signed-off-by: Namhyung Kim <namhyung@kernel.org> Cc: David Ahern <dsahern@gmail.com> Cc: Jiri Olsa <jolsa@redhat.com> Cc: Peter Zijlstra <a.p.zijlstra@chello.nl> Link: http://lkml.kernel.org/r/1431351879-23798-1-git-send-email-namhyung@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com> --- tools/perf/builtin-report.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-)
diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 4d642db..92fca21 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -333,15 +333,14 @@ static int perf_evlist__tty_browse_hists(struct perf_evlist *evlist, } if (sort_order == NULL && - parent_pattern == default_parent_pattern) { + parent_pattern == default_parent_pattern) fprintf(stdout, "#\n# (%s)\n#\n", help); - if (rep->show_threads) { - bool style = !strcmp(rep->pretty_printing_style, "raw"); - perf_read_values_display(stdout, &rep->show_threads_values, - style); - perf_read_values_destroy(&rep->show_threads_values); - } + if (rep->show_threads) { + bool style = !strcmp(rep->pretty_printing_style, "raw"); + perf_read_values_display(stdout, &rep->show_threads_values, + style); + perf_read_values_destroy(&rep->show_threads_values); } return 0;
| |