xfs: simplify xfs_rmap_lookup_le call sites

Source kernel commit: 5b7ca8b313621907d80460bfcc1fa876d2a38488

Most callers of xfs_rmap_lookup_le will retrieve the btree record
immediately if the lookup succeeds.  The overlapped version of this
function (xfs_rmap_lookup_le_range) will return the record if the lookup
succeeds, so make the regular version do it too.  Get rid of the useless
len argument, since it's not part of the lookup key.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

diff --git a/libxfs/xfs_rmap.c b/libxfs/xfs_rmap.c
index d6601a6573fe52a04181015c46082c7a106f60d0..0488dcde7a8f3ed2f570d1f330a04ed482e2aced 100644 (file)
--- a/libxfs/xfs_rmap.c
+++ b/libxfs/xfs_rmap.c
@@ -33,18 +33,32 @@ int
 xfs_rmap_lookup_le(
        struct xfs_btree_cur    *cur,
        xfs_agblock_t           bno,
-       xfs_extlen_t            len,
        uint64_t                owner,
        uint64_t                offset,
        unsigned int            flags,
+       struct xfs_rmap_irec    *irec,
        int                     *stat)
 {
+       int                     get_stat = 0;
+       int                     error;
+
        cur->bc_rec.r.rm_startblock = bno;
-       cur->bc_rec.r.rm_blockcount = len;
+       cur->bc_rec.r.rm_blockcount = 0;
        cur->bc_rec.r.rm_owner = owner;
        cur->bc_rec.r.rm_offset = offset;
        cur->bc_rec.r.rm_flags = flags;
-       return xfs_btree_lookup(cur, XFS_LOOKUP_LE, stat);
+
+       error = xfs_btree_lookup(cur, XFS_LOOKUP_LE, stat);
+       if (error || !(*stat) || !irec)
+               return error;
+
+       error = xfs_rmap_get_rec(cur, irec, &get_stat);
+       if (error)
+               return error;
+       if (!get_stat)
+               return -EFSCORRUPTED;
+
+       return 0;
 }
 
 /*
@@ -509,7 +523,7 @@ xfs_rmap_unmap(
         * for the AG headers at rm_startblock == 0 created by mkfs/growfs that
         * will not ever be removed from the tree.
         */
