> >> 1) rping, on the client side, clears the conn_params for the newly to be
> >> attempted connection, then sets:
> >>
> >>    conn_param.responder_resources = 1;
> >>    conn_param.initiator_depth = 1;
> >>    conn_param.retry_count = 10;
> >
> >
> >> 2) the qib driver enforces a maximum of 7 for retry_count.  I don't see
> >> anything in the spec that specifies a maximum for this entry, and in
> >> particular I know it doesn't call out for 7 to mean infinite retries
> >> like it does for rnr_retry_count.
> >
> > On IB, retry count is a 3-bit value.  See the CM REQ message.
> >
> > The kernel rdma cm should probably check for a valid value and reduce it
> accordingly.
> 
> Then it's also fair to say that right now rping passes a known invalid
> value and needs fixed ;-)

For IB, but I wasn't sure about iWarp.  It doesn't look like it's used there 
though.

If not, I'll apply a patch to rping, plus submit a fix for the kernel to 
validate the value.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to