Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-13 Thread Chuck Lever
On Jul 10, 2015, at 12:05 PM, J. Bruce Fields bfie...@fieldses.org wrote: On Fri, Jul 10, 2015 at 11:59:20AM -0400, Chuck Lever wrote: On Jul 10, 2015, at 11:54 AM, J. Bruce Fields bfie...@fieldses.org wrote: On Fri, Jul 10, 2015 at 10:59:48AM -0400, Chuck Lever wrote: On Jul 10, 2015,

Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-10 Thread J. Bruce Fields
On Fri, Jul 10, 2015 at 10:59:48AM -0400, Chuck Lever wrote: On Jul 10, 2015, at 10:18 AM, bfie...@fieldses.org wrote: On Thu, Jul 09, 2015 at 04:45:47PM -0400, Chuck Lever wrote: Increased to 1 megabyte. Why not more or less? Why do we even have this constant, why shouldn't we

Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-10 Thread Chuck Lever
On Jul 10, 2015, at 11:54 AM, J. Bruce Fields bfie...@fieldses.org wrote: On Fri, Jul 10, 2015 at 10:59:48AM -0400, Chuck Lever wrote: On Jul 10, 2015, at 10:18 AM, bfie...@fieldses.org wrote: On Thu, Jul 09, 2015 at 04:45:47PM -0400, Chuck Lever wrote: Increased to 1 megabyte. Why

Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-10 Thread J. Bruce Fields
On Fri, Jul 10, 2015 at 10:18:14AM -0400, J. Bruce Fields wrote: On Thu, Jul 09, 2015 at 04:45:47PM -0400, Chuck Lever wrote: Increased to 1 megabyte. Why not more or less? Why do we even have this constant, why shouldn't we just use RPCSVC_MAXPAYLOAD? (That one question aside these

Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-10 Thread J. Bruce Fields
On Thu, Jul 09, 2015 at 04:45:47PM -0400, Chuck Lever wrote: Increased to 1 megabyte. Why not more or less? Why do we even have this constant, why shouldn't we just use RPCSVC_MAXPAYLOAD? --b. Signed-off-by: Chuck Lever chuck.le...@oracle.com --- include/linux/sunrpc/svc_rdma.h |2

Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-10 Thread J. Bruce Fields
On Fri, Jul 10, 2015 at 11:59:20AM -0400, Chuck Lever wrote: On Jul 10, 2015, at 11:54 AM, J. Bruce Fields bfie...@fieldses.org wrote: On Fri, Jul 10, 2015 at 10:59:48AM -0400, Chuck Lever wrote: On Jul 10, 2015, at 10:18 AM, bfie...@fieldses.org wrote: On Thu, Jul 09, 2015 at

Re: [PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-10 Thread Chuck Lever
On Jul 10, 2015, at 10:18 AM, bfie...@fieldses.org wrote: On Thu, Jul 09, 2015 at 04:45:47PM -0400, Chuck Lever wrote: Increased to 1 megabyte. Why not more or less? Why do we even have this constant, why shouldn't we just use RPCSVC_MAXPAYLOAD? The payload size maximum for RDMA is

[PATCH v1 5/5] svcrdma: Boost NFS READ/WRITE payload size maximum

2015-07-09 Thread Chuck Lever
Increased to 1 megabyte. Signed-off-by: Chuck Lever chuck.le...@oracle.com --- include/linux/sunrpc/svc_rdma.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/linux/sunrpc/svc_rdma.h b/include/linux/sunrpc/svc_rdma.h index 13af61b..1bca6dd 100644 ---