[PATCH 7/8] xfs_repair: don't create block maps for data files

Darrick J. Wong
2024-03-26T03:27:22Z

From: Darrick J. Wong <djwong@kernel.org>

Repair only queries inode block maps for inode forks that map filesystem
metadata. IOWs, it only uses it for directories, quota files, symlinks,
and realtime space metadata. It doesn't use it for regular files or
realtime files, so exclude its use for these files to reduce processing
times for heavily fragmented regular files.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
---
repair/dinode.c | 10 ++++++++--
1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/repair/dinode.c b/repair/dinode.c
index bf93a5790877..94f5fdcb4a37 100644
--- a/repair/dinode.c
+++ b/repair/dinode.c
@@ -1930,8 +1930,14 @@ process_inode_data_fork(
if (*nextents > be64_to_cpu(dino->di_nblocks))
*nextents = 1;

-
- if (dino->di_format != XFS_DINODE_FMT_LOCAL && type != XR_INO_RTDATA)
+ /*
+ * Repair doesn't care about the block maps for regular file data
+ * because it never tries to read data blocks. Only spend time on
+ * constructing a block map for directories, quota files, symlinks,
+ * and realtime space metadata.
+ */
+ if (dino->di_format != XFS_DINODE_FMT_LOCAL &&
+ (type != XR_INO_RTDATA && type != XR_INO_DATA))
*dblkmap = blkmap_alloc(*nextents, XFS_DATA_FORK);
*nextents = 0;