On 04/20/2015 03:56 PM, James Smart wrote: > > The lpfc driver does not issue RFF_ID and RFT_ID in the correct sequence > > Signed-off-by: Dick Kennedy <dick.kennedy@xxxxxxxxxx> > Signed-off-by: James Smart <james.smart@xxxxxxxxxx> > --- > drivers/scsi/lpfc/lpfc_hbadisc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/lpfc/lpfc_hbadisc.c b/drivers/scsi/lpfc/lpfc_hbadisc.c > index 2500f15..f2db89f 100644 > --- a/drivers/scsi/lpfc/lpfc_hbadisc.c > +++ b/drivers/scsi/lpfc/lpfc_hbadisc.c > @@ -3868,11 +3868,11 @@ out: > > if (vport->port_state < LPFC_VPORT_READY) { > /* Link up discovery requires Fabric registration. */ > - lpfc_ns_cmd(vport, SLI_CTNS_RFF_ID, 0, 0); /* Do this first! */ > lpfc_ns_cmd(vport, SLI_CTNS_RNN_ID, 0, 0); > lpfc_ns_cmd(vport, SLI_CTNS_RSNN_NN, 0, 0); > lpfc_ns_cmd(vport, SLI_CTNS_RSPN_ID, 0, 0); > lpfc_ns_cmd(vport, SLI_CTNS_RFT_ID, 0, 0); > + lpfc_ns_cmd(vport, SLI_CTNS_RFF_ID, 0, 0); > > /* Issue SCR just before NameServer GID_FT Query */ > lpfc_issue_els_scr(vport, SCR_DID, 0); > Hehe. Someone screwed up with the comment ;-) Reviewed-by: Hannes Reinecke <hare@xxxxxxx> Cheers, Hannes -- Dr. Hannes Reinecke zSeries & Storage hare@xxxxxxx +49 911 74053 688 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html