[SCSI] bfa: Driver and BSG enhancements.

- Added a new module parameter max_xfer_size to
  set the max_sectors in the scsi_host template.
- Added logic to handle request_irq() failure so
  that msix vector resource is de-allocated immediately
  when failure happens.
- BSG enhancements to collect vHBA releated info and port log.
- Removed the workaround of incrementing the module refcnt on bsg request.

Signed-off-by: Krishna Gudipati <kgudipat@brocade.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com>

6 files changed, 100 insertions, 34 deletions

diff --git a/drivers/scsi/bfa/bfa_defs_svc.h b/drivers/scsi/bfa/bfa_defs_svc.h
index 4b6529c7a531..0b97525803fb 100644
--- a/drivers/scsi/bfa/bfa_defs_svc.h
+++ b/drivers/scsi/bfa/bfa_defs_svc.h
@@ -1043,6 +1043,19 @@ struct bfa_itnim_ioprofile_s {
 };
 
 /*
+ *      vHBA port attribute values.
+ */
+struct bfa_vhba_attr_s {
+        wwn_t   nwwn;       /* node wwn */
+        wwn_t   pwwn;       /* port wwn */
+        u32     pid;        /* port ID */
+        bfa_boolean_t       io_profile; /* get it from fcpim mod */
+        bfa_boolean_t       plog_enabled;   /* portlog is enabled */
+        u16     path_tov;
+        u8      rsvd[2];
+};
+
+/*
  * FC physical port statistics.
  */
 struct bfa_port_fc_stats_s {
diff --git a/drivers/scsi/bfa/bfad.c b/drivers/scsi/bfa/bfad.c
index 0ed260852159..beb30a748ea5 100644
--- a/drivers/scsi/bfa/bfad.c
+++ b/drivers/scsi/bfa/bfad.c
@@ -56,6 +56,7 @@ int		fdmi_enable = BFA_TRUE;
 int             pcie_max_read_reqsz;
 int             bfa_debugfs_enable = 1;
 int             msix_disable_cb = 0, msix_disable_ct = 0;
+int             max_xfer_size = BFAD_MAX_SECTORS >> 1;
 
 /* Firmware releated */
 u32     bfi_image_cb_size, bfi_image_ct_size, bfi_image_ct2_size;
@@ -144,6 +145,9 @@ MODULE_PARM_DESC(pcie_max_read_reqsz, "PCIe max read request size, default=0 "
 module_param(bfa_debugfs_enable, int, S_IRUGO | S_IWUSR);
 MODULE_PARM_DESC(bfa_debugfs_enable, "Enables debugfs feature, default=1,"
                 " Range[false:0|true:1]");
+module_param(max_xfer_size, int, S_IRUGO | S_IWUSR);
+MODULE_PARM_DESC(max_xfer_size, "default=32MB,"
+                " Range[64k|128k|256k|512k|1024k|2048k]");
 
 static void
 bfad_sm_uninit(struct bfad_s *bfad, enum bfad_sm_event event);
@@ -1015,6 +1019,12 @@ bfad_start_ops(struct bfad_s *bfad) {
         struct bfad_vport_s *vport, *vport_new;
         struct bfa_fcs_driver_info_s driver_info;
 
+        /* Limit min/max. xfer size to [64k-32MB] */
+        if (max_xfer_size < BFAD_MIN_SECTORS >> 1)
+                max_xfer_size = BFAD_MIN_SECTORS >> 1;
+        if (max_xfer_size > BFAD_MAX_SECTORS >> 1)
+                max_xfer_size = BFAD_MAX_SECTORS >> 1;
+
         /* Fill the driver_info info to fcs*/
         memset(&driver_info, 0, sizeof(driver_info));
         strncpy(driver_info.version, BFAD_DRIVER_VERSION,
@@ -1231,6 +1241,9 @@ bfad_install_msix_handler(struct bfad_s *bfad)
                                 free_irq(bfad->msix_tab[j].msix.vector,
                                                 &bfad->msix_tab[j]);
 
+                        bfad->bfad_flags &= ~BFAD_MSIX_ON;
+                        pci_disable_msix(bfad->pcidev);
+
                         return 1;
                 }
         }
@@ -1306,6 +1319,7 @@ line_based:
                 /* Enable interrupt handler failed */
                 return 1;
         }
+        bfad->bfad_flags |= BFAD_INTX_ON;
 
         return error;
 }
@@ -1322,7 +1336,7 @@ bfad_remove_intr(struct bfad_s *bfad)
 
                 pci_disable_msix(bfad->pcidev);
                 bfad->bfad_flags &= ~BFAD_MSIX_ON;
