scsi: fc: provide fc_bsg_to_rport() helper

Provide fc_bsg_to_rport() helper that will become handy when we're
moving from struct fc_bsg_job to a plain struct bsg_job. Also move all
LLDDs to use the new helper.

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Acked-by: Tyrel Datwyler <tyreld@linux.vnet.ibm.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/s390/scsi/zfcp_fc.c b/drivers/s390/scsi/zfcp_fc.c
index 3937debf7cb55775c0a8cfa865042fda62d5e1e4..c751003aea61b5a61f239a21b150ec51c2b99d0c 100644 (file)
--- a/drivers/s390/scsi/zfcp_fc.c
+++ b/drivers/s390/scsi/zfcp_fc.c
@@ -901,12 +901,13 @@ static struct zfcp_fc_wka_port *zfcp_fc_job_wka_port(struct fc_bsg_job *job)
        u8 gs_type;
        struct zfcp_adapter *adapter;
        struct fc_bsg_request *bsg_request = job->request;
+       struct fc_rport *rport = fc_bsg_to_rport(job);
        struct Scsi_Host *shost;
 
        preamble_word1 = bsg_request->rqst_data.r_ct.preamble_word1;
        gs_type = (preamble_word1 & 0xff000000) >> 24;
 
-       shost = fc_bsg_to_shost(job);
+       shost = rport ? rport_to_shost(rport) : fc_bsg_to_shost(job);
        adapter = (struct zfcp_adapter *) shost->hostdata[0];
 
        switch (gs_type) {
@@ -940,7 +941,7 @@ static int zfcp_fc_exec_els_job(struct fc_bsg_job *job,
                                struct zfcp_adapter *adapter)
 {
        struct zfcp_fsf_ct_els *els = job->dd_data;
-       struct fc_rport *rport = job->rport;
+       struct fc_rport *rport = fc_bsg_to_rport(job);
        struct fc_bsg_request *bsg_request = job->request;
        struct zfcp_port *port;
        u32 d_id;
@@ -988,8 +989,9 @@ int zfcp_fc_exec_bsg_job(struct fc_bsg_job *job)
        struct zfcp_adapter *adapter;
        struct zfcp_fsf_ct_els *ct_els = job->dd_data;
        struct fc_bsg_request *bsg_request = job->request;
+       struct fc_rport *rport = fc_bsg_to_rport(job);
 
-       shost = job->rport ? rport_to_shost(job->rport) : fc_bsg_to_shost(job);
+       shost = rport ? rport_to_shost(rport) : fc_bsg_to_shost(job);
        adapter = (struct zfcp_adapter *)shost->hostdata[0];
 
        if (!(atomic_read(&adapter->status) & ZFCP_STATUS_COMMON_OPEN))

diff --git a/drivers/scsi/ibmvscsi/ibmvfc.c b/drivers/scsi/ibmvscsi/ibmvfc.c
index 02df1f156f1ba163394644c7086fc9f31d9ec142..4c73fc735b1389b9dc2a1bc4dd59de5edacc5d1f 100644 (file)
--- a/drivers/scsi/ibmvscsi/ibmvfc.c
+++ b/drivers/scsi/ibmvscsi/ibmvfc.c
@@ -1822,7 +1822,7 @@ unlock_out:
 static int ibmvfc_bsg_request(struct fc_bsg_job *job)
 {
        struct ibmvfc_host *vhost = shost_priv(fc_bsg_to_shost(job));
-       struct fc_rport *rport = job->rport;
+       struct fc_rport *rport = fc_bsg_to_rport(job);
        struct ibmvfc_passthru_mad *mad;
        struct ibmvfc_event *evt;
        union ibmvfc_iu rsp_iu;

diff --git a/drivers/scsi/libfc/fc_lport.c b/drivers/scsi/libfc/fc_lport.c
index 3e3afe6e0cde8fbb950e4f095a71765f0b6bda91..5e24ca3118c3f8bfb3de4a4201fecbac24c5f689 100644 (file)
--- a/drivers/scsi/libfc/fc_lport.c
+++ b/drivers/scsi/libfc/fc_lport.c
@@ -2103,7 +2103,7 @@ int fc_lport_bsg_request(struct fc_bsg_job *job)
 
        switch (bsg_request->msgcode) {
        case FC_BSG_RPT_ELS:
-               rport = job->rport;
+               rport = fc_bsg_to_rport(job);
                if (!rport)
                        break;
 
@@ -2113,7 +2113,7 @@ int fc_lport_bsg_request(struct fc_bsg_job *job)
                break;
 
        case FC_BSG_RPT_CT:
-               rport = job->rport;
+               rport = fc_bsg_to_rport(job);
                if (!rport)
                        break;
 

diff --git a/drivers/scsi/lpfc/lpfc_bsg.c b/drivers/scsi/lpfc/lpfc_bsg.c
index 45184ee5fbaae75c70389c94186672494db58353..19847d7d4177360cf8957f61fc106d65798466a4 100644 (file)
--- a/drivers/scsi/lpfc/lpfc_bsg.c
+++ b/drivers/scsi/lpfc/lpfc_bsg.c
@@ -386,7 +386,7 @@ lpfc_bsg_send_mgmt_cmd(struct fc_bsg_job *job)
 {
        struct lpfc_vport *vport = shost_priv(fc_bsg_to_shost(job));
        struct lpfc_hba *phba = vport->phba;
-       struct lpfc_rport_data *rdata = job->rport->dd_data;
+       struct lpfc_rport_data *rdata = fc_bsg_to_rport(job)->dd_data;
        struct lpfc_nodelist *ndlp = rdata->pnode;
        struct fc_bsg_reply *bsg_reply = job->reply;
        struct ulp_bde64 *bpl = NULL;
@@ -660,7 +660,7 @@ lpfc_bsg_rport_els(struct fc_bsg_job *job)
 {
        struct lpfc_vport *vport = shost_priv(fc_bsg_to_shost(job));
        struct lpfc_hba *phba = vport->phba;
-       struct lpfc_rport_data *rdata = job->rport->dd_data;
+       struct lpfc_rport_data *rdata = fc_bsg_to_rport(job)->dd_data;
        struct lpfc_nodelist *ndlp = rdata->pnode;
        struct fc_bsg_request *bsg_request = job->request;
        struct fc_bsg_reply *bsg_reply = job->reply;

diff --git a/drivers/scsi/qla2xxx/qla_bsg.c b/drivers/scsi/qla2xxx/qla_bsg.c
index 109b852daada773da10287dc49f04b26e15ccfb0..917eafecc435fec6a02d0dc9f2337509435d8952 100644 (file)
--- a/drivers/scsi/qla2xxx/qla_bsg.c
+++ b/drivers/scsi/qla2xxx/qla_bsg.c
@@ -264,7 +264,7 @@ qla2x00_process_els(struct fc_bsg_job *bsg_job)
        uint16_t nextlid = 0;
 
        if (bsg_request->msgcode == FC_BSG_RPT_ELS) {
-               rport = bsg_job->rport;
+               rport = fc_bsg_to_rport(bsg_job);
                fcport = *(fc_port_t **) rport->dd_data;
                host = rport_to_shost(rport);
                vha = shost_priv(host);
@@ -2485,7 +2485,7 @@ qla24xx_bsg_request(struct fc_bsg_job *bsg_job)
        bsg_reply->reply_payload_rcv_len = 0;
 
        if (bsg_request->msgcode == FC_BSG_RPT_ELS) {
-               rport = bsg_job->rport;
+               rport = fc_bsg_to_rport(bsg_job);
                host = rport_to_shost(rport);
                vha = shost_priv(host);
        } else {

diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/scsi_transport_fc.c
index 53a59daddfb55b7cf23ca393ede290b03863a85a..45954a6edd9474fc8e89548a6ba8097f27b3a0c7 100644 (file)
--- a/drivers/scsi/scsi_transport_fc.c
+++ b/drivers/scsi/scsi_transport_fc.c
@@ -3640,11 +3640,12 @@ fc_bsg_job_timeout(struct request *req)
 {
        struct fc_bsg_job *job = (void *) req->special;
        struct Scsi_Host *shost = fc_bsg_to_shost(job);
+       struct fc_rport *rport = fc_bsg_to_rport(job);
        struct fc_internal *i = to_fc_internal(shost->transportt);
        unsigned long flags;
        int err = 0, done = 0;
 
-       if (job->rport && job->rport->port_state == FC_PORTSTATE_BLOCKED)
+       if (rport && rport->port_state == FC_PORTSTATE_BLOCKED)
                return BLK_EH_RESET_TIMER;
 
        spin_lock_irqsave(&job->job_lock, flags);

diff --git a/include/scsi/scsi_transport_fc.h b/include/scsi/scsi_transport_fc.h
index efb948816f5d0d7d1e400f061a8e226985823ddb..9f53fe3fa249fb546b8e1c9be70c1f4705f615fa 100644 (file)
--- a/include/scsi/scsi_transport_fc.h
+++ b/include/scsi/scsi_transport_fc.h
@@ -824,6 +824,11 @@ static inline struct Scsi_Host *fc_bsg_to_shost(struct fc_bsg_job *job)
        return job->shost;
 }
 
+static inline struct fc_rport *fc_bsg_to_rport(struct fc_bsg_job *job)
+{
+       return job->rport;
+}
+
 struct scsi_transport_template *fc_attach_transport(
                        struct fc_function_template *);
 void fc_release_transport(struct scsi_transport_template *);