ncr5380: Proceed with next command after NCR5380_select() calls scsi_done
authorFinn Thain <fthain@telegraphics.com.au>
Sun, 3 Jan 2016 05:05:17 +0000 (16:05 +1100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 7 Jan 2016 02:42:55 +0000 (21:42 -0500)
If a target disappears from the SCSI bus, NCR5380_select() may
subsequently fail with a time-out. In this situation, scsi_done is
called and NCR5380_select() returns 0. Both hostdata->connected and
hostdata->selecting are NULL and the main loop should proceed with
the next command in the issue queue. Clarify this logic.

Signed-off-by: Finn Thain <fthain@telegraphics.com.au>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Tested-by: Ondrej Zary <linux@rainbow-software.org>
Tested-by: Michael Schmitz <schmitzmic@gmail.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/NCR5380.c
drivers/scsi/atari_NCR5380.c

index 051684b08a775bf186f0167089be462fb2020d56..4ef4fc7bd6ebf1f06221298206c181324eb6ceed 100644 (file)
@@ -1007,14 +1007,18 @@ static void NCR5380_main(struct work_struct *work)
                                         */
 
                                        if (!NCR5380_select(instance, tmp)) {
-                                               break;
+                                               /* OK or bad target */
                                        } else {
+                                               /* Need to retry */
                                                LIST(tmp, hostdata->issue_queue);
                                                tmp->host_scribble = (unsigned char *) hostdata->issue_queue;
                                                hostdata->issue_queue = tmp;
                                                done = 0;
                                                dprintk(NDEBUG_MAIN|NDEBUG_QUEUES, "scsi%d : main(): select() failed, returned to issue_queue\n", instance->host_no);
                                        }
+                                       if (hostdata->connected ||
+                                           hostdata->selecting)
+                                               break;
                                        /* lock held here still */
                                }       /* if target/lun is not busy */
                        }       /* for */
@@ -1024,7 +1028,7 @@ static void NCR5380_main(struct work_struct *work)
                        tmp = (struct scsi_cmnd *) hostdata->selecting;
                        /* Selection will drop and retake the lock */
                        if (!NCR5380_select(instance, tmp)) {
-                               /* Ok ?? */
+                               /* OK or bad target */
                        } else {
                                /* RvC: device failed, so we wait a long time
                                   this is needed for Mustek scanners, that
index 80f6830d9f3c5dc5e725e47e694c3d9c45e2a29e..bedc7e2efa1dde2ba612b7d9822110c69e64325e 100644 (file)
@@ -1139,13 +1139,13 @@ static void NCR5380_main(struct work_struct *work)
                                        cmd_get_tag(tmp, tmp->cmnd[0] != REQUEST_SENSE);
 #endif
                                        if (!NCR5380_select(instance, tmp)) {
+                                               /* OK or bad target */
                                                local_irq_disable();
                                                hostdata->retain_dma_intr--;
-                                               /* release if target did not response! */
                                                maybe_release_dma_irq(instance);
                                                local_irq_restore(flags);
-                                               break;
                                        } else {
+                                               /* Need to retry */
                                                local_irq_disable();
                                                LIST(tmp, hostdata->issue_queue);
                                                SET_NEXT(tmp, hostdata->issue_queue);
@@ -1157,9 +1157,9 @@ static void NCR5380_main(struct work_struct *work)
                                                local_irq_restore(flags);
                                                dprintk(NDEBUG_MAIN, "scsi%d: main(): select() failed, "
                                                            "returned to issue_queue\n", HOSTNO);
-                                               if (hostdata->connected)
-                                                       break;
                                        }
+                                       if (hostdata->connected)
+                                               break;
                                } /* if target/lun/target queue is not busy */
                        } /* for issue_queue */
                } /* if (!hostdata->connected) */