[ovs-dev] meet you well

2016-08-03 Thread SUSAN DOUGLAS
I am Mrs.Zeinab Abdul Can i ask you for help ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] ovs-rcu: Add new ovsrcu_index type.

2016-08-03 Thread Daniele Di Proietto
Applied to master, thanks! On 03/08/2016 15:00, "Jarno Rajahalme" wrote: >Looks good to me, > >Acked-by: Jarno Rajahalme > >> On Aug 2, 2016, at 5:03 PM, Daniele Di Proietto >> wrote: >> >> With RCU in Open vSwitch it's very easy to

[ovs-dev] [PATCH 1/4] ovsdb: Rename replication related variable names.

2016-08-03 Thread Andy Zhou
Current replication code refers the other ovsdb-sever instance as a 'remote'. which is overloaded in ovsdb. Switching to use active/backup instead to make it less confusing. Active is the server that should be servicing the client, backup server is the server that boots with the --sync-from

[ovs-dev] [PATCH 2/4] ovsdb: add replication wait

2016-08-03 Thread Andy Zhou
Pollblock requires the run() function to be paired with a wait() function. Add one for replication. Signed-off-by: Andy Zhou --- ovsdb/ovsdb-server.c | 3 +++ ovsdb/replication.c | 8 ovsdb/replication.h | 1 + 3 files changed, 12 insertions(+) diff --git

[ovs-dev] [PATCH 4/4] ovsdb: Make OVSDB backup sever read only

2016-08-03 Thread Andy Zhou
When ovsdb-sever is running in the backup state, it would be nice to make sure there is no un-intended changes to the backup database. This patch makes the ovsdb server only accepts 'read' transactions as a backup server. When the server role is changed into an active server, all existing client

[ovs-dev] [PATCH 3/4] ovsdb: Fix bug, set rpc to NULL after freeing.

2016-08-03 Thread Andy Zhou
Found by inspection Signed-off-by: Andy Zhou --- ovsdb/replication.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/ovsdb/replication.c b/ovsdb/replication.c index 19626a5..de6eec1 100644 --- a/ovsdb/replication.c +++ b/ovsdb/replication.c @@ -150,6 +150,7

Re: [ovs-dev] [PATCH v3 1/3] Windows: Add internal switch port per OVS bridge

2016-08-03 Thread Sairam Venugopal
Nice work putting this together! I went over the changes and had some comments. Haven¹t tested it yet. Thanks, Sairam On 8/2/16, 12:46 PM, "Alin Serdean" wrote: >This patch updates the following commands in the vswitch: >ovs-vsctl add-br br-test >ovs-vsctl

[ovs-dev] Mail System Error - Returned Mail

2016-08-03 Thread tobias . lorenz
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] datapath: Remove incorrect WARN_ONCE().

2016-08-03 Thread Joe Stringer
On 3 August 2016 at 15:25, Jarno Rajahalme wrote: > Upstream commit: > ovs_ct_find_existing() issues a warning if an existing conntrack entry > classified as IP_CT_NEW is found, with the premise that this should > not happen. However, a newly confirmed, non-expected

Re: [ovs-dev] [PATCH v3 3/3] Windows: document multiple NIC support setup

2016-08-03 Thread Sairam Venugopal
Hi Alin, Thanks for the patch! Had some inlined comments about the documentation. I will run some tests and send out the review comments for other 2 patches. Thanks, Sairam On 8/2/16, 12:51 PM, "Alin Serdean" wrote: >This patch updates the documentation on

Re: [ovs-dev] [PATCH 6/6] datapath: compat: gso: tighen checks for compat GSO code.

2016-08-03 Thread pravin shelar
On Wed, Aug 3, 2016 at 12:48 PM, Jesse Gross wrote: > On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: >> Few function can be compiled out for non GSO case. This >> patch make it bit cleaner to understand GSO compat code. >> >> Signed-off-by: Pravin B

Re: [ovs-dev] [PATCH v1] ovn: Extend Address_Set to Macros to support define port name sets

