diff options
| author |   Dan Williams <dan.j.williams@intel.com> | 2012-06-21 23:36:15 -0700 |
|---|---|---|
| committer |   James Bottomley <JBottomley@Parallels.com> | 2012-07-20 08:58:52 +0100 |
| commit | 26f2f199ff150d8876b2641c41e60d1c92d2fb81 (patch) | |
| tree | 00e731f023ef8df4dd7ed80acec4e5d1945a7954 /drivers/scsi | |
| parent | [SCSI] libsas: sas_rediscover_dev did not look at the SMP exec status. (diff) | |
| download | linux-dev-26f2f199ff150d8876b2641c41e60d1c92d2fb81.tar.xz linux-dev-26f2f199ff150d8876b2641c41e60d1c92d2fb81.zip | |
[SCSI] libsas: continue revalidation
Continue running revalidation until no more broadcast devices are
discovered. Fixes cases where re-discovery completes too early in a
domain with multiple expanders with pending re-discovery events.
Servicing BCNs can get backed up behind error recovery.
Cc: <stable@vger.kernel.org>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers/scsi')
| -rw-r--r-- | drivers/scsi/libsas/sas_expander.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/scsi/libsas/sas_expander.c b/drivers/scsi/libsas/sas_expander.c index af659cc8308c..63c574227ebf 100644 --- a/drivers/scsi/libsas/sas_expander.c +++ b/drivers/scsi/libsas/sas_expander.c @@ -2114,9 +2114,7 @@ int sas_ex_revalidate_domain(struct domain_device *port_dev) struct domain_device *dev = NULL; res = sas_find_bcast_dev(port_dev, &dev); - if (res) - goto out; - if (dev) { + while (res == 0 && dev) { struct expander_device *ex = &dev->ex_dev; int i = 0, phy_id; @@ -2128,8 +2126,10 @@ int sas_ex_revalidate_domain(struct domain_device *port_dev) res = sas_rediscover(dev, phy_id); i = phy_id + 1; } while (i < ex->num_phys); + + dev = NULL; + res = sas_find_bcast_dev(port_dev, &dev); } -out: return res; } |