-        } else {
+        } else if (bfad->bfad_flags & BFAD_INTX_ON) {
                 free_irq(bfad->pcidev->irq, bfad);
         }
 }
diff --git a/drivers/scsi/bfa/bfad_bsg.c b/drivers/scsi/bfa/bfad_bsg.c
index 552bb250a210..89f863ed2334 100644
--- a/drivers/scsi/bfa/bfad_bsg.c
+++ b/drivers/scsi/bfa/bfad_bsg.c
@@ -22,30 +22,6 @@
 
 BFA_TRC_FILE(LDRV, BSG);
 
-/* bfad_im_bsg_get_kobject - increment the bfa refcnt */
-static void
-bfad_im_bsg_get_kobject(struct fc_bsg_job *job)
-{
-        struct Scsi_Host *shost = job->shost;
-        unsigned long flags;
-
-        spin_lock_irqsave(shost->host_lock, flags);
-        __module_get(shost->dma_dev->driver->owner);
-        spin_unlock_irqrestore(shost->host_lock, flags);
-}
-
-/* bfad_im_bsg_put_kobject - decrement the bfa refcnt */
-static void
-bfad_im_bsg_put_kobject(struct fc_bsg_job *job)
-{
-        struct Scsi_Host *shost = job->shost;
-        unsigned long flags;
-
-        spin_lock_irqsave(shost->host_lock, flags);
-        module_put(shost->dma_dev->driver->owner);
-        spin_unlock_irqrestore(shost->host_lock, flags);
-}
-
 int
 bfad_iocmd_ioc_enable(struct bfad_s *bfad, void *cmd)
 {
@@ -1468,6 +1444,25 @@ out:
 }
 
 int
+bfad_iocmd_vhba_query(struct bfad_s *bfad, void *cmd)
+{
+        struct bfa_bsg_vhba_attr_s *iocmd =
+                        (struct bfa_bsg_vhba_attr_s *)cmd;
+        struct bfa_vhba_attr_s *attr = &iocmd->attr;
+        unsigned long flags;
+
+        spin_lock_irqsave(&bfad->bfad_lock, flags);
+        attr->pwwn =  bfad->bfa.ioc.attr->pwwn;
+        attr->nwwn =  bfad->bfa.ioc.attr->nwwn;
+        attr->plog_enabled = (bfa_boolean_t)bfad->bfa.plog->plog_enabled;
+        attr->io_profile = bfa_fcpim_get_io_profile(&bfad->bfa);
+        attr->path_tov  = bfa_fcpim_path_tov_get(&bfad->bfa);
+        iocmd->status = BFA_STATUS_OK;
+        spin_unlock_irqrestore(&bfad->bfad_lock, flags);
+        return 0;
+}
+
+int
 bfad_iocmd_phy_update(struct bfad_s *bfad, void *cmd, unsigned int payload_len)
 {
         struct bfa_bsg_phy_s *iocmd = (struct bfa_bsg_phy_s *)cmd;
@@ -1497,6 +1492,25 @@ out:
         return 0;
 }
 
+int
+bfad_iocmd_porglog_get(struct bfad_s *bfad, void *cmd)
+{
+        struct bfa_bsg_debug_s *iocmd = (struct bfa_bsg_debug_s *)cmd;
+        void *iocmd_bufptr;
+
+        if (iocmd->bufsz < sizeof(struct bfa_plog_s)) {
+                bfa_trc(bfad, sizeof(struct bfa_plog_s));
+                iocmd->status = BFA_STATUS_EINVAL;
+                goto out;
+        }
+
+        iocmd->status = BFA_STATUS_OK;
+        iocmd_bufptr = (char *)iocmd + sizeof(struct bfa_bsg_debug_s);
+        memcpy(iocmd_bufptr, (u8 *) &bfad->plog_buf, sizeof(struct bfa_plog_s));
+out:
+        return 0;
+}
+
 static int
 bfad_iocmd_handler(struct bfad_s *bfad, unsigned int cmd, void *iocmd,
                 unsigned int payload_len)
@@ -1682,6 +1696,12 @@ bfad_iocmd_handler(struct bfad_s *bfad, unsigned int cmd, void *iocmd,
         case IOCMD_PHY_READ_FW:
                 rc = bfad_iocmd_phy_read(bfad, iocmd, payload_len);
                 break;
+        case IOCMD_VHBA_QUERY:
+                rc = bfad_iocmd_vhba_query(bfad, iocmd);
+                break;
+        case IOCMD_DEBUG_PORTLOG:
+                rc = bfad_iocmd_porglog_get(bfad, iocmd);
+                break;