[ovs-dev] New Doc 03-68

2016-08-11 Thread Virginia
Scanned by CamScanner Sent from Yahoo Mail on Android ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Offerta sito web professionale, aggiornabile personalmente.

2016-08-11 Thread Presenza sul web
Offerta speciale creazione sito web. Fai click qui per visualizzare online (

[ovs-dev] New Doc 9-8

2016-08-11 Thread Oscar
Scanned by CamScanner Sent from Yahoo Mail on Android ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Simon Horman
Hi Jan, On Thu, Aug 11, 2016 at 12:58:44PM +, Jan Scheurich wrote: > Hi Simon > > > > The implicit push_eth action introduced by Simon in his L3 tunnel port > > > patch mainly ensures the presence of the 14 byte MAC header on ports > > > where this is a must for syntactic interpretation of

[ovs-dev] [PATCH 1/1] FAQ: Update supported userspace features and DPDK version.

2016-08-11 Thread Ian Stokes
Add support for policing to userspace features and set DPDK v16.07 as the supported DPDK version for release 2.6.x. Signed-off-by: Ian Stokes --- FAQ.md |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/FAQ.md b/FAQ.md index 40e3a22..2f50b96

[ovs-dev] New Doc 01-8

2016-08-11 Thread Bianca
Scanned by CamScanner Sent from Yahoo Mail on Android ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn-controller: Remove the repeated code.

2016-08-11 Thread nickcooper-zhangtonghao
> On Aug 7, 2016, at 7:22 AM, Ryan Moats wrote: > > > From: nickcooper-zhangtonghao > > > > To: dev@openvswitch.org > > Cc: nickcooper-zhangtonghao

[ovs-dev] [PATCH v2] ovn-controller: Reset flow processing after (re)connection to switch

2016-08-11 Thread Numan Siddique
When ovn-controller reconnects to the ovs-vswitchd, it deletes all the OF flows in the switch. It doesn't install the flows again, leaving the datapath broken unless ovn-controller is restarted or ovn-northd updates the SB DB. The reason for this is - lflow_reset_processing() is not called

[ovs-dev] New Doc 92-2

2016-08-11 Thread Sung
Scanned by CamScanner Sent from Yahoo Mail on Android ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn-controller: Reset flow processing after (re)connection to switch

2016-08-11 Thread Numan Siddique
On Wed, Aug 10, 2016 at 6:07 AM, Numan Siddique wrote: > On Aug 9, 2016 8:28 PM, "Ryan Moats" wrote: > > > > Numan Siddique wrote on 08/09/2016 09:39:21 AM: > > > > > From: Numan Siddique > > > To: Ryan

Re: [ovs-dev] [RFC] systemd notification integration

2016-08-11 Thread Christian Svensson
On Wed, Aug 10, 2016 at 11:51 PM, Ben Pfaff wrote: > > Hmm. I think that OVS actually solves all of these problems. First, > the return code of the parent should be correct, in that if the child > fails to start properly the parent exits with a nonzero exit code. > Second,

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Jan Scheurich
Hi Simon > > The implicit push_eth action introduced by Simon in his L3 tunnel port > > patch mainly ensures the presence of the 14 byte MAC header on ports > > where this is a must for syntactic interpretation of the packet. It > > did not worry about if the resulting packet was semantically

Re: [ovs-dev] [PATCH] ovs-bugtool: Switch from MD5 to SHA-256.

2016-08-11 Thread Ryan Moats
"dev" wrote on 08/10/2016 11:14:27 PM: > From: Ben Pfaff > To: dev@openvswitch.org > Cc: Ben Pfaff > Date: 08/10/2016 11:14 PM > Subject: [ovs-dev] [PATCH] ovs-bugtool: Switch from MD5 to SHA-256. > Sent by: "dev"

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Simon Horman
Hi Jan, On Thu, Aug 11, 2016 at 11:04:22AM +, Jan Scheurich wrote: > > > [Jan] Should sending a packet after push_nsh to an output port be > > > allowed in general? For a VXLAN-GPE tunnel port this is OK, but my > > > expectation was that one must explicitly do push_eth (followed by > > >

[ovs-dev] New Doc 76-8

2016-08-11 Thread Della
Scanned by CamScanner Sent from Yahoo Mail on Android ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Emailing: Receipt(13)

2016-08-11 Thread Aileen
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Jan Scheurich
> > [Jan] Should sending a packet after push_nsh to an output port be > > allowed in general? For a VXLAN-GPE tunnel port this is OK, but my > > expectation was that one must explicitly do push_eth (followed by > > set_field for dl_src and dl_dst) to be able to transmit on a normal > > Ethernet

[ovs-dev] File: Scan(06)

2016-08-11 Thread Charlene
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovn-controller: Combine two Logical_Flow loops into one.

2016-08-11 Thread nickcooper-zhangtonghao
> On Aug 7, 2016, at 6:54 AM, Ryan Moats wrote: > > > From: nickcooper-zhangtonghao > > > > To: dev@openvswitch.org > > Cc: nickcooper-zhangtonghao

Re: [ovs-dev] [PATCH] ovn-controller:improve the ofctrl_run, pinctrl_run

2016-08-11 Thread nickcooper-zhangtonghao
> On Aug 7, 2016, at 5:44 AM, Ryan Moats wrote: > > "dev" wrote on 08/04/2016 06:50:03 AM: > > > From: nickcooper-zhangtonghao > > To: dev@openvswitch.org > > Cc: nickcooper-zhangtonghao

Re: [ovs-dev] [RFC/PATCH v2] Make the PID part of socket path configurable

2016-08-11 Thread Christian Svensson
On Tue, Aug 9, 2016 at 9:10 AM, Christian Svensson wrote: > > ovs-appctl -t $PWD/my-daemon.$( To compromise on the client side: How would you feel making ovs-appctl and socket_name_from_target look for both naming schemes? Psuedo code: if isfile(target +

[ovs-dev] File: Receipt(5876)

2016-08-11 Thread Heriberto
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] ovn-controller: Reset flow processing after (re)connection to switch

2016-08-11 Thread Numan Siddique
On Thu, Aug 11, 2016 at 7:45 PM, Ryan Moats wrote: > "dev" wrote on 08/11/2016 07:21:39 AM: > > > > > Signed-off-by: Numan Siddique > > --- > > This all looks does what it promises, so let's get the immediate > out of the

[ovs-dev] Copy: Receipt(6665)

2016-08-11 Thread Jerald
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] ovn-controller: Reset flow processing after (re)connection to switch

2016-08-11 Thread Ryan Moats
"dev" wrote on 08/11/2016 07:21:39 AM: > From: Numan Siddique > To: ovs dev > Date: 08/11/2016 07:21 AM > Subject: [ovs-dev] [PATCH v2] ovn-controller: Reset flow processing > after (re)connection to switch > Sent by: "dev"

Re: [ovs-dev] [PATCH v2 0/5] check-kernel: add 802.1ad tests

2016-08-11 Thread Eric Garver
On Wed, Aug 10, 2016 at 04:54:08PM -0700, Joe Stringer wrote: > On 10 August 2016 at 14:55, Eric Garver wrote: > > Hi, > > > > See two replies below. > > > > Thanks again! > > > > On Fri, Aug 05, 2016 at 10:03:24AM -0400, Eric Garver wrote: > >> Joe, > >> > >> Thanks for further

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-11 Thread Aaron Conole
Thanks so much for the review, Ben! Wall of text follows (apologies), so I suggest having your breakfast first :) Ben Pfaff writes: > On Mon, Aug 01, 2016 at 08:24:40AM -0400, Aaron Conole wrote: >> Aaron Conole writes: >> >> > Currently, when using Open

[ovs-dev] File: Scan(415)

2016-08-11 Thread Guadalupe
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Emailing: Document(078)

2016-08-11 Thread Josephine
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Attached: Receipt(8563)

2016-08-11 Thread Vilma
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] Add monitoring to ovsdb

