diff options
| author | Chuck Lever <chuck.lever@oracle.com> | 2006-10-20 02:28:41 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-10-20 13:26:39 -0400 |
| commit | 39cf8a1374dc51fea169190674d5e4996a7d7ea2 (patch) | |
| tree | ca6e201861ed911c2b78dd81b56505527308c9a4 | |
| parent | cd9ae2b6a75bb1fa0d370929c2d7a7da1ed719d9 (diff) | |
[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: Chuck Lever <chuck.lever@oracle.com>
Signed-off-by: Trond Myklebust <Trond.Myklebust@netapp.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | fs/nfs/dir.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/fs/nfs/dir.c b/fs/nfs/dir.c index 58d44057813e..c86a1ead4772 100644 --- a/fs/nfs/dir.c +++ b/fs/nfs/dir.c | |||
| @@ -1519,8 +1519,8 @@ static int nfs_symlink(struct inode *dir, struct dentry *dentry, const char *sym | |||
| 1519 | pagevec_init(&lru_pvec, 0); | 1519 | pagevec_init(&lru_pvec, 0); |
| 1520 | if (!add_to_page_cache(page, dentry->d_inode->i_mapping, 0, | 1520 | if (!add_to_page_cache(page, dentry->d_inode->i_mapping, 0, |
| 1521 | GFP_KERNEL)) { | 1521 | GFP_KERNEL)) { |
| 1522 | if (!pagevec_add(&lru_pvec, page)) | 1522 | pagevec_add(&lru_pvec, page); |
| 1523 | __pagevec_lru_add(&lru_pvec); | 1523 | pagevec_lru_add(&lru_pvec); |
| 1524 | SetPageUptodate(page); | 1524 | SetPageUptodate(page); |
| 1525 | unlock_page(page); | 1525 | unlock_page(page); |
| 1526 | } else | 1526 | } else |