[ovs-dev] Updating Logical_Switch_Port.up with ovn-controller being a read-only client to SB DB

2016-11-30 Thread Numan Siddique
​Hello, With the patches proposed to make ovn-controller not updating the Port_Binding.chassis column [1], the value of Logical_Switch_Port.up in the NB DB will not be the true value. Below is a proposal which solves this problem - ovn-northd and ovn-controller will make use of locking mechani

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

2016-11-30 Thread Thomas Morin
Hi Ben, 2016-11-30, Ben Pfaff: Do you have any idea what in your OpenFlow pipeline might do that, i.e. is there anything especially tricky in the OpenFlow flows? > > Are you willing to show us your OpenFlow flow table? The setup involves three OVS bridges connected with patch-ports: br-int --

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

2016-11-30 Thread Liran Schour
Darrell Ball wrote on 30/11/2016 02:29:01 AM: > On Tue, Nov 29, 2016 at 4:10 AM, Liran Schour wrote: > Ben Pfaff wrote on 29/11/2016 12:51:51 AM: > > > I hope that this version is ready to go in. Liran, are you happy with > > this version? > > > > I did some short evaluation and got the fol

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

2016-11-30 Thread Simon Horman
Hi Thomas, On Wed, Nov 30, 2016 at 12:05:46PM +0100, Thomas Morin wrote: > Hi Ben, > > 2016-11-30, Ben Pfaff: > >Do you have any idea what in your OpenFlow pipeline might do that, > >i.e. is there anything especially tricky in the OpenFlow flows? > > > > Are you willing to show us your OpenFlow f

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

2016-11-30 Thread Thomas Morin
Hi Simon, 2016-11-30, Simon Horman: Thomas Morin wrote: 2016-11-30, Ben Pfaff: Do you have any idea what in your OpenFlow pipeline might do that, i.e. is there anything especially tricky in the OpenFlow flows? Are you willing to show us your OpenFlow flow table? The setup involves three OVS

[ovs-dev] What is the minimum Linux kernel version ovs 2.6.1 build requires?

2016-11-30 Thread Yang, Yi Y
Hi, folks I tried to build ovs 2.6.1 in Ubuntu trusty 64 with the below configuration options, but it failed, it is ok when I use the same way to build in Ubuntu 16.04 which has Linux kernel 4.4. What is the minimum Linux kernel version ovs 2.6.1 build requires? ./configure --prefix=/ --with-l

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

2016-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2016 at 12:05:46PM +0100, Thomas Morin wrote: > Hi Ben, > > 2016-11-30, Ben Pfaff: > >Do you have any idea what in your OpenFlow pipeline might do that, > >i.e. is there anything especially tricky in the OpenFlow flows? > > > > Are you willing to show us your OpenFlow flow table? >

Re: [ovs-dev] [PATCH v2] ofp-actions: Add "ingress" and "egress" options to "sample" action.

2016-11-30 Thread Ben Pfaff
On Mon, Nov 28, 2016 at 04:47:30PM +0100, Simon Horman wrote: > On Sat, Nov 26, 2016 at 01:36:00PM -0800, Ben Pfaff wrote: > > On Fri, Nov 25, 2016 at 03:46:55PM +0100, Simon Horman wrote: > > > Hi Ben, > > > > > > On Wed, Nov 23, 2016 at 11:18:14PM -0800, Ben Pfaff wrote: > > > > Before Open vSwi

Re: [ovs-dev] [PATCH] datapath-windows: null comparison for icmp and tcp header

2016-11-30 Thread Sairam Venugopal
OvsConntrackValidateTcpPacket currently checks for NULL. I will update OvsConntrackValidateIcmpPacket to check for NULL as well. I acked this change to keep the checks consistent across different protocol. Thanks, Sairam On 11/29/16, 2:30 PM, "Yin Lin" wrote: >Can we decide if tcp and icmp i

[ovs-dev] [PATCH] datapath: backport: openvswitch: Fix skb leak in IPv6 reassembly.

2016-11-30 Thread Daniele Di Proietto
commit f92a80a9972175a6a1d36c6c44be47fb0efd020d Author: Daniele Di Proietto Date: Mon Nov 28 15:43:53 2016 -0800 openvswitch: Fix skb leak in IPv6 reassembly. If nf_ct_frag6_gather() returns an error other than -EINPROGRESS, it means that we still have a reference to th

[ovs-dev] Las 5´s de la CALIDAD TOTAL JAPONESA

2016-11-30 Thread Más ordenado de manera permanente
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

Re: [ovs-dev] [RFC] [PATCH] ovn: Support sample action in logical datapath

