]> git.ipfire.org Git - thirdparty/xfsprogs-dev.git/commitdiff
libfrog: always use the kernel GETFSMAP definitions
authorDarrick J. Wong <djwong@kernel.org>
Fri, 25 Feb 2022 22:37:50 +0000 (17:37 -0500)
committerEric Sandeen <sandeen@sandeen.net>
Fri, 25 Feb 2022 22:37:50 +0000 (17:37 -0500)
The GETFSMAP ioctl has been a part of the kernel since 4.12.  We have no
business shipping a stale copy of kernel header contents in the xfslibs
package, so get rid of it.  This means that xfs_scrub now has a hard
dependency on the build system having new kernel headers.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Eric Sandeen <sandeen@redhat.com>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>
include/linux.h
io/Makefile
io/fsmap.c
scrub/Makefile
scrub/phase6.c
scrub/phase7.c
scrub/spacemap.c
spaceman/Makefile
spaceman/freesp.c

index de8a71221146c9dda5cb9bbfe9b2dd6dcdf6bd2c..3d9f4e3dca8020380df17ec1e41c5631a53fb096 100644 (file)
@@ -251,111 +251,6 @@ struct fsxattr {
 #define FS_XFLAG_COWEXTSIZE    0x00010000      /* CoW extent size allocator hint */
 #endif
 
-#ifdef HAVE_GETFSMAP
-# include <linux/fsmap.h>
-#else
-/*
- *     Structure for FS_IOC_GETFSMAP.
- *
- *     The memory layout for this call are the scalar values defined in
- *     struct fsmap_head, followed by two struct fsmap that describe
- *     the lower and upper bound of mappings to return, followed by an
- *     array of struct fsmap mappings.
- *
- *     fmh_iflags control the output of the call, whereas fmh_oflags report
- *     on the overall record output.  fmh_count should be set to the
- *     length of the fmh_recs array, and fmh_entries will be set to the
- *     number of entries filled out during each call.  If fmh_count is
- *     zero, the number of reverse mappings will be returned in
- *     fmh_entries, though no mappings will be returned.  fmh_reserved
- *     must be set to zero.
- *
- *     The two elements in the fmh_keys array are used to constrain the
- *     output.  The first element in the array should represent the
- *     lowest disk mapping ("low key") that the user wants to learn
- *     about.  If this value is all zeroes, the filesystem will return
- *     the first entry it knows about.  For a subsequent call, the
- *     contents of fsmap_head.fmh_recs[fsmap_head.fmh_count - 1] should be
- *     copied into fmh_keys[0] to have the kernel start where it left off.
- *
- *     The second element in the fmh_keys array should represent the
- *     highest disk mapping ("high key") that the user wants to learn
- *     about.  If this value is all ones, the filesystem will not stop
- *     until it runs out of mapping to return or runs out of space in
- *     fmh_recs.
- *
- *     fmr_device can be either a 32-bit cookie representing a device, or
- *     a 32-bit dev_t if the FMH_OF_DEV_T flag is set.  fmr_physical,
- *     fmr_offset, and fmr_length are expressed in units of bytes.
- *     fmr_owner is either an inode number, or a special value if
- *     FMR_OF_SPECIAL_OWNER is set in fmr_flags.
- */
-struct fsmap {
-       __u32           fmr_device;     /* device id */
-       __u32           fmr_flags;      /* mapping flags */
-       __u64           fmr_physical;   /* device offset of segment */
-       __u64           fmr_owner;      /* owner id */
-       __u64           fmr_offset;     /* file offset of segment */
-       __u64           fmr_length;     /* length of segment */
-       __u64           fmr_reserved[3];        /* must be zero */
-};
-
-struct fsmap_head {
-       __u32           fmh_iflags;     /* control flags */
-       __u32           fmh_oflags;     /* output flags */
-       __u32           fmh_count;      /* # of entries in array incl. input */
-       __u32           fmh_entries;    /* # of entries filled in (output). */
-       __u64           fmh_reserved[6];        /* must be zero */
-
-       struct fsmap    fmh_keys[2];    /* low and high keys for the mapping search */
-       struct fsmap    fmh_recs[];     /* returned records */
-};
-
-/* Size of an fsmap_head with room for nr records. */
-static inline size_t
-fsmap_sizeof(
-       unsigned int    nr)
-{
-       return sizeof(struct fsmap_head) + nr * sizeof(struct fsmap);
-}
-
-/* Start the next fsmap query at the end of the current query results. */
-static inline void
-fsmap_advance(
-       struct fsmap_head       *head)
-{
-       head->fmh_keys[0] = head->fmh_recs[head->fmh_entries - 1];
-}
-
-/*     fmh_iflags values - set by XFS_IOC_GETFSMAP caller in the header. */
-/* no flags defined yet */
-#define FMH_IF_VALID           0
-
-/*     fmh_oflags values - returned in the header segment only. */
-#define FMH_OF_DEV_T           0x1     /* fmr_device values will be dev_t */
-
-/*     fmr_flags values - returned for each non-header segment */
-#define FMR_OF_PREALLOC                0x1     /* segment = unwritten pre-allocation */
-#define FMR_OF_ATTR_FORK       0x2     /* segment = attribute fork */
-#define FMR_OF_EXTENT_MAP      0x4     /* segment = extent map */
-#define FMR_OF_SHARED          0x8     /* segment = shared with another file */
-#define FMR_OF_SPECIAL_OWNER   0x10    /* owner is a special value */
-#define FMR_OF_LAST            0x20    /* segment is the last in the FS */
-
-/* Each FS gets to define its own special owner codes. */
-#define FMR_OWNER(type, code)  (((__u64)type << 32) | \
-                                ((__u64)code & 0xFFFFFFFFULL))
-#define FMR_OWNER_TYPE(owner)  ((__u32)((__u64)owner >> 32))
-#define FMR_OWNER_CODE(owner)  ((__u32)(((__u64)owner & 0xFFFFFFFFULL)))
-#define FMR_OWN_FREE           FMR_OWNER(0, 1) /* free space */
-#define FMR_OWN_UNKNOWN                FMR_OWNER(0, 2) /* unknown owner */
-#define FMR_OWN_METADATA       FMR_OWNER(0, 3) /* metadata */
-
-#define FS_IOC_GETFSMAP                _IOWR('X', 59, struct fsmap_head)
-
-#define HAVE_GETFSMAP
-#endif /* HAVE_GETFSMAP */
-
 #ifndef HAVE_MAP_SYNC
 #define MAP_SYNC 0
 #define MAP_SHARED_VALIDATE 0
