diff options
author | Sage Weil <sage@newdream.net> | 2011-05-24 16:06:09 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-05-26 07:26:50 -0400 |
commit | 5c5d3f3b871a719e2c6413b85009c723adeb27e1 (patch) | |
tree | 8047b35ed1d8d385b25f906d4d05a83b062ebcb1 | |
parent | a71905f0db41d4b2b01044fb40f97656fefc44a7 (diff) |
libfs: drop unneeded dentry_unhash
There are no libfs issues with dangling references to empty directories.
Signed-off-by: Sage Weil <sage@newdream.net>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
-rw-r--r-- | fs/libfs.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/libfs.c b/fs/libfs.c index 91a3710e0fe5..c88eab55aec9 100644 --- a/fs/libfs.c +++ b/fs/libfs.c | |||
@@ -311,8 +311,6 @@ int simple_rmdir(struct inode *dir, struct dentry *dentry) | |||
311 | if (!simple_empty(dentry)) | 311 | if (!simple_empty(dentry)) |
312 | return -ENOTEMPTY; | 312 | return -ENOTEMPTY; |
313 | 313 | ||
314 | dentry_unhash(dentry); | ||
315 | |||
316 | drop_nlink(dentry->d_inode); | 314 | drop_nlink(dentry->d_inode); |
317 | simple_unlink(dir, dentry); | 315 | simple_unlink(dir, dentry); |
318 | drop_nlink(dir); | 316 | drop_nlink(dir); |
@@ -325,9 +323,6 @@ int simple_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
325 | struct inode *inode = old_dentry->d_inode; | 323 | struct inode *inode = old_dentry->d_inode; |
326 | int they_are_dirs = S_ISDIR(old_dentry->d_inode->i_mode); | 324 | int they_are_dirs = S_ISDIR(old_dentry->d_inode->i_mode); |
327 | 325 | ||
328 | if (new_dentry->d_inode && S_ISDIR(new_dentry->d_inode->i_mode)) | ||
329 | dentry_unhash(new_dentry); | ||
330 | |||
331 | if (!simple_empty(new_dentry)) | 326 | if (!simple_empty(new_dentry)) |
332 | return -ENOTEMPTY; | 327 | return -ENOTEMPTY; |
333 | 328 | ||