2016-11-30 Thread Valentine Sinitsyn
On 30.11.2016 06:18, Ben Pfaff wrote: On Tue, Nov 29, 2016 at 07:22:32PM +0500, Valentine Sinitsyn wrote: On 29.11.2016 05:21, Ben Pfaff wrote: On Fri, Oct 14, 2016 at 04:35:46PM +0500, Valentine Sinitsyn wrote: This is a quick attempt to implement sample action at logical port level.The goal

[ovs-dev] [PATCH] CONTRIBUTING.rst: Update patch summary and description style guidelines.

2016-11-30 Thread Ben Pfaff
Suggested-by: Joe Stringer Suggested-at: https://mail.openvswitch.org/pipermail/ovs-dev/2016-November/325513.html Signed-off-by: Ben Pfaff --- CONTRIBUTING.rst | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/CONTRIBUTING.rst b/CONTRIBUTING.rst index 867562e..72

[ovs-dev] [RFC PATCH] ofp-actions: Add clone action.

2016-11-30 Thread William Tu
This patch adds OpenFlow clone action with syntax as below: "clone([action][,action...])". The clone() action makes a copy of the current packet and executes the list of actions against the packet, without affecting the packet after the "clone(...)" action. In other word, the packet before the cl

Re: [ovs-dev] [RFC PATCH] ofp-actions: Add clone action.

2016-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2016 at 01:13:35PM -0800, William Tu wrote: > This patch adds OpenFlow clone action with syntax as below: > "clone([action][,action...])". The clone() action makes a copy of the > current packet and executes the list of actions against the packet, > without affecting the packet aft

Re: [ovs-dev] [RFC PATCH] ofp-actions: Add clone action.

2016-11-30 Thread William Tu
OK, I will rebase and resubmit. On Wed, Nov 30, 2016 at 1:25 PM, Ben Pfaff wrote: > On Wed, Nov 30, 2016 at 01:13:35PM -0800, William Tu wrote: >> This patch adds OpenFlow clone action with syntax as below: >> "clone([action][,action...])". The clone() action makes a copy of the >> current packe

[ovs-dev] [RFC PATCHv2] ofp-actions: Add clone action.

2016-11-30 Thread William Tu
This patch adds OpenFlow clone action with syntax as below: "clone([action][,action...])". The clone() action makes a copy of the current packet and executes the list of actions against the packet, without affecting the packet after the "clone(...)" action. In other word, the packet before the cl

[ovs-dev] DEV, Problems with item delivery, n.0000484051

2016-11-30 Thread FedEx 2Day
Dear Dev, This is to confirm that one or more of your parcels has been shipped. Shipment Label is attached to this email. Warm regards, Jon Foster, FedEx Station Agent. ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/list

[ovs-dev] Planeación y Presupuestos para el 2017

2016-11-30 Thread obtenga beneficios para la economía de su empresa
obtenga beneficios para la economía de su empresa Planeación y Presupuestos para el 2017 14 de diciembre - Zona Reforma - 9:00 a 18:00 Imparte: C.P. C.P. Gerardo López Najera Para lograr este fin, es importante que quienes llevan el control y contabilidad de una empresa, conozcan y ap

Re: [ovs-dev] [PATCH 0/2] ofproto-dpif-xlate: remove unneeded vlan check

2016-11-30 Thread Ben Pfaff
On Tue, Nov 01, 2016 at 02:39:28AM +0800, Hunt Xu wrote: > At the very beginning I was just trying to modify packets ingressing from an > access port, using "mod_vlan_vid" & "NORMAL" to change the VLAN vid, expecting > them to be then switched to the ports which carries the dest VLAN. However, > I

Re: [ovs-dev] [PATCH 1/2] ofproto-dpif-xlate: don't check vlan in normal action if vid changed

2016-11-30 Thread Ben Pfaff
On Tue, Nov 01, 2016 at 02:39:29AM +0800, Hunt Xu wrote: > If a packet's vlan vid has been changed(striped or modified) before > entering the "normal" processing, it should be consider a packet on the > new VLAN. Therefore vlan checking is not needed. > > Signed-off-by: Hunt Xu I gave some gener

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

2016-11-30 Thread Ben Pfaff
This seems like a worthwhile optimization but the patch is corrupted. Will you rebase and resend? Thanks, Ben. On Tue, Nov 01, 2016 at 06:28:36PM +, Zoltán Balogh wrote: > > Hi, > > I measured the packet processing cost of OVS DPDK datapath for different > OpenFlow actions. I configured O

Re: [ovs-dev] [PATCH 1/2] ovsdb windows: Allow online compacting

2016-11-30 Thread Ben Pfaff
On Wed, Nov 09, 2016 at 12:58:16PM +, Alin Serdean wrote: > > > > > > > This had some bugs. > > > > The worst one was that it deleted all the data from the database. This > > wasn't > > evident from the test because it only deleted it from the on-disk > > representation and the test didn't