-       error = xfs_rmap_lookup_le(cur, bno, len, owner, offset, flags, &i);
+       error = xfs_rmap_lookup_le(cur, bno, owner, offset, flags, &ltrec, &i);
        if (error)
                goto out_error;
        if (XFS_IS_CORRUPT(mp, i != 1)) {
@@ -517,13 +531,6 @@ xfs_rmap_unmap(
                goto out_error;
        }
 
-       error = xfs_rmap_get_rec(cur, &ltrec, &i);
-       if (error)
-               goto out_error;
-       if (XFS_IS_CORRUPT(mp, i != 1)) {
-               error = -EFSCORRUPTED;
-               goto out_error;
-       }
        trace_xfs_rmap_lookup_le_range_result(cur->bc_mp,
                        cur->bc_ag.pag->pag_agno, ltrec.rm_startblock,
                        ltrec.rm_blockcount, ltrec.rm_owner,
@@ -785,18 +792,11 @@ xfs_rmap_map(
         * record for our insertion point. This will also give us the record for
         * start block contiguity tests.
         */
-       error = xfs_rmap_lookup_le(cur, bno, len, owner, offset, flags,
+       error = xfs_rmap_lookup_le(cur, bno, owner, offset, flags, &ltrec,
                        &have_lt);
        if (error)
                goto out_error;
        if (have_lt) {
-               error = xfs_rmap_get_rec(cur, &ltrec, &have_lt);
-               if (error)
-                       goto out_error;
-               if (XFS_IS_CORRUPT(mp, have_lt != 1)) {
-                       error = -EFSCORRUPTED;
-                       goto out_error;
-               }
                trace_xfs_rmap_lookup_le_range_result(cur->bc_mp,
                                cur->bc_ag.pag->pag_agno, ltrec.rm_startblock,
                                ltrec.rm_blockcount, ltrec.rm_owner,
@@ -1021,7 +1021,7 @@ xfs_rmap_convert(
         * record for our insertion point. This will also give us the record for
         * start block contiguity tests.
         */
-       error = xfs_rmap_lookup_le(cur, bno, len, owner, offset, oldext, &i);
+       error = xfs_rmap_lookup_le(cur, bno, owner, offset, oldext, &PREV, &i);
        if (error)
                goto done;
        if (XFS_IS_CORRUPT(mp, i != 1)) {
@@ -1029,13 +1029,6 @@ xfs_rmap_convert(
                goto done;
        }
 
-       error = xfs_rmap_get_rec(cur, &PREV, &i);
-       if (error)
-               goto done;
-       if (XFS_IS_CORRUPT(mp, i != 1)) {
-               error = -EFSCORRUPTED;
-               goto done;
-       }
        trace_xfs_rmap_lookup_le_range_result(cur->bc_mp,
                        cur->bc_ag.pag->pag_agno, PREV.rm_startblock,
                        PREV.rm_blockcount, PREV.rm_owner,
@@ -1139,7 +1132,7 @@ xfs_rmap_convert(
                        _RET_IP_);
 
        /* reset the cursor back to PREV */
-       error = xfs_rmap_lookup_le(cur, bno, len, owner, offset, oldext, &i);
+       error = xfs_rmap_lookup_le(cur, bno, owner, offset, oldext, NULL, &i);
        if (error)
                goto done;
        if (XFS_IS_CORRUPT(mp, i != 1)) {
@@ -2676,7 +2669,7 @@ xfs_rmap_record_exists(
        ASSERT(XFS_RMAP_NON_INODE_OWNER(owner) ||
               (flags & XFS_RMAP_BMBT_BLOCK));
 
-       error = xfs_rmap_lookup_le(cur, bno, len, owner, offset, flags,
+       error = xfs_rmap_lookup_le(cur, bno, owner, offset, flags, &irec,
                        &has_record);
        if (error)
                return error;
@@ -2685,14 +2678,6 @@ xfs_rmap_record_exists(
                return 0;
        }
 
-       error = xfs_rmap_get_rec(cur, &irec, &has_record);
-       if (error)
-               return error;
-       if (!has_record) {
-               *has_rmap = false;
-               return 0;
-       }
-
        *has_rmap = (irec.rm_owner == owner && irec.rm_startblock <= bno &&
                     irec.rm_startblock + irec.rm_blockcount >= bno + len);
        return 0;

diff --git a/libxfs/xfs_rmap.h b/libxfs/xfs_rmap.h
index b718ebeda372d67191801c39246b5198d798f7ab..11ec9406a0ea71a5f2be153a00582265309518e2 100644 (file)
--- a/libxfs/xfs_rmap.h
+++ b/libxfs/xfs_rmap.h
@@ -122,8 +122,8 @@ int xfs_rmap_free(struct xfs_trans *tp, struct xfs_buf *agbp,
                  const struct xfs_owner_info *oinfo);
 
 int xfs_rmap_lookup_le(struct xfs_btree_cur *cur, xfs_agblock_t bno,
-               xfs_extlen_t len, uint64_t owner, uint64_t offset,
-               unsigned int flags, int *stat);
+               uint64_t owner, uint64_t offset, unsigned int flags,
+               struct xfs_rmap_irec *irec, int *stat);
 int xfs_rmap_lookup_eq(struct xfs_btree_cur *cur, xfs_agblock_t bno,
                xfs_extlen_t len, uint64_t owner, uint64_t offset,
                unsigned int flags, int *stat);

diff --git a/repair/rmap.c b/repair/rmap.c
index b76a149d7465e84778bc8ae99fce7adbbb4d2ad4..51691fcceaae040a97963d532bf5f00e0066914c 100644 (file)
--- a/repair/rmap.c
+++ b/repair/rmap.c
@@ -905,18 +905,10 @@ rmap_lookup(
        struct xfs_rmap_irec    *tmp,
        int                     *have)
 {
-       int                     error;
-
        /* Use the regular btree retrieval routine. */
-       error = -libxfs_rmap_lookup_le(bt_cur, rm_rec->rm_startblock,
-                               rm_rec->rm_blockcount,
+       return -libxfs_rmap_lookup_le(bt_cur, rm_rec->rm_startblock,
                                rm_rec->rm_owner, rm_rec->rm_offset,
-                               rm_rec->rm_flags, have);
-       if (error)
-               return error;
-       if (*have == 0)
-               return error;
-       return -libxfs_rmap_get_rec(bt_cur, tmp, have);
+                               rm_rec->rm_flags, tmp, have);
 }
 
 /* Look for an rmap in the rmapbt that matches a given rmap. */