[tip:perfcounters/core] perf trace: Fix parsing of perf.data

tip-bot for Ingo Molnar
2009-09-03T14:28:15Z

Commit-ID: 8886f42d6d8dabeb488c706c339634a0e3e08df4
Gitweb: http://git.kernel.org/tip/8886f42d6d8dabeb488c706c339634a0e3e08df4
Author: Ingo Molnar <mingo@elte.hu>
AuthorDate: Thu, 3 Sep 2009 16:19:57 +0200
Committer: Ingo Molnar <mingo@elte.hu>
CommitDate: Thu, 3 Sep 2009 16:19:57 +0200

perf trace: Fix parsing of perf.data

We started parsing perf.data at head 0. This caused -D to
segfault and it could possibly also case incorrect trace
entries to be displayed.

Parse it at data_offset instead.

Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>

---
tools/perf/builtin-trace.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c
index d59bf8a..914ab36 100644
--- a/tools/perf/builtin-trace.c
+++ b/tools/perf/builtin-trace.c
@@ -196,6 +196,7 @@ static int __cmd_trace(void)
exit(0);
}
header = perf_header__read(input);
+ head = header->data_offset;
sample_type = perf_header__sample_type(header);

if (!(sample_type & PERF_SAMPLE_RAW))