Messages in this thread Patch in this message | | | Subject | [RFC][PATCH] perf_events: Fix FORK events | From | Peter Zijlstra <> | Date | Fri, 05 Feb 2010 16:37:23 +0100 |
| |
Hi,
While looking into a problem reported by Pekka, I noticed that I wasn't receiving any FORK events for a workload that did fork (see attachment).
After making the below change, stuff started working again, thing is, I'm not sure why.
The main change is sending the FORK event to the parent instead of the child thread, however perf_event_fork() is at the end of copy_process(), which is after perf_event_init_task() which inherits all the counters, so it should all have worked as it was.
We could of course just slam the commit in and not worry about it, but that just doesn't feel right.
Alternatively, I'm really really dense and I'm totally missing the obvious.
Signed-ff-by: Peter Zijlstra <a.p.zijlstra@chello.nl> --- kernel/perf_event.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-)
diff --git a/kernel/perf_event.c b/kernel/perf_event.c index ab8a312..01655fa 100644 --- a/kernel/perf_event.c +++ b/kernel/perf_event.c @@ -3399,8 +3399,6 @@ static void perf_event_task_output(struct perf_event *event, task_event->event_id.tid = perf_event_tid(event, task); task_event->event_id.ptid = perf_event_tid(event, current); - task_event->event_id.time = perf_clock(); - perf_output_put(&handle, task_event->event_id); perf_output_end(&handle); @@ -3440,7 +3438,7 @@ static void perf_event_task_event(struct perf_task_event *task_event) cpuctx = &get_cpu_var(perf_cpu_context); perf_event_task_ctx(&cpuctx->ctx, task_event); if (!ctx) - ctx = rcu_dereference(task_event->task->perf_event_ctxp); + ctx = rcu_dereference(current->perf_event_ctxp); if (ctx) perf_event_task_ctx(ctx, task_event); put_cpu_var(perf_cpu_context); @@ -3471,6 +3469,7 @@ static void perf_event_task(struct task_struct *task, /* .ppid */ /* .tid */ /* .ptid */ + .time = perf_clock(), }, };
#include <stdlib.h> #include <stdio.h> #include <time.h>
main () { int i;
fork(); fork();
for (i = 0; i < 100000000; i++) { asm("nop"); asm("nop"); asm("nop"); asm("nop"); asm("nop"); asm("nop"); asm("nop"); } wait(NULL); wait(NULL); wait(NULL); wait(NULL); }
| |