[SCSI] libfc: use PRLI hook to get parameters when sending outgoing PRLI
authorJoe Eykholt <jeykholt@cisco.com>
Sat, 29 Jan 2011 00:04:23 +0000 (16:04 -0800)
committerJames Bottomley <James.Bottomley@suse.de>
Sat, 12 Feb 2011 17:02:22 +0000 (11:02 -0600)
When sending an outgoing PRLI as an initiator, get the parameters
from registered providers so that they all get a chance to decide
on roles.

The passive provider is called last, and could override the
initiator role.

Signed-off-by: Joe Eykholt <jeykholt@cisco.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/libfc/fc_rport.c

index a92954c..9ded612 100644 (file)
@@ -961,6 +961,8 @@ static void fc_rport_prli_resp(struct fc_seq *sp, struct fc_frame *fp,
                struct fc_els_prli prli;
                struct fc_els_spp spp;
        } *pp;
+       struct fc_els_spp temp_spp;
+       struct fc4_prov *prov;
        u32 roles = FC_RPORT_ROLE_UNKNOWN;
        u32 fcp_parm = 0;
        u8 op;
@@ -1009,6 +1011,13 @@ static void fc_rport_prli_resp(struct fc_seq *sp, struct fc_frame *fp,
                if (fcp_parm & FCP_SPPF_RETRY)
                        rdata->flags |= FC_RP_FLAGS_RETRY;
 
+               prov = fc_passive_prov[FC_TYPE_FCP];
+               if (prov) {
+                       memset(&temp_spp, 0, sizeof(temp_spp));
+                       prov->prli(rdata, pp->prli.prli_spp_len,
+                                  &pp->spp, &temp_spp);
+               }
+
                rdata->supported_classes = FC_COS_CLASS3;
                if (fcp_parm & FCP_SPPF_INIT_FCN)
                        roles |= FC_RPORT_ROLE_FCP_INITIATOR;
@@ -1045,6 +1054,7 @@ static void fc_rport_enter_prli(struct fc_rport_priv *rdata)
                struct fc_els_spp spp;
        } *pp;
        struct fc_frame *fp;
+       struct fc4_prov *prov;
 
        /*
         * If the rport is one of the well known addresses
@@ -1066,9 +1076,20 @@ static void fc_rport_enter_prli(struct fc_rport_priv *rdata)
                return;
        }
 
-       if (!lport->tt.elsct_send(lport, rdata->ids.port_id, fp, ELS_PRLI,
-                                 fc_rport_prli_resp, rdata,
-                                 2 * lport->r_a_tov))
+       fc_prli_fill(lport, fp);
+
+       prov = fc_passive_prov[FC_TYPE_FCP];
+       if (prov) {
+               pp = fc_frame_payload_get(fp, sizeof(*pp));
+               prov->prli(rdata, sizeof(pp->spp), NULL, &pp->spp);
+       }
+
+       fc_fill_fc_hdr(fp, FC_RCTL_ELS_REQ, rdata->ids.port_id,
+                      fc_host_port_id(lport->host), FC_TYPE_ELS,
+                      FC_FC_FIRST_SEQ | FC_FC_END_SEQ | FC_FC_SEQ_INIT, 0);
+
+       if (!lport->tt.exch_seq_send(lport, fp, fc_rport_prli_resp,
+                                   NULL, rdata, 2 * lport->r_a_tov))
                fc_rport_error_retry(rdata, NULL);
        else
                kref_get(&rdata->kref);