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

[Pradeep Satyanarayana]

Michael S. Tsirkin wrote:
>> @@ -159,11 +214,14 @@ static struct ib_qp *ipoib_cm_create_rx_
>>  		.recv_cq = priv->cq,
>>  		.srq = priv->cm.srq,
>>  		.cap.max_send_wr = 1, /* FIXME: 0 Seems not to work */
>> +		.cap.max_recv_wr = ipoib_recvq_size + 1,
>>  		.cap.max_send_sge = 1, /* FIXME: 0 Seems not to work */
>>  		.sq_sig_type = IB_SIGNAL_ALL_WR,
>>  		.qp_type = IB_QPT_RC,
>>  		.qp_context = p,
>>  	};
> 
> Why aren't you using UC QPs here? With retry count 0, what is
> the benefit of RC?
> 
The issue with switching only NOSRQ to UC is interoperability between 
HCAs. Switching IPOIB CM to UC mode would be good, but let us do all of 
it at one go.

Pradeep