Re: [ovs-dev] [RFC v2 0/5] ovn: add distributed NAT capability

2016-11-30 Thread Ben Pfaff
On Thu, Nov 03, 2016 at 12:09:21AM -0700, Mickey Spiegel wrote: > This patch set is being sent out early to solicit feedback on the > approach. North/south DNAT and SNAT are working, including some > automated tests. There is another patch required to get east/west NAT > working, but work has not

Re: [ovs-dev] [PATCH] datapath: backport: openvswitch: Fix skb leak in IPv6 reassembly.

2016-11-30 Thread Joe Stringer
On 30 November 2016 at 10:41, Daniele Di Proietto wrote: > commit f92a80a9972175a6a1d36c6c44be47fb0efd020d > Author: Daniele Di Proietto > Date: Mon Nov 28 15:43:53 2016 -0800 > > openvswitch: Fix skb leak in IPv6 reassembly. > > If nf_ct_frag6_gather() returns an error othe

[ovs-dev] Packet type-aware pipeline, L3 tunneling, EXT-382 and NSH: Minutes call 2016-11-29

2016-11-30 Thread Jan Scheurich
A small team is driving the implementation of NSH based on a number of new features to be implemented in OVS: * Packet type-aware pipeline (PTAP, EXT-112) in OF 1.5 * Generic tunnel encap/decap actions (EXT-382) intended for OF 1.6 * "versatile" tunnel ports (aka L3 tunnels) The

[ovs-dev] Organización y Orden para el Éxito

2016-11-30 Thread Las 5´s de la Calidad total Japonesa
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

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

2016-11-30 Thread Daniele Di Proietto
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_eth_xstats_get (port_id=3 '\003', xstats=xstats@entry=0x2e6eaa0, n=n

Re: [ovs-dev] [PATCH] datapath: backport: openvswitch: Fix skb leak in IPv6 reassembly.

2016-11-30 Thread Daniele Di Proietto
On 30/11/2016 14:40, "Joe Stringer" wrote: >On 30 November 2016 at 10:41, Daniele Di Proietto > wrote: >> commit f92a80a9972175a6a1d36c6c44be47fb0efd020d >> Author: Daniele Di Proietto >> Date: Mon Nov 28 15:43:53 2016 -0800 >> >> openvswitch: Fix skb leak in IPv6 reassemb

Re: [ovs-dev] [PATCH] CONTRIBUTING.rst: Update patch summary and description style guidelines.

2016-11-30 Thread Justin Pettit
> On Nov 30, 2016, at 12:40 PM, Ben Pfaff wrote: > > Suggested-by: Joe Stringer > Suggested-at: > https://mail.openvswitch.org/pipermail/ovs-dev/2016-November/325513.html > Signed-off-by: Ben Pfaff Acked-by: Justin Pettit --Justin ___ dev mai

Re: [ovs-dev] [RFC v2 0/5] ovn: add distributed NAT capability

2016-11-30 Thread Mickey Spiegel
On Wed, Nov 30, 2016 at 2:18 PM, Ben Pfaff wrote: > On Thu, Nov 03, 2016 at 12:09:21AM -0700, Mickey Spiegel wrote: > > This patch set is being sent out early to solicit feedback on the > > approach. North/south DNAT and SNAT are working, including some > > automated tests. There is another pat

[ovs-dev] Contact

2016-11-30 Thread Ms. Kish
 Can you assist to handle this transaction? More details will sent to you as soon as i receive your response. Kish ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH net-next v2] ofproto: Honor OFPFF_RESET_COUNTS flag in flow modify message.

2016-11-30 Thread Jarno Rajahalme
While a flow modify must keep the original flow's flags, it must reset counts if (and only if) the reset_counts flag is present in the flow mod message. Behavior prior to this patch is broken in a few ways: - OpenFlow 1.0 and 1.1 mod-flows did reset the counts, if the flow had reset_counts flag

Re: [ovs-dev] [PATCH 03/14] doc: Convert AUTHORS to rST

2016-11-30 Thread Russell Bryant
On Tue, Nov 22, 2016 at 1:18 PM, Stephen Finucane wrote: > Signed-off-by: Stephen Finucane > I've applied patches 1-3 to master. -- Russell Bryant ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [PATCH 04/14] Add initial sphinx configuration

2016-11-30 Thread Russell Bryant
On Tue, Nov 22, 2016 at 1:18 PM, Stephen Finucane wrote: > 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

Re: [ovs-dev] [PATCH 04/14] Add initial sphinx configuration

