[PATCH] powerpc/uaccess: Check CONFIG_PPC_E500

[PATCH] powerpc/uaccess: Check CONFIG_PPC_E500

[Pengpeng Hou]

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.

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);

Re: [PATCH] powerpc/uaccess: Check CONFIG_PPC_E500

[Christophe Leroy (CS GROUP)]

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);
>