Revert "perf record: Intercept all events"
authorHitoshi Mitake <mitake@dcl.info.waseda.ac.jp>
Sat, 30 Jan 2010 11:55:41 +0000 (20:55 +0900)
committerIngo Molnar <mingo@elte.hu>
Sun, 31 Jan 2010 07:27:52 +0000 (08:27 +0100)
This reverts commit f5a2c3dce03621b55f84496f58adc2d1a87ca16f.

This patch is required for making "perf lock rec" work.
The commit f5a2c3dce0 changes write_event() of builtin-record.c
. And changed write_event() sometimes doesn't stop with perf
lock rec.

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>
[ that commit also causes perf record to not be Ctrl-C-able,
  and it's concetually wrong to parse the data at record time
  (unconditionally - even when not needed), as we eventually
  want to be able to do zero-copy recording, at least for
  non-archive recordings.  ]
Signed-off-by: Ingo Molnar <mingo@elte.hu>
tools/perf/builtin-record.c

index 90345223908ca074984bae6714355533de34fda2..eea56910b91c3a203b1eb2584325c4d1496a8f7a 100644 (file)
@@ -113,24 +113,16 @@ static void write_output(void *buf, size_t size)
 
 static void write_event(event_t *buf, size_t size)
 {
-       size_t processed_size = buf->header.size;
-       event_t *ev = buf;
-
-       do {
-               /*
-               * Add it to the list of DSOs, so that when we finish this
-                * record session we can pick the available build-ids.
-                */
-               if (ev->header.type == PERF_RECORD_MMAP) {
-                       struct list_head *head = &dsos__user;
-                       if (ev->header.misc == 1)
-                               head = &dsos__kernel;
-                       __dsos__findnew(head, ev->mmap.filename);
-               }
-
-               ev = ((void *)ev) + ev->header.size;
-               processed_size += ev->header.size;
-       } while (processed_size < size);
+       /*
+       * Add it to the list of DSOs, so that when we finish this
+        * record session we can pick the available build-ids.
+        */
+       if (buf->header.type == PERF_RECORD_MMAP) {
+               struct list_head *head = &dsos__user;
+               if (buf->mmap.header.misc == 1)
+                       head = &dsos__kernel;
+               __dsos__findnew(head, buf->mmap.filename);
+       }
 
        write_output(buf, size);
 }