[ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Yi Yang
In ovs compat mode, ovs won't use LWT in current kernel, this is to make sure ovs can work on the old kernels, Linux kernel v4.7 includes VxLAN-gpe support but many Linux distributions' kernels are odler than v4.7, this fix will ensure that ovs can create VxLAN-gpe port correctly on old kernels, it

Re: [ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Jiri Benc
On Thu, 8 Dec 2016 16:20:10 +0800, Yi Yang wrote: > In ovs compat mode, ovs won't use LWT in current kernel, this is to > make sure ovs can work on the old kernels, Linux kernel v4.7 includes > VxLAN-gpe support but many Linux distributions' kernels are odler than > v4.7, this fix will ensure that

[ovs-dev] [PATCH] netdev: Set the default number of queues at removal from the database

2016-12-08 Thread Ilya Maximets
Expected behavior for attribute removal from the database is resetting it to default value. Currently this doesn't work for n_rxq/n_txq options of pmd netdevs (last requested value used): # ovs-vsctl set interface dpdk0 options:n_rxq=4 # ovs-vsctl remove interface dpdk0 options n_r

Re: [ovs-dev] [RFC PATCH] datapath: allow tunnels to be created with rtnetlink

2016-12-08 Thread Jiri Benc
On Wed, 7 Dec 2016 16:35:59 -0800, Pravin Shelar wrote: > In compat mode, OVS tunnel devices are not used in same way as LWT, > since OVS do support kernel version that does not have core LWT > support. Therefore we have to use legacy vport APIs to create these > ports. I see. Yes, that's unfortun

Re: [ovs-dev] [PATCH v2] ovn-trace: Implement DHCP option support.

2016-12-08 Thread Numan Siddique
On Wed, Dec 7, 2016 at 1:49 PM, Numan Siddique wrote: > > > On Tue, Dec 6, 2016 at 4:12 AM, Ben Pfaff wrote: > >> On Sun, Oct 23, 2016 at 02:32:11PM -0700, Ben Pfaff wrote: >> > The put_dhcp_opts(v6) logical action didn't really work because >> ovn-trace >> > didn't handle DHCP options. This fi

Re: [ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Yang, Yi
On Thu, Dec 08, 2016 at 09:54:00AM +0100, Jiri Benc wrote: > On Thu, 8 Dec 2016 16:20:10 +0800, Yi Yang wrote: > > In ovs compat mode, ovs won't use LWT in current kernel, this is to > > make sure ovs can work on the old kernels, Linux kernel v4.7 includes > > VxLAN-gpe support but many Linux dist

Re: [ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Jiri Benc
On Thu, 8 Dec 2016 18:57:51 +0800, Yang, Yi wrote: > So ovs out of tree modules need to adapt to upstream kernel, any > kernel-related changes must be accepted by Linux kernel at first. I'm perfectly aware of that and I'm saying that your patch is unacceptable for upstream kernel. This is a long s

Re: [ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Yang, Yi
On Thu, Dec 08, 2016 at 12:37:56PM +0100, Jiri Benc wrote: > On Thu, 8 Dec 2016 18:57:51 +0800, Yang, Yi wrote: > > So ovs out of tree modules need to adapt to upstream kernel, any > > kernel-related changes must be accepted by Linux kernel at first. > > I'm perfectly aware of that and I'm saying

[ovs-dev] [PATCH V2] netdev-dpdk: fix ifindex assignment for DPDK ports

2016-12-08 Thread Przemyslaw Lal
In current implementation port_id is used as an ifindex for all netdev-dpdk interfaces. For physical DPDK interfaces using port_id as ifindex causes that '0' is set as ifindex for 'dpdk0' interface, '1' for 'dpdk1' and so on. For the DPDK vHost interfaces ifindexes are not even assigned (0 is used

[ovs-dev] [PATCH v2] sflow: Expose ethernet stats via sFlow

2016-12-08 Thread Robert Wojciechowicz
Expose existing netdev stats via sFlow. Export sFlow ETHERNET structure with available counters. Map existing stats to counters in the GENERIC INTERFACE sFlow structure. Adjust unit test to accommodate these new counters. Signed-off-by: Robert Wojciechowicz --- v2: - remove VLAN counters --- ofp

[ovs-dev] [PATCH v2 02/11] doc: Create directory structure

2016-12-08 Thread Stephen Finucane
Create a series of sections, all of which are currently empty, using the general design established by Jacob Kaplan-Moss and the Django project [1]. Five sections are provided: - intro - tutorials - topics - howto - ref - faq - internals The purpose of each section is described in the documents t

[ovs-dev] [PATCH v2 00/11] Sphinx-ification of documentation

2016-12-08 Thread Stephen Finucane
This series is the first in a number of planned series that rework and expand the OVS documentation. This particular series aims to achieve the following: * Configure the basics required for Sphinx documentation generation * Create a number of document sections and move all documents, with the

[ovs-dev] [PATCH v2 01/11] Add initial sphinx configuration

2016-12-08 Thread Stephen Finucane
This is essentially the output of 'sphinx-quickstart' but with the following changes: - Parts of the generated Makefile are merged into the existing Documentation/automake.mk Makefile - A license is added to the index.rst file - The OVS logo is added - A 'contents' page is added, so we don't nee

[ovs-dev] [PATCH v2 05/11] doc: Populate 'install', 'howto' sections

2016-12-08 Thread Stephen Finucane
This is a dumb move of all 'INSTALL*' docs, with very little refactoring (mostly updating links and making the titles a little more consistent. Additional refactoring will be done in subsequent changes. Signed-off-by: Stephen Finucane --- Documentation/automake.mk | 16 +

[ovs-dev] [PATCH v2 06/11] doc: Further populate the 'howto' section

2016-12-08 Thread Stephen Finucane
There are a couple of references to these for various build systems. The website is going to be our "one true resource" for all docs, so simply remove these references. Signed-off-by: Stephen Finucane --- Documentation/automake.mk | 4 Documentation/howto/i

[ovs-dev] [PATCH v2 03/11] doc: Populate 'ref' section

2016-12-08 Thread Stephen Finucane
This is a simple table with links to the manpages published on openvswitch.org. Something fancier can be done in the future. Signed-off-by: Stephen Finucane --- Documentation/ref/index.rst | 144 +++- 1 file changed, 143 insertions(+), 1 deletion(-) diff

[ovs-dev] [PATCH v2 09/11] doc: Move WHY-OVS

2016-12-08 Thread Stephen Finucane
This is moved separately due to the sheer number of references to this file in the codebase. Signed-off-by: Stephen Finucane --- Documentation/automake.mk | 1 + Documentation/intro/index.rst | 1 + WHY-OVS.rst => Documentation/intro/why-ovs.rst | 0 FAQ.rst

[ovs-dev] [PATCH v2 07/11] doc: Populate 'topics' section

2016-12-08 Thread Stephen Finucane
There are many docs that don't need to kept at the top level, along with many more hidden in random folders. Move them all. This also allows us to add the '-W' flag to Sphinx, ensuring unindexed docs result in build failures. Signed-off-by: Stephen Finucane --- Documentation/automake.mk

[ovs-dev] [PATCH v2 08/11] doc: Populate 'tutorials' section

2016-12-08 Thread Stephen Finucane
Rename 'tutorial' to 'ovs-advanced' and 'ovn-tutorial' to 'ovn-basics'. Signed-off-by: Stephen Finucane --- Documentation/automake.mk| 2 ++ Documentation/index.rst | 3 ++- Documentation/tutorials/index.rst

[ovs-dev] [PATCH v2 04/11] doc: Populate 'internals' section

2016-12-08 Thread Stephen Finucane
This is mostly docs moved from the top-level directory and content scraped from the Open vSwitch website source [1]. [1] https://github.com/openvswitch/openvswitch.github.io/ Signed-off-by: Stephen Finucane --- CONTRIBUTING.rst | 430 +--- Docum

[ovs-dev] [PATCH v2 11/11] doc: Remove documentation from distdoc target

2016-12-08 Thread Stephen Finucane
Basic Sphinx integration is now complete. Remove the documentation aspects of the 'dist-docs' target in favor of the htmldocs target. Signed-off-by: Stephen Finucane --- Documentation/automake.mk | 4 +--- Makefile.am | 7 +-- build-aux/dist-docs | 50 +-

Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-08 Thread Numan Siddique
On Thu, Dec 1, 2016 at 9:07 PM, Lance Richardson wrote: > Add support for SSL connections to OVN northbound and/or > southbound databases. > > To improve security, the NB and SB ovsdb daemons no longer > have open ptcp connections by default. This is a change in > behavior from previous versions

Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-08 Thread Lance Richardson
> From: "Numan Siddique" > To: "Lance Richardson" > Cc: "ovs dev" > Sent: Thursday, December 8, 2016 8:01:07 AM > Subject: Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db > connections > > On Thu, Dec 1, 2016 at 9:07 PM, Lance Richardson > wrote: > > > Add support for SSL con

Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-08 Thread Russell Bryant
On Thu, Dec 8, 2016 at 8:37 AM, Lance Richardson wrote: > > From: "Numan Siddique" > > To: "Lance Richardson" > > Cc: "ovs dev" > > Sent: Thursday, December 8, 2016 8:01:07 AM > > Subject: Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db > connections > > > > On Thu, Dec 1, 2016

Re: [ovs-dev] [PATCH 3/3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-08 Thread Lance Richardson
> From: "Russell Bryant" > What would you think of changing the DEFAULT_REMOTE option to be > CREATE_REMOTE to more clearly indicate that the option is telling ovn-ctl > that we want it to go ahead and set up a remote for us. > Sounds good, I think CREATE_REMOTE is a better name. > When CREATE

[ovs-dev] Las 5´s de la Calidad total Japonesa

2016-12-08 Thread cómo organizar todas las áreas de su compañía
En línea y en Vivo / Para todo su Equipo con una sola Conexión Las 5´s de la CALIDAD TOTAL JAPONESA Y cómo ponerlo en marcha HOY en su compañía 15 de diciembre - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00 Hrs Conozca las cinco reglas japonesas que revolucionaron el concepto de

[ovs-dev] [PATCH] datapath-windows: Remove dead code from PacketIO

2016-12-08 Thread Alin Serdean
Assigning value to 'nativeNbls' has no effect outside the function and the variable is not used inside the function. Signed-off-by: Alin Gabriel Serdean --- datapath-windows/ovsext/PacketIO.c | 1 - 1 file changed, 1 deletion(-) diff --git a/datapath-windows/ovsext/PacketIO.c b/datapath-window

[ovs-dev] [PATCH v2 0/3] ovn: support ssl connections to nb/sb dbs

2016-12-08 Thread Lance Richardson
This series adds support for SSL connections to the northbound and southbound OVN database servers and removes the previous default TCP connection type. v2: - Changed DB_NB_DEFAULT_REMOTE to DB_NB_CREATE_REMOTE. - Changed DB_SB_DEFAULT_REMOTE to DB_SB_CREATE_REMOTE. - Create default remot

[ovs-dev] [PATCH v2 1/3] ovn-nb: remote connection management in nb db

2016-12-08 Thread Lance Richardson
Add support for managing remote connections, including SSL configuration, to northbound db schema, and add necessary commands to ovn-nbctl. Signed-off-by: Lance Richardson --- NEWS | 2 + ovn/ovn-nb.ovsschema | 53 +++- ovn/ovn-nb.xml| 288

[ovs-dev] [PATCH v2 2/3] ovn-sb: remote connection management in sb db

2016-12-08 Thread Lance Richardson
Add support for managing remote connections, including SSL configuration, to southbound db schema, and add necessary commands to ovn-sbctl. Signed-off-by: Lance Richardson --- NEWS | 2 +- manpages.mk | 6 ++ ovn/ovn-sb.ovsschema | 21 +++- o

[ovs-dev] [PATCH v2 3/3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-08 Thread Lance Richardson
Add support for SSL connections to OVN northbound and/or southbound databases. To improve security, the NB and SB ovsdb daemons no longer have open ptcp connections by default. This is a change in behavior from previous versions, users wishing to use TCP connections to the NB/SB daemons can eithe

[ovs-dev] [PATCH] treewide: Fix recent flake8-check.

2016-12-08 Thread Joe Stringer
Without this patch, I see errors like this on master: ../ofproto/ipfix-gen-entities:115:1: E305 expected 2 blank lines after class or function definition, found 1 Signed-off-by: Joe Stringer --- ofproto/ipfix-gen-entities | 1 + utilities/ovs-pcap.in | 3 +++ utilities/ovs-tcpdump.in | 1

Re: [ovs-dev] [PATCH v2 0/3] ovn: support ssl connections to nb/sb dbs

2016-12-08 Thread Lance Richardson
> From: "Lance Richardson" > To: d...@openvswitch.org, b...@ovn.org, russ...@ovn.org, nusid...@redhat.com > Sent: Thursday, December 8, 2016 1:12:22 PM > Subject: [ovs-dev] [PATCH v2 0/3] ovn: support ssl connections to nb/sb dbs > > This series adds support for SSL connections to the northbound

[ovs-dev] [PATCH] datapath-windows: Conntrack move null checks to functions

2016-12-08 Thread Alin Serdean
Add null checks inside OvsConntrackValidateIcmpPacket, OvsConntrackValidateTcpPacket to make the functions self-contained. Signed-off-by: Alin Gabriel Serdean Suggested-by: Yin Lin --- datapath-windows/ovsext/Conntrack-icmp.c | 4 datapath-windows/ovsext/Conntrack-tcp.c | 6 +- datapa

Re: [ovs-dev] [PATCH] treewide: Fix recent flake8-check.

2016-12-08 Thread Daniele Di Proietto
2016-12-08 10:22 GMT-08:00 Joe Stringer : > Without this patch, I see errors like this on master: > ../ofproto/ipfix-gen-entities:115:1: E305 expected 2 blank lines after > class or function definition, found 1 > > Signed-off-by: Joe Stringer Interesting, I pushed a related fix a few days ago, an

Re: [ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Pravin Shelar
On Thu, Dec 8, 2016 at 12:20 AM, Yi Yang wrote: > In ovs compat mode, ovs won't use LWT in current kernel, this is to > make sure ovs can work on the old kernels, Linux kernel v4.7 includes > VxLAN-gpe support but many Linux distributions' kernels are odler than > v4.7, this fix will ensure that o

Re: [ovs-dev] [PATCH] netdev: Set the default number of queues at removal from the database

2016-12-08 Thread Stokes, Ian
> Expected behavior for attribute removal from the database is resetting it > to default value. Currently this doesn't work for n_rxq/n_txq options of > pmd netdevs (last requested value used): > > # ovs-vsctl set interface dpdk0 options:n_rxq=4 > # ovs-vsctl remove interface dpdk0 opt

Re: [ovs-dev] [RFC PATCH] datapath: allow tunnels to be created with rtnetlink

2016-12-08 Thread Eric Garver
On Wed, Dec 07, 2016 at 08:47:56AM +0800, Yang, Yi wrote: > On Tue, Dec 06, 2016 at 09:38:09AM -0500, Eric Garver wrote: > > On Tue, Dec 06, 2016 at 07:17:20AM +, Yang, Yi Y wrote: > > > Hi, guys > > > > Hi Yi, > > > > > This patch isn't updated from June on, Cascardo said he/Eric is still >

Re: [ovs-dev] [PATCH] ovn: fix OVNDB process is stopped when master node demote to the slave by pacemaker

2016-12-08 Thread Andy Zhou
On Wed, Dec 7, 2016 at 11:42 PM, Guoshuai Li wrote: > > On 2016/12/8 5:36, Andy Zhou wrote: > > > > On Tue, Dec 6, 2016 at 9:41 PM, Guoshuai Li wrote: > >> When the master node's OVNDB process fails, the local node demote to the >> slave. >> Failure cause is that the OVNDB process is stop, So th

[ovs-dev] [PATCH] datapath-windows: Fix issues related to packet completion

2016-12-08 Thread Shashank Ram
- In OvsTunnelPortTx() function, for packets coming from the VIF port, the srcVportNo, srcPortId and srcNicIndex were getting modified for the original NBL prior to creation of newNbl. This is not correct since modifying the original packet's forwarding detail can cause completion issues.

Re: [ovs-dev] [PATCH] treewide: Fix recent flake8-check.

2016-12-08 Thread Joe Stringer
On 8 December 2016 at 11:32, Daniele Di Proietto wrote: > 2016-12-08 10:22 GMT-08:00 Joe Stringer : >> Without this patch, I see errors like this on master: >> ../ofproto/ipfix-gen-entities:115:1: E305 expected 2 blank lines after >> class or function definition, found 1 >> >> Signed-off-by: Joe S

Re: [ovs-dev] OVS-Hyper-V-Discovery-Agent Design Document

2016-12-08 Thread Yin Lin
Hi Alin, Sorry for the late response. I somehow that the message slip by. How do you use vswitchd/netlink to watch VIF creation/deletion and Hyper-V switch changes? Note that through WMI, we monitor the following events: 1. VIF creation/deletion. 2. Hyper-V switch creation/deletion. 3. OVS extens

[ovs-dev] [PATCH v2] Windows: Implement Hyper-V VIF discovery agent.

2016-12-08 Thread Yin Lin
Add Makefile changes to build the daemon from console. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH] datapath-windows: Conntrack move null checks to functions

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/8/16, 10:44 AM, "Alin Serdean" wrote: >Add null checks inside OvsConntrackValidateIcmpPacket, >OvsConntrackValidateTcpPacket to make the functions self-contained. > >Signed-off-by: Alin Gabriel Serdean >Suggested-by: Yin Lin >--- > datapath-windows/ovsext/Con

Re: [ovs-dev] [PATCH] windows: Change driver and MSI company name to LF

2016-12-08 Thread Sairam Venugopal
+Ben Do we need to make this change now? I would rather wait until we have commitment from Linux Foundation to sign the MSI/Driver. Without that we will be advertising the MSI and Driver as manufactured by Linux Foundation instead of Open vSwitch. Thanks, Sairam On 12/6/16, 3:30 PM, "Alin Serd

Re: [ovs-dev] [PATCH 1/5] datapath-windows: Ignore VStudio 2015 temporary files

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 7:39 AM, "Alin Serdean" wrote: >Ignore the temporary files created by Visual Studio 2015 when opening >a solution. > >Signed-off-by: Alin Gabriel Serdean >--- > datapath-windows/.gitignore | 2 ++ > 1 file changed, 2 insertions(+) > >diff --git a/datapath

Re: [ovs-dev] [PATCH 2/5] datapath-windows: Add Windows 10 family to solution

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 7:39 AM, "Alin Serdean" wrote: >This patch adds two more compiling targets: > - one for Windows 10 release > - one for Windows 10 Debug > >The new targets are flagged properly to use the new Windows 10 kernel mode >driver and its toolchain. > >Signed-of

Re: [ovs-dev] [PATCH 3/5] datapath-windows: Remove unused section from driver inf

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 7:39 AM, "Alin Serdean" wrote: >The new tool chain for Windows 10 driver contains a inf file checker. >While compiling it found the following issue: > - Common.Params.reg is missing. > >This patch removes the entry since it is not used, and at the moment >

Re: [ovs-dev] [PATCH 4/5] datapath-windows: Fix function prototypes

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 7:39 AM, "Alin Serdean" wrote: >There is a mismatch between OvsInitCompletionList and >OvsAddPktCompletionList >prototypes. > >Eg: >https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_openvswitc >h_ovs_blob_master_datapath-2Dwindows_ovsext_Pac

Re: [ovs-dev] [PATCH 5/5] datapath-windows: Conntrack disable type truncation warning

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 7:39 AM, "Alin Serdean" wrote: >Compiling with the WDK 10 gave the following warning: >WarningC4311 'type cast': pointer truncation from 'POVS_CT_ENTRY' to >'UINT32' >ovsext (OVSExt\ovsext) Conntrack.c 1139 > >This patch disables the warnin

Re: [ovs-dev] [PATCH 2/3 v6] datapath-windows: Add multiple switch internal ports

2016-12-08 Thread Sairam Venugopal
+Shashank Ram Alin - I think the other patch - https://patchwork.ozlabs.org/patch/697999/ should probably be merged instead. The issue is with the current state is that the routine returns without waiting. While your patch addresses that issue, it¹s not the right fix. Your change makes it wait fo

Re: [ovs-dev] [PATCH 1/4] datapath-windows: Force driver version to depend on a variable

2016-12-08 Thread Sairam Venugopal
Thanks for sending this out. On 12/5/16, 6:58 PM, "Alin Serdean" wrote: >The following components use Windows driver information: >- System (inf file); used during device installation >- Resource file (rc file); used by applications when looking over the >driver >file(sys) > >Curr

Re: [ovs-dev] [PATCH 1/4] datapath-windows: Force driver version to depend on a variable

2016-12-08 Thread Sairam Venugopal
Thanks for sending this out. Acked-by: Sairam Venugopal On 12/5/16, 6:58 PM, "Alin Serdean" wrote: >The following components use Windows driver information: >- System (inf file); used during device installation >- Resource file (rc file); used by applications when looking over the

Re: [ovs-dev] [PATCH 2/4] build-windows: Propagate PACKAGE_VERSION to the driver files

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 6:58 PM, "Alin Serdean" wrote: >This patch propagates the automake value 'PACKAGE_VERSION' to the driver >specific information files, overwriting the Visual Studio default value of >Version, when building the driver via msys. > >Signed-off-by: Alin Gabriel

Re: [ovs-dev] [PATCH 3/4] msi-windows: Add version variable

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 6:58 PM, "Alin Serdean" wrote: >Currently we have a predefined variable for the product version. > >This patch adds a new variable called 'Version' to the MSI project and >uses it for the product version. > >If the variable is not specified via the comman

Re: [ovs-dev] [PATCH 4/4] build-windows: Propagate PACKAGE_VERSION to the MSI

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/5/16, 6:58 PM, "Alin Serdean" wrote: >This patch propagates the automake variable PACKAGE_VERSION when building >the MSI via msys. > >Signed-off-by: Alin Gabriel Serdean >--- > windows/automake.mk | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >diff

Re: [ovs-dev] [PATCH] datapath-windows: Remove dead code from PacketIO

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/8/16, 9:57 AM, "Alin Serdean" wrote: >Assigning value to 'nativeNbls' has no effect outside the function and >the variable is not used inside the function. > >Signed-off-by: Alin Gabriel Serdean >--- > datapath-windows/ovsext/PacketIO.c | 1 - > 1 file changed,

Re: [ovs-dev] [PATCH] datapath-windows: Fix issues related to packet completion

2016-12-08 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 12/8/16, 1:31 PM, "Shashank Ram" wrote: >- In OvsTunnelPortTx() function, for packets coming from the > VIF port, the srcVportNo, srcPortId and srcNicIndex were > getting modified for the original NBL prior to creation > of newNbl. This is not correct since mod

[ovs-dev] Administración de Sueldos

2016-12-08 Thread Programas de Compensación
En línea y en Vivo / Para todo su Equipo con una sola Conexión Administración de SUELDOS y Programas de COMPENSACIÓN 14 de diciembre - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00 Hrs Mantenga salarios competitivos sin dañar la economía de su empresa. Las presiones económicas y

Re: [ovs-dev] [PATCH net-next] openvswitch: fix VxLAN-gpe port can't be created in ovs compat mode

2016-12-08 Thread Yang, Yi
On Thu, Dec 08, 2016 at 11:41:58AM -0800, Pravin Shelar wrote: > On Thu, Dec 8, 2016 at 12:20 AM, Yi Yang wrote: > > > > Signed-off-by: Yi Yang > > --- > > include/uapi/linux/openvswitch.h | 1 + > > net/openvswitch/vport-vxlan.c| 15 +++ > > 2 files changed, 16 insertions(+) >

Re: [ovs-dev] [RFC PATCH] datapath: allow tunnels to be created with rtnetlink

2016-12-08 Thread Yang, Yi
On Thu, Dec 08, 2016 at 03:24:35PM -0500, Eric Garver wrote: > On Wed, Dec 07, 2016 at 08:47:56AM +0800, Yang, Yi wrote: > > > > I notice if we fallback to ovs compat modules to create vxlan, it will > > use generic netlink but not rtnetlink, do you meam you're changing > > generic netlink in func

[ovs-dev] [PATCH] rpms: Remove more OVN files form openvswitch rpm builds

2016-12-08 Thread Andy Zhou
OVN is packaged with openvswitch-fedora.spec.in, but not with openvswitch.spec.in. Remove OVN files from openvswitch.spec.in builds to make rpm build happy. Signed-off-by: Andy Zhou --- rhel/openvswitch.spec.in | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/rhel/openvswitc

[ovs-dev] Did you get my message this time?

2016-12-08 Thread Sir friedrich mayhofer.....
This is the second time i am sending you this mail. I, Friedrich Mayrhofer Donate $ 1,000,000.00 to You, Email  Me personally for more details. Regards. Friedrich Mayrhofer ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailma

[ovs-dev] I Hope You Get My Message This Time

2016-12-08 Thread Friedrich Mayrhofer
This is the second time i am sending you this mail. I, Friedrich Mayrhofer Donate $ 1,000,000.00 to You, Email  Me personally for more details. Regards. Friedrich Mayrhofer ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailma

[ovs-dev] [PATCH 1/3] datapath: compat: Fix build on RHEL 7.3

2016-12-08 Thread Yi-Hung Wei
RHEL 7.3 provides upstream tunnel but it does not support name_assign_type attribute in net-device. This patch fixes the build problem by backporting functions with name_assign_type, and using proper flags in acinclude.m4 to invoke backport functions. Tested on RHEL 7.3 with kernel 3.10.0-514.el7.

[ovs-dev] [PATCH 2/3] system-traffic: Skip test cases if netcat is not installed.

2016-12-08 Thread Yi-Hung Wei
Test cases that use netcat will fail if netcat is not installed. This patch detects if netcat is present, and skips those test cases if netcat is not there. Singed-off-by: Yi-Hung Wei --- tests/atlocal.in| 17 + tests/system-traffic.at | 5 + 2 files changed, 22 inse

[ovs-dev] [PATCH 3/3] system-traffic: Skip test cases if firewalld is on.

2016-12-08 Thread Yi-Hung Wei
On RHEL 7.3, test cases that use vxlan, gre, and geneve tunnels fail because traffic is blocked by default firewall configuration. This commit detects the status of firewalld, and skips the tests if firewalld is on. Signed-off-by: Yi-Hung Wei --- tests/system-common-macros.at | 16 +-

[ovs-dev] [PATCH 0/3] Fix testsuite failures on big endian platforms.

2016-12-08 Thread Daniele Di Proietto
openvswitch branch-2.6 fails to build on Debian on big endian platforms, as reported in https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=840770. This series aims to fix some endianness related failures: 1168: ofproto-dpif.at:8503 ofproto-dpif - conntrack - ICMP related 1174: ofproto-dpif.at:87

[ovs-dev] [PATCH 1/3] csum: Fix csum_continue() on big endian with an odd number of bytes.

2016-12-08 Thread Daniele Di Proietto
Even though it reads 16 bits at a time, csum_continue() is almost neutral to endianness (see RFC 1071 1.2 (B), "Byte Order Independence"). Consider a buffer like the following: : XX YY XX YY XX YY XX YY ZZ Each couple of bytes is interpreted on little endian as: *data = 0xYYXX while on

[ovs-dev] [PATCH 2/3] ovn: Encode dhcpv6 PACKET_IN userdata as big endian.

2016-12-08 Thread Daniele Di Proietto
The packet in userdata generated by ovn-controller when translating the put_dhcpv6_opt action includes 16-bit integers. Currently these 16-bit integers are encoded with native endianness. This is ok becase the only consumer of that userdata is ovn-controller itself, but it means that the OpenFlow

[ovs-dev] [PATCH 3/3] tests: Ignore options order in dhcpv4 ovn test.

2016-12-08 Thread Daniele Di Proietto
The order of the options in the packet generated by ovs-controller depends on the hash function. I believe that murmur hash (our default) produces different outputs depending on the endianness of the system. This commit fixes the test by reordering the options in the packet before checking them.

Re: [ovs-dev] [PATCH] netdev-dpdk: Add vHost User PMD

2016-12-08 Thread Daniele Di Proietto
2016-12-06 7:10 GMT-08:00 Loftus, Ciara : >> >> Thanks for the patch. >> >> I experience a crash with this patch applied by starting ovs and >> immediately adding a vhostuserclient port. It's not reproducible 100% >> of the times. >> >> Program received signal SIGSEGV, Segmentation fault. >> rte_e

[ovs-dev] [PATCH] python: Currently stream supported SSL

2016-12-08 Thread Guoshuai Li
Signed-off-by: Guoshuai Li --- python/ovs/stream.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/ovs/stream.py b/python/ovs/stream.py index b43e105..d81bedb 100644 --- a/python/ovs/stream.py +++ b/python/ovs/stream.py @@ -104,7 +104,7 @@ class Stream(object):

Re: [ovs-dev] dev The world has opened up to me in several ways

2016-12-08 Thread Kerry
you are awesome really This drug is constructed to offer you a increase of energyhttps://shar.es/18xm2nmy self-confidence level has truly been through the roof ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-d

Re: [ovs-dev] [PATCH] datapath: Linux 4.9 compat.

2016-12-08 Thread Andy Zhou
On Wed, Dec 7, 2016 at 4:31 PM, Jarno Rajahalme wrote: > This patch allows openvswitch kernel module in the OVS tree to be > compiled against the current net-next Linux kernel. The changes are > due to these upstream commits: > > 56989f6d856 ("genetlink: mark families as __ro_after_init") > 4891

[ovs-dev] [PATCH] datapath: enable vxlangpe creation in compat mode

2016-12-08 Thread Yi Yang
Current out-of-tree ovs vxlan module has included vxlangpe code, but it can't create vxlangpe port, this patch enables out-of-tree ovs vxlan module to create vxlangpe port without Linux kernel v4.7 or above needed, this can ensure we can create vxlangpe port on any Linux system as long as Linux ker

Re: [ovs-dev] [RFC PATCH] datapath: allow tunnels to be created with rtnetlink

2016-12-08 Thread Pravin Shelar
On Thu, Dec 8, 2016 at 1:14 AM, Jiri Benc wrote: > On Wed, 7 Dec 2016 16:35:59 -0800, Pravin Shelar wrote: >> In compat mode, OVS tunnel devices are not used in same way as LWT, >> since OVS do support kernel version that does not have core LWT >> support. Therefore we have to use legacy vport API