xfs: remove xfs_bunmapi() firstblock param

All callers pass ->t_firstblock from the current transaction.

Signed-off-by: Brian Foster <bfoster@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>

diff --git a/fs/xfs/libxfs/xfs_attr_remote.c b/fs/xfs/libxfs/xfs_attr_remote.c
index 2db9ef186e052a14e4e1a36f4eb9478457a3d09c..f02c705965fffc206d06ed5dc8b8d6f5624ef6bd 100644 (file)
--- a/fs/xfs/libxfs/xfs_attr_remote.c
+++ b/fs/xfs/libxfs/xfs_attr_remote.c
@@ -631,8 +631,7 @@ xfs_attr_rmtval_remove(
                xfs_defer_init(args->trans, args->trans->t_dfops,
                               &args->trans->t_firstblock);
                error = xfs_bunmapi(args->trans, args->dp, lblkno, blkcnt,
-                                   XFS_BMAPI_ATTRFORK, 1,
-                                   &args->trans->t_firstblock, &done);
+                                   XFS_BMAPI_ATTRFORK, 1, &done);
                if (error)
                        goto out_defer_cancel;
                xfs_defer_ijoin(args->trans->t_dfops, args->dp);

diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index ca30e972288afb04214b9b5754d0b01ac3d2682c..619d3adc592321e460f6954911940e9054ba7a3a 100644 (file)
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -5122,9 +5122,7 @@ __xfs_bunmapi(
        xfs_fileoff_t           start,          /* first file offset deleted */
        xfs_filblks_t           *rlen,          /* i/o: amount remaining */
        int                     flags,          /* misc flags */
-       xfs_extnum_t            nexts,          /* number of extents max */
-       xfs_fsblock_t           *firstblock)    /* first allocated block
-                                                  controls a.g. for allocs */
+       xfs_extnum_t            nexts)          /* number of extents max */
 {
        struct xfs_btree_cur    *cur;           /* bmap btree cursor */
        struct xfs_bmbt_irec    del;            /* extent being deleted */
@@ -5198,7 +5196,7 @@ __xfs_bunmapi(
        if (ifp->if_flags & XFS_IFBROOT) {
                ASSERT(XFS_IFORK_FORMAT(ip, whichfork) == XFS_DINODE_FMT_BTREE);
                cur = xfs_bmbt_init_cursor(mp, tp, ip, whichfork);
-               cur->bc_private.b.firstblock = *firstblock;
+               cur->bc_private.b.firstblock = tp->t_firstblock;
                cur->bc_private.b.flags = 0;
        } else
                cur = NULL;
@@ -5314,7 +5312,7 @@ __xfs_bunmapi(
                        del.br_state = XFS_EXT_UNWRITTEN;
                        error = xfs_bmap_add_extent_unwritten_real(tp, ip,
                                        whichfork, &icur, &cur, &del,
-                                       firstblock, &logflags);
+                                       &tp->t_firstblock, &logflags);
                        if (error)
                                goto error0;
                        goto nodelete;
@@ -5371,7 +5369,8 @@ __xfs_bunmapi(
                                prev.br_state = XFS_EXT_UNWRITTEN;
                                error = xfs_bmap_add_extent_unwritten_real(tp,
                                                ip, whichfork, &icur, &cur,
-                                               &prev, firstblock, &logflags);
+                                               &prev, &tp->t_firstblock,
+                                               &logflags);
                                if (error)
                                        goto error0;
                                goto nodelete;
@@ -5380,7 +5379,8 @@ __xfs_bunmapi(
                                del.br_state = XFS_EXT_UNWRITTEN;
                                error = xfs_bmap_add_extent_unwritten_real(tp,
                                                ip, whichfork, &icur, &cur,
-                                               &del, firstblock, &logflags);
+                                               &del, &tp->t_firstblock,
+                                               &logflags);
                                if (error)
                                        goto error0;
                                goto nodelete;
@@ -5427,8 +5427,8 @@ nodelete:
         */
        if (xfs_bmap_needs_btree(ip, whichfork)) {
                ASSERT(cur == NULL);
-               error = xfs_bmap_extents_to_btree(tp, ip, firstblock, &cur, 0,
-                               &tmp_logflags, whichfork);
+               error = xfs_bmap_extents_to_btree(tp, ip, &tp->t_firstblock,
+                               &cur, 0, &tmp_logflags, whichfork);
                logflags |= tmp_logflags;
                if (error)
                        goto error0;
@@ -5467,7 +5467,7 @@ error0:
                xfs_trans_log_inode(tp, ip, logflags);
        if (cur) {
                if (!error) {
-                       *firstblock = cur->bc_private.b.firstblock;
+                       tp->t_firstblock = cur->bc_private.b.firstblock;
                        cur->bc_private.b.allocated = 0;
                }
                xfs_btree_del_cursor(cur,
@@ -5485,12 +5485,11 @@ xfs_bunmapi(
        xfs_filblks_t           len,
        int                     flags,
        xfs_extnum_t            nexts,
-       xfs_fsblock_t           *firstblock,
        int                     *done)
 {
        int                     error;
 
-       error = __xfs_bunmapi(tp, ip, bno, &len, flags, nexts, firstblock);
+       error = __xfs_bunmapi(tp, ip, bno, &len, flags, nexts);
        *done = (len == 0);
        return error;
 }
@@ -6148,7 +6147,7 @@ xfs_bmap_finish_one(
                break;
        case XFS_BMAP_UNMAP:
                error = __xfs_bunmapi(tp, ip, startoff, blockcount,
-                               XFS_BMAPI_REMAP, 1, &tp->t_firstblock);
+                               XFS_BMAPI_REMAP, 1);
                break;
        default:
                ASSERT(0);

diff --git a/fs/xfs/libxfs/xfs_bmap.h b/fs/xfs/libxfs/xfs_bmap.h
index 88c2b5dc499edf1c6fedacd611d8661f815b2938..108a3073d6589cdb1c5e47e3c8a7c14ba058816c 100644 (file)
--- a/fs/xfs/libxfs/xfs_bmap.h
+++ b/fs/xfs/libxfs/xfs_bmap.h
@@ -204,11 +204,10 @@ int       xfs_bmapi_write(struct xfs_trans *tp, struct xfs_inode *ip,
                xfs_extlen_t total, struct xfs_bmbt_irec *mval, int *nmap);
 int    __xfs_bunmapi(struct xfs_trans *tp, struct xfs_inode *ip,
                xfs_fileoff_t bno, xfs_filblks_t *rlen, int flags,
-               xfs_extnum_t nexts, xfs_fsblock_t *firstblock);
+               xfs_extnum_t nexts);
 int    xfs_bunmapi(struct xfs_trans *tp, struct xfs_inode *ip,
                xfs_fileoff_t bno, xfs_filblks_t len, int flags,
-               xfs_extnum_t nexts, xfs_fsblock_t *firstblock,
-               int *done);
+               xfs_extnum_t nexts, int *done);
 int    xfs_bmap_del_extent_delay(struct xfs_inode *ip, int whichfork,
                struct xfs_iext_cursor *cur, struct xfs_bmbt_irec *got,
                struct xfs_bmbt_irec *del);

diff --git a/fs/xfs/libxfs/xfs_da_btree.c b/fs/xfs/libxfs/xfs_da_btree.c
index fe4a192696ae2d5b61a5cc0943e77f7ff17dfff1..9efbd2038ffbf0a06143716d045ab17c03bf5f9f 100644 (file)
--- a/fs/xfs/libxfs/xfs_da_btree.c
+++ b/fs/xfs/libxfs/xfs_da_btree.c
@@ -2392,8 +2392,7 @@ xfs_da_shrink_inode(
                 * the last block to the place we want to kill.
                 */
                error = xfs_bunmapi(tp, dp, dead_blkno, count,
-                                   xfs_bmapi_aflag(w), 0, &tp->t_firstblock,
-                                   &done);
+                                   xfs_bmapi_aflag(w), 0, &done);
                if (error == -ENOSPC) {
                        if (w != XFS_DATA_FORK)
                                break;

diff --git a/fs/xfs/libxfs/xfs_dir2.c b/fs/xfs/libxfs/xfs_dir2.c
index 5db73d96b99ed4475db70aa39be02d4578af0d9f..4ea1fddb126f69801f52cc54824dc1b8604492eb 100644 (file)
--- a/fs/xfs/libxfs/xfs_dir2.c
+++ b/fs/xfs/libxfs/xfs_dir2.c
@@ -656,8 +656,7 @@ xfs_dir2_shrink_inode(
        da = xfs_dir2_db_to_da(args->geo, db);
 
        /* Unmap the fsblock(s). */
-       error = xfs_bunmapi(tp, dp, da, args->geo->fsbcount, 0, 0,
-                           &tp->t_firstblock, &done);
+       error = xfs_bunmapi(tp, dp, da, args->geo->fsbcount, 0, 0, &done);
        if (error) {
                /*
                 * ENOSPC actually can happen if we're in a removename with no

diff --git a/fs/xfs/xfs_bmap_util.c b/fs/xfs/xfs_bmap_util.c
index 4f4b1d3fb898bdb4a458d7f395395deefa98001a..f225707c89beefd511c780c0198e587c2a716cc0 100644 (file)
--- a/fs/xfs/xfs_bmap_util.c
+++ b/fs/xfs/xfs_bmap_util.c
@@ -1041,8 +1041,7 @@ xfs_unmap_extent(
        xfs_trans_ijoin(tp, ip, 0);
 
        xfs_defer_init(tp, &dfops, &tp->t_firstblock);
-       error = xfs_bunmapi(tp, ip, startoffset_fsb, len_fsb, 0, 2,
-                           &tp->t_firstblock, done);
+       error = xfs_bunmapi(tp, ip, startoffset_fsb, len_fsb, 0, 2, done);
        if (error)
                goto out_bmap_cancel;
 

diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c
index 3cdfd795a50c2ea913c602d21f240045e7b7d2a5..5e1eaa26435ce564cdbd541bce58781415253997 100644 (file)
--- a/fs/xfs/xfs_inode.c
+++ b/fs/xfs/xfs_inode.c
@@ -1581,8 +1581,7 @@ xfs_itruncate_extents_flags(
        while (!done) {
                xfs_defer_init(tp, &dfops, &tp->t_firstblock);
                error = xfs_bunmapi(tp, ip, first_unmap_block, unmap_len, flags,
-                                   XFS_ITRUNC_MAX_EXTENTS, &tp->t_firstblock,
-                                   &done);
+                                   XFS_ITRUNC_MAX_EXTENTS, &done);
                if (error)
                        goto out_bmap_cancel;
 

diff --git a/fs/xfs/xfs_reflink.c b/fs/xfs/xfs_reflink.c
index 7010f3453c29e6c1cc974eb2fe5f43d7e6d6af6f..2972efeee5ccc1a32eccea85679613f65813df15 100644 (file)
--- a/fs/xfs/xfs_reflink.c
+++ b/fs/xfs/xfs_reflink.c
@@ -696,8 +696,7 @@ xfs_reflink_end_cow(
                /* Unmap the old blocks in the data fork. */
                xfs_defer_init(tp, &dfops, &tp->t_firstblock);
                rlen = del.br_blockcount;
-               error = __xfs_bunmapi(tp, ip, del.br_startoff, &rlen, 0, 1,
-                               &tp->t_firstblock);
+               error = __xfs_bunmapi(tp, ip, del.br_startoff, &rlen, 0, 1);
                if (error)
                        goto out_defer;
 
@@ -1044,8 +1043,7 @@ xfs_reflink_remap_extent(
        rlen = unmap_len;
        while (rlen) {
                xfs_defer_init(tp, &dfops, &tp->t_firstblock);
-               error = __xfs_bunmapi(tp, ip, destoff, &rlen, 0, 1,
-                                     &tp->t_firstblock);
+               error = __xfs_bunmapi(tp, ip, destoff, &rlen, 0, 1);
                if (error)
                        goto out_defer;
 

diff --git a/fs/xfs/xfs_symlink.c b/fs/xfs/xfs_symlink.c
index 94301b63525f24e9e195f1e97ec85f600b025794..a3dc552a5b9740969aadfb7f77358a090d640912 100644 (file)
--- a/fs/xfs/xfs_symlink.c
+++ b/fs/xfs/xfs_symlink.c
@@ -460,8 +460,7 @@ xfs_inactive_symlink_rmt(
        /*
         * Unmap the dead block(s) to the dfops.
         */
-       error = xfs_bunmapi(tp, ip, 0, size, 0, nmaps, &tp->t_firstblock,
-                           &done);
+       error = xfs_bunmapi(tp, ip, 0, size, 0, nmaps, &done);
        if (error)
                goto error_bmap_cancel;
        ASSERT(done);