[ovs-dev] Estimado usuario

Estimado usuario Actualmente estamos corriendo nuestro servidor actualización de verificación, para mejorar la eficiencia y eliminar cuentas que ya no están activas. Por favor ingrese sus datos a continuación para verificar y actualizar su cuenta: (1) Correo electrónico: (2) Nombre: (3) Contrase

[ovs-dev] HUMBLE BUSINESS ASSISTANCE!

___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

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

On Thu, 8 Dec 2016 22:49:56 -0800, Pravin Shelar wrote: > OVS out of tree kernel module is using compat tunnel code upto kernel > 4.5 kernel even thought LWT is available in these kernels. This is due > to missing features on these kernel which are backported to OVS > module. In future we could bum

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

Hi Jarno, 2016-12-03, Jarno Rajahalme: I sent a patch to fix this on OVS master. After it is reviewed I’ll backport it to branch-2.5 (it only needs trivial fixes to apply, so you may want to try that as well). I tested the patch (on branch-2.5) and have mixed results. In the same setup as th

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

On Fri, Dec 9, 2016 at 8:20 AM, Daniele Di Proietto wrote: > 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 t

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

2016-12-09, Thomas Morin: In the same setup as the one on which the bug was observed, [...] I was confused, I in fact tested the patch (branch-2.5) on a /different/ setup as the one on which we hit the bug, using MPLS over a GRE tunnel port, rather than plain MPLS over an eth port. Sorry if

[ovs-dev] [PATCH 0/2] dpdk: fix memory release issues

