On Mon, Aug 1, 2016 at 12:09 PM, Russell Bryant <russ...@ovn.org> wrote:
> > > On Mon, Aug 1, 2016 at 12:00 PM, Ben Pfaff <b...@ovn.org> wrote: > >> On Mon, Aug 01, 2016 at 02:55:26AM -0700, nickcooper-zhangtonghao wrote: >> > If the chassis doesn't configure the 'external-ids:ovn-bridge-mappings' >> in >> > the OVSDB, the 'add_bridge_mappings' should return directly. >> > >> > Signed-off-by: nickcooper-zhangtonghao >> <nickcooper-zhangtong...@opencloud.tech> >> >> What's the benefit? > > > I was looking at this earlier before I got on some calls. > > There doesn't appear to be any bug here, but this patch does skip some > unnecessary code. The most notable improvement seems to be skipping a loop > over all Port_Binding records in every call to patch_run. > Acked-by: Russell Bryant <russ...@ovn.org> -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
