From: "J. Bruce Fields" <bfields@fieldses.org>
To: Andy Adamson <andros@netapp.com>
Cc: Alexandros Batsakis <batsakis@netapp.com>,
"Adamson, Andy" <William.Adamson@netapp.com>,
linux-nfs@vger.kernel.org, pnfs@linux-nfs.org
Subject: Re: [pnfs] [PATCH 4/5] nfsd41: set the session maximum response sizecached
Date: Mon, 6 Jul 2009 17:48:24 -0400 [thread overview]
Message-ID: <20090706214824.GS1601@fieldses.org> (raw)
In-Reply-To: <4F969509-933A-45B1-B365-CB70F812A120@netapp.com>
On Mon, Jul 06, 2009 at 03:27:43PM -0400, Andy Adamson wrote:
> Here is what I suggest.
>
> 1) max response size should be the DRC buffer slot size (currently 512) +
> 44 + 32. The RPC GSS verifier will still not be accounted for, but the
> rest of the known sizes are.
Sounds good.
> 2) Perhaps bump the 512 to 512+64. The current code shows we are
> cutting it a bit close, we want to ensure that nfsd can service normal
> operation responses with several GETATTR's with a lot of attributes.
576 sounds a little close too. 768? 1k?
--b.
next prev parent reply other threads:[~2009-07-06 21:48 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-24 19:37 [PATCH 0/14] NFSv4.1 Server DRC rewrite Version 4 andros
2009-06-24 19:37 ` [PATCH 1/5] nfsd41: use globals for DRC limits andros
2009-06-24 19:37 ` [PATCH 2/5] nfsd41: change from page to memory based drc limits andros
2009-06-24 19:37 ` [PATCH 3/5] nfsd41: reclaim DRC memory on session free andros
2009-06-24 19:37 ` [PATCH 4/5] nfsd41: set the session maximum response size cached andros
2009-06-24 19:37 ` [PATCH 5/5] nfsd41: remove redundant forechannel max requests check andros
2009-06-24 19:37 ` [PATCH 06/14] nfsd41: change check_slot_seqid parameters andros
2009-06-24 19:37 ` [PATCH 07/14] nfsd41: replace create_session DRC with xdr structure andros
2009-06-24 19:37 ` [PATCH 08/14] nfsd41: replace page based DRC with buffer based DRC andros
2009-06-24 19:37 ` [PATCH 09/14] nfsd41: rename nfsd4_enc_uncached_replay andros
2009-06-24 19:37 ` [PATCH 10/14] nfsd41: encode replay sequence from the slot values andros
2009-06-24 19:37 ` [PATCH 11/14] nfsd41: fix nfsd4_replay_cache_entry comments andros
2009-06-24 19:37 ` [PATCH 12/14] nfsd41: support 16 slots per session andros
2009-06-24 19:37 ` [PATCH 13/14] nfsd41: add test for failed sequence operation andros
2009-06-24 19:37 ` [PATCH 14/14] nfsd41: remove redundant failed sequence check andros
2009-07-01 23:09 ` [pnfs] [PATCH 4/5] nfsd41: set the session maximum response sizecached Alexandros Batsakis
2009-07-06 19:27 ` Andy Adamson
2009-07-06 21:48 ` J. Bruce Fields [this message]
2009-07-16 14:18 ` [pnfs] [PATCH 3/5] nfsd41: reclaim DRC memory on session free J. Bruce Fields
2009-07-16 15:49 ` William A. (Andy) Adamson
2009-07-27 22:57 ` J. Bruce Fields
2009-07-31 8:38 ` Andy Adamson
2009-07-16 14:17 ` [pnfs] [PATCH 2/5] nfsd41: change from page to memory based drc limits J. Bruce Fields
2009-07-16 18:50 ` J. Bruce Fields
2009-07-16 14:24 ` J. Bruce Fields
2009-06-28 16:01 ` [pnfs] [PATCH 0/14] NFSv4.1 Server DRC rewrite Version 4 Benny Halevy
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=20090706214824.GS1601@fieldses.org \
--to=bfields@fieldses.org \
--cc=William.Adamson@netapp.com \
--cc=andros@netapp.com \
--cc=batsakis@netapp.com \
--cc=linux-nfs@vger.kernel.org \
--cc=pnfs@linux-nfs.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