[PATCH] ARM: perf-events: fix v7 event selection mask
Jean Pihet
jpihet at mvista.com
Fri Feb 26 04:31:26 EST 2010
Will,
On Thursday 25 February 2010 19:18:31 Will Deacon wrote:
> The event selection mask for ARMv7 cores [ARMV7_EVTSEL_MASK]
> is incorrectly set to 0x7f. This means that the top bit of an
> event ID is ignored, so counting branch misses (id=0x10) and
> ISBs (id=0x90) give the same results.
Good catch! It is a left over from the Cortex-A8 only code. Cortex-A9
introduces more events.
Signed-off-by: Jean Pihet <jpihet at mvista.com>
Thanks,
Jean
>
> This patch sets the event selection mask to the correct value
> of 0xff.
>
> Cc: Jean Pihet <jpihet at mvista.com>
> Signed-off-by: Will Deacon <will.deacon at arm.com>
> ---
> arch/arm/kernel/perf_event.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/arch/arm/kernel/perf_event.c b/arch/arm/kernel/perf_event.c
> index b44d159..c45a155 100644
> --- a/arch/arm/kernel/perf_event.c
> +++ b/arch/arm/kernel/perf_event.c
> @@ -1625,7 +1625,7 @@ enum armv7_counters {
> /*
> * EVTSEL: Event selection reg
> */
> -#define ARMV7_EVTSEL_MASK 0x7f /* Mask for writable bits */
> +#define ARMV7_EVTSEL_MASK 0xff /* Mask for writable bits */
>
> /*
> * SELECT: Counter selection reg
More information about the linux-arm-kernel
mailing list