1 files changed, 214 insertions, 119 deletions

diff --git a/drivers/s390/scsi/zfcp_fc.c b/drivers/s390/scsi/zfcp_fc.c
index 30cf91a787a3..297e6b71ce9c 100644
--- a/drivers/s390/scsi/zfcp_fc.c
+++ b/drivers/s390/scsi/zfcp_fc.c
@@ -11,11 +11,14 @@
 
 #include <linux/types.h>
 #include <linux/slab.h>
+#include <linux/utsname.h>
 #include <scsi/fc/fc_els.h>
 #include <scsi/libfc.h>
 #include "zfcp_ext.h"
 #include "zfcp_fc.h"
 
+struct kmem_cache *zfcp_fc_req_cache;
+
 static u32 zfcp_fc_rscn_range_mask[] = {
         [ELS_ADDR_FMT_PORT]             = 0xFFFFFF,
         [ELS_ADDR_FMT_AREA]             = 0xFFFF00,
@@ -260,24 +263,18 @@ void zfcp_fc_incoming_els(struct zfcp_fsf_req *fsf_req)
                 zfcp_fc_incoming_rscn(fsf_req);
 }
 
-static void zfcp_fc_ns_gid_pn_eval(void *data)
+static void zfcp_fc_ns_gid_pn_eval(struct zfcp_fc_req *fc_req)
 {
-        struct zfcp_fc_gid_pn *gid_pn = data;
-        struct zfcp_fsf_ct_els *ct = &gid_pn->ct;
-        struct zfcp_fc_gid_pn_req *gid_pn_req = sg_virt(ct->req);
-        struct zfcp_fc_gid_pn_resp *gid_pn_resp = sg_virt(ct->resp);
-        struct zfcp_port *port = gid_pn->port;
+        struct zfcp_fsf_ct_els *ct_els = &fc_req->ct_els;
+        struct zfcp_fc_gid_pn_rsp *gid_pn_rsp = &fc_req->u.gid_pn.rsp;
 
-        if (ct->status)
+        if (ct_els->status)
                 return;
-        if (gid_pn_resp->ct_hdr.ct_cmd != FC_FS_ACC)
+        if (gid_pn_rsp->ct_hdr.ct_cmd != FC_FS_ACC)
                 return;
 
-        /* paranoia */
-        if (gid_pn_req->gid_pn.fn_wwpn != port->wwpn)
-                return;
         /* looks like a valid d_id */
-        port->d_id = ntoh24(gid_pn_resp->gid_pn.fp_fid);
+        ct_els->port->d_id = ntoh24(gid_pn_rsp->gid_pn.fp_fid);
 }
 
 static void zfcp_fc_complete(void *data)
@@ -285,69 +282,73 @@ static void zfcp_fc_complete(void *data)
         complete(data);
 }
 
