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.

-- 
Russell Bryant
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to