Commit 8cbfa53b authored by Sage Weil's avatar Sage Weil Committed by Al Viro

exofs: remove unnecessary dentry_unhash on rmdir/rename_dir

Exofs has no problems with lingering references to unlinked directory
inodes.

CC: Benny Halevy <bhalevy@panasas.com>
CC: osd-dev@open-osd.org
Acked-by: default avatarBoaz Harrosh <bharrosh@panasas.com>
Signed-off-by: default avatarSage Weil <sage@newdream.net>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 052e2a1b
...@@ -227,8 +227,6 @@ static int exofs_rmdir(struct inode *dir, struct dentry *dentry) ...@@ -227,8 +227,6 @@ static int exofs_rmdir(struct inode *dir, struct dentry *dentry)
struct inode *inode = dentry->d_inode; struct inode *inode = dentry->d_inode;
int err = -ENOTEMPTY; int err = -ENOTEMPTY;
dentry_unhash(dentry);
if (exofs_empty_dir(inode)) { if (exofs_empty_dir(inode)) {
err = exofs_unlink(dir, dentry); err = exofs_unlink(dir, dentry);
if (!err) { if (!err) {
...@@ -251,9 +249,6 @@ static int exofs_rename(struct inode *old_dir, struct dentry *old_dentry, ...@@ -251,9 +249,6 @@ static int exofs_rename(struct inode *old_dir, struct dentry *old_dentry,
struct exofs_dir_entry *old_de; struct exofs_dir_entry *old_de;
int err = -ENOENT; int err = -ENOENT;
if (new_inode && S_ISDIR(new_inode->i_mode))
dentry_unhash(new_dentry);
old_de = exofs_find_entry(old_dir, old_dentry, &old_page); old_de = exofs_find_entry(old_dir, old_dentry, &old_page);
if (!old_de) if (!old_de)
goto out; goto out;
......
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