+static void zfcp_fc_ct_ns_init(struct fc_ct_hdr *ct_hdr, u16 cmd, u16 mr_size)
+{
+        ct_hdr->ct_rev = FC_CT_REV;
+        ct_hdr->ct_fs_type = FC_FST_DIR;
+        ct_hdr->ct_fs_subtype = FC_NS_SUBTYPE;
+        ct_hdr->ct_cmd = cmd;
+        ct_hdr->ct_mr_size = mr_size / 4;
+}
+
 static int zfcp_fc_ns_gid_pn_request(struct zfcp_port *port,
-                                     struct zfcp_fc_gid_pn *gid_pn)
+                                     struct zfcp_fc_req *fc_req)
 {
         struct zfcp_adapter *adapter = port->adapter;
         DECLARE_COMPLETION_ONSTACK(completion);
+        struct zfcp_fc_gid_pn_req *gid_pn_req = &fc_req->u.gid_pn.req;
+        struct zfcp_fc_gid_pn_rsp *gid_pn_rsp = &fc_req->u.gid_pn.rsp;
         int ret;
 
         /* setup parameters for send generic command */
-        gid_pn->port = port;
-        gid_pn->ct.handler = zfcp_fc_complete;
-        gid_pn->ct.handler_data = &completion;
-        gid_pn->ct.req = &gid_pn->sg_req;
-        gid_pn->ct.resp = &gid_pn->sg_resp;
-        sg_init_one(&gid_pn->sg_req, &gid_pn->gid_pn_req,
-                    sizeof(struct zfcp_fc_gid_pn_req));
-        sg_init_one(&gid_pn->sg_resp, &gid_pn->gid_pn_resp,
-                    sizeof(struct zfcp_fc_gid_pn_resp));
-
-        /* setup nameserver request */
-        gid_pn->gid_pn_req.ct_hdr.ct_rev = FC_CT_REV;
-        gid_pn->gid_pn_req.ct_hdr.ct_fs_type = FC_FST_DIR;
-        gid_pn->gid_pn_req.ct_hdr.ct_fs_subtype = FC_NS_SUBTYPE;
-        gid_pn->gid_pn_req.ct_hdr.ct_options = 0;
-        gid_pn->gid_pn_req.ct_hdr.ct_cmd = FC_NS_GID_PN;
-        gid_pn->gid_pn_req.ct_hdr.ct_mr_size = ZFCP_FC_CT_SIZE_PAGE / 4;
-        gid_pn->gid_pn_req.gid_pn.fn_wwpn = port->wwpn;
-
-        ret = zfcp_fsf_send_ct(&adapter->gs->ds, &gid_pn->ct,
+        fc_req->ct_els.port = port;
+        fc_req->ct_els.handler = zfcp_fc_complete;
+        fc_req->ct_els.handler_data = &completion;
+        fc_req->ct_els.req = &fc_req->sg_req;
+        fc_req->ct_els.resp = &fc_req->sg_rsp;
+        sg_init_one(&fc_req->sg_req, gid_pn_req, sizeof(*gid_pn_req));
+        sg_init_one(&fc_req->sg_rsp, gid_pn_rsp, sizeof(*gid_pn_rsp));
+
+        zfcp_fc_ct_ns_init(&gid_pn_req->ct_hdr,
+                           FC_NS_GID_PN, ZFCP_FC_CT_SIZE_PAGE);
+        gid_pn_req->gid_pn.fn_wwpn = port->wwpn;
+
+        ret = zfcp_fsf_send_ct(&adapter->gs->ds, &fc_req->ct_els,
                                adapter->pool.gid_pn_req,
                                ZFCP_FC_CTELS_TMO);
         if (!ret) {
                 wait_for_completion(&completion);
-                zfcp_fc_ns_gid_pn_eval(gid_pn);
+                zfcp_fc_ns_gid_pn_eval(fc_req);
         }
         return ret;
 }
 
 /**
- * zfcp_fc_ns_gid_pn_request - initiate GID_PN nameserver request
+ * zfcp_fc_ns_gid_pn - initiate GID_PN nameserver request
  * @port: port where GID_PN request is needed
  * return: -ENOMEM on error, 0 otherwise
  */
 static int zfcp_fc_ns_gid_pn(struct zfcp_port *port)
 {
         int ret;
-        struct zfcp_fc_gid_pn *gid_pn;
+        struct zfcp_fc_req *fc_req;
         struct zfcp_adapter *adapter = port->adapter;
 
-        gid_pn = mempool_alloc(adapter->pool.gid_pn, GFP_ATOMIC);
-        if (!gid_pn)
+        fc_req = mempool_alloc(adapter->pool.gid_pn, GFP_ATOMIC);
+        if (!fc_req)
                 return -ENOMEM;
 
-        memset(gid_pn, 0, sizeof(*gid_pn));
+        memset(fc_req, 0, sizeof(*fc_req));
 
         ret = zfcp_fc_wka_port_get(&adapter->gs->ds);
         if (ret)
                 goto out;
 
-        ret = zfcp_fc_ns_gid_pn_request(port, gid_pn);
+        ret = zfcp_fc_ns_gid_pn_request(port, fc_req);
 
         zfcp_fc_wka_port_put(&adapter->gs->ds);
 out:
-        mempool_free(gid_pn, adapter->pool.gid_pn);
+        mempool_free(fc_req, adapter->pool.gid_pn);
         return ret;
 }
 
@@ -419,11 +420,11 @@ void zfcp_fc_plogi_evaluate(struct zfcp_port *port, struct fc_els_flogi *plogi)
 
 static void zfcp_fc_adisc_handler(void *data)
 {
-        struct zfcp_fc_els_adisc *adisc = data;
-        struct zfcp_port *port = adisc->els.port;
-        struct fc_els_adisc *adisc_resp = &adisc->adisc_resp;
+        struct zfcp_fc_req *fc_req = data;
+        struct zfcp_port *port = fc_req->ct_els.port;
+        struct fc_els_adisc *adisc_resp = &fc_req->u.adisc.rsp;
 
-        if (adisc->els.status) {
+        if (fc_req->ct_els.status) {
                 /* request rejected or timed out */
                 zfcp_erp_port_forced_reopen(port, ZFCP_STATUS_COMMON_ERP_FAILED,
                                             "fcadh_1");
@@ -445,42 +446,42 @@ static void zfcp_fc_adisc_handler(void *data)
  out:
         atomic_clear_mask(ZFCP_STATUS_PORT_LINK_TEST, &port->status);
         put_device(&port->dev);
-        kmem_cache_free(zfcp_data.adisc_cache, adisc);
+        kmem_cache_free(zfcp_fc_req_cache, fc_req);
 }
 
 static int zfcp_fc_adisc(struct zfcp_port *port)
 {
-        struct zfcp_fc_els_adisc *adisc;
+        struct zfcp_fc_req *fc_req;
         struct zfcp_adapter *adapter = port->adapter;
+        struct Scsi_Host *shost = adapter->scsi_host;
         int ret;
 
-        adisc = kmem_cache_zalloc(zfcp_data.adisc_cache, GFP_ATOMIC);
-        if (!adisc)
+        fc_req = kmem_cache_zalloc(zfcp_fc_req_cache, GFP_ATOMIC);
+        if (!fc_req)
                 return -ENOMEM;
 
-        adisc->els.port = port;
-        adisc->els.req = &adisc->req;
-        adisc->els.resp = &adisc->resp;
-        sg_init_one(adisc->els.req, &adisc->adisc_req,
+        fc_req->ct_els.port = port;
+        fc_req->ct_els.req = &fc_req->sg_req;
+        fc_req->ct_els.resp = &fc_req->sg_rsp;
+        sg_init_one(&fc_req->sg_req, &fc_req->u.adisc.req,
                     sizeof(struct fc_els_adisc));
-        sg_init_one(adisc->els.resp, &adisc->adisc_resp,
+        sg_init_one(&fc_req->sg_rsp, &fc_req->u.adisc.rsp,
                     sizeof(struct fc_els_adisc));
 
-        adisc->els.handler = zfcp_fc_adisc_handler;
-        adisc->els.handler_data = adisc;
+        fc_req->ct_els.handler = zfcp_fc_adisc_handler;
+        fc_req->ct_els.handler_data = fc_req;
 
         /* acc. to FC-FS, hard_nport_id in ADISC should not be set for ports
            without FC-AL-2 capability, so we don't set it */
-        adisc->adisc_req.adisc_wwpn = fc_host_port_name(adapter->scsi_host);
-        adisc->adisc_req.adisc_wwnn = fc_host_node_name(adapter->scsi_host);
-        adisc->adisc_req.adisc_cmd = ELS_ADISC;
-        hton24(adisc->adisc_req.adisc_port_id,
-               fc_host_port_id(adapter->scsi_host));
+        fc_req->u.adisc.req.adisc_wwpn = fc_host_port_name(shost);
+        fc_req->u.adisc.req.adisc_wwnn = fc_host_node_name(shost);
+        fc_req->u.adisc.req.adisc_cmd = ELS_ADISC;
+        hton24(fc_req->u.adisc.req.adisc_port_id, fc_host_port_id(shost));
 
-        ret = zfcp_fsf_send_els(adapter, port->d_id, &adisc->els,
+        ret = zfcp_fsf_send_els(adapter, port->d_id, &fc_req->ct_els,
                                 ZFCP_FC_CTELS_TMO);
         if (ret)
-                kmem_cache_free(zfcp_data.adisc_cache, adisc);
+                kmem_cache_free(zfcp_fc_req_cache, fc_req);
 
         return ret;
 }
@@ -528,68 +529,42 @@ void zfcp_fc_test_link(struct zfcp_port *port)
                 put_device(&port->dev);
 }
 
-static void zfcp_free_sg_env(struct zfcp_fc_gpn_ft *gpn_ft, int buf_num)
+static struct zfcp_fc_req *zfcp_alloc_sg_env(int buf_num)
 {
-        struct scatterlist *sg = &gpn_ft->sg_req;
-
-        kmem_cache_free(zfcp_data.gpn_ft_cache, sg_virt(sg));
-        zfcp_sg_free_table(gpn_ft->sg_resp, buf_num);
-
-        kfree(gpn_ft);
-}
+        struct zfcp_fc_req *fc_req;
 
-static struct zfcp_fc_gpn_ft *zfcp_alloc_sg_env(int buf_num)
-{
-        struct zfcp_fc_gpn_ft *gpn_ft;
-        struct zfcp_fc_gpn_ft_req *req;
-
-        gpn_ft = kzalloc(sizeof(*gpn_ft), GFP_KERNEL);
-        if (!gpn_ft)
+        fc_req = kmem_cache_zalloc(zfcp_fc_req_cache, GFP_KERNEL);
+        if (!fc_req)
                 return NULL;
 
-        req = kmem_cache_zalloc(zfcp_data.gpn_ft_cache, GFP_KERNEL);
-        if (!req) {
-                kfree(gpn_ft);
-                gpn_ft = NULL;
-                goto out;
+        if (zfcp_sg_setup_table(&fc_req->sg_rsp, buf_num)) {
+                kmem_cache_free(zfcp_fc_req_cache, fc_req);
+                return NULL;
         }
-        sg_init_one(&gpn_ft->sg_req, req, sizeof(*req));
 
-        if (zfcp_sg_setup_table(gpn_ft->sg_resp, buf_num)) {
-                zfcp_free_sg_env(gpn_ft, buf_num);
-                gpn_ft = NULL;
-        }
-out:
-        return gpn_ft;
-}
+        sg_init_one(&fc_req->sg_req, &fc_req->u.gpn_ft.req,
+                    sizeof(struct zfcp_fc_gpn_ft_req));
 
+        return fc_req;
+}
 
-static int zfcp_fc_send_gpn_ft(struct zfcp_fc_gpn_ft *gpn_ft,
+static int zfcp_fc_send_gpn_ft(struct zfcp_fc_req *fc_req,
                                struct zfcp_adapter *adapter, int max_bytes)
 {
-        struct zfcp_fsf_ct_els *ct = &gpn_ft->ct;
-        struct zfcp_fc_gpn_ft_req *req = sg_virt(&gpn_ft->sg_req);
+        struct zfcp_fsf_ct_els *ct_els = &fc_req->ct_els;
+        struct zfcp_fc_gpn_ft_req *req = &fc_req->u.gpn_ft.req;
         DECLARE_COMPLETION_ONSTACK(completion);
         int ret;
 
-        /* prepare CT IU for GPN_FT */
-        req->ct_hdr.ct_rev = FC_CT_REV;
-        req->ct_hdr.ct_fs_type = FC_FST_DIR;
-        req->ct_hdr.ct_fs_subtype = FC_NS_SUBTYPE;
-        req->ct_hdr.ct_options = 0;
-        req->ct_hdr.ct_cmd = FC_NS_GPN_FT;
-        req->ct_hdr.ct_mr_size = max_bytes / 4;
-        req->gpn_ft.fn_domain_id_scope = 0;
-        req->gpn_ft.fn_area_id_scope = 0;
+        zfcp_fc_ct_ns_init(&req->ct_hdr, FC_NS_GPN_FT, max_bytes);
         req->gpn_ft.fn_fc4_type = FC_TYPE_FCP;
 
-        /* prepare zfcp_send_ct */
-        ct->handler = zfcp_fc_complete;
-        ct->handler_data = &completion;
-        ct->req = &gpn_ft->sg_req;
-        ct->resp = gpn_ft->sg_resp;
+        ct_els->handler = zfcp_fc_complete;
+        ct_els->handler_data = &completion;
+        ct_els->req = &fc_req->sg_req;
+        ct_els->resp = &fc_req->sg_rsp;
 
-        ret = zfcp_fsf_send_ct(&adapter->gs->ds, ct, NULL,
+        ret = zfcp_fsf_send_ct(&adapter->gs->ds, ct_els, NULL,
                                ZFCP_FC_CTELS_TMO);
         if (!ret)
                 wait_for_completion(&completion);
@@ -610,11 +585,11 @@ static void zfcp_fc_validate_port(struct zfcp_port *port, struct list_head *lh)
         list_move_tail(&port->list, lh);
 }
 
-static int zfcp_fc_eval_gpn_ft(struct zfcp_fc_gpn_ft *gpn_ft,
+static int zfcp_fc_eval_gpn_ft(struct zfcp_fc_req *fc_req,
                                struct zfcp_adapter *adapter, int max_entries)
 {
-        struct zfcp_fsf_ct_els *ct = &gpn_ft->ct;
-        struct scatterlist *sg = gpn_ft->sg_resp;
+        struct zfcp_fsf_ct_els *ct_els = &fc_req->ct_els;
+        struct scatterlist *sg = &fc_req->sg_rsp;
         struct fc_ct_hdr *hdr = sg_virt(sg);
         struct fc_gpn_ft_resp *acc = sg_virt(sg);
         struct zfcp_port *port, *tmp;
@@ -623,7 +598,7 @@ static int zfcp_fc_eval_gpn_ft(struct zfcp_fc_gpn_ft *gpn_ft,
         u32 d_id;
         int ret = 0, x, last = 0;
 
-        if (ct->status)
+        if (ct_els->status)
                 return -EIO;
 
         if (hdr->ct_cmd != FC_FS_ACC) {
@@ -687,7 +662,7 @@ void zfcp_fc_scan_ports(struct work_struct *work)
         struct zfcp_adapter *adapter = container_of(work, struct zfcp_adapter,
                                                     scan_work);
         int ret, i;
-        struct zfcp_fc_gpn_ft *gpn_ft;
+        struct zfcp_fc_req *fc_req;
         int chain, max_entries, buf_num, max_bytes;
 
         chain = adapter->adapter_features & FSF_FEATURE_ELS_CT_CHAINED_SBALS;
@@ -702,25 +677,145 @@ void zfcp_fc_scan_ports(struct work_struct *work)
         if (zfcp_fc_wka_port_get(&adapter->gs->ds))
                 return;
 
-        gpn_ft = zfcp_alloc_sg_env(buf_num);
-        if (!gpn_ft)
+        fc_req = zfcp_alloc_sg_env(buf_num);
+        if (!fc_req)
                 goto out;
 
         for (i = 0; i < 3; i++) {
-                ret = zfcp_fc_send_gpn_ft(gpn_ft, adapter, max_bytes);
+                ret = zfcp_fc_send_gpn_ft(fc_req, adapter, max_bytes);
                 if (!ret) {
-                        ret = zfcp_fc_eval_gpn_ft(gpn_ft, adapter, max_entries);
+                        ret = zfcp_fc_eval_gpn_ft(fc_req, adapter, max_entries);
                         if (ret == -EAGAIN)
                                 ssleep(1);
                         else
                                 break;
                 }
         }
-        zfcp_free_sg_env(gpn_ft, buf_num);
+        zfcp_sg_free_table(&fc_req->sg_rsp, buf_num);
+        kmem_cache_free(zfcp_fc_req_cache, fc_req);
 out:
         zfcp_fc_wka_port_put(&adapter->gs->ds);
 }
 
+static int zfcp_fc_gspn(struct zfcp_adapter *adapter,
+                        struct zfcp_fc_req *fc_req)
+{
+        DECLARE_COMPLETION_ONSTACK(completion);
+        char devno[] = "DEVNO:";
+        struct zfcp_fsf_ct_els *ct_els = &fc_req->ct_els;
+        struct zfcp_fc_gspn_req *gspn_req = &fc_req->u.gspn.req;
+        struct zfcp_fc_gspn_rsp *gspn_rsp = &fc_req->u.gspn.rsp;
+        int ret;
+
+        zfcp_fc_ct_ns_init(&gspn_req->ct_hdr, FC_NS_GSPN_ID,
+                           FC_SYMBOLIC_NAME_SIZE);
+        hton24(gspn_req->gspn.fp_fid, fc_host_port_id(adapter->scsi_host));
+
+        sg_init_one(&fc_req->sg_req, gspn_req, sizeof(*gspn_req));
+        sg_init_one(&fc_req->sg_rsp, gspn_rsp, sizeof(*gspn_rsp));
+
+        ct_els->handler = zfcp_fc_complete;
+        ct_els->handler_data = &completion;
+        ct_els->req = &fc_req->sg_req;
+        ct_els->resp = &fc_req->sg_rsp;
+
+        ret = zfcp_fsf_send_ct(&adapter->gs->ds, ct_els, NULL,
+                               ZFCP_FC_CTELS_TMO);
+        if (ret)
+                return ret;
+
+        wait_for_completion(&completion);
+        if (ct_els->status)
+                return ct_els->status;
+
+        if (fc_host_port_type(adapter->scsi_host) == FC_PORTTYPE_NPIV &&
+            !(strstr(gspn_rsp->gspn.fp_name, devno)))
+                snprintf(fc_host_symbolic_name(adapter->scsi_host),
+                         FC_SYMBOLIC_NAME_SIZE, "%s%s %s NAME: %s",
+                         gspn_rsp->gspn.fp_name, devno,
+                         dev_name(&adapter->ccw_device->dev),
+                         init_utsname()->nodename);
+        else
+                strlcpy(fc_host_symbolic_name(adapter->scsi_host),
+                        gspn_rsp->gspn.fp_name, FC_SYMBOLIC_NAME_SIZE);
+
+        return 0;
+}
+
+static void zfcp_fc_rspn(struct zfcp_adapter *adapter,
+                         struct zfcp_fc_req *fc_req)
+{
+        DECLARE_COMPLETION_ONSTACK(completion);
+        struct Scsi_Host *shost = adapter->scsi_host;
+        struct zfcp_fsf_ct_els *ct_els = &fc_req->ct_els;
+        struct zfcp_fc_rspn_req *rspn_req = &fc_req->u.rspn.req;
+        struct fc_ct_hdr *rspn_rsp = &fc_req->u.rspn.rsp;
+        int ret, len;
+
+        zfcp_fc_ct_ns_init(&rspn_req->ct_hdr, FC_NS_RSPN_ID,
+                           FC_SYMBOLIC_NAME_SIZE);
+        hton24(rspn_req->rspn.fr_fid.fp_fid, fc_host_port_id(shost));
+        len = strlcpy(rspn_req->rspn.fr_name, fc_host_symbolic_name(shost),
+                      FC_SYMBOLIC_NAME_SIZE);
+        rspn_req->rspn.fr_name_len = len;
+
+        sg_init_one(&fc_req->sg_req, rspn_req, sizeof(*rspn_req));
+        sg_init_one(&fc_req->sg_rsp, rspn_rsp, sizeof(*rspn_rsp));
+
+        ct_els->handler = zfcp_fc_complete;
+        ct_els->handler_data = &completion;
+        ct_els->req = &fc_req->sg_req;
+        ct_els->resp = &fc_req->sg_rsp;
+
+        ret = zfcp_fsf_send_ct(&adapter->gs->ds, ct_els, NULL,
+                               ZFCP_FC_CTELS_TMO);
+        if (!ret)
+                wait_for_completion(&completion);
+}
+
+/**
+ * zfcp_fc_sym_name_update - Retrieve and update the symbolic port name
+ * @work: ns_up_work of the adapter where to update the symbolic port name
+ *
+ * Retrieve the current symbolic port name that may have been set by
+ * the hardware using the GSPN request and update the fc_host
+ * symbolic_name sysfs attribute. When running in NPIV mode (and hence
+ * the port name is unique for this system), update the symbolic port
+ * name to add Linux specific information and update the FC nameserver
+ * using the RSPN request.
+ */
+void zfcp_fc_sym_name_update(struct work_struct *work)
+{
+        struct zfcp_adapter *adapter = container_of(work, struct zfcp_adapter,
+                                                    ns_up_work);
+        int ret;
+        struct zfcp_fc_req *fc_req;
+
+        if (fc_host_port_type(adapter->scsi_host) != FC_PORTTYPE_NPORT &&
+            fc_host_port_type(adapter->scsi_host) != FC_PORTTYPE_NPIV)
+                return;
+
+        fc_req = kmem_cache_zalloc(zfcp_fc_req_cache, GFP_KERNEL);
+        if (!fc_req)
+                return;
+
+        ret = zfcp_fc_wka_port_get(&adapter->gs->ds);
+        if (ret)
+                goto out_free;
+
+        ret = zfcp_fc_gspn(adapter, fc_req);
+        if (ret || fc_host_port_type(adapter->scsi_host) != FC_PORTTYPE_NPIV)
+                goto out_ds_put;
+
+        memset(fc_req, 0, sizeof(*fc_req));
+        zfcp_fc_rspn(adapter, fc_req);
+
+out_ds_put:
+        zfcp_fc_wka_port_put(&adapter->gs->ds);
+out_free:
+        kmem_cache_free(zfcp_fc_req_cache, fc_req);
+}
+
 static void zfcp_fc_ct_els_job_handler(void *data)
 {
         struct fc_bsg_job *job = data;