On Sun, 3 Jul 2022 at 09:25, Richard Henderson
<richard.hender...@linaro.org> wrote:
>
> The comment was correct, but the test was not:
> disable mte if tagged is *not* set.
>
> Signed-off-by: Richard Henderson <richard.hender...@linaro.org>
> ---
> target/arm/sve_helper.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/arm/sve_helper.c b/target/arm/sve_helper.c
> index 1654c0bbf9..db15d03ded 100644
> --- a/target/arm/sve_helper.c
> +++ b/target/arm/sve_helper.c
> @@ -5986,7 +5986,7 @@ void sve_ldnfff1_r(CPUARMState *env, void *vg, const
> target_ulong addr,
> * Disable MTE checking if the Tagged bit is not set. Since TBI must
> * be set within MTEDESC for MTE, !mtedesc => !mte_active.
> */
> - if (arm_tlb_mte_tagged(&info.page[0].attrs)) {
> + if (!arm_tlb_mte_tagged(&info.page[0].attrs)) {
> mtedesc = 0;
> }
Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>
Since this is a standalone bugfix I've put it in target-arm.next
so it doesn't get held up while we review the rest of the series.
thanks
-- PMM
