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

Regards,
Quinn Tran







-----Original Message-----
From: Bart Van Assche <bart.vanass...@sandisk.com>
Date: Thursday, March 31, 2016 at 8:22 AM
To: Quinn Tran <quinn.t...@qlogic.com>, James Bottomley 
<j...@linux.vnet.ibm.com>, "Martin K. Petersen" <martin.peter...@oracle.com>
Cc: Himanshu Madhani <himanshu.madh...@qlogic.com>, Christoph Hellwig 
<h...@lst.de>, linux-scsi <linux-scsi@vger.kernel.org>
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�����r��y����b�X��ǧv�^�)޺{.n�+����{���"�{ay�ʇڙ�,j��f���h���z��w���
���j:+v���w�j�m��������zZ+�����ݢj"��!�i

Reply via email to