diff options
author | Ingo Molnar <mingo@elte.hu> | 2009-06-04 07:41:22 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2009-06-04 07:41:37 -0400 |
commit | df97992c6e25ffc66c549c8bc59262dc627c6d17 (patch) | |
tree | 5d666762c78320adeb2e15fe362c3b2909e0b66b /Documentation/perf_counter/builtin-report.c | |
parent | 3aff27ca84fa94311ae99189e54fed8d83b69fc1 (diff) |
perf record/report: Fix PID/COMM handling
Fix two bugs causing lost comm mappings:
- initial PID is not 0 but getpid()
- when we are unable to handle an mmap event, dont assume the event
itself is broken - try to parse the stream. This way we wont lose
comm events.
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Mike Galbraith <efault@gmx.de>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Marcelo Tosatti <mtosatti@redhat.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'Documentation/perf_counter/builtin-report.c')
-rw-r--r-- | Documentation/perf_counter/builtin-report.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Documentation/perf_counter/builtin-report.c b/Documentation/perf_counter/builtin-report.c index 86f23f0991f1..ff6f657476a9 100644 --- a/Documentation/perf_counter/builtin-report.c +++ b/Documentation/perf_counter/builtin-report.c | |||
@@ -852,7 +852,7 @@ process_mmap_event(event_t *event, unsigned long offset, unsigned long head) | |||
852 | 852 | ||
853 | if (thread == NULL || map == NULL) { | 853 | if (thread == NULL || map == NULL) { |
854 | dprintf("problem processing PERF_EVENT_MMAP, skipping event.\n"); | 854 | dprintf("problem processing PERF_EVENT_MMAP, skipping event.\n"); |
855 | return -1; | 855 | return 0; |
856 | } | 856 | } |
857 | 857 | ||
858 | thread__insert_map(thread, map); | 858 | thread__insert_map(thread, map); |