[ovs-git] Open vSwitch: ofproto-dpif: Update 'facet->rule' stats in facet_push_stats(). (master)

dev at openvswitch.org dev at openvswitch.org
Thu May 9 19:48:41 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  eafed69b66fa5b7b69035fe5aa2ae2102d66d6f6 (commit)
       via  0f49659a40f6e19cf908b9a3f571cf738bba2881 (commit)
       via  79e15b78532f770ed14d91714def962b6b5c3f16 (commit)
      from  3abe1835dbf92fb770d63a9754c77285e3994bd9 (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 eafed69b66fa5b7b69035fe5aa2ae2102d66d6f6
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=eafed69b66fa5b7b69035fe5aa2ae2102d66d6f6
Author: Ethan Jackson <ethan at nicira.com>
ofproto-dpif: Update 'facet->rule' stats in facet_push_stats().
Before this patch, stats for 'facet->rule' were handled differently
than stats which 'facet' resubmitted into.  The former were
maintained in 'facet' until it was destroyed, while the latter were
pushed regularly in facet_push_stats().

This inconsistent behavior was not only confusing, it was often
incorrect.  In some circumstances, if a facet changed rules, it
could carry the statistics from the entirety of its lifetime from
the old rule to the new one.  This patch remedies the issue by
handling all rule stats in facet_push_stats().

Reported-by: Tmusic <Tmusic987 at gmail.com>
Signed-off-by: Ethan Jackson <ethan at nicira.com>

commit 0f49659a40f6e19cf908b9a3f571cf738bba2881
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=0f49659a40f6e19cf908b9a3f571cf738bba2881
Author: Ethan Jackson <ethan at nicira.com>
ofproto-dpif: Simplify send_packet().
Before this patch, send_packet() manually constructed the required
datapath output actions.  This worked fine when these actions were
simple, however as outputting to tunnels and patch ports became
more complex, this required quite a bit of complex duplicated code.
This patch solves the problem by running through the standard
xlate_actions() code path instead.

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

commit 79e15b78532f770ed14d91714def962b6b5c3f16
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=79e15b78532f770ed14d91714def962b6b5c3f16
Author: Ethan Jackson <ethan at nicira.com>
ofproto-dpif: Remove ofproto_update_local_port_stats().
It's a bit odd, and not particularly useful.

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


Summary of changes:
 ofproto/ofproto-dpif.c |  112 ++++++++++++------------------------------------
 1 files changed, 27 insertions(+), 85 deletions(-)

Open vSwitch

More information about the git mailing list