Around line 1340 of osm_sa_mcmember_record.c, there is the code:

  /* the mcmember_record should have mtu_sel, rate_sel and pkt_lifetime_sel = 2 
*/
  (*pp_mgrp)->mcmember_rec.mtu |= 2<<6; /* exactly */
  (*pp_mgrp)->mcmember_rec.rate |= 2<<6; /* exactly */
  (*pp_mgrp)->mcmember_rec.pkt_life |= 2<<6; /* exactly */

  /* Initialize the mgrp */
  (*pp_mgrp)->mcmember_rec = mcm_rec;
  (*pp_mgrp)->mcmember_rec.mlid = mlid;

I don't know exactly what this is trying to do, but it looks very
fishy to me: as far as I can see, the second block of code overwrites
the effects of the first three lines.  So either those "/* exactly */"
lines aren't needed, or they need to be moved after the mgrp is
initialized.

 - R.

_______________________________________________
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