2016-08-11 Thread Conner Herriges
Ben Pfaff wrote on 08/10/2016 11:22:49 AM: > From: Ben Pfaff > To: Conner Herriges/US/IBM@IBM > Cc: dev@openvswitch.org > Date: 08/10/2016 11:23 AM > Subject: Re: [ovs-dev] [PATCH v2] Add monitoring to ovsdb > > On Wed, Aug 10, 2016 at 11:10:25AM -0500, Conner

[ovs-dev] [PATCH v4 2/3] netdev-dpdk: Consistent naming for vhost

2016-08-11 Thread Ciara Loftus
A mix of vhost_user_ and vhost_ is used when naming vhost functions. The 'user_' has been dropped for consistency. Also remove empty init functions for netdev dpdk classes. Suggested-by: Daniele Di Proietto Acked-by: Daniele Di Proietto Acked-by: Flavio Leitner

[ovs-dev] [PATCH v4 1/3] netdev-dpdk: Remove dpdkvhostcuse ports

2016-08-11 Thread Ciara Loftus
This commit removes the 'dpdkvhostcuse' port type from the userspace datapath. vhost-cuse ports are quickly becoming obsolete as the vhost-user port type begins to support a greater feature-set thanks to the addition of things like vhost-user multiqueue and potential upcoming features like

