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