Re: [PATCH 22/43] qla2xxx: Add switch command to simplify fabric discovery

Madhani, Himanshu
2017-12-21T00:04:03Z

Hi Ewan,

> On Dec 20, 2017, at 12:09 PM, Ewan D. Milne <emilne@redhat.com> 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