[ovs-git] Open vSwitch: ofproto: Increase default flow-eviction-threshold. (branch-1.10)

dev at openvswitch.org dev at openvswitch.org
Mon Apr 1 23:04:21 UTC 2013


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, branch-1.10 has been updated
       via  be87803d64dc0c29472d4702db43e6b6ed0fefdd (commit)
       via  8ed3c277e0990157652ce5557ccc8e0396deb694 (commit)
       via  3ff576b335ca6cad1dfc1d916c9cc5a98297f2ef (commit)
       via  9eb0b058ce080da913900cece35e0ce2d1bb35b5 (commit)
      from  7e844f8e5817fd71a4439e924658f882f748c5ef (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit be87803d64dc0c29472d4702db43e6b6ed0fefdd
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=be87803d64dc0c29472d4702db43e6b6ed0fefdd
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto: Increase default flow-eviction-threshold.
		
The flow-eviction-threshold presents a trade off between the
expense of maintaining large numbers of datapath flows, and the
benefit of avoid unnecessary flow misses.  In some large Open
vSwitch deployments, we've seen the previous default flow eviction
threshold negatively impact performance with reasonably typical
traffic patterns.  This patch increases the default to a level
which should represent a better trade off: still relatively safe,
but much more amenable to large numbers of long lived flows.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 8ed3c277e0990157652ce5557ccc8e0396deb694
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=8ed3c277e0990157652ce5557ccc8e0396deb694
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: Push statistics less frequently.
		
The most natural place to push facet statistics is in
update_stats() where they're pulled from the datapath.  However,
under load, update_stats() can be called as many as 10 times per
second causing us to push statistics so frequently it hurts
performance.  By pushing statistics much less frequently, this
patch generates a roughly 8% improvement in TCP_CRR performance.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 3ff576b335ca6cad1dfc1d916c9cc5a98297f2ef
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=3ff576b335ca6cad1dfc1d916c9cc5a98297f2ef
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: Run fast internally.
		
ofproto-dpif is responsible for quite a few book keeping tasks in
addition to handling flow misses.  Many of these tasks (flow
expiration, flow revalidation, etc) can take many hundreds of
milliseconds, during which no misses can be handled.  The ideal
long term solution to this problem, is to isolate flow miss
handling into it's own thread.  However, for now this patch
provides a 5% increase in TCP_CRR performance, and smooths out
results during revalidations.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


commit 9eb0b058ce080da913900cece35e0ce2d1bb35b5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=9eb0b058ce080da913900cece35e0ce2d1bb35b5
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: Systematically push stats upon request.
		
Commit bf1e8ff (ofproto-dpif: Push statistics in rule_get_stats()),
started down the road towards pushing stats on demand, but it
didn't go quite far enough.  First, it neglected to push stats in
port_get_stats() and mirror_get_stats().  Second, it only pushes
stats for a single ofproto, making it incomplete when patch ports
are used.

Signed-off-by: Ethan Jackson <ethan at nicira.com>


-----------------------------------------------------------------------

Summary of changes:
 ofproto/ofproto-dpif.c |  114 +++++++++++++++++++++++++++++++++++++++++-------
 ofproto/ofproto.h      |    2 +-
 2 files changed, 99 insertions(+), 17 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list