Commit 52e65d1c authored by Bart Van Assche's avatar Bart Van Assche Committed by Martin K. Petersen

scsi: esas2r: Call scsi_done() directly

Conditional statements are faster than indirect calls. Hence call
scsi_done() directly.

Link: https://lore.kernel.org/r/20211007202923.2174984-31-bvanassche@acm.orgSigned-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent e6ed928e
...@@ -828,7 +828,7 @@ int esas2r_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd) ...@@ -828,7 +828,7 @@ int esas2r_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *cmd)
if (unlikely(test_bit(AF_DEGRADED_MODE, &a->flags))) { if (unlikely(test_bit(AF_DEGRADED_MODE, &a->flags))) {
cmd->result = DID_NO_CONNECT << 16; cmd->result = DID_NO_CONNECT << 16;
cmd->scsi_done(cmd); scsi_done(cmd);
return 0; return 0;
} }
...@@ -988,7 +988,7 @@ int esas2r_eh_abort(struct scsi_cmnd *cmd) ...@@ -988,7 +988,7 @@ int esas2r_eh_abort(struct scsi_cmnd *cmd)
scsi_set_resid(cmd, 0); scsi_set_resid(cmd, 0);
cmd->scsi_done(cmd); scsi_done(cmd);
return SUCCESS; return SUCCESS;
} }
...@@ -1054,7 +1054,7 @@ int esas2r_eh_abort(struct scsi_cmnd *cmd) ...@@ -1054,7 +1054,7 @@ int esas2r_eh_abort(struct scsi_cmnd *cmd)
scsi_set_resid(cmd, 0); scsi_set_resid(cmd, 0);
cmd->scsi_done(cmd); scsi_done(cmd);
return SUCCESS; return SUCCESS;
} }
...@@ -1535,7 +1535,7 @@ void esas2r_complete_request_cb(struct esas2r_adapter *a, ...@@ -1535,7 +1535,7 @@ void esas2r_complete_request_cb(struct esas2r_adapter *a,
scsi_set_resid(rq->cmd, 0); scsi_set_resid(rq->cmd, 0);
} }
rq->cmd->scsi_done(rq->cmd); scsi_done(rq->cmd);
esas2r_free_request(a, rq); esas2r_free_request(a, rq);
} }
......
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