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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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





[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux