Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-13 Thread Jason Gunthorpe
On Tue, Apr 13, 2021 at 09:29:41AM +0300, Leon Romanovsky wrote: > On Mon, Apr 12, 2021 at 07:58:47PM -0300, Jason Gunthorpe wrote: > > On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: > > > ib_modify_qp() is an expensive operation on some HCAs running > > > virtualized. This series rem

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-13 Thread Leon Romanovsky
On Tue, Apr 13, 2021 at 11:13:38AM +, Haakon Bugge wrote: > > > > On 13 Apr 2021, at 08:29, Leon Romanovsky wrote: > > > > On Mon, Apr 12, 2021 at 07:58:47PM -0300, Jason Gunthorpe wrote: > >> On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: > >>> ib_modify_qp() is an expensive

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-13 Thread Haakon Bugge
> On 13 Apr 2021, at 08:29, Leon Romanovsky wrote: > > On Mon, Apr 12, 2021 at 07:58:47PM -0300, Jason Gunthorpe wrote: >> On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: >>> ib_modify_qp() is an expensive operation on some HCAs running >>> virtualized. This series removes two ib_m

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-12 Thread Leon Romanovsky
On Mon, Apr 12, 2021 at 07:58:47PM -0300, Jason Gunthorpe wrote: > On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: > > ib_modify_qp() is an expensive operation on some HCAs running > > virtualized. This series removes two ib_modify_qp() calls from RDS. > > > > I am sending this as a v

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-12 Thread Jason Gunthorpe
On Wed, Mar 31, 2021 at 08:43:12PM +0200, Håkon Bugge wrote: > ib_modify_qp() is an expensive operation on some HCAs running > virtualized. This series removes two ib_modify_qp() calls from RDS. > > I am sending this as a v3, even though it is the first sent to > net. This because the IB Core comm

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-12 Thread Jason Gunthorpe
On Mon, Apr 12, 2021 at 06:35:35PM +, Haakon Bugge wrote: > > > > On 7 Apr 2021, at 18:41, Haakon Bugge wrote: > > > > > > > >> On 1 Apr 2021, at 19:51, Jason Gunthorpe wrote: > >> > >> On Wed, Mar 31, 2021 at 07:54:17PM +, Santosh Shilimkar wrote: > >>> [...] > >>> > >>> Thanks Ha

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-12 Thread Haakon Bugge
> On 7 Apr 2021, at 18:41, Haakon Bugge wrote: > > > >> On 1 Apr 2021, at 19:51, Jason Gunthorpe wrote: >> >> On Wed, Mar 31, 2021 at 07:54:17PM +, Santosh Shilimkar wrote: >>> [...] >>> >>> Thanks Haakon. Patchset looks fine by me. >>> Acked-by: Santosh Shilimkar >> >> Jakub/Dave ar

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-07 Thread Haakon Bugge
> On 1 Apr 2021, at 19:51, Jason Gunthorpe wrote: > > On Wed, Mar 31, 2021 at 07:54:17PM +, Santosh Shilimkar wrote: >> [...] >> >> Thanks Haakon. Patchset looks fine by me. >> Acked-by: Santosh Shilimkar > > Jakub/Dave are you OK if I take this RDS patch rdma to rdma's tree? Let me kno

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-04-01 Thread Jason Gunthorpe
On Wed, Mar 31, 2021 at 07:54:17PM +, Santosh Shilimkar wrote: > [...] > > Thanks Haakon. Patchset looks fine by me. > Acked-by: Santosh Shilimkar Jakub/Dave are you OK if I take this RDS patch rdma to rdma's tree? Thanks, Jason

Re: [PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-03-31 Thread Santosh Shilimkar
[...] Thanks Haakon. Patchset looks fine by me. Acked-by: Santosh Shilimkar

[PATCH for-next v3 0/2] Introduce rdma_set_min_rnr_timer() and use it in RDS

2021-03-31 Thread Håkon Bugge
ib_modify_qp() is an expensive operation on some HCAs running virtualized. This series removes two ib_modify_qp() calls from RDS. I am sending this as a v3, even though it is the first sent to net. This because the IB Core commit has reach v3. Håkon Bugge (2): IB/cma: Introduce rdma_set_min_rnr