summaryrefslogtreecommitdiffstats
path: root/fs/xfs/libxfs/xfs_bmap.c
diff options
context:
space:
mode:
authorChristoph Hellwig2017-10-17 23:16:20 +0200
committerDarrick J. Wong2017-10-27 00:38:20 +0200
commitb213d69293cf003e6f0c63adfee6fcc70e3afee6 (patch)
treed4ca773f9f0bc60a7f77c590681a7ad94d0850b2 /fs/xfs/libxfs/xfs_bmap.c
parentxfs: use xfs_iext_get_extent instead of open coding it (diff)
downloadkernel-qcow2-linux-b213d69293cf003e6f0c63adfee6fcc70e3afee6.tar.gz
kernel-qcow2-linux-b213d69293cf003e6f0c63adfee6fcc70e3afee6.tar.xz
kernel-qcow2-linux-b213d69293cf003e6f0c63adfee6fcc70e3afee6.zip
xfs: don't set XFS_BTCUR_BPRV_WASDEL in xfs_bunmapi
The XFS_BTCUR_BPRV_WASDEL flag is supposed to indicate that we are converting a delayed allocation to a real one, which isn't the case in xfs_bunmapi. Setting it could theoretically lead to misaccounting here, but it's unlikely that we ever hit it in practice. Signed-off-by: Christoph Hellwig <hch@lst.de> Reviewed-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>
Diffstat (limited to 'fs/xfs/libxfs/xfs_bmap.c')
-rw-r--r--fs/xfs/libxfs/xfs_bmap.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c
index e7146026e8f6..9db566b7e45e 100644
--- a/fs/xfs/libxfs/xfs_bmap.c
+++ b/fs/xfs/libxfs/xfs_bmap.c
@@ -5723,11 +5723,7 @@ __xfs_bunmapi(
XFS_QMOPT_RES_REGBLKS);
}
ip->i_delayed_blks -= del.br_blockcount;
- if (cur)
- cur->bc_private.b.flags |=
- XFS_BTCUR_BPRV_WASDEL;
- } else if (cur)
- cur->bc_private.b.flags &= ~XFS_BTCUR_BPRV_WASDEL;
+ }
error = xfs_bmap_del_extent(ip, tp, &lastx, dfops, cur, &del,
&tmp_logflags, whichfork, flags);