Hmm, right you are. I'll try to be more careful in future.

On 20 November 2013 11:23, Ben Pfaff <b...@nicira.com> wrote:
> 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