diff options
author | Dan Williams <dan.j.williams@intel.com> | 2011-12-21 18:19:56 -0500 |
---|---|---|
committer | James Bottomley <JBottomley@Parallels.com> | 2012-02-29 13:55:12 -0500 |
commit | 3a9c5560f677690f65038f399f4f598c79b83186 (patch) | |
tree | 1e15d427fd86260fc26670bc4ceff61d51ef025e /drivers | |
parent | 0508c2f3b701f3cd8ed52b2a4abbb2a670f69ce2 (diff) |
[SCSI] libsas: check for 'gone' expanders in smp_execute_task()
No sense in issuing or retrying commands to an expander that has been
removed.
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/scsi/libsas/sas_expander.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/scsi/libsas/sas_expander.c b/drivers/scsi/libsas/sas_expander.c index 7701ab588404..6fb1f3afd1e0 100644 --- a/drivers/scsi/libsas/sas_expander.c +++ b/drivers/scsi/libsas/sas_expander.c | |||
@@ -74,6 +74,11 @@ static int smp_execute_task(struct domain_device *dev, void *req, int req_size, | |||
74 | 74 | ||
75 | mutex_lock(&dev->ex_dev.cmd_mutex); | 75 | mutex_lock(&dev->ex_dev.cmd_mutex); |
76 | for (retry = 0; retry < 3; retry++) { | 76 | for (retry = 0; retry < 3; retry++) { |
77 | if (test_bit(SAS_DEV_GONE, &dev->state)) { | ||
78 | res = -ECOMM; | ||
79 | break; | ||
80 | } | ||
81 | |||
77 | task = sas_alloc_task(GFP_KERNEL); | 82 | task = sas_alloc_task(GFP_KERNEL); |
78 | if (!task) { | 83 | if (!task) { |
79 | res = -ENOMEM; | 84 | res = -ENOMEM; |