ocfs2: remove unnecessary dentry_unhash on rmdir/rename_dir
authorSage Weil <sage@newdream.net>
Tue, 24 May 2011 20:06:21 +0000 (13:06 -0700)
committerAl Viro <viro@zeniv.linux.org.uk>
Thu, 26 May 2011 11:26:58 +0000 (07:26 -0400)
Ocfs2 has no issues with lingering references to unlinked directory inodes.

CC: Mark Fasheh <mfasheh@suse.com>
CC: ocfs2-devel@oss.oracle.com
Acked-by: Joel Becker <jlbec@evilplan.org>
Signed-off-by: Sage Weil <sage@newdream.net>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/ocfs2/namei.c

index f3582a6a6dac87595d0a3ccad45f0ed4d5c3b9a2..e5d738cd9cc0540d508873818bd0b6035f2c1f20 100644 (file)
@@ -810,9 +810,6 @@ static int ocfs2_unlink(struct inode *dir,
                           (unsigned long long)OCFS2_I(dir)->ip_blkno,
                           (unsigned long long)OCFS2_I(inode)->ip_blkno);
 
-       if (S_ISDIR(inode->i_mode))
-               dentry_unhash(dentry);
-
        dquot_initialize(dir);
 
        BUG_ON(dentry->d_parent->d_inode != dir);
@@ -1066,9 +1063,6 @@ static int ocfs2_rename(struct inode *old_dir,
        struct ocfs2_dir_lookup_result orphan_insert = { NULL, };
        struct ocfs2_dir_lookup_result target_insert = { NULL, };
 
-       if (new_inode && S_ISDIR(new_inode->i_mode))
-               dentry_unhash(new_dentry);
-
        /* At some point it might be nice to break this function up a
         * bit. */