[ovs-dev] [bundle 7/7] bundle: New action "bundle_load".

Ben Pfaff blp at nicira.com
Fri Jul 22 23:00:48 UTC 2011


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.



More information about the dev mailing list