From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Jeff Garzik <jeff@garzik.org>
Cc: linux-scsi <linux-scsi@vger.kernel.org>,
"Darrick J. Wong" <djwong@us.ibm.com>
Subject: Re: [PATCH 2/5] aic94xx: plumb in I_T_nexus_reset task management function
Date: Sun, 24 Feb 2008 00:01:25 -0600 [thread overview]
Message-ID: <1203832885.3139.33.camel@localhost.localdomain> (raw)
In-Reply-To: <47C105FB.8090401@garzik.org>
On Sun, 2008-02-24 at 00:51 -0500, Jeff Garzik wrote:
> Tangent:
>
> Now that patches for ipr and sata_sx4 exist, libsas is the last holdout
> using libata's old error handling paths.
>
> I think Darrick was either scoping that work, or had preliminary patches?
>
> Anyway, that's holding back removal of a lot of libata code, so I wanted
> to make sure the issue remained in people's minds...
I've not really been following what the issue actually is, so hopefully
Darrick has.
At the moment, we don't use any of libata for error handling in libsas.
(In fact, there's a big issue with ATA error handling in libsas in
general, since the sequencer abort task doesn't know what to do with a
task that's at an ATA drive ... we need to detect that condition and
then apply the SATL abort function, which depends on whether we're NCQ
or not...SATA error handling was next on my list of things to look at).
James
prev parent reply other threads:[~2008-02-24 6:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-24 5:37 [PATCH 2/5] aic94xx: plumb in I_T_nexus_reset task management function James Bottomley
2008-02-24 5:51 ` Jeff Garzik
2008-02-24 6:01 ` James Bottomley [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=1203832885.3139.33.camel@localhost.localdomain \
--to=james.bottomley@hansenpartnership.com \
--cc=djwong@us.ibm.com \
--cc=jeff@garzik.org \
--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