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 /drivers/scsi/libata-eh.c | |
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>
Diffstat (limited to 'drivers/scsi/libata-eh.c')
-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"); |