Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit f1af6208 authored May 04, 2010 by Giridhar Malavali Committed by James Bottomley May 16, 2010

[SCSI] qla2xxx: Updates to ISP82xx support.



1) Allow transition to NEED RESET state only from READY state for ISP82xx.
2) Avoid infinite ISP aborts when chip reset fails.
3) Code cleanup to remove some of the unused debug code.

Signed-off-by: Lalit Chandivade <lalit.chandivade@qlogic.com>
Signed-off-by: Santosh Vernekar <santosh.vernekar@qlogic.com>
Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>

parent f4c496c1

drivers/scsi/qla2xxx/qla_def.h

+0 −2

Original line number	Diff line number	Diff line
		@@ -2764,8 +2764,6 @@ struct qla_hw_data {
		unsigned long nx_pcibase; /* Base I/O address */
		uint8_t nxdb_rd_ptr; / Doorbell read pointer */
		unsigned long nxdb_wr_ptr; /* Door bell write pointer */
		unsigned long first_page_group_start;
		unsigned long first_page_group_end;

		uint32_t crb_win;
		uint32_t curr_window;

drivers/scsi/qla2xxx/qla_init.c

+0 −33

Original line number	Diff line number	Diff line
		@@ -5296,39 +5296,6 @@ qla82xx_restart_isp(scsi_qla_host_t *vha)
		"(%d).\n", rval);
		}
		}
		} else { /* failed the ISP abort */
		vha->flags.online = 1;
		if (test_bit(ISP_ABORT_RETRY, &vha->dpc_flags)) {
		if (ha->isp_abort_cnt == 0) {
		qla_printk(KERN_WARNING, ha,
		"ISP error recovery failed - "
		"board disabled\n");
		/*
		* The next call disables the board
		* completely.
		*/
		ha->isp_ops->reset_adapter(vha);
		vha->flags.online = 0;
		clear_bit(ISP_ABORT_RETRY,
		&vha->dpc_flags);
		status = 0;
		} else { /* schedule another ISP abort */
		ha->isp_abort_cnt--;
		qla_printk(KERN_INFO, ha,
		"qla%ld: ISP abort - "
		"retry remaining %d\n",
		vha->host_no, ha->isp_abort_cnt);
		status = 1;
		}
		} else {
		ha->isp_abort_cnt = MAX_RETRIES_OF_ISP_ABORT;
		qla_printk(KERN_INFO, ha,
		"(%ld): ISP error recovery "
		"- retrying (%d) more times\n",
		vha->host_no, ha->isp_abort_cnt);
		set_bit(ISP_ABORT_RETRY, &vha->dpc_flags);
		status = 1;
		}
		}

		if (!status) {

drivers/scsi/qla2xxx/qla_nx.c

+103 −102

Original line number	Diff line number	Diff line
		@@ -30,16 +30,6 @@
		#define QLA82XX_PCI_CAMQM_2M_BASE (0x000ff800UL)
		#define CRB_INDIRECT_2M (0x1e0000UL)

		static inline void qla82xx_pci_base_offsetfset(struct qla_hw_data ha,
		unsigned long off)
		{
		if ((off < ha->first_page_group_end) &&
		(off >= ha->first_page_group_start))
		return (void *)(ha->nx_pcibase + off);

		return NULL;
		}

		#define MAX_CRB_XFORM 60
		static unsigned long crb_addr_xform[MAX_CRB_XFORM];
		int qla82xx_crb_table_initialized;
		@@ -335,6 +325,18 @@ unsigned qla82xx_crb_hub_agt[64] = {
		0,
		};

		/* Device states */
		char *qdev_state[] = {
		"Unknown",
		"Cold",
		"Initializing",
		"Ready",
		"Need Reset",
		"Need Quiescent",
		"Failed",
		"Quiescent",
		};

		/*
		* In: 'off' is offset from CRB space in 128M pci map
		* Out: 'off' is 2M pci map addr
		@@ -661,7 +663,7 @@ static int qla82xx_pci_mem_read_direct(struct qla_hw_data *ha,
		u64 off, void *data, int size)
		{
		unsigned long flags;
		void *addr;
		void *addr = NULL;
		int ret = 0;
		u64 start;
		uint8_t *mem_ptr = NULL;
		@@ -684,8 +686,6 @@ static int qla82xx_pci_mem_read_direct(struct qla_hw_data *ha,
		return -1;
		}

		addr = qla82xx_pci_base_offsetfset(ha, start);
		if (!addr) {
		write_unlock_irqrestore(&ha->hw_lock, flags);
		mem_base = pci_resource_start(ha->pdev, 0);
		mem_page = start & PAGE_MASK;
		@@ -703,7 +703,6 @@ static int qla82xx_pci_mem_read_direct(struct qla_hw_data *ha,
		addr = mem_ptr;
		addr += start & (PAGE_SIZE - 1);
		write_lock_irqsave(&ha->hw_lock, flags);
		}

		switch (size) {
		case 1:
		@@ -734,7 +733,7 @@ qla82xx_pci_mem_write_direct(struct qla_hw_data *ha,
		u64 off, void *data, int size)
		{
		unsigned long flags;
		void *addr;
		void *addr = NULL;
		int ret = 0;
		u64 start;
		uint8_t *mem_ptr = NULL;
		@@ -757,8 +756,6 @@ qla82xx_pci_mem_write_direct(struct qla_hw_data *ha,
		return -1;
		}

		addr = qla82xx_pci_base_offsetfset(ha, start);
		if (!addr) {
		write_unlock_irqrestore(&ha->hw_lock, flags);
		mem_base = pci_resource_start(ha->pdev, 0);
		mem_page = start & PAGE_MASK;
		@@ -775,7 +772,6 @@ qla82xx_pci_mem_write_direct(struct qla_hw_data *ha,
		addr = mem_ptr;
		addr += start & (PAGE_SIZE - 1);
		write_lock_irqsave(&ha->hw_lock, flags);
		}

		switch (size) {
		case 1:
		@@ -1866,6 +1862,14 @@ void qla82xx_config_rings(struct scsi_qla_host *vha)
		WRT_REG_DWORD((unsigned long __iomem *)&reg->rsp_q_out[0], 0);
		}

		void qla82xx_reset_adapter(struct scsi_qla_host *vha)
		{
		struct qla_hw_data *ha = vha->hw;
		vha->flags.online = 0;
		qla2x00_try_to_stop_firmware(vha);
		ha->isp_ops->disable_intrs(ha);
		}

		int qla82xx_fw_load_from_blob(struct qla_hw_data *ha)
		{
		u64 *ptr64;
		@@ -2093,15 +2097,7 @@ qla82xx_intr_handler(int irq, void *dev_id)

		if (RD_REG_DWORD(&reg->host_int)) {
		stat = RD_REG_DWORD(&reg->host_status);
		if (stat & HSRX_RISC_PAUSED) {
		if (pci_channel_offline(ha->pdev))
		break;

		qla_printk(KERN_INFO, ha, "RISC paused\n");
		set_bit(ISP_ABORT_NEEDED, &vha->dpc_flags);
		qla2xxx_wake_dpc(vha);
		break;
		} else if ((stat & HSRX_RISC_INT) == 0)
		if ((stat & HSRX_RISC_INT) == 0)
		break;

		switch (stat & 0xff) {
		@@ -2177,15 +2173,7 @@ qla82xx_msix_default(int irq, void *dev_id)
		do {
		if (RD_REG_DWORD(&reg->host_int)) {
		stat = RD_REG_DWORD(&reg->host_status);
		if (stat & HSRX_RISC_PAUSED) {
		if (pci_channel_offline(ha->pdev))
		break;

		qla_printk(KERN_INFO, ha, "RISC paused\n");
		set_bit(ISP_ABORT_NEEDED, &vha->dpc_flags);
		qla2xxx_wake_dpc(vha);
		break;
		} else if ((stat & HSRX_RISC_INT) == 0)
		if ((stat & HSRX_RISC_INT) == 0)
		break;

		switch (stat & 0xff) {
		@@ -3348,6 +3336,9 @@ qla82xx_need_reset_handler(scsi_qla_host_t *vha)
		}

		dev_state = qla82xx_rd_32(ha, QLA82XX_CRB_DEV_STATE);
		qla_printk(KERN_INFO, ha, "3:Device state is 0x%x = %s\n", dev_state,
		dev_state < MAX_STATES ? qdev_state[dev_state] : "Unknown");

		/* Force to DEV_COLD unless someone else is starting a reset */
		if (dev_state != QLA82XX_DEV_INITIALIZING) {
		qla_printk(KERN_INFO, ha, "HW State: COLD/RE-INIT\n");
		@@ -3398,7 +3389,6 @@ int
		qla82xx_device_state_handler(scsi_qla_host_t *vha)
		{
		uint32_t dev_state;
		uint32_t drv_active;
		int rval = QLA_SUCCESS;
		unsigned long dev_init_timeout;
		struct qla_hw_data *ha = vha->hw;
		@@ -3407,38 +3397,9 @@ qla82xx_device_state_handler(scsi_qla_host_t *vha)
		if (!vha->flags.init_done)
		qla82xx_set_drv_active(vha);

		/* Set cold state*/
		if (!PCI_FUNC(ha->pdev->devfn & 1)) {

		/* Check if other functions alive, else set dev state
		* to cold
		*/
		drv_active = qla82xx_rd_32(ha, QLA82XX_CRB_DRV_ACTIVE);
		drv_active &= ~(1 << (ha->portnum * 4));
		drv_active &= ~(1 << ((ha->portnum + 1) * 4));

		dev_state = qla82xx_rd_32(ha, QLA82XX_CRB_DEV_STATE);
		if (!drv_active) {

		switch (dev_state) {
		case QLA82XX_DEV_COLD:
		case QLA82XX_DEV_READY:
		case QLA82XX_DEV_INITIALIZING:
		case QLA82XX_DEV_NEED_RESET:
		case QLA82XX_DEV_NEED_QUIESCENT:
		case QLA82XX_DEV_QUIESCENT:
		case QLA82XX_DEV_FAILED:
		break;
		default:
		qla_printk(KERN_INFO, ha,
		"No other function exist,"
		" resetting dev state to COLD\n");
		qla82xx_wr_32(ha, QLA82XX_CRB_DEV_STATE,
		QLA82XX_DEV_COLD);
		break;
		}
		}
		}
		qla_printk(KERN_INFO, ha, "1:Device state is 0x%x = %s\n", dev_state,
		dev_state < MAX_STATES ? qdev_state[dev_state] : "Unknown");

		/* wait for 30 seconds for device to go ready */
		dev_init_timeout = jiffies + (ha->nx_dev_init_timeout * HZ);
		@@ -3453,6 +3414,11 @@ qla82xx_device_state_handler(scsi_qla_host_t *vha)
		break;
		}
		dev_state = qla82xx_rd_32(ha, QLA82XX_CRB_DEV_STATE);
		qla_printk(KERN_INFO, ha,
		"2:Device state is 0x%x = %s\n", dev_state,
		dev_state < MAX_STATES ?
		qdev_state[dev_state] : "Unknown");

		switch (dev_state) {
		case QLA82XX_DEV_READY:
		goto exit;
		@@ -3545,12 +3511,14 @@ qla82xx_abort_isp(scsi_qla_host_t *vha)

		qla82xx_idc_lock(ha);
		dev_state = qla82xx_rd_32(ha, QLA82XX_CRB_DEV_STATE);
		if (dev_state != QLA82XX_DEV_INITIALIZING) {
		if (dev_state == QLA82XX_DEV_READY) {
		qla_printk(KERN_INFO, ha, "HW State: NEED RESET\n");
		qla82xx_wr_32(ha, QLA82XX_CRB_DEV_STATE,
		QLA82XX_DEV_NEED_RESET);
		} else
		qla_printk(KERN_INFO, ha, "HW State: DEVICE INITIALIZING\n");
		qla_printk(KERN_INFO, ha, "HW State: %s\n",
		dev_state < MAX_STATES ?
		qdev_state[dev_state] : "Unknown");
		qla82xx_idc_unlock(ha);

		rval = qla82xx_device_state_handler(vha);
		@@ -3561,6 +3529,39 @@ qla82xx_abort_isp(scsi_qla_host_t *vha)

		if (rval == QLA_SUCCESS)
		qla82xx_restart_isp(vha);

		if (rval) {
		vha->flags.online = 1;
		if (test_bit(ISP_ABORT_RETRY, &vha->dpc_flags)) {
		if (ha->isp_abort_cnt == 0) {
		qla_printk(KERN_WARNING, ha,
		"ISP error recovery failed - "
		"board disabled\n");
		/*
		* The next call disables the board
		* completely.
		*/
		ha->isp_ops->reset_adapter(vha);
		vha->flags.online = 0;
		clear_bit(ISP_ABORT_RETRY,
		&vha->dpc_flags);
		rval = QLA_SUCCESS;
		} else { /* schedule another ISP abort */
		ha->isp_abort_cnt--;
		DEBUG(qla_printk(KERN_INFO, ha,
		"qla%ld: ISP abort - retry remaining %d\n",
		vha->host_no, ha->isp_abort_cnt));
		rval = QLA_FUNCTION_FAILED;
		}
		} else {
		ha->isp_abort_cnt = MAX_RETRIES_OF_ISP_ABORT;
		DEBUG(qla_printk(KERN_INFO, ha,
		"(%ld): ISP error recovery - retrying (%d) "
		"more times\n", vha->host_no, ha->isp_abort_cnt));
		set_bit(ISP_ABORT_RETRY, &vha->dpc_flags);
		rval = QLA_FUNCTION_FAILED;
		}
		}
		return rval;
		}

drivers/scsi/qla2xxx/qla_nx.h

+8 −7

Original line number	Diff line number	Diff line
		@@ -552,6 +552,7 @@
		#define QLA82XX_DEV_NEED_QUIESCENT 5
		#define QLA82XX_DEV_FAILED 6
		#define QLA82XX_DEV_QUIESCENT 7
		#define MAX_STATES 8 /* Increment if new state added */

		#define QLA82XX_IDC_VERSION 1
		#define QLA82XX_ROM_DEV_INIT_TIMEOUT 30