From: Paolo Bonzini <pbonzini@redhat.com>
To: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
Cc: target-devel <target-devel@vger.kernel.org>,
linux-scsi <linux-scsi@vger.kernel.org>,
Christoph Hellwig <hch@lst.de>,
Roland Dreier <roland@purestorage.com>
Subject: Re: [PATCH 0/3] target: Fix zero-length regressions in v3.8-rc1 code
Date: Wed, 30 Jan 2013 10:24:02 +0100 [thread overview]
Message-ID: <5108E6B2.8090101@redhat.com> (raw)
In-Reply-To: <1359498375-8040-1-git-send-email-nab@linux-iscsi.org>
Il 29/01/2013 23:26, Nicholas A. Bellinger ha scritto:
> From: Nicholas Bellinger <nab@linux-iscsi.org>
>
> Hi folks,
>
> The following are a handful of zero-length CDB regression bugfixes to address
> breakage introduced by the recent sense_reason_t conversion in v3.8-rc1 code,
> which incorrectly assumed CHECK_CONDITION status (in all CDB emulation cases)
> when NULL was returned by transport_kmap_data_sg().
>
> Please review, as I'd like to get these into v3.8-rc6.
>
> Thank you,
>
> --nab
>
> Nicholas Bellinger (3):
> target: Fix zero-length INQUIRY additional sense code regression
> target: Fix zero-length MODE_SENSE regression
> target: Fix zero-length READ_CAPACITY_16 regression
>
> drivers/target/target_core_sbc.c | 18 +++++++--------
> drivers/target/target_core_spc.c | 44 +++++++++----------------------------
> 2 files changed, 19 insertions(+), 43 deletions(-)
>
Looks good, but given the mess I made in my own zero-length patches,
don't really count it as a Reviewed-by.
Paolo
prev parent reply other threads:[~2013-01-30 9:24 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-29 22:26 [PATCH 0/3] target: Fix zero-length regressions in v3.8-rc1 code Nicholas A. Bellinger
2013-01-29 22:26 ` [PATCH 1/3] target: Fix zero-length INQUIRY additional sense code regression Nicholas A. Bellinger
2013-01-29 22:26 ` [PATCH 2/3] target: Fix zero-length MODE_SENSE regression Nicholas A. Bellinger
2013-01-29 22:26 ` [PATCH 3/3] target: Fix zero-length READ_CAPACITY_16 regression Nicholas A. Bellinger
2013-01-30 9:24 ` Paolo Bonzini [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=5108E6B2.8090101@redhat.com \
--to=pbonzini@redhat.com \
--cc=hch@lst.de \
--cc=linux-scsi@vger.kernel.org \
--cc=nab@linux-iscsi.org \
--cc=roland@purestorage.com \
--cc=target-devel@vger.kernel.org \
/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.