xfs: refactor allocation tree fixup code

Source kernel commit: 4a65b7c2c72c3940a4472c49743fdb65a03a9935

Both algorithms duplicate the same btree allocation code. Eliminate
the duplication and reuse the fallback algorithm codepath.

Signed-off-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>
Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

diff --git a/libxfs/xfs_alloc.c b/libxfs/xfs_alloc.c
index dec1a79f5b2d3e99e7b3165f47dd63225faaca95..8df4e7153749809c56e4f616e9ac9d05f7d7af6b 100644 (file)
--- a/libxfs/xfs_alloc.c
+++ b/libxfs/xfs_alloc.c
@@ -1328,23 +1328,8 @@ restart:
                if (acur.len == 0)
                        break;
 
-               /*
-                * Allocate at the bno/len tracked in the cursor.
-                */
-               args->agbno = acur.bno;
-               args->len = acur.len;
-               ASSERT(acur.bno >= acur.rec_bno);
-               ASSERT(acur.bno + acur.len <= acur.rec_bno + acur.rec_len);
-               ASSERT(acur.rec_bno + acur.rec_len <=
-                      be32_to_cpu(XFS_BUF_TO_AGF(args->agbp)->agf_length));
-
-               error = xfs_alloc_fixup_trees(acur.cnt, acur.bnolt,
-                               acur.rec_bno, acur.rec_len, acur.bno, acur.len,
-                               0);
-               if (error)
-                       goto out;
                trace_xfs_alloc_near_first(args);
-               goto out;
+               goto alloc;
        }
 
        /*
@@ -1429,6 +1414,7 @@ restart:
                goto out;
        }
 
+alloc:
        args->agbno = acur.bno;
        args->len = acur.len;
        ASSERT(acur.bno >= acur.rec_bno);