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

dev at openvswitch.org dev at openvswitch.org
Fri Mar 22 21:47:42 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  8a02b5ee895add9b5ec4407ad5e525654a41a3dc (commit)
      from  99c8d7931003c800db581d3fe734186d011ea896 (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 8a02b5ee895add9b5ec4407ad5e525654a41a3dc
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=8a02b5ee895add9b5ec4407ad5e525654a41a3dc
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(-)

Open vSwitch

More information about the git mailing list