On Thu, Jun 02, 2022 at 08:10:28AM -0700, Brian Bunker wrote: > static void scsi_io_completion_reprep(struct scsi_cmnd *cmd, > - struct request_queue *q) > + struct request_queue *q, > unsigned long msecs) > { > /* A new command will be prepared and issued. */ > - scsi_mq_requeue_cmd(cmd); > + scsi_mq_requeue_cmd(cmd, msecs); q is unused. But I think it is better if we just kill this pointless wrapper anyway. > + case ACTION_DELAYED_REPREP: > + scsi_io_completion_reprep(cmd, q, ALUA_TRANSITION_REPREP_DELAY); This is using spaces where it should use tabs.