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

scsi: mvumi: Call scsi_done() directly

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

Link: https://lore.kernel.org/r/20211007202923.2174984-54-bvanassche@acm.orgSigned-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent b0c30079
...@@ -1328,7 +1328,7 @@ static void mvumi_complete_cmd(struct mvumi_hba *mhba, struct mvumi_cmd *cmd, ...@@ -1328,7 +1328,7 @@ static void mvumi_complete_cmd(struct mvumi_hba *mhba, struct mvumi_cmd *cmd,
dma_unmap_sg(&mhba->pdev->dev, scsi_sglist(scmd), dma_unmap_sg(&mhba->pdev->dev, scsi_sglist(scmd),
scsi_sg_count(scmd), scsi_sg_count(scmd),
scmd->sc_data_direction); scmd->sc_data_direction);
cmd->scmd->scsi_done(scmd); scsi_done(scmd);
mvumi_return_cmd(mhba, cmd); mvumi_return_cmd(mhba, cmd);
} }
...@@ -2104,7 +2104,7 @@ static int mvumi_queue_command(struct Scsi_Host *shost, ...@@ -2104,7 +2104,7 @@ static int mvumi_queue_command(struct Scsi_Host *shost,
out_return_cmd: out_return_cmd:
mvumi_return_cmd(mhba, cmd); mvumi_return_cmd(mhba, cmd);
scmd->scsi_done(scmd); scsi_done(scmd);
spin_unlock_irqrestore(shost->host_lock, irq_flags); spin_unlock_irqrestore(shost->host_lock, irq_flags);
return 0; return 0;
} }
......
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