public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
From: Christoph Hellwig <hch@infradead.org>
To: Markus Trippelsdorf <markus@trippelsdorf.de>
Cc: xfs@oss.sgi.com
Subject: Re: xfs performance problem
Date: Mon, 2 May 2011 06:14:01 -0400	[thread overview]
Message-ID: <20110502101401.GA9155@infradead.org> (raw)
In-Reply-To: <20110501182442.GA1635@x4.trippels.de>

On Sun, May 01, 2011 at 08:24:42PM +0200, Markus Trippelsdorf wrote:
> Another thing is that with the recent updates to block FLUSH handling,
> using FUA might even be less efficient.  The new implementation
> aggressively merges those commit writes and flushes.  IOW, depending
> on timing, multiple consecutive commit writes can be merged as,
> 
>  FLUSH + commit writes + FLUSH
> 
> or
> 
>  FLUSH + some commit writes + FLUSH + other commit writes + FLUSH
> 
> and so on,

Except that writing multiple log buffers right next to each other
is rather unusual - you'd have to have a burst of metadata only
operations to get there.  What's more common is that a log write
interrupts streams of actual data I/O, and the longer we drain
the queue the more performance impact it has.

Moreover I'm working on avoiding the pre-flush if it's not needed,
e.g. there were no appending writes, and there as no pushing of the
log tail required, in which case the log write will only be
a write with FUA set, with no FLUSH and thus no queue draining on
SATA at all.

Also when you move away from SATA to higher latency links like FC
or iSCSI (maybe even over a WAN) avoiding protocol roundtrips buys
you a lot of performance.

_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs

  reply	other threads:[~2011-05-02 10:10 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-26 19:44 xfs performance problem Benjamin Schindler
2011-04-26 22:12 ` Stan Hoeppner
2011-04-26 23:23   ` Benjamin Schindler
2011-04-26 23:59   ` Benjamin Schindler
2011-04-29 15:00     ` Peter Grandi
2011-04-30 20:36       ` Michael Monnerie
2011-05-01  8:49       ` Dave Chinner
2011-05-01 14:38         ` Peter Grandi
2011-05-01 15:08           ` Peter Grandi
2011-05-01 15:32           ` Michael Monnerie
2011-05-01 17:04             ` Peter Grandi
2011-05-02  2:50           ` Dave Chinner
2011-05-02 20:10             ` Emmanuel Florac
2011-05-01 13:33     ` Peter Grandi
2011-05-01 16:32     ` Peter Grandi
2011-04-27  7:55   ` Michael Weissenbacher
2011-04-27  8:09     ` Benjamin Schindler
2011-04-27  2:35 ` Dave Chinner
2011-04-29 16:27   ` Martin Steigerwald
2011-05-01  8:52     ` Dave Chinner
2011-05-01 16:55       ` Christoph Hellwig
2011-05-01 18:24         ` Markus Trippelsdorf
2011-05-02 10:14           ` Christoph Hellwig [this message]
  -- strict thread matches above, loose matches on Subject: below --
2011-04-29 16:28 Martin Steigerwald
2011-04-29 19:51 ` Peter Grandi
2011-05-01 16:56 ` Benjamin Schindler

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=20110502101401.GA9155@infradead.org \
    --to=hch@infradead.org \
    --cc=markus@trippelsdorf.de \
    --cc=xfs@oss.sgi.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