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 rdma-next 00/10] Enable relaxed ordering for ULPs

2021-04-12 Thread Haakon Bugge
> On 10 Apr 2021, at 15:30, David Laight wrote: > > From: Tom Talpey >> Sent: 09 April 2021 18:49 >> On 4/9/2021 12:27 PM, Haakon Bugge wrote: >>> >>> >>>> On 9 Apr 2021, at 17:32, Tom Talpey wrote: >>>> >>>> On 4

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

Re: [PATCH rdma-next 00/10] Enable relaxed ordering for ULPs

2021-04-09 Thread Haakon Bugge
> On 9 Apr 2021, at 17:32, Tom Talpey wrote: > > On 4/9/2021 10:45 AM, Chuck Lever III wrote: >>> On Apr 9, 2021, at 10:26 AM, Tom Talpey wrote: >>> >>> On 4/6/2021 7:49 AM, Jason Gunthorpe wrote: On Mon, Apr 05, 2021 at 11:42:31PM +, Chuck Lever III wrote: > We need to ge

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 v2] net/rds: Fix a use after free in rds_message_map_pages

2021-03-31 Thread Haakon Bugge
> On 31 Mar 2021, at 03:59, Lv Yunlong wrote: > > In rds_message_map_pages, the rm is freed by rds_message_put(rm). > But rm is still used by rm->data.op_sg in return value. > > My patch assigns ERR_CAST(rm->data.op_sg) to err before the rm is > freed to avoid the uaf. > > Fixes: 7dba92037baf