From: James Smart <James.Smart@emulex.com>
To: Hannes Reinecke <hare@suse.de>
Cc: "Elliott, Robert (Server Storage)" <Elliott@hp.com>,
"emilne@redhat.com" <emilne@redhat.com>,
SCSI Mailing List <linux-scsi@vger.kernel.org>,
Andrew Vasquez <andrew.vasquez@qlogic.com>,
Chad Dupuis <chad.dupuis@qlogic.com>,
James Bottomley <James.Bottomley@HansenPartnership.com>
Subject: Re: Error handling on FC devices
Date: Thu, 29 Nov 2012 11:02:29 -0500 [thread overview]
Message-ID: <50B78715.2060109@emulex.com> (raw)
In-Reply-To: <50B5B8C4.1040503@suse.de>
Always possible - but.... Our f/w works at the FCP level and below,
which means it doesn't know/do SCSI commands - e.g what the cdb within
the FCP CMD frame is; know anything about SCSI device classes and state;
etc. And it shouldn't be required to do so. Anytime this has been there
in the past, it's been problematic.
if we want to do this - we should add it to the midlayer/transport.
-- james s
On 11/28/2012 2:09 AM, Hannes Reinecke wrote:
> On 11/27/2012 09:29 PM, Elliott, Robert (Server Storage) wrote:
>> There is a new command in SPC-4 called REMOVE I_T NEXUS that is
>> intended to help
> > that situation. REMOVE I_T NEXUS lets the application client use a
> good I_T nexus
> > to abort commands that were being processed on a bad I_T nexus, so
> it can safely
> > re-issue those commands on the good I_T nexus without worrying that
> the original
> > commands might resume.
>>
>> In contrast:
>> - the ABORT TASK, ABORT TASK SET, and CLEAR TASK SET must use the
>> same I_T nexus
> > as the commands being aborted, so are not viable if that I_T nexus
> is bad
>> - LOGICAL UNIT RESET aborts commands from every I_T nexus, so in
>> addition to
> > aborting commands from the bad I_T nexus it also affects commands
> on the
> > good I_T nexus
>>
> Hmm. Nice in principle, but the problem here is that we cannot
> guarantee the nexus is still intact.
> So one would need to implement this in the HBA firmware; the firmware
> would need to be able to process the TMF, and do the appropriate
> things for the FC stack like dropping the rport etc.
>
> Good idea, though.
>
> James, Andrew, Chad?
> Any chance of having a firmware supporting REMOVE IT-NEXUS ?
>
> Cheers,
>
> Hannes
next prev parent reply other threads:[~2012-11-29 16:02 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-19 12:41 Error handling on FC devices Hannes Reinecke
2012-11-26 22:32 ` James Smart
2012-11-27 20:03 ` Ewan Milne
2012-11-27 20:29 ` Elliott, Robert (Server Storage)
2012-11-28 7:09 ` Hannes Reinecke
2012-11-29 16:02 ` James Smart [this message]
2012-11-30 11:44 ` Hannes Reinecke
2012-11-30 16:54 ` Mike Christie
2012-12-03 7:15 ` Hannes Reinecke
2012-12-03 17:19 ` Jeremy Linton
2012-12-03 22:52 ` Elliott, Robert (Server Storage)
2012-12-04 15:56 ` Kipp Aldrich
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=50B78715.2060109@emulex.com \
--to=james.smart@emulex.com \
--cc=Elliott@hp.com \
--cc=James.Bottomley@HansenPartnership.com \
--cc=andrew.vasquez@qlogic.com \
--cc=chad.dupuis@qlogic.com \
--cc=emilne@redhat.com \
--cc=hare@suse.de \
--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 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.