From: Ric Wheeler <ric@emc.com>
To: Tejun Heo <htejun@gmail.com>
Cc: Jens Axboe <axboe@kernel.dk>, Alan Cox <alan@lxorguk.ukuu.org.uk>,
Jeff Garzik <jgarzik@pobox.com>,
linux-ide@vger.kernel.org
Subject: Re: [PATCH 2/2] libata: turn off NCQ if queue depth is adjusted to 1
Date: Sat, 30 Sep 2006 20:52:24 -0400 [thread overview]
Message-ID: <451F1148.5050604@emc.com> (raw)
In-Reply-To: <451F0BF7.1000200@gmail.com>
Tejun Heo wrote:
> Ric Wheeler wrote:
>
>
>> We have to be careful with the blacklisting - specifically, drive
>> model is often less critical than the version of the firmware (which
>> gets updated as people work drives through qualification, etc).
>> Updating drive firmware for end users is pretty rare (and almost all
>> tools are still DOS based ;-)).
>>
>> Certain drives should default to non-NCQ (based on model), but we
>> should be able to enable it if the firmware supports it.
>>
>> Most of the newest drives are fine, but we will still need something
>> like Tejun's fix to be able to turn it off for the odd cases that
>> show up in certain odd applications, etc.
>
>
> Ric, can you press harddisk vendors hard enough such that those model
> and revision numbers squeeze out of them?
>
> Thanks.
I will certainly push to get drive vendors to work with us on this. It
is in everyone's interest to have the drives avoid stressing features
that are not stable ;-)
Even better, we can usually try to work the bugs out with vendors before
the drives become public which should help minimize the issues.
next prev parent reply other threads:[~2006-10-01 0:58 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-30 10:44 [PATCH 1/2] libata: cosmetic changes to constants Tejun Heo
2006-09-30 10:45 ` [PATCH 2/2] libata: turn off NCQ if queue depth is adjusted to 1 Tejun Heo
2006-09-30 18:04 ` Jens Axboe
2006-09-30 20:51 ` Alan Cox
2006-09-30 20:26 ` Jens Axboe
2006-10-01 0:17 ` Ric Wheeler
2006-10-01 0:29 ` Tejun Heo
2006-10-01 0:52 ` Ric Wheeler [this message]
2006-10-01 12:56 ` Ric Wheeler
2006-12-16 17:19 ` Jeff Garzik
2006-12-18 10:04 ` Jens Axboe
2006-10-01 19:55 ` Jens Axboe
2006-10-04 13:37 ` saeed bishara
2006-10-04 13:37 ` Jens Axboe
2006-10-05 9:08 ` Jens Axboe
2006-09-30 11:39 ` [PATCH 1/2] libata: cosmetic changes to constants Jeff Garzik
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=451F1148.5050604@emc.com \
--to=ric@emc.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=axboe@kernel.dk \
--cc=htejun@gmail.com \
--cc=jgarzik@pobox.com \
--cc=linux-ide@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.