Commit 9f48db0d authored by Aditya Pakki's avatar Aditya Pakki Committed by Jason Gunthorpe

RDMA/srpt: Remove unnecessary assertion in srpt_queue_response

Since ch has already been de-referenced by the time we get to the BUG_ON,
it is useless. The back trace alone is enough to tell what is going on,
delete the redundant BUG_ON.

Link: https://lore.kernel.org/r/20191217194437.25568-1-pakki001@umn.eduSigned-off-by: default avatarAditya Pakki <pakki001@umn.edu>
Reviewed-by: default avatarBart Van Assche <bvanassche@acm.org>
Reviewed-by: default avatarJason Gunthorpe <jgg@mellanox.com>
Signed-off-by: default avatarJason Gunthorpe <jgg@mellanox.com>
parent a242c369
...@@ -2810,8 +2810,6 @@ static void srpt_queue_response(struct se_cmd *cmd) ...@@ -2810,8 +2810,6 @@ static void srpt_queue_response(struct se_cmd *cmd)
int resp_len, ret, i; int resp_len, ret, i;
u8 srp_tm_status; u8 srp_tm_status;
BUG_ON(!ch);
state = ioctx->state; state = ioctx->state;
switch (state) { switch (state) {
case SRPT_STATE_NEW: case SRPT_STATE_NEW:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment