[PATCH v1] arm64/mm: use GENMASK_ULL for TTBR_BADDR_MASK_52

[Kristina Martsenko]

On 08/07/2022 15:00, Joey Gouly wrote:
> The comment says this should be GENMASK_ULL(47, 12), so do that!
> 
> GENMASK_ULL() is available in assembly since:
>     95b980d62d52 ("linux/bits.h: make BIT(), GENMASK(), and friends available in assembly")
> 
> Signed-off-by: Joey Gouly <joey.gouly at arm.com>
> Link: https://lore.kernel.org/all/20171221164851.edxq536yobjuagwe@armageddon.cambridge.arm.com/
> Cc: Catalin Marinas <catalin.marinas at arm.com>
> Cc: Will Deacon <will at kernel.org>
> Cc: Kristina Martsenko <kristina.martsenko at arm.com>

FWIW,

Reviewed-by: Kristina Martsenko <kristina.martsenko at arm.com>

> ---
>  arch/arm64/include/asm/pgtable-hwdef.h | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/include/asm/pgtable-hwdef.h b/arch/arm64/include/asm/pgtable-hwdef.h
> index dd3d12bce07b..5ab8d163198f 100644
> --- a/arch/arm64/include/asm/pgtable-hwdef.h
> +++ b/arch/arm64/include/asm/pgtable-hwdef.h
> @@ -281,10 +281,9 @@
>   */
>  #ifdef CONFIG_ARM64_PA_BITS_52
>  /*
> - * This should be GENMASK_ULL(47, 2).
>   * TTBR_ELx[1] is RES0 in this configuration.
>   */
> -#define TTBR_BADDR_MASK_52	(((UL(1) << 46) - 1) << 2)
> +#define TTBR_BADDR_MASK_52	GENMASK_ULL(47, 2)
>  #endif
>  
>  #ifdef CONFIG_ARM64_VA_BITS_52