configure: don't check for HDIO_GETGEO

HDIO_GETGEO has been around longer than XFS.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
Signed-off-by: Carlos Maiolino <cem@kernel.org>

diff --git a/configure.ac b/configure.ac
index ae95a3dab7d12df7a511d70d0848d3ab221d9e5f..e0713e9bcd03f3616daf8b8c868f8dc6b2dc8716 100644 (file)
--- a/configure.ac
+++ b/configure.ac
@@ -182,7 +182,6 @@ if test "$enable_scrub" = "yes"; then
                 AC_MSG_ERROR([libicu not found.])
         fi
 fi
-AC_HAVE_HDIO_GETGEO
 AC_CONFIG_SYSTEMD_SYSTEM_UNIT_DIR
 AC_CONFIG_CROND_DIR
 AC_CONFIG_UDEV_RULE_DIR

diff --git a/include/builddefs.in b/include/builddefs.in
index ba5a55d79565e742994a75efb8d4c9657b4f1e01..644ed1cb1125e0ddea061766fa8d6a81d57fcf8e 100644 (file)
--- a/include/builddefs.in
+++ b/include/builddefs.in
@@ -102,7 +102,6 @@ HAVE_MALLINFO = @have_mallinfo@
 HAVE_MALLINFO2 = @have_mallinfo2@
 HAVE_LIBATTR = @have_libattr@
 HAVE_LIBICU = @have_libicu@
-HAVE_HDIO_GETGEO = @have_hdio_getgeo@
 HAVE_SYSTEMD = @have_systemd@
 SYSTEMD_SYSTEM_UNIT_DIR = @systemd_system_unit_dir@
 HAVE_CROND = @have_crond@

diff --git a/m4/package_libcdev.m4 b/m4/package_libcdev.m4
index 8a86bab5a5c31b1c0b932f1e636043b22b660fda..de64c9af7fdef2d50fd3fd27f3d4b8c34f0c0803 100644 (file)
--- a/m4/package_libcdev.m4
+++ b/m4/package_libcdev.m4
@@ -177,25 +177,6 @@ test = mallinfo2();
     AC_SUBST(have_mallinfo2)
   ])
 
-#
-# Check if we have the HDIO_GETGEO ioctl
-#
-AC_DEFUN([AC_HAVE_HDIO_GETGEO],
-  [ AC_MSG_CHECKING([for struct hd_geometry ])
-    AC_COMPILE_IFELSE(
-    [  AC_LANG_PROGRAM([[
-#include <linux/hdreg.h>
-#include <sys/ioctl.h>
-       ]], [[
-struct hd_geometry hdr;
-ioctl(0, HDIO_GETGEO, &hdr);
-       ]])
-    ], have_hdio_getgeo=yes
-       AC_MSG_RESULT(yes),
-       AC_MSG_RESULT(no))
-    AC_SUBST(have_hdio_getgeo)
-  ])
-
 AC_DEFUN([AC_PACKAGE_CHECK_LTO],
   [ AC_MSG_CHECKING([if C compiler supports LTO])
     OLD_CFLAGS="$CFLAGS"

diff --git a/scrub/Makefile b/scrub/Makefile
index 2f123ef005e169d218c9b929b728d541ece160b3..c11c2b5fe84e4520a8bd2c1cb09270155a310eba 100644 (file)
--- a/scrub/Makefile
+++ b/scrub/Makefile
@@ -98,10 +98,6 @@ CFILES += unicrash.c
 LCFLAGS += -DHAVE_LIBICU $(LIBICU_CFLAGS)
 endif
 
-ifeq ($(HAVE_HDIO_GETGEO),yes)
-LCFLAGS += -DHAVE_HDIO_GETGEO
-endif
-
 LDIRT = $(XFS_SCRUB_ALL_PROG) $(XFS_SCRUB_FAIL_PROG) *.service *.cron
 
 default: depend $(LTCOMMAND) $(XFS_SCRUB_ALL_PROG) $(XFS_SCRUB_FAIL_PROG) $(OPTIONAL_TARGETS)

diff --git a/scrub/disk.c b/scrub/disk.c
index 0ec29d965bd6a807e8c42b42dad9be55c6296959..2cf84d91887587d110753959ca1ed1d8a6166ed6 100644 (file)
--- a/scrub/disk.c
+++ b/scrub/disk.c
@@ -11,9 +11,7 @@
 #include <sys/types.h>
 #include <sys/statvfs.h>
 #include <scsi/sg.h>
-#ifdef HAVE_HDIO_GETGEO
-# include <linux/hdreg.h>
-#endif
+#include <linux/hdreg.h>
 #include "platform_defs.h"
 #include "libfrog/util.h"
 #include "libfrog/paths.h"
@@ -184,9 +182,7 @@ struct disk *
 disk_open(
        const char              *pathname)
 {
-#ifdef HAVE_HDIO_GETGEO
        struct hd_geometry      bdgeo;
-#endif
        struct disk             *disk;
        bool                    suspicious_disk = false;
        int                     error;
@@ -218,7 +214,6 @@ disk_open(
                error = ioctl(disk->d_fd, BLKBSZGET, &disk->d_blksize);
                if (error)
                        disk->d_blksize = 0;
-#ifdef HAVE_HDIO_GETGEO
                error = ioctl(disk->d_fd, HDIO_GETGEO, &bdgeo);
                if (!error) {
                        /*
@@ -234,7 +229,6 @@ disk_open(
                                suspicious_disk = true;
                        disk->d_start = bdgeo.start << BBSHIFT;
                } else
-#endif
                        disk->d_start = 0;
        } else {
                disk->d_size = disk->d_sb.st_size;