| From 38d12b560cf2044b498e8bbef3cf6acf9610e7ac Mon Sep 17 00:00:00 2001 |
| From: Dan Williams <dan.j.williams@intel.com> |
| Date: Thu, 21 Jun 2012 23:36:15 -0700 |
| Subject: [PATCH] libsas: continue revalidation |
| |
| commit 26f2f199ff150d8876b2641c41e60d1c92d2fb81 upstream. |
| |
| 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. |
| |
| Signed-off-by: Dan Williams <dan.j.williams@intel.com> |
| Signed-off-by: James Bottomley <JBottomley@Parallels.com> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| --- |
| drivers/scsi/libsas/sas_expander.c | 8 ++++---- |
| 1 file changed, 4 insertions(+), 4 deletions(-) |
| |
| diff --git a/drivers/scsi/libsas/sas_expander.c b/drivers/scsi/libsas/sas_expander.c |
| index 4ee42bb48dcd..cb4964b54191 100644 |
| --- a/drivers/scsi/libsas/sas_expander.c |
| +++ b/drivers/scsi/libsas/sas_expander.c |
| @@ -1947,9 +1947,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; |
| |
| @@ -1961,8 +1959,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; |
| } |
| |
| -- |
| 1.8.5.2 |
| |