ocfs2/dlm: clean up useless BUG_ON default case in dlm_finalize_reco_handler()
authorpiaojun <piaojun@huawei.com>
Tue, 13 Dec 2016 00:41:08 +0000 (16:41 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 13 Dec 2016 02:55:06 +0000 (18:55 -0800)
The value of 'stage' must be between 1 and 2, so the switch can't reach
the default case.

Link: http://lkml.kernel.org/r/57FB5EB2.7050002@huawei.com
Signed-off-by: Jun Piao <piaojun@huawei.com>
Cc: Mark Fasheh <mfasheh@versity.com>
Cc: Joel Becker <jlbec@evilplan.org>
Cc: Junxiao Bi <junxiao.bi@oracle.com>
Cc: Joseph Qi <jiangqi903@gmail.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 dd5cb8bcefd13820ae0d242dc1e02da5798736c6..74407c6dd592a51a6bf2c83ef39e77479e5ded85 100644 (file)
@@ -2966,8 +2966,6 @@ int dlm_finalize_reco_handler(struct o2net_msg *msg, u32 len, void *data,
                        spin_unlock(&dlm->spinlock);
                        dlm_kick_recovery_thread(dlm);
                        break;
-               default:
-                       BUG();
        }
 
        mlog(0, "%s: recovery done, reco master was %u, dead now %u, master now %u\n",