xfs: move some more code into xfs_bmap_del_extent_real
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>
This commit is contained in:
parent
e1d7553faf
commit
1b24b633aa
|
@ -5078,7 +5078,7 @@ xfs_bmap_del_extent_real(
|
||||||
int do_fx; /* free extent at end of routine */
|
int do_fx; /* free extent at end of routine */
|
||||||
xfs_bmbt_rec_host_t *ep; /* current extent entry pointer */
|
xfs_bmbt_rec_host_t *ep; /* current extent entry pointer */
|
||||||
int error; /* error return value */
|
int error; /* error return value */
|
||||||
int flags; /* inode logging flags */
|
int flags = 0;/* inode logging flags */
|
||||||
xfs_bmbt_irec_t got; /* current extent entry */
|
xfs_bmbt_irec_t got; /* current extent entry */
|
||||||
xfs_fileoff_t got_endoff; /* first offset past got */
|
xfs_fileoff_t got_endoff; /* first offset past got */
|
||||||
int i; /* temp state */
|
int i; /* temp state */
|
||||||
|
@ -5110,10 +5110,25 @@ xfs_bmap_del_extent_real(
|
||||||
got_endoff = got.br_startoff + got.br_blockcount;
|
got_endoff = got.br_startoff + got.br_blockcount;
|
||||||
ASSERT(got_endoff >= del_endoff);
|
ASSERT(got_endoff >= del_endoff);
|
||||||
ASSERT(!isnullstartblock(got.br_startblock));
|
ASSERT(!isnullstartblock(got.br_startblock));
|
||||||
flags = XFS_ILOG_CORE;
|
|
||||||
qfield = 0;
|
qfield = 0;
|
||||||
error = 0;
|
error = 0;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* If it's the case where the directory code is running with no block
|
||||||
|
* reservation, and the deleted block is in the middle of its extent,
|
||||||
|
* and the resulting insert of an extent would cause transformation to
|
||||||
|
* btree format, then reject it. The calling code will then swap blocks
|
||||||
|
* around instead. We have to do this now, rather than waiting for the
|
||||||
|
* conversion to btree format, since the transaction will be dirty then.
|
||||||
|
*/
|
||||||
|
if (tp->t_blk_res == 0 &&
|
||||||
|
XFS_IFORK_FORMAT(ip, whichfork) == XFS_DINODE_FMT_EXTENTS &&
|
||||||
|
XFS_IFORK_NEXTENTS(ip, whichfork) >=
|
||||||
|
XFS_IFORK_MAXEXT(ip, whichfork) &&
|
||||||
|
del->br_startoff > got.br_startoff && del_endoff < got_endoff)
|
||||||
|
return -ENOSPC;
|
||||||
|
|
||||||
|
flags = XFS_ILOG_CORE;
|
||||||
if (whichfork == XFS_DATA_FORK && XFS_IS_REALTIME_INODE(ip)) {
|
if (whichfork == XFS_DATA_FORK && XFS_IS_REALTIME_INODE(ip)) {
|
||||||
xfs_fsblock_t bno;
|
xfs_fsblock_t bno;
|
||||||
xfs_filblks_t len;
|
xfs_filblks_t len;
|
||||||
|
@ -5599,28 +5614,6 @@ __xfs_bunmapi(
|
||||||
error = xfs_bmap_del_extent_delay(ip, whichfork, &lastx,
|
error = xfs_bmap_del_extent_delay(ip, whichfork, &lastx,
|
||||||
&got, &del);
|
&got, &del);
|
||||||
} else {
|
} else {
|
||||||
/*
|
|
||||||
* If it's the case where the directory code is running
|
|
||||||
* with no block reservation, and the deleted block is
|
|
||||||
* in the middle of its extent, and the resulting insert
|
|
||||||
* of an extent would cause transformation to btree
|
|
||||||
* format, then reject it. The calling code will then
|
|
||||||
* swap blocks around instead. We have to do this now,
|
|
||||||
* rather than waiting for the conversion to btree
|
|
||||||
* format, since the transaction will be dirty.
|
|
||||||
*/
|
|
||||||
if (tp->t_blk_res == 0 &&
|
|
||||||
XFS_IFORK_FORMAT(ip, whichfork) ==
|
|
||||||
XFS_DINODE_FMT_EXTENTS &&
|
|
||||||
XFS_IFORK_NEXTENTS(ip, whichfork) >=
|
|
||||||
XFS_IFORK_MAXEXT(ip, whichfork) &&
|
|
||||||
del.br_startoff > got.br_startoff &&
|
|
||||||
del.br_startoff + del.br_blockcount <
|
|
||||||
got.br_startoff + got.br_blockcount) {
|
|
||||||
error = -ENOSPC;
|
|
||||||
goto error0;
|
|
||||||
}
|
|
||||||
|
|
||||||
error = xfs_bmap_del_extent_real(ip, tp, &lastx, dfops,
|
error = xfs_bmap_del_extent_real(ip, tp, &lastx, dfops,
|
||||||
cur, &del, &tmp_logflags, whichfork,
|
cur, &del, &tmp_logflags, whichfork,
|
||||||
flags);
|
flags);
|
||||||
|
|
Loading…
Reference in New Issue