diff options
author | Ingo Molnar <mingo@elte.hu> | 2009-09-03 16:19:57 +0200 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-09-03 16:19:57 +0200 |
commit | 8886f42d6d8dabeb488c706c339634a0e3e08df4 (patch) | |
tree | 21774d1711bfe4f4e49c126f27ff31a5d059bd0c /tools/perf/builtin-trace.c | |
parent | perf trace: Sample timestamps as well (diff) | |
download | linux-8886f42d6d8dabeb488c706c339634a0e3e08df4.tar.xz linux-8886f42d6d8dabeb488c706c339634a0e3e08df4.zip |
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>
Diffstat (limited to 'tools/perf/builtin-trace.c')
-rw-r--r-- | 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 d59bf8a86743..914ab366e369 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)) |