[ovs-dev] [PATCH v4 3/3] netdev-dpdk: vHost client mode and reconnect

2016-08-11 Thread Ciara Loftus
Until now, vHost ports in OVS have only been able to operate in 'server' mode whereby OVS creates and manages the vHost socket and essentially acts as the vHost 'server'. With this commit a new mode, 'client' mode, is available. In this mode, OVS acts as the vHost 'client' and connects to the

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 01:00:21PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > Neither of those is a simple "return server->listen_fd;" function. Any > > chance that would be accepted? It would solve the problem just fine. > > I don't know, but I'll outline my plans, and

Re: [ovs-dev] [PATCH v2] Add monitoring to ovsdb

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 10:12:13AM -0500, Conner Herriges wrote: > Ben Pfaff wrote on 08/10/2016 11:22:49 AM: > > > From: Ben Pfaff > > To: Conner Herriges/US/IBM@IBM > > Cc: dev@openvswitch.org > > Date: 08/10/2016 11:23 AM > > Subject: Re: [ovs-dev] [PATCH v2] Add

Re: [ovs-dev] [PATCH v1] ovn: rename Address_Set to Set to reflect a more broad purpose

2016-08-11 Thread Russell Bryant
On Tue, Aug 9, 2016 at 11:55 PM, Zong Kai LI wrote: > This patch renames table Address_Set to Set, Address_Set.addresses to > Set.members to reflect a more broad purpose, that we can define other > types > of sets than address set. > > Per discussion around [1] and [2], this

Re: [ovs-dev] Failing Test 2214 on i686

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 10:08:12AM -0700, Ben Pfaff wrote: > On Fri, Aug 05, 2016 at 03:25:47PM +0200, Christian Ehrhardt wrote: > > Hi, > > while checking latest master we found that there were several tests that > > failed sometimes. > > 15: bfd - bfd decay > > 1149: ofproto-dpif - in place

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Log flow in XLATE_REPORT().

2016-08-11 Thread Ben Pfaff
On Thu, Aug 04, 2016 at 02:58:38PM -0700, Joe Stringer wrote: > To assist debugging pipelines when resubmit resource checks fail, print > the base_flow from the translation context. This base flow can then be > used from ofproto/trace to figure out which parts of the pipeline lead > to this

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 11:33:59AM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > On Mon, Aug 01, 2016 at 08:24:40AM -0400, Aaron Conole wrote: > >> Aaron Conole writes: > >> > >> > Currently, when using Open vSwitch with DPDK and qemu guests, the > >> >

Re: [ovs-dev] [PATCH v2 3/3] netdev-dpdk: vHost client mode and reconnect

