[SCSI] bfa: fix prli retry issues

Add a max retry limit for PRLI retries. Max retry limit (5) is same as used
in rport PLOGI. Once the retries are exhausted, invoke rport offline so that
existing logic of rport re-discovery can kick-in. Also fixed a bug in rport.c
where one less retry was happening.

Signed-off-by: Jing Huang <huangj@brocade.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>

1 files changed, 11 insertions, 2 deletions

diff --git a/drivers/scsi/bfa/fcpim.c b/drivers/scsi/bfa/fcpim.c
index 7a0207e56693..d090f7a6368a 100644
--- a/drivers/scsi/bfa/fcpim.c
+++ b/drivers/scsi/bfa/fcpim.c
@@ -110,6 +110,7 @@ bfa_fcs_itnim_sm_offline(struct bfa_fcs_itnim_s *itnim,
         switch (event) {
         case BFA_FCS_ITNIM_SM_ONLINE:
                 bfa_sm_set_state(itnim, bfa_fcs_itnim_sm_prli_send);
+                itnim->prli_retries = 0;
                 bfa_fcs_itnim_send_prli(itnim, NULL);
                 break;
 
@@ -218,8 +219,16 @@ bfa_fcs_itnim_sm_prli_retry(struct bfa_fcs_itnim_s *itnim,
 
         switch (event) {
         case BFA_FCS_ITNIM_SM_TIMEOUT:
-                bfa_sm_set_state(itnim, bfa_fcs_itnim_sm_prli_send);
-                bfa_fcs_itnim_send_prli(itnim, NULL);
+                if (itnim->prli_retries < BFA_FCS_RPORT_MAX_RETRIES) {
+                        itnim->prli_retries++;
+                        bfa_trc(itnim->fcs, itnim->prli_retries);
+                        bfa_sm_set_state(itnim, bfa_fcs_itnim_sm_prli_send);
+                        bfa_fcs_itnim_send_prli(itnim, NULL);
+                } else {
+                        /* invoke target offline */
+                        bfa_sm_set_state(itnim, bfa_fcs_itnim_sm_offline);
+                        bfa_fcs_rport_logo_imp(itnim->rport);
+                }
                 break;
 
         case BFA_FCS_ITNIM_SM_OFFLINE: