[ovs-git] Open vSwitch: ofproto-dpif: xlate actions once with subfacets. (master)

dev at openvswitch.org dev at openvswitch.org
Thu Mar 21 20:19:23 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, master has been updated
       via  5c0243a9db5cdb2a2751caa834534687730c6141 (commit)
      from  1b035ef20084e14b90537fb3873af99f45d40e34 (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 5c0243a9db5cdb2a2751caa834534687730c6141
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=5c0243a9db5cdb2a2751caa834534687730c6141
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: xlate actions once with subfacets.
		
Before this patch, when ofproto-dpif decided that a particular flow
miss needed a facet, it would do action translation multiple times.
Once in subfacet_make_actions(), and once per packet in
subfacet_update_stats().  In the common case (once per miss), this
would double the amount of work required in xlate_actions().

The call to facet_push_stats() in subfacet_update_stats() is
unnecessary.  If the packets are simply accounted to the facet,
they will eventually be pushed to the relevant rules in
update_stats() or when the facet is removed.   Removing the
unnecessary step gives us a 20% improvement of the netperf TCP_CRR
benchmark with the complex flow tables installed by our controller.

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


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

Summary of changes:
 ofproto/ofproto-dpif.c |    1 -
 tests/ofproto-dpif.at  |    1 +
 2 files changed, 1 insertions(+), 1 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list