> On Aug 2, 2016, at 12:09 AM, Russell Bryant <russ...@ovn.org> wrote: > > > > On Mon, Aug 1, 2016 at 12:00 PM, Ben Pfaff <b...@ovn.org > <mailto: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. > > -- > Russell Bryant
I’am with Russell. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
