diff options
| author |   Darrick J. Wong <darrick.wong@oracle.com> | 2017-12-10 18:03:53 -0800 |
|---|---|---|
| committer | Darrick J. Wong <darrick.wong@oracle.com> | 2017-12-14 09:20:10 -0800 |
| commit | 8c57b88637d78a723e0854fc3d06c6d4c31a1e0c (patch) | |
| tree | 9f071dde548f79a51ccbd2ba8b9bd764bbb6e493 /fs | |
| parent | xfs: hold xfs_buf locked between shortform->leaf conversion and the addition of an attribute (diff) | |
| download | linux-dev-8c57b88637d78a723e0854fc3d06c6d4c31a1e0c.tar.xz linux-dev-8c57b88637d78a723e0854fc3d06c6d4c31a1e0c.zip | |
xfs: account for null transactions in bunmapi
In e1a4e37cc7b665 ("xfs: try to avoid blowing out the transaction
reservation when bunmaping a shared extent"), we try to constrain the
amount of real extents we unmap from the data fork in a given call so
that we don't blow out transaction reservations.
However, not all bunmapi operations require a transaction -- if we're
only removing a delalloc extent, no transaction is needed, so we have to
code against that.
Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'fs')
| -rw-r--r-- | fs/xfs/libxfs/xfs_bmap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bmap.c index 1210f684d3c2..1bddbba6b80c 100644 --- a/fs/xfs/libxfs/xfs_bmap.c +++ b/fs/xfs/libxfs/xfs_bmap.c @@ -5136,7 +5136,7 @@ __xfs_bunmapi( * blowing out the transaction with a mix of EFIs and reflink * adjustments. */ - if (xfs_is_reflink_inode(ip) && whichfork == XFS_DATA_FORK) + if (tp && xfs_is_reflink_inode(ip) && whichfork == XFS_DATA_FORK) max_len = min(len, xfs_refcount_max_unmap(tp->t_log_res)); else max_len = len; |