[ovs-git] Open vSwitch: ovs-thread: Implement OVS specific barrier. (master)

dev at openvswitch.org dev at openvswitch.org
Sat Jun 14 00:04:19 UTC 2014


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  d8043da7182aa0700e8e85a7c53929f3be054ef9 (commit)
       via  a5d4fadd0027b25cd393ba8535594227aa09b86a (commit)
       via  b56ea5d54e072105b398d26421f9a4578fa6e05b (commit)
       via  94f4d44763d49ce082ab70704af672d432cd4ad2 (commit)
       via  200d6ac4cd63bded337765959c5a124dfdce0826 (commit)
       via  1637ebb2e40a328a65b63cea7604d7037dda145f (commit)
       via  8b768391b9d9ff7e4923c7475e4b05df7b29eeac (commit)
       via  b57c6aaa85c01118b32627708a7956b93d6e0aa5 (commit)
      from  f0e5aa1107407bbcabb3a816ee40a98b8ba2050c (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 d8043da7182aa0700e8e85a7c53929f3be054ef9
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=d8043da7182aa0700e8e85a7c53929f3be054ef9
Author: Alex Wang <alexw at nicira.com>
		
ovs-thread: Implement OVS specific barrier.
		
Non-leader revalidator thread uses pthread_barrier_* functions in their
main loop to synchronize with leader thread.  However, since those threads
only call poll_block() intermittently, the poll interval check in
poll_block() can wrongly take the time since last call as poll interval
and issue the following warnings:

"Unreasonably long XXXXms poll interval".

To prevent it, this commit implements the barrier struct and operations
for OVS which allow thread to block on barrier via poll_block().

Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>



commit a5d4fadd0027b25cd393ba8535594227aa09b86a
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a5d4fadd0027b25cd393ba8535594227aa09b86a
Author: Jesse Gross <jesse at nicira.com>
		
netdev-vport: Use dpif_port as base for tunnel backing port.
		
In most cases, tunnel ports specify a dpif name to act as the backing
port in the datapath. However, in the case of UDP tunnels the type is
used with the port number appended. This is potentially a problem for
IPsec tunnels because they have different types but should have the
same backing port. The hasn't been a problem in practice though because
no UDP tunnels are currently used with IPsec.

This switches to use the dpif_port in all cases plus a port number if
necessary. It does this by making the names short enough to accomodate
ports, which also makes the naming more consistent.

Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>


commit b56ea5d54e072105b398d26421f9a4578fa6e05b
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=b56ea5d54e072105b398d26421f9a4578fa6e05b
Author: Ben Pfaff <blp at nicira.com>
		
stream-ssl: Enable TLSv1.1 and TLSv1.2.
		
The Open vSwitch SSL code was inadvertently enabling only TLSv1, not
later versions.  This commit should fix it.

See https://www.openssl.org/docs/ssl/SSL_CTX_new.html
and http://www.postgresql.org/message-id/20131203213049.GA8259@gmail.com
for more information.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Reported-by: Abhinav Singhal <Abhinav.Singhal at spirent.com>
Acked-by: Gurucharan Shetty <gshetty at nicira.com>


commit 94f4d44763d49ce082ab70704af672d432cd4ad2
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=94f4d44763d49ce082ab70704af672d432cd4ad2
Author: Ben Pfaff <blp at nicira.com>
		
odp-util: Make it harder to forget to update ODPUTIL_FLOW_KEY_BYTES.
		
We've forgotten more than once.  Most updates coincide with adding
new flow members, so a build assertion on FLOW_WC_SEQ seems appropriate.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Andy Zhou <azhou at nicira.com>


commit 200d6ac4cd63bded337765959c5a124dfdce0826
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=200d6ac4cd63bded337765959c5a124dfdce0826
Author: Simon Horman <horms at verge.net.au>
		
ofproto: Honour NXFMF_OWN flag of flow monitors
		
If NXFMF_OWN is set then full updates should be sent.
Prior to this change an abbreviated update is always sent
to the controller to which a monitor belongs, that is,
the NXFMF_OWN flag is ignored.

Also:
* Update existing flow tests for this chagne
* Add a test to exercise NX flow monitor with !own

I noticed this while working on support for OpenFlow1.4 flow monitors.

Signed-off-by: Simon Horman <horms at verge.net.au>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit 1637ebb2e40a328a65b63cea7604d7037dda145f
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=1637ebb2e40a328a65b63cea7604d7037dda145f
Author: Simon Horman <horms at verge.net.au>
		
ofproto: Break out perl code for sorting multiline messages
		
The flow monitoring tests includes a perl script to sort multiline
messages.  This commit breaks it out of an individual test so that it can
be used elsewhere.

Signed-off-by: Simon Horman <horms at verge.net.au>
[blp at nicira.com changed approach from a separate file to a shell function]
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit 8b768391b9d9ff7e4923c7475e4b05df7b29eeac
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=8b768391b9d9ff7e4923c7475e4b05df7b29eeac
Author: Linda Sun <lsun at vmware.com>
		
Move setsockopt TCP_NODELAY to when TCP is connected.
		
On windows platform, TCP_NODELAY can only be set when TCP
is established.  If the conection is not immediately returning
success, call it when state is changed from TCP_CONNECTING
to SSL_CONNECTING.

Signed-off-by: Linda Sun <lsun at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>


commit b57c6aaa85c01118b32627708a7956b93d6e0aa5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=b57c6aaa85c01118b32627708a7956b93d6e0aa5
Author: Ben Pfaff <blp at nicira.com>
		
util: Move definition of HANDLE here.
		
A few Open vSwitch source files use a type named HANDLE on Windows systems,
in a way that makes it easier to avoid #ifdefs if we have a dummy
definition on other platforms.  <linux/types.h> was a really weird place
for this dummy definition.  This commit moves it to util.h.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Gurucharan Shetty <gshetty at nicira.com>


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

Summary of changes:
 AUTHORS                       |    1 +
 include/linux/types.h         |    6 +--
 lib/netdev-vport.c            |   37 ++++++++++--------
 lib/netdev.c                  |    2 +-
 lib/odp-util.h                |    7 ++--
 lib/ovs-thread.c              |   46 ++++++++++++++++-------
 lib/ovs-thread.h              |   19 +++++++---
 lib/stream-ssl.c              |   48 ++++++++++++++++++------
 lib/util.h                    |    4 ++
 ofproto/connmgr.c             |    3 +-
 ofproto/ofproto-dpif-upcall.c |   13 +++----
 tests/ofproto.at              |   83 ++++++++++++++++++++++++++++++++---------
 tests/ovs-vsctl.at            |   16 ++++----
 13 files changed, 195 insertions(+), 90 deletions(-)


hooks/post-receive
-- 
Open vSwitch



More information about the git mailing list