From: Sathya Prakash Veerichetty <sathya.prakash@broadcom.com>
To: "Martin K. Petersen" <martin.petersen@oracle.com>,
Alexey Khoroshilov <khoroshilov@ispras.ru>
Cc: Sreekanth Reddy <Sreekanth.Reddy@avagotech.com>,
Chaitra Basappa <chaitra.basappa@broadcom.com>,
Suganath Prabu Subramani <suganath-prabu.subramani@broadcom.com>,
PDL-MPT-FUSIONLINUX <mpt-fusionlinux.pdl@broadcom.com>,
linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org,
ldv-project@linuxtesting.org
Subject: RE: [PATCH] mptsas: fix checks for dma mapping errors
Date: Fri, 29 Apr 2016 13:13:36 -0600 [thread overview]
Message-ID: <e4d00c4701ebcea4239019574ffc25fd@mail.gmail.com> (raw)
In-Reply-To: <yq1wpnicyzq.fsf@sermon.lab.mkp.net>
Please consider this patch as Ack-by: Sathya Prakash
Veerichetty<sathya.prakash@broadcom.com>
PS: We don't have test environment to test this patch as this is for an
old controller. So ACKing based on code review and similar mpt3sas driver
code.
-----Original Message-----
From: Martin K. Petersen [mailto:martin.petersen@oracle.com]
Sent: Wednesday, April 27, 2016 7:18 PM
To: Alexey Khoroshilov
Cc: Sreekanth Reddy; Sathya Prakash; Chaitra P B; Suganath Prabu
Subramani; MPT-FusionLinux.pdl@broadcom.com; linux-scsi@vger.kernel.org;
linux-kernel@vger.kernel.org; ldv-project@linuxtesting.org
Subject: Re: [PATCH] mptsas: fix checks for dma mapping errors
>>>>> "Alexey" == Alexey Khoroshilov <khoroshilov@ispras.ru> writes:
Alexey> mptsas_smp_handler() checks for dma mapping errors by comparison
Alexey> returned address with zero, while pci_dma_mapping_error() should
Alexey> be used.
Broadcom folks, please review!
--
Martin K. Petersen Oracle Linux Engineering
prev parent reply other threads:[~2016-04-29 19:13 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-15 23:12 [PATCH] mptsas: fix checks for dma mapping errors Alexey Khoroshilov
2016-04-28 1:18 ` Martin K. Petersen
2016-04-29 19:13 ` Sathya Prakash Veerichetty [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=e4d00c4701ebcea4239019574ffc25fd@mail.gmail.com \
--to=sathya.prakash@broadcom.com \
--cc=Sreekanth.Reddy@avagotech.com \
--cc=chaitra.basappa@broadcom.com \
--cc=khoroshilov@ispras.ru \
--cc=ldv-project@linuxtesting.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=mpt-fusionlinux.pdl@broadcom.com \
--cc=suganath-prabu.subramani@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).