> 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. Ethan _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
