diff options
| author | Bart Van Assche <bvanassche@acm.org> | 2021-10-07 22:28:34 +0200 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2021-10-17 03:28:47 +0200 |
| commit | 0233196eb238bc18fa89be9249b4b321af0c3fc1 (patch) | |
| tree | b354830e6d66dda8048b4932892c91a762b11c42 /drivers/scsi/imm.c | |
| parent | scsi: ibmvscsi: Call scsi_done() directly (diff) | |
| download | linux-0233196eb238bc18fa89be9249b4b321af0c3fc1.tar.xz linux-0233196eb238bc18fa89be9249b4b321af0c3fc1.zip | |
scsi: imm: Call scsi_done() directly
Conditional statements are faster than indirect calls. Hence call
scsi_done() directly.
Link: https://lore.kernel.org/r/20211007202923.2174984-40-bvanassche@acm.org
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'drivers/scsi/imm.c')
| -rw-r--r-- | drivers/scsi/imm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/imm.c b/drivers/scsi/imm.c index 943c9102a7eb..be8edcff0177 100644 --- a/drivers/scsi/imm.c +++ b/drivers/scsi/imm.c @@ -769,7 +769,7 @@ static void imm_interrupt(struct work_struct *work) spin_lock_irqsave(host->host_lock, flags); dev->cur_cmd = NULL; - cmd->scsi_done(cmd); + scsi_done(cmd); spin_unlock_irqrestore(host->host_lock, flags); return; } @@ -922,7 +922,6 @@ static int imm_queuecommand_lck(struct scsi_cmnd *cmd, dev->failed = 0; dev->jstart = jiffies; dev->cur_cmd = cmd; - cmd->scsi_done = done; cmd->result = DID_ERROR << 16; /* default return code */ cmd->SCp.phase = 0; /* bus free */ |