aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/scsi/isci/remote_device.c
diff options
context:
space:
mode:
authorDan Williams <dan.j.williams@intel.com>2011-06-20 18:11:22 -0400
committerDan Williams <dan.j.williams@intel.com>2011-07-03 07:04:50 -0400
commit980d3aeb3828b0fdf2a0b2e893d238130b014575 (patch)
tree76fcb476d6cb9ab4d8307ec7ff4dcfffbf1d296f /drivers/scsi/isci/remote_device.c
parent77c852f312243192b1f2ce7fc56d678784786692 (diff)
isci: fix isci_terminate_pending() list management
Walk through the list of pending requests being careful to consider that multiple requests can be terminated when the lock is dropped (i.e. invalidating the 'next' reference established by list_for_each_entry_safe). Also noticed that all callers to isci_terminate_pending_requests() specifying terminating, so just drop the parameter. Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/scsi/isci/remote_device.c')
-rw-r--r--drivers/scsi/isci/remote_device.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/isci/remote_device.c b/drivers/scsi/isci/remote_device.c
index 3b555dcbe56..45592ad33c3 100644
--- a/drivers/scsi/isci/remote_device.c
+++ b/drivers/scsi/isci/remote_device.c
@@ -1272,7 +1272,7 @@ void isci_remote_device_nuke_requests(struct isci_host *ihost, struct isci_remot
1272 "%s: idev = %p\n", __func__, idev); 1272 "%s: idev = %p\n", __func__, idev);
1273 1273
1274 /* Cleanup all requests pending for this device. */ 1274 /* Cleanup all requests pending for this device. */
1275 isci_terminate_pending_requests(ihost, idev, terminating); 1275 isci_terminate_pending_requests(ihost, idev);
1276 1276
1277 dev_dbg(&ihost->pdev->dev, 1277 dev_dbg(&ihost->pdev->dev,
1278 "%s: idev = %p, done\n", __func__, idev); 1278 "%s: idev = %p, done\n", __func__, idev);