[ovs-git] Open vSwitch: ofproto: Rename struct rule's evict lock and use it more widely. (branch-2.0)

[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, branch-2.0 has been updated
       via  d617c36f029c3804d4f3c701e63e42cd4573ec5d (commit)
       via  f50b0780416946199780bd09bda414a00089a1ff (commit)
       via  a07dfd5d746ad984f7541f84628f884419314c4b (commit)
      from  059ba5813e9e711b1a0dbc34ea7cbe9d360e31a6 (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 d617c36f029c3804d4f3c701e63e42cd4573ec5d
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=d617c36f029c3804d4f3c701e63e42cd4573ec5d
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto: Rename struct rule's evict lock and use it more widely.
		
There are a few fields in struct rule which are accessible by
functions in ofproto-dpif and therefore need to accessed in a thread
safe manner.  This patch achieves this by generalizing the rules evict
rwlock and requiring a writelock to be held to edit them.

Signed-off-by: Ethan Jackson <ethan at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


commit f50b0780416946199780bd09bda414a00089a1ff
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=f50b0780416946199780bd09bda414a00089a1ff
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: Don't manage eviction groups from threads.
		
Managing eviction groups from threads was going to be difficult to do
in a performant thread-safe manner, so this patch punts the problem to
the main thread.

Signed-off-by: Ethan Jackson <ethan at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


commit a07dfd5d746ad984f7541f84628f884419314c4b
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a07dfd5d746ad984f7541f84628f884419314c4b
Author: Ethan Jackson <ethan at nicira.com>
		
ofproto-dpif: Hide struct rule_dpif internally.
		
By hiding struct rule_dpif inside ofproto-dpif, it becomes very clear
which attributes are accessed by multiple threads and need to be
protected by locks.

Signed-off-by: Ethan Jackson <ethan at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


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

Summary of changes:
 ofproto/ofproto-dpif-upcall.c |    6 +--
 ofproto/ofproto-dpif-xlate.c  |   27 ++++++------
 ofproto/ofproto-dpif-xlate.h  |    2 +
 ofproto/ofproto-dpif.c        |   98 ++++++++++++++++++++++++++++++++---------
 ofproto/ofproto-dpif.h        |   58 ++++++++++--------------
 ofproto/ofproto-provider.h    |   24 +++++-----
 ofproto/ofproto.c             |   90 +++++++++++++++++++++++--------------
 7 files changed, 192 insertions(+), 113 deletions(-)


hooks/post-receive
-- 
Open vSwitch