diff options
author | J. Bruce Fields <bfields@redhat.com> | 2012-12-21 19:48:59 -0500 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-12-21 20:07:45 -0500 |
commit | 10532b560bacf23766f9c7dc09778b31b198ff45 (patch) | |
tree | 14eb44215dbcb3143222f3a865ae7f728a60628e | |
parent | 184e2516614f7055d4c3a2e63fd8a3eb95fff6d6 (diff) |
Revert "nfsd: warn on odd reply state in nfsd_vfs_read"
This reverts commit 79f77bf9a4e3dd5ead006b8f17e7c4ff07d8374e.
This is obviously wrong, and I have no idea how I missed seeing the
warning in testing: I must just not have looked at the right logs. The
caller bumps rq_resused/rq_next_page, so it will always be hit on a
large enough read.
Reported-by: Dave Jones <davej@redhat.com>
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | fs/nfsd/vfs.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/nfsd/vfs.c b/fs/nfsd/vfs.c index f0a6d88d7fff..d586117fa94a 100644 --- a/fs/nfsd/vfs.c +++ b/fs/nfsd/vfs.c | |||
@@ -934,7 +934,6 @@ nfsd_vfs_read(struct svc_rqst *rqstp, struct svc_fh *fhp, struct file *file, | |||
934 | .u.data = rqstp, | 934 | .u.data = rqstp, |
935 | }; | 935 | }; |
936 | 936 | ||
937 | WARN_ON_ONCE(rqstp->rq_next_page != rqstp->rq_respages + 1); | ||
938 | rqstp->rq_next_page = rqstp->rq_respages + 1; | 937 | rqstp->rq_next_page = rqstp->rq_respages + 1; |
939 | host_err = splice_direct_to_actor(file, &sd, nfsd_direct_splice_actor); | 938 | host_err = splice_direct_to_actor(file, &sd, nfsd_direct_splice_actor); |
940 | } else { | 939 | } else { |