NFS: Fix the return value in nfs_page_mkwrite()
authorTrond Myklebust <Trond.Myklebust@netapp.com>
Tue, 12 May 2009 06:23:51 +0000 (16:23 +1000)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 20 May 2009 05:20:10 +0000 (22:20 -0700)
commit 2b2ec7554cf7ec5e4412f89a5af6abe8ce950700 upstream

Commit c2ec175c39f62949438354f603f4aa170846aabb ("mm: page_mkwrite
change prototype to match fault") exposed a bug in the NFS
implementation of page_mkwrite.  We should be returning 0 on success...

Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Nick Piggin <npiggin@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
fs/nfs/file.c

index 62e5b28..a7c9f7b 100644 (file)
@@ -477,8 +477,6 @@ static int nfs_vm_page_mkwrite(struct vm_area_struct *vma, struct vm_fault *vmf)
                goto out_unlock;
 
        ret = nfs_updatepage(filp, page, 0, pagelen);
-       if (ret == 0)
-               ret = pagelen;
 out_unlock:
        unlock_page(page);
        if (ret)