Re: [PATCH 3/3] qla2xxx: Assign names to the flags in cmd_flags

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

 



Agreed.  It’s one of our pending action items for cleanup.  

Regards,
Quinn Tran







-----Original Message-----
From: Bart Van Assche <bart.vanassche@xxxxxxxxxxx>
Date: Thursday, March 31, 2016 at 8:22 AM
To: Quinn Tran <quinn.tran@xxxxxxxxxx>, James Bottomley <jejb@xxxxxxxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Cc: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, linux-scsi <linux-scsi@xxxxxxxxxxxxxxx>
Subject: Re: [PATCH 3/3] qla2xxx: Assign names to the flags in cmd_flags

>On 03/31/16 08:16, Quinn Tran wrote:
>> We use these flags for debugging purpose in crash cases.
>
>Hello Quinn,
>
>It must be inconvenient to have to look up the meaning of the individual 
>bits of the cmd_flags variable when analyzing a kernel crash. How about 
>preserving all these flags and to assign a meaningful name to each flag 
>instead of using bit numbers?
>
>Thanks,
>
>Bart.
>
��.n��������+%������w��{.n�����{������ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f




[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