On Thu, Jul 21, 2011 at 03:58:55PM -0700, Ethan Jackson wrote: > The bundle_load action behaves the same as the bundle action, > except instead of outputting, it writes its result to a register.
The update to the comment on nx_action_multipath could have gone into the commit that generalized the multipath action output choices. The comment on the 'dst' member of nx_action_bundle says that it is a register, but that's not required, right? 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. Otherwise looks good, thank you. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
