Re: [PATCH 0/10] qla2xxx: Reduce the number of Coverity warnings

2013-06-12 Thread Saurav Kashyap
Hi Bart, Thanks for patchset. There are minor issue with some patches like updating the missing msg_ids in qla_dbg.c file. I am replying to each of the patch. Thanks, ~Saurav On 06/07/13 21:06, Saurav Kashyap wrote: Thanks for the patches. Please share the warnings reported by Coverity, so

Re: [PATCH 0/10] qla2xxx: Reduce the number of Coverity warnings

2013-06-08 Thread Bart Van Assche
On 06/07/13 21:06, Saurav Kashyap wrote: Thanks for the patches. Please share the warnings reported by Coverity, so that its easy for us to review the patches. Hello Saurav, The Coverity warnings that led me to developing this patch series are as follows: - For patches 1, 2, 3, 8: Logically

Re: [PATCH 0/10] qla2xxx: Reduce the number of Coverity warnings

2013-06-07 Thread Saurav Kashyap
Hi Bart, Thanks for the patches. Please share the warnings reported by Coverity, so that its easy for us to review the patches. Thanks, ~Saurav The current implementation of the qla2xxx Fibre Channel initiator driver triggers multiple warnings when analyzed with the Coverity static analyzer.