xfs: rename bno to end in __xfs_bunmapi
Rename the bno variable that's used as the end of the range in __xfs_bunmapi to end, which better describes it. Additionally change the start variable which takes the initial value of bno to be the function parameter itself. 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
b213d69293
commit
8280f6ed46
|
@ -5408,7 +5408,7 @@ int /* error */
|
|||
__xfs_bunmapi(
|
||||
xfs_trans_t *tp, /* transaction pointer */
|
||||
struct xfs_inode *ip, /* incore inode */
|
||||
xfs_fileoff_t bno, /* starting offset to unmap */
|
||||
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 */
|
||||
|
@ -5427,7 +5427,6 @@ __xfs_bunmapi(
|
|||
int logflags; /* transaction logging flags */
|
||||
xfs_extlen_t mod; /* rt extent offset */
|
||||
xfs_mount_t *mp; /* mount structure */
|
||||
xfs_fileoff_t start; /* first file offset deleted */
|
||||
int tmp_logflags; /* partial logging flags */
|
||||
int wasdel; /* was a delayed alloc extent */
|
||||
int whichfork; /* data or attribute fork */
|
||||
|
@ -5435,8 +5434,9 @@ __xfs_bunmapi(
|
|||
xfs_filblks_t len = *rlen; /* length to unmap in file */
|
||||
xfs_fileoff_t max_len;
|
||||
xfs_agnumber_t prev_agno = NULLAGNUMBER, agno;
|
||||
xfs_fileoff_t end;
|
||||
|
||||
trace_xfs_bunmap(ip, bno, len, flags, _RET_IP_);
|
||||
trace_xfs_bunmap(ip, start, len, flags, _RET_IP_);
|
||||
|
||||
whichfork = xfs_bmapi_whichfork(flags);
|
||||
ASSERT(whichfork != XFS_COW_FORK);
|
||||
|
@ -5475,17 +5475,16 @@ __xfs_bunmapi(
|
|||
}
|
||||
XFS_STATS_INC(mp, xs_blk_unmap);
|
||||
isrt = (whichfork == XFS_DATA_FORK) && XFS_IS_REALTIME_INODE(ip);
|
||||
start = bno;
|
||||
bno = start + len - 1;
|
||||
end = start + len - 1;
|
||||
|
||||
/*
|
||||
* Check to see if the given block number is past the end of the
|
||||
* file, back up to the last block if so...
|
||||
*/
|
||||
if (!xfs_iext_lookup_extent(ip, ifp, bno, &lastx, &got)) {
|
||||
if (!xfs_iext_lookup_extent(ip, ifp, end, &lastx, &got)) {
|
||||
ASSERT(lastx > 0);
|
||||
xfs_iext_get_extent(ifp, --lastx, &got);
|
||||
bno = got.br_startoff + got.br_blockcount - 1;
|
||||
end = got.br_startoff + got.br_blockcount - 1;
|
||||
}
|
||||
|
||||
logflags = 0;
|
||||
|
@ -5509,13 +5508,13 @@ __xfs_bunmapi(
|
|||
}
|
||||
|
||||
extno = 0;
|
||||
while (bno != (xfs_fileoff_t)-1 && bno >= start && lastx >= 0 &&
|
||||
while (end != (xfs_fileoff_t)-1 && end >= start && lastx >= 0 &&
|
||||
(nexts == 0 || extno < nexts) && max_len > 0) {
|
||||
/*
|
||||
* Is the found extent after a hole in which bno lives?
|
||||
* Is the found extent after a hole in which end lives?
|
||||
* Just back up to the previous extent, if so.
|
||||
*/
|
||||
if (got.br_startoff > bno) {
|
||||
if (got.br_startoff > end) {
|
||||
if (--lastx < 0)
|
||||
break;
|
||||
xfs_iext_get_extent(ifp, lastx, &got);
|
||||
|
@ -5524,9 +5523,9 @@ __xfs_bunmapi(
|
|||
* Is the last block of this extent before the range
|
||||
* we're supposed to delete? If so, we're done.
|
||||
*/
|
||||
bno = XFS_FILEOFF_MIN(bno,
|
||||
end = XFS_FILEOFF_MIN(end,
|
||||
got.br_startoff + got.br_blockcount - 1);
|
||||
if (bno < start)
|
||||
if (end < start)
|
||||
break;
|
||||
/*
|
||||
* Then deal with the (possibly delayed) allocated space
|
||||
|
@ -5551,8 +5550,8 @@ __xfs_bunmapi(
|
|||
if (!wasdel)
|
||||
del.br_startblock += start - got.br_startoff;
|
||||
}
|
||||
if (del.br_startoff + del.br_blockcount > bno + 1)
|
||||
del.br_blockcount = bno + 1 - del.br_startoff;
|
||||
if (del.br_startoff + del.br_blockcount > end + 1)
|
||||
del.br_blockcount = end + 1 - del.br_startoff;
|
||||
|
||||
/* How much can we safely unmap? */
|
||||
if (max_len < del.br_blockcount) {
|
||||
|
@ -5578,10 +5577,10 @@ __xfs_bunmapi(
|
|||
* This piece is unwritten, or we're not
|
||||
* using unwritten extents. Skip over it.
|
||||
*/
|
||||
ASSERT(bno >= mod);
|
||||
bno -= mod > del.br_blockcount ?
|
||||
ASSERT(end >= mod);
|
||||
end -= mod > del.br_blockcount ?
|
||||
del.br_blockcount : mod;
|
||||
if (bno < got.br_startoff) {
|
||||
if (end < got.br_startoff) {
|
||||
if (--lastx >= 0)
|
||||
xfs_iext_get_extent(ifp, lastx,
|
||||
&got);
|
||||
|
@ -5630,9 +5629,9 @@ __xfs_bunmapi(
|
|||
* Can't make it unwritten. There isn't
|
||||
* a full extent here so just skip it.
|
||||
*/
|
||||
ASSERT(bno >= del.br_blockcount);
|
||||
bno -= del.br_blockcount;
|
||||
if (got.br_startoff > bno && --lastx >= 0)
|
||||
ASSERT(end >= del.br_blockcount);
|
||||
end -= del.br_blockcount;
|
||||
if (got.br_startoff > end && --lastx >= 0)
|
||||
xfs_iext_get_extent(ifp, lastx, &got);
|
||||
continue;
|
||||
} else if (del.br_state == XFS_EXT_UNWRITTEN) {
|
||||
|
@ -5735,24 +5734,24 @@ __xfs_bunmapi(
|
|||
xfs_mod_fdblocks(mp, (int64_t)del.br_blockcount, false);
|
||||
|
||||
max_len -= del.br_blockcount;
|
||||
bno = del.br_startoff - 1;
|
||||
end = del.br_startoff - 1;
|
||||
nodelete:
|
||||
/*
|
||||
* If not done go on to the next (previous) record.
|
||||
*/
|
||||
if (bno != (xfs_fileoff_t)-1 && bno >= start) {
|
||||
if (end != (xfs_fileoff_t)-1 && end >= start) {
|
||||
if (lastx >= 0) {
|
||||
xfs_iext_get_extent(ifp, lastx, &got);
|
||||
if (got.br_startoff > bno && --lastx >= 0)
|
||||
if (got.br_startoff > end && --lastx >= 0)
|
||||
xfs_iext_get_extent(ifp, lastx, &got);
|
||||
}
|
||||
extno++;
|
||||
}
|
||||
}
|
||||
if (bno == (xfs_fileoff_t)-1 || bno < start || lastx < 0)
|
||||
if (end == (xfs_fileoff_t)-1 || end < start || lastx < 0)
|
||||
*rlen = 0;
|
||||
else
|
||||
*rlen = bno - start + 1;
|
||||
*rlen = end - start + 1;
|
||||
|
||||
/*
|
||||
* Convert to a btree if necessary.
|
||||
|
|
Loading…
Reference in a new issue