From: Douglas Gilbert <dgilbert@interlog.com>
To: Akinobu Mita <akinobu.mita@gmail.com>
Cc: linux-scsi@vger.kernel.org,
"James E.J. Bottomley" <JBottomley@parallels.com>,
"Martin K. Petersen" <martin.petersen@oracle.com>
Subject: Re: [PATCH 0/3] scsi_debug: fix data integrity support
Date: Wed, 24 Apr 2013 19:42:57 -0400 [thread overview]
Message-ID: <51786E01.9090101@interlog.com> (raw)
In-Reply-To: <1366535879-20775-1-git-send-email-akinobu.mita@gmail.com>
On 13-04-21 05:17 AM, Akinobu Mita wrote:
> When I tried testing the data integrity support in scsi_debug on x86_32,
> I got CONFIG_DEBUG_HIGHMEM warnings and protection errors. This was
> triggered due to misused kmap_atomic/kunmap_atomic.
>
> And then, while I was testing the fix of the above issue with several
> combination with module parameters dix and dif, I found that doing
> 'modprobe scsi_debug dif=0 dix=1' causes kernel crash.
>
> This patch set includes these fixes and cleanup which is related to data
> integrity support.
>
> Cc: "James E.J. Bottomley" <JBottomley@parallels.com>
> Cc: Douglas Gilbert <dgilbert@interlog.com>
> Cc: "Martin K. Petersen" <martin.petersen@oracle.com>
> Cc: linux-scsi@vger.kernel.org
>
> Akinobu Mita (3):
> scsi_debug: fix data integrity support on highmem machine
> scsi_debug: fix NULL pointer dereference with parameters dif=0 dix=1
> scsi_debug: simplify offset calculation for dif_storep
>
> drivers/scsi/scsi_debug.c | 29 ++++++++++++-----------------
> 1 file changed, 12 insertions(+), 17 deletions(-)
Again, I'd like to see some feedback from Martin Petersen
on this set of patches. For my part, for this patch series
(1/3 to 3/3):
Acked-by: Douglas Gilbert <dgilbert@interlog.com>
prev parent reply other threads:[~2013-04-24 23:43 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-21 9:17 [PATCH 0/3] scsi_debug: fix data integrity support Akinobu Mita
2013-04-21 9:17 ` [PATCH 1/3] scsi_debug: fix data integrity support on highmem machine Akinobu Mita
2013-04-25 2:13 ` Martin K. Petersen
2013-04-27 8:51 ` Akinobu Mita
2013-04-21 9:17 ` [PATCH 2/3] scsi_debug: fix NULL pointer dereference with parameters dif=0 dix=1 Akinobu Mita
2013-04-25 2:29 ` Martin K. Petersen
2013-04-27 8:37 ` Akinobu Mita
2013-04-21 9:17 ` [PATCH 3/3] scsi_debug: simplify offset calculation for dif_storep Akinobu Mita
2013-04-25 2:18 ` Martin K. Petersen
2013-04-24 23:42 ` Douglas Gilbert [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=51786E01.9090101@interlog.com \
--to=dgilbert@interlog.com \
--cc=JBottomley@parallels.com \
--cc=akinobu.mita@gmail.com \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.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.