iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

scsi: storvsc_drv: Call scsi_done() directly

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

Link: https://lore.kernel.org/r/20211007202923.2174984-74-bvanassche@acm.org
Reviewed-by: Haiyang Zhang <haiyangz@microsoft.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
This commit is contained in:
Bart Van Assche 2021-10-07 13:29:08 -07:00 committed by Martin K. Petersen
parent 4acf838e80
commit 0c31fa0e66
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
drivers/scsi/storvsc_drv.c
View File

@ -1154,7 +1154,7 @@ static void storvsc_command_completion(struct storvsc_cmd_request *cmd_request,
scsi_set_resid(scmnd, scsi_set_resid(scmnd,
cmd_request->payload->range.len - data_transfer_length); cmd_request->payload->range.len - data_transfer_length);
scmnd->scsi_done(scmnd); scsi_done(scmnd);
if (payload_sz > if (payload_sz >
sizeof(struct vmbus_channel_packet_multipage_buffer)) sizeof(struct vmbus_channel_packet_multipage_buffer))
@ -1753,7 +1753,7 @@ static int storvsc_queuecommand(struct Scsi_Host *host, struct scsi_cmnd *scmnd)
* future versions of the host. * future versions of the host.
*/ */
if (!storvsc_scsi_cmd_ok(scmnd)) { if (!storvsc_scsi_cmd_ok(scmnd)) {
scmnd->scsi_done(scmnd); scsi_done(scmnd);
return 0; return 0;
} }
} }