summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorxuejiufei <xuejiufei@huawei.com>2016-02-05 15:36:47 -0800
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2016-02-19 14:22:38 -0800
commitbb169b2b585790c58d59aa09e1408e365a7acccc (patch)
tree5091ed1aa6af7bb6734a07cb9b5995ebb865a338
parent854e6fc9de5bf94952095b544daa6c339b41014b (diff)
downloadlwn-bb169b2b585790c58d59aa09e1408e365a7acccc.tar.gz
lwn-bb169b2b585790c58d59aa09e1408e365a7acccc.zip
ocfs2/dlm: clear refmap bit of recovery lock while doing local recovery cleanup
commit c95a51807b730e4681e2ecbdfd669ca52601959e upstream. When recovery master down, dlm_do_local_recovery_cleanup() only remove the $RECOVERY lock owned by dead node, but do not clear the refmap bit. Which will make umount thread falling in dead loop migrating $RECOVERY to the dead node. Signed-off-by: xuejiufei <xuejiufei@huawei.com> Reviewed-by: Joseph Qi <joseph.qi@huawei.com> Cc: Mark Fasheh <mfasheh@suse.de> Cc: Joel Becker <jlbec@evilplan.org> Cc: Junxiao Bi <junxiao.bi@oracle.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--fs/ocfs2/dlm/dlmrecovery.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/ocfs2/dlm/dlmrecovery.c b/fs/ocfs2/dlm/dlmrecovery.c
index 9bd981cd3142..01c69f24e416 100644
--- a/fs/ocfs2/dlm/dlmrecovery.c
+++ b/fs/ocfs2/dlm/dlmrecovery.c
@@ -2326,6 +2326,8 @@ static void dlm_do_local_recovery_cleanup(struct dlm_ctxt *dlm, u8 dead_node)
break;
}
}
+ dlm_lockres_clear_refmap_bit(dlm, res,
+ dead_node);
spin_unlock(&res->spinlock);
continue;
}