diff options
author | Yan, Zheng <zheng.z.yan@intel.com> | 2014-11-04 21:56:02 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@kernel.org> | 2015-02-18 11:16:07 -0500 |
commit | 5a158c3ccd2183a7b0866be6685d001fe653430f (patch) | |
tree | b270d5bca5169d27285abead59f8cbf74dda683a /kernel/events | |
parent | 4af57ef28c2c1047fda9e1a5be02aa7a6a69cf9e (diff) |
perf: Always switch pmu specific data during context switch
If two tasks were both forked from the same parent task, Events in
their perf task contexts can be the same. Perf core may leave out
switching the perf event contexts.
Previous patch inroduces pmu specific data. The data is for saving
the LBR stack, it is task specific. So we need to switch the data
even when context switch is optimized out.
Signed-off-by: Yan, Zheng <zheng.z.yan@intel.com>
Signed-off-by: Kan Liang <kan.liang@intel.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: eranian@google.com
Cc: jolsa@redhat.com
Link: http://lkml.kernel.org/r/1415156173-10035-7-git-send-email-kan.liang@intel.com
Signed-off-by: Ingo Molnar <mingo@kernel.org>
Diffstat (limited to 'kernel/events')
-rw-r--r-- | kernel/events/core.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/kernel/events/core.c b/kernel/events/core.c index 688086bb7144..84451c0debba 100644 --- a/kernel/events/core.c +++ b/kernel/events/core.c | |||
@@ -2562,6 +2562,9 @@ static void perf_event_context_sched_out(struct task_struct *task, int ctxn, | |||
2562 | next->perf_event_ctxp[ctxn] = ctx; | 2562 | next->perf_event_ctxp[ctxn] = ctx; |
2563 | ctx->task = next; | 2563 | ctx->task = next; |
2564 | next_ctx->task = task; | 2564 | next_ctx->task = task; |
2565 | |||
2566 | swap(ctx->task_ctx_data, next_ctx->task_ctx_data); | ||
2567 | |||
2565 | do_switch = 0; | 2568 | do_switch = 0; |
2566 | 2569 | ||
2567 | perf_event_sync_stat(ctx, next_ctx); | 2570 | perf_event_sync_stat(ctx, next_ctx); |