perf sched map: Always show task comm with -v
authorNamhyung Kim <namhyung@kernel.org>
Mon, 24 Oct 2016 02:02:44 +0000 (11:02 +0900)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Tue, 25 Oct 2016 18:05:09 +0000 (15:05 -0300)
I'd like to see the name of tasks with perf sched map, but it only shows
name of new tasks and then use short names after all.  This is not good
for long running tasks since it's hard for users to track the short
names.  This patch makes it show the names (except the idle task) when
-v option is used.  Probably we may make it as default behavior.

Signed-off-by: Namhyung Kim <namhyung@kernel.org>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/20161024020246.14928-2-namhyung@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-sched.c

index a8ad8591446628276881eeff0692286292991f6f..1f33d15314a556e2fa1f2956046667583733d8ed 100644 (file)
@@ -1493,7 +1493,7 @@ static int map_switch_event(struct perf_sched *sched, struct perf_evsel *evsel,
                goto out;
 
        color_fprintf(stdout, color, "  %12.6f secs ", (double)timestamp / NSEC_PER_SEC);
-       if (new_shortname) {
+       if (new_shortname || (verbose && sched_in->tid)) {
                const char *pid_color = color;
 
                if (thread__has_color(sched_in))