xfs: refactor creation of bmap btree roots

Now that we've created inode fork helpers to allocate and free btree
roots, create a new bmap btree helper to create a new bmbt root, and
refactor the extents <-> btree conversion functions to use our new
helpers.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>

diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index 5dcb736f07781aa55c1a4079856296195bd12fd9..6a26ff1689db8976532d53f9b000d8ef36db77b5 100644 (file)
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -615,7 +615,7 @@ xfs_bmap_btree_to_extents(
        xfs_trans_binval(tp, cbp);
        if (cur->bc_levels[0].bp == cbp)
                cur->bc_levels[0].bp = NULL;
-       xfs_iroot_realloc(ip, -1, whichfork);
+       xfs_iroot_free(ip, whichfork);
        ASSERT(ifp->if_broot == NULL);
        ifp->if_format = XFS_DINODE_FMT_EXTENTS;
        *logflagsp |= XFS_ILOG_CORE | xfs_ilog_fext(whichfork);
@@ -656,19 +656,10 @@ xfs_bmap_extents_to_btree(
        ASSERT(ifp->if_format == XFS_DINODE_FMT_EXTENTS);
 
        /*
-        * Make space in the inode incore. This needs to be undone if we fail
-        * to expand the root.
-        */
-       xfs_iroot_realloc(ip, 1, whichfork);
-
-       /*
-        * Fill in the root.
-        */
-       block = ifp->if_broot;
-       xfs_bmbt_init_block(ip, block, NULL, 1, 1);
-       /*
-        * Need a cursor.  Can't allocate until bb_level is filled in.
+        * Fill in the root, create a cursor.  Can't allocate until bb_level is
+        * filled in.
         */
+       xfs_bmbt_iroot_alloc(ip, whichfork);
        cur = xfs_bmbt_init_cursor(mp, tp, ip, whichfork);
        if (wasdel)
                cur->bc_flags |= XFS_BTREE_BMBT_WASDEL;
@@ -725,6 +716,7 @@ xfs_bmap_extents_to_btree(
        /*
         * Fill in the root key and pointer.
         */
+       block = ifp->if_broot;
        kp = xfs_bmbt_key_addr(mp, block, 1);
        arp = xfs_bmbt_rec_addr(mp, ablock, 1);
        kp->br_startoff = cpu_to_be64(xfs_bmbt_disk_get_startoff(arp));
@@ -746,7 +738,7 @@ xfs_bmap_extents_to_btree(
 out_unreserve_dquot:
        xfs_trans_mod_dquot_byino(tp, ip, XFS_TRANS_DQ_BCOUNT, -1L);
 out_root_realloc:
-       xfs_iroot_realloc(ip, -1, whichfork);
+       xfs_iroot_free(ip, whichfork);
        ifp->if_format = XFS_DINODE_FMT_EXTENTS;
        ASSERT(ifp->if_broot == NULL);
        xfs_btree_del_cursor(cur, XFS_BTREE_ERROR);

diff --git a/fs/xfs/libxfs/xfs_bmap_btree.c b/fs/xfs/libxfs/xfs_bmap_btree.c
index 3695b3ad07d4d3f4a531d16953ad0ddc7e672aea..bbcf099d8cf396df871f9207fff36b319d29773f 100644 (file)
--- a/fs/xfs/libxfs/xfs_bmap_btree.c
+++ b/fs/xfs/libxfs/xfs_bmap_btree.c
@@ -759,3 +759,18 @@ xfs_bmbt_destroy_cur_cache(void)
        kmem_cache_destroy(xfs_bmbt_cur_cache);
        xfs_bmbt_cur_cache = NULL;
 }
+
+/* Create an incore bmbt btree root block. */
+void
+xfs_bmbt_iroot_alloc(
+       struct xfs_inode        *ip,
+       int                     whichfork)
+{
+       struct xfs_ifork        *ifp = xfs_ifork_ptr(ip, whichfork);
+
+       xfs_iroot_alloc(ip, whichfork,
+                       xfs_bmap_broot_space_calc(ip->i_mount, 1));
+
+       /* Fill in the root. */
+       xfs_bmbt_init_block(ip, ifp->if_broot, NULL, 1, 1);
+}

diff --git a/fs/xfs/libxfs/xfs_bmap_btree.h b/fs/xfs/libxfs/xfs_bmap_btree.h
index af47174fca084ef97150cd8b5ada4816da11f2b5..921c3e02469dd0ef1afcf2d32b8c48bed74de7de 100644 (file)
--- a/fs/xfs/libxfs/xfs_bmap_btree.h
+++ b/fs/xfs/libxfs/xfs_bmap_btree.h
@@ -197,4 +197,6 @@ xfs_bmap_bmdr_space(struct xfs_btree_block *bb)
        return xfs_bmdr_space_calc(be16_to_cpu(bb->bb_numrecs));
 }
 
+void xfs_bmbt_iroot_alloc(struct xfs_inode *ip, int whichfork);
+
 #endif /* __XFS_BMAP_BTREE_H__ */