The following series is composed of a PATCH which corrects both a leak and a double free, as well as an RFC/PATCH which aims to remove the deferred release for DPDK. There's a bit of confusion with respect to how DPDK uses the argc/argv parameters. I've submitted a patch to that list (found here:

[ovs-dev] [PATCH 1/2] lib/dpdk: fix double free on exit

The DPDK EAL library intents that all argc/argv arguments passed on the command line will be in the form: progname dpdk arguments program arguments This means the argv array will look something like: argv[0] = progname argv[1..x] = dpdk arguments argv[x..y] = program arguments When

[ovs-dev] [RFC-PATCH 2/2] lib/dpdk: No more deferred release

DPDK documentation is recently updated to reflect that DPDK does not hold any references to, nor take ownership of, the argv/argc elements. With that understanding, let's just release the memory asap. Signed-off-by: Aaron Conole --- NOTE: if the patch at http://dpdk.org/ml/archives/dev/2016

[ovs-dev] Open vSwitch 2.7 release dates and process

Documentation/release-process.rst generally lays out the following dates for the Open vSwitch 2.7 release (adjusted slightly for better alignment to workweeks): Jan. 2, 2017: Fork branch-2.7 from master. Feb. 17, 2017: Release branch-2.7 as version 2.7.0. However, after our positive expe

Re: [ovs-dev] [patch_v10] ovn: Add datapaths of interest filtering.

From: Mickey Spiegel Date: Wednesday, December 7, 2016 at 3:50 PM To: Darrell Ball Cc: Darrell Ball , ovs dev Subject: Re: [ovs-dev] [patch_v10] ovn: Add datapaths of interest filtering. On Tue, Dec 6, 2016 at 7:53 PM, Darrell Ball mailto:db...@vmware.com>> wrote: On 12/4/16, 9:48 PM, "o

[ovs-dev] Normas de Información Financiera (NIF)

En línea y en Vivo / Para todo su Equipo con una sola Conexión Normas de Información Financiera (NIF) CONTROLES INTERNOS 16 de diciembre - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00 Hrs Tome el verdadero control de su empresa. Aprenda la manera óptima de regular, registrar y

[ovs-dev] [patch_v13] ovn: Add datapaths of interest filtering.

This patch adds datapaths of interest support where only datapaths of local interest are monitored by the ovn-controller ovsdb client. The idea is to do a flood fill in ovn-controller of datapath associations calculated by northd. A new column is added to the SB database datapath_binding table - r

[ovs-dev] [PATCH 1/2] ovn-sbctl: Fix a few minor typesetting errors.

In manpages, \- is used primarily for literal "-" in literal text, that is, it should usually be used for bold (literal) text. (It's also used for minus signs in mathematics.) It should not generally be used for hyphens in English text, as it was used here in italic (metasyntactic variable) text.

[ovs-dev] [PATCH 2/2] ovn-controller: Tie OpenFlow and logical flows using OpenFlow cookie.

This makes it easy to find the logical flow that generated a particular OpenFlow flow, by running "ovn-sbctl dump-flows ". Later, this can be refined (and automated for "ofproto/trace"), but this is still a significant advance. Signed-off-by: Ben Pfaff --- Justin, I know that you had some ideas

Re: [ovs-dev] [PATCH 1/4] datapath-windows: Conntrack - Fix OvsGetTcpPayloadLength()

Acked-by: Shashank Ram From: Sairam Venugopal Sent: Friday, December 9, 2016 10:48:23 AM To: Shashank Ram Cc: Sairam Venugopal Subject: [PATCH 1/4] datapath-windows: Conntrack - Fix OvsGetTcpPayloadLength() Move the OvsGetTcpPayloadLength() to common header. Upd

Re: [ovs-dev] [PATCH 1/4] datapath-windows: Conntrack - Fix OvsGetTcpPayloadLength()

I will look into using it in a different patch. My main concern is with OvsParseTcp¹s usage of the POVS_PACKET_HDR_INFO. This field is not passed around and there are cases where the POVS_PACKET_HDR_INFO isn¹t set completely. Eg: l4Payload is set only in User.c context and not in PacketParser.c T

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

Acked-by: Jarno Rajahalme > On Dec 8, 2016, at 6:50 PM, Daniele Di Proietto > wrote: > > 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

Re: [ovs-dev] [PATCH 1/2] ovn-sbctl: Fix a few minor typesetting errors.

On Fri, Dec 9, 2016 at 1:38 PM, Ben Pfaff wrote: > In manpages, \- is used primarily for literal "-" in literal text, that is, > it should usually be used for bold (literal) text. (It's also used for > minus signs in mathematics.) It should not generally be used for hyphens > in English text, a

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

On Thu, Dec 8, 2016 at 8:31 PM, Andy Zhou wrote: > 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 > Acked-by: Russell Bryant but 1 minor suggestion ..

Re: [ovs-dev] [RFC] ovn: Securing from compromised host - removal of MAC_Binding

On Mon, Dec 5, 2016 at 4:31 PM, Ben Pfaff wrote: > On Fri, Nov 18, 2016 at 10:13:23AM +0530, Babu Shanmugam wrote: > > 2) When the destination for an ARP reply is an OVN distributed router, > > broadcast the response to all instances of the logical router so that the > > result is available in th

[ovs-dev] [PATCH v2 0/4] datapath-windows: Enable support for tracking FTP connections

Add support for maintaining and tracking related connections. This patch introduces the concept of related-connections table. There is an FTP parser in place to parse FTP PASV and PORT commands. Support for traking extended FTP commands will be added in subsequently. v2: Addressed Alin's and Sh

[ovs-dev] [PATCH v2 3/4] datapath-windows: Conntrack - Introduce support for tracking related connections

Introduce a new table to track related connections. This table will be used to track FTP data connections based on the control connection. There is a new Conntrack-ftp.c to parse incoming FTP messages to determine the related data ports. It creates a new entry in the related connections tracker tab

[ovs-dev] [PATCH v2 2/4] datapath-windows: Cleanup Conntrack definitions and introduce related entries

Consolidate the reusable structs and includes. Introduce the new OVS_CT_REL_ENTRY to track related connections. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Conntrack.c | 12 - datapath-windows/ovsext/Conntrack.h | 51 ++--- 2 files changed,

[ovs-dev] [PATCH v2 4/4] datapath-windows: Conntrack - Enable FTP support

Enable the support for tracking FTP connections in the Connection tracker. This checks an incoming ftp control connection to extract the related data connection. When a matching data connection arrives, it would then update the connection entry to point to the original control connection. Signed-o

[ovs-dev] [PATCH v2 1/4] datapath-windows: Conntrack - Fix OvsGetTcpPayloadLength()

Move the OvsGetTcpPayloadLength() to common header. Update the code to check for null references and the correct size of the TCP header. Signed-off-by: Sairam Venugopal --- datapath-windows/ovsext/Conntrack-tcp.c | 15 --- datapath-windows/ovsext/Conntrack.h | 22

Re: [ovs-dev] [PATCH v2 2/2] ovs-vswitchd: Avoid segfault for "netdev" datapath.

2016-12-07 10:04 GMT-08:00 nickcooper-zhangtonghao : > When the datapath, whose type is "netdev", processes packets > in userspce action, it may cause a segmentation fault. In the > dp_execute_userspace_action(), we pass the "wc" argument to > dp_netdev_upcall() using NULL. In the dp_netdev_upcall(

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

On 09/12/2016 11:04, "Jarno Rajahalme" wrote: >Acked-by: Jarno Rajahalme Thanks Jarno, I pushed this to master and up to branch-2.3 > >> On Dec 8, 2016, at 6:50 PM, Daniele Di Proietto >> wrote: >> >> Even though it reads 16 bits at a time, csum_continue() is almost >> neutral to endia

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

On Fri, Dec 9, 2016 at 11:21 AM, Russell Bryant wrote: > > > On Thu, Dec 8, 2016 at 8:31 PM, Andy Zhou wrote: > >> 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-b

[ovs-dev] Tome el verdadero control de su empresa

En línea y en Vivo / Para todo su Equipo con una sola Conexión Normas de Información Financiera (NIF) CONTROLES INTERNOS 16 de diciembre - Online en Vivo - 10:00 a 13:00 y de 15:00 a 18:00 Hrs Tome el verdadero control de su empresa. Aprenda la manera óptima de regular, registrar y

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

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") > 48911

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

On 7 December 2016 at 16:31, 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") > 489111e5

[ovs-dev] [PATCH] ofproto-dpif-ipfix: Fix assertion failure for bad configuration.

The assertions in dpif_ipfix_set_options() made some bad assumptions about flow exporters. The code that added and removed exporters would add a flow exporter even if it had an invalid configuration ("broken"), but the assertions checked that broken flow exporters were not added. Thus, the when a

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

On Fri, Dec 9, 2016 at 12:43 AM, Jiri Benc wrote: > On Thu, 8 Dec 2016 22:49:56 -0800, Pravin Shelar wrote: >> OVS out of tree kernel module is using compat tunnel code upto kernel >> 4.5 kernel even thought LWT is available in these kernels. This is due >> to missing features on these kernel whic

[ovs-dev] [PATCH] ofp-parse: Allow match field names in actions and brackets in matches.

Allow using match field names in addition to the canonical register names in actions (including 'load', 'move', 'push', 'pop', 'output', 'multipath', 'bundle_load', and 'learn'). Allow also leaving out the trailing '[]' to indicate full field. These changes allow simpler syntax similar to 'set_fi

Re: [ovs-dev] [PATCH v2 2.5] netdev-dpdk: Use instant sending instead of queueing of packets.

On 17/11/2016 01:43, "Thadeu Lima de Souza Cascardo" wrote: >From: Ilya Maximets > >Current implementarion of TX packet's queueing is broken in several ways: > > * TX queue flushing implemented on receive assumes that all > core_id-s are sequential and starts from zero. This

Re: [ovs-dev] [PATCH] ofproto-dpif-ipfix: Fix assertion failure for bad configuration.

Acked-by: Jarno Rajahalme With questions for clarification below, Jarno > On Dec 9, 2016, at 3:12 PM, Ben Pfaff wrote: > > The assertions in dpif_ipfix_set_options() made some bad assumptions about > flow exporters. The code that added and removed exporters would add a flow > exporter even

Re: [ovs-dev] assertion failing in commit_set_ipv4_action(), flow->nw_proto != base_flow->nw_proto

> On Dec 9, 2016, at 7:04 AM, Thomas Morin wrote: > > 2016-12-09, Thomas Morin: >> In the same setup as the one on which the bug was observed, [...] > > I was confused, I in fact tested the patch (branch-2.5) on a /different/ > setup as the one on which we hit the bug, using MPLS over a GRE tu

Re: [ovs-dev] [PATCH] ofproto-dpif-ipfix: Fix assertion failure for bad configuration.

On Fri, Dec 09, 2016 at 03:44:16PM -0800, Jarno Rajahalme wrote: > Acked-by: Jarno Rajahalme > > With questions for clarification below, Thanks! I applied this to master and to release branches back to 2.0. > > On Dec 9, 2016, at 3:12 PM, Ben Pfaff wrote: > > > > The assertions in dpif_ipfix

[ovs-dev] [PATCH] doc: Add required packages for RHEL build dependency

Two more packages are needed to build ovs package on RHEL. Signed-off-by: Yi-Hung Wei --- INSTALL.RHEL.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/INSTALL.RHEL.rst b/INSTALL.RHEL.rst index 125c55d..ee2bf86 100644 --- a/INSTALL.RHEL.rst +++ b/INSTALL.RHEL.rst @@ -74,7

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

> On Dec 9, 2016, at 2:47 PM, Joe Stringer wrote: > > On 7 December 2016 at 16:31, 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

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

Thanks for the reviews! Pushed to master with a better title and a compiletime_assert() for GENL_ID_GENERATE, if it is defined. Jarno > On Dec 9, 2016, at 2:36 PM, Pravin Shelar wrote: > > On Wed, Dec 7, 2016 at 4:31 PM, Jarno Rajahalme > wrote: >> This patch allows o

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

> On Dec 8, 2016, at 9:38 PM, Andy Zhou wrote: > > > > 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 upstr

[ovs-dev] [PATCH] datapath: Fix compile time assertion.

compiletime_assert() cannot be used in file scope, so use preprocessor directives instead. Reported-by: Joe Stringer Signed-off-by: Jarno Rajahalme --- datapath/linux/compat/include/linux/genetlink.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/datapath/linux/compat/in

Re: [ovs-dev] [PATCH] datapath: Fix compile time assertion.

On 9 December 2016 at 17:53, Jarno Rajahalme wrote: > compiletime_assert() cannot be used in file scope, so use preprocessor > directives instead. > > Reported-by: Joe Stringer > Signed-off-by: Jarno Rajahalme Thanks for the quick fix. Acked-by: Joe Stringer __

Re: [ovs-dev] [PATCH v2] odp-execute: Optimize IP header modification in OVS datapath

Thanks for the patch, it looks good to me and I can confirm the throughput improvements. Have you tried avoiding also computing the new field if the mask is 0? For example, if mask->ipv4_src is 0, there's not reason to compute new_ip_src, or to extract ip_src_nh. 2016-12-02 3:46 GMT-08:00 Zoltán

Re: [ovs-dev] [PATCH] datapath: Fix compile time assertion.

> On Dec 9, 2016, at 17:57, Joe Stringer wrote: > >> On 9 December 2016 at 17:53, Jarno Rajahalme wrote: >> compiletime_assert() cannot be used in file scope, so use preprocessor >> directives instead. >> >> Reported-by: Joe Stringer >> Signed-off-by: Jarno Rajahalme > > Thanks for the quic

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

On 08/12/2016 12:18, "Stokes, Ian" wrote: >> 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

Re: [ovs-dev] [PATCH 1/2] ovn-sbctl: Fix a few minor typesetting errors.

On Fri, Dec 09, 2016 at 02:18:23PM -0500, Russell Bryant wrote: > On Fri, Dec 9, 2016 at 1:38 PM, Ben Pfaff wrote: > > > In manpages, \- is used primarily for literal "-" in literal text, that is, > > it should usually be used for bold (literal) text. (It's also used for > > minus signs in mathe

[ovs-dev] [PATCH v2 0/2] Clearly relate OVN and OpenFlow flows.

v1->v2: - Patch 1 in v1 was committed and therefore dropped from the series. - Patch 2 in v2 is new. Ben Pfaff (2): ovn-controller: Tie OpenFlow and logical flows using OpenFlow cookie. ovn-trace: New --ovs option to also print OpenFlow flows. NEWS | 1 + inclu

[ovs-dev] [PATCH v2 1/2] ovn-controller: Tie OpenFlow and logical flows using OpenFlow cookie.

This makes it easy to find the logical flow that generated a particular OpenFlow flow, by running "ovn-sbctl dump-flows ". Later, this can be refined (and automated for "ofproto/trace"), but this is still a significant advance. Signed-off-by: Ben Pfaff --- lib/uuid.c | 22 +++

[ovs-dev] [PATCH v2 2/2] ovn-trace: New --ovs option to also print OpenFlow flows.

Sometimes seeing the OpenFlow flows that back a given logical flow can provide additional insight. This commit adds a new --ovs option to ovn-trace that makes it connect to Open vSwitch over OpenFlow and retrieve and print the OpenFlow flows behind each logical flow encountered during a trace. Si

Re: [ovs-dev] [PATCH 2/2] ovn-controller: Tie OpenFlow and logical flows using OpenFlow cookie.

On Fri, Dec 09, 2016 at 10:38:32AM -0800, Ben Pfaff wrote: > This makes it easy to find the logical flow that generated a particular > OpenFlow flow, by running "ovn-sbctl dump-flows ". > > Later, this can be refined (and automated for "ofproto/trace"), but this > is still a significant advance. >