diff options
| author | Sage Weil <sage@newdream.net> | 2011-05-27 16:41:57 -0400 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2011-05-28 01:02:51 -0400 |
| commit | 44a8e6364e48ab93a1d86385b5fc9efe81395fa9 (patch) | |
| tree | 763a035f0e4a7eac8a65398430fa57aa90381c15 | |
| parent | cf0f0536fa65ca1353476c49feed34891f3f7134 (diff) | |
jfs: remove unnecessary dentry_unhash from rmdir, dir rename
jfs does not have problems with references to unlinked directories.
CC: Dave Kleikamp <shaggy@kernel.org>
CC: jfs-discussion@lists.sourceforge.net
Signed-off-by: Sage Weil <sage@newdream.net>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
| -rw-r--r-- | fs/jfs/namei.c | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/fs/jfs/namei.c b/fs/jfs/namei.c index 865df16a6cf3..eaaf2b511e89 100644 --- a/fs/jfs/namei.c +++ b/fs/jfs/namei.c | |||
| @@ -360,8 +360,6 @@ static int jfs_rmdir(struct inode *dip, struct dentry *dentry) | |||
| 360 | 360 | ||
| 361 | jfs_info("jfs_rmdir: dip:0x%p name:%s", dip, dentry->d_name.name); | 361 | jfs_info("jfs_rmdir: dip:0x%p name:%s", dip, dentry->d_name.name); |
| 362 | 362 | ||
| 363 | dentry_unhash(dentry); | ||
| 364 | |||
| 365 | /* Init inode for quota operations. */ | 363 | /* Init inode for quota operations. */ |
| 366 | dquot_initialize(dip); | 364 | dquot_initialize(dip); |
| 367 | dquot_initialize(ip); | 365 | dquot_initialize(ip); |
| @@ -1097,9 +1095,6 @@ static int jfs_rename(struct inode *old_dir, struct dentry *old_dentry, | |||
| 1097 | jfs_info("jfs_rename: %s %s", old_dentry->d_name.name, | 1095 | jfs_info("jfs_rename: %s %s", old_dentry->d_name.name, |
| 1098 | new_dentry->d_name.name); | 1096 | new_dentry->d_name.name); |
| 1099 | 1097 | ||
| 1100 | if (new_dentry->d_inode && S_ISDIR(new_dentry->d_inode->i_mode)) | ||
| 1101 | dentry_unhash(new_dentry); | ||
| 1102 | |||
| 1103 | dquot_initialize(old_dir); | 1098 | dquot_initialize(old_dir); |
| 1104 | dquot_initialize(new_dir); | 1099 | dquot_initialize(new_dir); |
| 1105 | 1100 | ||
