* Driver does a PCIe read to check whether HBA is hot unplugged or
not. If the returned vendor ID is 0xFFFFFFFF/0x0, then this
indicates that the device might have been hot removed and the device
will be removed from driver.
* In the PCI device remove() callback function, flush out all
the outstanding IOs with 'DID_NO_CONNECT' before removing the drives
attached to the HBA.
* In the TM abort() callback function return the scmd with
"DID_NO_CONNECT" status and free the smid, if driver detects that
HBA is hot unplugged.
* In the hard reset flush out all the outstanding IOs even if diag reset
fails and also if driver detects that HBA is hot unplugged.
v1 change set:
unlock mutex before goto "out_unlocked",
if active reset is in progress.
Signed-off-by: Suganath Prabu S <suganath-prabu.subram...@broadcom.com>
---
drivers/scsi/mpt3sas/mpt3sas_base.c | 62 +++++++++++++++++++++++++++++++++++-
drivers/scsi/mpt3sas/mpt3sas_base.h | 3 +-
drivers/scsi/mpt3sas/mpt3sas_scsih.c | 42 +++++++++++++++++++++---
3 files changed, 100 insertions(+), 7 deletions(-)
diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.c
b/drivers/scsi/mpt3sas/mpt3sas_base.c
index 836b2e1..8b33670 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_base.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_base.c
@@ -61,7 +61,7 @@
#include <linux/kthread.h>
#include <asm/page.h> /* To get host page size per arch */
#include <linux/aer.h>
-
+#include <../drivers/pci/pci.h>
#include "mpt3sas_base.h"
@@ -543,6 +543,44 @@ static int mpt3sas_remove_dead_ioc_func(void *arg)
}
/**
+ * mpt3sas_base_pci_device_is_unplugged - Check whether HBA device is
+ * hot unplugged or not
+ * @ioc: per adapter object
+ *
+ * Return 1 if the HBA device is hot unplugged else return 0.
+ */
+u8
+mpt3sas_base_pci_device_is_unplugged(struct MPT3SAS_ADAPTER *ioc)
+{
+ struct pci_dev *pdev = ioc->pdev;
+ int devfn = pdev->devfn;
+ u32 l;
+
+ if (pci_bus_read_dev_vendor_id(pdev->bus, devfn, &l, 0))
+ return 0;
+
+ return 1;
+}
+
+/**
+ * mpt3sas_base_pci_device_is_available - check whether pci device is
+ * available for any transactions with FW
+ *
+ * @ioc: per adapter object
+ *
+ * Return 1 if pci device state is up and running else return 0.
+ */
+u8
+mpt3sas_base_pci_device_is_available(struct MPT3SAS_ADAPTER *ioc)
+{
+ if (ioc->pci_error_recovery ||
+ mpt3sas_base_pci_device_is_unplugged(ioc))
+ return 0;
+
+ return 1;
+}
+
+/**
* _base_fault_reset_work - workq handling ioc fault conditions
* @work: input argument, used to derive ioc
*
@@ -6853,6 +6891,14 @@ mpt3sas_wait_for_commands_to_complete(struct
MPT3SAS_ADAPTER *ioc)
ioc->pending_io_count = 0;
+ if (!mpt3sas_base_pci_device_is_available(ioc)) {
+ pr_err(MPT3SAS_FMT
+ "%s: pci error recovery reset or"
+ " pci device unplug occurred\n",
+ ioc->name, __func__);
+ return;
+ }
+
ioc_state = mpt3sas_base_get_iocstate(ioc, 0);
if ((ioc_state & MPI2_IOC_STATE_MASK) != MPI2_IOC_STATE_OPERATIONAL)
return;
@@ -6899,6 +6945,20 @@ mpt3sas_base_hard_reset_handler(struct MPT3SAS_ADAPTER
*ioc,
/* wait for an active reset in progress to complete */
mutex_lock(&ioc->reset_in_progress_mutex);
+ if (!mpt3sas_base_pci_device_is_available(ioc)) {
+ pr_err(MPT3SAS_FMT
+ "%s: pci error recovery reset or"
+ " pci device unplug occurred\n",
+ ioc->name, __func__);
+ if (mpt3sas_base_pci_device_is_unplugged(ioc))
+ ioc->schedule_dead_ioc_flush_running_cmds(ioc);
+ r = 0;
+ mutex_unlock(&ioc->reset_in_progress_mutex);
+ goto out_unlocked;
+ }
+
+ mpt3sas_halt_firmware(ioc);
+
spin_lock_irqsave(&ioc->ioc_reset_in_progress_lock, flags);
ioc->shost_recovery = 1;
spin_unlock_irqrestore(&ioc->ioc_reset_in_progress_lock, flags);
diff --git a/drivers/scsi/mpt3sas/mpt3sas_base.h
b/drivers/scsi/mpt3sas/mpt3sas_base.h
index 96dc15e..8ee3ba7 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_base.h
+++ b/drivers/scsi/mpt3sas/mpt3sas_base.h
@@ -1474,7 +1474,8 @@ void mpt3sas_base_update_missing_delay(struct
MPT3SAS_ADAPTER *ioc,
u16 device_missing_delay, u8 io_missing_delay);
int mpt3sas_port_enable(struct MPT3SAS_ADAPTER *ioc);
-
+u8 mpt3sas_base_pci_device_is_unplugged(struct MPT3SAS_ADAPTER *ioc);
+u8 mpt3sas_base_pci_device_is_available(struct MPT3SAS_ADAPTER *ioc);
void
mpt3sas_wait_for_commands_to_complete(struct MPT3SAS_ADAPTER *ioc);
diff --git a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
index 8dd3d67..eeee9da 100644
--- a/drivers/scsi/mpt3sas/mpt3sas_scsih.c
+++ b/drivers/scsi/mpt3sas/mpt3sas_scsih.c
@@ -2846,9 +2846,19 @@ scsih_abort(struct scsi_cmnd *scmd)
"attempting task abort! scmd(%p)\n", scmd);
_scsih_tm_display_info(ioc, scmd);
+ if (mpt3sas_base_pci_device_is_unplugged(ioc) || ioc->remove_host) {
+ sdev_printk(KERN_INFO, scmd->device, "%s scmd(%p)\n",
+ ((ioc->remove_host) ? ("shost is getting removed!") :
+ ("pci device been removed!")), scmd);
+ if (st && st->smid)
+ mpt3sas_base_free_smid(ioc, st->smid);
+ scmd->result = DID_NO_CONNECT << 16;
+ r = FAST_IO_FAIL;
+ goto out;
+ }
+
sas_device_priv_data = scmd->device->hostdata;
- if (!sas_device_priv_data || !sas_device_priv_data->sas_target ||
- ioc->remove_host) {
+ if (!sas_device_priv_data || !sas_device_priv_data->sas_target) {
sdev_printk(KERN_INFO, scmd->device,
"device been deleted! scmd(%p)\n", scmd);
scmd->result = DID_NO_CONNECT << 16;
@@ -2918,6 +2928,15 @@ scsih_dev_reset(struct scsi_cmnd *scmd)
"attempting device reset! scmd(%p)\n", scmd);
_scsih_tm_display_info(ioc, scmd);
+ if (mpt3sas_base_pci_device_is_unplugged(ioc) || ioc->remove_host) {
+ sdev_printk(KERN_INFO, scmd->device, "%s scmd(%p)\n",
+ ((ioc->remove_host) ? ("shost is getting removed!") :
+ ("pci device been removed!")), scmd);
+ scmd->result = DID_NO_CONNECT << 16;
+ r = FAST_IO_FAIL;
+ goto out;
+ }
+
sas_device_priv_data = scmd->device->hostdata;
if (!sas_device_priv_data || !sas_device_priv_data->sas_target ||
ioc->remove_host) {
@@ -2995,9 +3014,17 @@ scsih_target_reset(struct scsi_cmnd *scmd)
scmd);
_scsih_tm_display_info(ioc, scmd);
+ if (mpt3sas_base_pci_device_is_unplugged(ioc) || ioc->remove_host) {
+ sdev_printk(KERN_INFO, scmd->device, "%s scmd(%p)\n",
+ ((ioc->remove_host) ? ("shost is getting removed!") :
+ ("pci device been removed!")), scmd);
+ scmd->result = DID_NO_CONNECT << 16;
+ r = FAST_IO_FAIL;
+ goto out;
+ }
+
sas_device_priv_data = scmd->device->hostdata;
- if (!sas_device_priv_data || !sas_device_priv_data->sas_target ||
- ioc->remove_host) {
+ if (!sas_device_priv_data || !sas_device_priv_data->sas_target) {
starget_printk(KERN_INFO, starget, "target been deleted!
scmd(%p)\n",
scmd);
scmd->result = DID_NO_CONNECT << 16;
@@ -4474,7 +4501,9 @@ _scsih_flush_running_cmds(struct MPT3SAS_ADAPTER *ioc)
st = scsi_cmd_priv(scmd);
mpt3sas_base_clear_st(ioc, st);
scsi_dma_unmap(scmd);
- if (ioc->pci_error_recovery || ioc->remove_host)
+
+ if ((!mpt3sas_base_pci_device_is_available(ioc))
+ || ioc->remove_host)
scmd->result = DID_NO_CONNECT << 16;
else
scmd->result = DID_RESET << 16;
@@ -9726,6 +9755,9 @@ _scsih_ir_shutdown(struct MPT3SAS_ADAPTER *ioc)
if (list_empty(&ioc->raid_device_list))
return;
+ if (mpt3sas_base_pci_device_is_unplugged(ioc))
+ return;
+
mutex_lock(&ioc->scsih_cmds.mutex);
if (ioc->scsih_cmds.status != MPT3_CMD_NOT_USED) {
--
1.8.3.1
