Use -Werror for all kernel configure tests.

As a matter of fact, we're already using -Werror for most tests because
of a bug in kernel-bio-empty-barrier.m4 which sets -Werror without
reverting it afterwards. This meant that all tests which ran after this
one was using -Werror.

This patch simply makes it clear that we're using -Werror and makes
the code more readable and more predictable.

Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #1317

diff --git a/config/kernel-bio-empty-barrier.m4 b/config/kernel-bio-empty-barrier.m4
index 99549fe24a217d40a66316c01a834043124844ff..fda16b3abf85e7ab92ef88e077d9af01b427377a 100644 (file)
--- a/config/kernel-bio-empty-barrier.m4
+++ b/config/kernel-bio-empty-barrier.m4
@@ -4,7 +4,6 @@ dnl # Empty write barriers are now supported and we should use them.
 dnl #
 AC_DEFUN([ZFS_AC_KERNEL_BIO_EMPTY_BARRIER], [
        AC_MSG_CHECKING([whether bio_empty_barrier() is defined])
-       EXTRA_KCFLAGS="-Werror"
        ZFS_LINUX_TRY_COMPILE([
                #include <linux/bio.h>
        ],[

diff --git a/config/kernel-bio-end-io-t-args.m4 b/config/kernel-bio-end-io-t-args.m4
index ced486193d9acfd096a4b32d0124ac206b049328..44e4826bfb13c67626111c935c030458195caefc 100644 (file)
--- a/config/kernel-bio-end-io-t-args.m4
+++ b/config/kernel-bio-end-io-t-args.m4
@@ -9,8 +9,6 @@ dnl # to void.
 dnl #
 AC_DEFUN([ZFS_AC_KERNEL_BIO_END_IO_T_ARGS], [
        AC_MSG_CHECKING([whether bio_end_io_t wants 2 args])
-       tmp_flags="$EXTRA_KCFLAGS"
-       EXTRA_KCFLAGS="-Werror"
        ZFS_LINUX_TRY_COMPILE([
                #include <linux/bio.h>
        ],[
@@ -25,5 +23,4 @@ AC_DEFUN([ZFS_AC_KERNEL_BIO_END_IO_T_ARGS], [
        ],[
                AC_MSG_RESULT(no)
        ])
-       EXTRA_KCFLAGS="$tmp_flags"
 ])

diff --git a/config/kernel-security-inode-init.m4 b/config/kernel-security-inode-init.m4
index c21588af75681f8778c15950b10bb796ef7436d5..a62176d42b546d30e548a0fdacc0d5dd41ae6762 100644 (file)
--- a/config/kernel-security-inode-init.m4
+++ b/config/kernel-security-inode-init.m4
@@ -7,8 +7,6 @@ dnl # security checks will just be skipped.
 dnl #
 AC_DEFUN([ZFS_AC_KERNEL_6ARGS_SECURITY_INODE_INIT_SECURITY], [
        AC_MSG_CHECKING([whether security_inode_init_security wants 6 args])
-       tmp_flags="$EXTRA_KCFLAGS"
-       EXTRA_KCFLAGS="-Werror"
        ZFS_LINUX_TRY_COMPILE([
                #include <linux/security.h>
        ],[
@@ -27,7 +25,6 @@ AC_DEFUN([ZFS_AC_KERNEL_6ARGS_SECURITY_INODE_INIT_SECURITY], [
        ],[
                AC_MSG_RESULT(no)
        ])
-       EXTRA_KCFLAGS="$tmp_flags"
 ])
 
 dnl #
@@ -39,8 +36,6 @@ dnl # and the EVM xattr.
 dnl #
 AC_DEFUN([ZFS_AC_KERNEL_CALLBACK_SECURITY_INODE_INIT_SECURITY], [
        AC_MSG_CHECKING([whether security_inode_init_security wants callback])
-       tmp_flags="$EXTRA_KCFLAGS"
-       EXTRA_KCFLAGS="-Werror"
        ZFS_LINUX_TRY_COMPILE([
                #include <linux/security.h>
        ],[
@@ -57,5 +52,4 @@ AC_DEFUN([ZFS_AC_KERNEL_CALLBACK_SECURITY_INODE_INIT_SECURITY], [
        ],[
                AC_MSG_RESULT(no)
        ])
-       EXTRA_KCFLAGS="$tmp_flags"
 ])

diff --git a/config/kernel.m4 b/config/kernel.m4
index 6d28074d0bbb3726c17ec280885eea0a774cc303..5623785fb3ac8a2b774916db1073c99b64b3a7b0 100644 (file)
--- a/config/kernel.m4
+++ b/config/kernel.m4
@@ -488,7 +488,7 @@ AC_DEFUN([ZFS_LINUX_COMPILE_IFELSE], [
        modpost_flag=''
        test "x$enable_linux_builtin" = xyes && modpost_flag='modpost=true' # fake modpost stage
        AS_IF(
-               [AC_TRY_COMMAND(cp conftest.c build && make [$2] -C $LINUX_OBJ EXTRA_CFLAGS="-Werror-implicit-function-declaration $EXTRA_KCFLAGS" $ARCH_UM M=$PWD/build $modpost_flag) >/dev/null && AC_TRY_COMMAND([$3])],
+               [AC_TRY_COMMAND(cp conftest.c build && make [$2] -C $LINUX_OBJ EXTRA_CFLAGS="-Werror $EXTRA_KCFLAGS" $ARCH_UM M=$PWD/build $modpost_flag) >/dev/null && AC_TRY_COMMAND([$3])],
                [$4],
                [_AC_MSG_LOG_CONFTEST m4_ifvaln([$5],[$5])]
        )