From a3588c68f7026c49291a7ec92a78c69b22e15df1 Mon Sep 17 00:00:00 2001 From: Alexander Date: Wed, 8 Sep 2021 21:59:43 +0200 Subject: [PATCH] Linux 5.15 compat: standalone Kernel commits 39f75da7bcc8 ("isystem: trim/fixup stdarg.h and other headers") c0891ac15f04 ("isystem: ship and use stdarg.h") 564f963eabd1 ("isystem: delete global -isystem compile option") (for now can be found in linux-next.git tree, will land into the Linus' tree during the ongoing 5.15 cycle with one of akpm merges) removed the -isystem flag and disallowed the inclusion of any compiler header files. They also introduced a minimal as a replacement for . include/os/linux/spl/sys/cmn_err.h in the ZFS source tree includes unconditionally. Introduce a test for and include it instead of the compiler's one to prevent module build breakage. Reviewed-by: Tony Nguyen Reviewed-by: Brian Behlendorf Signed-off-by: Alexander Lobakin Closes #12531 --- config/kernel-stdarg.m4 | 32 ++++++++++++++++++++++++++++++ config/kernel.m4 | 2 ++ include/os/linux/spl/sys/cmn_err.h | 4 ++++ 3 files changed, 38 insertions(+) create mode 100644 config/kernel-stdarg.m4 diff --git a/config/kernel-stdarg.m4 b/config/kernel-stdarg.m4 new file mode 100644 index 00000000000000..5bc8dd859d6b6d --- /dev/null +++ b/config/kernel-stdarg.m4 @@ -0,0 +1,32 @@ +dnl # +dnl # Linux 5.15 gets rid of -isystem and external inclusion +dnl # and ships its own . Check if this header file does +dnl # exist and provide all necessary definitions for variable argument +dnl # functions. Adjust the inclusion of according to the +dnl # results. +dnl # +AC_DEFUN([ZFS_AC_KERNEL_SRC_STANDALONE_LINUX_STDARG], [ + ZFS_LINUX_TEST_SRC([has_standalone_linux_stdarg], [ + #include + + #if !defined(va_start) || !defined(va_end) || \ + !defined(va_arg) || !defined(va_copy) + #error " is invalid" + #endif + ],[]) +]) + +AC_DEFUN([ZFS_AC_KERNEL_STANDALONE_LINUX_STDARG], [ + dnl # + dnl # Linux 5.15 ships its own stdarg.h and doesn't allow to + dnl # include compiler headers. + dnl # + AC_MSG_CHECKING([whether standalone exists]) + ZFS_LINUX_TEST_RESULT([has_standalone_linux_stdarg], [ + AC_MSG_RESULT([yes]) + AC_DEFINE(HAVE_STANDALONE_LINUX_STDARG, 1, + [standalone exists]) + ],[ + AC_MSG_RESULT([no]) + ]) +]) diff --git a/config/kernel.m4 b/config/kernel.m4 index 5ea2286dbcc3d5..0b94f3bd9cb6e5 100644 --- a/config/kernel.m4 +++ b/config/kernel.m4 @@ -133,6 +133,7 @@ AC_DEFUN([ZFS_AC_KERNEL_TEST_SRC], [ ZFS_AC_KERNEL_SRC_SIGINFO ZFS_AC_KERNEL_SRC_SET_SPECIAL_STATE ZFS_AC_KERNEL_SRC_VFS_SET_PAGE_DIRTY_NOBUFFERS + ZFS_AC_KERNEL_SRC_STANDALONE_LINUX_STDARG AC_MSG_CHECKING([for available kernel interfaces]) ZFS_LINUX_TEST_COMPILE_ALL([kabi]) @@ -239,6 +240,7 @@ AC_DEFUN([ZFS_AC_KERNEL_TEST_RESULT], [ ZFS_AC_KERNEL_SIGINFO ZFS_AC_KERNEL_SET_SPECIAL_STATE ZFS_AC_KERNEL_VFS_SET_PAGE_DIRTY_NOBUFFERS + ZFS_AC_KERNEL_STANDALONE_LINUX_STDARG ]) dnl # diff --git a/include/os/linux/spl/sys/cmn_err.h b/include/os/linux/spl/sys/cmn_err.h index 79297067c17d2c..d2088371c6bc14 100644 --- a/include/os/linux/spl/sys/cmn_err.h +++ b/include/os/linux/spl/sys/cmn_err.h @@ -24,7 +24,11 @@ #ifndef _SPL_CMN_ERR_H #define _SPL_CMN_ERR_H +#if defined(_KERNEL) && defined(HAVE_STANDALONE_LINUX_STDARG) +#include +#else #include +#endif #define CE_CONT 0 /* continuation */ #define CE_NOTE 1 /* notice */