[ovs-git] [openvswitch/ovs] 620b17: INSTALL.md: Add a few notes for Windows builds.
GitHub
noreply at github.com
Sun Apr 12 17:22:40 UTC 2015
Branch: refs/heads/ovn
Home: https://github.com/openvswitch/ovs
Commit: 620b176267a5d5684d85e2d90e98e29dd4808942
https://github.com/openvswitch/ovs/commit/620b176267a5d5684d85e2d90e98e29dd4808942
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M INSTALL.md
Log Message:
-----------
INSTALL.md: Add a few notes for Windows builds.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Gurucharan Shetty <gshetty at nicira.com>
Commit: 86b3f7e1c805fa1a9f78b9df2772a1780402fbcb
https://github.com/openvswitch/ovs/commit/86b3f7e1c805fa1a9f78b9df2772a1780402fbcb
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M INSTALL.md
Log Message:
-----------
INSTALL.md: Note that Clang and ccache aren't completely compatible.
CC: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Commit: c7952afb1258f4282140de52baa51ab59ae169e6
https://github.com/openvswitch/ovs/commit/c7952afb1258f4282140de52baa51ab59ae169e6
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M AUTHORS
M lib/netdev-linux.c
M vswitchd/bridge.c
Log Message:
-----------
netdev-linux: Be more careful about integer overflow in policing.
Otherwise the policing limits could make no sense if large rates were
specified.
Reported-by: Zhangguanghui <zhang.guanghui at h3c.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Ansis Atteka <aatteka at nicira.com>
Commit: ed9c9e3e087ab8892229154c2098410eee17b554
https://github.com/openvswitch/ovs/commit/ed9c9e3e087ab8892229154c2098410eee17b554
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
Log Message:
-----------
ofproto-dpif-xlate: Preserve and execute action_set on a patch port.
Prevent a peer bridge from seeing or modifying the action_set of the
first bridge.
Also execute the accumulated action set on the peer bridge, if any.
Found by inspection.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: f3d5b47390c3c1db31444b8dd5d7f635438a805d
https://github.com/openvswitch/ovs/commit/f3d5b47390c3c1db31444b8dd5d7f635438a805d
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
Log Message:
-----------
ofproto-dpif-xlate: Do not exit if peer bridge exits.
The fact that the peer bridge exits (for any reason) does not mean
that the original bridge should exit. Specifically, if the peer
bridge recirculates (which typically modifies the packet), the
original bridge must continue processing with the original, not the
recirculated packet!
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 0792d095fbc93a253fefd56c8e1d625cfed16c1c
https://github.com/openvswitch/ovs/commit/0792d095fbc93a253fefd56c8e1d625cfed16c1c
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M include/openflow/openflow-common.h
Log Message:
-----------
Use NTR rather than NTR as prefix for Netronome vendor id
As pointed out by Ben Pfaff NTR (NetronoMe eXtension) is confusingly
similar to NXM (Nicira eXtended Match?) which is already used widely
in the Open vSwitch tree.
To get ease future eye strain use NTR (NeTRonome) instead of NTR,
starting with the only instance currently present in the tree,
the Netronome vendor id.
This is in preparation for actually using the Netronome vendor id.
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 71d6ab4533bd3320acaee738d65ccd98a5ecdb78
https://github.com/openvswitch/ovs/commit/71d6ab4533bd3320acaee738d65ccd98a5ecdb78
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M include/openflow/automake.mk
Log Message:
-----------
include/openflow: Simplify structure checking for OpenFlow headers.
This means that anyone who adds another openflow include file automatically
gets it checked. It does mean that changing any of the OpenFlow headers
causes all of them to be checked, but that doesn't seem like a big deal
(it's quick).
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Thomas Graf <tgraf at noironetworks.com>
Commit: 38b01df60dc8688f934bfe13e9d28cb43f3a5829
https://github.com/openvswitch/ovs/commit/38b01df60dc8688f934bfe13e9d28cb43f3a5829
Author: Gurucharan Shetty <gshetty at nicira.com>
Date: 2015-03-17 (Tue, 17 Mar 2015)
Changed paths:
M AUTHORS
M INSTALL.Windows.md
Log Message:
-----------
INSTALL.Windows.md: Mention the easy way to get into bash.
Just typing 'bash' from MSVC developers command prompt _can_ cause
MSYS to use the Windows version of 'sort' instead of the GNU sort.
Reported-by: Hemanth Kumar Mantri <mantri at nutanix.com>
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Commit: ac3e3aaa0a9be3f5fed6b9b65509efd6b377778a
https://github.com/openvswitch/ovs/commit/ac3e3aaa0a9be3f5fed6b9b65509efd6b377778a
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-18 (Wed, 18 Mar 2015)
Changed paths:
M AUTHORS
M lib/netdev-linux.c
Log Message:
-----------
netdev-linux: Avoid RTM_GETQDISC bug workaround on new-enough kernels.
Otherwise we can't detect classless qdiscs.
This has no useful effect for the currently supported qdiscs, which all
have classes, but it makes it possible to add support for new classless
qdiscs.
This suddenly makes netdev-linux complain about qdiscs it doesn't know
about (e.g. pfifo_fast), which isn't too useful, so this commit also
demotes that INFO message to DBG level.
Reported-by: Jonathan Vestin <jonavest at kau.se>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: b6b0e049fd51e1d434ca2ecbbb98c11d4f805f76
https://github.com/openvswitch/ovs/commit/b6b0e049fd51e1d434ca2ecbbb98c11d4f805f76
Author: Alex Wang <alexw at nicira.com>
Date: 2015-03-18 (Wed, 18 Mar 2015)
Changed paths:
M utilities/ovs-appctl-bashcomp.bash
Log Message:
-----------
ovs-appctl-bashcomp: Use better function to complete file.
This commit uses the _filedir function defined in bash_completion
module for file completion. It will take care of the '/' suffix
for directory.
Signed-off-by: Alex Wang <alexw at nicira.com>
Commit: 58be9c9fd732b5bdd3d4c2e9b8cc2313f570094d
https://github.com/openvswitch/ovs/commit/58be9c9fd732b5bdd3d4c2e9b8cc2313f570094d
Author: Mark D. Gray <mark.d.gray at intel.com>
Date: 2015-03-19 (Thu, 19 Mar 2015)
Changed paths:
M vswitchd/automake.mk
M vtep/automake.mk
Log Message:
-----------
automake.mk: Improve schema checksum error message.
Signed-off-by: Mark D. Gray <mark.d.gray at intel.com>
Signed-off-by: Billy O'Mahony <billy.o.mahony at intel.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 58397e6c1e6c0b68a0f3f32ded463df8087021d8
https://github.com/openvswitch/ovs/commit/58397e6c1e6c0b68a0f3f32ded463df8087021d8
Author: Kevin Traynor <kevin.traynor at intel.com>
Date: 2015-03-19 (Thu, 19 Mar 2015)
Changed paths:
M INSTALL.DPDK.md
M Makefile.am
M NEWS
M acinclude.m4
M lib/netdev-dpdk.c
M lib/netdev.c
M utilities/automake.mk
A utilities/qemu-wrap.py
M vswitchd/ovs-vswitchd.c
Log Message:
-----------
netdev-dpdk: add dpdk vhost-cuse ports
This patch adds support for a new port type to userspace datapath
called dpdkvhost. This allows KVM (QEMU) to offload the servicing
of virtio-net devices to its associated dpdkvhost port. Instructions
for use are in INSTALL.DPDK.
This has been tested on Intel multi-core platforms and with clients
that have virtio-net interfaces.
Signed-off-by: Ciara Loftus <ciara.loftus at intel.com>
Signed-off-by: Kevin Traynor <kevin.traynor at intel.com>
Signed-off-by: Maryam Tahhan <maryam.tahhan at intel.com>
Signed-off-by: Pravin B Shelar <pshelar at nicira.com>
Commit: c557ca04169bbe0af253bd910a5328ac3eee98e7
https://github.com/openvswitch/ovs/commit/c557ca04169bbe0af253bd910a5328ac3eee98e7
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-20 (Fri, 20 Mar 2015)
Changed paths:
M utilities/ovs-vsctl.c
Log Message:
-----------
ovs-vsctl: Create AutoAttach records on demand, for backward compatibility.
The AutoAttach table is new in OVS 2.3.90, but ovs-vsctl was creating a
record in the table unconditionally whenever it created a new bridge.
This caused a gratuitous incompatibility with older databases, which can
be a problem during upgrade. This commit switches to creating the
AutoAttaach record for a bridge lazily, only when the first mapping is
created for the bridge.
VMware-BZ: #1413562
CC: Dennis Flynn <drflynn at avaya.com>
Reported-by: Alex Wang <alexw at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Commit: d6f115f56a88d66f3f6b5f657a45ce6e37099702
https://github.com/openvswitch/ovs/commit/d6f115f56a88d66f3f6b5f657a45ce6e37099702
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-20 (Fri, 20 Mar 2015)
Changed paths:
M utilities/ovs-vsctl.c
Log Message:
-----------
ovs-vsctl: Only monitor AutoAttach columns when useful.
Otherwise this creates a gratuitous incompatibility with older databases,
which can be a problem in upgrade situations.
VMware-BZ: #1413562
Reported-by: Alex Wang <alexw at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Commit: 3c94676c3727166384372b5dce3de3184ad8ae16
https://github.com/openvswitch/ovs/commit/3c94676c3727166384372b5dce3de3184ad8ae16
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-03-20 (Fri, 20 Mar 2015)
Changed paths:
M acinclude.m4
Log Message:
-----------
acinclude.m4: Restore --whole-archive option for DPDK.
The --whole-archive option is required to link vswitchd with DPDK,
otherwise the PMD drivers are not going to be included. Omitting the
option is not going to cause build failures, but OVS won't be able to
use most physical NICs.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 777cb78767fd0ca821dad79691ee3200de66f06e
https://github.com/openvswitch/ovs/commit/777cb78767fd0ca821dad79691ee3200de66f06e
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-03-20 (Fri, 20 Mar 2015)
Changed paths:
M .travis/build.sh
M .travis/prepare.sh
M INSTALL.DPDK.md
Log Message:
-----------
DPDK: Update documentation and travis build for vhost.
DPDK vhost support is mandatory to compile OVS with DPDK. This commit
updates INSTALL.DPDK.md and the travis build procedure to reflect that.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: bb5dbe78592b3f39f0e7e6361c8796a5023363e5
https://github.com/openvswitch/ovs/commit/bb5dbe78592b3f39f0e7e6361c8796a5023363e5
Author: Alex Wang <alexw at nicira.com>
Date: 2015-03-20 (Fri, 20 Mar 2015)
Changed paths:
M tests/completion.at
M utilities/ovs-vsctl-bashcomp.bash
Log Message:
-----------
ovs-vsctl-bashcomp: Avoid setting the COMP_WORDBREAKS.
Modifying $COMP_WORDBREAKS in completion script is not the recommended
as it is a global variable and the modification could affect the behavior
of other completion scripts. As a workaround, this commit uses the
_get_comp_words_by_ref which allows user to exclude characters out of
$COMP_WORDBREAKS and reassemble input command line. However, as a side
effect, the bash completion module cannot handle characters defined in
$COMP_WORDBREAKS (e.g. ':' and '=') correctly in the resulting completions.
Thusly, we need to trim the colon-word and equal-word prefixes from reply.
Signed-off-by: Alex Wang <alexw at nicira.com>
Commit: 2c9907cde3be90e12efb3c2729c0a8e98e898a2d
https://github.com/openvswitch/ovs/commit/2c9907cde3be90e12efb3c2729c0a8e98e898a2d
Author: Alex Wang <alexw at nicira.com>
Date: 2015-03-20 (Fri, 20 Mar 2015)
Changed paths:
M Makefile.am
M debian/openvswitch-common.install
M debian/openvswitch-switch.install
M rhel/openvswitch.spec.in
M utilities/automake.mk
M utilities/ovs-command-bashcomp.INSTALL.md
M xenserver/openvswitch-xen.spec.in
Log Message:
-----------
bashcomp: Install and package completion scripts.
This commit makes the bash completion scripts be installed to
$(sysconfdir)/bash_completion.d/ through 'make install' and to
/etc/bash_completion.d/ through package installation. This will
make the scripts available for each bash session.
An alternative is to put scripts to /usr/share/bash_completion/
directory. However, this is not supported by earlier version of
bash completion.
Signed-off-by: Alex Wang <alexw at nicira.com>
Commit: 5568661cbff1095b595c6020f2a311d8743dc47f
https://github.com/openvswitch/ovs/commit/5568661cbff1095b595c6020f2a311d8743dc47f
Author: Russell Bryant <rbryant at redhat.com>
Date: 2015-03-23 (Mon, 23 Mar 2015)
Changed paths:
M INSTALL.DPDK.md
Log Message:
-----------
INSTALL.DPDK.md: Terminate code section.
Add a missing terminator for a code section. Without this, the
rendering on github at least shows the rest of the file as a code
block.
Signed-off-by: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 14691214d171d0209ec6e96717f89550a3d3119b
https://github.com/openvswitch/ovs/commit/14691214d171d0209ec6e96717f89550a3d3119b
Author: Dennis Flynn <drflynn at avaya.com>
Date: 2015-03-23 (Mon, 23 Mar 2015)
Changed paths:
M lib/lldp/aa-structs.h
M lib/lldp/lldp-tlv.h
M lib/lldp/lldp.c
M lib/ovs-lldp.c
M tests/test-aa.c
Log Message:
-----------
auto-attach: Support latest version of auto-attach LLDP TLVs
The following enhancements to the auto-attach feature are provided
- Support recent modifications to the AA element discovery TLV
- Support recent Avaya Organizationally Unique ID (OUI) change.
(This will change to IEEE assigned OUI once AA standard has been ratified)
- Remove some Avaya specific #defines
The primary purpose of this commit is to catch up with the latest changes made
to the auto attach TLVs as the Auto Attach feature progresses through the
802.1Q IEEE standards committee. Most notably this includes some minor rework
of the AA element discovery TLV and a recent change to the Avaya OUI value.
Signed-off-by: Dennis Flynn <drflynn at avaya.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 677d9158fc0aa16f875198d83c7bd8f87238aed5
https://github.com/openvswitch/ovs/commit/677d9158fc0aa16f875198d83c7bd8f87238aed5
Author: Jonathan Vestin <jonavest at kau.se>
Date: 2015-03-23 (Mon, 23 Mar 2015)
Changed paths:
M AUTHORS
M NEWS
M lib/netdev-linux.c
M vswitchd/bridge.c
M vswitchd/vswitch.xml
Log Message:
-----------
netdev-linux: Support for SFQ, FQ_CoDel and CoDel qdiscs.
This patch adds support for SFQ, CoDel and FQ_CoDel classless qdiscs to Open vSwitch. It also removes the requirement for a QoS to have at least one Queue (as this makes no sense when using classless qdiscs). I have also not implemented class_{get,set,delete,get_stats,dump_stats} because they are meant for qdiscs with classes.
Signed-off-by: Jonathan Vestin <jonavest at kau.se>
[blp at nicira.com mostly applied stylistic changes]
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 59a33335dcc5c750b55dbd2a7961e91a816697b7
https://github.com/openvswitch/ovs/commit/59a33335dcc5c750b55dbd2a7961e91a816697b7
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Datapath.c
M datapath-windows/ovsext/TunnelFilter.c
M datapath-windows/ovsext/TunnelIntf.h
Log Message:
-----------
datapath-windows: Updated WFP system provider handling
If the Base Filtering Engine (BFE) is not started, the WFP system
provider failed to be added because no session to the engine could
be acquired.
The solution for this was to registered a BFE notification callback
that is called whenever the BFE's state changes. Only if the BFE's
state is running the WFP system provider is added.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/65
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 842733c37ccb51d574795782143ed99b0a76a44d
https://github.com/openvswitch/ovs/commit/842733c37ccb51d574795782143ed99b0a76a44d
Author: Mark D. Gray <mark.d.gray at intel.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M lib/sset.c
M lib/sset.h
M vswitchd/bridge.c
M vswitchd/vswitch.ovsschema
M vswitchd/vswitch.xml
Log Message:
-----------
vswitch.ovsschema: Add datapath_types and port_types.
At startup enumerate datapath and port types and add this information to
the datapath_types and port_types columns in the ovsdb.
This allows an ovsdb client to query the datapath in order to determine
if certain datapath and port types exist. For example, by querying the
port_types column, an ovsdb client will be able to determine if this
instance of ovs-vswitchd was compiled with DPDK support.
Signed-off-by: Mark D. Gray <mark.d.gray at intel.com>
Signed-off-by: Billy O'Mahony <billy.o.mahony at intel.com>
[blp at nicira.com made several changes]
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 89a994c9e7b2bb0ed4259e560c2a8926d51f02ba
https://github.com/openvswitch/ovs/commit/89a994c9e7b2bb0ed4259e560c2a8926d51f02ba
Author: Oleg Gashev <oleg at gashev.net>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M rhel/openvswitch-fedora.spec.in
Log Message:
-----------
rhel: Fix RPM build errors.
On exec rpmbuild -ba openvswitch-fedora.spec displayed:
error: Installed (but unpackaged) file(s) found:
/etc/bash_completion.d/ovs-appctl-bashcomp.bash
/etc/bash_completion.d/ovs-vsctl-bashcomp.bash
RPM build errors:
Installed (but unpackaged) file(s) found:
/etc/bash_completion.d/ovs-appctl-bashcomp.bash
/etc/bash_completion.d/ovs-vsctl-bashcomp.bash
/etc/bash_completion.d/ovs-appctl-bashcomp.bash and
/etc/bash_completion.d/ovs-vsctl-bashcomp.bash added to files list.
Signed-off-by: Oleg Gashev <oleg at gashev.net>
Acked-by: Alex Wang <alexw at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 8c254406a36cf87a6665566130a1b1838d2229bd
https://github.com/openvswitch/ovs/commit/8c254406a36cf87a6665566130a1b1838d2229bd
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
A Documentation/automake.mk
A Documentation/group-selection-method-property.txt
M Makefile.am
Log Message:
-----------
Documentation: Add documentation of group selection method property
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 4aa6cf580bc1292e4e7dd5ec42802ed99311cd6b
https://github.com/openvswitch/ovs/commit/4aa6cf580bc1292e4e7dd5ec42802ed99311cd6b
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M include/openflow/automake.mk
A include/openflow/netronome-ext.h
Log Message:
-----------
Add types for NTR selection method
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: bc65c25ac604d5787714c8f16c8df8e392e8dae9
https://github.com/openvswitch/ovs/commit/bc65c25ac604d5787714c8f16c8df8e392e8dae9
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M lib/meta-flow.c
M lib/meta-flow.h
M lib/nx-match.c
M lib/nx-match.h
M lib/ofp-util.c
M lib/ofp-util.h
M ofproto/ofproto-provider.h
M ofproto/ofproto.c
Log Message:
-----------
Support decoding of NTR selection method
This is in preparation for supporting group mod and desc reply
messages with an NTR selection method group experimenter property.
Currently decoding always fails as it only allows properties for known
selection methods and no selection methods are known yet. A subsequent
patch will propose a hash selection method.
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 53eb84a52cbd8ff279a32f3e7fc2c51e608e5e7a
https://github.com/openvswitch/ovs/commit/53eb84a52cbd8ff279a32f3e7fc2c51e608e5e7a
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M lib/nx-match.c
M lib/nx-match.h
M lib/ofp-print.c
M lib/ofp-util.c
M ofproto/ofproto.c
Log Message:
-----------
Support encoding of NTR selection method
Include NTR selection method experimenter group property in
in group mod request and group desc reply.
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 7565c3e4fe1f98f5eb1a4b2151ef3d22ceb37b00
https://github.com/openvswitch/ovs/commit/7565c3e4fe1f98f5eb1a4b2151ef3d22ceb37b00
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
M ofproto/ofproto-dpif.c
M ofproto/ofproto-dpif.h
Log Message:
-----------
Support translation of NTR selection method
Only the default existing behaviour is translated.
All other methods are rejected for now.
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: b879391e0f2fed8e52ae20ad21ad8ae52cea8363
https://github.com/openvswitch/ovs/commit/b879391e0f2fed8e52ae20ad21ad8ae52cea8363
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M lib/ofp-parse.c
M tests/ofproto.at
M utilities/ovs-ofctl.8.in
Log Message:
-----------
Support NTR selection method in ovs-ofctl group commands
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 0c4b9393b6a5792b888d4b580e04cc3fa64ebc12
https://github.com/openvswitch/ovs/commit/0c4b9393b6a5792b888d4b580e04cc3fa64ebc12
Author: Simon Horman <simon.horman at netronome.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M NEWS
M lib/meta-flow.c
M lib/meta-flow.h
M lib/ofp-util.c
M ofproto/ofproto-dpif-xlate.c
M ofproto/ofproto-dpif.c
M ofproto/ofproto-dpif.h
M tests/ofp-print.at
M tests/ofproto-dpif.at
Log Message:
-----------
Implement hash fields select group
This is intended as a usable demonstration of how
the NTR selection method extension might may be used.
NTR selection method
Signed-off-by: Simon Horman <simon.horman at netronome.com>
[blp at nicira.com added a NEWS entry]
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 7f5fe537de3357e36bbb5c385c0aab2fc2bf6a08
https://github.com/openvswitch/ovs/commit/7f5fe537de3357e36bbb5c385c0aab2fc2bf6a08
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
Log Message:
-----------
ofproto-dpif-xlate: Restore was_mpls after a patch port.
The peer bridge popping MPLS should have no effect on the original
bridge.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: e93ef1c79617c67200835ee05f0605b5c949cc07
https://github.com/openvswitch/ovs/commit/e93ef1c79617c67200835ee05f0605b5c949cc07
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
M ofproto/ofproto-dpif-xlate.h
Log Message:
-----------
ofproto-dpif-xlate: Remove bond recirculation parameters from context.
The bond recirculation parameters in the translation context are short
lived and only carry parameters to immediate downstream functions. As
such they are better served as normal function parameters. This
simplifies the translation context, which needs to be essentially
restored after traversing patch ports.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 5b09e56908a1b1e79d056e7be8e8a97783b1570d
https://github.com/openvswitch/ovs/commit/5b09e56908a1b1e79d056e7be8e8a97783b1570d
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
M tests/ofproto-dpif.at
Log Message:
-----------
ofproto-dpif-xlate: Roll back group bucket actions after every bucket.
We used to roll back group bucket changes only for 'all' and
'indirect' group types, but the expected semantics of all group types
is that any changes by the group bucket are not visible after the
group has been executed.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 80e3509d72acecc44d4661f0da5f50a716e23e18
https://github.com/openvswitch/ovs/commit/80e3509d72acecc44d4661f0da5f50a716e23e18
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
Log Message:
-----------
ofproto-dpif-xlate: More robust wildcarding for select group.
The flow key should be the same regardless of whether a live bucket is
found or not, as it would be confusing that the flow key would be
different (different mask bits) after the last group bucket goes dead.
In general, the megaflow algorithm expects the mask bits be set as
soon as we read the header bits, regardless of what happens
afterwards.
Also, use flow_mask_hash_fields() instead of individually setting mask
fields. This immediately brings in IPv6 support, and helps keeping
masks in sync with potential algorithm changes to flow hashing
functions.
Found by inspection.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 4752cc0c26cf6ddfded8f2675b58b6cd3529662a
https://github.com/openvswitch/ovs/commit/4752cc0c26cf6ddfded8f2675b58b6cd3529662a
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M FAQ.md
M NEWS
M lib/netdev-vport.c
M vswitchd/vswitch.xml
Log Message:
-----------
tunnels: Enable UDP checksum computation for Geneve and VXLAN.
The kernel module can already support outer UDP checksums for
Geneve and VXLAN using the standard checksum flag in tunnel
metadata. This makes userspace aware of the capability so that
users can enable it on tunnel ports.
There is a complication in that there is no way for userspace to
probe or detect if the kernel does not support this capability
in order to warn the user. In this case, connectivity will appear
to function normally but packets will not be checksum protected.
This is mainly an issue for VXLAN which has existed in the kernel
for a some time without checksum support - while there are also
a few kernel versions that support Geneve only without checksums,
they are much less common.
There isn't a particularly good solution to the compatibility
issue without introducing a larger capabilities structure. However,
UDP checksums are likely to be used only rarely at this point in
time and the VXLAN spec (where the main problem lies) recommends
against them. Therefore, this is considered to be an advanced user
feature and we settle for just documenting the issue.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pritesh Kothari <pritesh.kothari at cisco.com>
Commit: 2f4298ce26b1130c63d2e8b21da13c22986bdeef
https://github.com/openvswitch/ovs/commit/2f4298ce26b1130c63d2e8b21da13c22986bdeef
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M lib/netdev-linux.c
Log Message:
-----------
netdev-linux: Fix build with old kernel headers.
Commit 677d9158fc0a (netdev-linux: Support for SFQ, FQ_CoDel and CoDel
qdiscs.) added support for new qdiscs. The commit uses TCA_CODEL_* and
TCA_FQ_CODEL_* not in old kernel headers, causing a build failure against
such headers. This commit should fix the problem by defining these values
ourselves. (I haven't tested it against old headers, so I might have
missed something, but it's a straightforward change and at worst won't do
harm.)
It appears that sfq (also added by the same commit) was in Linux before
2.6.32, so it seems unlikely that we need any compatibility code there.
CC: Jonathan Vestin <jonavest at kau.se>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: da79ce2b71dd879e7f20fdddc715568f6a74185a
https://github.com/openvswitch/ovs/commit/da79ce2b71dd879e7f20fdddc715568f6a74185a
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-03-24 (Tue, 24 Mar 2015)
Changed paths:
M lib/netdev-dpdk.c
Log Message:
-----------
netdev-dpdk: create smaller mempools in case of failure
If rte_mempool_create() fails with ENOMEM, try asking for a smaller
mempools. This patch enables OVS DPDK to run on systems without 1GB
hugepages
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: 13dd4a9738e99684a56b10ce2f1a5ee2d2ec2f9f
https://github.com/openvswitch/ovs/commit/13dd4a9738e99684a56b10ce2f1a5ee2d2ec2f9f
Author: Joe Stringer <joestringer at nicira.com>
Date: 2015-03-25 (Wed, 25 Mar 2015)
Changed paths:
M acinclude.m4
M datapath/compat.h
M datapath/linux/compat/genetlink-openvswitch.c
M datapath/linux/compat/include/linux/skbuff.h
M datapath/linux/compat/include/net/genetlink.h
M datapath/linux/compat/include/net/ip.h
M datapath/linux/compat/include/net/udp.h
M datapath/linux/compat/udp.c
Log Message:
-----------
compat: Fix RHEL7 build.
Tested against 3.10.0-229.el7.x86_64.
Signed-off-by: Joe Stringer <joestringer at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: e672ff9b4d224f4d7ef12b289811b0759a333e48
https://github.com/openvswitch/ovs/commit/e672ff9b4d224f4d7ef12b289811b0759a333e48
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M lib/ofp-actions.c
M lib/ofp-actions.h
M ofproto/bond.c
M ofproto/ofproto-dpif-rid.c
M ofproto/ofproto-dpif-rid.h
M ofproto/ofproto-dpif-upcall.c
M ofproto/ofproto-dpif-xlate.c
M ofproto/ofproto-dpif-xlate.h
M ofproto/ofproto-dpif.c
M ofproto/ofproto-dpif.h
M tests/mpls-xlate.at
M tests/ofproto-dpif.at
Log Message:
-----------
ofproto-dpif: Restore metadata and registers on recirculation.
xlate_actions() now considers an optional recirculation context (via
'xin') and restores OpenFlow pipeline metadata (registers, 'metadata',
etc.) based on it. The recirculation context may contain an action
set and stack to be restored and further actions to be executed upon
recirculation. It also contains a table_id number to be used for rule
lookup in cases where no post-recirculation actions are used.
The translation context internal metadata is restored using a new
internal action: UNROLL_XLATE action stores the translation context
data visible to OpenFlow controllers via PACKET_IN messages. This
includes the current table number and the current rule cookie.
UNROLL_XLATE actions are inserted only when the remaining actions may
generate PACKET_IN messages.
These changes allow the post-MPLS recirculation to properly continue
with the pipeline metadata that existed at the time of recirculation.
The internal table is still consulted for bonds.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 1d741d6dfce5752b8a04568e18da15a2788b70b0
https://github.com/openvswitch/ovs/commit/1d741d6dfce5752b8a04568e18da15a2788b70b0
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif-xlate.c
Log Message:
-----------
ofproto-dpif-xlate: Fix MPLS recirculation.
Prior to this patch MPLS recirculation was not performed on a table
lookup following an MPLS_POP action. This patch refactors MPLS
recirculation triggering so that a table action can be re-done after
recirculation if that table action follows an MPLS_POP action.
Recirculation for a patch port traversal (which also does a table
lookup) after an MPLS_POP action does not need to store the output
action, as recirculation without any post-recirculation actions causes
the table lookup to happen anyway.
Furthermore, the stack actions now have the same post-MPLS_POP
optimization as the SET_FIELD and MOVE actions had already:
recirculation is triggered only if the register in the action is L3 or
higher.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: aa63bc9de230f688692831ee9719a0b9b5eddc6b
https://github.com/openvswitch/ovs/commit/aa63bc9de230f688692831ee9719a0b9b5eddc6b
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Util.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Support for allocating/releasing memory with tag
Added functions for allocating and releasing memory with specified
tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 5bf619159c26044b5643a50ec83aaa7de5b52aef
https://github.com/openvswitch/ovs/commit/5bf619159c26044b5643a50ec83aaa7de5b52aef
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Datapath.c
M datapath-windows/ovsext/Datapath.h
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for datapath code
All memory allocations within datapath code have 'DSVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 48b3e646d13f96d7d4d256966ae56910a557ed19
https://github.com/openvswitch/ovs/commit/48b3e646d13f96d7d4d256966ae56910a557ed19
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/BufferMgmt.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for buffermgmt code
All MDL memory allocations within buffermgmt code have 'BSVO' pool
tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 84726095b13af12d34161842691c3e5a05fc8d20
https://github.com/openvswitch/ovs/commit/84726095b13af12d34161842691c3e5a05fc8d20
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Event.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for event code
All memory allocations within event code have 'ESVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 99075373dab5cdd0a4305e9e7036c9e125579e0a
https://github.com/openvswitch/ovs/commit/99075373dab5cdd0a4305e9e7036c9e125579e0a
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Flow.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for flow code
All memory allocations within flow code have 'LSVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: cc26fc9215f551fafd6aa64dffcd8d111ac5a40c
https://github.com/openvswitch/ovs/commit/cc26fc9215f551fafd6aa64dffcd8d111ac5a40c
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Util.h
M datapath-windows/ovsext/Vxlan.c
Log Message:
-----------
datapath-windows: Added specific pool tag for vxlan code
All memory allocations within vxlan code have 'XSVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: f68ddba96638e82bb1c43a2649e08c94aef3cdff
https://github.com/openvswitch/ovs/commit/f68ddba96638e82bb1c43a2649e08c94aef3cdff
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/IpHelper.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for iphelper code
All memory allocations within iphelper code have 'HSVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 5b55d0abac9a20aeaf0820baf6b3954bf88efe58
https://github.com/openvswitch/ovs/commit/5b55d0abac9a20aeaf0820baf6b3954bf88efe58
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Oid.c
M datapath-windows/ovsext/Oid.h
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for oid code
All memory allocations within oid code have 'ASVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: ba7f9d9019b28ee3331893b1771aa239b0bed005
https://github.com/openvswitch/ovs/commit/ba7f9d9019b28ee3331893b1771aa239b0bed005
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Switch.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for switch code
All memory allocations within switch code have 'SSVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: d016f8414d47c1fe93ba80902c1a5ee46b6fa9be
https://github.com/openvswitch/ovs/commit/d016f8414d47c1fe93ba80902c1a5ee46b6fa9be
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/User.c
M datapath-windows/ovsext/Util.h
Log Message:
-----------
datapath-windows: Added specific pool tag for user code
All memory allocations within user code have 'USVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 2be9ec263e98138602f0a130923ef85fa6f544fd
https://github.com/openvswitch/ovs/commit/2be9ec263e98138602f0a130923ef85fa6f544fd
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M datapath-windows/ovsext/Util.h
M datapath-windows/ovsext/Vport.c
Log Message:
-----------
datapath-windows: Added specific pool tag for vport code
All memory allocations within vport code have 'PSVO' pool tag.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/56
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: e6d9ab56ee1f417e8823a6405d5a085c70fe8982
https://github.com/openvswitch/ovs/commit/e6d9ab56ee1f417e8823a6405d5a085c70fe8982
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M lib/flow.c
M tests/ofp-print.at
M tests/ofproto-dpif.at
M tests/ofproto.at
Log Message:
-----------
flow_format: Do not format "in_port=0".
flow_format() is used mainly for formating the headers of packets
embedded in OpenFlow PACKET_IN messages. In this case the flow does
not have a valid port number, and printing out "in_port=0" only
confuses the resulting output. Besides, 0 is not a valid OpenFlow
port number.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: c876a4bb9bcc881befac5647a1e311b5af09d549
https://github.com/openvswitch/ovs/commit/c876a4bb9bcc881befac5647a1e311b5af09d549
Author: Ricky Li <ricky.li at intel.com>
Date: 2015-03-26 (Thu, 26 Mar 2015)
Changed paths:
M lib/netdev-provider.h
M lib/netdev-vport.c
M lib/netdev.c
M lib/netdev.h
M ofproto/tunnel.c
M tests/tunnel-push-pop.at
Log Message:
-----------
netdev: Fix user space tunneling for set_tunnel action.
e.g. Set tunnel id for encapsulated VxLAN packet (out_key=flow):
ovs-vsctl add-port int-br vxlan0 -- set interface vxlan0 \
type=vxlan options:remote_ip=172.168.1.2 options:out_key=flow
ovs-ofctl add-flow int-br in_port=LOCAL, icmp,\
actions=set_tunnel:3, output:1 (1 is the port# of vxlan0)
Output tunnel ID should be modified to 3 with this patch.
Signed-off-by: Ricky Li <ricky.li at intel.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 6b2771c347547b835cf425878d1d3e636631b1d5
https://github.com/openvswitch/ovs/commit/6b2771c347547b835cf425878d1d3e636631b1d5
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-27 (Fri, 27 Mar 2015)
Changed paths:
M tutorial/ovs-sandbox
Log Message:
-----------
ovs-sandbox: Initialize database before starting ovs-vswitchd.
Otherwise ovs-vswitchd can't immediately start working (until some other
call to ovs-vsctl initializes the database). This is most obvious if one
runs "ovs-vsctl list Open_vSwitch ." as the first command, because the
output will not show the changes that ovs-vswitchd will make to the
database at startup (in particular initializing datapath_types and
iface_types), which is confusing.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Commit: c1ddf0915c4ebdf8626d9fbeb91fab8e7719cd9a
https://github.com/openvswitch/ovs/commit/c1ddf0915c4ebdf8626d9fbeb91fab8e7719cd9a
Author: Flavio Leitner <fbl at redhat.com>
Date: 2015-03-27 (Fri, 27 Mar 2015)
Changed paths:
M rhel/openvswitch-fedora.spec.in
Log Message:
-----------
openvswitch-fedora: remove requires to kernel
That RPM requires helped to work around a bug in
the kernel that was triggered by userspace package.
However, it doesn't force the kernel to be running,
so it's a weak solution. Also, it makes difficult
to re-use the same spec on other RPM distros.
This patch comments out the requires.
Signed-off-by: Flavio Leitner <fbl at redhat.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: a19083996c4f7ec20d5f6abf8ea4e25157cf9820
https://github.com/openvswitch/ovs/commit/a19083996c4f7ec20d5f6abf8ea4e25157cf9820
Author: Alex Wang <alexw at nicira.com>
Date: 2015-03-27 (Fri, 27 Mar 2015)
Changed paths:
M lib/netdev-vport.c
Log Message:
-----------
netdev-vport: Do not update netdev when there is no config change.
When there is any update from ovsdb, ovs will call netdev_set_config()
for every vport. Even though the change is not related to vport, the
current implementation will always increment the per-netdev sequence
number. Subsequently this could cause even more unwanted effects,
e.g. the recreation of 'struct tnl_port' in ofproto level.
This commit fixes the issue by only updating the netdev when there
is actual configuration change.
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 618f44f7a406d6c3e90110420a7fd183d40f1bff
https://github.com/openvswitch/ovs/commit/618f44f7a406d6c3e90110420a7fd183d40f1bff
Author: Kevin Traynor <kevin.traynor at intel.com>
Date: 2015-03-27 (Fri, 27 Mar 2015)
Changed paths:
M lib/netdev-dpdk.c
Log Message:
-----------
netdev-dpdk: Put cuse thread into quiescent state.
ovsrcu_synchronize() is used when setting virtio_dev to NULL.
This results in an ovsrcu_quiesce_end() call which means the
cuse thread may not go into quiescent state again for an
indefinite time. Add an ovsrcu_quiesce_start() call to prevent
this.
Signed-off-by: Kevin Traynor <kevin.traynor at intel.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 4f631ccd515ffb95dbe3f5c22d74b01c64a7d9b8
https://github.com/openvswitch/ovs/commit/4f631ccd515ffb95dbe3f5c22d74b01c64a7d9b8
Author: Alex Wang <alexw at nicira.com>
Date: 2015-03-27 (Fri, 27 Mar 2015)
Changed paths:
M lib/netdev-linux.c
Log Message:
-----------
netdev-linux: Make htb quantum always no less than mtu.
Currently, ovs uses hardcoded rate2quantum = 10 for each htb qdisc.
When qdisc class's rate is small, the resulting quantum (calculated
by min_rate / rate2quantum) will be smaller than MTU. This is not
recommended and tc will keep complaining the following in syslog.
localhost kernel: HTB: quantum of class 10003 is small. Consider r2q change.
localhost kernel: HTB: quantum of class 10004 is small. Consider r2q change.
localhost kernel: HTB: quantum of class 10005 is small. Consider r2q change.
localhost kernel: HTB: quantum of class 10006 is small. Consider r2q change.
To fix the issue, this commit makes ovs always use htb quantum no less
than the MTU.
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 4e5e44e30133939b792a013a812c84f564ffa8aa
https://github.com/openvswitch/ovs/commit/4e5e44e30133939b792a013a812c84f564ffa8aa
Author: Alex Wang <alexw at nicira.com>
Date: 2015-03-28 (Sat, 28 Mar 2015)
Changed paths:
M ofproto/ofproto-dpif.c
Log Message:
-----------
ofproto-dpif: Set need_revalidate when removing cfm from ofport.
When cfm is deleted from a port, all modules should release their
reference so that the cfm struct can be removed from the global hmap
and freed. Therein, the reference held by xlate module can only be
released when the need_revalidate flag is set (e.g set to
REV_RECONFIGURE). And this flag should be set while removing cfm
from ofport. Unfortunately, this has never been done before and the
bug was hidden by another bug fixed in recent commit a190839
(netdev-vport: Do not update netdev when there is no config change.)
To fix this issue, this commit makes the code set need_revalidate
when removing cfm from ofport.
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 0d42597cd1b9fa32fdfc6c033c97d28c09087b57
https://github.com/openvswitch/ovs/commit/0d42597cd1b9fa32fdfc6c033c97d28c09087b57
Author: Alin Serdean <aserdean at cloudbasesolutions.com>
Date: 2015-03-30 (Mon, 30 Mar 2015)
Changed paths:
M build-aux/cccl
Log Message:
-----------
build-aux/cccl: Enhance --with-debug option
This patch changes the behaviour in case the configure argument: --with-debug
was specified.
Currently the optimization flag in the case of debugging is the following:
https://msdn.microsoft.com/en-us/library/f9534wye.aspx
which does not fully disable optimization, that is why it was changed with
the following flag:
https://msdn.microsoft.com/en-us/library/aafb762y.aspx
which disables all code optimization.
Also this patch includes the definition of the following preprocessor
definitions:
_DEBUG - in case --with-debug is specified
The above definitions usually are defined when compiling with the following
flags:
https://msdn.microsoft.com/en-us/library/2kzt1wy3.aspx
Since we are not compiling with the above flag, mimic the behaviour the
debug becahviour.
Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 7ad20cbd96b9b33a2e01be36dc51674fde8b67bf
https://github.com/openvswitch/ovs/commit/7ad20cbd96b9b33a2e01be36dc51674fde8b67bf
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-03-30 (Mon, 30 Mar 2015)
Changed paths:
M lib/dpif-netdev.c
Log Message:
-----------
dpif-netdev: Account for and free lost packets.
Packets for which an upcall has failed (lost packets) must be deleted.
We also need to count them as MISS and LOST.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: 4adb03cb6e62d938b586e68aae7af06587e58f49
https://github.com/openvswitch/ovs/commit/4adb03cb6e62d938b586e68aae7af06587e58f49
Author: Thomas Graf <tgraf at noironetworks.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M datapath/linux/compat/include/net/udp_tunnel.h
Log Message:
-----------
datapath: Use alternate name for udp_sock_create() backport
Account for kernels which provide udp_sock_create() in an
insufficient version.
Avoids the following error when inserting openvswitch.ko on
respective kernels:
openvswitch: exports duplicate symbol udp_sock_create (owned by udp_tunnel)
Fixes: eb6eebd28 ("datapath: Account for "udp: Add udp_sock_create for UDP tunnels to open listener socket")
Signed-off-by: Thomas Graf <tgraf at noironetworks.com>
Cc: Jesse Gross <jesse at nicira.com>
Acked-by: Jesse Gross <jesse at nicira.com>
Commit: 2e2f242e6730a658f077ee4be2b8a9c3f0cc9a04
https://github.com/openvswitch/ovs/commit/2e2f242e6730a658f077ee4be2b8a9c3f0cc9a04
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M lib/jsonrpc.c
Log Message:
-----------
jsonrpc: Add function comments.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Commit: 8d66822447710e1f9a365cb2158a75ba8ad8cc36
https://github.com/openvswitch/ovs/commit/8d66822447710e1f9a365cb2158a75ba8ad8cc36
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M lib/ovsdb-idl.c
Log Message:
-----------
ovsdb-idl: Add function comments.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Commit: 5617ae6abe492498ee6350fe657a53e0ba21f8ba
https://github.com/openvswitch/ovs/commit/5617ae6abe492498ee6350fe657a53e0ba21f8ba
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M lib/cfm.c
M lib/hmap.h
M lib/lacp.c
M lib/netdev-dummy.c
M lib/ovs-lldp.h
Log Message:
-----------
hmap: Don't include ovs-atomic.h unnecessarily.
GNU C++ isn't too happy with ovs-atomic.h. We could fix that (maybe we
should) but the report I received from a C++ user implied to me that it
would be just as useful to just drop the unnecessary #include
"ovs-atomic.h" from hmap.h.
Reported-by: Michael Hu <humichael at vmware.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Jarno Rajahalme <jrajahalme at nicira.com>
Commit: d314fc84e09301f485733fc9fc7c64ce781eb513
https://github.com/openvswitch/ovs/commit/d314fc84e09301f485733fc9fc7c64ce781eb513
Author: Alin Serdean <aserdean at cloudbasesolutions.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M lib/lockfile.c
Log Message:
-----------
lockfile: Support \-delimited file names in lockfile_name().
Currently paths that have only forward slashes like the following
"C:/package/binaries/conf.db" work seamlessly.
If we try the native windows filepaths i.e. "C:\package\binaries\conf.db" we
will hit the following problem:
2015-03-31T15:54:17Z|00001|lockfile|WARN|.c:\package\binaries\conf.db.~lock~:
failed to open lock file: Invalid argument
2015-03-31T15:54:17Z|00002|lockfile|WARN|.c:\package\binaries\conf.db.~lock~:
failed to lock file: Invalid argument
ovsdb-server: I/O error: c:\package\binaries\conf.db: failed to lock lockfile
(Invalid argument)
In this patch we update the lockfile_name function to also look for
backslashes, and also accommodate if we have a mix of backslashes and forward
slashes.
Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
[blp at nicira.com simplified the code]
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: da1e25d5af82d0d17fa9560cb8bb3a29afcd7818
https://github.com/openvswitch/ovs/commit/da1e25d5af82d0d17fa9560cb8bb3a29afcd7818
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M AUTHORS
M vswitchd/vswitch.xml
Log Message:
-----------
vswitch.xml: Improve description of forward-bpdu.
The description was at best incomplete and difficult to understand.
Reported-by: Brian Field <Brian_Field at cable.comcast.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Commit: d18e52e3d97ee160b19a55927876423add24a13a
https://github.com/openvswitch/ovs/commit/d18e52e3d97ee160b19a55927876423add24a13a
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M lib/ovsdb-idl.c
M lib/ovsdb-parser.c
M lib/ovsdb-parser.h
M tests/automake.mk
A tests/idltest2.ovsschema
M tests/ovsdb-idl.at
Log Message:
-----------
ovsdb-idl: Tolerate missing tables and columns.
Until now, if ovs-vsctl (or another client of the C ovsdb-idl library) was
compiled against a schema that had a column or table that was not in the
database actually being used (e.g. during an upgrade), and the column or
table was selected for monitoring, then ovsdb-idl would fail to get any
data at all because ovsdb-server would report an error due to a request
about a column or a table it didn't know about.
This commit fixes the problem by making ovsdb-idl retrieve the database
schema from the database server and omit any tables or columns that don't
exist from its monitoring request. This works OK for the kinds of upgrades
that OVSDB otherwise supports gracefully because it will simply make the
missing columns or tables appear empty, which clients of the ovsdb-idl
library already have to tolerate.
VMware-BZ: #1413562
Reported-by: Alex Wang <alexw at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Alex Wang <alexw at nicira.com>
Commit: 508624b691e1517e4d426aa69165ea1efba80d53
https://github.com/openvswitch/ovs/commit/508624b691e1517e4d426aa69165ea1efba80d53
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-03-31 (Tue, 31 Mar 2015)
Changed paths:
M ovsdb/jsonrpc-server.c
M ovsdb/ovsdb-server.1.in
M tests/ovsdb-server.at
Log Message:
-----------
ovsdb-server: Correct malformed error replies to certain JSON-RPC requests.
I realized that this bug existed only a few weeks ago, but it has been
there since the earliest ovsdb-server code.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Alex Wang <alexw at nicira.com>
Commit: b3cceba0b7c4013f46b01f8987e8716d7857c6db
https://github.com/openvswitch/ovs/commit/b3cceba0b7c4013f46b01f8987e8716d7857c6db
Author: Alex Wang <alexw at nicira.com>
Date: 2015-04-01 (Wed, 01 Apr 2015)
Changed paths:
M vswitchd/bridge.c
Log Message:
-----------
bridge: Execute bridge_run() only after retrieving db contents.
During upgrade of ovs-vswitchd, we do not want to recreate the already
configured kernel interfaces. Especially when IP address is assigned to
the internal port, the recreation will cause the lost of connection.
Therefore, ovs-vswitchd should read current ovsdb content first and then
reuse the existing kernel interfaces that are configured in ovsdb. In
terms of the code language, ovs-vswitchd should only execute bridge_run()
after it finishes reading the ovsdb content.
However, this expected behavior is broken by the recent commit d18e52e
(ovsdb-idl: Tolerate missing tables and columns.) which causes the
execution of bridge_run() before getting the hint of configured interfaces
from ovsdb.
To fix the issue, this commit makes sure that the execution of bridge_run()
happens only after retrieving the ovsdb contents.
VMware-BZ: #1424342
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 190cf5338956c53596f60b20fa7672ba052ee815
https://github.com/openvswitch/ovs/commit/190cf5338956c53596f60b20fa7672ba052ee815
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-04-02 (Thu, 02 Apr 2015)
Changed paths:
M datapath-windows/include/OvsDpInterfaceExt.h
M datapath-windows/ovsext/Datapath.c
M datapath-windows/ovsext/Flow.c
M lib/netlink-socket.c
Log Message:
-----------
datapath-windows: Make GET_PID a separate IOCTL
Added a new IOCTL in order to retrieve the PID from the kernel datapath.
The new method uses a direct and cleaner way, as opposed to the old way
of using a Netlink transaction, avoiding the unnecessary overhead.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/31
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Tested-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 73c33892acabe104ddd3d87561d368f3b75d3047
https://github.com/openvswitch/ovs/commit/73c33892acabe104ddd3d87561d368f3b75d3047
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-02 (Thu, 02 Apr 2015)
Changed paths:
M INSTALL.md
Log Message:
-----------
INSTALL.md: Document how to add custom compiler flags.
CC: Mark B Kavanagh <mark.b.kavanagh at intel.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Commit: 97447f55a9678739a64d10db9401858cf040abcf
https://github.com/openvswitch/ovs/commit/97447f55a9678739a64d10db9401858cf040abcf
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-04-02 (Thu, 02 Apr 2015)
Changed paths:
M lib/dpif-netdev.c
Log Message:
-----------
dpif-netdev: Remove support for DPIF_FP_ZERO_STATS flag
Since flow statistics are thread local and updated without any lock,
it is not correct to do a memset from another thread.
This commit simply removes the support for the flag. It is not needed
by ofproto-dpif, it is only exposed by dpctl commands.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: 4f7e5c274bfff6187313588a66491bcf45b7a709
https://github.com/openvswitch/ovs/commit/4f7e5c274bfff6187313588a66491bcf45b7a709
Author: Russell Bryant <rbryant at redhat.com>
Date: 2015-04-02 (Thu, 02 Apr 2015)
Changed paths:
M INSTALL.md
Log Message:
-----------
INSTALL.md: Add note about EXTRA_CFLAGS.
Add a note about the use of EXTRA_CFLAGS to provide custom CFLAGS for
the build of the Linux kernel module.
This addition is technically in the "configuring the sources" section
of the document. However, it's the spot where custom CFLAGS is
discussed already, so that seemed like the best place to put it.
Alternatively, it could go in the "Building the Sources" section
instead.
Signed-off-by: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 416e71322f82beea869b2e5928be5ae60f2577de
https://github.com/openvswitch/ovs/commit/416e71322f82beea869b2e5928be5ae60f2577de
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-03 (Fri, 03 Apr 2015)
Changed paths:
M acinclude.m4
Log Message:
-----------
acinclude: Always assume buggy strtok_r() for glibc < 2.8.
Lately our internal build system has been seeing intermittent failures that
I can't explain. With old glibc versions, the "configure" time check will
pass, but the equivalent (almost identical) "make check" test will fail.
One possibility, I guess, is that occasionally address space randomization
will put valid data at the 0xc0ffee address that the test assumes will
segfault, and another is that some change in compiler optimization flags
is making a difference. At any rate, I think it's safe to just always
assume that this strtok_r() bug is present whenever glibc before 2.8 is
in use.
Specifically we've seen this happen intermittently when building against
the XenServer DDK 5.6.100 build 39265, which uses glibc 2.5.
Reported-by: Alex Wang <alexw at nicira.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Alex Wang <alexw at nicira.com>
Commit: 5a38795f5e6623b39004b53c7fede08399a09d79
https://github.com/openvswitch/ovs/commit/5a38795f5e6623b39004b53c7fede08399a09d79
Author: Thomas Graf <tgraf at noironetworks.com>
Date: 2015-04-04 (Sat, 04 Apr 2015)
Changed paths:
M datapath/Modules.mk
M datapath/datapath.c
M datapath/linux/Kbuild.in
M datapath/linux/compat/flow_dissector.c
M datapath/linux/compat/geneve.c
M datapath/linux/compat/gre.c
M datapath/linux/compat/ip_tunnels_core.c
M datapath/linux/compat/udp_tunnel.c
M datapath/linux/compat/vxlan.c
M datapath/vport-geneve.c
M datapath/vport-gre.c
M datapath/vport-internal_dev.c
M datapath/vport-lisp.c
M datapath/vport-netdev.c
M datapath/vport-netdev.h
M datapath/vport-vxlan.c
M datapath/vport.c
M datapath/vport.h
Log Message:
-----------
datapath: Turn vports with dependencies into separate modules
Upstream commit:
The internal and netdev vport remain part of openvswitch.ko. Encap
vports including vxlan, gre, and geneve can be built as separate
modules and are loaded on demand. Modules can be unloaded after use.
Datapath ports keep a reference to the vport module during their
lifetime.
Allows to remove the error prone maintenance of the global list
vport_ops_list.
Signed-off-by: Thomas Graf <tgraf at suug.ch>
Signed-off-by: David S. Miller <davem at davemloft.net>
Also folds in the follow-up commits 9ba559d9ca3 to turned the non-GPL
symbol exports to GPL exports, and fa2d8ff4e35 which fixes a module
reference release bug.
Exports various backwards compat functions linked into the main
openvswitch module as GPL symbols to ensure vport modules can use them.
Some fiddling with the Makefile was needed to work around the fact
that Makefile variables can't contain '-' characters needed to define
'vport-xxx' module sources. Also, Kbuild complains heavily if a
$(module)-y = $(module).o is defined which is actually backed with a .c
file of the same name. Therefore, a new $(build_multi_modules) variable
is defined which lists all module which consist of more than one source
file.
Upstream: 62b9c8d0372 ("ovs: Turn vports with dependencies into separate modules")
Upstream: 9ba559d9ca3 ("openvswitch: Export symbols as GPL symbols.")
Upstream: fa2d8ff4e35 ("openvswitch: Return vport module ref before destruction")
Signed-off-by: Thomas Graf <tgraf at noironetworks.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 1547aff66926f9f66f1e45c4d4723a1d9c940418
https://github.com/openvswitch/ovs/commit/1547aff66926f9f66f1e45c4d4723a1d9c940418
Author: Kevin Lo <kevlo at FreeBSD.org>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M lib/netdev-bsd.c
Log Message:
-----------
netdev-bsd: Fix sign extension bug in ifr_flags on FreeBSD.
FreeBSD fills the int return value with ifr_flagshigh in the high
16 bits and ifr_flags in the low 16 bits rather than blindly promoting
ifr_flags to an int, which will preserve the sign.
This commit makes sure the flags returned isn't negative and apply mask
0xffff to flags.
Signed-off-by: Kevin Lo <kevlo at FreeBSD.org>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: e1333833fa02571f1ae068e402b95bbc75c455b2
https://github.com/openvswitch/ovs/commit/e1333833fa02571f1ae068e402b95bbc75c455b2
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M AUTHORS
Log Message:
-----------
AUTHORS: Add Kevin Lo.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 4207d631d4841ddc38abdc0f2aea3b9fcb5ddfdc
https://github.com/openvswitch/ovs/commit/4207d631d4841ddc38abdc0f2aea3b9fcb5ddfdc
Author: Justin Pettit <jpettit at nicira.com>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M Makefile.am
Log Message:
-----------
Makefile.am: Clarify error message about missing distribution files.
The error message did not make it clear that the problem was due to
files being in git but not the distribution.
Signed-off-by: Justin Pettit <jpettit at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: fdc14c7b33338c78b27d277c864f47c8a37adb36
https://github.com/openvswitch/ovs/commit/fdc14c7b33338c78b27d277c864f47c8a37adb36
Author: Russell Bryant <rbryant at redhat.com>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M tutorial/ovs-sandbox
Log Message:
-----------
ovs-sandbox: Tell gdb to start the daemon.
The current gdb support launches gdb but doesn't start the daemon.
If you start ovsdb-server with gdb, ovs-sandbox produces an error as
it tries to run ovs-vsctl before ovsdb-server is running. Telling gdb
to start the daemon immediately avoids this error.
There are cases where it's useful to go straight to the gdb prompt,
too. For example, someone may want to set a breakpoint. In that
case, it's easy enough to just kill it, set a breakpoint, and execute
'run' again.
In passing, fix indentation to use spaces instead of tabs.
Signed-off-by: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 7e84c25dbbee56ef39434cacac0d8ce755da88bb
https://github.com/openvswitch/ovs/commit/7e84c25dbbee56ef39434cacac0d8ce755da88bb
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M xenserver/openvswitch-xen.spec.in
Log Message:
-----------
xenserver: Package vport-*.ko with kernel module.
Fixes a build error due to unpackaged files.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Commit: 31efbdb998b0267e245b8edc32b34c0dcc45e723
https://github.com/openvswitch/ovs/commit/31efbdb998b0267e245b8edc32b34c0dcc45e723
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M rhel/openvswitch-kmod-fedora.spec.in
Log Message:
-----------
rhel: Package vport-*.ko in Fedora kernel package.
Fixes a presumed build break due to unpackaged files. (Only "presumed" at
this point because it is masked in the VMware build system by the similar
XenServer build failure.)
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Commit: 19c12401d654308b9ee0333c83a2df8e28161385
https://github.com/openvswitch/ovs/commit/19c12401d654308b9ee0333c83a2df8e28161385
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-05 (Sun, 05 Apr 2015)
Changed paths:
M utilities/ovs-dev.py
Log Message:
-----------
ovs-dev.py: Remove vport-*.ko at same time as openvswitch.ko.
My guess is that this is the intent.
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: ce5112573fa58d34ca87dc488ec0d996f8c7f0d8
https://github.com/openvswitch/ovs/commit/ce5112573fa58d34ca87dc488ec0d996f8c7f0d8
Author: Alin Serdean <aserdean at cloudbasesolutions.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M datapath-windows/ovsext/Vport.c
Log Message:
-----------
datapath-windows: Allow NdisSwitchPortTypeSynthetic to be updated
Allow the port to be updated if it the type is only:
NdisSwitchPortTypeSynthetic
Acked-by: Nithin Raju <nithin at vmware.com>
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 6c6045a72a93f72317567cc6075ac306ef0a3c44
https://github.com/openvswitch/ovs/commit/6c6045a72a93f72317567cc6075ac306ef0a3c44
Author: Alin Serdean <aserdean at cloudbasesolutions.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M datapath-windows/ovsext/Vport.c
Log Message:
-----------
datapath-windows Release lock on HVUpdateNIC
Release switchContext->dispatchLock in case the vport has not been found.
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Acked-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 660527a420c29bdbc0cddd51438980ee712ba591
https://github.com/openvswitch/ovs/commit/660527a420c29bdbc0cddd51438980ee712ba591
Author: Dennis Flynn <drflynn at avaya.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M lib/ovs-lldp.c
Log Message:
-----------
auto-attach: Fix segfault when enabling auto attach.
This commit fixes a segmentation fault observed when enabling lldp prior to
establishing auto attach mappings.
Signed-off-by: Dennis Flynn <drflynn at avaya.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Commit: 5f03c98321092da7ff0246117098a7df197b2ce2
https://github.com/openvswitch/ovs/commit/5f03c98321092da7ff0246117098a7df197b2ce2
Author: Jarno Rajahalme <jrajahalme at nicira.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M lib/dp-packet.c
M lib/list.h
M lib/lldp/lldpd-structs.c
M lib/lldp/lldpd.c
M lib/mcast-snooping.c
M lib/ofp-util.c
M lib/ofpbuf.c
M lib/ovs-lldp.c
M lib/ovs-numa.c
M lib/ovs-rcu.c
M lib/vconn.c
M ofproto/bundles.c
M ofproto/connmgr.c
M ofproto/ofproto-dpif-rid.c
M ofproto/ofproto-dpif-xlate.c
M ofproto/ofproto-dpif.c
M ofproto/ofproto.c
M tests/library.at
M tests/test-list.c
Log Message:
-----------
lib/list: Add LIST_FOR_EACH_POP.
Makes popping each member of the list a bit easier.
Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
Acked-by: Russell Bryant <rbryant at redhat.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 500ce35e8d2ef05295dd9e3b6147cd55dd79df55
https://github.com/openvswitch/ovs/commit/500ce35e8d2ef05295dd9e3b6147cd55dd79df55
Author: Alex Wang <alexw at nicira.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M vswitchd/bridge.c
Log Message:
-----------
bridge: Do not wait for status update when there is no bridge.
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 131d04dcc4190d4202d605bd982047084e987fba
https://github.com/openvswitch/ovs/commit/131d04dcc4190d4202d605bd982047084e987fba
Author: Alex Wang <alexw at nicira.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M vswitchd/bridge.c
Log Message:
-----------
bridge: Wait for previous stats update transation when it is incomplete.
When ovsdb happens to get blocked for few seconds, the 'stats_timer'
will not be updated due to incompletion of previous transaction.
When the current time passes the 'stats_timer', the call to
poll_timer_wait_until(stats_timer) will keep waking up ovs-vswitchd,
causing 100% cpu utilization.
This commit fixes this issue by making ovs-vswitchd wait on the
previous idl transaction when it is incomplete rather than the
'stats_timer'.
Signed-off-by: Alex Wang <alexw at nicira.com>
Acked-by: Ben Pfaff <blp at nicira.com>
Commit: 48627758026da115f30ca71a5f651dbc434a8664
https://github.com/openvswitch/ovs/commit/48627758026da115f30ca71a5f651dbc434a8664
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M lib/hmap.h
Log Message:
-----------
hmap: Add comment warning about pitfall in HMAP_FOR_EACH_* usage.
CC: Russell Bryant <rbryant at redhat.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Justin Pettit <jpettit at nicira.com>
Commit: dd2a1e9c59c75dc9bac4f5bc4aee70a1af568e07
https://github.com/openvswitch/ovs/commit/dd2a1e9c59c75dc9bac4f5bc4aee70a1af568e07
Author: Ben Pfaff <blp at nicira.com>
Date: 2015-04-06 (Mon, 06 Apr 2015)
Changed paths:
M rhel/openvswitch-kmod-fedora.spec.in
Log Message:
-----------
rhel: Make Fedora %files and %install section wildcards match.
Since %install puts every .ko file into /lib/modules, %files should make
sure that all of them are packaged.
Suggested-by: Thomas Graf <tgraf at noironetworks.com>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Flavio Leitner <fbl at redhat.com>
Commit: f293a803d9ece7157ec5d098deb1bf29e5de5915
https://github.com/openvswitch/ovs/commit/f293a803d9ece7157ec5d098deb1bf29e5de5915
Author: Kevin Lo <kevlo at FreeBSD.org>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M INSTALL.userspace.md
Log Message:
-----------
specify -w to set variables for sysctl(8) on NetBSD
We have to specify -w to set tunable sysctls on NetBSD.
Signed-off-by: Kevin Lo <kevlo at FreeBSD.org>
Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: YAMAMOTO Takashi <yamamoto at valinux.co.jp>
Commit: 6c4e7adb0807155b603eef30a7929c42667bfa38
https://github.com/openvswitch/ovs/commit/6c4e7adb0807155b603eef30a7929c42667bfa38
Author: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M datapath-windows/ovsext/Datapath.c
M datapath-windows/ovsext/Switch.c
M datapath-windows/ovsext/Switch.h
Log Message:
-----------
datapath-windows: Solved BSOD when uninstalling the driver (race condition)
The BSOD occurred because the FilterAttach routine released the switch
context, while there were IRPs in processing.
The solution was to add a reference count to prevent premature deallocation of the
global switch context structure, gOvsSwitchContext.
Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>
Reported-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
Reported-at: https://github.com/openvswitch/ovs-issues/issues/58
Acked-by: Eitan Eliahu <eliahue at vmware.com>
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Commit: 6625245743a8621a8eee1fd67add15120ed042f8
https://github.com/openvswitch/ovs/commit/6625245743a8621a8eee1fd67add15120ed042f8
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
Log Message:
-----------
tunneling: Include IP TTL in flow metadata.
The IP TTL is currently omitted in the extracted tunnel information
that is stored in the flow for userspace tunneling. This includes it
so that the same logic used by the kernel also applies.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pritesh Kothari <pritesh.kothari at cisco.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 6432e527ce4bc1d67b8fe8ec111817b52953b7c3
https://github.com/openvswitch/ovs/commit/6432e527ce4bc1d67b8fe8ec111817b52953b7c3
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
M tests/tunnel-push-pop.at
Log Message:
-----------
tunneling: Add check for GRE protocol is Ethernet.
On receive, the userspace GRE code doesn't check the protocol
field. Since OVS only understands Ethernet packets, this adds a
check that the inner protocol is Ethernet and discards other types
of packets.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pritesh Kothari <pritesh.kothari at cisco.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: d804d31e24a47d51bf066052c87524ed865f3f4a
https://github.com/openvswitch/ovs/commit/d804d31e24a47d51bf066052c87524ed865f3f4a
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
M lib/odp-util.c
Log Message:
-----------
tunneling: Fix location of GRE checksums.
The GRE checksum is a 16 bit field stored in a 32 bit option (the
rest is reserved). The current code treats the checksum as a 32-bit
field and places it in the right place for little endian systems but
not big endian. This fixes the problem by storing the 16 bit field
directly.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pritesh Kothari <pritesh.kothari at cisco.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 61cf6b70243ee5bb7e948cabf5298a21a66e7c77
https://github.com/openvswitch/ovs/commit/61cf6b70243ee5bb7e948cabf5298a21a66e7c77
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
Log Message:
-----------
tunneling: Use flow flag for GRE checksum calculation.
The indication to calculate the GRE checksum is currently the port
config rather than the tunnel flow. Currently there is a one to one
mapping between the two so there is no difference. However, the
kernel datapath must use the flow and it is also potentially more
flexible, so this switches how we decide whether to calculate the
checksum.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pritesh Kothari <pritesh.kothari at cisco.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: a92a3aae09f6f9c169d65486b9053292844ca5f2
https://github.com/openvswitch/ovs/commit/a92a3aae09f6f9c169d65486b9053292844ca5f2
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/odp-util.c
M tests/odp.at
M tests/tunnel-push-pop.at
Log Message:
-----------
odp-util: Shift VXLAN VNI when printing/parsing.
Currently when printing a userspace tunnel action for VXLAN, the
VNI is treated as a 32 bit field rather than 24 bit. Even if this
is the representation that we use internally, we should still show
the right VNI to avoid confusing people.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pritesh Kothari <pritesh.kothari at cisco.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 83fbb69b50325ad01604d035dbcf6c5fdd61d564
https://github.com/openvswitch/ovs/commit/83fbb69b50325ad01604d035dbcf6c5fdd61d564
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
Log Message:
-----------
vxlan: Set FLOW_TNL_F_KEY for received packets.
The VNI is always present in the VXLAN header, so we should
set the FLOW_TNL_F_KEY flag to indicate this. However, the
userspace implementation of VXLAN currently does not.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Commit: e066f78fea7ba17d293f97c51cd15d69d2c75dbf
https://github.com/openvswitch/ovs/commit/e066f78fea7ba17d293f97c51cd15d69d2c75dbf
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
M lib/odp-util.c
Log Message:
-----------
tunneling: Factor out common UDP tunnel code.
Currently, the userspace VXLAN implementation contains the code
for generating and parsing both the UDP and VXLAN headers. This
pulls out the UDP portion for better layering and to make it
easier to support additional UDP based tunnels and features.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: e5a1caeed45ae4dc8783a2fc1ea466c9e1cc79a7
https://github.com/openvswitch/ovs/commit/e5a1caeed45ae4dc8783a2fc1ea466c9e1cc79a7
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M NEWS
M lib/netdev-vport.c
M lib/odp-util.c
M lib/packets.h
M tests/odp.at
M tests/tunnel-push-pop.at
Log Message:
-----------
tunneling: Add userspace tunnel support for Geneve.
This adds basic userspace dataplane support for the Geneve
tunneling protocol. The rest of userspace only has the ability
to handle Geneve without options and this follows that pattern
for the time being. However, when the rest of userspace is updated
it should be easy to extend the dataplane as well.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 0292a0c9d99f6ec5980bd26a7094fc2a4148ebe8
https://github.com/openvswitch/ovs/commit/0292a0c9d99f6ec5980bd26a7094fc2a4148ebe8
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/packets.c
M lib/packets.h
M tests/library.at
M tests/test-csum.c
Log Message:
-----------
packet: Add IP pseudoheader checksum calculation.
As OVS adds userspace support for being the endpoint in protocols
like tunnels, it will need to be able to calculate pseudoheaders
as part of the checksum calculation.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 8e45fe7c9e20a068a879d8e39cf14c551b9d77ef
https://github.com/openvswitch/ovs/commit/8e45fe7c9e20a068a879d8e39cf14c551b9d77ef
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/netdev-vport.c
M lib/odp-util.c
M tests/odp.at
M tests/tunnel-push-pop.at
Log Message:
-----------
tunneling: Add UDP checksum support for userspace tunnels.
Kernel based OVS recently added the ability to support checksums
for UDP based tunnels (Geneve and VXLAN). This adds similar support
for the userspace datapath to bring feature parity.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 8fc55661c57b26ab84a2baa5ef235f1a79c20633
https://github.com/openvswitch/ovs/commit/8fc55661c57b26ab84a2baa5ef235f1a79c20633
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/packets.h
Log Message:
-----------
packet: Avoid array of struct with zero length member.
Windows doesn't like that the Geneve header has an array of
options with each have a zero length member (the variable data).
Nothing is accessing the data now, so just replace the member with
a comment - we can use pointer arithmetic when necessary.
Reported-by: Gurucharan Shetty <shettyg at nicira.com>
Signed-off-by: Jesse Gross <jesse at nicira.com>
Commit: 46e7137c77d845c488e17b718eac7c3fb97cedcc
https://github.com/openvswitch/ovs/commit/46e7137c77d845c488e17b718eac7c3fb97cedcc
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-07 (Tue, 07 Apr 2015)
Changed paths:
M lib/odp-util.c
Log Message:
-----------
geneve: Zero header before parsing userspace tunneling action.
When we parse the text representation of the Geneve action the
header is not fully initialized. Besides the obvious potential
to generate an action that the user did not actually specify, this
also causes intermittent unit test failures when an action is
read in and printed out and the result is different.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Commit: dcd032186066c5899c8b14394fa91fed26a501d7
https://github.com/openvswitch/ovs/commit/dcd032186066c5899c8b14394fa91fed26a501d7
Author: Flavio Leitner <fbl at redhat.com>
Date: 2015-04-08 (Wed, 08 Apr 2015)
Changed paths:
M tests/automake.mk
Log Message:
-----------
testsuite: work around patch bug
Due to a CVE bug, the patch was modified to not allow following
symlinks out of the workdir. However, this causes a bug when
there is a regular file out of the workdir which breaks the current
testsuite build.
Steps to reproduce:
./boot.sh
mkdir _gcc && cd _gcc
../configure
make
[...]
/bin/sh /home/fleitner/ovs/testsuite/build-aux/missing autom4te
--language=autotest -I '..' -o ../tests/testsuite.tmp
../tests/testsuite.at
patch -p0 ../tests/testsuite.tmp ../tests/testsuite.patch
Invalid file name ../tests/testsuite.tmp -- skipping patch
Makefile:5155: recipe for target '../tests/testsuite' failed
make[2]: *** [../tests/testsuite] Error 1
make[2]: Leaving directory
'/home/fleitner/NetworkingServices/openvswitch/repo/testsuite/_gcc'
Makefile:4087: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
Since it's just a temporary file, this patch changes to use the
workdir as the place to put the file.
Signed-off-by: Flavio Leitner <fbl at redhat.com>
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Commit: 0a46bca787884c914c045ac3838595cbf19e172c
https://github.com/openvswitch/ovs/commit/0a46bca787884c914c045ac3838595cbf19e172c
Author: Nithin Raju <nithin at vmware.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M INSTALL.Windows.md
Log Message:
-----------
INSTALL.Windows: update documentation
The documentation to run the executables has fallen behind quite
a bit. In this patch, we make a few updates. There's more to come.
The patch assumes the user has not run 'make install' or does not
have access to an installer.
Signed-off-by: Nithin Raju <nithin at vmware.com>
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Commit: 5f5ebd4cc8bfede3410fbb5db574693eae8c29e0
https://github.com/openvswitch/ovs/commit/5f5ebd4cc8bfede3410fbb5db574693eae8c29e0
Author: Flavio Leitner <fbl at redhat.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M tests/ofproto-dpif.at
Log Message:
-----------
testsuite: ofproto-dpif: fix test names
Some tests were not included when running the
make TESTSUITEFLAGS="-k ofproto-dpif" check because
the test name was out of the expected pattern.
Signed-off-by: Flavio Leitner <fbl at redhat.com>
Acked-by: Joe Stringer <joestringer at nicira.com>
Commit: d0d808fda035e86602f6e2b4ba414494055e4579
https://github.com/openvswitch/ovs/commit/d0d808fda035e86602f6e2b4ba414494055e4579
Author: Gurucharan Shetty <gshetty at nicira.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M INSTALL.Windows.md
Log Message:
-----------
INSTALL.Windows: Add details about local build and installation.
This commits adds the following.
* Clearly callout to set pthread-win32's dll path in $PATH
* Details about running parallel make and parallel unit tests.
* Installing Open vSwitch locally via 'make install'
* Update the steps to run the user processes by relying on
the previous 'make install'. The steps now run the daemons
in the background with the creation of a logfile and pidfile.
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Commit: 0be55e389d2f12dbbf5dca5c16a7b2c8177e58ff
https://github.com/openvswitch/ovs/commit/0be55e389d2f12dbbf5dca5c16a7b2c8177e58ff
Author: Gurucharan Shetty <gshetty at nicira.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M INSTALL.Windows.md
Log Message:
-----------
INSTALL.Windows: Add documentation about Windows services.
Signed-off-by: Gurucharan Shetty <gshetty at nicira.com>
Acked-by: Nithin Raju <nithin at vmware.com>
Commit: d625fbd13ec3254981df5000bf2587d715401f36
https://github.com/openvswitch/ovs/commit/d625fbd13ec3254981df5000bf2587d715401f36
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M lib/netdev-provider.h
M lib/netdev-vport.c
M lib/netdev.c
Log Message:
-----------
tunneling: Convert tunnel push/pop functions to act on single packets.
The userspace tunneling API for pushing and popping tunnel headers
is currently based on processing batches of packets. However, there
is no obvious way to take advantage of batching for these operations
and so each tunnel operation has a pair of loops to process the
batch. This changes the API to operate on single packets to enable
better code reuse.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: e62d903bca43235e5c92ded0753b1c13096e790a
https://github.com/openvswitch/ovs/commit/e62d903bca43235e5c92ded0753b1c13096e790a
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M lib/netdev.c
Log Message:
-----------
tunneling: Invalid packets should be cleared.
If we receive a packet with an invalid tunnel header, we
should drop the packet without further processing. Currently
we do this by removing any parsed tunnel metadata. However,
this is not sufficient to stop processing - this only results
in the packet getting dropped by chance when something
usually runs across part of the packet that does not make
sense. Since both the packet and its metadata are in an
inconsistent state, it's also possible that the result is
an ovs-vswitchd crash or forwarding of a mangled packet.
Rather than clear the metadata, an alternate solution is to
remove all of the packet data. This guarantees that the
packet gets dropped during the next round of processing.
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: b13c9440d4e5e561ec4df011f1d62fffec5a8a66
https://github.com/openvswitch/ovs/commit/b13c9440d4e5e561ec4df011f1d62fffec5a8a66
Author: Jesse Gross <jesse at nicira.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M datapath/linux/compat/geneve.c
M datapath/linux/compat/vxlan.c
Log Message:
-----------
datapath: Account for "udptunnels: Call handle_offloads after inserting vlan tag."
Upstream commit:
udptunnels: Call handle_offloads after inserting vlan tag.
handle_offloads() calls skb_reset_inner_headers() to store
the layer pointers to the encapsulated packet. However, we
currently push the vlag tag (if there is one) onto the packet
afterwards. This changes the MAC header for the encapsulated
packet but it is not reflected in skb->inner_mac_header, which
breaks GSO and drivers which attempt to use this for encapsulation
offloads.
Fixes: b736a623 ("vxlan: Add tx-vlan offload support.")
Signed-off-by: Jesse Gross <jesse at nicira.com>
Signed-off-by: David S. Miller <davem at davemloft.net>
Upstream: 4aac884d ("udptunnels: Call handle_offloads after inserting vlan tag.")
Signed-off-by: Jesse Gross <jesse at nicira.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>
Commit: 60fc3b7ba4c0dccc9799c7cec6cc77c0d19aa17a
https://github.com/openvswitch/ovs/commit/60fc3b7ba4c0dccc9799c7cec6cc77c0d19aa17a
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M lib/dpif-netdev.c
Log Message:
-----------
dpif-netdev: Group statistics updates in the slow path.
Since statistics updates might require locking (in future commits)
grouping them will reduce the locking overhead.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: eb94da30aeb32cb004c85945a593e3e5c96d0b03
https://github.com/openvswitch/ovs/commit/eb94da30aeb32cb004c85945a593e3e5c96d0b03
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M lib/dpif-netdev.c
Log Message:
-----------
dpif-netdev: Make datapath and flow stats atomic.
A read operation from a non atomic shared value (without external
locking) can return incorrect values. Using the atomic semantics
prevents this from happening.
However:
* No memory barriers are used. We don't need that kind of consistency
for statistics (we use relaxed operations).
* The updates are not atomic, just the loads and stores. This is ok
because there's a single writer.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: abcf3ef4c3f75963f46a11501685363e3ceb7c0e
https://github.com/openvswitch/ovs/commit/abcf3ef4c3f75963f46a11501685363e3ceb7c0e
Author: Daniele Di Proietto <diproiettod at vmware.com>
Date: 2015-04-09 (Thu, 09 Apr 2015)
Changed paths:
M lib/dpif-netdev.c
Log Message:
-----------
dpif-netdev: Count exact match cache hits.
We used to count exact match cache hits and masked classifier hits
together. This commit splits the DP_STAT_HIT counter into two.
This change will be used by future commits.
Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ethan Jackson <ethan at nicira.com>
Commit: 2f86335b8a143a475ab44492ca18927876e36a0c
https://github.com/openvswitch/ovs/commit/2f86335b8a143a475ab44492ca18927876e36a0c
Author: Flavio Leitner <fbl at redhat.com>
Date: 2015-04-10 (Fri, 10 Apr 2015)
Changed paths:
M tests/test-csum.c
Log Message:
-----------
test-csum: endianess fix
The math is done in network byte order so the 32 bits won't
match between big and little endian, so the assert fails.
Reduce to 16 bits and then compare to the converted constant.
Signed-off-by: Flavio Leitner <fbl at redhat.com>
Signed-off-by: Jesse Gross <jesse at nicira.com>
Commit: e8fe6ad03aa3d25b5ae99190d5aa065705a1b3c8
https://github.com/openvswitch/ovs/commit/e8fe6ad03aa3d25b5ae99190d5aa065705a1b3c8
Author: Flavio Leitner <fbl at redhat.com>
Date: 2015-04-10 (Fri, 10 Apr 2015)
Changed paths:
M lib/odp-util.c
M tests/odp.at
M tests/tunnel-push-pop.at
Log Message:
-----------
tests: gre: fix flags endianness
The flags field is 16 bits so use network byte order in the
test case and use the proper conversion methods when parsing
and dumping.
Signed-off-by: Flavio Leitner <fbl at redhat.com>
Signed-off-by: Jesse Gross <jesse at nicira.com>
Commit: daaeeec0bd4348b1f6ac5ca5a302e57bd11cca01
https://github.com/openvswitch/ovs/commit/daaeeec0bd4348b1f6ac5ca5a302e57bd11cca01
Author: Justin Pettit <jpettit at nicira.com>
Date: 2015-04-12 (Sun, 12 Apr 2015)
Changed paths:
M .travis/build.sh
M .travis/prepare.sh
M AUTHORS
A Documentation/automake.mk
A Documentation/group-selection-method-property.txt
M FAQ.md
M INSTALL.DPDK.md
M INSTALL.Windows.md
M INSTALL.md
M INSTALL.userspace.md
M Makefile.am
M NEWS
M acinclude.m4
M build-aux/cccl
M datapath-windows/include/OvsDpInterfaceExt.h
M datapath-windows/ovsext/BufferMgmt.c
M datapath-windows/ovsext/Datapath.c
M datapath-windows/ovsext/Datapath.h
M datapath-windows/ovsext/Event.c
M datapath-windows/ovsext/Flow.c
M datapath-windows/ovsext/IpHelper.c
M datapath-windows/ovsext/Oid.c
M datapath-windows/ovsext/Oid.h
M datapath-windows/ovsext/Switch.c
M datapath-windows/ovsext/Switch.h
M datapath-windows/ovsext/TunnelFilter.c
M datapath-windows/ovsext/TunnelIntf.h
M datapath-windows/ovsext/User.c
M datapath-windows/ovsext/Util.c
M datapath-windows/ovsext/Util.h
M datapath-windows/ovsext/Vport.c
M datapath-windows/ovsext/Vxlan.c
M datapath/Modules.mk
M datapath/compat.h
M datapath/datapath.c
M datapath/linux/Kbuild.in
M datapath/linux/compat/flow_dissector.c
M datapath/linux/compat/genetlink-openvswitch.c
M datapath/linux/compat/geneve.c
M datapath/linux/compat/gre.c
M datapath/linux/compat/include/linux/skbuff.h
M datapath/linux/compat/include/net/genetlink.h
M datapath/linux/compat/include/net/ip.h
M datapath/linux/compat/include/net/udp.h
M datapath/linux/compat/include/net/udp_tunnel.h
M datapath/linux/compat/ip_tunnels_core.c
M datapath/linux/compat/udp.c
M datapath/linux/compat/udp_tunnel.c
M datapath/linux/compat/vxlan.c
M datapath/vport-geneve.c
M datapath/vport-gre.c
M datapath/vport-internal_dev.c
M datapath/vport-lisp.c
M datapath/vport-netdev.c
M datapath/vport-netdev.h
M datapath/vport-vxlan.c
M datapath/vport.c
M datapath/vport.h
M debian/openvswitch-common.install
M debian/openvswitch-switch.install
M include/openflow/automake.mk
A include/openflow/netronome-ext.h
M include/openflow/openflow-common.h
M lib/cfm.c
M lib/dp-packet.c
M lib/dpif-netdev.c
M lib/flow.c
M lib/hmap.h
M lib/jsonrpc.c
M lib/lacp.c
M lib/list.h
M lib/lldp/aa-structs.h
M lib/lldp/lldp-tlv.h
M lib/lldp/lldp.c
M lib/lldp/lldpd-structs.c
M lib/lldp/lldpd.c
M lib/lockfile.c
M lib/mcast-snooping.c
M lib/meta-flow.c
M lib/meta-flow.h
M lib/netdev-bsd.c
M lib/netdev-dpdk.c
M lib/netdev-dummy.c
M lib/netdev-linux.c
M lib/netdev-provider.h
M lib/netdev-vport.c
M lib/netdev.c
M lib/netdev.h
M lib/netlink-socket.c
M lib/nx-match.c
M lib/nx-match.h
M lib/odp-util.c
M lib/ofp-actions.c
M lib/ofp-actions.h
M lib/ofp-parse.c
M lib/ofp-print.c
M lib/ofp-util.c
M lib/ofp-util.h
M lib/ofpbuf.c
M lib/ovs-lldp.c
M lib/ovs-lldp.h
M lib/ovs-numa.c
M lib/ovs-rcu.c
M lib/ovsdb-idl.c
M lib/ovsdb-parser.c
M lib/ovsdb-parser.h
M lib/packets.c
M lib/packets.h
M lib/sset.c
M lib/sset.h
M lib/vconn.c
M ofproto/bond.c
M ofproto/bundles.c
M ofproto/connmgr.c
M ofproto/ofproto-dpif-rid.c
M ofproto/ofproto-dpif-rid.h
M ofproto/ofproto-dpif-upcall.c
M ofproto/ofproto-dpif-xlate.c
M ofproto/ofproto-dpif-xlate.h
M ofproto/ofproto-dpif.c
M ofproto/ofproto-dpif.h
M ofproto/ofproto-provider.h
M ofproto/ofproto.c
M ofproto/tunnel.c
M ovsdb/jsonrpc-server.c
M ovsdb/ovsdb-server.1.in
M rhel/openvswitch-fedora.spec.in
M rhel/openvswitch-kmod-fedora.spec.in
M rhel/openvswitch.spec.in
M tests/automake.mk
M tests/completion.at
A tests/idltest2.ovsschema
M tests/library.at
M tests/mpls-xlate.at
M tests/odp.at
M tests/ofp-print.at
M tests/ofproto-dpif.at
M tests/ofproto.at
M tests/ovsdb-idl.at
M tests/ovsdb-server.at
M tests/test-aa.c
M tests/test-csum.c
M tests/test-list.c
M tests/tunnel-push-pop.at
M tutorial/ovs-sandbox
M utilities/automake.mk
M utilities/ovs-appctl-bashcomp.bash
M utilities/ovs-command-bashcomp.INSTALL.md
M utilities/ovs-dev.py
M utilities/ovs-ofctl.8.in
M utilities/ovs-vsctl-bashcomp.bash
M utilities/ovs-vsctl.c
A utilities/qemu-wrap.py
M vswitchd/automake.mk
M vswitchd/bridge.c
M vswitchd/ovs-vswitchd.c
M vswitchd/vswitch.ovsschema
M vswitchd/vswitch.xml
M vtep/automake.mk
M xenserver/openvswitch-xen.spec.in
Log Message:
-----------
Merge remote-tracking branch 'origin/master' into ovn
Conflicts:
tutorial/ovs-sandbox
Compare: https://github.com/openvswitch/ovs/compare/717c7fc50804...daaeeec0bd43
More information about the git
mailing list