[ovs-git] Open vSwitch: revalidator: Use xcache when revalidation is required. (master)

[dev at openvswitch.org]

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  a48c85b2d672505b89e488d28066538705b94942 (commit)
       via  feca8bd78a2e8bf5bb37ac8d07e6040d6ff75bd5 (commit)
       via  2608616d87e4669531541b2a64db84c8d1426c2f (commit)
      from  143859ec63d45e5184356984dbf10c142e0ded39 (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 a48c85b2d672505b89e488d28066538705b94942
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a48c85b2d672505b89e488d28066538705b94942
Author: Joe Stringer <joestringer at nicira.com>
		
revalidator: Use xcache when revalidation is required.
		
One of the reasons that xlate_cache was introduced was to ensure that
statistics were attributed to the correct rules and interfaces according
to the flow that was installed into the datapath, rather than according
to the current state of the flow table.

This patch makes the revalidators use the xlate_cache to attribute stats
when full revalidation is required, as the statistics belong to the old
ruleset. However, when revalidating, the rules may have changed while
leaving the datapath flows intact, so we still re-create the xcache at
that point.

Signed-off-by: Joe Stringer <joestringer at nicira.com>
Acked-by: Alex Wang <alexw at nicira.com>


commit feca8bd78a2e8bf5bb37ac8d07e6040d6ff75bd5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=feca8bd78a2e8bf5bb37ac8d07e6040d6ff75bd5
Author: Joe Stringer <joestringer at nicira.com>
		
revalidator: Refactor ukey creation/lookup.
		
This patch refactors the code around ukey creation and lookup to
simplify the code for callers. A new function ukey_acquire() combines
these functions and attempts to acquire a lock on the ukey. Failure to
acquire a lock on the ukey is usually a sign that another thread is
handling the same flow concurrently, which means the flow does not need
to be handled anyway.

Signed-off-by: Joe Stringer <joestringer at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>


commit 2608616d87e4669531541b2a64db84c8d1426c2f
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=2608616d87e4669531541b2a64db84c8d1426c2f
Author: Joe Stringer <joestringer at nicira.com>
		
tests: Rename "ofproto-if"->"ofproto-dpif"
		
It's unclear why these tests are named "ofproto-if..." unlike all of the
other ofproto-dpif tests. Rename them to be more consistent.

Signed-off-by: Joe Stringer <joestringer at nicira.com>
Acked-by: Simon Horman <horms at verge.net.au>


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

Summary of changes:
 ofproto/ofproto-dpif-upcall.c |  111 ++++++++++++++++++++---------------------
 tests/ofproto-dpif.at         |    8 +--
 2 files changed, 57 insertions(+), 62 deletions(-)


hooks/post-receive
-- 
Open vSwitch