From: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
To: target-devel <target-devel@vger.kernel.org>
Cc: linux-scsi <linux-scsi@vger.kernel.org>,
Roland Dreier <roland@purestorage.com>
Subject: Re: [PATCH 0/4] target: LUN_RESET bugfixes for HW target mode
Date: Thu, 29 Sep 2011 22:25:12 -0700 [thread overview]
Message-ID: <1317360312.27478.89.camel@haakon2.linux-iscsi.org> (raw)
In-Reply-To: <1317358511-10664-1-git-send-email-nab@linux-iscsi.org>
On Fri, 2011-09-30 at 04:55 +0000, Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger <nab@linux-iscsi.org>
>
> Hi folks,
>
> This patch series addresses a number of issues that have come up recently for
> TMR LUN_RESET operation with HW target mode + qla2xxx/tcm_qla2xxx code. Aside
> from the various bugfixes in this series, the big change is the re-org of
> core_tmr_lun_reset() logic to prevent possible list corruption while scanning
> the various lists and breaking this out of this code into seperate functions.
>
> I know the re-org patch diff is very difficult to read without more context, so
> apologies ahead of time for that. At this point it makes sense to queue these
> for v3.2, and CC stable to ensure they are also picked up for v3.1 target code
> as well.
>
> Please let us know if you have any comments.
>
> Thank you,
>
> --nab
>
> Nicholas Bellinger (3):
> target: Re-org of core_tmr_lun_reset + FREE_CMD_INTR bugfix
> target: Fix transport_cmd_finish_abort queue removal bug
> target: Prevent transport_send_task_abort when CHECK_CONDITION status
>
> Roland Dreier (1):
> target: Prevent cmd->se_queue_node double add
>
> drivers/target/target_core_tmr.c | 228 ++++++++++++++++++++------------
> drivers/target/target_core_transport.c | 45 ++++---
> 2 files changed, 169 insertions(+), 104 deletions(-)
>
Hi again folks,
The full context for 'Re-org of core_tmr_lun_reset + FREE_CMD_INTR
bugfix' can be found here for review.
http://www.risingtidesystems.com/git/?p=lio-core-2.6.git;a=blob;f=drivers/target/target_core_tmr.c;h=89459086e68628a87a77dd086c6136871db86f0b;hb=HEAD#l104
Thanks,
--nab
prev parent reply other threads:[~2011-09-30 5:25 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-09-30 4:55 [PATCH 0/4] target: LUN_RESET bugfixes for HW target mode Nicholas A. Bellinger
2011-09-30 4:55 ` [PATCH 1/4] target: Prevent cmd->se_queue_node double add Nicholas A. Bellinger
2011-10-09 1:24 ` Christoph Hellwig
2011-09-30 4:55 ` [PATCH 2/4] target: Re-org of core_tmr_lun_reset + FREE_CMD_INTR bugfix Nicholas A. Bellinger
2011-10-09 1:30 ` Christoph Hellwig
2011-10-09 2:00 ` Nicholas A. Bellinger
2011-09-30 4:55 ` [PATCH 3/4] target: Fix transport_cmd_finish_abort queue removal bug Nicholas A. Bellinger
2011-10-09 1:33 ` Christoph Hellwig
2011-10-09 2:03 ` Nicholas A. Bellinger
2011-10-09 2:27 ` Christoph Hellwig
2011-10-09 9:06 ` Nicholas A. Bellinger
2011-09-30 4:55 ` [PATCH 4/4] target: Prevent transport_send_task_abort when CHECK_CONDITION status Nicholas A. Bellinger
2011-09-30 5:25 ` Nicholas A. Bellinger [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=1317360312.27478.89.camel@haakon2.linux-iscsi.org \
--to=nab@linux-iscsi.org \
--cc=linux-scsi@vger.kernel.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.