diff options
| author |   Peter Zijlstra <a.p.zijlstra@chello.nl> | 2010-09-13 11:15:58 +0200 |
|---|---|---|
| committer |   Ingo Molnar <mingo@elte.hu> | 2010-09-13 17:08:42 +0200 |
| commit | 0c67b40872326a5340cab51d79a192a5fbaeb484 (patch) | |
| tree | e92b420dcb1375e159842d8ffa03a5230976ab3a | |
| parent | perf: Sanitize the RCU logic (diff) | |
| download | linux-dev-0c67b40872326a5340cab51d79a192a5fbaeb484.tar.xz linux-dev-0c67b40872326a5340cab51d79a192a5fbaeb484.zip | |
perf: Fix free_event()
With the context rework stuff we can actually end up freeing an event
before it gets attached to a context.
Reported-by: Cyrill Gorcunov <gorcunov@gmail.com>
Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
LKML-Reference: <new-submission>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
| -rw-r--r-- | kernel/perf_event.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/kernel/perf_event.c b/kernel/perf_event.c index bc46bff69620..440f9ca067b2 100644 --- a/kernel/perf_event.c +++ b/kernel/perf_event.c @@ -2150,7 +2150,9 @@ static void free_event(struct perf_event *event) if (event->destroy) event->destroy(event); - put_ctx(event->ctx); + if (event->ctx) + put_ctx(event->ctx); + call_rcu(&event->rcu_head, free_event_rcu); } |