Commit ee70fca0 authored by Martin Brandenburg's avatar Martin Brandenburg Committed by Mike Marshall

orangefs: don't d_drop in d_revalidate since the caller will

Signed-off-by: default avatarMartin Brandenburg <martin@omnibond.com>
Signed-off-by: default avatarMike Marshall <hubcap@omnibond.com>
parent ee3b8d37
......@@ -82,7 +82,6 @@ static int orangefs_revalidate_lookup(struct dentry *dentry)
out_drop:
gossip_debug(GOSSIP_DCACHE_DEBUG, "%s:%s:%d revalidate failed\n",
__FILE__, __func__, __LINE__);
d_drop(dentry);
goto out_release_op;
}
......@@ -109,10 +108,8 @@ static int orangefs_d_revalidate(struct dentry *dentry, unsigned int flags)
* If this passes, the positive dentry still exists or the negative
* dentry still does not exist.
*/
if (!orangefs_revalidate_lookup(dentry)) {
d_drop(dentry);
if (!orangefs_revalidate_lookup(dentry))
return 0;
}
/* We do not need to continue with negative dentries. */
if (!dentry->d_inode)
......@@ -125,13 +122,10 @@ static int orangefs_d_revalidate(struct dentry *dentry, unsigned int flags)
if (ret < 0) {
gossip_debug(GOSSIP_DCACHE_DEBUG, "%s:%s:%d getattr failure.\n",
__FILE__, __func__, __LINE__);
d_drop(dentry);
return 0;
}
if (ret == 0) {
d_drop(dentry);
if (ret == 0)
return 0;
}
out:
gossip_debug(GOSSIP_DCACHE_DEBUG,
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment