[Openib-windows] Bug or by design?

[Fabian Tillier]

Hi Saul,

On 8/14/06, Saul <26oo at cox.net> wrote:
> I later discovered that you cannot go from RTS -> RTR -> RTS, so I scrapped
> that idea.  I made another post to the list with the "patch" that I put into
> mthca_qp.c to fix things.

I never saw your patch come through.  Could you resend?

>  It appears to me that there just needed to be
> some required params specified for the RTS->RTS state change, just like
> there are for the RTR->RTS change.  Without anything being specified as
> required, but with the routine mlnx_conv_qp_modify_attr() forcing
> IB_QP_SQ_PSN to be required (line 743) it was a no win situation.  If there
> was an IB_MOD_QP_xxx flag for SQ_PSN I would be fine, but since there is not
> I was stuck.

The ib_qp_mod_t structure is laid out to segregate required from
optional parameters.  These are generally documented in the IB
specification, but the structure will help.

The "opts" field in the structure serves as delimiter for required vs.
optional parameters.  For the RTS case, sq_psn, retry_cnt,
rnr_retry_cnt, local_ack_timeout, init_depth are all required.

- Fab