On Mon, Jan 08, 2024 at 05:09:04PM -0800, Florian Fainelli wrote:
> Commit d8b0f5465012 ("wire up syscalls for statmount/listmount") added
> two new system calls to arch/arm64/include/asm/unistd32.h but forgot to
> update the __NR_compat_syscalls number, thus causing the following build
> failures:
>
> ./arch/arm64/include/asm/unistd32.h:922:24: error: array index in initializer exceeds array bounds
> 922 | #define __NR_statmount 457
> | ^~~
> arch/arm64/kernel/sys32.c:130:34: note: in definition of macro '__SYSCALL'
> 130 | #define __SYSCALL(nr, sym) [nr] = __arm64_##sym,
> | ^~
>
> Bump up the number by two to accomodate for the new system calls added.
>
> Fixes: d8b0f5465012 ("wire up syscalls for statmount/listmount")
> Signed-off-by: Florian Fainelli <[email protected]>
> ---
> arch/arm64/include/asm/unistd.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/include/asm/unistd.h b/arch/arm64/include/asm/unistd.h
> index 531effca5f1f..b63f870debaf 100644
> --- a/arch/arm64/include/asm/unistd.h
> +++ b/arch/arm64/include/asm/unistd.h
> @@ -39,7 +39,7 @@
> #define __ARM_NR_compat_set_tls (__ARM_NR_COMPAT_BASE + 5)
> #define __ARM_NR_COMPAT_END (__ARM_NR_COMPAT_BASE + 0x800)
>
> -#define __NR_compat_syscalls 457
> +#define __NR_compat_syscalls 459
> #endif
>
> #define __ARCH_WANT_SYS_CLONE
> --
> 2.34.1
Acked-by: Will Deacon <[email protected]>
Will
On 1/9/24 01:50, Will Deacon wrote:
> On Mon, Jan 08, 2024 at 05:09:04PM -0800, Florian Fainelli wrote:
>> Commit d8b0f5465012 ("wire up syscalls for statmount/listmount") added
>> two new system calls to arch/arm64/include/asm/unistd32.h but forgot to
>> update the __NR_compat_syscalls number, thus causing the following build
>> failures:
>>
>> ./arch/arm64/include/asm/unistd32.h:922:24: error: array index in initializer exceeds array bounds
>> 922 | #define __NR_statmount 457
>> | ^~~
>> arch/arm64/kernel/sys32.c:130:34: note: in definition of macro '__SYSCALL'
>> 130 | #define __SYSCALL(nr, sym) [nr] = __arm64_##sym,
>> | ^~
>>
>> Bump up the number by two to accomodate for the new system calls added.
>>
>> Fixes: d8b0f5465012 ("wire up syscalls for statmount/listmount")
>> Signed-off-by: Florian Fainelli <[email protected]>
>> ---
>> arch/arm64/include/asm/unistd.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/include/asm/unistd.h b/arch/arm64/include/asm/unistd.h
>> index 531effca5f1f..b63f870debaf 100644
>> --- a/arch/arm64/include/asm/unistd.h
>> +++ b/arch/arm64/include/asm/unistd.h
>> @@ -39,7 +39,7 @@
>> #define __ARM_NR_compat_set_tls (__ARM_NR_COMPAT_BASE + 5)
>> #define __ARM_NR_COMPAT_END (__ARM_NR_COMPAT_BASE + 0x800)
>>
>> -#define __NR_compat_syscalls 457
>> +#define __NR_compat_syscalls 459
>> #endif
>>
>> #define __ARCH_WANT_SYS_CLONE
>> --
>> 2.34.1
>
> Acked-by: Will Deacon <[email protected]>
Thanks Will, looks like Linus already fast tracked the change:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f0a78b3e2a0c842cc7b4c2686f4a35681f02ca72
--
Florian