All of lore.kernel.org
 help / color / mirror / Atom feed
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



  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.