diff options
| author |   Eric W. Biederman <ebiederm@xmission.com> | 2014-02-13 09:46:25 -0800 |
|---|---|---|
| committer |   Al Viro <viro@zeniv.linux.org.uk> | 2014-10-09 02:38:57 -0400 |
| commit | 5542aa2fa7f6cddb03c4ac3135e390adffda98ca (patch) | |
| tree | adfb3c0923c8f00e7946e85c1d8742dc89f0cce7 /fs/namei.c | |
| parent | vfs: Merge check_submounts_and_drop and d_invalidate (diff) | |
| download | linux-dev-5542aa2fa7f6cddb03c4ac3135e390adffda98ca.tar.xz linux-dev-5542aa2fa7f6cddb03c4ac3135e390adffda98ca.zip | |
vfs: Make d_invalidate return void
Now that d_invalidate can no longer fail, stop returning a useless
return code. For the few callers that checked the return code update
remove the handling of d_invalidate failure.
Reviewed-by: Miklos Szeredi <miklos@szeredi.hu>
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/namei.c')
| -rw-r--r-- | fs/namei.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/fs/namei.c b/fs/namei.c index 2ba10904dba0..d20d579a022e 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1306,7 +1306,8 @@ static struct dentry *lookup_dcache(struct qstr *name, struct dentry *dir, if (error < 0) { dput(dentry); return ERR_PTR(error); - } else if (!d_invalidate(dentry)) { + } else { + d_invalidate(dentry); dput(dentry); dentry = NULL; } @@ -1435,10 +1436,9 @@ unlazy: dput(dentry); return status; } - if (!d_invalidate(dentry)) { - dput(dentry); - goto need_lookup; - } + d_invalidate(dentry); + dput(dentry); + goto need_lookup; } path->mnt = mnt; |