> On 31 Mar 2021, at 01:12, Jason Gunthorpe <jgg@xxxxxxxxxx> wrote: > > On Thu, Mar 25, 2021 at 02:05:47PM +0100, Håkon Bugge wrote: >> Introduce the ability for both user-space and kernel ULPs to adjust >> the minimum RNR Retry timer. The INIT -> RTR transition executed by >> RDMA CM will be used for this adjustment. This avoids an additional >> ib_modify_qp() call. > > Can't userspace override the ibv_modify_qp() call the librdmacm wants > to make to do this? Not sure I understand. The point is, that user-land which intends to set said timer, can do so without an additional ibv_modify_qp() call. May be I should have added: Shamelessly-inspired-by: 2c1619edef61 ("IB/cma: Define option to set ack timeout and pack tos_set") > You'll need to make the rdma-core patches before this can go ahead Ooh, I thought it was the other way around. Will do. I also intend to send an RDS patch, so there will be at least two users of this. Thxs, Håkon