Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Håkon Bugge
> On 13 Jun 2019, at 22:25, Doug Ledford wrote: > > On Thu, 2019-06-13 at 18:58 +0200, Håkon Bugge wrote: >>> On 13 Jun 2019, at 16:25, Doug Ledford wrote: >>> >>> On Tue, 2019-02-26 at 08:57 +0100, Håkon Bugge wrote: During certain workloads, the default CM response timeout is too

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Jason Gunthorpe
On Thu, Jun 13, 2019 at 07:39:24PM +0200, Håkon Bugge wrote: > > > > On 13 Jun 2019, at 19:23, Jason Gunthorpe wrote: > > > > On Thu, Jun 13, 2019 at 06:58:30PM +0200, Håkon Bugge wrote: > > > >> If you refer to the backlog parameter in rdma_listen(), I cannot see > >> it being used at all for

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Doug Ledford
On Thu, 2019-06-13 at 18:58 +0200, Håkon Bugge wrote: > > On 13 Jun 2019, at 16:25, Doug Ledford wrote: > > > > On Tue, 2019-02-26 at 08:57 +0100, Håkon Bugge wrote: > > > During certain workloads, the default CM response timeout is too > > > short, leading to excessive retries. Hence, make it co

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Håkon Bugge
> On 13 Jun 2019, at 16:25, Doug Ledford wrote: > > On Tue, 2019-02-26 at 08:57 +0100, Håkon Bugge wrote: >> During certain workloads, the default CM response timeout is too >> short, leading to excessive retries. Hence, make it configurable >> through sysctl. While at it, also make number of

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Håkon Bugge
> On 13 Jun 2019, at 19:23, Jason Gunthorpe wrote: > > On Thu, Jun 13, 2019 at 06:58:30PM +0200, Håkon Bugge wrote: > >> If you refer to the backlog parameter in rdma_listen(), I cannot see >> it being used at all for IB. >> >> For CX-3, which is paravirtualized wrt. MAD packets, it is the p

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Jason Gunthorpe
On Thu, Jun 13, 2019 at 06:58:30PM +0200, Håkon Bugge wrote: > If you refer to the backlog parameter in rdma_listen(), I cannot see > it being used at all for IB. > > For CX-3, which is paravirtualized wrt. MAD packets, it is the proxy > UD receive queue length for the PF driver that can be const

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Doug Ledford
On Thu, 2019-06-13 at 08:28 -0700, Bart Van Assche wrote: > On 6/13/19 7:25 AM, Doug Ledford wrote: > > So, to revive this patch, what I'd like to see is some attempt to > > actually quantify a reasonable timeout for the default backlog > > depth, > > then the patch should actually change the defau

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Bart Van Assche
On 6/13/19 7:25 AM, Doug Ledford wrote: So, to revive this patch, what I'd like to see is some attempt to actually quantify a reasonable timeout for the default backlog depth, then the patch should actually change the default to that reasonable timeout, and then put in the ability to adjust the t

Re: [PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-06-13 Thread Doug Ledford
On Tue, 2019-02-26 at 08:57 +0100, Håkon Bugge wrote: > During certain workloads, the default CM response timeout is too > short, leading to excessive retries. Hence, make it configurable > through sysctl. While at it, also make number of CM retries > configurable. > > The defaults are not changed

[PATCH v2] RDMA/cma: Make CM response timeout and # CM retries configurable

2019-02-25 Thread Håkon Bugge
During certain workloads, the default CM response timeout is too short, leading to excessive retries. Hence, make it configurable through sysctl. While at it, also make number of CM retries configurable. The defaults are not changed. Signed-off-by: Håkon Bugge --- v1 -> v2: * Added unregiste