repair: do not prefetch holes in sparse inode chunks

The repair prefetch mechanism reads all inode chunks in advance of
repair processing to improve performance. Inode buffer verification and
processing can occur within the prefetch mechanism such as when
directories are being processed. Prefetch currently assumes fully
populated inode chunks which leads to corruption errors attempting to
verify inode buffers that do not contain inodes.

Update prefetch to check the previously scanned sparse inode bits and
skip inode buffer reads of clusters that are sparse. We check sparse
state per-inode cluster because the cluster size is the min. allowable
inode chunk hole granularity.

Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Dave Chinner <david@fromorbit.com>

diff --git a/repair/prefetch.c b/repair/prefetch.c
index d6246ce3c149929c6efa1c5368f13f279d4d7834..157797181763fdf8e7b3182e555bbb399216980c 100644 (file)
--- a/repair/prefetch.c
+++ b/repair/prefetch.c
@@ -679,6 +679,7 @@ pf_queuing_worker(
        xfs_agblock_t           bno;
        int                     i;
        int                     err;
+       uint64_t                sparse;
 
        blks_per_cluster = mp->m_inode_cluster_size >> mp->m_sb.sb_blocklog;
        if (blks_per_cluster == 0)
@@ -736,17 +737,27 @@ pf_queuing_worker(
 
                num_inos = 0;
                bno = XFS_AGINO_TO_AGBNO(mp, cur_irec->ino_startnum);
+               sparse = cur_irec->ir_sparse;
 
                do {
                        struct xfs_buf_map      map;
 
                        map.bm_bn = XFS_AGB_TO_DADDR(mp, args->agno, bno);
                        map.bm_len = XFS_FSB_TO_BB(mp, blks_per_cluster);
-                       pf_queue_io(args, &map, 1,
-                                   (cur_irec->ino_isa_dir != 0) ?  B_DIR_INODE
-                                                                : B_INODE);
+
+                       /*
+                        * Queue I/O for each non-sparse cluster. We can check
+                        * sparse state in cluster sized chunks as cluster size
+                        * is the min. granularity of sparse irec regions.
+                        */
+                       if ((sparse & ((1 << inodes_per_cluster) - 1)) == 0)
+                               pf_queue_io(args, &map, 1,
+                                           (cur_irec->ino_isa_dir != 0) ?
+                                            B_DIR_INODE : B_INODE);
+
                        bno += blks_per_cluster;
                        num_inos += inodes_per_cluster;
+                       sparse >>= inodes_per_cluster;
                } while (num_inos < mp->m_ialloc_inos);
        }