On Wed, Jun 27, 2018 at 05:08:31PM -0700, Justin Pettit wrote:
> 
> > On Jun 21, 2018, at 3:53 PM, Ben Pfaff <[email protected]> wrote:
> > 
> > The port_add() function checks whether the port about to be added to the
> > dpif is already present and adds it only if it is not.  This duplicates a
> > check also present (and necessary) in each dpif and races with it as well.
> > When a dpif has a large number of ports, the check can be expensive (it is
> > not efficiently implemented).  It would be nice to made the check cheaper,
> > but it also seems reasonable to do as done in this patch and just let the
> > dpif report the duplication.
> > 
> > Reported-by: Haifeng Lin <[email protected]>
> > Signed-off-by: Ben Pfaff <[email protected]>
> 
> Acked-by: Justin Pettit <[email protected]>

Thanks, applied to master.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to