2016-08-11 Thread Loftus, Ciara
> > > 2016-08-08 7:18 GMT-07:00 Loftus, Ciara : > > > > The patch mostly looks good to me, thanks. > > I'm not 100% sure about the interface.  Can we make the flag interface > > specific? > > I'm not 100% sure about making the flag interface specific :) Do you think >

Re: [ovs-dev] [PATCH] ovs-bugtool: Switch from MD5 to SHA-256.

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 07:59:10AM -0500, Ryan Moats wrote: > > > "dev" wrote on 08/10/2016 11:14:27 PM: > > > From: Ben Pfaff > > To: dev@openvswitch.org > > Cc: Ben Pfaff > > Date: 08/10/2016 11:14 PM > > Subject: [ovs-dev] [PATCH]

Re: [ovs-dev] Failing Test 2214 on i686

2016-08-11 Thread Ben Pfaff
On Fri, Aug 05, 2016 at 03:25:47PM +0200, Christian Ehrhardt wrote: > Hi, > while checking latest master we found that there were several tests that > failed sometimes. > 15: bfd - bfd decay > 1149: ofproto-dpif - in place modification (vlan) > But one of them seems to reliably fail in i686 mode

Re: [ovs-dev] [PATCH v3] Add monitoring to ovsdb

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 11:33:17AM -0500, Conner Herriges wrote: > Monitoring is a valid option on POSIX based platforms. The monitor > creates an additional process to monitor the ovsdb-server daemon. If > the daemon dies due to a programming error, then the monitor process > starts a new copy of

[ovs-dev] [PATCH v3] Add monitoring to ovsdb

2016-08-11 Thread Conner Herriges
Monitoring is a valid option on POSIX based platforms. The monitor creates an additional process to monitor the ovsdb-server daemon. If the daemon dies due to a programming error, then the monitor process starts a new copy of it. If the daemon dies or exits for another reason, the monitor process

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-11 Thread Aaron Conole
Ben Pfaff writes: > On Thu, Aug 11, 2016 at 11:33:59AM -0400, Aaron Conole wrote: >> Ben Pfaff writes: >> >> > On Mon, Aug 01, 2016 at 08:24:40AM -0400, Aaron Conole wrote: >> >> Aaron Conole writes: >> >> >> >> > Currently, when using Open

[ovs-dev] [PATCHv2] ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.

2016-08-11 Thread Joe Stringer
To assist debugging pipelines when resubmit resource checks fail, print the base_flow from the translation context. This base flow can then be used from ofproto/trace to figure out which parts of the pipeline lead to this translation error. Signed-off-by: Joe Stringer ---

Re: [ovs-dev] [PATCH 1/3] datapath: compat: keep skb encapsulation zero on older kernel.

2016-08-11 Thread Jesse Gross
On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar wrote: > When using compat GSO there is no need to turn on skb > encapsulation bit since OVS does not use any tunnel GSO > functionality from the networking stack. > > Signed-off-by: Pravin B Shelar I'm not sure

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Log flow in XLATE_REPORT().

2016-08-11 Thread Joe Stringer
On 11 August 2016 at 11:04, Ben Pfaff wrote: > On Thu, Aug 04, 2016 at 02:58:38PM -0700, Joe Stringer wrote: >> To assist debugging pipelines when resubmit resource checks fail, print >> the base_flow from the translation context. This base flow can then be >> used from

[ovs-dev] [PATCH] ovn: make external_ids naming uniform

2016-08-11 Thread Mauricio Vasquez B
In some places it is called "external-ids" while in others "external_ids" is used. This commit changes it to external_ids in all of them. Signed-off-by: Mauricio Vasquez B --- ovn/controller/ovn-controller.8.xml | 12 ++--

Re: [ovs-dev] [PATCH] sandbox: launch SB backup server when running in OVN mode

