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
next prev parent 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.