From 347ce015f3a904685245b3da65b1ebd3e6025ab9 Mon Sep 17 00:00:00 2001 From: Lepton Wu Date: Tue, 16 Oct 2007 23:30:04 -0700 Subject: [PATCH] --- yaml --- r: 70683 b: refs/heads/master c: fb46f341d9868fe993626536c7449c2a1aec62a3 h: refs/heads/master i: 70681: ea69cc0bcabfc8c417ba89396376dabc5bc8f74d 70679: 57d95c1a4836052bc442b72c94c6b2f2623e3828 v: v3 --- [refs] | 2 +- trunk/fs/reiserfs/super.c | 18 +++++++++++++++--- 2 files changed, 16 insertions(+), 4 deletions(-) diff --git a/[refs] b/[refs] index c4f47fc9d9b1..1c9ccd1a055d 100644 --- a/[refs] +++ b/[refs] @@ -1,2 +1,2 @@ --- -refs/heads/master: b9ec0339d8e22cadf2d9d1b010b51dc53837dfb0 +refs/heads/master: fb46f341d9868fe993626536c7449c2a1aec62a3 diff --git a/trunk/fs/reiserfs/super.c b/trunk/fs/reiserfs/super.c index edfd74f9f7e4..b82897ae090b 100644 --- a/trunk/fs/reiserfs/super.c +++ b/trunk/fs/reiserfs/super.c @@ -145,7 +145,7 @@ static int finish_unfinished(struct super_block *s) { INITIALIZE_PATH(path); struct cpu_key max_cpu_key, obj_key; - struct reiserfs_key save_link_key; + struct reiserfs_key save_link_key, last_inode_key; int retval = 0; struct item_head *ih; struct buffer_head *bh; @@ -166,6 +166,8 @@ static int finish_unfinished(struct super_block *s) set_cpu_key_k_offset(&max_cpu_key, ~0U); max_cpu_key.key_length = 3; + memset(&last_inode_key, 0, sizeof(last_inode_key)); + #ifdef CONFIG_QUOTA /* Needed for iput() to work correctly and not trash data */ if (s->s_flags & MS_ACTIVE) { @@ -278,8 +280,18 @@ static int finish_unfinished(struct super_block *s) REISERFS_I(inode)->i_flags |= i_link_saved_unlink_mask; /* not completed unlink (rmdir) found */ reiserfs_info(s, "Removing %k..", INODE_PKEY(inode)); - /* removal gets completed in iput */ - retval = 0; + if (memcmp(&last_inode_key, INODE_PKEY(inode), + sizeof(last_inode_key))){ + last_inode_key = *INODE_PKEY(inode); + /* removal gets completed in iput */ + retval = 0; + } else { + reiserfs_warning(s, "Dead loop in " + "finish_unfinished detected, " + "just remove save link\n"); + retval = remove_save_link_only(s, + &save_link_key, 0); + } } iput(inode);