aboutsummaryrefslogtreecommitdiffstats
path: root/fs/ocfs2/namei.c
diff options
context:
space:
mode:
authorMark Fasheh <mfasheh@suse.com>2009-02-18 14:41:38 -0500
committerMark Fasheh <mfasheh@suse.com>2009-04-03 14:39:17 -0400
commitb80b549c3520b31d3bbc4b36e37e0a5102da0b94 (patch)
tree403d548a9be6a1e1ae8063a732d520a06defd83c /fs/ocfs2/namei.c
parent3a8df2b9c365a9f366b76bbbce479e1cc18ddc16 (diff)
ocfs2: re-order ocfs2_empty_dir checks
ocfs2_empty_dir() is far more expensive than checking link count. Since both need to be checked at the same time, we can improve performance by checking link count first. Signed-off-by: Mark Fasheh <mfasheh@suse.com>
Diffstat (limited to 'fs/ocfs2/namei.c')
-rw-r--r--fs/ocfs2/namei.c9
1 files changed, 3 insertions, 6 deletions
diff --git a/fs/ocfs2/namei.c b/fs/ocfs2/namei.c
index 67113cfddc9b..58c318d2f061 100644
--- a/fs/ocfs2/namei.c
+++ b/fs/ocfs2/namei.c
@@ -834,10 +834,7 @@ static int ocfs2_unlink(struct inode *dir,
834 child_locked = 1; 834 child_locked = 1;
835 835
836 if (S_ISDIR(inode->i_mode)) { 836 if (S_ISDIR(inode->i_mode)) {
837 if (!ocfs2_empty_dir(inode)) { 837 if (inode->i_nlink != 2 || !ocfs2_empty_dir(inode)) {
838 status = -ENOTEMPTY;
839 goto leave;
840 } else if (inode->i_nlink != 2) {
841 status = -ENOTEMPTY; 838 status = -ENOTEMPTY;
842 goto leave; 839 goto leave;
843 } 840 }
@@ -1280,8 +1277,8 @@ static int ocfs2_rename(struct inode *old_dir,
1280 1277
1281 if (target_exists) { 1278 if (target_exists) {
1282 if (S_ISDIR(new_inode->i_mode)) { 1279 if (S_ISDIR(new_inode->i_mode)) {
1283 if (!ocfs2_empty_dir(new_inode) || 1280 if (new_inode->i_nlink != 2 ||
1284 new_inode->i_nlink != 2) { 1281 !ocfs2_empty_dir(new_inode)) {
1285 status = -ENOTEMPTY; 1282 status = -ENOTEMPTY;
1286 goto bail; 1283 goto bail;
1287 } 1284 }