e191b54ecf
Certain versions of gcc generate an 'unrecognized command
line option' error message when -Wunused-but-set-variable
is used unconditionally. This in turn can cause several
of the autoconf tests to misdetect an interface.
Now, the use of -Wunused-but-set-variable in the autoconf
tests was introduced by commit b9c59ec8 to address a gcc
4.6 compatibility problem. So we really only need to pass
this option for version of gcc which are known to support it.
Therefore, the tests have been updated to use the result of
the existing ZFS_AC_CONFIG_ALWAYS_NO_UNUSED_BUT_SET_VARIABLE
which determines if gcc supports this option.
Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov>
Closes #1004
42 lines
1.1 KiB
Plaintext
42 lines
1.1 KiB
Plaintext
dnl #
|
|
dnl # 2.6.29 API change
|
|
dnl # In the 2.6.29 kernel blk_rq_bytes() was available as a GPL-only symbol.
|
|
dnl # So we need to check the symbol license as well. As of 2.6.31 the
|
|
dnl blk_rq_bytes() helper was changed to a static inline which we can use.
|
|
dnl #
|
|
AC_DEFUN([ZFS_AC_KERNEL_BLK_RQ_BYTES], [
|
|
AC_MSG_CHECKING([whether blk_rq_bytes() is available])
|
|
tmp_flags="$EXTRA_KCFLAGS"
|
|
EXTRA_KCFLAGS="${NO_UNUSED_BUT_SET_VARIABLE}"
|
|
ZFS_LINUX_TRY_COMPILE([
|
|
#include <linux/blkdev.h>
|
|
],[
|
|
struct request *req = NULL;
|
|
(void) blk_rq_bytes(req);
|
|
],[
|
|
AC_MSG_RESULT(yes)
|
|
AC_DEFINE(HAVE_BLK_RQ_BYTES, 1,
|
|
[blk_rq_bytes() is available])
|
|
],[
|
|
AC_MSG_RESULT(no)
|
|
])
|
|
|
|
AC_MSG_CHECKING([whether blk_rq_bytes() is GPL-only])
|
|
ZFS_LINUX_TRY_COMPILE([
|
|
#include <linux/module.h>
|
|
#include <linux/blkdev.h>
|
|
|
|
MODULE_LICENSE("CDDL");
|
|
],[
|
|
struct request *req = NULL;
|
|
(void) blk_rq_bytes(req);
|
|
],[
|
|
AC_MSG_RESULT(no)
|
|
],[
|
|
AC_MSG_RESULT(yes)
|
|
AC_DEFINE(HAVE_BLK_RQ_BYTES_GPL_ONLY, 1,
|
|
[blk_rq_bytes() is GPL-only])
|
|
])
|
|
EXTRA_KCFLAGS="$tmp_flags"
|
|
])
|