On Fri, Jul 22, 2011 at 03:58:44PM -0700, Ethan Jackson wrote: > > The update to the comment on nx_action_multipath could have gone into > > the commit that generalized the multipath action output choices. > > Ooops, simple rebasing error. > > > The comment on the 'dst' member of nx_action_bundle says that it is a > > register, but that's not required, right? > > I'll fix that before merging. > > > I think that it might be a good idea for nxm_dst_check() to log a more > > detailed explanation than its caller here (and elsewhere in other > > patches) does. ?Then the callers themselves wouldn't have to bother > > logging, and anyone debugging will have a better idea of the actual > > problem. > > I sent out an incremental which does this. I'll update this patch as well.
All sounds good, thank you. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
