Re: [PATCH v2 3/9] target/arm: Introduce core_to_aa64_mmu_idx

[Peter Maydell]

On Mon, 2 Mar 2020 at 17:58, Richard Henderson
<address@hidden> wrote:
>
> If by context we know that we're in AArch64 mode, we need not
> test for M-profile when reconstructing the full ARMMMUIdx.
>
> Signed-off-by: Richard Henderson <address@hidden>
> ---
>  target/arm/internals.h     | 6 ++++++
>  target/arm/translate-a64.c | 2 +-
>  2 files changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/target/arm/internals.h b/target/arm/internals.h
> index 9f96a2359f..e633aff36e 100644
> --- a/target/arm/internals.h
> +++ b/target/arm/internals.h
> @@ -783,6 +783,12 @@ static inline ARMMMUIdx core_to_arm_mmu_idx(CPUARMState 
> *env, int mmu_idx)
>      }
>  }
>
> +static inline ARMMMUIdx core_to_aa64_mmu_idx(int mmu_idx)
> +{
> +    /* AArch64 is always a-profile. */
> +    return mmu_idx | ARM_MMU_IDX_A;
> +}
> +
>  int arm_mmu_idx_to_el(ARMMMUIdx mmu_idx);

Reviewed-by: Peter Maydell <address@hidden>

thanks
-- PMM