On Thu, Nov 14, 2013 at 03:28:27PM -0800, Joe Stringer wrote:
> When dealing with a large number of ports, bundle_run() and
> bundle_wait() add significant unnecessary processing to the main run
> loop, even when there are no bonds and lacp is not configured. This
> patch skips such execution if it is unneeded, reducing CPU usage from
> about 25% to about 20% in a test environment of 5000 internal ports and
> 50 tunnel ports running bfd.
> 
> Signed-off-by: Joe Stringer <joestrin...@nicira.com>

If I'm reading this right, it suppresses all calls to port_run(), not
just bundle_run(), unless lacp or bonds are enabled.  That doesn't seem
right to me: port_run() deals with a lot more than bonds and LACP.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to