2016-08-11 Thread Russell Bryant
On Tue, Aug 9, 2016 at 7:49 PM, Andy Zhou wrote: > Automatically launch backup server for OVN SB database that replicates > all transactions of the active server. This can be handy for > experimenting with the newly added replication feature. > > Signed-off-by: Andy Zhou

Re: [ovs-dev] [PATCH 3/3] datapath: compat: vxlan: fix vxlan_notify_add_rx_port().

2016-08-11 Thread Jesse Gross
On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar wrote: > Same as earlier patch this fixes vxlan recieve offload implementation. > > Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross ___ dev

[ovs-dev] [PATCH v2] datapath: compat: keep skb encapsulation zero on older kernel.

2016-08-11 Thread Pravin B Shelar
When using compat GSO there is no need to turn on skb encapsulation bit since OVS does not use any tunnel GSO functionality from the networking stack. Signed-off-by: Pravin B Shelar --- datapath/linux/compat/gso.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-)

Re: [ovs-dev] [PATCH v2] datapath: compat: keep skb encapsulation zero on older kernel.

2016-08-11 Thread Jesse Gross
On Thu, Aug 11, 2016 at 2:12 PM, Pravin B Shelar wrote: > When using compat GSO there is no need to turn on skb > encapsulation bit since OVS does not use any tunnel GSO > functionality from the networking stack. > > Signed-off-by: Pravin B Shelar Acked-by:

Re: [ovs-dev] [PATCHv2] ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.

2016-08-11 Thread Joe Stringer
On 11 August 2016 at 13:42, Ben Pfaff wrote: > On Thu, Aug 11, 2016 at 12:36:16PM -0700, Joe Stringer wrote: >> To assist debugging pipelines when resubmit resource checks fail, print >> the base_flow from the translation context. This base flow can then be >> used from

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

2016-08-11 Thread Darrell Ball
I have various comments; I focused on the big picture mainly and did not review most details except ovs-vtep and one or two selected spots elsewhere, pending agreement on the big picture. Some comments are the same as I made in the first patch. Your patch did not apply cleanly due to previous

Re: [ovs-dev] [PATCH v2] datapath: compat: keep skb encapsulation zero on older kernel.

2016-08-11 Thread pravin shelar
On Thu, Aug 11, 2016 at 2:23 PM, Jesse Gross wrote: > On Thu, Aug 11, 2016 at 2:12 PM, Pravin B Shelar wrote: >> When using compat GSO there is no need to turn on skb >> encapsulation bit since OVS does not use any tunnel GSO >> functionality from the

Re: [ovs-dev] [PATCH 2/3] datapath: compat: geneve: fix geneve_notify_add_rx_port()

2016-08-11 Thread Jesse Gross
On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar wrote: > Remove mutual exclusion between udp-gro registration and geneve receive port > registration. > > Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross

Re: [ovs-dev] [PATCH 1/3] datapath: compat: keep skb encapsulation zero on older kernel.

2016-08-11 Thread pravin shelar
On Thu, Aug 11, 2016 at 1:17 PM, Jesse Gross wrote: > On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar wrote: >> When using compat GSO there is no need to turn on skb >> encapsulation bit since OVS does not use any tunnel GSO >> functionality from the

Re: [ovs-dev] [PATCH] ovn: make external_ids naming uniform

2016-08-11 Thread Russell Bryant
On Thu, Aug 11, 2016 at 4:23 PM, Mauricio Vasquez B < mauricio.vasquezber...@studenti.polito.it> wrote: > In some places it is called "external-ids" while in others "external_ids" > is used. This commit changes it to external_ids in all of them. > > Signed-off-by: Mauricio Vasquez B

Re: [ovs-dev] [PATCHv2] ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 12:36:16PM -0700, Joe Stringer wrote: > To assist debugging pipelines when resubmit resource checks fail, print > the base_flow from the translation context. This base flow can then be > used from ofproto/trace to figure out which parts of the pipeline lead > to this

Re: [ovs-dev] [PATCH] ovn-controller: Combine two Logical_Flow loops into one.

