summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristoph Hellwig <hch@infradead.org>2011-09-18 20:40:43 +0000
committerAlex Elder <aelder@sgi.com>2011-10-11 21:15:03 -0500
commitecee76ba9d91fdcbdff933ca1bd41465ca4c4fdb (patch)
tree70baed19a4cab18ce053e961a16ba313c3bd4220
parentb9b984d7846e37c57e5b3f8cd883ad45e8ebc2cf (diff)
downloadop-kernel-dev-ecee76ba9d91fdcbdff933ca1bd41465ca4c4fdb.zip
op-kernel-dev-ecee76ba9d91fdcbdff933ca1bd41465ca4c4fdb.tar.gz
xfs: remove the nextents variable in xfs_bmapi
Instead of using a local variable that needs to updated when we modify the extent map just check ifp->if_bytes directly where we use it. Signed-off-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Alex Elder <aelder@sgi.com>
-rw-r--r--fs/xfs/xfs_bmap.c6
1 files changed, 1 insertions, 5 deletions
diff --git a/fs/xfs/xfs_bmap.c b/fs/xfs/xfs_bmap.c
index ebcd38b..05d38adb 100644
--- a/fs/xfs/xfs_bmap.c
+++ b/fs/xfs/xfs_bmap.c
@@ -4294,7 +4294,6 @@ xfs_bmapi(
xfs_mount_t *mp; /* xfs mount structure */
int n; /* current extent index */
int nallocs; /* number of extents alloc'd */
- xfs_extnum_t nextents; /* number of extents in file */
xfs_fileoff_t obno; /* old block number (offset) */
xfs_bmbt_irec_t prev; /* previous file extent record */
int tmp_logflags; /* temp flags holder */
@@ -4380,7 +4379,6 @@ xfs_bmapi(
goto error0;
ep = xfs_bmap_search_extents(ip, bno, whichfork, &eof, &lastx, &got,
&prev);
- nextents = ifp->if_bytes / (uint)sizeof(xfs_bmbt_rec_t);
n = 0;
end = bno + len;
obno = bno;
@@ -4622,7 +4620,6 @@ xfs_bmapi(
if (error)
goto error0;
ep = xfs_iext_get_ext(ifp, lastx);
- nextents = ifp->if_bytes / (uint)sizeof(xfs_bmbt_rec_t);
xfs_bmbt_get_all(ep, &got);
ASSERT(got.br_startoff <= aoff);
ASSERT(got.br_startoff + got.br_blockcount >=
@@ -4723,7 +4720,6 @@ xfs_bmapi(
if (error)
goto error0;
ep = xfs_iext_get_ext(ifp, lastx);
- nextents = ifp->if_bytes / (uint)sizeof(xfs_bmbt_rec_t);
xfs_bmbt_get_all(ep, &got);
/*
* We may have combined previously unwritten
@@ -4781,7 +4777,7 @@ xfs_bmapi(
* Else go on to the next record.
*/
prev = got;
- if (++lastx < nextents) {
+ if (++lastx < ifp->if_bytes / sizeof(xfs_bmbt_rec_t)) {
ep = xfs_iext_get_ext(ifp, lastx);
xfs_bmbt_get_all(ep, &got);
} else {
OpenPOWER on IntegriCloud