IB/srp: Remove unnecessary unlikely()
authorIgor Stoppa <igor.stoppa@gmail.com>
Fri, 31 Aug 2018 20:16:45 +0000 (23:16 +0300)
committerJason Gunthorpe <jgg@mellanox.com>
Wed, 5 Sep 2018 21:52:06 +0000 (15:52 -0600)
WARN_ON() already contains an unlikely(), so it's not necessary to wrap it
into another.

Signed-off-by: Igor Stoppa <igor.stoppa@huawei.com>
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>
drivers/infiniband/ulp/srpt/ib_srpt.c

index f37cbad..447d21e 100644 (file)
@@ -2708,7 +2708,7 @@ static void srpt_queue_response(struct se_cmd *cmd)
                break;
        }
 
-       if (unlikely(WARN_ON_ONCE(state == SRPT_STATE_CMD_RSP_SENT)))
+       if (WARN_ON_ONCE(state == SRPT_STATE_CMD_RSP_SENT))
                return;
 
        /* For read commands, transfer the data to the initiator. */