Commit 56dd2c06 authored by Darrick J. Wong's avatar Darrick J. Wong Committed by James Bottomley

[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: default avatarDarrick J. Wong <djwong@us.ibm.com>
Tested-by: default avatarHaipao Fan <haipao.fan@intel.com>
Signed-off-by: default avatarMaciej Trela <maciej.trela@intel.com>
Signed-off-by: default avatarDan Williams <dan.j.williams@intel.com>
Signed-off-by: default avatarJames Bottomley <James.Bottomley@suse.de>
parent 39a98554
...@@ -162,6 +162,10 @@ static unsigned int sas_ata_qc_issue(struct ata_queued_cmd *qc) ...@@ -162,6 +162,10 @@ static unsigned int sas_ata_qc_issue(struct ata_queued_cmd *qc)
unsigned int xfer = 0; unsigned int xfer = 0;
unsigned int si; unsigned int si;
/* If the device fell off, no sense in issuing commands */
if (dev->gone)
return AC_ERR_SYSTEM;
task = sas_alloc_task(GFP_ATOMIC); task = sas_alloc_task(GFP_ATOMIC);
if (!task) if (!task)
return AC_ERR_SYSTEM; return AC_ERR_SYSTEM;
......
...@@ -1724,6 +1724,7 @@ static void sas_unregister_ex_tree(struct domain_device *dev) ...@@ -1724,6 +1724,7 @@ static void sas_unregister_ex_tree(struct domain_device *dev)
struct domain_device *child, *n; struct domain_device *child, *n;
list_for_each_entry_safe(child, n, &ex->children, siblings) { list_for_each_entry_safe(child, n, &ex->children, siblings) {
child->gone = 1;
if (child->dev_type == EDGE_DEV || if (child->dev_type == EDGE_DEV ||
child->dev_type == FANOUT_DEV) child->dev_type == FANOUT_DEV)
sas_unregister_ex_tree(child); sas_unregister_ex_tree(child);
...@@ -1744,6 +1745,7 @@ static void sas_unregister_devs_sas_addr(struct domain_device *parent, ...@@ -1744,6 +1745,7 @@ static void sas_unregister_devs_sas_addr(struct domain_device *parent,
&ex_dev->children, siblings) { &ex_dev->children, siblings) {
if (SAS_ADDR(child->sas_addr) == if (SAS_ADDR(child->sas_addr) ==
SAS_ADDR(phy->attached_sas_addr)) { SAS_ADDR(phy->attached_sas_addr)) {
child->gone = 1;
if (child->dev_type == EDGE_DEV || if (child->dev_type == EDGE_DEV ||
child->dev_type == FANOUT_DEV) child->dev_type == FANOUT_DEV)
sas_unregister_ex_tree(child); sas_unregister_ex_tree(child);
...@@ -1752,6 +1754,7 @@ static void sas_unregister_devs_sas_addr(struct domain_device *parent, ...@@ -1752,6 +1754,7 @@ static void sas_unregister_devs_sas_addr(struct domain_device *parent,
break; break;
} }
} }
parent->gone = 1;
sas_disable_routing(parent, phy->attached_sas_addr); sas_disable_routing(parent, phy->attached_sas_addr);
} }
memset(phy->attached_sas_addr, 0, SAS_ADDR_SIZE); memset(phy->attached_sas_addr, 0, SAS_ADDR_SIZE);
......
...@@ -228,6 +228,13 @@ int sas_queuecommand(struct scsi_cmnd *cmd, ...@@ -228,6 +228,13 @@ int sas_queuecommand(struct scsi_cmnd *cmd,
goto out; goto out;
} }
/* If the device fell off, no sense in issuing commands */
if (dev->gone) {
cmd->result = DID_BAD_TARGET << 16;
scsi_done(cmd);
goto out;
}
res = -ENOMEM; res = -ENOMEM;
task = sas_create_task(cmd, dev, GFP_ATOMIC); task = sas_create_task(cmd, dev, GFP_ATOMIC);
if (!task) if (!task)
......
...@@ -205,6 +205,7 @@ struct domain_device { ...@@ -205,6 +205,7 @@ struct domain_device {
}; };
void *lldd_dev; void *lldd_dev;
int gone;
}; };
struct sas_discovery_event { struct sas_discovery_event {
......
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