2016-08-03 Thread Ben Pfaff
On Wed, Aug 03, 2016 at 11:14:20PM +0800, Zong Kai LI wrote: > This patch aims to extend Address_Set to Macros, make it more common to > accept variable set, not only address. And by that, we can skinny down ACLs, > if we use Macros to defines port name sets for ACL to use, since lots of ACL >

[ovs-dev] [PATCH] datapath: Remove incorrect WARN_ONCE().

2016-08-03 Thread Jarno Rajahalme
Upstream commit: ovs_ct_find_existing() issues a warning if an existing conntrack entry classified as IP_CT_NEW is found, with the premise that this should not happen. However, a newly confirmed, non-expected conntrack entry remains IP_CT_NEW as long as no reply direction traffic

Re: [ovs-dev] [PATCH v4 1/4] Add support for 802.1ad (QinQ tunneling)

2016-08-03 Thread Ben Pfaff
On Wed, Aug 03, 2016 at 10:25:21AM -0400, Eric Garver wrote: > On Sun, Jul 31, 2016 at 08:22:47AM +0800, Xiao Liang wrote: > > On Thu, Jul 28, 2016 at 2:40 AM, Ben Pfaff wrote: > > > On Tue, Jul 12, 2016 at 11:38:54PM +0800, Xiao Liang wrote: > .. snip .. > > > I'm concerned about

Re: [ovs-dev] [PATCH 1/6] datapath: compat: Use udp-checksum function for compat case.

2016-08-03 Thread pravin shelar
On Wed, Aug 3, 2016 at 2:08 PM, Jesse Gross wrote: > On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: >> udp_set_csum() has bug fix that is not relevant for upstream >> (commit c77d947191b0). >> So OVS need to use compat function. This function is also

Re: [ovs-dev] [PATCH v4 1/4] Add support for 802.1ad (QinQ tunneling)

2016-08-03 Thread Ben Pfaff
Thanks for the replies, I have some further responses below. On Sun, Jul 31, 2016 at 08:22:47AM +0800, Xiao Liang wrote: > On Thu, Jul 28, 2016 at 2:40 AM, Ben Pfaff wrote: > > I'm concerned about backward compatibility. Consider some application > > built on Open vSwitch using

Re: [ovs-dev] [PATCH] ovs-rcu: Add new ovsrcu_index type.

2016-08-03 Thread Jarno Rajahalme
Looks good to me, Acked-by: Jarno Rajahalme > On Aug 2, 2016, at 5:03 PM, Daniele Di Proietto > wrote: > > With RCU in Open vSwitch it's very easy to protect objects accessed by > a pointer, but sometimes a pointer is not available. > > One example is

Re: [ovs-dev] [PATCH 2/6] datapath: compat: Use checksum offload for outer header.

2016-08-03 Thread pravin shelar
On Wed, Aug 3, 2016 at 2:07 PM, Jesse Gross wrote: > On Wed, Aug 3, 2016 at 9:00 AM, pravin shelar wrote: >> On Tue, Aug 2, 2016 at 4:55 PM, Jesse Gross wrote: >>> On Tue, Aug 2, 2016 at 3:55 PM, pravin shelar wrote: On

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Ben Pfaff
On Wed, Aug 03, 2016 at 11:58:52AM -0400, Russell Bryant wrote: > On Wed, Aug 3, 2016 at 11:39 AM, Kyle Mestery wrote: > > > On Wed, Aug 3, 2016 at 10:30 AM, Ryan Moats wrote: > > > > > > Russell Bryant wrote on 08/03/2016 10:11:57 AM: >

Re: [ovs-dev] [PATCH 2/6] datapath: compat: Use checksum offload for outer header.

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 9:00 AM, pravin shelar wrote: > On Tue, Aug 2, 2016 at 4:55 PM, Jesse Gross wrote: >> On Tue, Aug 2, 2016 at 3:55 PM, pravin shelar wrote: >>> On Tue, Aug 2, 2016 at 3:11 PM, Jesse Gross wrote: On

Re: [ovs-dev] [PATCH 1/6] datapath: compat: Use udp-checksum function for compat case.

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: > udp_set_csum() has bug fix that is not relevant for upstream > (commit c77d947191b0). > So OVS need to use compat function. This function is also > used from UDP xmit path so we have to check USE_UPSTREAM_TUNNEL. >

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

2016-08-03 Thread Christian Svensson
On Tue, Aug 2, 2016 at 12:42 PM, Christian Svensson wrote: > I just noticed that the first patch only was a URL, retrying with the real > patch attached. This is weird. I'm unable to attach the patch, and 'git send-email' based patches never show up in the archive, even

Re: [ovs-dev] [PATCH 6/6] datapath: compat: gso: tighen checks for compat GSO code.

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: > Few function can be compiled out for non GSO case. This > patch make it bit cleaner to understand GSO compat code. > > Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross

Re: [ovs-dev] [PATCH 5/6] datapath: backport: geneve: fix max_mtu setting

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: > Upstream commit: > commit d5d5e8d55732c7c35c354e45e3b0af2795978a57 > Author: Haishuang Yan > Date: Sat Jul 2 15:02:48 2016 +0800 > > geneve: fix max_mtu setting > >

Re: [ovs-dev] [PATCH 4/6] datapath: backport: openvswitch: fix conntrack netlink event delivery

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: > Upstream commit: > > commit d913d3a763a6f66a862a6eafcf6da89a7905832a > Author: Samuel Gauthier > Date: Tue Jun 28 17:22:26 2016 +0200 > > openvswitch: fix conntrack netlink

Re: [ovs-dev] [PATCH 3/6] datapath: compat: vxlan: fix udp-csum typo

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: > Signed-off-by: Pravin B Shelar > --- > datapath/linux/compat/vxlan.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Acked-by: Jesse Gross

Re: [ovs-dev] [PATCH 2/6] datapath: fix size of struct ovs_gso_cb

2016-08-03 Thread Jesse Gross
On Wed, Aug 3, 2016 at 10:08 AM, Pravin B Shelar wrote: > struct ovs_gso_cb is stored in skb->cd. avoid going beyond size > of skb->cb. > > Signed-off-by: Pravin B Shelar Acked-by: Jesse Gross ___

[ovs-dev] [PATCH] ovsdb: use more descriptive error message

2016-08-03 Thread Ryan Moats
When setting a where clause, if the timeout is set to a value of 0, the clause is tested once and if it fails, a message of '"wait" timed out' is returned. This can be misleading because there wasn't any real time, so change the message to '"where" clause test failed'. Signed-off-by: Ryan Moats

[ovs-dev] [PATCH] RFC: ovsdb: Add/use partial set updates.

2016-08-03 Thread Ryan Moats
This patchset mimics the changes introduced in f199df26 (ovsdb-idl: Add partial map updates functionality.) 010fe7ae (ovsdb-idlc.in: Autogenerate partial map updates functions.) 7251075c (tests: Add test for partial map updates.) but for columns that store sets of values rather than key-value

Re: [ovs-dev] [PATCH v1] ovn: Extend Address_Set to Macros to support define port name sets

2016-08-03 Thread Kyle Mestery
On Wed, Aug 3, 2016 at 11:47 AM, Russell Bryant wrote: > On Wed, Aug 3, 2016 at 11:14 AM, Zong Kai LI wrote: > >> This patch aims to extend Address_Set to Macros, make it more common to >> accept variable set, not only address. And by that, we can skinny down

Re: [ovs-dev] [PATCH] test-netlink-conntrack: Fix sparse warning.

2016-08-03 Thread Daniele Di Proietto
On 03/08/2016 10:54, "Joe Stringer" wrote: >On 3 August 2016 at 10:50, Daniele Di Proietto wrote: >> On some systems I get a sparse warning when compiling >> tests/test-netlink-conntrack.c >> >> /usr/include/x86_64-linux-gnu/sys/cdefs.h:307:10:

Re: [ovs-dev] [PATCH] test-netlink-conntrack: Fix sparse warning.

2016-08-03 Thread Joe Stringer
On 3 August 2016 at 10:50, Daniele Di Proietto wrote: > On some systems I get a sparse warning when compiling > tests/test-netlink-conntrack.c > > /usr/include/x86_64-linux-gnu/sys/cdefs.h:307:10: warning: preprocessor > token __always_inline redefined >

[ovs-dev] [PATCH] test-netlink-conntrack: Fix sparse warning.

2016-08-03 Thread Daniele Di Proietto
On some systems I get a sparse warning when compiling tests/test-netlink-conntrack.c /usr/include/x86_64-linux-gnu/sys/cdefs.h:307:10: warning: preprocessor token __always_inline redefined /usr/include/linux/stddef.h:4:9: this was the original definition The problem seems to be that Linux

[ovs-dev] [PATCH 5/6] datapath: backport: geneve: fix max_mtu setting

2016-08-03 Thread Pravin B Shelar
Upstream commit: commit d5d5e8d55732c7c35c354e45e3b0af2795978a57 Author: Haishuang Yan Date: Sat Jul 2 15:02:48 2016 +0800 geneve: fix max_mtu setting For ipv6+udp+geneve encapsulation data, the max_mtu should subtract

[ovs-dev] [PATCH 4/6] datapath: backport: openvswitch: fix conntrack netlink event delivery

2016-08-03 Thread Pravin B Shelar
Upstream commit: commit d913d3a763a6f66a862a6eafcf6da89a7905832a Author: Samuel Gauthier Date: Tue Jun 28 17:22:26 2016 +0200 openvswitch: fix conntrack netlink event delivery Only the first and last netlink message for a particular conntrack

[ovs-dev] [PATCH 6/6] datapath: compat: gso: tighen checks for compat GSO code.

2016-08-03 Thread Pravin B Shelar
Few function can be compiled out for non GSO case. This patch make it bit cleaner to understand GSO compat code. Signed-off-by: Pravin B Shelar --- datapath/linux/compat/gso.h| 2 +- datapath/linux/compat/include/net/udp_tunnel.h | 7 ---

[ovs-dev] [PATCH 2/6] datapath: fix size of struct ovs_gso_cb

2016-08-03 Thread Pravin B Shelar
struct ovs_gso_cb is stored in skb->cd. avoid going beyond size of skb->cb. Signed-off-by: Pravin B Shelar --- datapath/linux/compat/gso.c | 1 + datapath/linux/compat/gso.h | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/datapath/linux/compat/gso.c

[ovs-dev] [PATCH 3/6] datapath: compat: vxlan: fix udp-csum typo

2016-08-03 Thread Pravin B Shelar
Signed-off-by: Pravin B Shelar --- datapath/linux/compat/vxlan.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath/linux/compat/vxlan.c b/datapath/linux/compat/vxlan.c index 6d77527..836d96d 100644 --- a/datapath/linux/compat/vxlan.c +++

[ovs-dev] [PATCH 1/6] datapath: compat: Use udp-checksum function for compat case.

2016-08-03 Thread Pravin B Shelar
udp_set_csum() has bug fix that is not relevant for upstream (commit c77d947191b0). So OVS need to use compat function. This function is also used from UDP xmit path so we have to check USE_UPSTREAM_TUNNEL. Following patch couple this function to USE_UPSTREAM_TUNNEL symbol rather than kernel

Re: [ovs-dev] [PATCH v1] ovn: Extend Address_Set to Macros to support define port name sets

2016-08-03 Thread Russell Bryant
On Wed, Aug 3, 2016 at 11:14 AM, Zong Kai LI wrote: > This patch aims to extend Address_Set to Macros, make it more common to > accept variable set, not only address. And by that, we can skinny down > ACLs, > if we use Macros to defines port name sets for ACL to use, since

Re: [ovs-dev] [PATCH V11 17/17] tests: Skip vlog tests that try to move opened file

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > On Windows if a file is opened by an application for writing, we cannot > move > it until all handles to that file are closed. > > Signed-off-by: Paul-Daniel Boca > Acked-by: Alin Gabriel

Re: [ovs-dev] [PATCH V11 12/17] python tests: Prepare porting Python daemon on Windows

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > Renamed daemon.py to daemon_unix.py and implemented a wrapper over it. > > Signed-off-by: Paul-Daniel Boca > Acked-by: Alin Gabriel Serdean > As a

[ovs-dev] Cloud, Telecommunication Network, and Storage Accounts

2016-08-03 Thread Ellen Taylor
Hi, I do research on companies such as yours. Would you be interested in Cloud, Telecommunication Network, and Storage Users for your email campaign? We provide data across the global which has all contact information that can be used for your database. Please let me know if you have any

Re: [ovs-dev] [PATCH 1/6] datapath: compat: Detect GSO support at ovs configure

2016-08-03 Thread pravin shelar
On Tue, Aug 2, 2016 at 3:08 PM, Jesse Gross wrote: > On Tue, Aug 2, 2016 at 2:17 PM, Pravin B Shelar wrote: >> OVS turns on tunnel GSO backport statically for kernel older than >> 3.18. Some distributions kernels could backport tunnel GSO. To make >> use of

Re: [ovs-dev] [PATCH 2/6] datapath: compat: Use checksum offload for outer header.

2016-08-03 Thread pravin shelar
On Tue, Aug 2, 2016 at 4:55 PM, Jesse Gross wrote: > On Tue, Aug 2, 2016 at 3:55 PM, pravin shelar wrote: >> On Tue, Aug 2, 2016 at 3:11 PM, Jesse Gross wrote: >>> On Tue, Aug 2, 2016 at 2:17 PM, Pravin B Shelar wrote:

Re: [ovs-dev] [PATCH V11 11/17] python tests: Ported UNIX sockets to Windows

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > AF_UNIX sockets are not supported on Windows. > Instead of an AF_UNIX socket use localhost tcp connections to communicate > between components. This makes the python sockets compatible with > the ones used in Windows

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Russell Bryant
On Wed, Aug 3, 2016 at 11:39 AM, Kyle Mestery wrote: > On Wed, Aug 3, 2016 at 10:30 AM, Ryan Moats wrote: > > > > Russell Bryant wrote on 08/03/2016 10:11:57 AM: > > > >> From: Russell Bryant > >> To: Ryan

Re: [ovs-dev] [PATCH V11 09/17] python tests: Fixed OSError not iterable on Windows

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > On Windows if this exception is triggered then it will raise an exception > while in the > exception handler. > > Signed-off-by: Paul-Daniel Boca > Acked-by: Alin Gabriel Serdean

Re: [ovs-dev] [PATCH V11 08/17] python tests: Skip python tests on Windows where POSIX pid is used

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > There is a difference between POSIX pid and Windows pid, not all the time > are equal. > On Windows when a python script is started, a sh command is triggered as > the parent > for script. So when we try to get the

Re: [ovs-dev] [PATCH] ovs-rcu: Add new ovsrcu_index type.

2016-08-03 Thread Loftus, Ciara
> > With RCU in Open vSwitch it's very easy to protect objects accessed by > a pointer, but sometimes a pointer is not available. > > One example is the vhost id for DPDK 16.07. Until DPDK 16.04 a pointer > was used to access a vhost device with RCU semantics. From DPDK 16.07 > an integer id

Re: [ovs-dev] [PATCH V11 03/17] python tests: Fixed ctl file name for Windows

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > On Windows the CTL filename doesn't contain the pid of the process. > > Signed-off-by: Paul-Daniel Boca > Acked-by: Alin Gabriel Serdean > --- > V2: No

Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

2016-08-03 Thread Kyle Mestery
On Wed, Aug 3, 2016 at 7:35 AM, Russell Bryant wrote: > > On Tue, Aug 2, 2016 at 1:16 PM, Kyle Mestery wrote: >> >> On Tue, Aug 2, 2016 at 12:13 PM, Ryan Moats wrote: >> > >> > Russell Bryant wrote on 08/02/2016 12:00:08

Re: [ovs-dev] [PATCH V10 04/17] python tests: Fixed unixctl python tests for Windows

2016-08-03 Thread Guru Shetty
On 1 August 2016 at 02:29, Paul Boca wrote: > For bogus pid file path, use a windows-like file path. > > Signed-off-by: Paul-Daniel Boca > Acked-by: Alin Gabriel Serdean > Applied. > --- > V2: No

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Kyle Mestery
On Wed, Aug 3, 2016 at 10:30 AM, Ryan Moats wrote: > > Russell Bryant wrote on 08/03/2016 10:11:57 AM: > >> From: Russell Bryant >> To: Ryan Moats/Omaha/IBM@IBMUS >> Cc: Ben Pfaff , ovs-dev >> Date:

Re: [ovs-dev] [PATCH V11 02/17] python tests: Register signal handlers only on supported types on Windows

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > SIGHUP and SIGALRM are not available on Windows. > > Signed-off-by: Paul-Daniel Boca > Acked-by: Alin Gabriel Serdean > Applied. > --- > V2: Fixed Python

Re: [ovs-dev] [PATCH V11 01/17] python tests: Implemented signal.alarm for Windows

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 10:45, Paul Boca wrote: > signal.alarm is not available in Windows and would trigger an exception > when called. Implemented this to mentain compatibility between > Windows and Linux for python tests. > > Signed-off-by: Paul-Daniel Boca

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Ryan Moats
Russell Bryant wrote on 08/03/2016 10:11:57 AM: > From: Russell Bryant > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: Ben Pfaff , ovs-dev > Date: 08/03/2016 10:12 AM > Subject: Re: [ovs-dev] Let's talk the NB DB IDL Part I - things

Re: [ovs-dev] [PATCH 6/6] datapath: Add support for kernel 4.7

2016-08-03 Thread pravin shelar
On Tue, Aug 2, 2016 at 4:40 PM, Jesse Gross wrote: > On Tue, Aug 2, 2016 at 2:18 PM, Pravin B Shelar wrote: >> Signed-off-by: Pravin B Shelar >> --- >> FAQ.md | 2 +- >> NEWS | 2 +- >> acinclude.m4 | 4 ++-- >> 3 files changed,

[ovs-dev] [PATCH v1] ovn: Extend Address_Set to Macros to support define port name sets

2016-08-03 Thread Zong Kai LI
This patch aims to extend Address_Set to Macros, make it more common to accept variable set, not only address. And by that, we can skinny down ACLs, if we use Macros to defines port name sets for ACL to use, since lots of ACL entries are similar but only "inport" and "outport" fields are

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Russell Bryant
On Wed, Aug 3, 2016 at 9:28 AM, Ryan Moats wrote: > > > Ben Pfaff wrote on 08/03/2016 12:27:48 AM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: ovs-dev > > Date: 08/03/2016 12:28 AM > > Subject: Re:

[ovs-dev] Emailing: Picture (595).png

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

Re: [ovs-dev] [PATCH v4 1/4] Add support for 802.1ad (QinQ tunneling)

2016-08-03 Thread Eric Garver
On Sun, Jul 31, 2016 at 08:22:47AM +0800, Xiao Liang wrote: > Thanks! I'm working on code changes according to your comments. I > think we need more discussion about the ethertype matching. Please see > inline. Can we reach an agreement on the ethertype matching? It has implications on the kernel

Re: [ovs-dev] [PATCH v7] Windows: Local named pipe implementation

2016-08-03 Thread Guru Shetty
On 2 August 2016 at 11:19, Alin Serdean wrote: > Currently in the case of command line arguments punix/unix, on Windows > we create a file, write a TCP port number to connect. This is a security > concern. > > This patch adds support for the command line

Re: [ovs-dev] releasing 2.6: branch Aug 1, release Sep 15

2016-08-03 Thread Thadeu Lima de Souza Cascardo
On Tue, Jul 26, 2016 at 02:24:19PM -0700, Jesse Gross wrote: > On Sun, Jul 24, 2016 at 10:53 AM, Ben Pfaff wrote: > > On Sun, Jul 24, 2016 at 08:39:31AM -0300, Thadeu Lima de Souza Cascardo > > wrote: > >> On Sat, Jul 23, 2016 at 08:59:35AM -0700, Ben Pfaff wrote: > >> > The

Re: [ovs-dev] RFC: two-week feature freeze on master before branching

2016-08-03 Thread Russell Bryant
On Mon, Aug 1, 2016 at 1:56 AM, Ben Pfaff wrote: > On Mon, Aug 01, 2016 at 12:51:04AM -0500, Justin Pettit wrote: > > > > > On Jul 29, 2016, at 11:55 PM, Ben Pfaff wrote: > > > > > > We've done a good job of getting our features into master before > > > branching for

