Re: [PATCH] target/arm: Count PMU events when MDCR.SPME is set

[Peter Maydell]

On Tue, 25 Aug 2020 at 15:49, Aaron Lindsay
<aaron@os.amperecomputing.com> wrote:
>
> This check was backwards when introduced in commit
> 033614c47de78409ad3fb39bb7bd1483b71c6789:
>
>     target/arm: Filter cycle counter based on PMCCFILTR_EL0
>
> Signed-off-by: Aaron Lindsay <aaron@os.amperecomputing.com>
> ---
>  target/arm/helper.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/arm/helper.c b/target/arm/helper.c
> index 6b4f0eb533..8e7730c7e3 100644
> --- a/target/arm/helper.c
> +++ b/target/arm/helper.c
> @@ -1452,7 +1452,7 @@ static bool pmu_counter_enabled(CPUARMState *env, 
> uint8_t counter)
>          }
>      } else {
>          prohibited = arm_feature(env, ARM_FEATURE_EL3) &&
> -           (env->cp15.mdcr_el3 & MDCR_SPME);
> +           !(env->cp15.mdcr_el3 & MDCR_SPME);
>      }
>
>      if (prohibited && counter == 31) {



Applied to target-arm.next, thanks.

-- PMM