From: Anna Schumaker <Anna.Schumaker@netapp.com>
To: Christoph Hellwig <hch@infradead.org>
Cc: <Trond.Myklebust@primarydata.com>, <linux-nfs@vger.kernel.org>
Subject: Re: [PATCH v3 0/5] NFS: Add READ_PLUS support
Date: Tue, 17 Mar 2015 08:30:24 -0400 [thread overview]
Message-ID: <55081E60.9010200@Netapp.com> (raw)
In-Reply-To: <20150317103145.GA27478@infradead.org>
On 03/17/2015 06:31 AM, Christoph Hellwig wrote:
> On Mon, Mar 16, 2015 at 05:17:41PM -0400, Anna Schumaker wrote:
>> Test | v4.1 | no READ_PLUS | READ_PLUS
>> ---------------------------------------------------
>> generic/013 | 135s | 143s | 123s
>> generic/075 | 4s | 11s | 4s
>> generic/091 | 9s | 16s | 15s
>> generic/112 | 4s | 7s | 6s
>> generic/127 | 62s | 117s | 114s
>> generic/213 | [not run] | 1s | 1s
>> generic/214 | [not run] | 0s | 0s
>> generic/228 | [not run] | 1s | 2s
>> generic/236 | 1s | 1s | 1s
>> generic/263 | 4s | 6s | 8s
>> generic/285 | 0s | 1s | 3s
>> generic/315 | [not run] | 2s | 1s
>> ---------------------------------------------------
>> Total | 3:47.47 | 5:11.85 | 4:43.77
>
> So why is 4.2 generally slower than 4.1? I guess we really should look
> into that first.
>
I did! There are several tests that run on v4.1 but skip over fallocate() calls. v4.2 no longer skips these tests.
next prev parent reply other threads:[~2015-03-17 12:30 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-16 21:17 [PATCH v3 0/5] NFS: Add READ_PLUS support Anna Schumaker
2015-03-16 21:17 ` [PATCH v3 1/5] SUNRPC: Split out a function for setting current page Anna Schumaker
2015-03-16 21:17 ` [PATCH v3 2/5] SUNRPC: Add the ability to expand holes in data pages Anna Schumaker
2015-03-16 21:17 ` [PATCH v3 3/5] NFS: Add basic READ_PLUS support Anna Schumaker
2015-03-17 19:14 ` J. Bruce Fields
2015-03-16 21:17 ` [PATCH v3 4/5] SUNRPC: Add the ability to shift data to a specific offset Anna Schumaker
2015-03-16 21:17 ` [PATCH v3 5/5] NFS: Add support for decoding multiple segments Anna Schumaker
2015-03-17 10:31 ` [PATCH v3 0/5] NFS: Add READ_PLUS support Christoph Hellwig
2015-03-17 12:30 ` Anna Schumaker [this message]
2015-03-17 19:08 ` J. Bruce Fields
2015-03-17 19:16 ` Anna Schumaker
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=55081E60.9010200@Netapp.com \
--to=anna.schumaker@netapp.com \
--cc=Trond.Myklebust@primarydata.com \
--cc=hch@infradead.org \
--cc=linux-nfs@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).