Commit 39cf8a13 authored by Chuck Lever's avatar Chuck Lever Committed by Linus Torvalds

[PATCH] NFS: fix minor bug in new NFS symlink code

The original code confused a zero return code from pagevec_add() as success.

Test plan:
None.
Signed-off-by: default avatarChuck Lever <chuck.lever@oracle.com>
Signed-off-by: default avatarTrond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent cd9ae2b6
...@@ -1519,8 +1519,8 @@ static int nfs_symlink(struct inode *dir, struct dentry *dentry, const char *sym ...@@ -1519,8 +1519,8 @@ static int nfs_symlink(struct inode *dir, struct dentry *dentry, const char *sym
pagevec_init(&lru_pvec, 0); pagevec_init(&lru_pvec, 0);
if (!add_to_page_cache(page, dentry->d_inode->i_mapping, 0, if (!add_to_page_cache(page, dentry->d_inode->i_mapping, 0,
GFP_KERNEL)) { GFP_KERNEL)) {
if (!pagevec_add(&lru_pvec, page)) pagevec_add(&lru_pvec, page);
__pagevec_lru_add(&lru_pvec); pagevec_lru_add(&lru_pvec);
SetPageUptodate(page); SetPageUptodate(page);
unlock_page(page); unlock_page(page);
} else } else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment