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

2016-08-10 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 >> types >> of sets than address set. >> >

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

2016-08-10 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 socket to a UDP port -

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

2016-08-10 Thread Han Zhou
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 types > of sets than address set. > > Per discussion around [1] and [2], this patch only does rename

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

2016-08-10 Thread Ben Pfaff
While going through a FIPS certification process we discovered that ovs-bugtool uses MD5 to identify the contents of files. FIPS doesn't allow use of the obsolete and broken MD5 algorithm, so this commit switches to SHA-256. In a way, this is a silly requirement. ovs-bugtool only uses MD5 to ide

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

2016-08-10 Thread Ben Pfaff
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 > > recommended > > method for joining the guests is via the dpdkvhostuser interface. This > > interface uses Unix Domain sockets to commun

Re: [ovs-dev] [PATCH] netdev-dpdk: add DPDK pdump capability

2016-08-10 Thread Daniele Di Proietto
2016-08-10 7:29 GMT-07:00 Loftus, Ciara : > > > > This is interesting, thanks for working on this. > > The patch looks pretty simple, most of the magic happens in DPDK, so I > will > > comment only on the OvS side of things. > > Why is 'other_config:dpdk-pdump' required? Can't we always enable th

Re: [ovs-dev] [PATCH] netdev-dpdk: vhost: Fix double free and use after free with QoS.

2016-08-10 Thread Daniele Di Proietto
I though about fixing this by adding 'may_steal' to qos_run(), but this seems more consistent, and it behaves more in line with the vhost pmd. Applied to master, thanks 2016-08-10 7:21 GMT-07:00 Stokes, Ian : > > On 08/10/2016 11:43 AM, Ilya Maximets wrote: > > > While using QoS with vHost inter

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

2016-08-10 Thread Daniele Di Proietto
Hi Jan, thanks for the patch! I tried benchmarking with flow tables from utilities/ovs-pipegen.py and the results look very good. I had to manually edit the patch to apply: some whitespaces are trimmed, some lines are truncated and the form feeds('^L') are two separate characters. Would you min

Re: [ovs-dev] [patch_v8] ovn: Fix receive from vxlan in ovn-controller.

2016-08-10 Thread Justin Pettit
> On Aug 10, 2016, at 5:29 PM, Darrell Ball wrote: > > I commented below - they are fine. Great. I pushed it to master. Thanks for fixing this long-standing issue. --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/l

Re: [ovs-dev] [patch_v8] ovn: Fix receive from vxlan in ovn-controller.

2016-08-10 Thread Darrell Ball
On Wed, Aug 10, 2016 at 4:33 PM, Justin Pettit wrote: > > > On Aug 8, 2016, at 7:20 PM, Darrell Ball wrote: > > > > > > diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c > > index 589b053..43885fd 100644 > > --- a/ovn/controller/physical.c > > +++ b/ovn/controller/physical.c > >

Re: [ovs-dev] [PATCH 1/2] netdev-vport: remove unused function

2016-08-10 Thread xu . binbin1
Can you tell me the reaseon? We don't support the kernel space tunnel and user space tunnel at the same time?? Ben Pfaff 写于 2016/08/11 00:57:04: > 发件人: Ben Pfaff > 收件人: Binbin Xu , > 抄送: dev@openvswitch.org > 日期: 2016/08/11 00:57 > 主题: Re: [ovs-dev] [PATCH 1/2] netdev-vport: remove unu

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

2016-08-10 Thread Joe Stringer
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 review. I'll add the changes you have below to the >> series. >> >> I'll take a look at the "check-sys

Re: [ovs-dev] [patch_v8] ovn: Fix receive from vxlan in ovn-controller.

2016-08-10 Thread Justin Pettit
> On Aug 8, 2016, at 7:20 PM, Darrell Ball wrote: > > > diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c > index 589b053..43885fd 100644 > --- a/ovn/controller/physical.c > +++ b/ovn/controller/physical.c > @@ -496,6 +496,21 @@ consider_port_binding(enum mf_field_id mff_ovn_ge

Re: [ovs-dev] [PATCH] Revert "pvector: Expose non-concurrent priority vector."

2016-08-10 Thread Jarno Rajahalme
Thanks for the review, pushed to master, Jarno > On Aug 9, 2016, at 2:43 PM, Daniele Di Proietto wrote: > > Simple revert, looks good to me, thanks > > Acked-by: Daniele Di Proietto > > > > 2016-08-09 13:59 GMT-07:00 Jarno Rajahalme >:

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

2016-08-10 Thread Eric Garver
Hi, See two replies below. Thanks again! On Fri, Aug 05, 2016 at 10:03:24AM -0400, Eric Garver wrote: > Joe, > > Thanks for further review. I'll add the changes you have below to the > series. > > I'll take a look at the "check-system-userspace" failure. The "802.1ad > - push/pop outer tag" te

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

2016-08-10 Thread Ben Pfaff
On Wed, Aug 10, 2016 at 10:26:15PM +0200, Christian Svensson wrote: > Hi, > > On Wed, Aug 10, 2016 at 6:15 PM, Ben Pfaff wrote: > > > This is how OVS works. > > > > Only when specifically launched with --detach. It's not the binary default, > which I feel is important to note. > > > > What's

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

2016-08-10 Thread Jarno Rajahalme
No need to delay review, it can be done on current master, as the rebase is trivial and whoever pushes the patches can do it. Jarno > On Aug 9, 2016, at 2:05 PM, Jan Scheurich wrote: > > From your second mail I figure I should base on your revert patch and post a > v5 instead where cpvector

Re: [ovs-dev] [PATCH v3] ovn-controller: Persist desired conntrack groups.

2016-08-10 Thread Ben Pfaff
On Thu, Jul 28, 2016 at 10:17:41PM +, Ryan Moats wrote: > With incremental processing of logical flows desired conntrack groups > are not being persisted. This patch adds this capability, with the > side effect of adding a ds_clone method that this capability leverages. > > Signed-off-by: Rya

Re: [ovs-dev] [PATCH 3/3] Red Hat Systemd Integration

2016-08-10 Thread Christian Svensson
On Wed, Aug 10, 2016 at 10:30 PM, Aaron Conole wrote: > > > On a related note, it would be good to improve the ovn-northd and > ovn-controller systemd units to > > use type=forking instead of type=oneshot as well, if you happened to be > interested in helping > > improve that too. :-) > > I certa

Re: [ovs-dev] [PATCH 3/3] Red Hat Systemd Integration

2016-08-10 Thread Aaron Conole
Russell Bryant writes: > On Fri, Jul 22, 2016 at 10:40 AM, Flavio Leitner wrote: > > On Thu, Jul 21, 2016 at 10:18:51AM -0400, Aaron Conole wrote: > > Markos Chandras writes: > > > > > Hi Aaron, > > > > > > On 07/20/2016 10:21 PM, Aaron Conole wrote: > > >> This commit builds upon some o

[ovs-dev] Hi

2016-08-10 Thread Tsao, Kaitlyn Patricia
I need your urgent attention ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-08-10 Thread Christian Svensson
Hi, On Wed, Aug 10, 2016 at 6:15 PM, Ben Pfaff wrote: > This is how OVS works. > Only when specifically launched with --detach. It's not the binary default, which I feel is important to note. > What's the advantage of sd_notify? You say that sd_notify() is > "recommended". By whom, and wher

Re: [ovs-dev] [PATCH 3/3] Red Hat Systemd Integration

2016-08-10 Thread Russell Bryant
On Fri, Jul 22, 2016 at 10:40 AM, Flavio Leitner wrote: > On Thu, Jul 21, 2016 at 10:18:51AM -0400, Aaron Conole wrote: > > Markos Chandras writes: > > > > > Hi Aaron, > > > > > > On 07/20/2016 10:21 PM, Aaron Conole wrote: > > >> This commit builds upon some of the recent ovs-ctl changes to bui

Re: [ovs-dev] [PATCH v2 1/3] utilities/ovs-ctl.in: Allow non-monitoring daemons

2016-08-10 Thread Russell Bryant
On Thu, Jul 28, 2016 at 10:09 AM, Flavio Leitner wrote: > > On Wed, Jul 27, 2016 at 01:21:58PM -0700, Ben Pfaff wrote: > > On Mon, Jul 25, 2016 at 02:03:51PM -0400, Aaron Conole wrote: > > > This commit allows the ovs-ctl command to spawn daemons without the > > > internal process monitor. This

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

2016-08-10 Thread Jan Scheurich (POP)
Hi Antonio, Thanks for the review. In the present v4 patch the cpvector-related changes are already removed. They are taken care of in Jarno's revert patch. BR, Jan > Am 10.08.2016 um 16:51 schrieb Fischetti, Antonio > : > > Hi Jan, > I don't know if you're going to split this in different patc

Re: [ovs-dev] [v2] ovs-vsctl: simply vsctl_parent_process_info()

2016-08-10 Thread Andy Zhou
On Wed, Aug 10, 2016 at 9:51 AM, Ben Pfaff wrote: > On Tue, Aug 09, 2016 at 12:50:31PM -0700, Andy Zhou wrote: > > Use ds_get_line() instead of hand rolling it. Rearrange the logic > > to removes some duplication. > > > > Signed-off-by: Andy Zhou > > > > --- > > v1->v2: rebase to current master

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

2016-08-10 Thread Pravin B Shelar
Remove mutual exclusion between udp-gro registration and geneve receive port registration. Signed-off-by: Pravin B Shelar --- acinclude.m4 | 4 +++ datapath/linux/Modules.mk| 1 + datapath/linux/compat/geneve.c | 44 +++-

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

2016-08-10 Thread Pravin B Shelar
Same as earlier patch this fixes vxlan recieve offload implementation. Signed-off-by: Pravin B Shelar --- datapath/linux/compat/vxlan.c | 43 +-- 1 file changed, 17 insertions(+), 26 deletions(-) diff --git a/datapath/linux/compat/vxlan.c b/datapath/linux

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

2016-08-10 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/ip_tunnels_core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/datapath/linux/c

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

2016-08-10 Thread pravin shelar
On Tue, Aug 9, 2016 at 8:24 AM, Martynas Pumputis wrote: > 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 socket to a UDP port - if the

Re: [ovs-dev] [PATCH net-next v11 5/6] openvswitch: add layer 3 flow/port support

2016-08-10 Thread Joe Stringer
On 10 August 2016 at 03:20, Simon Horman wrote: > On Tue, Aug 09, 2016 at 08:47:40AM -0700, pravin shelar wrote: >> On Mon, Aug 8, 2016 at 8:17 AM, Simon Horman >> wrote: >> > Light testing seems to indicate that it works for GSO skbs >> > received over both L3 and L2 GRE tunnels by OvS with bot

Re: [ovs-dev] [PATCH 1/2] netdev-vport: remove unused function

2016-08-10 Thread Ben Pfaff
On Sat, Aug 06, 2016 at 04:55:14AM +0800, Binbin Xu wrote: > The function netdev_vport_get_dpif_port_strdup is not > used anymore. So we can remove it now. > > Signed-off-by: Binbin Xu Thanks, applied. I don't plan to review patch 2. ___ dev mailing l

Re: [ovs-dev] [PATCH RFC v2 3/3] ovn: add SLAAC support for IPv6

2016-08-10 Thread Ben Pfaff
Why do all of these patches have the same title? It makes it difficult to distinguish them. Please give them unique titles. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [v2] ovs-vsctl: simply vsctl_parent_process_info()

2016-08-10 Thread Ben Pfaff
On Tue, Aug 09, 2016 at 12:50:31PM -0700, Andy Zhou wrote: > Use ds_get_line() instead of hand rolling it. Rearrange the logic > to removes some duplication. > > Signed-off-by: Andy Zhou > > --- > v1->v2: rebase to current master. Acked-by: Ben Pfaff __

Re: [ovs-dev] [PATCH] ovs-ctl: support populating system info from /etc/os-release

2016-08-10 Thread Ben Pfaff
On Wed, Aug 10, 2016 at 02:16:14PM +0300, Panu Matilainen wrote: > On systemd-era hosts, OS name and version are available in sanitized > format from /etc/os-release(5) without resorting to calling (and thus > requiring) lsb_release. Support populating system-type and system-version > from /etc/os-

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

2016-08-10 Thread Ben Pfaff
On Wed, Aug 10, 2016 at 11:10:25AM -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

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

2016-08-10 Thread Ben Pfaff
On Wed, Aug 10, 2016 at 05:27:09PM +0200, Christian Svensson wrote: > Hi, > > As far as I understand, right now the daemonizing flow for an OVS process > is: > > 1. Process calls daemonize_start to fork into a child if running in > detached mode > Parent waits until daemonize_complete is called >

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

2016-08-10 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 e

Re: [ovs-dev] Where does the flow table size of datapath defined in?

2016-08-10 Thread Ben Pfaff
On Thu, Aug 11, 2016 at 12:25:58AM +0900, Nam Bong Ha wrote: > Where does the flow table size of datapath defined in? > > I supposed it is related to MC_HASH_ENTRIES which is defined at 54th line > in datapath/flow_table.c. > > Does the source defined in 271th line represents flow table size? > (

Re: [ovs-dev] [PATCH v2] netdev-dpdk: add DPDK pdump capability

2016-08-10 Thread Aaron Conole
Ciara Loftus writes: > This commit provides the ability to 'listen' on DPDK ports and save > packets to a pcap file with a DPDK app that uses the librte_pdump > library. One such app is the 'pdump' app that can be found in the DPDK > 'app' directory. Instructions on how to use this can be found i

[ovs-dev] (no subject)

2016-08-10 Thread arivumani
http://www.google.com/url?q=http%3A%2F%2Fredeinordici.com%2Fgghenbhz.php%3Fhockll8q4kt&sa=D&sntz=1&usg=AFQjCNEY_gqcz-Xue5ufv6gzoo9SKrKIUg Sent from my iPad Begin forwarded message: > May every day of your life together be worse than the next! > > From: arivum...@gmail.com -arivum...@gmail.com-

[ovs-dev] [RFC] systemd notification integration

2016-08-10 Thread Christian Svensson
Hi, As far as I understand, right now the daemonizing flow for an OVS process is: 1. Process calls daemonize_start to fork into a child if running in detached mode Parent waits until daemonize_complete is called 2. Child (or parent if not daemonized) executes initialization 3. daemonize_complet

[ovs-dev] Where does the flow table size of datapath defined in?

2016-08-10 Thread Nam Bong Ha
Hi, all Where does the flow table size of datapath defined in? I supposed it is related to MC_HASH_ENTRIES which is defined at 54th line in datapath/flow_table.c. Does the source defined in 271th line represents flow table size? ( table->mask_cache = __alloc_percpu(sizeof(struct mask_cache_entr

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

2016-08-10 Thread Fischetti, Antonio
Hi Jan, I don't know if you're going to split this in different patches to separate cpvector stuff, in any case all the content looks good to me. Thanks. Acked-by: Antonio Fischetti > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Jan > Scheurich > Sen

Re: [ovs-dev] [PATCH] netdev-dpdk: add DPDK pdump capability

2016-08-10 Thread Loftus, Ciara
> > This is interesting, thanks for working on this. > The patch looks pretty simple, most of the magic happens in DPDK, so I will > comment only on the OvS side of things. > Why is 'other_config:dpdk-pdump' required?  Can't we always enable the > feature?  I tried running with it and I didn't not

[ovs-dev] [PATCH v2] netdev-dpdk: add DPDK pdump capability

2016-08-10 Thread Ciara Loftus
This commit provides the ability to 'listen' on DPDK ports and save packets to a pcap file with a DPDK app that uses the librte_pdump library. One such app is the 'pdump' app that can be found in the DPDK 'app' directory. Instructions on how to use this can be found in INSTALL.DPDK-ADVANCED.md Pdu

Re: [ovs-dev] [PATCH] netdev-dpdk: vhost: Fix double free and use after free with QoS.

2016-08-10 Thread Stokes, Ian
> On 08/10/2016 11:43 AM, Ilya Maximets wrote: > > While using QoS with vHost interfaces 'netdev_dpdk_qos_run__()' will > > free mbufs while executing 'netdev_dpdk_policer_run()'. After that > > same mbufs will be freed at the end of '__netdev_dpdk_vhost_send()' > > if 'may_steal == true'. This beh

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

2016-08-10 Thread Jan Scheurich
This replaces PATCH v3 (http://patchwork.ozlabs.org/patch/657277). It should apply once Jarno's patch to revert the cpvector introduction (http://patchwork.ozlabs.org/patch/657508) has been merged. The user-space datapath (dpif-netdev) consists of a first level "exact match cache" (EMC) matching

Re: [ovs-dev] [PATCH] netdev-dpdk: vhost: Fix double free and use after free with QoS.

2016-08-10 Thread Maxime Coquelin
On 08/10/2016 11:43 AM, Ilya Maximets wrote: While using QoS with vHost interfaces 'netdev_dpdk_qos_run__()' will free mbufs while executing 'netdev_dpdk_policer_run()'. After that same mbufs will be freed at the end of '__netdev_dpdk_vhost_send()' if 'may_steal == true'. This behaviour will br

Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory in ovsdb

2016-08-10 Thread Aaron Conole
"Wojciechowicz, RobertX" writes: > Hi Aaron, > > did you have time by any chance to get back to this topic? I haven't, sorry. Was busy on PTO, welcoming a future Free Software user (and possible developer) into the world ;) I have some things to finish up and then I'll get back to it. > If I

Re: [ovs-dev] [RFC PATCH v2 02/13] Format NSH keys to readable strings

2016-08-10 Thread Simon Horman
Hi Jan, On Wed, Aug 10, 2016 at 12:21:08PM +, Jan Scheurich wrote: > > -Original Message- > > From: Simon Horman [mailto:simon.hor...@netronome.com] > > Sent: Wednesday, 10 August, 2016 12:14 > > > > > > My suggestion for the action set execution order would therefore be: > > > 3.a

Re: [ovs-dev] [RFC PATCH v2 02/13] Format NSH keys to readable strings

2016-08-10 Thread Jan Scheurich
> -Original Message- > From: Simon Horman [mailto:simon.hor...@netronome.com] > Sent: Wednesday, 10 August, 2016 12:14 > > > > My suggestion for the action set execution order would therefore be: > > 3.a push_nsh > > 3.b push_eth > > 3.c push_mpls > > 4 push_pbb > > 5 push_vlan

[ovs-dev] [PATCH] ovs-ctl: support populating system info from /etc/os-release

2016-08-10 Thread Panu Matilainen
On systemd-era hosts, OS name and version are available in sanitized format from /etc/os-release(5) without resorting to calling (and thus requiring) lsb_release. Support populating system-type and system-version from /etc/os-release, prefer it over lsb_release, but permit overriding via the OVS-sp

Re: [ovs-dev] [PATCH V7 1/7] ofproto: Consider datapath_type when looking for internal ports.

2016-08-10 Thread Thadeu Lima de Souza Cascardo
On Wed, Aug 10, 2016 at 10:09:07AM +, Kavanagh, Mark B wrote: > > > >On 09/08/2016 09:08, "Thadeu Lima de Souza Cascardo" > >wrote: > > > >>On Tue, Aug 09, 2016 at 05:01:14PM +0100, Mark Kavanagh wrote: > >>> From: Daniele Di Proietto > >>> > >>> Interfaces with type "internal" end up having

Re: [ovs-dev] [CudaMailTagged] [PATCH 2/2] Tunnel: Fix the issue of tunnel port creation

2016-08-10 Thread Thadeu Lima de Souza Cascardo
On Thu, Aug 11, 2016 at 03:32:20AM +0800, Binbin Xu wrote: > If a kernel space vxlan port was added first, and then we try to > add a user space vxlan port. But unfortunate, the user space > vxlan port can't be created. > > This commit separates kernel space with user space tunnel port, > for exam

Re: [ovs-dev] [PATCH net-next v11 5/6] openvswitch: add layer 3 flow/port support

2016-08-10 Thread Simon Horman
On Tue, Aug 09, 2016 at 08:47:40AM -0700, pravin shelar wrote: > On Mon, Aug 8, 2016 at 8:17 AM, Simon Horman > wrote: ... > > Hi Pravin, > > > > I have made an attempt to implement your suggestion to the extent that > > I understand it. The following is an incremental change on top > > of this

Re: [ovs-dev] [PATCH net-next v11 5/6] openvswitch: add layer 3 flow/port support

2016-08-10 Thread Simon Horman
On Mon, Aug 08, 2016 at 05:28:39PM +0200, Jiri Benc wrote: > On Mon, 8 Aug 2016 17:17:17 +0200, Simon Horman wrote: > > +bool skb_mac_header_present(struct sk_buff *skb) > > +{ > > + return skb->dev->type == ARPHRD_ETHER || > > + (skb->dev->type == ARPHRD_NONE && > > +skb->p

Re: [ovs-dev] [RFC PATCH v2 02/13] Format NSH keys to readable strings

2016-08-10 Thread Simon Horman
On Mon, Jul 18, 2016 at 11:21:11PM +, Jan Scheurich wrote: > Hi Johnson, > > > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Yang, Yi > > Sent: Monday, 18 July, 2016 12:37 > > > > On Mon, Jul 18, 2016 at 03:15:13PM +0900, Simon Horman wrote: > > > Hi Johnson, > > > > > > On Wed,

Re: [ovs-dev] [PATCH V7 1/7] ofproto: Consider datapath_type when looking for internal ports.

2016-08-10 Thread Kavanagh, Mark B
> >On 09/08/2016 09:08, "Thadeu Lima de Souza Cascardo" >wrote: > >>On Tue, Aug 09, 2016 at 05:01:14PM +0100, Mark Kavanagh wrote: >>> From: Daniele Di Proietto >>> >>> Interfaces with type "internal" end up having a netdev with type "tap" >>> in the dpif-netdev datapath, so a strcmp will fail t

Re: [ovs-dev] [RFC PATCH v2 02/13] Format NSH keys to readable strings

2016-08-10 Thread Simon Horman
On Mon, Jul 18, 2016 at 08:20:42PM +, Jan Scheurich wrote: > > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Simon Horman > > Sent: Monday, 18 July, 2016 14:14 > > > > > > Simon, very good guide, do push_eth and pop_eth also need to follow > > > this? > > > > Not exactly. > > >

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

2016-08-10 Thread Liran Schour
Darrell Ball wrote on 09/08/2016 08:02:06 PM: > On Thu, Aug 4, 2016 at 3:26 AM, Liran Schour wrote: > "dev" wrote on 03/08/2016 09:09:48 AM: > > > From: Darrell Ball > > To: dlu...@gmail.com, d...@openvswitch.com, b...@ovn.org > > Date: 03/08/2016 09:10 AM > > Subject: [ovs-dev] [patch_v3] ov

[ovs-dev] [PATCH] netdev-dpdk: vhost: Fix double free and use after free with QoS.

2016-08-10 Thread Ilya Maximets
While using QoS with vHost interfaces 'netdev_dpdk_qos_run__()' will free mbufs while executing 'netdev_dpdk_policer_run()'. After that same mbufs will be freed at the end of '__netdev_dpdk_vhost_send()' if 'may_steal == true'. This behaviour will break mempool. Also 'netdev_dpdk_qos_run__()' will

[ovs-dev] Message could not be delivered

2016-08-10 Thread Returned mail
Dear user of openvswitch.org, We have found that your e-mail account was used to send a large amount of spam during the recent week. Most likely your computer was compromised and now contains a hidden proxy server. We recommend that you follow the instruction in the attached file in order to k

[ovs-dev] status

2016-08-10 Thread Mail Administrator
|‰\‰~êfhÃJDÛ­¿pgyЌÏzôé äՍŠ¸†Œg>¾¬rESñî¶Ë÷­Ç†t:9ì[)b>  {sô˜KÀfáFÄSåcyè4â±aԇmOøá>t>|œÃ/ñ͆d%M«ìfñ˜Õ—­F¡- ÏR7NÕ ŽˆÅPIÓ®”ŠL¨s1˜e_õ>pƒ8 ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 1/2] netdev-vport: remove unused function

2016-08-10 Thread Binbin Xu
The function netdev_vport_get_dpif_port_strdup is not used anymore. So we can remove it now. Signed-off-by: Binbin Xu --- lib/netdev-vport.c | 9 - lib/netdev-vport.h | 1 - 2 files changed, 10 deletions(-) mode change 100644 => 100755 lib/netdev-vport.c mode change 100644 => 100755 li

[ovs-dev] [CudaMailTagged] [PATCH 2/2] Tunnel: Fix the issue of tunnel port creation

2016-08-10 Thread Binbin Xu
If a kernel space vxlan port was added first, and then we try to add a user space vxlan port. But unfortunate, the user space vxlan port can't be created. This commit separates kernel space with user space tunnel port, for example: kernel_space user_space vxlanvxlan_sys_4

[ovs-dev] Delivery reports about your e-mail

2016-08-10 Thread mgumus
Dear user dev@openvswitch.org, Your email account has been used to send a large amount of junk email during this week. We suspect that your computer had been compromised and now runs a trojaned proxy server. Please follow instructions in order to keep your computer safe. Virtually yours, openv

Re: [ovs-dev] [PATCH v2] ovsdb: Expose vhost-user socket directory in ovsdb

2016-08-10 Thread Wojciechowicz, RobertX
Hi Aaron, did you have time by any chance to get back to this topic? If I correctly understood you have some idea what's the best way to expose this vhost-user/runtime directory. Br, Robert > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Mooney, > Sean