From: Linus Torvalds <torvalds@linux-foundation.org>
To: Theodore Tso <tytso@mit.edu>
Cc: Jens Axboe <jens.axboe@oracle.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 0/8][RFC] IO latency/throughput fixes
Date: Mon, 6 Apr 2009 13:10:50 -0700 (PDT) [thread overview]
Message-ID: <alpine.LFD.2.00.0904061300310.6058@localhost.localdomain> (raw)
In-Reply-To: <alpine.LFD.2.00.0904061255160.4134@localhost.localdomain>
On Mon, 6 Apr 2009, Linus Torvalds wrote:
>
> Yes. I want to go back and see how much this all helps for the
> data=ordered case (if at all), but I think we should at least consider
> trying 'data=writeback' as a default.
Hmm.
So I'm back to "data=ordered", and quite frankly, so far it's a horrible
experience.
The difference is rather startlingly huge, which it was _not_ before.
Sure, data=writeback was better before too, but in the end, the difference
between the occasional 5+ second pause and the occasional 10+ second pause
wasn't really all that interesting. They were both unusuable, and both
made me kill the background writer almost immediately.
This time I really forced myself to keep it around, just to get the whole
horridness of the experience.
Here's wahat I just got from data=writeback and all the latest patches
(which includes one extra experimental one from Jens to further improve on
things):
fsync(6) = 0 <0.839397>
fsync(6) = 0 <0.879706>
fsync(6) = 0 <0.240856>
fsync(6) = 0 <0.006497>
fsync(6) = 0 <0.207197>
fsync(4) = 0 <0.318396>
fsync(4) = 0 <0.044499>
fsync(4) = 0 <0.088381>
fsync(6) = 0 <0.001057>
fsync(9) = 0 <0.245389>
fsync(7) = 0 <0.042728>
fsync(7) = 0 <1.148262>
fsync(6) = 0 <0.200707>
fsync(6) = 0 <0.045259>
fsync(6) = 0 <0.111846>
that's all quite usable. Stuttering, yes, but never anything feeling rally
bad, or like something died.
This is with data=ordered on the same kernel:
fsync(5) = 0 <0.000949>
fsync(8) = 0 <19.674681>
fsync(8) = 0 <1.169357>
fsync(8) = 0 <17.994631>
fsync(8) = 0 <5.711143>
fsync(8) = 0 <4.759515>
fsync(4) = 0 <17.283171>
fsync(4) = 0 <4.371930>
fsync(4) = 0 <0.008327>
fsync(8) = 0 <0.000636>
fsync(11) = 0 <9.802027>
fsync(9) = 0 <14.109262>
fsync(9) = 0 <4.081919>
fsync(8) = 0 <0.000490>
fsync(8) = 0 <0.001588>
fsync(11) = 0 <18.822826>
fsync(9) = 0 <3.917717>
fsync(9) = 0 <0.001035>
fsync(8) = 0 <0.000138>
fsync(9) = 0 <17.466156>
where basically each 'sync' on the big file triggers that 15-20s total
failure. I do wonder if it's actually gotten _worse_ for the data=ordered
case, but at the same time it does make a pretty compelling argument for
switching the defaults around.
Linus
next prev parent reply other threads:[~2009-04-06 20:13 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-06 12:48 Jens Axboe
2009-04-06 12:48 ` [PATCH 1/8] block: change the request allocation/congestion logic to be sync/async based Jens Axboe
2009-04-06 12:48 ` [PATCH 2/8] Add WRITE_SYNC_PLUG and SWRITE_SYNC_PLUG Jens Axboe
2009-04-06 12:48 ` [PATCH 3/8] block: fsync_buffers_list() should use SWRITE_SYNC_PLUG Jens Axboe
2009-04-06 12:48 ` [PATCH 4/8] jbd: use WRITE_SYNC_PLUG instead of WRITE_SYNC Jens Axboe
2009-04-06 12:48 ` [PATCH 5/8] jbd2: " Jens Axboe
2009-04-06 12:48 ` [PATCH 6/8] block: enabling plugging on SSD devices that don't do queuing Jens Axboe
2009-04-06 12:48 ` [PATCH 7/8] block: Add flag for telling the IO schedulers NOT to anticipate more IO Jens Axboe
2009-04-06 12:48 ` [PATCH 8/8] block: switch sync_dirty_buffer() over to WRITE_SYNC Jens Axboe
2009-04-06 13:04 ` [PATCH 0/8][RFC] IO latency/throughput fixes Jens Axboe
2009-04-06 13:13 ` Jens Axboe
2009-04-06 15:37 ` Linus Torvalds
2009-04-06 16:57 ` Jens Axboe
2009-04-07 3:28 ` Chris Mason
2009-04-06 15:04 ` Linus Torvalds
2009-04-06 15:10 ` Jens Axboe
2009-04-06 15:45 ` Linus Torvalds
2009-04-06 17:01 ` Jens Axboe
2009-04-06 18:31 ` Theodore Tso
2009-04-06 19:57 ` Linus Torvalds
2009-04-06 20:10 ` Linus Torvalds [this message]
2009-04-06 21:26 ` Theodore Tso
2009-04-06 20:12 ` Hua Zhong
2009-04-06 20:20 ` Linus Torvalds
2009-04-06 21:19 ` Theodore Tso
2009-04-06 21:35 ` Hua Zhong
2009-04-06 22:04 ` Ray Lee
2009-04-06 22:17 ` Linus Torvalds
2009-04-06 23:10 ` Linus Torvalds
2009-04-07 7:51 ` Geert Uytterhoeven
2009-04-07 10:36 ` Ingo Molnar
2009-04-07 14:10 ` Diego Calleja
2009-04-08 12:04 ` Ingo Molnar
2009-04-08 12:56 ` Denys Vlasenko
2009-04-08 13:27 ` Ingo Molnar
2009-04-07 13:35 ` Mark Lord
2009-04-07 14:33 ` Linus Torvalds
2009-04-07 19:24 ` Mark Lord
2009-04-07 19:45 ` Jeff Garzik
2009-04-07 20:53 ` Mike Galbraith
2009-04-09 2:40 ` Eric Sandeen
2009-04-09 14:01 ` Ric Wheeler
2009-04-06 22:25 ` Hua Zhong
2009-04-06 22:48 ` Ray Lee
2009-04-06 22:52 ` Hua Zhong
2009-04-06 23:19 ` Alan Cox
2009-04-07 3:52 ` Chris Mason
2009-04-07 4:13 ` Trenton D. Adams
2009-04-07 4:27 ` Linus Torvalds
2009-04-07 4:48 ` Trenton D. Adams
2009-04-07 5:02 ` Linus Torvalds
2009-04-07 5:23 ` Hua Zhong
2009-04-07 6:27 ` Trenton D. Adams
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=alpine.LFD.2.00.0904061300310.6058@localhost.localdomain \
--to=torvalds@linux-foundation.org \
--cc=jens.axboe@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=tytso@mit.edu \
/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
all inboxes | Powered by JetHome®