aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/libsas/sas_expander.c
diff options
context:
space:
mode:
authorDarrick J. Wong <djwong@us.ibm.com>2010-10-01 16:55:47 -0400
committerJames Bottomley <James.Bottomley@suse.de>2010-10-08 17:03:42 -0400
commit56dd2c0691a5a387b7b05835fe547dc6fade9407 (patch)
treed7ff3abef5b42f554128b30072aaa1e2aa7eab08 /drivers/scsi/libsas/sas_expander.c
parent39a985547cbfcbb0b23667b69b8ae82a6cf312ac (diff)
[SCSI] libsas: Don't issue commands to devices that have been hot-removed
sd will get hung up issuing commands to flush write cache if a SAS device behind the expander is unplugged without warning. Change libsas to reject commands to domain devices that have already gone away. [maciej.trela@intel.com: removed setting ->gone in sas_deform_port() to permit sync cache commands at module removal] Signed-off-by: Darrick J. Wong <djwong@us.ibm.com> Tested-by: Haipao Fan <haipao.fan@intel.com> Signed-off-by: Maciej Trela <maciej.trela@intel.com> Signed-off-by: Dan Williams <dan.j.williams@intel.com> Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/libsas/sas_expander.c')
-rw-r--r--drivers/scsi/libsas/sas_expander.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/scsi/libsas/sas_expander.c b/drivers/scsi/libsas/sas_expander.c
index 83dd5070a15c..61d81f858a5a 100644
--- a/drivers/scsi/libsas/sas_expander.c
+++ b/drivers/scsi/libsas/sas_expander.c
@@ -1724,6 +1724,7 @@ static void sas_unregister_ex_tree(struct domain_device *dev)
1724 struct domain_device *child, *n; 1724 struct domain_device *child, *n;
1725 1725
1726 list_for_each_entry_safe(child, n, &ex->children, siblings) { 1726 list_for_each_entry_safe(child, n, &ex->children, siblings) {
1727 child->gone = 1;
1727 if (child->dev_type == EDGE_DEV || 1728 if (child->dev_type == EDGE_DEV ||
1728 child->dev_type == FANOUT_DEV) 1729 child->dev_type == FANOUT_DEV)
1729 sas_unregister_ex_tree(child); 1730 sas_unregister_ex_tree(child);
@@ -1744,6 +1745,7 @@ static void sas_unregister_devs_sas_addr(struct domain_device *parent,
1744 &ex_dev->children, siblings) { 1745 &ex_dev->children, siblings) {
1745 if (SAS_ADDR(child->sas_addr) == 1746 if (SAS_ADDR(child->sas_addr) ==
1746 SAS_ADDR(phy->attached_sas_addr)) { 1747 SAS_ADDR(phy->attached_sas_addr)) {
1748 child->gone = 1;
1747 if (child->dev_type == EDGE_DEV || 1749 if (child->dev_type == EDGE_DEV ||
1748 child->dev_type == FANOUT_DEV) 1750 child->dev_type == FANOUT_DEV)
1749 sas_unregister_ex_tree(child); 1751 sas_unregister_ex_tree(child);
@@ -1752,6 +1754,7 @@ static void sas_unregister_devs_sas_addr(struct domain_device *parent,
1752 break; 1754 break;
1753 } 1755 }
1754 } 1756 }
1757 parent->gone = 1;
1755 sas_disable_routing(parent, phy->attached_sas_addr); 1758 sas_disable_routing(parent, phy->attached_sas_addr);
1756 } 1759 }
1757 memset(phy->attached_sas_addr, 0, SAS_ADDR_SIZE); 1760 memset(phy->attached_sas_addr, 0, SAS_ADDR_SIZE);