On Wed, Nov 06, 2013 at 12:57:38PM +0530, Shyam Kaushik wrote: > Hi Bruce, > > This hack works great. All 32 of configured NFSD threads end up doing > nfsd_write() which is great & I get higher IOPs/bandwidth from NFS > client side. > > What do you think if we vary the signature of > typedef __be32(*nfsd4_dec)(struct nfsd4_compoundargs *argp, void *); > > to include an additional return argument of the size estimate. This > way we get size estimate from the decoders (like nfsd4_decode_read > could return this estimate as rd_length + overhead) & in the worst > case if decoder says cant estimate (like a special return code -1) we > dont do svc_reserve() & leave it like it is. So when we run through > the compound we have a sum of size estimate & just do svc_reserve() at > the end of nfsd4_decode_compound() like your hack has. > > Does this sound reasonable to you? If not, perhaps can we just use the > hack for now & worry about readdir etc when they support >4K buffer? Yep. Actually looking at it again I think it needs a couple more special cases (for readlink, readdir), but that should be good enough for now. For the future.... I'd rather not add an extra argument to every encoder but maybe that is the simplest thing to do. --b. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
