diff options
| author | Tejun Heo <htejun@gmail.com> | 2006-06-12 01:11:01 -0400 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2006-06-12 02:10:26 -0400 |
| commit | f9be7113b56eb5d013a611d9c65e8d1d2d3f7c94 (patch) | |
| tree | e500b57b88989563a4b073f34a1ce8a9a1f48353 | |
| parent | 551e8889509c30cda31e464e5cbe7572c4e1febd (diff) | |
[PATCH] libata: fix ATA_EH_REVALIDATE clearing
ATA_EH_REVALIDATE should be cleared after all devices on the target
port have been revalidated. Fix ata_eh_revalidate_and_attach()
accordingly.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
| -rw-r--r-- | drivers/scsi/libata-eh.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/scsi/libata-eh.c b/drivers/scsi/libata-eh.c index f82799e22588..782dfba573a9 100644 --- a/drivers/scsi/libata-eh.c +++ b/drivers/scsi/libata-eh.c | |||
| @@ -1556,8 +1556,6 @@ static int ata_eh_revalidate_and_attach(struct ata_port *ap, | |||
| 1556 | 1556 | ||
| 1557 | /* schedule the scsi_rescan_device() here */ | 1557 | /* schedule the scsi_rescan_device() here */ |
| 1558 | queue_work(ata_aux_wq, &(ap->scsi_rescan_task)); | 1558 | queue_work(ata_aux_wq, &(ap->scsi_rescan_task)); |
| 1559 | |||
| 1560 | ehc->i.action &= ~ATA_EH_REVALIDATE; | ||
| 1561 | } else if (dev->class == ATA_DEV_UNKNOWN && | 1559 | } else if (dev->class == ATA_DEV_UNKNOWN && |
| 1562 | ehc->tries[dev->devno] && | 1560 | ehc->tries[dev->devno] && |
| 1563 | ata_class_enabled(ehc->classes[dev->devno])) { | 1561 | ata_class_enabled(ehc->classes[dev->devno])) { |
| @@ -1578,7 +1576,9 @@ static int ata_eh_revalidate_and_attach(struct ata_port *ap, | |||
| 1578 | } | 1576 | } |
| 1579 | } | 1577 | } |
| 1580 | 1578 | ||
| 1581 | if (rc) | 1579 | if (rc == 0) |
| 1580 | ehc->i.action &= ~ATA_EH_REVALIDATE; | ||
| 1581 | else | ||
| 1582 | *r_failed_dev = dev; | 1582 | *r_failed_dev = dev; |
| 1583 | 1583 | ||
| 1584 | DPRINTK("EXIT\n"); | 1584 | DPRINTK("EXIT\n"); |
