ocfs2: dlm: remove redundant code
authorJunxiao Bi <junxiao.bi@oracle.com>
Thu, 14 Jan 2016 23:17:47 +0000 (15:17 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 15 Jan 2016 00:00:49 +0000 (16:00 -0800)
Found this when do patch review, remove to make it clear and save a
little cpu time.

Signed-off-by: Junxiao Bi <junxiao.bi@oracle.com>
Cc: Mark Fasheh <mfasheh@suse.de>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Joseph Qi <joseph.qi@huawei.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/dlm/dlmrecovery.c

index 86fb536..c5bdf02 100644 (file)
@@ -2457,11 +2457,7 @@ static void __dlm_hb_node_down(struct dlm_ctxt *dlm, int idx)
         * perhaps later we can genericize this for other waiters. */
        wake_up(&dlm->migration_wq);
 
-       if (test_bit(idx, dlm->recovery_map))
-               mlog(0, "domain %s, node %u already added "
-                    "to recovery map!\n", dlm->name, idx);
-       else
-               set_bit(idx, dlm->recovery_map);
+       set_bit(idx, dlm->recovery_map);
 }
 
 void dlm_hb_node_down_cb(struct o2nm_node *node, int idx, void *data)