Quoting r. Sean Hefty <[EMAIL PROTECTED]>: > > Finally, the patch in question also seems to introduce more cleanups and > > such. It would be less controversial if it was just an API change. > > The cleanups are part of the change. Once ib_join_multicast() has been > invoked, > ib_free_multicast() must be called exactly once. Proper state tracking for > this > is required.
We already do state tracking to join/leave groups properly. It would be easier to review this change if you left killing off mcast->done and other such improvements for a separate patch. -- MST _______________________________________________ openib-general mailing list openib-general@openib.org http://openib.org/mailman/listinfo/openib-general To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general