2023-05-01 22:32:26

by Drew Fustini

[permalink] [raw]
Subject: [PATCH] riscv: compat_syscall_table: Fixup compile warning

From: Guo Ren <[email protected]>

../arch/riscv/kernel/compat_syscall_table.c:12:41: warning: initialized
field overwritten [-Woverride-init]
12 | #define __SYSCALL(nr, call) [nr] = (call),
| ^
../include/uapi/asm-generic/unistd.h:567:1: note: in expansion of macro
'__SYSCALL'
567 | __SYSCALL(__NR_semget, sys_semget)

Fixes: 59c10c52f573 ("riscv: compat: syscall: Add compat_sys_call_table implementation")
Reviewed-by: Conor Dooley <[email protected]>
Reported-by: kernel test robot <[email protected]>
Tested-by: Jisheng Zhang <[email protected]>
Signed-off-by: Guo Ren <[email protected]>
Signed-off-by: Guo Ren <[email protected]>
Signed-off-by: Drew Fustini <[email protected]>
---
This fix was in the v9 of "riscv: Add GENERIC_ENTRY support and related
features" [1] but was dropped in v10 [2] with the prospect that it would
be submitted separately. That seems to have not happened so I am
submitting it now. I want to silence the riscv warnings when using W=1.
With this patch applied to riscv/for-next, I no longer see any warnings
for riscv.

[1] https://lore.kernel.org/linux-riscv/[email protected]/
[2] https://lore.kernel.org/lkml/[email protected]/

arch/riscv/kernel/Makefile | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/riscv/kernel/Makefile b/arch/riscv/kernel/Makefile
index 0fee73a20c87..c2ff6440767c 100644
--- a/arch/riscv/kernel/Makefile
+++ b/arch/riscv/kernel/Makefile
@@ -9,6 +9,7 @@ CFLAGS_REMOVE_patch.o = $(CC_FLAGS_FTRACE)
CFLAGS_REMOVE_sbi.o = $(CC_FLAGS_FTRACE)
endif
CFLAGS_syscall_table.o += $(call cc-option,-Wno-override-init,)
+CFLAGS_compat_syscall_table.o += $(call cc-option,-Wno-override-init,)

ifdef CONFIG_KEXEC
AFLAGS_kexec_relocate.o := -mcmodel=medany $(call cc-option,-mno-relax)
--
2.34.1


2023-05-02 01:05:36

by Palmer Dabbelt

[permalink] [raw]
Subject: Re: [PATCH] riscv: compat_syscall_table: Fixup compile warning


On Mon, 01 May 2023 15:33:54 -0700, Drew Fustini wrote:
> ../arch/riscv/kernel/compat_syscall_table.c:12:41: warning: initialized
> field overwritten [-Woverride-init]
> 12 | #define __SYSCALL(nr, call) [nr] = (call),
> | ^
> ../include/uapi/asm-generic/unistd.h:567:1: note: in expansion of macro
> '__SYSCALL'
> 567 | __SYSCALL(__NR_semget, sys_semget)
>
> [...]

Applied, thanks!

[1/1] riscv: compat_syscall_table: Fixup compile warning
https://git.kernel.org/palmer/c/f9c4bbddece7

Best regards,
--
Palmer Dabbelt <[email protected]>

Subject: Re: [PATCH] riscv: compat_syscall_table: Fixup compile warning

Hello:

This patch was applied to riscv/linux.git (fixes)
by Palmer Dabbelt <[email protected]>:

On Mon, 1 May 2023 15:33:54 -0700 you wrote:
> From: Guo Ren <[email protected]>
>
> ../arch/riscv/kernel/compat_syscall_table.c:12:41: warning: initialized
> field overwritten [-Woverride-init]
> 12 | #define __SYSCALL(nr, call) [nr] = (call),
> | ^
> ../include/uapi/asm-generic/unistd.h:567:1: note: in expansion of macro
> '__SYSCALL'
> 567 | __SYSCALL(__NR_semget, sys_semget)
>
> [...]

Here is the summary with links:
- riscv: compat_syscall_table: Fixup compile warning
https://git.kernel.org/riscv/c/f9c4bbddece7

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html