Messages in this thread Patch in this message | | | Date | Fri, 09 Jul 2010 10:21:19 +0200 | From | Peter Zijlstra <> | Subject | [PATCH 02/13] sparc64: Fix maybe_change_configuration() PCR setting. |
| |
From: David S. Miller <davem@davemloft.net>
Need to mask out the existing event bits before OR'ing in the new ones.
Reported-by: Peter Zijlstra <a.p.zijlstra@chello.nl> Signed-off-by: David S. Miller <davem@davemloft.net> Signed-off-by: Peter Zijlstra <a.p.zijlstra@chello.nl> --- arch/sparc/kernel/perf_event.c | 1 + 1 file changed, 1 insertion(+)
Index: linux-2.6/arch/sparc/kernel/perf_event.c =================================================================== --- linux-2.6.orig/arch/sparc/kernel/perf_event.c +++ linux-2.6/arch/sparc/kernel/perf_event.c @@ -657,6 +657,7 @@ static u64 maybe_change_configuration(st cpuc->current_idx[i] = idx; enc = perf_event_get_enc(cpuc->events[i]); + pcr &= ~mask_for_index(idx); pcr |= event_encoding(enc, idx); } out:
| |