From: Johannes Thumshirn <jthumshirn@suse.de>
To: Raghava Aditya Renukunta <RaghavaAditya.Renukunta@pmcs.com>,
"JBottomley@Parallels.com" <JBottomley@Parallels.com>,
"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>
Cc: Mahesh Rajashekhara <Mahesh.Rajashekhara@pmcs.com>,
Murthy Bhat <Murthy.Bhat@pmcs.com>,
Santosh Akula <Santosh.Akula@pmcs.com>,
Gana Sridaran <Gana.Sridaran@pmcs.com>,
"aacraid@pmc-sierra.com" <aacraid@pmc-sierra.com>,
Rich Bono <Rich.Bono@pmcs.com>
Subject: Re: [PATCH 07/10] aacraid: Fix AIF triggered IOP_RESET
Date: Thu, 03 Dec 2015 09:03:32 +0100 [thread overview]
Message-ID: <1449129812.3311.3.camel@suse.de> (raw)
In-Reply-To: <198D06D448A18D4E93F08FB849C4E39D7D1EE83A@BBYEXM01.pmc-sierra.internal>
Hi Raghava,
On Wed, 2015-12-02 at 22:29 +0000, Raghava Aditya Renukunta wrote:
> Hello Johannes,
>
> > -----Original Message-----
[...]
> Originally the problem was that dev->adapter_shutdown was set after the
> Shutdown command was sent to the controller, leading to a race
> condition If after the controller was shut down and before
> dev->adapter_shutdown was set , the driver received a AIF
> command from management tool , it would still be sent to the
> controller in aac_cfg_ioctl() leading to a IOP_RESET.
>
> A fix would be to set dev->adapter_shutdown at the very
> beginning of aac_send_shutdown().
That's what I though as well
Johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next prev parent reply other threads:[~2015-12-03 8:03 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-01 12:39 [PATCH 00/10] aacraid: Patchset for aacraid driver version 41052 Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 01/10] aacraid: SCSI blk tag support Raghava Aditya Renukunta
2015-12-02 10:49 ` Johannes Thumshirn
2015-12-03 15:52 ` Tomas Henzl
2015-12-03 21:25 ` Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 02/10] aacraid: Fix RRQ overload Raghava Aditya Renukunta
2015-12-02 9:26 ` Johannes Thumshirn
2015-12-04 14:11 ` Tomas Henzl
2015-12-01 12:39 ` [PATCH 03/10] aacraid: Added EEH support Raghava Aditya Renukunta
2015-12-02 9:41 ` Johannes Thumshirn
2015-12-02 23:14 ` Raghava Aditya Renukunta
2015-12-04 14:20 ` Tomas Henzl
2015-12-01 12:39 ` [PATCH 04/10] aacraid: Fix memory leak in aac_fib_map_free Raghava Aditya Renukunta
2015-12-02 9:44 ` Johannes Thumshirn
2015-12-04 14:34 ` Tomas Henzl
2015-12-05 0:40 ` Raghava Aditya Renukunta
2015-12-07 14:05 ` Tomas Henzl
2015-12-07 19:07 ` Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 05/10] aacraid: Set correct msix count for EEH recovery Raghava Aditya Renukunta
2015-12-02 10:27 ` Johannes Thumshirn
2015-12-02 22:59 ` Raghava Aditya Renukunta
2015-12-04 14:10 ` Tomas Henzl
2015-12-05 0:15 ` Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 06/10] aacraid: Fundamental reset support for Series 7 Raghava Aditya Renukunta
2015-12-02 9:49 ` Johannes Thumshirn
2015-12-01 12:39 ` [PATCH 07/10] aacraid: Fix AIF triggered IOP_RESET Raghava Aditya Renukunta
2015-12-02 10:00 ` Johannes Thumshirn
2015-12-02 22:29 ` Raghava Aditya Renukunta
2015-12-03 8:03 ` Johannes Thumshirn [this message]
2015-12-01 12:39 ` [PATCH 08/10] aacraid: Disable device ID wildcard Raghava Aditya Renukunta
2015-12-02 10:02 ` Johannes Thumshirn
2015-12-03 15:54 ` Tomas Henzl
2015-12-03 21:32 ` Raghava Aditya Renukunta
2015-12-04 8:33 ` Christoph Hellwig
2015-12-07 19:07 ` Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 09/10] aacraid: Fix character device re-initialization Raghava Aditya Renukunta
2015-12-02 10:13 ` Johannes Thumshirn
2015-12-02 22:30 ` Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 09/10] aacraid: Fix character device re initialization Raghava Aditya Renukunta
2015-12-02 9:18 ` Johannes Thumshirn
2015-12-02 21:59 ` Raghava Aditya Renukunta
2015-12-01 12:39 ` [PATCH 10/10] aacraid: Update driver version Raghava Aditya Renukunta
2015-12-02 10:14 ` Johannes Thumshirn
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=1449129812.3311.3.camel@suse.de \
--to=jthumshirn@suse.de \
--cc=Gana.Sridaran@pmcs.com \
--cc=JBottomley@Parallels.com \
--cc=Mahesh.Rajashekhara@pmcs.com \
--cc=Murthy.Bhat@pmcs.com \
--cc=RaghavaAditya.Renukunta@pmcs.com \
--cc=Rich.Bono@pmcs.com \
--cc=Santosh.Akula@pmcs.com \
--cc=aacraid@pmc-sierra.com \
--cc=linux-scsi@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 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).