Regressions on mips malta_defconfig build failed with gcc-12 on the Linux next tag next-20250521 and next-20250522.
First seen on the next-20250521 Good: next-20250516 Bad: next-20250521
Regressions found on mips: - build/gcc-12-malta_defconfig
Regression Analysis: - New regression? Yes - Reproducible? Yes
Build regression: mips gcc-12 malta_defconfig 'SOCK_COREDUMP' undeclared (first use in this function); did you mean 'SOCK_RDM'?
Reported-by: Linux Kernel Functional Testing lkft@linaro.org
## Build log net/unix/af_unix.c: In function 'unix_find_bsd': net/unix/af_unix.c:1152:21: error: 'SOCK_COREDUMP' undeclared (first use in this function); did you mean 'SOCK_RDM'? 1152 | if (flags & SOCK_COREDUMP) { | ^~~~~~~~~~~~~ | SOCK_RDM
fs/coredump.c: In function 'do_coredump': fs/coredump.c:899:64: error: 'SOCK_COREDUMP' undeclared (first use in this function); did you mean 'SOCK_RDM'? 899 | addr_len, O_NONBLOCK | SOCK_COREDUMP); | ^~~~~~~~~~~~~ | SOCK_RDM fs/coredump.c:899:64: note: each undeclared identifier is reported only once for each function it appears in make[4]: *** [scripts/Makefile.build:203: fs/coredump.o] Error 1
## Source * Kernel version: 6.15.0-rc7 * Git tree: https://kernel.googlesource.com/pub/scm/linux/kernel/git/next/linux-next.git * Git sha: 460178e842c7a1e48a06df684c66eb5fd630bcf7 * Git describe: next-20250522
## Build * Build log: https://qa-reports.linaro.org/api/testruns/28516701/log_file/ * Build history: https://qa-reports.linaro.org/lkft/linux-next-master/build/next-20250522/tes... * Build link: https://storage.tuxsuite.com/public/linaro/lkft/builds/2xRo9ld0H5IJGyGHQxUSo... * Kernel config: https://storage.tuxsuite.com/public/linaro/lkft/builds/2xRo9ld0H5IJGyGHQxUSo...
-- Linaro LKFT https://lkft.linaro.org
On Thu, May 22, 2025, at 15:22, Naresh Kamboju wrote:
## Build log net/unix/af_unix.c: In function 'unix_find_bsd': net/unix/af_unix.c:1152:21: error: 'SOCK_COREDUMP' undeclared (first use in this function); did you mean 'SOCK_RDM'? 1152 | if (flags & SOCK_COREDUMP) {
SOCK_COREDUMP should be defined outside of ARCH_HAS_SOCKET_TYPES. How about reducing the scope of that check like this?
Arnd
diff --git a/arch/mips/include/asm/socket.h b/arch/mips/include/asm/socket.h index 4724a563c5bf..43a09f0dd3ff 100644 --- a/arch/mips/include/asm/socket.h +++ b/arch/mips/include/asm/socket.h @@ -36,15 +36,6 @@ enum sock_type { SOCK_PACKET = 10, };
-#define SOCK_MAX (SOCK_PACKET + 1) -/* Mask which covers at least up to SOCK_MASK-1. The - * * remaining bits are used as flags. */ -#define SOCK_TYPE_MASK 0xf - -/* Flags for socket, socketpair, paccept */ -#define SOCK_CLOEXEC O_CLOEXEC -#define SOCK_NONBLOCK O_NONBLOCK - #define ARCH_HAS_SOCKET_TYPES 1
#endif /* _ASM_SOCKET_H */ diff --git a/include/linux/net.h b/include/linux/net.h index 139c85d0f2ea..f60fff91e1df 100644 --- a/include/linux/net.h +++ b/include/linux/net.h @@ -70,6 +70,7 @@ enum sock_type { SOCK_DCCP = 6, SOCK_PACKET = 10, }; +#endif /* ARCH_HAS_SOCKET_TYPES */
#define SOCK_MAX (SOCK_PACKET + 1) /* Mask which covers at least up to SOCK_MASK-1. The @@ -83,8 +84,6 @@ enum sock_type { #endif #define SOCK_COREDUMP O_NOCTTY
-#endif /* ARCH_HAS_SOCKET_TYPES */ - /** * enum sock_shutdown_cmd - Shutdown types * @SHUT_RD: shutdown receptions
On Thu, May 22, 2025 at 04:01:53PM +0200, Arnd Bergmann wrote:
On Thu, May 22, 2025, at 15:22, Naresh Kamboju wrote:
## Build log net/unix/af_unix.c: In function 'unix_find_bsd': net/unix/af_unix.c:1152:21: error: 'SOCK_COREDUMP' undeclared (first use in this function); did you mean 'SOCK_RDM'? 1152 | if (flags & SOCK_COREDUMP) {
SOCK_COREDUMP should be defined outside of ARCH_HAS_SOCKET_TYPES. How about reducing the scope of that check like this?
Arnd
I applied the appended patch.
Hi Arnd,
On Thu, 22 May 2025 at 19:32, Arnd Bergmann arnd@arndb.de wrote:
On Thu, May 22, 2025, at 15:22, Naresh Kamboju wrote:
## Build log net/unix/af_unix.c: In function 'unix_find_bsd': net/unix/af_unix.c:1152:21: error: 'SOCK_COREDUMP' undeclared (first use in this function); did you mean 'SOCK_RDM'? 1152 | if (flags & SOCK_COREDUMP) {
SOCK_COREDUMP should be defined outside of ARCH_HAS_SOCKET_TYPES. How about reducing the scope of that check like this?
Arnd
Thanks for sharing a quick fix patch. This below patch is applied and build test pass.
Tested-by: Naresh Kamboju naresh.kamboju@linaro.org
diff --git a/arch/mips/include/asm/socket.h b/arch/mips/include/asm/socket.h index 4724a563c5bf..43a09f0dd3ff 100644 --- a/arch/mips/include/asm/socket.h +++ b/arch/mips/include/asm/socket.h @@ -36,15 +36,6 @@ enum sock_type { SOCK_PACKET = 10, };
-#define SOCK_MAX (SOCK_PACKET + 1) -/* Mask which covers at least up to SOCK_MASK-1. The
- remaining bits are used as flags. */
-#define SOCK_TYPE_MASK 0xf
-/* Flags for socket, socketpair, paccept */ -#define SOCK_CLOEXEC O_CLOEXEC -#define SOCK_NONBLOCK O_NONBLOCK
#define ARCH_HAS_SOCKET_TYPES 1
#endif /* _ASM_SOCKET_H */ diff --git a/include/linux/net.h b/include/linux/net.h index 139c85d0f2ea..f60fff91e1df 100644 --- a/include/linux/net.h +++ b/include/linux/net.h @@ -70,6 +70,7 @@ enum sock_type { SOCK_DCCP = 6, SOCK_PACKET = 10, }; +#endif /* ARCH_HAS_SOCKET_TYPES */
#define SOCK_MAX (SOCK_PACKET + 1) /* Mask which covers at least up to SOCK_MASK-1. The @@ -83,8 +84,6 @@ enum sock_type { #endif #define SOCK_COREDUMP O_NOCTTY
-#endif /* ARCH_HAS_SOCKET_TYPES */
/**
- enum sock_shutdown_cmd - Shutdown types
- @SHUT_RD: shutdown receptions