diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2014-02-13 12:34:30 -0500 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2014-10-09 02:38:56 -0400 |
commit | 9b053f3207e8887fed88162a339fdd4001abcdb2 (patch) | |
tree | 7751f057ee6c8ecd6cdb8d613c94db5c6dd632f7 /fs/afs | |
parent | 8ed936b5671bfb33d89bc60bdcc7cf0470ba52fe (diff) |
vfs: Remove unnecessary calls of check_submounts_and_drop
Now that check_submounts_and_drop can not fail and is called from
d_invalidate there is no longer a need to call check_submounts_and_drom
from filesystem d_revalidate methods so remove it.
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/afs')
-rw-r--r-- | fs/afs/dir.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/afs/dir.c b/fs/afs/dir.c index 529300327f45..a1645b88fe8a 100644 --- a/fs/afs/dir.c +++ b/fs/afs/dir.c | |||
@@ -669,7 +669,6 @@ static int afs_d_revalidate(struct dentry *dentry, unsigned int flags) | |||
669 | 669 | ||
670 | out_valid: | 670 | out_valid: |
671 | dentry->d_fsdata = dir_version; | 671 | dentry->d_fsdata = dir_version; |
672 | out_skip: | ||
673 | dput(parent); | 672 | dput(parent); |
674 | key_put(key); | 673 | key_put(key); |
675 | _leave(" = 1 [valid]"); | 674 | _leave(" = 1 [valid]"); |
@@ -682,10 +681,6 @@ not_found: | |||
682 | spin_unlock(&dentry->d_lock); | 681 | spin_unlock(&dentry->d_lock); |
683 | 682 | ||
684 | out_bad: | 683 | out_bad: |
685 | /* don't unhash if we have submounts */ | ||
686 | if (check_submounts_and_drop(dentry) != 0) | ||
687 | goto out_skip; | ||
688 | |||
689 | _debug("dropping dentry %s/%s", | 684 | _debug("dropping dentry %s/%s", |
690 | parent->d_name.name, dentry->d_name.name); | 685 | parent->d_name.name, dentry->d_name.name); |
691 | dput(parent); | 686 | dput(parent); |