diff options
author | Trond Myklebust <Trond.Myklebust@netapp.com> | 2009-04-24 17:32:22 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-05-18 16:35:07 -0700 |
commit | 5da3b37d5eb68af30eb9e8bb553bed8c616024eb (patch) | |
tree | c4ae8cb9f6684c1b8a7904b2ab94446a5fcafa89 /fs | |
parent | 668d834cdaebbd0280e4bd497205ea66ee0bf23f (diff) |
NFS: Close page_mkwrite() races
commit 7fdf523067666b0eaff330f362401ee50ce187c4 upstream.
Follow up to Nick Piggin's patches to ensure that nfs_vm_page_mkwrite
returns with the page lock held, and sets the VM_FAULT_LOCKED flag.
See http://bugzilla.kernel.org/show_bug.cgi?id=12913
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/nfs/file.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/fs/nfs/file.c b/fs/nfs/file.c index f4a5bbbed14f..523e7e021805 100644 --- a/fs/nfs/file.c +++ b/fs/nfs/file.c @@ -481,10 +481,10 @@ static int nfs_vm_page_mkwrite(struct vm_area_struct *vma, struct vm_fault *vmf) ret = nfs_updatepage(filp, page, 0, pagelen); out_unlock: + if (!ret) + return VM_FAULT_LOCKED; unlock_page(page); - if (ret) - ret = VM_FAULT_SIGBUS; - return ret; + return VM_FAULT_SIGBUS; } static struct vm_operations_struct nfs_file_vm_ops = { |