Re: [PATCH for-next] IB/cma: Introduce rdma_set_min_rnr_timer()

Jason Gunthorpe
2021-03-31T17:51:17Z

On Wed, Mar 31, 2021 at 05:45:30PM +0000, Haakon Bugge wrote:

> True. Local ACK Timeout is part of CM REQ. But that makes
> 2c1619edef61 ("IB/cma: Define option to set ack timeout and pack
> tos_set") fuzzy, as it claims in the commentary: "The timeout will
> affect the local side of the QP, it is not negotiated with remote
> side ..."

Hmm, is it intended to replace the CM provided local ack timeout on
the responder side?

It doesn't look like it copied into the MAD though, and the
qp_attr->timeout only looks like it is et for the IB_QPT_XRC_TGT case??

Very confusing indeed. Parav?

Jason