Re: [PATCH] powerpc/uaccess: Check CONFIG_PPC_E500

["Christophe Leroy (CS GROUP)" <chleroy@kernel.org>]

Hi,

Le 24/06/2026 à 10:15, Pengpeng Hou a écrit :
> The powerpc CPU Kconfig defines `PPC_E500`.  There is no generated
> `CONFIG_E500` symbol for `IS_ENABLED()` to test here.
> 
> `mask_user_address()` therefore checks an impossible symbol before
> choosing the E500-specific masking helper.  Use `CONFIG_PPC_E500`, which
> matches the actual Kconfig provider for this CPU family.

There is already a fix in the queue waiting to be applied:

https://lore.kernel.org/all/20260615233729.29386-1-enelsonmoore@gmail.com/


> 
> Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
> ---
>   arch/powerpc/include/asm/uaccess.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/powerpc/include/asm/uaccess.h b/arch/powerpc/include/asm/uaccess.h
> --- a/arch/powerpc/include/asm/uaccess.h
> +++ b/arch/powerpc/include/asm/uaccess.h
> @@ -537,7 +537,7 @@ static __always_inline unsigned long mask_user_address(const void __user *ptr)
>   
>   	if (IS_ENABLED(CONFIG_PPC64))
>   		return mask_user_address_simple(ptr);
> -	if (IS_ENABLED(CONFIG_E500))
> +	if (IS_ENABLED(CONFIG_PPC_E500))
>   		return mask_user_address_isel(ptr);
>   	if (TASK_SIZE <= UL(SZ_2G) && border >= UL(SZ_2G))
>   		return mask_user_address_simple(ptr);
>