From: Chris Mason <clm@fb.com>
To: Morten Stevens <mstevens@fedoraproject.org>,
<linux-btrfs@vger.kernel.org>
Subject: Re: btrfs kernel workqueues performance regression
Date: Tue, 15 Jul 2014 13:39:11 -0400 [thread overview]
Message-ID: <53C5673F.6010702@fb.com> (raw)
In-Reply-To: <CAKSJeFKteZNUnJmN3dnYaHLnayQhZNNTKX5co_smMqwy1JgQNA@mail.gmail.com>
On 07/15/2014 11:26 AM, Morten Stevens wrote:
> Hi,
>
> I see that btrfs is using kernel workqueues since linux 3.15. After
> some tests I noticed performance regressions with fs_mark.
>
> mount options: rw,relatime,compress=lzo,space_cache
>
> fs_mark on Kernel 3.14.9:
>
> # fs_mark -d /mnt/btrfs/fsmark -D 512 -t 16 -n 4096 -s 51200 -L5 -S0
> FSUse% Count Size Files/sec App Overhead
> 1 65536 51200 17731.4 723894
> 1 131072 51200 16832.6 685444
> 1 196608 51200 19604.5 652294
> 1 262144 51200 18663.6 630067
> 1 327680 51200 20112.2 692769
>
> The results are really nice! compress=lzo performs very good.
>
> fs_mark after upgrading to Kernel 3.15.4:
>
> # fs_mark -d /mnt/btrfs/fsmark -D 512 -t 16 -n 4096 -s 51200 -L5 -S0
> FSUse% Count Size Files/sec App Overhead
> 0 65536 51200 10718.1 749540
> 0 131072 51200 8601.2 853050
> 0 196608 51200 11623.2 558546
> 0 262144 51200 11534.2 536342
> 0 327680 51200 11167.4 578562
>
> That's really a big performance regression :(
>
> What do you think? It's easy to reproduce with fs_mark.
I wasn't able to trigger regressions here when we first merged it, but I
was sure that something would pop up. fs_mark is sensitive to a few
different factors outside just the worker threads, so it could easily be
another change as well.
With 16 threads, the btree locking also has a huge impact, and we've
made change there too.
I'll reproduce here, thanks for sending it in.
-chris
next prev parent reply other threads:[~2014-07-15 17:43 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-15 15:26 btrfs kernel workqueues performance regression Morten Stevens
2014-07-15 17:39 ` Chris Mason [this message]
2014-07-22 23:39 ` Dave Chinner
2014-07-23 13:33 ` Chris Mason
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=53C5673F.6010702@fb.com \
--to=clm@fb.com \
--cc=linux-btrfs@vger.kernel.org \
--cc=mstevens@fedoraproject.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.