perf tools: Fix thread comm resolution in perf sched
authorFrederic Weisbecker <fweisbec@gmail.com>
Thu, 8 Oct 2009 19:04:17 +0000 (21:04 +0200)
committerIngo Molnar <mingo@elte.hu>
Thu, 8 Oct 2009 19:10:21 +0000 (21:10 +0200)
commit97ea1a7fa62af0d8d49a0fc12796b0073537c9d8
tree3be620dc044bd978e71b1bdd997d8e38a3c9b2a8
parent2e538c4a1847291cf01218d4fe7bb4dc60fef7cf
perf tools: Fix thread comm resolution in perf sched

This reverts commit 9a92b479b2f088ee2d3194243f4c8e59b1b8c9c2 ("perf
tools: Improve thread comm resolution in perf sched") and fixes the
real bug.

The bug was elsewhere:

We are failing to resolve thread names in perf sched because the
table of threads we are building, on top of comm events, has a per
process granularity. But perf sched, unlike the other perf tools,
needs a per thread granularity as we are profiling every tasks
individually.

So fix it by building our threads table using the tid instead of
the pid as the thread identifier.

v2: Revert the previous fix - it is not really needed

Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
LKML-Reference: <1255028657-11158-1-git-send-email-fweisbec@gmail.com>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
tools/perf/builtin-sched.c
tools/perf/util/thread.c
tools/perf/util/thread.h