Libor Michalek wrote:
  I found a couple more spots in the active connect case of incorrect
byte ordering, this time around starting psn. Here is a tested patch.

Thanks for finding this.

-       cm_req_set_starting_psn(req_msg, cpu_to_be32(param->starting_psn));
+       cm_req_set_starting_psn(req_msg, param->starting_psn);

This fix implies that the user is giving the CM the starting PSN in network-byte order, which probably isn't the case. Looking at where the PSN values are set, I think it's swapped in 3 places. (For example, I don't think it's being swapped on the receiving end of the REQ, which is why this patch works.)


I'm re-working the PSN handling portions now for a correct patch.

- Sean
_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to