RISC-V: Improve /proc/cpuinfo output for ISA extensions

[linux.git] / drivers / scsi / scsi_error.c

diff --git a/drivers/scsi/scsi_error.c b/drivers/scsi/scsi_error.c
index 3eae2392ef158f2060f1f585dacebc015562709a..60a6ae9d1219f9e4ca34d3b5ec750c0ad769e552 100644 (file)
--- a/drivers/scsi/scsi_error.c
+++ b/drivers/scsi/scsi_error.c
@@ -133,23 +133,6 @@ static bool scsi_eh_should_retry_cmd(struct scsi_cmnd *cmd)
        return true;
 }
 
-static void scsi_eh_complete_abort(struct scsi_cmnd *scmd, struct Scsi_Host *shost)
-{
-       unsigned long flags;
-
-       spin_lock_irqsave(shost->host_lock, flags);
-       list_del_init(&scmd->eh_entry);
-       /*
-        * If the abort succeeds, and there is no further
-        * EH action, clear the ->last_reset time.
-        */
-       if (list_empty(&shost->eh_abort_list) &&
-           list_empty(&shost->eh_cmd_q))
-               if (shost->eh_deadline != -1)
-                       shost->last_reset = 0;
-       spin_unlock_irqrestore(shost->host_lock, flags);
-}
-
 /**
  * scmd_eh_abort_handler - Handle command aborts
  * @work:      command to be aborted.
@@ -166,54 +149,72 @@ scmd_eh_abort_handler(struct work_struct *work)
        struct scsi_cmnd *scmd =
                container_of(work, struct scsi_cmnd, abort_work.work);
        struct scsi_device *sdev = scmd->device;
+       struct Scsi_Host *shost = sdev->host;
        enum scsi_disposition rtn;
        unsigned long flags;
 
-       if (scsi_host_eh_past_deadline(sdev->host)) {
+       if (scsi_host_eh_past_deadline(shost)) {
                SCSI_LOG_ERROR_RECOVERY(3,
                        scmd_printk(KERN_INFO, scmd,
                                    "eh timeout, not aborting\n"));
-       } else {
-               SCSI_LOG_ERROR_RECOVERY(3,
+               goto out;
+       }
+
+       SCSI_LOG_ERROR_RECOVERY(3,
                        scmd_printk(KERN_INFO, scmd,
                                    "aborting command\n"));
-               rtn = scsi_try_to_abort_cmd(sdev->host->hostt, scmd);
-               if (rtn == SUCCESS) {
-                       set_host_byte(scmd, DID_TIME_OUT);
-                       if (scsi_host_eh_past_deadline(sdev->host)) {
-                               SCSI_LOG_ERROR_RECOVERY(3,
-                                       scmd_printk(KERN_INFO, scmd,
-                                                   "eh timeout, not retrying "
-                                                   "aborted command\n"));
-                       } else if (!scsi_noretry_cmd(scmd) &&
-                                  scsi_cmd_retry_allowed(scmd) &&
-                               scsi_eh_should_retry_cmd(scmd)) {
-                               SCSI_LOG_ERROR_RECOVERY(3,
-                                       scmd_printk(KERN_WARNING, scmd,
-                                                   "retry aborted command\n"));
-                               scsi_eh_complete_abort(scmd, sdev->host);
-                               scsi_queue_insert(scmd, SCSI_MLQUEUE_EH_RETRY);
-                               return;
-                       } else {
-                               SCSI_LOG_ERROR_RECOVERY(3,
-                                       scmd_printk(KERN_WARNING, scmd,
-                                                   "finish aborted command\n"));
-                               scsi_eh_complete_abort(scmd, sdev->host);
-                               scsi_finish_command(scmd);
-                               return;
-                       }
-               } else {
-                       SCSI_LOG_ERROR_RECOVERY(3,
-                               scmd_printk(KERN_INFO, scmd,
-                                           "cmd abort %s\n",
-                                           (rtn == FAST_IO_FAIL) ?
-                                           "not send" : "failed"));
-               }
+       rtn = scsi_try_to_abort_cmd(shost->hostt, scmd);
+       if (rtn != SUCCESS) {
+               SCSI_LOG_ERROR_RECOVERY(3,
+                       scmd_printk(KERN_INFO, scmd,
+                                   "cmd abort %s\n",
+                                   (rtn == FAST_IO_FAIL) ?
+                                   "not send" : "failed"));
+               goto out;
+       }
+       set_host_byte(scmd, DID_TIME_OUT);
+       if (scsi_host_eh_past_deadline(shost)) {
+               SCSI_LOG_ERROR_RECOVERY(3,
+                       scmd_printk(KERN_INFO, scmd,
+                                   "eh timeout, not retrying "
+                                   "aborted command\n"));
+               goto out;
        }
 
-       spin_lock_irqsave(sdev->host->host_lock, flags);
+       spin_lock_irqsave(shost->host_lock, flags);
        list_del_init(&scmd->eh_entry);
-       spin_unlock_irqrestore(sdev->host->host_lock, flags);
+
+       /*
+        * If the abort succeeds, and there is no further
+        * EH action, clear the ->last_reset time.
+        */
+       if (list_empty(&shost->eh_abort_list) &&
+           list_empty(&shost->eh_cmd_q))
+               if (shost->eh_deadline != -1)
+                       shost->last_reset = 0;
+
+       spin_unlock_irqrestore(shost->host_lock, flags);
+
+       if (!scsi_noretry_cmd(scmd) &&
+           scsi_cmd_retry_allowed(scmd) &&
+           scsi_eh_should_retry_cmd(scmd)) {
+               SCSI_LOG_ERROR_RECOVERY(3,
+                       scmd_printk(KERN_WARNING, scmd,
+                                   "retry aborted command\n"));
+               scsi_queue_insert(scmd, SCSI_MLQUEUE_EH_RETRY);
+       } else {
+               SCSI_LOG_ERROR_RECOVERY(3,
+                       scmd_printk(KERN_WARNING, scmd,
+                                   "finish aborted command\n"));
+               scsi_finish_command(scmd);
+       }
+       return;
+
+out:
+       spin_lock_irqsave(shost->host_lock, flags);
+       list_del_init(&scmd->eh_entry);
+       spin_unlock_irqrestore(shost->host_lock, flags);
+
        scsi_eh_scmd_add(scmd);
 }
 
@@ -1429,7 +1430,8 @@ static int scsi_eh_try_stu(struct scsi_cmnd *scmd)
                enum scsi_disposition rtn = NEEDS_RETRY;
 
                for (i = 0; rtn == NEEDS_RETRY && i < 2; i++)
-                       rtn = scsi_send_eh_cmnd(scmd, stu_command, 6, scmd->device->request_queue->rq_timeout, 0);
+                       rtn = scsi_send_eh_cmnd(scmd, stu_command, 6,
+                                               scmd->device->eh_timeout, 0);
 
                if (rtn == SUCCESS)
                        return 0;