summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChuck Lever <chuck.lever@oracle.com>2010-02-01 14:17:23 -0500
committerTrond Myklebust <Trond.Myklebust@netapp.com>2010-02-10 08:31:03 -0500
commit4184dcf2dbde481b34d370e1704f2b91a8c9f0d1 (patch)
treefed7dd67648a78c00bd687ed38bdd133fe45124a
parentc2459dc46269728e4a080ec8d5a316b2bba2e142 (diff)
downloadop-kernel-dev-4184dcf2dbde481b34d370e1704f2b91a8c9f0d1.zip
op-kernel-dev-4184dcf2dbde481b34d370e1704f2b91a8c9f0d1.tar.gz
NFS: Fix byte accounting for generic NFS reads
Currently, the NFS I/O counters count the number of bytes requested by applications, rather than the number of bytes actually read by the system calls. The number of bytes requested for reads is actually not that useful, because the value is usually a buffer size for reads. That is, that requested number is usually a maximum, and frequently doesn't reflect the actual number of bytes read. Signed-off-by: Chuck Lever <chuck.lever@oracle.com> Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
-rw-r--r--fs/nfs/file.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/nfs/file.c b/fs/nfs/file.c
index 57cf94f..7f4910c 100644
--- a/fs/nfs/file.c
+++ b/fs/nfs/file.c
@@ -262,9 +262,11 @@ nfs_file_read(struct kiocb *iocb, const struct iovec *iov,
(unsigned long) count, (unsigned long) pos);
result = nfs_revalidate_mapping(inode, iocb->ki_filp->f_mapping);
- nfs_add_stats(inode, NFSIOS_NORMALREADBYTES, count);
- if (!result)
+ if (!result) {
result = generic_file_aio_read(iocb, iov, nr_segs, pos);
+ if (result > 0)
+ nfs_add_stats(inode, NFSIOS_NORMALREADBYTES, result);
+ }
return result;
}
OpenPOWER on IntegriCloud