diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2014-02-13 12:46:25 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2014-10-09 02:38:57 -0400 |
commit | 5542aa2fa7f6cddb03c4ac3135e390adffda98ca (patch) | |
tree | adfb3c0923c8f00e7946e85c1d8742dc89f0cce7 /fs/namei.c | |
parent | 1ffe46d11cc88479797b262f60d92e5fb461b411 (diff) |
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, | |||
1306 | if (error < 0) { | 1306 | if (error < 0) { |
1307 | dput(dentry); | 1307 | dput(dentry); |
1308 | return ERR_PTR(error); | 1308 | return ERR_PTR(error); |
1309 | } else if (!d_invalidate(dentry)) { | 1309 | } else { |
1310 | d_invalidate(dentry); | ||
1310 | dput(dentry); | 1311 | dput(dentry); |
1311 | dentry = NULL; | 1312 | dentry = NULL; |
1312 | } | 1313 | } |
@@ -1435,10 +1436,9 @@ unlazy: | |||
1435 | dput(dentry); | 1436 | dput(dentry); |
1436 | return status; | 1437 | return status; |
1437 | } | 1438 | } |
1438 | if (!d_invalidate(dentry)) { | 1439 | d_invalidate(dentry); |
1439 | dput(dentry); | 1440 | dput(dentry); |
1440 | goto need_lookup; | 1441 | goto need_lookup; |
1441 | } | ||
1442 | } | 1442 | } |
1443 | 1443 | ||
1444 | path->mnt = mnt; | 1444 | path->mnt = mnt; |