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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.