From: Tejun Heo <htejun@gmail.com>
To: Robert Hancock <hancockr@shaw.ca>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
linux-ide@vger.kernel.org, edmudama@gmail.com,
Nicolas.Mailhot@LaPoste.net
Subject: Re: libata FUA revisited
Date: Mon, 12 Feb 2007 00:31:07 -0800 [thread overview]
Message-ID: <45D025CB.6070205@gmail.com> (raw)
In-Reply-To: <45CFDE27.5030607@shaw.ca>
Hello,
Robert Hancock wrote:
[--correct summary snipped--]
> Given the above, what I'm proposing to do is:
>
> -Remove the blacklisting of Maxtor BANC1G10 firmware for FUA. If we need
> to FUA-blacklist any drives this should likely be added to the existing
> "horkage" mechanism we now have. However, at this point I don't think
> that's needed, considering that I've seen no conclusive evidence that
> any drive has ever been established to have broken FUA.
Agreed.
> -Add a new port flag ATA_FLAG_NO_FUA to indicate that a controller can't
> handle FUA commands, and add that flag to sata_sil. Force FUA off on any
> drive connected to a controller with this bit set.
>
> There was some talk that sata_mv might have this problem, but I believe
> the conclusion was that it didn't. The only controllers that would are
> ones that actually try to interpret the ATA command codes and don't know
> about WRITE DMA FUA.
I think it would be better to add ATA_FLAG_FUA instead of ATA_FLAG_NO_FUA.
> -Change the fua module option to control FUA enable/disable to have a
> third value, "enable for NCQ-supporting drives only", which would become
> the new default. That case seems less likely to cause problems since FUA
> on NCQ is just another bit in the command whereas FUA on non-NCQ is an
> entirely different, potentially unsupported command.
Not enabling FUA doesn't result in huge performance hit. I'm not sure
whether we should go such far. Just supporting FUA on known good
controllers sounds good enough to me.
> Any comments?
>
> As an aside, I came across a comment that the Silicon Image Windows
> drivers for NCQ-supporting controllers have some blacklist entries for
> drives with broken NCQ in their .inf files. We only seem to have one in
> the libata NCQ blacklist, we may want to add some more of these. The
> ones in the current SiI3124 and 3132 drivers' .inf files for
> "DisableSataQueueing" appear to be:
>
> Model Firmware
> Maxtor 7B250S0 BANC1B70
> HTS541060G9SA00 MB3OC60D
> HTS541080G9SA00 MB4OC60D
> HTS541010G9SA00 MBZOC60D
>
> (the latter 3 being Hitachi Travelstar drives)
Yeah, I don't think we need to be too careful about blacklisting NCQ
considering the sorry state of many early NCQ firmwares. Please submit
a patch. It would be nice if you add a comment why the drives are added
for documentation.
Thanks.
--
tejun
next prev parent reply other threads:[~2007-02-12 8:31 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-02-12 3:25 libata FUA revisited Robert Hancock
2007-02-12 8:31 ` Tejun Heo [this message]
2007-02-16 18:14 ` Jeff Garzik
[not found] <fa.S80SRyQbD/hm4SxliPUKU88BaCo@ifi.uio.no>
2007-02-12 5:47 ` Robert Hancock
[not found] ` <fa.Q/csgyCHkAsD84yi+bN78H1WNNM@ifi.uio.no>
2007-02-13 0:23 ` Robert Hancock
2007-02-13 15:20 ` Tejun Heo
2007-02-14 0:07 ` Robert Hancock
2007-02-14 0:50 ` Tejun Heo
2007-02-15 18:00 ` Jens Axboe
2007-02-19 19:46 ` Robert Hancock
2007-02-21 8:37 ` Tejun Heo
2007-02-21 8:46 ` Jens Axboe
2007-02-21 8:57 ` Tejun Heo
2007-02-21 9:01 ` Jens Axboe
2007-02-22 22:44 ` Ric Wheeler
2007-02-22 22:40 ` Ric Wheeler
2007-02-21 14:06 ` Robert Hancock
2007-02-22 22:34 ` Ric Wheeler
2007-02-23 0:04 ` Robert Hancock
2007-02-21 8:44 ` Jens Axboe
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=45D025CB.6070205@gmail.com \
--to=htejun@gmail.com \
--cc=Nicolas.Mailhot@LaPoste.net \
--cc=edmudama@gmail.com \
--cc=hancockr@shaw.ca \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@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.