Hi Ewan, > On Dec 20, 2017, at 12:09 PM, Ewan D. Milne <emilne@xxxxxxxxxx> wrote: > > On Tue, 2017-12-19 at 22:56 -0800, Himanshu Madhani wrote: >> @@ -4915,12 +4941,32 @@ static int qlt_24xx_handle_els(struct scsi_qla_host *vha, >> } >> >> if (sess != NULL) { >> + bool delete = false; >> spin_lock_irqsave(&tgt->ha->tgt.sess_lock, flags); >> switch (sess->fw_login_state) { >> + case DSC_LS_PLOGI_PEND: >> case DSC_LS_PLOGI_COMP: >> case DSC_LS_PRLI_COMP: >> break; >> default: >> + delete = true; > > Nitpick: git quiltimport complains about the whitespace error > (space before tab) on the above line. > > Thanks for review. Will fix in v2 Thanks, - Himanshu