2016-11-30 Thread Russell Bryant
On Wed, Nov 30, 2016 at 9:02 PM, Russell Bryant wrote: > > > On Tue, Nov 22, 2016 at 1:18 PM, Stephen Finucane > wrote: > >> This is essentially the output of 'sphinx-quickstart' but with the >> following changes: >> >> - Parts of the generated Makefile are merged into the existing >> Document

Re: [ovs-dev] Packet type-aware pipeline, L3 tunneling, EXT-382 and NSH: Minutes call 2016-11-29

2016-11-30 Thread Jarno Rajahalme
Thanks for the minutes, Jan! The time difference between Central Europe and Pacific Standard Time is 9 hours, so the next meeting will start at 8am PST. Jarno > On Nov 30, 2016, at 2:52 PM, Jan Scheurich wrote: > > A small team is driving the implementation of NSH based on a number of new

Re: [ovs-dev] [PATCH] ovs-vtep: vtep-ctl and ovs-vtep support of adding explicit tunnel key

2016-11-30 Thread Justin Pettit
> On Oct 27, 2016, at 8:12 AM, itamaro wrote: > > From: itamarofek Sorry for the delay. I had a bunch of suggested changes in my drafts folder and apparently never hit send. The previous patches had version numbers. I assume this is v5. Can you please add v6 to the next version? It's als

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

2016-11-30 Thread Jarno Rajahalme
> On Nov 30, 2016, at 8:41 AM, Ben Pfaff wrote: > > On Wed, Nov 30, 2016 at 12:05:46PM +0100, Thomas Morin wrote: >> Hi Ben, >> >> 2016-11-30, Ben Pfaff: >>> Do you have any idea what in your OpenFlow pipeline might do that, >>> i.e. is there anything especially tricky in the OpenFlow flows? >>

[ovs-dev] [PATCH] build: fix rpm-fedora target breakage

2016-11-30 Thread Lance Richardson
Since commit 3deca69b08f2 ("doc: Convert AUTHORS to rST"), the rpm-fedora target fails to build with: *** No rule to make target `AUTHORS.rst', needed by `debian/copyright'. Fix by adding AUTHORS.rst to the docs list to ensure that it is included in the dist archive. Fixes: 3deca69b08f2 ("doc:

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

2016-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2016 at 06:58:57PM -0800, Jarno Rajahalme wrote: > > > On Nov 30, 2016, at 8:41 AM, Ben Pfaff wrote: > > > > On Wed, Nov 30, 2016 at 12:05:46PM +0100, Thomas Morin wrote: > >> Hi Ben, > >> > >> 2016-11-30, Ben Pfaff: > >>> Do you have any idea what in your OpenFlow pipeline migh

Re: [ovs-dev] [PATCH] build: fix rpm-fedora target breakage

2016-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2016 at 10:57:25PM -0500, Lance Richardson wrote: > Since commit 3deca69b08f2 ("doc: Convert AUTHORS to rST"), the rpm-fedora > target fails to build with: > > *** No rule to make target `AUTHORS.rst', needed by `debian/copyright'. > > Fix by adding AUTHORS.rst to the docs list

Re: [ovs-dev] [PATCH net-next v2] ofproto: Honor OFPFF_RESET_COUNTS flag in flow modify message.

2016-11-30 Thread Ben Pfaff
On Wed, Nov 30, 2016 at 05:51:12PM -0800, Jarno Rajahalme wrote: > While a flow modify must keep the original flow's flags, it must reset > counts if (and only if) the reset_counts flag is present in the flow > mod message. > > Behavior prior to this patch is broken in a few ways: > > - OpenFlow

[ovs-dev] OVS NSH patches

2016-11-30 Thread Yang, Yi Y
Hi, folks Although official ovs release doesn't support NSH, but we do have a repo to host OVS NSH patches, it is https://github.com/yyang13/ovs_nsh_patches/, now we have ported it to ovs 2.6.1 in order that it can support new Linux kernels and conntrack in case of OVS DPDK. The patchset for o

[ovs-dev] [PATCH] ovn-controller: Remove obsolete concept of "stale" patched datapaths.

2016-11-30 Thread Ben Pfaff
When ovn-controller implemented incremental processing, the set of patched datapaths was revised on each trip through the main loop, so it was necessary to notice datapaths that shouldn't exist anymore and remove them. With incremental processing gone, the set of patched datapaths is built and tor

[ovs-dev] [PATCH] physical: Remove obsolete comments.

2016-11-30 Thread Ben Pfaff
These comments were added above code that removed flows from the flow table. Now that that code was deleted, the comments no longer make sense. Signed-off-by: Ben Pfaff --- ovn/controller/physical.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/ovn/controller/physical.c b/ovn/control