Browse Source

xfs: move more RT specific code under CONFIG_XFS_RT

Various utility functions and interfaces that iterate internal
devices try to reference the realtime device even when RT support is
not compiled into the kernel.

Make sure this code is excluded from the CONFIG_XFS_RT=n build,
and where appropriate stub functions to return fatal errors if
they ever get called when RT support is not present.

Signed-Off-By: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Dave Chinner 7 years ago
parent
commit
bb9c2e5433
3 changed files with 27 additions and 0 deletions
  1. 2 0
      fs/xfs/xfs_bmap_util.c
  2. 13 0
      fs/xfs/xfs_bmap_util.h
  3. 12 0
      fs/xfs/xfs_fsmap.c

+ 2 - 0
fs/xfs/xfs_bmap_util.c

@@ -84,6 +84,7 @@ xfs_zero_extent(
 		GFP_NOFS, 0);
 		GFP_NOFS, 0);
 }
 }
 
 
+#ifdef CONFIG_XFS_RT
 int
 int
 xfs_bmap_rtalloc(
 xfs_bmap_rtalloc(
 	struct xfs_bmalloca	*ap)	/* bmap alloc argument struct */
 	struct xfs_bmalloca	*ap)	/* bmap alloc argument struct */
@@ -190,6 +191,7 @@ xfs_bmap_rtalloc(
 	}
 	}
 	return 0;
 	return 0;
 }
 }
+#endif /* CONFIG_XFS_RT */
 
 
 /*
 /*
  * Check if the endoff is outside the last extent. If so the caller will grow
  * Check if the endoff is outside the last extent. If so the caller will grow

+ 13 - 0
fs/xfs/xfs_bmap_util.h

@@ -28,7 +28,20 @@ struct xfs_mount;
 struct xfs_trans;
 struct xfs_trans;
 struct xfs_bmalloca;
 struct xfs_bmalloca;
 
 
+#ifdef CONFIG_XFS_RT
 int	xfs_bmap_rtalloc(struct xfs_bmalloca *ap);
 int	xfs_bmap_rtalloc(struct xfs_bmalloca *ap);
+#else /* !CONFIG_XFS_RT */
+/*
+ * Attempts to allocate RT extents when RT is disable indicates corruption and
+ * should trigger a shutdown.
+ */
+static inline int
+xfs_bmap_rtalloc(struct xfs_bmalloca *ap)
+{
+	return -EFSCORRUPTED;
+}
+#endif /* CONFIG_XFS_RT */
+
 int	xfs_bmap_eof(struct xfs_inode *ip, xfs_fileoff_t endoff,
 int	xfs_bmap_eof(struct xfs_inode *ip, xfs_fileoff_t endoff,
 		     int whichfork, int *eof);
 		     int whichfork, int *eof);
 int	xfs_bmap_punch_delalloc_range(struct xfs_inode *ip,
 int	xfs_bmap_punch_delalloc_range(struct xfs_inode *ip,

+ 12 - 0
fs/xfs/xfs_fsmap.c

@@ -521,6 +521,7 @@ __xfs_getfsmap_rtdev(
 	return query_fn(tp, info);
 	return query_fn(tp, info);
 }
 }
 
 
+#ifdef CONFIG_XFS_RT
 /* Actually query the realtime bitmap. */
 /* Actually query the realtime bitmap. */
 STATIC int
 STATIC int
 xfs_getfsmap_rtdev_rtbitmap_query(
 xfs_getfsmap_rtdev_rtbitmap_query(
@@ -561,6 +562,7 @@ xfs_getfsmap_rtdev_rtbitmap(
 	return __xfs_getfsmap_rtdev(tp, keys, xfs_getfsmap_rtdev_rtbitmap_query,
 	return __xfs_getfsmap_rtdev(tp, keys, xfs_getfsmap_rtdev_rtbitmap_query,
 			info);
 			info);
 }
 }
+#endif /* CONFIG_XFS_RT */
 
 
 /* Execute a getfsmap query against the regular data device. */
 /* Execute a getfsmap query against the regular data device. */
 STATIC int
 STATIC int
@@ -795,7 +797,15 @@ xfs_getfsmap_check_keys(
 	return false;
 	return false;
 }
 }
 
 
+/*
+ * There are only two devices if we didn't configure RT devices at build time.
+ */
+#ifdef CONFIG_XFS_RT
 #define XFS_GETFSMAP_DEVS	3
 #define XFS_GETFSMAP_DEVS	3
+#else
+#define XFS_GETFSMAP_DEVS	2
+#endif /* CONFIG_XFS_RT */
+
 /*
 /*
  * Get filesystem's extents as described in head, and format for
  * Get filesystem's extents as described in head, and format for
  * output.  Calls formatter to fill the user's buffer until all
  * output.  Calls formatter to fill the user's buffer until all
@@ -853,10 +863,12 @@ xfs_getfsmap(
 		handlers[1].dev = new_encode_dev(mp->m_logdev_targp->bt_dev);
 		handlers[1].dev = new_encode_dev(mp->m_logdev_targp->bt_dev);
 		handlers[1].fn = xfs_getfsmap_logdev;
 		handlers[1].fn = xfs_getfsmap_logdev;
 	}
 	}
+#ifdef CONFIG_XFS_RT
 	if (mp->m_rtdev_targp) {
 	if (mp->m_rtdev_targp) {
 		handlers[2].dev = new_encode_dev(mp->m_rtdev_targp->bt_dev);
 		handlers[2].dev = new_encode_dev(mp->m_rtdev_targp->bt_dev);
 		handlers[2].fn = xfs_getfsmap_rtdev_rtbitmap;
 		handlers[2].fn = xfs_getfsmap_rtdev_rtbitmap;
 	}
 	}
+#endif /* CONFIG_XFS_RT */
 
 
 	xfs_sort(handlers, XFS_GETFSMAP_DEVS, sizeof(struct xfs_getfsmap_dev),
 	xfs_sort(handlers, XFS_GETFSMAP_DEVS, sizeof(struct xfs_getfsmap_dev),
 			xfs_getfsmap_dev_compare);
 			xfs_getfsmap_dev_compare);