From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alexey Khoroshilov Subject: Re: [PATCH v2] mptlan: add checks for dma mapping errors Date: Mon, 25 Jan 2016 18:08:27 +0100 Message-ID: <56A6568B.9020707@ispras.ru> References: <1453509715-13146-1-git-send-email-khoroshilov@ispras.ru> <56A640E4.2040509@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <56A640E4.2040509@redhat.com> Sender: linux-kernel-owner@vger.kernel.org To: Tomas Henzl , Sreekanth Reddy Cc: MPT-FusionLinux.pdl@avagotech.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org, ldv-project@linuxtesting.org List-Id: linux-scsi@vger.kernel.org On 25.01.2016 16:36, Tomas Henzl wrote: > On 23.1.2016 01:41, 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. >> >> Found by Linux Driver Verification project (linuxtesting.org). >> >> Signed-off-by: Alexey Khoroshilov >> --- >> drivers/message/fusion/mptlan.c | 14 ++++++++++++++ >> 1 file changed, 14 insertions(+) >> >> diff --git a/drivers/message/fusion/mptlan.c b/drivers/message/fusion/mptlan.c >> index cbe96072a6cc..3b6c8a755713 100644 >> --- a/drivers/message/fusion/mptlan.c >> +++ b/drivers/message/fusion/mptlan.c >> @@ -734,6 +734,12 @@ mpt_lan_sdu_send (struct sk_buff *skb, struct net_device *dev) >> >> dma = pci_map_single(mpt_dev->pcidev, skb->data, skb->len, >> PCI_DMA_TODEVICE); >> + if (pci_dma_mapping_error(mpt_dev->pcidev, dma)) { >> + netif_stop_queue(dev); > > Hi Alexey, > isn't a mpt_put_msg_frame needed here ? > and a similar de-alloc in next chunk too? > -tms Hi Tomas, You are right, thank you! I'll resend the patch.