xfs: rename bno to end in __xfs_bunmapi
authorChristoph Hellwig <hch@lst.de>
Tue, 17 Oct 2017 21:16:21 +0000 (14:16 -0700)
committerDarrick J. Wong <darrick.wong@oracle.com>
Thu, 26 Oct 2017 22:38:20 +0000 (15:38 -0700)
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>
fs/xfs/libxfs/xfs_bmap.c

index 9db566b7e45e547d1bd11a3e6e5087fdc419fab9..f4469e542c18cca70f9fd7889797bdc27d3579d5 100644 (file)
@@ -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.