mirror of
https://github.com/raspberrypi/linux.git
synced 2025-12-06 10:00:17 +00:00
nfs: don't create zero-length requests
commit 149a4fddd0 upstream.
NFS doesn't expect requests with wb_bytes set to zero and may make
unexpected decisions about how to handle that request at the page IO layer.
Skip request creation if we won't have any wb_bytes in the request.
Signed-off-by: Benjamin Coddington <bcodding@redhat.com>
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Reviewed-by: Weston Andros Adamson <dros@primarydata.com>
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
committed by
popcornmix
parent
6b2badc8c4
commit
0de13b858a
@@ -1261,6 +1261,9 @@ int nfs_updatepage(struct file *file, struct page *page,
|
|||||||
dprintk("NFS: nfs_updatepage(%pD2 %d@%lld)\n",
|
dprintk("NFS: nfs_updatepage(%pD2 %d@%lld)\n",
|
||||||
file, count, (long long)(page_file_offset(page) + offset));
|
file, count, (long long)(page_file_offset(page) + offset));
|
||||||
|
|
||||||
|
if (!count)
|
||||||
|
goto out;
|
||||||
|
|
||||||
if (nfs_can_extend_write(file, page, inode)) {
|
if (nfs_can_extend_write(file, page, inode)) {
|
||||||
count = max(count + offset, nfs_page_length(page));
|
count = max(count + offset, nfs_page_length(page));
|
||||||
offset = 0;
|
offset = 0;
|
||||||
@@ -1271,7 +1274,7 @@ int nfs_updatepage(struct file *file, struct page *page,
|
|||||||
nfs_set_pageerror(page);
|
nfs_set_pageerror(page);
|
||||||
else
|
else
|
||||||
__set_page_dirty_nobuffers(page);
|
__set_page_dirty_nobuffers(page);
|
||||||
|
out:
|
||||||
dprintk("NFS: nfs_updatepage returns %d (isize %lld)\n",
|
dprintk("NFS: nfs_updatepage returns %d (isize %lld)\n",
|
||||||
status, (long long)i_size_read(inode));
|
status, (long long)i_size_read(inode));
|
||||||
return status;
|
return status;
|
||||||
|
|||||||
Reference in New Issue
Block a user