[ofa-general] Re: IPOIB CM (NOSRQ)[PATCH V2] patch for review

Michael S. Tsirkin mst at dev.mellanox.co.il
Tue Apr 24 21:05:00 PDT 2007


> Quoting Sean Hefty <mshefty at ichips.intel.com>:
> Subject: Re: [ofa-general] Re: IPOIB CM (NOSRQ)[PATCH V2] patch for review
> 
> >If the ACK delays on both sides are not being taken into account
> >properly when establishing a connection, then I guess that is a bug in
> >our CM.
> 
> I looked, and the cm does not take into account the ca ack delay.  This can 
> be worked around by bumping up the qp timeout value between calling 
> ib_cm_init_qp_attr() and ib_modify_qp(), or by increasing the path record 
> packet_life_time.

What really should happen is that the field Local Ack Timeout in REQ
should be (2 * PacketLifeTime + Local CA’s ACK delay) (see 12.7.34)
and then the responder should use this for it's QP.

This does not sound too hard - why can't we just fix CM to do this, then?

-- 
MST



More information about the general mailing list