linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Bernd Schubert <bernd.schubert@fastmail.fm>
To: Christoph Hellwig <hch@infradead.org>,
	Robert Hancock <hancockrwd@gmail.com>,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
	linux-scsi@vger.kernel.org, Jeff Garzik <jeff@garzik.org>,
	Zheng Liu <wenqing.lz@taobao.com>
Subject: Re: [RFC][PATCH] libata: enable SATA disk fua detection on default
Date: Wed, 09 May 2012 15:20:56 +0200	[thread overview]
Message-ID: <4FAA6F38.6080801@fastmail.fm> (raw)
In-Reply-To: <20120509124804.GA13329@gmail.com>

On 05/09/2012 02:48 PM, Zheng Liu wrote:
> On Wed, May 09, 2012 at 07:12:43AM -0400, Christoph Hellwig wrote:
>> On Wed, May 09, 2012 at 05:30:16PM +0800, Zheng Liu wrote:
>>> IMHO, FUA provides a solution that we can get better performance in write
>>> cache mode when we do many flush operations.  If I set the disk to write
>>> through mode, I won't get this benefit.  Am I missing something?
>>
>> If you set the disk to write through mode you never have to flush the
>> cache.  So as soonas your number of flushes gets close to the number of
>> writes it tends to be a clear win - for ATA the tradeoff is even more in
>> favour of write through because the flush command can't be queued yetin
>> commonly available standards versions.  So if you have a workload that
>> basically needs to flush out every write you win - if you have workloads
>> where you have a lot more writes than cache flushes write back mode
>> wins.
>
> Thanks for your explanation.  It seems that there still has a problem.
> If I set the disk to write through mode, I need to modify my application
> to remove all of flush/sync operations.  It is unacceptable for us.

Why? You still need to flush the linux page cache. Just the disk cache 
does not need to be flushed anymore. So changing the application would 
be the wrong thing to do.


Cheers,
Bernd

  reply	other threads:[~2012-05-09 13:20 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 [this message]
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
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=4FAA6F38.6080801@fastmail.fm \
    --to=bernd.schubert@fastmail.fm \
    --cc=hancockrwd@gmail.com \
    --cc=hch@infradead.org \
    --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).