Re: [PATCH 2/2] sunrpc: socket buffer size module parameter

[bpm@sgi.com]

Hey Chuck,

On Mon, Feb 22, 2010 at 05:33:10PM -0800, Chuck Lever wrote:
> On 02/22/2010 01:54 PM, Ben Myers wrote:
>> +int            tcp_rcvbuf_nrpc = 6;
>
> Just curious, is this '6' a typo?

Not a typo.  The original setting for tcp receive buffer was hardcoded
at

3 (in svc_tcp_init and svc_tcp_recv_record)
	*
sv_max_mesg
	*
2 (in svc_sock_setbufsize)
	
That's where I came up with the 6 for the tcp recv buffer.  The setting
hasn't changed.



The UDP send/recv buffer settings and TCP send buffer settings were
going to be

( 4 (default number of kernel threads on sles11)
	+
3 (as in svc_udp_recvfrom, etc) )
	*
sv_max_mesg
	* 
2 (in svc_sock_setbufsize)

but 14 wasn't a very round number, so I went with 16 which also happened
to match the slot_table_entries default.

> Perhaps it would be nice to have a  
> single macro defined as the default value for all of these.
>
> Do we have a high degree of confidence that these new default settings  
> will not adversely affect workloads that already perform well?

This patch has been in several releases of SGI's nfsd respin and I've
heard nothing to suggest there is an issue.  I didn't spend much time
taking measurements on UDP and didn't keep my TCP measurements.  If you
feel measurements are essential I'll be happy to provide a few, but
won't be able to get around to it for a little while.

Thanks,
	Ben