On 07/06/2016 05:14 PM, Himanshu Madhani wrote: > From: Giridhar Malavali <[email protected]> > > When VP decoupling enabled, there could be a window where, FLOGI from > initiators > can be dropped before VP0 is enabled, causing link level recovery. > Retry FLOGI to avoid link level recovery. > > Signed-off-by: Giridhar Malavali <[email protected]> > Signed-off-by: Himanshu Madhani <[email protected]> > --- > drivers/scsi/qla2xxx/qla_dbg.c | 2 +- > drivers/scsi/qla2xxx/qla_init.c | 24 ++++++++++++++++++++++++ > 2 files changed, 25 insertions(+), 1 deletion(-) > > diff --git a/drivers/scsi/qla2xxx/qla_dbg.c b/drivers/scsi/qla2xxx/qla_dbg.c > index 0671f07..907a7ac 100644 > --- a/drivers/scsi/qla2xxx/qla_dbg.c > +++ b/drivers/scsi/qla2xxx/qla_dbg.c > @@ -16,7 +16,7 @@ > * | | | 0x016e-0x0170 | > * | Mailbox commands | 0x1192 | | > * | | | | > - * | Device Discovery | 0x2016 | 0x2020-0x2022, | > + * | Device Discovery | 0x2003 | 0x2016 > | > * | | | 0x2011-0x2012, | > * | | | 0x2099-0x20a4 | > * | Queue Command and IO tracing | 0x3074 | 0x300b | > diff --git a/drivers/scsi/qla2xxx/qla_init.c b/drivers/scsi/qla2xxx/qla_init.c > index c56cdb3..a3685b6 100644 > --- a/drivers/scsi/qla2xxx/qla_init.c > +++ b/drivers/scsi/qla2xxx/qla_init.c > @@ -2053,6 +2053,14 @@ qla2x00_update_fw_options(scsi_qla_host_t *vha) > if (IS_QLA6312(ha)) > ha->fw_options[2] |= BIT_13; > > + /* Set Retry FLOGI in case of P2P connection */ > + if (ha->operating_mode == P2P) { > + ha->fw_options[2] |= BIT_3; > + ql_dbg(ql_dbg_disc, vha, 0x2100, > + "(%s): Setting FLOGI retry BIT in fw_options[2]: 0x%x\n", > + __func__, ha->fw_options[2]); > + } > + > /* Update firmware options. */ > qla2x00_set_fw_options(vha, ha->fw_options); > } > @@ -2070,6 +2078,14 @@ qla24xx_update_fw_options(scsi_qla_host_t *vha) > if (ql2xfwholdabts) > ha->fw_options[3] |= BIT_12; > > + /* Set Retry FLOGI in case of P2P connection */ > + if (ha->operating_mode == P2P) { > + ha->fw_options[2] |= BIT_3; > + ql_dbg(ql_dbg_disc, vha, 0x2101, > + "(%s): Setting FLOGI retry BIT in fw_options[2]: 0x%x\n", > + __func__, ha->fw_options[2]); > + } > + > /* Update Serial Link options. */ > if ((le16_to_cpu(ha->fw_seriallink_options24[0]) & BIT_0) == 0) > return; > @@ -6513,6 +6529,14 @@ qla81xx_update_fw_options(scsi_qla_host_t *vha) > if (ql2xfwholdabts) > ha->fw_options[3] |= BIT_12; > > + /* Set Retry FLOGI in case of P2P connection */ > + if (ha->operating_mode == P2P) { > + ha->fw_options[2] |= BIT_3; > + ql_dbg(ql_dbg_disc, vha, 0x2103, > + "(%s): Setting FLOGI retry BIT in fw_options[2]: 0x%x\n", > + __func__, ha->fw_options[2]); > + } > + > if (!ql2xetsenable) > goto out; > > Is this bit available for all firmware versions? Or does one need a newer firmware here?
Cheers, Hannes -- Dr. Hannes Reinecke Teamlead Storage & Networking [email protected] +49 911 74053 688 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton HRB 21284 (AG Nürnberg) -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
