]> git.apps.os.sepia.ceph.com Git - xfsprogs-dev.git/commitdiff
xfs: remove the unnecessary daddr paramter to _init_block
authorDarrick J. Wong <djwong@kernel.org>
Mon, 22 Apr 2024 17:00:59 +0000 (10:00 -0700)
committerDarrick J. Wong <djwong@kernel.org>
Mon, 3 Jun 2024 18:37:37 +0000 (11:37 -0700)
Source kernel commit: 11388f6581f40e7d5a69ce5f8b13264eca7c2c5c

Now that all of the callers pass XFS_BUF_DADDR_NULL as the daddr
parameter, we can elide that too.

Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
libxfs/xfs_bmap_btree.c
libxfs/xfs_btree.c
libxfs/xfs_btree.h
libxfs/xfs_btree_staging.c

index 65ba3ae8a5499c32da31e59aba4d96afae401876..2d84118099d8fc9a604e05c6e2327ba141378281 100644 (file)
@@ -37,8 +37,8 @@ xfs_bmbt_init_block(
                xfs_btree_init_buf(ip->i_mount, bp, &xfs_bmbt_ops, level,
                                numrecs, ip->i_ino);
        else
-               xfs_btree_init_block(ip->i_mount, buf, &xfs_bmbt_ops,
-                               XFS_BUF_DADDR_NULL, level, numrecs, ip->i_ino);
+               xfs_btree_init_block(ip->i_mount, buf, &xfs_bmbt_ops, level,
+                               numrecs, ip->i_ino);
 }
 
 /*
index 372a521c1af8dac47858fe710e2e1c361c6a4191..2386084a531dff9640d880489ae21a68c4a67244 100644 (file)
@@ -1150,8 +1150,8 @@ xfs_btree_set_sibling(
        }
 }
 
-void
-xfs_btree_init_block(
+static void
+__xfs_btree_init_block(
        struct xfs_mount        *mp,
        struct xfs_btree_block  *buf,
        const struct xfs_btree_ops *ops,
@@ -1192,6 +1192,19 @@ xfs_btree_init_block(
        }
 }
 
+void
+xfs_btree_init_block(
+       struct xfs_mount        *mp,
+       struct xfs_btree_block  *block,
+       const struct xfs_btree_ops *ops,
+       __u16                   level,
+       __u16                   numrecs,
+       __u64                   owner)
+{
+       __xfs_btree_init_block(mp, block, ops, XFS_BUF_DADDR_NULL, level,
+                       numrecs, owner);
+}
+
 void
 xfs_btree_init_buf(
        struct xfs_mount                *mp,
@@ -1201,7 +1214,7 @@ xfs_btree_init_buf(
        __u16                           numrecs,
        __u64                           owner)
 {
-       xfs_btree_init_block(mp, XFS_BUF_TO_BLOCK(bp), ops,
+       __xfs_btree_init_block(mp, XFS_BUF_TO_BLOCK(bp), ops,
                        xfs_buf_daddr(bp), level, numrecs, owner);
 }
 
index 56901d2591ed73b2ba8b3a05400ed3ceea092dfa..80be40ca89547c363bc8362970dc0a155b956b55 100644 (file)
@@ -440,7 +440,7 @@ void xfs_btree_init_buf(struct xfs_mount *mp, struct xfs_buf *bp,
                __u64 owner);
 void xfs_btree_init_block(struct xfs_mount *mp,
                struct xfs_btree_block *buf, const struct xfs_btree_ops *ops,
-               xfs_daddr_t blkno, __u16 level, __u16 numrecs, __u64 owner);
+               __u16 level, __u16 numrecs, __u64 owner);
 
 /*
  * Common btree core entry points.
index 47ef8e23a59e40bf9cfbcbf6d146301a9156269e..39e95a771c3b52ca22a24b3e28bd3fffd0fca4e8 100644 (file)
@@ -410,9 +410,8 @@ xfs_btree_bload_prep_block(
                ifp->if_broot_bytes = (int)new_size;
 
                /* Initialize it and send it out. */
-               xfs_btree_init_block(cur->bc_mp, ifp->if_broot,
-                               cur->bc_ops, XFS_BUF_DADDR_NULL, level,
-                               nr_this_block, cur->bc_ino.ip->i_ino);
+               xfs_btree_init_block(cur->bc_mp, ifp->if_broot, cur->bc_ops,
+                               level, nr_this_block, cur->bc_ino.ip->i_ino);
 
                *bpp = NULL;
                *blockp = ifp->if_broot;