diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2012-03-30 14:48:04 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2012-03-31 16:03:17 -0400 |
| commit | 81e6f520898edbda56e8680d338ace4f5694874e (patch) | |
| tree | d516e74ac37250452b6deb8cc3a5ef686bad6a81 /fs/namei.c | |
| parent | a32555466caee38faeef4e44d7878ecbff1199bc (diff) | |
| download | lwn-81e6f520898edbda56e8680d338ace4f5694874e.tar.gz lwn-81e6f520898edbda56e8680d338ace4f5694874e.zip | |
untangling do_lookup() - take __lookup_hash()-calling case out of line.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'fs/namei.c')
| -rw-r--r-- | fs/namei.c | 31 |
1 files changed, 16 insertions, 15 deletions
diff --git a/fs/namei.c b/fs/namei.c index a919affd1531..5eeec562a03d 100644 --- a/fs/namei.c +++ b/fs/namei.c @@ -1208,22 +1208,14 @@ unlazy: dentry = __d_lookup(parent, name); } - if (dentry && unlikely(d_need_lookup(dentry))) { + if (unlikely(!dentry)) + goto need_lookup; + + if (unlikely(d_need_lookup(dentry))) { dput(dentry); - dentry = NULL; + goto need_lookup; } -retry: - if (unlikely(!dentry)) { - struct inode *dir = parent->d_inode; - BUG_ON(nd->inode != dir); - mutex_lock(&dir->i_mutex); - dentry = __lookup_hash(name, parent, nd); - mutex_unlock(&dir->i_mutex); - if (IS_ERR(dentry)) - return PTR_ERR(dentry); - goto done; - } if (unlikely(dentry->d_flags & DCACHE_OP_REVALIDATE) && need_reval) status = d_revalidate(dentry, nd); if (unlikely(status <= 0)) { @@ -1233,8 +1225,7 @@ retry: } if (!d_invalidate(dentry)) { dput(dentry); - dentry = NULL; - goto retry; + goto need_lookup; } } done: @@ -1249,6 +1240,16 @@ done: nd->flags |= LOOKUP_JUMPED; *inode = path->dentry->d_inode; return 0; + +need_lookup: + BUG_ON(nd->inode != parent->d_inode); + + mutex_lock(&parent->d_inode->i_mutex); + dentry = __lookup_hash(name, parent, nd); + mutex_unlock(&parent->d_inode->i_mutex); + if (IS_ERR(dentry)) + return PTR_ERR(dentry); + goto done; } static inline int may_lookup(struct nameidata *nd) |