From: Michael Tokarev <mjt@tls.msk.ru>
To: Jeff Garzik <jeff@garzik.org>,
linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
linux-scsi@vger.kernel.org, Zheng Liu <wenqing.lz@taobao.com>
Subject: Re: [RFC][PATCH] libata: enable SATA disk fua detection on default
Date: Wed, 04 Jul 2012 10:36:40 +0400 [thread overview]
Message-ID: <4FF3E478.50609@msgid.tls.msk.ru> (raw)
In-Reply-To: <20120704024715.GA17095@gmail.com>
On 04.07.2012 06:47, Zheng Liu wrote:
[]
>> I guess it can be enabled after LOTS of testing of various drives out
>> there, which shows that no current drives have issues with FUA anymore,
>> and after building a black-list of devices which misbehave. It is quite
>> a big project, I think. But what it gives us in return?
>>
>> (I've no idea if this is the right answer, speaking of myself only)
>
> Thanks for the reply. Indeed it is quite a big project but we enable
> FUA feature for SAS disk. Is there any differences?
Yes, there's a very big difference with SAS disks. Even in parallel SCSI
world DPO/FUA has been enabled since the day it has been implemented IIRC,
because, apparently, hardware raid controllers enabled it too. In other
words, it has been tested and proved to be working even before linux
implementation. When first SAS disks started appearing, DPO/FUA were
enabled for them in linux already -- at that time any breakage were
solely due to the particular disk model, and were easy to blacklist,
if necessary, since only a few disk models were in production.
With SATA disks, initial hardware implementation proved to be more
non-functional than functional, ie, initially there were more drives
with non-working FUA. I have a few not-so-old SATA drives here which
behaves strangely when FUA is enabled (I don't remember exact details,
but I had to disable FA again after I tried to enable it once, the
system started behaving not as good as before). So, for SATA drives,
we've exactly the opposite picture: we've some proof that "generally,
drives dislikes FUA", and now when fua has been disabled for a lot
of drives and users, turning it on by default needs lots of testing.
But I ask again: what is the benefit of turning FUA on to start with?
Thanks,
/mjt
next prev parent reply other threads:[~2012-07-04 6:36 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-08 3:24 [RFC][PATCH] libata: enable SATA disk fua detection on default Zheng Liu
2012-05-09 5:38 ` Robert Hancock
2012-05-09 6:19 ` Zheng Liu
2012-05-09 8:25 ` Christoph Hellwig
2012-05-09 9:30 ` Zheng Liu
2012-05-09 11:12 ` Christoph Hellwig
2012-05-09 12:48 ` Zheng Liu
2012-05-09 13:20 ` Bernd Schubert
2012-05-09 13:23 ` Christoph Hellwig
2012-07-03 9:04 ` Zheng Liu
2012-07-03 20:11 ` Michael Tokarev
2012-07-04 2:47 ` Zheng Liu
2012-07-04 6:36 ` Michael Tokarev [this message]
2012-07-04 7:06 ` Zheng Liu
2012-09-09 20:34 ` Arvydas Sidorenko
2012-09-09 20:39 ` Jeff Garzik
2012-08-17 18:06 ` Enabling FUA for SATA drives (was Re: [RFC][PATCH] libata: enable SATA disk fua detection on default) 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=4FF3E478.50609@msgid.tls.msk.ru \
--to=mjt@tls.msk.ru \
--cc=jeff@garzik.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=wenqing.lz@taobao.com \
/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).