index 7174192681683022d9e2951451a04e3e659b8be5..498174cfc43e32006c751a84883059a319c9706b 100644 (file)
@@ -104,10 +104,9 @@ LLDLIBS += $(LIBDEVMAPPER)
 LCFLAGS += -DHAVE_DEVMAPPER
 endif
 
-# On linux we get fsmap from the system or define it ourselves
-# so include this unconditionally.  If this reverts to only
-# the autoconf check w/o local definition, test HAVE_GETFSMAP
+ifeq ($(HAVE_GETFSMAP),yes)
 CFILES += fsmap.c
+endif
 
 ifeq ($(HAVE_STATFS_FLAGS),yes)
 LCFLAGS += -DHAVE_STATFS_FLAGS
index f540a7c04409c0ba3a91f7631d88df1aee5842d7..9dd19cc04a9eb064b407847d22b53e441b494ab5 100644 (file)
@@ -4,6 +4,7 @@
  * Author: Darrick J. Wong <darrick.wong@oracle.com>
  */
 #include "platform_defs.h"
+#include <linux/fsmap.h>
 #include "command.h"
 #include "init.h"
 #include "libfrog/paths.h"
index fd6bb6796986712918097b6eaf60a1add4753029..335e1e8de3799e4342ed080d02f76d573bf1c931 100644 (file)
@@ -6,12 +6,9 @@ TOPDIR = ..
 builddefs=$(TOPDIR)/include/builddefs
 include $(builddefs)
 
-# On linux we get fsmap from the system or define it ourselves
-# so include this based on platform type.  If this reverts to only
-# the autoconf check w/o local definition, change to testing HAVE_GETFSMAP
-SCRUB_PREREQS=$(HAVE_OPENAT)$(HAVE_FSTATAT)
+SCRUB_PREREQS=$(HAVE_OPENAT)$(HAVE_FSTATAT)$(HAVE_GETFSMAP)
 
-ifeq ($(SCRUB_PREREQS),yesyes)
+ifeq ($(SCRUB_PREREQS),yesyesyes)
 LTCOMMAND = xfs_scrub
 INSTALL_SCRUB = install-scrub
 XFS_SCRUB_ALL_PROG = xfs_scrub_all
index 87828b60fbed90a1dfd9d681512672a774f83f6a..afdb16b689cc84693aa0a8ecf8931f7e1aaef9bc 100644 (file)
@@ -7,6 +7,7 @@
 #include <stdint.h>
 #include <dirent.h>
 #include <sys/statvfs.h>
+#include <linux/fsmap.h>
 #include "handle.h"
 #include "libfrog/paths.h"
 #include "libfrog/workqueue.h"
index bc652ab6f44aeb790602b4dcc8a3775b9f20ea49..84546b1cb5cc2b29047f151cf892ef5b5e8038f0 100644 (file)
@@ -7,6 +7,7 @@
 #include <stdint.h>
 #include <stdlib.h>
 #include <sys/statvfs.h>
+#include <linux/fsmap.h>
 #include "libfrog/paths.h"
 #include "libfrog/ptvar.h"
 #include "list.h"
index a5508d568a4386d2be40ec7e9c57d47f64d65cbd..03440d3a85486b4b9cec09356322b86653f606e0 100644 (file)
@@ -8,6 +8,7 @@
 #include <string.h>
 #include <pthread.h>
 #include <sys/statvfs.h>
+#include <linux/fsmap.h>
 #include "libfrog/workqueue.h"
 #include "libfrog/paths.h"
 #include "xfs_scrub.h"
index 2a3669183a40ee86b6da04c587d85be5b77d1c4a..1f048d54a4d2b4ce54f43f46987651bbdb65dc1b 100644 (file)
@@ -18,10 +18,9 @@ ifeq ($(ENABLE_EDITLINE),yes)
 LLDLIBS += $(LIBEDITLINE) $(LIBTERMCAP)
 endif
 
-# On linux we get fsmap from the system or define it ourselves
-# so include this unconditionally.  If this reverts to only
-# the autoconf check w/o local definition, test HAVE_GETFSMAP
+ifeq ($(HAVE_GETFSMAP),yes)
 CFILES += freesp.c
+endif
 
 default: depend $(LTCOMMAND)
 
index de301c195fb3b476073e65663e3d574f464e9a33..423568a42488d7518ee316b81c6914c807a6afc9 100644 (file)
@@ -8,6 +8,7 @@
 
 #include "libxfs.h"
 #include <linux/fiemap.h>
+#include <linux/fsmap.h>
 #include "libfrog/fsgeom.h"
 #include "command.h"
 #include "init.h"