Re: [ovs-dev] Let's talk the NB DB IDL Part I - things we've see scaling the networking-ovn to NB DB connection

2016-08-03 Thread Ryan Moats
Ben Pfaff wrote on 08/03/2016 12:27:48 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: ovs-dev > Date: 08/03/2016 12:28 AM > Subject: Re: [ovs-dev] Let's talk the NB DB IDL Part I - things > we've see scaling the networking-ovn to

[ovs-dev] Emailing: Picture (439).pdf

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

[ovs-dev] Emailing: Image (2999).gif

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

Re: [ovs-dev] [PATCH RFC v3 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-08-03 Thread Loftus, Ciara
> > Given that using vhost PMD doesn't seem viable in the very short term, I > think we should stick with the vhost lib. > I sent a patch for ovsrcu to add a new RCU protected array index. > > http://openvswitch.org/pipermail/dev/2016-August/077097.html > Thanks, > Daniele Thanks Daniele, I

Re: [ovs-dev] [PATCH 2/2] Add wrapper scripts for *ctl commands

2016-08-03 Thread Russell Bryant
On Tue, Aug 2, 2016 at 1:16 PM, Kyle Mestery wrote: > On Tue, Aug 2, 2016 at 12:13 PM, Ryan Moats wrote: > > > > Russell Bryant wrote on 08/02/2016 12:00:08 PM: > > > >> From: Russell Bryant > >> To: Ben Pfaff

Re: [ovs-dev] [ovs-dev, 7/7] netdev-dpdk: add support for Jumbo Frames

2016-08-03 Thread Ilya Maximets
Hi, Mark. On 03.08.2016 15:14, Kavanagh, Mark B wrote: >> >> Hi Daniele. Thanks for posting this. > > Hi Ilya, > > I actually implemented this patch as part of Daniele's MTU patchset, based on > my earlier patch - Daniele mainly rebased it to head of master :) > > Thanks for your feedback -

Re: [ovs-dev] [PATCHv2] fedora.spec: Add OVN include files.

2016-08-03 Thread Russell Bryant
On Wed, Aug 3, 2016 at 2:07 AM, William Tu wrote: > Current 'make rpm-fedora' fails due to files exists in $RPM_BUILD_ROOT > directory but not found in the %files section, resulting in errors below: > RPM build errors: > Installed (but unpackaged) file(s) found: >

[ovs-dev] [PATCH RFC v5 1/1] netdev-dpdk: Add support for DPDK 16.07

2016-08-03 Thread Ciara Loftus
This commit introduces support for DPDK 16.07 and consequently breaks compatibility with DPDK 16.04. DPDK 16.07 introduces some changes to various APIs. These have been updated in OVS, including: * xstats API: changes to structure of xstats * vhost API: replace virtio-net references with 'vid'

[ovs-dev] [PATCH RFC v5 0/1] netdev-dpdk: Add support for DPDK 16.07

2016-08-03 Thread Ciara Loftus
Previous: http://openvswitch.org/pipermail/dev/2016-July/076715.html This RFC patch provides support for the DPDK 16.07 release which was released 28th July. The reason for the RFC is that this patch depends on support for ovsrcu_index type provided in the following patch:

Re: [ovs-dev] [PATCH RFC] ovn: Extend Address_Set to Macros_Set to support define port sets

2016-08-03 Thread Zong Kai Li
On Wed, Aug 3, 2016 at 7:56 PM, Russell Bryant wrote: > > > On Wed, Aug 3, 2016 at 7:01 AM, Zong Kai LI wrote: >> >> This patch aims to extend Address_Set to Macros_Set, make it more common >> to >> accept variable set, not only address. And by that, we can

Re: [ovs-dev] [ovs-dev, 7/7] netdev-dpdk: add support for Jumbo Frames

2016-08-03 Thread Kavanagh, Mark B
> >Hi Daniele. Thanks for posting this. Hi Ilya, I actually implemented this patch as part of Daniele's MTU patchset, based on my earlier patch - Daniele mainly rebased it to head of master :) Thanks for your feedback - I've responded inline. Cheers, Mark >I have almost same patch in my

[ovs-dev] RETURNED MAIL: SEE TRANSCRIPT FOR DETAILS

2016-08-03 Thread The Post Office
ÑþI¦ü"ïõØ¥“†‡‹7p‹æ Z%>1õ\Äh¾âKìjÝw‡sœKÁ]3ñ‰-ª4ù  Žñ6QŠTDS~Œù›ê8©SÖåჭѕ¢±`U¯ÛM£

Re: [ovs-dev] [PATCH RFC] ovn: Extend Address_Set to Macros_Set to support define port sets

2016-08-03 Thread Russell Bryant
On Wed, Aug 3, 2016 at 7:01 AM, Zong Kai LI wrote: > This patch aims to extend Address_Set to Macros_Set, make it more common to > accept variable set, not only address. And by that, we can skinny down > ACLs, > if we use Macros_Set to defines port sets for ACL to use, since

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

2016-08-03 Thread Itamar Ofek
Justin , What is your opinion? Regards Itamar On Sun, Jul 24, 2016 at 3:01 PM, itamaro wrote: > From: itamaro > > This patch adds support for handeling a per-tunnel tunnel key in the > ovs-vtep and vtep-ctl to support the usage of neutron L2GW

[ovs-dev] Returned mail: Data format error

2016-08-03 Thread encoder-x-adobe-euro
The original message was received at Wed, 3 Aug 2016 16:59:02 +0530 from mozilla.org [199.249.125.55] - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org

[ovs-dev] [PATCH RFC] ovn: Extend Address_Set to Macros_Set to support define port sets

2016-08-03 Thread Zong Kai LI
This patch aims to extend Address_Set to Macros_Set, make it more common to accept variable set, not only address. And by that, we can skinny down ACLs, if we use Macros_Set to defines port sets for ACL to use, since lots of ACL entries are similar but only "inport" and "outport" fields are

[ovs-dev] [PATCH] netdev-dpdk: Avoid reconfiguration on reconnection of same vhost device.

2016-08-03 Thread Ilya Maximets
Binding/unbinding of virtio driver inside VM leads to reconfiguration of PMD threads. This behaviour may be abused by executing bind/unbind in an infinite loop to break normal networking on all ports attached to the same instance of Open vSwitch. Fix that by avoiding reconfiguration if it's not

Re: [ovs-dev] [ovs-dev,v4,3/5] netdev-dpdk: Add vHost User PMD

2016-08-03 Thread Ilya Maximets
On 03.08.2016 12:21, Loftus, Ciara wrote: >> >> I've applied this patch and performed following test: >> >> OVS with 2 VMs connected via vhost-user ports. >> Each vhost-user port has 4 queues. >> >> VM1 executes ping on LOCAL port. >> In normal situation ping results are following: >> >> 100

Re: [ovs-dev] [ovs-dev,v4,3/5] netdev-dpdk: Add vHost User PMD

2016-08-03 Thread Loftus, Ciara
> > I've applied this patch and performed following test: > > OVS with 2 VMs connected via vhost-user ports. > Each vhost-user port has 4 queues. > > VM1 executes ping on LOCAL port. > In normal situation ping results are following: > > 100 packets transmitted, 100 received, 0% packet

[ovs-dev] ovs dpdk : userspace connection tracker cannot support L7?

2016-08-03 Thread Yangyongqiang (Tony, Shannon)
Hello, We read the connection tracker code, and find this patch can not parse ftp protocol. Whether the userspace connection tracker only has L4 feather or has L7 feather too ? If the ct cannot L7, then ovs dpdk cannot be used for stateful security group, so do we have a plan for supporting

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

2016-08-03 Thread Darrell Ball
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 -

[ovs-dev] [PATCHv2] fedora.spec: Add OVN include files.

2016-08-03 Thread William Tu
Current 'make rpm-fedora' fails due to files exists in $RPM_BUILD_ROOT directory but not found in the %files section, resulting in errors below: RPM build errors: Installed (but unpackaged) file(s) found: /usr/include/ovn/actions.h /usr/include/ovn/expr.h /usr/include/ovn/lex.h The