scsi: aic79xx: Restore modes when exiting ahd_linux_queue_abort_cmd()
authorHannes Reinecke <hare@suse.de>
Tue, 14 Jul 2020 16:03:01 +0000 (18:03 +0200)
committerMartin K. Petersen <martin.petersen@oracle.com>
Sat, 25 Jul 2020 02:09:54 +0000 (22:09 -0400)
ahd_linux_queue_abort_cmd() calls ahd_save_modes() without calling
ahd_restore_modes() before exiting.

Link: https://lore.kernel.org/r/20200714160301.4482-1-hare@suse.de
Signed-off-by: Hannes Reinecke <hare@suse.de>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/aic7xxx/aic79xx_osm.c

index 3782a20..d019e3f 100644 (file)
@@ -2143,7 +2143,7 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
        u_int  last_phase;
        u_int  saved_scsiid;
        u_int  cdb_byte;
-       int    retval;
+       int    retval = SUCCESS;
        int    was_paused;
        int    paused;
        int    wait;
@@ -2181,8 +2181,7 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
                 * so we must not still own the command.
                 */
                scmd_printk(KERN_INFO, cmd, "Is not an active device\n");
-               retval = SUCCESS;
-               goto no_cmd;
+               goto done;
        }
 
        /*
@@ -2195,7 +2194,7 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
 
        if (pending_scb == NULL) {
                scmd_printk(KERN_INFO, cmd, "Command not found\n");
-               goto no_cmd;
+               goto done;
        }
 
        if ((pending_scb->flags & SCB_RECOVERY_SCB) != 0) {
@@ -2203,7 +2202,7 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
                 * We can't queue two recovery actions using the same SCB
                 */
                retval = FAILED;
-               goto  done;
+               goto done;
        }
 
        /*
@@ -2218,7 +2217,7 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
 
        if ((pending_scb->flags & SCB_ACTIVE) == 0) {
                scmd_printk(KERN_INFO, cmd, "Command already completed\n");
-               goto no_cmd;
+               goto done;
        }
 
        printk("%s: At time of recovery, card was %spaused\n",
@@ -2235,7 +2234,6 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
                printk("%s:%d:%d:%d: Cmd aborted from QINFIFO\n",
                       ahd_name(ahd), cmd->device->channel, 
                       cmd->device->id, (u8)cmd->device->lun);
-               retval = SUCCESS;
                goto done;
        }
 
@@ -2332,17 +2330,10 @@ ahd_linux_queue_abort_cmd(struct scsi_cmnd *cmd)
        } else {
                scmd_printk(KERN_INFO, cmd, "Unable to deliver message\n");
                retval = FAILED;
-               goto done;
        }
 
-no_cmd:
-       /*
-        * Our assumption is that if we don't have the command, no
-        * recovery action was required, so we return success.  Again,
-        * the semantics of the mid-layer recovery engine are not
-        * well defined, so this may change in time.
-        */
-       retval = SUCCESS;
+
+       ahd_restore_modes(ahd, saved_modes);
 done:
        if (paused)
                ahd_unpause(ahd);