2016-08-11 Thread Russell Bryant
On Thu, Aug 11, 2016 at 8:09 AM, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > > > On Aug 7, 2016, at 6:54 AM, Ryan Moats wrote: > > > > > From: nickcooper-zhangtonghao

Re: [ovs-dev] [PATCH] ovn-controller: Remove the repeated code.

2016-08-11 Thread Russell Bryant
On Thu, Aug 11, 2016 at 8:09 AM, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > > > On Aug 7, 2016, at 7:22 AM, Ryan Moats wrote: > > > > > From: nickcooper-zhangtonghao

Re: [ovs-dev] [PATCH 3/3] datapath: compat: vxlan: fix vxlan_notify_add_rx_port().

2016-08-11 Thread pravin shelar
On Thu, Aug 11, 2016 at 1:54 PM, Jesse Gross wrote: > On Wed, Aug 10, 2016 at 10:34 AM, Pravin B Shelar wrote: >> Same as earlier patch this fixes vxlan recieve offload implementation. >> >> Signed-off-by: Pravin B Shelar > > Acked-by: Jesse

Re: [ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

2016-08-11 Thread Ryan Moats
"dev" wrote on 08/11/2016 07:20:34 PM: > From: Jesse Gross > To: dev@openvswitch.org > Date: 08/11/2016 07:22 PM > Subject: [ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP > checksums when creating Geneve tunnels. > Sent by: "dev"

[ovs-dev] [PATCH] datapath: compat: keep skb mark across tunnel devices.

2016-08-11 Thread Pravin B Shelar
Older kernel skb_scrub_packet() has bug which resets skb mark for all packet. It is fixed during 3.18 release where it is reset only for packets crossing namespace. So OVS is forced to use compat skb_scrub_packet() on older kernel. This is related to upstream bug fix commit ca7c7b9059e3 ("skbuff:

[ovs-dev] (no subject)

2016-08-11 Thread dev-bounces
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 09:53:50PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > On Thu, Aug 11, 2016 at 01:00:21PM -0400, Aaron Conole wrote: > >> Ben Pfaff writes: > >> > Neither of those is a simple "return server->listen_fd;" function. Any > >> > chance

[ovs-dev] [PATCH] ofproto-dpif-xlate: Fix VLOG_ERR_RL() call.

2016-08-11 Thread Joe Stringer
a716ef9a7a73 ("ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.") inadvertantly broke build on clang due to improper passing of the ds cstring into the VLOG() function: error: format string is not a string literal (potentially insecure) [-Werror,-Wformat-security]

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Fix VLOG_ERR_RL() call.

2016-08-11 Thread Daniele Di Proietto
2016-08-11 17:54 GMT-07:00 Joe Stringer : > a716ef9a7a73 ("ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.") > inadvertantly broke build on clang due to improper passing of the ds > cstring into the VLOG() function: > > error: format string is not a string literal >

Re: [ovs-dev] [PATCH] ofproto-dpif-xlate: Fix VLOG_ERR_RL() call.

2016-08-11 Thread Joe Stringer
On 11 August 2016 at 17:58, Daniele Di Proietto wrote: > > > 2016-08-11 17:54 GMT-07:00 Joe Stringer : >> >> a716ef9a7a73 ("ofproto-dpif-xlate: Log flow in XLATE_REPORT_ERROR.") >> inadvertantly broke build on clang due to improper passing of the ds >> cstring

Re: [ovs-dev] Connection Tracking with OVS

2016-08-11 Thread Joe Stringer
On 11 August 2016 at 17:21, Joji Mekkatt wrote: > Hello, > > I am trying to use connection tracking with OVS 2.5. I see the ovs-ofctl > documentation on this. Is there a corresponding support/extensions in > openflow for this? If so, which version of the spec should I refer to?

Re: [ovs-dev] [PATCH v3 0/3] vhost-user: Add the ability to control ownership/permissions

2016-08-11 Thread Aaron Conole
Ben Pfaff writes: > On Thu, Aug 11, 2016 at 01:00:21PM -0400, Aaron Conole wrote: >> Ben Pfaff writes: >> > Neither of those is a simple "return server->listen_fd;" function. Any >> > chance that would be accepted? It would solve the problem just fine. >> >> I

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Li, Johnson
> > Hi Jan, > > On Thu, Aug 11, 2016 at 12:58:44PM +, Jan Scheurich wrote: > > Hi Simon > > > > > > The implicit push_eth action introduced by Simon in his L3 tunnel > > > > port patch mainly ensures the presence of the 14 byte MAC header > > > > on ports where this is a must for syntactic

Re: [ovs-dev] [PATCH rebase 1/3] ovn-controller: Fix memory leak when updating tunnels.

2016-08-11 Thread Ryan Moats
"dev" wrote on 08/11/2016 07:20:32 PM: > From: Jesse Gross > To: dev@openvswitch.org > Date: 08/11/2016 07:21 PM > Subject: [ovs-dev] [PATCH rebase 1/3] ovn-controller: Fix memory > leak when updating tunnels. > Sent by: "dev"

[ovs-dev] [PATCH rebase 1/3] ovn-controller: Fix memory leak when updating tunnels.

2016-08-11 Thread Jesse Gross
When a tunnel possibly needs to be updated, we are currently allocating a new name for it. This is not necessary and in fact nothing uses the name, which then results in the memory being leaked. Fixes: 1d45d5a9 ("ovn-controller: Change encaps_run to work incrementally.") Signed-off-by: Jesse

[ovs-dev] [PATCH rebase 0/3] Enable Geneve checksums in OVN

2016-08-11 Thread Jesse Gross
The primary goal of this series is to enable outer UDP checksums for Geneve-encapsulated packets with OVN. The benefits of this are described in more detail in the final patch. Since this touches the encap code in ovn-controller, I noticed several issues in the existing implementation as a result

[ovs-dev] Connection Tracking with OVS

2016-08-11 Thread Joji Mekkatt
Hello, I am trying to use connection tracking with OVS 2.5. I see the ovs-ofctl documentation on this. Is there a corresponding support/extensions in openflow for this? If so, which version of the spec should I refer to? Any pointer is greatly appreciated. Thanks, Joji

Re: [ovs-dev] Failing Test 2214 on i686

2016-08-11 Thread Christian Ehrhardt
Hi Ben, we ran last week with openvswitch-2.6.0~git20160804.0a0f39d The patch you asked for was already in at the time. Maybe the attachements got dropped somewhere - my sent mail still had them. Anyway lets make sure you get the content via pastebin at least: I had logs when I

Re: [ovs-dev] [PATCH] datapath: compat: keep skb mark across tunnel devices.

2016-08-11 Thread Joe Stringer
On 11 August 2016 at 19:43, Pravin B Shelar wrote: > Older kernel skb_scrub_packet() has bug which resets skb mark for > all packet. It is fixed during 3.18 release where it is reset > only for packets crossing namespace. So OVS is forced to use > compat skb_scrub_packet() on

Re: [ovs-dev] [PATCH v1] ovn: rename Address_Set to Set to reflect a more broad purpose

2016-08-11 Thread Han Zhou
Make sense! Thanks for explain :) On Wednesday, August 10, 2016, Zong Kai Li wrote: > On Thu, Aug 11, 2016 at 1:51 PM, Han Zhou > wrote: > > > > > > On Tue, Aug 9, 2016 at 8:55 PM, Zong Kai LI > wrote: >

Re: [ovs-dev] [PATCH] openvswitch: do not ignore netdev errors when creating tunnel vports

2016-08-11 Thread David Miller
From: Martynas Pumputis Date: Tue, 9 Aug 2016 16:24:50 +0100 > The creation of a tunnel vport (geneve, gre, vxlan) brings up a > corresponding netdev, a multi-step operation which can fail. > > For example, changing a vxlan vport's netdev state to 'up' binds the > vport's

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Jan Scheurich
+ [ovs-dev] Further comments below. BR, Jan > -Original Message- > From: Li, Johnson [mailto:johnson...@intel.com] > Sent: Thursday, 11 August, 2016 03:36 > Subject: RE: NSH Option 2 implementation > > > > > Hi Danny, > > > > thanks, for your quick response. I look forward to hearing

Re: [ovs-dev] [PATCH v1] ovn: rename Address_Set to Set to reflect a more broad purpose

2016-08-11 Thread Zong Kai Li
On Thu, Aug 11, 2016 at 1:51 PM, Han Zhou wrote: > > > On Tue, Aug 9, 2016 at 8:55 PM, Zong Kai LI wrote: >> >> This patch renames table Address_Set to Set, Address_Set.addresses to >> Set.members to reflect a more broad purpose, that we can define other >>

Re: [ovs-dev] [PATCH v4] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-11 Thread Jan Scheurich
Hi Daniele, Thanks for the review. I will send v5 asap. A few answers inline below. BR, Jan From: Daniele Di Proietto [mailto:diproiet...@ovn.org] Sent: Thursday, 11 August, 2016 02:59 To: Jan Scheurich Cc: dev@openvswitch.org; antonio.fische...@intel.com Subject: Re: [ovs-dev] [PATCH v4]

Re: [ovs-dev] NSH Option 2 implementation

2016-08-11 Thread Li, Johnson
> > + [ovs-dev] > > Further comments below. > > BR, Jan > > > -Original Message- > > From: Li, Johnson [mailto:johnson...@intel.com] > > Sent: Thursday, 11 August, 2016 03:36 > > Subject: RE: NSH Option 2 implementation > > > > > > > > Hi Danny, > > > > > > thanks, for your quick

[ovs-dev] [PATCH v5] dpif-netdev: dpcls per in_port with sorted subtables

2016-08-11 Thread Jan Scheurich
The user-space datapath (dpif-netdev) consists of a first level "exact match cache" (EMC) matching on 5-tuples and the normal megaflow classifier. With many parallel packet flows (e.g. TCP connections) the EMC becomes inefficient and the OVS forwarding performance is determined by the megaflow

Re: [ovs-dev] [PATCH rebase 1/3] ovn-controller: Fix memory leak when updating tunnels.

2016-08-11 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 05:20:32PM -0700, Jesse Gross wrote: > When a tunnel possibly needs to be updated, we are currently allocating > a new name for it. This is not necessary and in fact nothing uses the > name, which then results in the memory being leaked. > > Fixes: 1d45d5a9

[ovs-dev] Mail System Error - Returned Mail

2016-08-11 Thread Returned mail
ÂN ƒÊ’e£©×ä’C"[‡fš<µOs«â{-ÏxœT´Øf\( )lјùpÙÞ®1èçŠÍu hé”Ik¸_ÌðAh¾8HDÃB–¸÷ܛ¤àdtúØïj(BjHë}ÞÔëCÜ:,p©üú䗘p}Å&‘÷í]Ò&˜“®®ŽÝð¸>”ŸòíÑí$½Ÿ[¾Â,‚Ò  yÅ©xPmÎÒâpǂ«ÇbÖ\²ÔžK25¯òznԓpš`…9}æ[³f¯ÑU…l!,?¢úV­*ü…±–#Gäû;4à™Á£TšÎHjÅÇì Ú¤l,à6¿ªM

[ovs-dev] [PATCH rebase 2/3] ovn-controller: Make encap processing more robust against changes.

2016-08-11 Thread Jesse Gross
Originally, processing of encapsulations simply iterated over all tables on every wakeup and would replace anything that changed. This is somewhat inefficient but it captured all changes. Incremental processing avoided the need to do so much work but it could miss several types of changes. In

[ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

2016-08-11 Thread Jesse Gross
Currently metadata transmitted by OVN over Geneve tunnels is unprotected by any checksum other than the one provided by the link layer - this includes both the VNI and data stored in options. Turning on UDP checksums which cover this data has obvious benefits in terms of integrity protection. In