All of lore.kernel.org
 help / color / mirror / Atom feed
From: Willy Tarreau <w@1wt.eu>
To: Eduardo Valentin <eduval@amazon.com>
Cc: Greg KH <gregkh@linuxfoundation.org>,
	stable@vger.kernel.org, shli@fb.com, neilb@suse.com,
	colyli@suse.de
Subject: Re: [stable v4.9.y] Backports to fix fstrim time / CPU load on raid0
Date: Fri, 22 Sep 2017 20:11:22 +0200	[thread overview]
Message-ID: <20170922181122.GA13515@1wt.eu> (raw)
In-Reply-To: <20170922174048.GB11575@u40b0340c692b58f6553c.ant.amazon.com>

On Fri, Sep 22, 2017 at 10:40:48AM -0700, Eduardo Valentin wrote:
> On Fri, Sep 22, 2017 at 07:31:45PM +0200, Greg KH wrote:
> > On Fri, Sep 22, 2017 at 10:16:06AM -0700, Eduardo Valentin wrote:
> > > Hello GregKH,
> > > 
> > > I have been seeing several reports of performance issue with raid0 while performing fstrim on v4.9.y.
> > > Currently, if one performs:
> > > 
> > > # fio --name fio_test_file --direct=1 --rw=randwrite --bs=4k --size=5G --numjobs=8 --group_reporting --directory=/mount/raid0 
> > > # rm -rf /media/nvme-raid0 
> > > # time fstrim -vvv -a 
> > > real	3m41.102s 
> > > user	0m0.000s 
> > > sys	0m4.964s 
> > 
> > Also, is this a regression from older kernels?
> 
> I personally did not try older than 4.9 kernels. but looking at git history,
> and the commit message of the fix, looks like this is long known issue that
> got fixed on v4.12.

Or it may simply be something that couldn't be achieved without significantly
improving the underlying infrastructure using the patches you've spotted (and
possibly a few others that you didn't notice but are required for stability
or to avoid breaking other subsystems).

While I use 4.9 on many of my machines, I'd rather favor maximal stability
over an optimization for some operations that don't appear *that* often.
That said if the relevant maintainers consider it safe to backport, I'll
certainly welcome some performance improvements on my machines, but that's
not what I'm primarily looking for in stable kernels.

And if we start to backport performance improvements into LTS kernels,
what will encourage users to upgrade to the next LTS ?

Cheers,
Willy

  reply	other threads:[~2017-09-22 18:11 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-09-22 17:16 [stable v4.9.y] Backports to fix fstrim time / CPU load on raid0 Eduardo Valentin
2017-09-22 17:31 ` Greg KH
2017-09-22 17:39   ` Eduardo Valentin
2017-09-22 17:31 ` Greg KH
2017-09-22 17:40   ` Eduardo Valentin
2017-09-22 18:11     ` Willy Tarreau [this message]
2017-09-22 18:38       ` Eduardo Valentin
2017-09-22 19:05         ` Greg KH

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=20170922181122.GA13515@1wt.eu \
    --to=w@1wt.eu \
    --cc=colyli@suse.de \
    --cc=eduval@amazon.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=neilb@suse.com \
    --cc=shli@fb.com \
    --cc=stable@vger.kernel.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.