Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1034225AbcJ1Rq1 (ORCPT ); Fri, 28 Oct 2016 13:46:27 -0400 Received: from terminus.zytor.com ([198.137.202.10]:32952 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756347AbcJ1RqZ (ORCPT ); Fri, 28 Oct 2016 13:46:25 -0400 Date: Fri, 28 Oct 2016 10:46:11 -0700 From: tip-bot for Namhyung Kim Message-ID: Cc: linux-kernel@vger.kernel.org, jolsa@kernel.org, acme@redhat.com, mingo@kernel.org, tglx@linutronix.de, namhyung@kernel.org, a.p.zijlstra@chello.nl, dsahern@gmail.com, hpa@zytor.com Reply-To: acme@redhat.com, jolsa@kernel.org, linux-kernel@vger.kernel.org, namhyung@kernel.org, a.p.zijlstra@chello.nl, dsahern@gmail.com, hpa@zytor.com, mingo@kernel.org, tglx@linutronix.de In-Reply-To: <20161024020246.14928-2-namhyung@kernel.org> References: <20161024020246.14928-2-namhyung@kernel.org> To: linux-tip-commits@vger.kernel.org Subject: [tip:perf/core] perf sched map: Always show task comm with -v Git-Commit-ID: e107f129e2e0e75ddf1cd7995a9f5ffff2307766 X-Mailer: tip-git-log-daemon Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Content-Type: text/plain; charset=UTF-8 Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1622 Lines: 38 Commit-ID: e107f129e2e0e75ddf1cd7995a9f5ffff2307766 Gitweb: http://git.kernel.org/tip/e107f129e2e0e75ddf1cd7995a9f5ffff2307766 Author: Namhyung Kim AuthorDate: Mon, 24 Oct 2016 11:02:44 +0900 Committer: Arnaldo Carvalho de Melo CommitDate: Tue, 25 Oct 2016 15:05:09 -0300 perf sched map: Always show task comm with -v 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 Acked-by: Jiri Olsa Cc: David Ahern Cc: Peter Zijlstra Link: http://lkml.kernel.org/r/20161024020246.14928-2-namhyung@kernel.org Signed-off-by: Arnaldo Carvalho de Melo --- tools/perf/builtin-sched.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c index a8ad859..1f33d15 100644 --- a/tools/perf/builtin-sched.c +++ b/tools/perf/builtin-sched.c @@ -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))