On Wed, Mar 04, 2015 at 10:29:36PM -0800, Justin Pettit wrote: > > > On Mar 4, 2015, at 10:00 PM, Ben Pfaff <b...@nicira.com> wrote: > > > > On Wed, Mar 04, 2015 at 06:09:17PM -0800, Justin Pettit wrote: > >> Signed-off-by: Justin Pettit <jpet...@nicira.com> > >> --- > >> v1->v2: - Add more comments based on suggestions from Ben. > >> - Fix whitespace issue. > > > > Doesn't appear to apply to current master or ovn branches, but you > > didn't repost the whole series. I think that I acked the other patches, > > do you want to apply them? > > Okay, I just pushed them, so this should now cleanly apply to master by > itself.
It applies for me now. Some of the comment wording is odd, e.g. on ovsrec_autoattach_set_mappings(): * The value must be key in range 0 to 16,777,215, value in range 0 to 4,095. or on ovsrec_bridge_set_flow_tables(): * The value must be key in range 0 to 254. That's all I noticed. Seems like a nice improvement! Acked-by: Ben Pfaff <b...@nicira.com> _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
