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

Reply via email to