linux-nfs.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "J. Bruce Fields" <bfields@fieldses.org>
To: James Vanns <james.vanns@framestore.com>
Cc: Linux NFS Mailing List <linux-nfs@vger.kernel.org>
Subject: Re: Where in the server code is fsinfo rtpref calculated?
Date: Wed, 15 May 2013 13:42:45 -0400	[thread overview]
Message-ID: <20130515174245.GN16811@fieldses.org> (raw)
In-Reply-To: <1706515764.20038371.1368635535864.JavaMail.root@framestore.com>

On Wed, May 15, 2013 at 05:32:15PM +0100, James Vanns wrote:
> <snip>
> 
> > > I've just returned from nfsd3_proc_fsinfo() and found what I would
> > > consider an odd decision - perhaps nothing better was suggested at
> > > the time. It seems to me that in response to an FSINFO call the
> > > reply stuffs the max_block_size value in  both the maximum *and*
> > > preferred block sizes for both read and write. A 1MB block size
> > > for a preferred default is a little high! If a disk is reading at
> > > 33MB/s and we have just a single server running 64 knfsd and each
> > > READ call is requesting 1MB of data then all of a sudden we have
> > > an aggregate read speed of ~512k/s
> > 
> > I lost you here.
> 
> OK, so what we're seeing is the large majority of our nr. ~700 clients
> (all Linux 2.6.32 based NFS clients) issuing READ requests of 1MB in
> size.

Knowing nothing about your situation, I'd assume the clients are doing
that because they actually want that 1MB of data.

Would you prefer they each send 1024 1k READs?  I don't understand why
it's the read size you're focused on here.

--b.

> 
> After the initial MOUNT request has been granted an FSINFO call is
> made. The contents of the REPLY from the server (another Linux 2.6.32
> server) include rtmax, rtpref, wtmax and wtpref all of which are set
> to 1MB. This 1MB appears to come from that code/explanation I
> described earlier  - all values are basically getting set to whatever
> comes out of nfsd_get_default_max_blksize().



  reply	other threads:[~2013-05-15 17:42 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-14 11:17 Where in the server code is fsinfo rtpref calculated? James Vanns
2013-05-14 22:01 ` J. Bruce Fields
2013-05-15  9:21   ` James Vanns
2013-05-15 13:42   ` James Vanns
2013-05-15 14:15     ` J. Bruce Fields
2013-05-15 14:34       ` James Vanns
2013-05-15 14:47         ` J. Bruce Fields
2013-05-15 15:20           ` Myklebust, Trond
2013-05-15 16:32           ` James Vanns
2013-05-15 17:42             ` J. Bruce Fields [this message]
2013-05-17 11:43               ` James Vanns
2013-05-17 13:56                 ` J. Bruce Fields

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=20130515174245.GN16811@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=james.vanns@framestore.com \
    --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).