diff options
| author |   Bart Van Assche <bvanassche@acm.org> | 2021-10-07 13:28:56 -0700 |
|---|---|---|
| committer |   Martin K. Petersen <martin.petersen@oracle.com> | 2021-10-16 21:31:40 -0400 |
| commit | 7bc195c7513430aef09b3ef1f7438b58b3b7f12b (patch) | |
| tree | 06698aac9a93e7129a2e9fdb2c191a6685b1af99 | |
| parent | scsi: pmcraid: Call scsi_done() directly (diff) | |
| download | linux-dev-7bc195c7513430aef09b3ef1f7438b58b3b7f12b.tar.xz linux-dev-7bc195c7513430aef09b3ef1f7438b58b3b7f12b.zip | |
scsi: ppa: Call scsi_done() directly
Conditional statements are faster than indirect calls. Hence call
scsi_done() directly.
Link: https://lore.kernel.org/r/20211007202923.2174984-62-bvanassche@acm.org
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
| -rw-r--r-- | drivers/scsi/ppa.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/ppa.c b/drivers/scsi/ppa.c index 977315fdc254..799ad8562e24 100644 --- a/drivers/scsi/ppa.c +++ b/drivers/scsi/ppa.c @@ -665,7 +665,7 @@ static void ppa_interrupt(struct work_struct *work) dev->cur_cmd = NULL; - cmd->scsi_done(cmd); + scsi_done(cmd); } static int ppa_engine(ppa_struct *dev, struct scsi_cmnd *cmd) @@ -798,7 +798,6 @@ static int ppa_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 */ |