ocfs2/dlm: Remove BUG_ON in dlm recovery when freeing locks of a dead node
authorSunil Mushran <sunil.mushran@oracle.com>
Tue, 2 Feb 2010 01:34:58 +0000 (17:34 -0800)
committerJoel Becker <joel.becker@oracle.com>
Thu, 4 Feb 2010 01:51:41 +0000 (17:51 -0800)
commitcda70ba8c05a8661f882862c4699a31d215ab151
tree3aec5061ae7a361dc2706225ad38928ccd860a02
parent079b805782f94f4b278132286a8c9bc4655d1c51
ocfs2/dlm: Remove BUG_ON in dlm recovery when freeing locks of a dead node

During recovery, the dlm frees the locks for the dead node. If it finds a
lock in a resource for the dead node, it expects that node to also have a
ref in that lock resource. If not, it BUGs.

ossbz#1175 was filed with the above BUG. Now, while it is correct that we
should be expecting the ref, I see no reason why we have to BUG. After all,
we are freeing up the lock and clearing the ref.

This patch replaces the BUG_ON with a printk(). Hopefully, that will give
us more clues next time this happens.

http://oss.oracle.com/bugzilla/show_bug.cgi?id=1175

Signed-off-by: Sunil Mushran <sunil.mushran@oracle.com>
Acked-by: Mark Fasheh <mfasheh@suse.com>
Signed-off-by: Joel Becker <joel.becker@oracle.com>
fs/ocfs2/dlm/dlmrecovery.c