scsi: lpfc: Deprecate lpfc_soft_wwn parameter
authorJames Smart <james.smart@broadcom.com>
Mon, 19 Dec 2016 23:07:22 +0000 (15:07 -0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Thu, 5 Jan 2017 05:21:12 +0000 (00:21 -0500)
Deprecate lpfc_soft_wwn parameter.
No longer allow override of hw-assigned wwns

Signed-off-by: Dick Kennedy <dick.kennedy@broadcom.com>
Signed-off-by: James Smart <james.smart@broadcom.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/lpfc/lpfc.h
drivers/scsi/lpfc/lpfc_attr.c
drivers/scsi/lpfc/lpfc_init.c

index 8a20b4e862241c7333e0eb927347816919e51a8f..341b656e9d9d4ec7fb15e9fc52515f95940f8b79 100644 (file)
@@ -729,8 +729,6 @@ struct lpfc_hba {
        uint32_t cfg_sg_seg_cnt;
        uint32_t cfg_prot_sg_seg_cnt;
        uint32_t cfg_sg_dma_buf_size;
-       uint64_t cfg_soft_wwnn;
-       uint64_t cfg_soft_wwpn;
        uint32_t cfg_hba_queue_depth;
        uint32_t cfg_enable_hba_reset;
        uint32_t cfg_enable_hba_heartbeat;
@@ -835,8 +833,6 @@ struct lpfc_hba {
 #define VPD_PORT            0x8         /* valid vpd port data */
 #define VPD_MASK            0xf         /* mask for any vpd data */
 
-       uint8_t soft_wwn_enable;
-
        struct timer_list fcp_poll_timer;
        struct timer_list eratt_poll;
        uint32_t eratt_poll_interval;
index c84775562c65e9b486eee549594c800c03945fd7..98c40a734a3648cced75acfd89ebdc9b8ccd8d9f 100644 (file)
@@ -1987,7 +1987,6 @@ static DEVICE_ATTR(protocol, S_IRUGO, lpfc_sli4_protocol_show, NULL);
 static DEVICE_ATTR(lpfc_xlane_supported, S_IRUGO, lpfc_oas_supported_show,
                   NULL);
 
-static char *lpfc_soft_wwn_key = "C99G71SL8032A";
 #define WWN_SZ 8
 /**
  * lpfc_wwn_set - Convert string to the 8 byte WWN value.
@@ -2031,216 +2030,6 @@ lpfc_wwn_set(const char *buf, size_t cnt, char wwn[])
        }
        return 0;
 }
-/**
- * lpfc_soft_wwn_enable_store - Allows setting of the wwn if the key is valid
- * @dev: class device that is converted into a Scsi_host.
- * @attr: device attribute, not used.
- * @buf: containing the string lpfc_soft_wwn_key.
- * @count: must be size of lpfc_soft_wwn_key.
- *
- * Returns:
- * -EINVAL if the buffer does not contain lpfc_soft_wwn_key
- * length of buf indicates success
- **/
-static ssize_t
-lpfc_soft_wwn_enable_store(struct device *dev, struct device_attribute *attr,
-                          const char *buf, size_t count)
-{
-       struct Scsi_Host  *shost = class_to_shost(dev);
-       struct lpfc_vport *vport = (struct lpfc_vport *) shost->hostdata;
-       struct lpfc_hba   *phba = vport->phba;
-       unsigned int cnt = count;
-
-       /*
-        * We're doing a simple sanity check for soft_wwpn setting.
-        * We require that the user write a specific key to enable
-        * the soft_wwpn attribute to be settable. Once the attribute
-        * is written, the enable key resets. If further updates are
-        * desired, the key must be written again to re-enable the
-        * attribute.
-        *
-        * The "key" is not secret - it is a hardcoded string shown
-        * here. The intent is to protect against the random user or
-        * application that is just writing attributes.
-        */
-
-       /* count may include a LF at end of string */
-       if (buf[cnt-1] == '\n')
-               cnt--;
-
-       if ((cnt != strlen(lpfc_soft_wwn_key)) ||
-           (strncmp(buf, lpfc_soft_wwn_key, strlen(lpfc_soft_wwn_key)) != 0))
-               return -EINVAL;
-
-       phba->soft_wwn_enable = 1;
-       return count;
-}
-static DEVICE_ATTR(lpfc_soft_wwn_enable, S_IWUSR, NULL,
-                  lpfc_soft_wwn_enable_store);
-
-/**
- * lpfc_soft_wwpn_show - Return the cfg soft ww port name of the adapter
- * @dev: class device that is converted into a Scsi_host.
- * @attr: device attribute, not used.
- * @buf: on return contains the wwpn in hexadecimal.
- *
- * Returns: size of formatted string.
- **/
-static ssize_t
-lpfc_soft_wwpn_show(struct device *dev, struct device_attribute *attr,
-                   char *buf)
-{
-       struct Scsi_Host  *shost = class_to_shost(dev);
-       struct lpfc_vport *vport = (struct lpfc_vport *) shost->hostdata;
-       struct lpfc_hba   *phba = vport->phba;
-
-       return snprintf(buf, PAGE_SIZE, "0x%llx\n",
-                       (unsigned long long)phba->cfg_soft_wwpn);
-}
-
-/**
- * lpfc_soft_wwpn_store - Set the ww port name of the adapter
- * @dev class device that is converted into a Scsi_host.
- * @attr: device attribute, not used.
- * @buf: contains the wwpn in hexadecimal.
- * @count: number of wwpn bytes in buf
- *
- * Returns:
- * -EACCES hba reset not enabled, adapter over temp
- * -EINVAL soft wwn not enabled, count is invalid, invalid wwpn byte invalid
- * -EIO error taking adapter offline or online
- * value of count on success
- **/
-static ssize_t
-lpfc_soft_wwpn_store(struct device *dev, struct device_attribute *attr,
-                    const char *buf, size_t count)
-{
-       struct Scsi_Host  *shost = class_to_shost(dev);
-       struct lpfc_vport *vport = (struct lpfc_vport *) shost->hostdata;
-       struct lpfc_hba   *phba = vport->phba;
-       struct completion online_compl;
-       int stat1 = 0, stat2 = 0;
-       unsigned int cnt = count;
-       u8 wwpn[WWN_SZ];
-       int rc;
-
-       if (!phba->cfg_enable_hba_reset)
-               return -EACCES;
-       spin_lock_irq(&phba->hbalock);
-       if (phba->over_temp_state == HBA_OVER_TEMP) {
-               spin_unlock_irq(&phba->hbalock);
-               return -EACCES;
-       }
-       spin_unlock_irq(&phba->hbalock);
-       /* count may include a LF at end of string */
-       if (buf[cnt-1] == '\n')
-               cnt--;
-
-       if (!phba->soft_wwn_enable)
-               return -EINVAL;
-
-       /* lock setting wwpn, wwnn down */
-       phba->soft_wwn_enable = 0;
-
-       rc = lpfc_wwn_set(buf, cnt, wwpn);
-       if (!rc) {
-               /* not able to set wwpn, unlock it */
-               phba->soft_wwn_enable = 1;
-               return rc;
-       }
-
-       phba->cfg_soft_wwpn = wwn_to_u64(wwpn);
-       fc_host_port_name(shost) = phba->cfg_soft_wwpn;
-       if (phba->cfg_soft_wwnn)
-               fc_host_node_name(shost) = phba->cfg_soft_wwnn;
-
-       dev_printk(KERN_NOTICE, &phba->pcidev->dev,
-                  "lpfc%d: Reinitializing to use soft_wwpn\n", phba->brd_no);
-
-       stat1 = lpfc_do_offline(phba, LPFC_EVT_OFFLINE);
-       if (stat1)
-               lpfc_printf_log(phba, KERN_ERR, LOG_INIT,
-                               "0463 lpfc_soft_wwpn attribute set failed to "
-                               "reinit adapter - %d\n", stat1);
-       init_completion(&online_compl);
-       rc = lpfc_workq_post_event(phba, &stat2, &online_compl,
-                                  LPFC_EVT_ONLINE);
-       if (rc == 0)
-               return -ENOMEM;
-
-       wait_for_completion(&online_compl);
-       if (stat2)
-               lpfc_printf_log(phba, KERN_ERR, LOG_INIT,
-                               "0464 lpfc_soft_wwpn attribute set failed to "
-                               "reinit adapter - %d\n", stat2);
-       return (stat1 || stat2) ? -EIO : count;
-}
-static DEVICE_ATTR(lpfc_soft_wwpn, S_IRUGO | S_IWUSR,
-                  lpfc_soft_wwpn_show, lpfc_soft_wwpn_store);
-
-/**
- * lpfc_soft_wwnn_show - Return the cfg soft ww node name for the adapter
- * @dev: class device that is converted into a Scsi_host.
- * @attr: device attribute, not used.
- * @buf: on return contains the wwnn in hexadecimal.
- *
- * Returns: size of formatted string.
- **/
-static ssize_t
-lpfc_soft_wwnn_show(struct device *dev, struct device_attribute *attr,
-                   char *buf)
-{
-       struct Scsi_Host *shost = class_to_shost(dev);
-       struct lpfc_hba *phba = ((struct lpfc_vport *)shost->hostdata)->phba;
-       return snprintf(buf, PAGE_SIZE, "0x%llx\n",
-                       (unsigned long long)phba->cfg_soft_wwnn);
-}
-
-/**
- * lpfc_soft_wwnn_store - sets the ww node name of the adapter
- * @cdev: class device that is converted into a Scsi_host.
- * @buf: contains the ww node name in hexadecimal.
- * @count: number of wwnn bytes in buf.
- *
- * Returns:
- * -EINVAL soft wwn not enabled, count is invalid, invalid wwnn byte invalid
- * value of count on success
- **/
-static ssize_t
-lpfc_soft_wwnn_store(struct device *dev, struct device_attribute *attr,
-                    const char *buf, size_t count)
-{
-       struct Scsi_Host *shost = class_to_shost(dev);
-       struct lpfc_hba *phba = ((struct lpfc_vport *)shost->hostdata)->phba;
-       unsigned int cnt = count;
-       u8 wwnn[WWN_SZ];
-       int rc;
-
-       /* count may include a LF at end of string */
-       if (buf[cnt-1] == '\n')
-               cnt--;
-
-       if (!phba->soft_wwn_enable)
-               return -EINVAL;
-
-       rc = lpfc_wwn_set(buf, cnt, wwnn);
-       if (!rc) {
-               /* Allow wwnn to be set many times, as long as the enable
-                * is set. However, once the wwpn is set, everything locks.
-                */
-               return rc;
-       }
-
-       phba->cfg_soft_wwnn = wwn_to_u64(wwnn);
-
-       dev_printk(KERN_NOTICE, &phba->pcidev->dev,
-                  "lpfc%d: soft_wwnn set. Value will take effect upon "
-                  "setting of the soft_wwpn\n", phba->brd_no);
-
-       return count;
-}
-static DEVICE_ATTR(lpfc_soft_wwnn, S_IRUGO | S_IWUSR,
-                  lpfc_soft_wwnn_show, lpfc_soft_wwnn_store);
 
 /**
  * lpfc_oas_tgt_show - Return wwpn of target whose luns maybe enabled for
@@ -4750,9 +4539,6 @@ struct device_attribute *lpfc_hba_attrs[] = {
        &dev_attr_lpfc_fcp_cpu_map,
        &dev_attr_lpfc_fcp_io_channel,
        &dev_attr_lpfc_enable_bg,
-       &dev_attr_lpfc_soft_wwnn,
-       &dev_attr_lpfc_soft_wwpn,
-       &dev_attr_lpfc_soft_wwn_enable,
        &dev_attr_lpfc_enable_hba_reset,
        &dev_attr_lpfc_enable_hba_heartbeat,
        &dev_attr_lpfc_EnableXLane,
@@ -5765,8 +5551,6 @@ lpfc_get_cfgparam(struct lpfc_hba *phba)
        else
                phba->cfg_poll = lpfc_poll;
 
-       phba->cfg_soft_wwnn = 0L;
-       phba->cfg_soft_wwpn = 0L;
        lpfc_sg_seg_cnt_init(phba, lpfc_sg_seg_cnt);
        lpfc_prot_sg_seg_cnt_init(phba, lpfc_prot_sg_seg_cnt);
        lpfc_hba_queue_depth_init(phba, lpfc_hba_queue_depth);
index 4776fd85514f5886a195d409c25fca6f389468da..43d2b06bdc82baa0949d928421dc13b32b390af4 100644 (file)
@@ -319,36 +319,26 @@ lpfc_dump_wakeup_param_cmpl(struct lpfc_hba *phba, LPFC_MBOXQ_t *pmboxq)
 
 /**
  * lpfc_update_vport_wwn - Updates the fc_nodename, fc_portname,
- *     cfg_soft_wwnn, cfg_soft_wwpn
  * @vport: pointer to lpfc vport data structure.
  *
- *
  * Return codes
  *   None.
  **/
 void
 lpfc_update_vport_wwn(struct lpfc_vport *vport)
 {
-       /* If the soft name exists then update it using the service params */
-       if (vport->phba->cfg_soft_wwnn)
-               u64_to_wwn(vport->phba->cfg_soft_wwnn,
-                          vport->fc_sparam.nodeName.u.wwn);
-       if (vport->phba->cfg_soft_wwpn)
-               u64_to_wwn(vport->phba->cfg_soft_wwpn,
-                          vport->fc_sparam.portName.u.wwn);
-
        /*
-        * If the name is empty or there exists a soft name
+        * If the name is empty
         * then copy the service params name, otherwise use the fc name
         */
-       if (vport->fc_nodename.u.wwn[0] == 0 || vport->phba->cfg_soft_wwnn)
+       if (vport->fc_nodename.u.wwn[0] == 0)
                memcpy(&vport->fc_nodename, &vport->fc_sparam.nodeName,
                        sizeof(struct lpfc_name));
        else
                memcpy(&vport->fc_sparam.nodeName, &vport->fc_nodename,
                        sizeof(struct lpfc_name));
 
-       if (vport->fc_portname.u.wwn[0] == 0 || vport->phba->cfg_soft_wwpn)
+       if (vport->fc_portname.u.wwn[0] == 0)
                memcpy(&vport->fc_portname, &vport->fc_sparam.portName,
                        sizeof(struct lpfc_name));
        else