Re: [PATCH v3] mptlan: add checks for dma mapping errors

2016-01-27 Thread James Bottomley
On Wed, 2016-01-27 at 17:14 +0100, Tomas Henzl wrote: > On 27.1.2016 06:44, Sathya Prakash wrote: > > There is no fusion based network card and resources exists today in > > Avago(LSI) to test this patch so we prefer to leave it as is. We > > would > > like to prevent any new changes on MPT (FC/SCS

Re: [PATCH v3] mptlan: add checks for dma mapping errors

2016-01-27 Thread Tomas Henzl
Alexey Khoroshilov; Sreekanth Reddy; > mpt-fusionlinux@avagotech.com; linux-s...@vger.kernel.org; > linux-kernel@vger.kernel.org; ldv-proj...@linuxtesting.org > Subject: Re: [PATCH v3] mptlan: add checks for dma mapping errors > >>>>>> "Tomas" == Tomas Henzl w

RE: [PATCH v3] mptlan: add checks for dma mapping errors

2016-01-26 Thread Sathya Prakash
kanth Reddy; mpt-fusionlinux@avagotech.com; linux-s...@vger.kernel.org; linux-kernel@vger.kernel.org; ldv-proj...@linuxtesting.org Subject: Re: [PATCH v3] mptlan: add checks for dma mapping errors >>>>> "Tomas" == Tomas Henzl writes: Tomas> Other than that - previ

Re: [PATCH v3] mptlan: add checks for dma mapping errors

2016-01-26 Thread Martin K. Petersen
> "Tomas" == Tomas Henzl writes: Tomas> Other than that - previous patch for this driver came in in 2010 Tomas> - six years ago and the driver seems unmaintained now. I'm not Tomas> sure if we should fix hw we can't test and when there is not an Tomas> user bug report. This example nicely sh

Re: [PATCH v3] mptlan: add checks for dma mapping errors

2016-01-26 Thread Tomas Henzl
On 25.1.2016 19:02, Alexey Khoroshilov wrote: > mpt_lan_sdu_send() and mpt_lan_post_receive_buckets() do not check > if mapping dma memory succeed. > The patch adds the checks and failure handling. > > v3: Fix resource deallocation (reported by Tomas Henzl). > > Found by Linux Driver Verification p

[PATCH v3] mptlan: add checks for dma mapping errors

2016-01-25 Thread Alexey Khoroshilov
mpt_lan_sdu_send() and mpt_lan_post_receive_buckets() do not check if mapping dma memory succeed. The patch adds the checks and failure handling. v3: Fix resource deallocation (reported by Tomas Henzl). Found by Linux Driver Verification project (linuxtesting.org). Signed-off-by: Alexey Khoroshi