Commit 705c1a5b authored by Dan Williams's avatar Dan Williams Committed by Ben Hutchings

libsas: continue revalidation

commit 26f2f199 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: default avatarDan Williams <dan.j.williams@intel.com>
Signed-off-by: default avatarJames Bottomley <JBottomley@Parallels.com>
Signed-off-by: default avatarBen Hutchings <ben@decadent.org.uk>
parent c6e92669
...@@ -1983,9 +1983,7 @@ int sas_ex_revalidate_domain(struct domain_device *port_dev) ...@@ -1983,9 +1983,7 @@ int sas_ex_revalidate_domain(struct domain_device *port_dev)
struct domain_device *dev = NULL; struct domain_device *dev = NULL;
res = sas_find_bcast_dev(port_dev, &dev); res = sas_find_bcast_dev(port_dev, &dev);
if (res) while (res == 0 && dev) {
goto out;
if (dev) {
struct expander_device *ex = &dev->ex_dev; struct expander_device *ex = &dev->ex_dev;
int i = 0, phy_id; int i = 0, phy_id;
...@@ -1997,8 +1995,10 @@ int sas_ex_revalidate_domain(struct domain_device *port_dev) ...@@ -1997,8 +1995,10 @@ int sas_ex_revalidate_domain(struct domain_device *port_dev)
res = sas_rediscover(dev, phy_id); res = sas_rediscover(dev, phy_id);
i = phy_id + 1; i = phy_id + 1;
} while (i < ex->num_phys); } while (i < ex->num_phys);
dev = NULL;
res = sas_find_bcast_dev(port_dev, &dev);
} }
out:
return res; return res;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment