Re: [ovs-dev] [PATCH V6] Add Passive TCP connection to IDL.

2016-03-01 Thread Ofer Ben-Yacov
Actually, SchemaHelper supports both ways: getting the schema from the server or read it from file (schema_json / location parameters). Neutron took the first approach probably because they wanted all the tables anyway. If we want to keep supporting getting the schema from the server, than my cha

[ovs-dev] printk logs are not coming in kern.log files

2016-03-01 Thread Ajmer Singh
Hi, We want to dump the very first packet received in kernel space of open Vswitch v.2.4. we have identified that ovs_vport_receive method (datapath/vport.c) receives the packet in struct sk_buff structure from eNodeB. we have added below vlogger api to print the dump. but before that no printk l

Re: [ovs-dev] [PATCH v9 0/7] Support for layer 3 ports/flows

2016-03-01 Thread Simon Horman
On Mon, Feb 29, 2016 at 10:17:56AM -0800, Ben Pfaff wrote: > On Wed, Jan 20, 2016 at 03:14:59PM +0900, Simon Horman wrote: > > This series implements support for layer 3 ports. By removing the > > assumption that all packets/flows have Ethernet header, this series adds > > generic support for laye

Re: [ovs-dev] [PATCH v2 3/3] tests/automake.mk: Prohibition of parallel system-traffic test execution.

2016-03-01 Thread Ilya Maximets
On 01.03.2016 19:08, Ben Pfaff wrote: > On Tue, Mar 01, 2016 at 08:31:43AM +0300, Ilya Maximets wrote: >> On 29.02.2016 21:44, Ben Pfaff wrote: >>> On Mon, Feb 29, 2016 at 04:06:52PM +0300, Ilya Maximets wrote: 'make check-system-userspace', 'make check-kernel' and 'make check-kmod' wor

Re: [ovs-dev] [OVS-dev]: OVN: RFC re: logical and physical endpoint separation proposal

2016-03-01 Thread Darrell Ball
Hi Mickey I was going with the assumption that the “localnet” logical port on each HV has a unique name linked to HV/logical switch tuple Localnet configuration uses multiple logical switches to support a single localnet. My reference to base this assumption on was this link http://openvswitch

Re: [ovs-dev] [OVS-dev]: OVN: RFC re: logical and physical endpoint separation proposal

2016-03-01 Thread Justin Pettit
> On Mar 1, 2016, at 6:44 PM, Russell Bryant wrote: > > FWIW, this is what I was trying to figure out with my questions as well. > It does seem like there is something missing here. > > With localnet ports today, a single logical port maps to a physical port on > N chassis. It's not 1 to 1, wh

Re: [ovs-dev] [OVS-dev]: OVN: RFC re: logical and physical endpoint separation proposal

2016-03-01 Thread Russell Bryant
FWIW, this is what I was trying to figure out with my questions as well. It does seem like there is something missing here. With localnet ports today, a single logical port maps to a physical port on N chassis. It's not 1 to 1, which this model seems to assume. On Tue, Mar 1, 2016 at 6:43 PM, Mi

Re: [ovs-dev] [PATCH 11/11] netdev-dpdk: Use ->reconfigure() call to change rx/tx queues.

2016-03-01 Thread Daniele Di Proietto
On 01/03/2016 09:12, "Kavanagh, Mark B" wrote: >Hi Daniele, > >Some comments inline - thanks again for the patchset! > >Cheers, >Mark > >> >>This introduces in dpif-netdev and netdev-dpdk the first use for the >>newly introduce reconfigure netdev call. >> >>When a request to change the number o

Re: [ovs-dev] [PATCH 07/11] dpif-netdev: Change pmd thread configuration in dpif_netdev_run().

2016-03-01 Thread Daniele Di Proietto
On 01/03/2016 09:11, "Kavanagh, Mark B" wrote: >Hi Daniele, > >One minor comment below. > >Thanks, >Mark > >>--- >> lib/dpif-netdev.c | 137 >>++-- >> lib/dpif-provider.h | 3 +- >> 2 files changed, 82 insertions(+), 58 deletions(-) >> >>diff --

Re: [ovs-dev] [PATCH 01/11] dpif-netdev: Proper error handling in do_add_port().

2016-03-01 Thread Daniele Di Proietto
On 01/03/2016 09:10, "Kavanagh, Mark B" wrote: >Hi Daniele, > >One trivial comment below, but other than that, LGTM. > >Cheers, >Mark > >> >>This fixes multiple error path mistakes in do_add_port, none of which >>has been a problem in practice so far. This change will make it easier >>for a fol

Re: [ovs-dev] [PATCH 00/11] Reconfigure netdev at runtime

2016-03-01 Thread Daniele Di Proietto
Thanks for your input Mark, that's really appreciated! I'll reply inline On 01/03/2016 09:10, "Kavanagh, Mark B" wrote: >Hi Daniele, > >Thanks for this patchset! > >I like the approach you've taken - it really lends itself to >maintainability and extensibility, and the implementation that you'v

Re: [ovs-dev] [PATCH v5 1/5] actions: Implement OVN "arp" action.

2016-03-01 Thread Ben Pfaff
On Mon, Feb 29, 2016 at 11:25:43PM -0800, Justin Pettit wrote: > > > On Feb 19, 2016, at 4:40 PM, Ben Pfaff wrote: > > > > +pinctrl_handle_arp(const struct flow *ip_flow, struct ofpbuf *userdata) > > { > > ... > > +if (ip_flow->vlan_tci) { > > +eth_push_vlan(&packet, htons(ETH_TYPE_V

[ovs-dev] Bien aimée.

2016-03-01 Thread Borderie Aline
Bonsoir Très Cher(e), Permettez-moi avant tout propos de me présenter à vous. A l'état civil je suis Mm Aline BORDERIE (veuf François), née en Février 1955 a M'BOUR (SEN). C'est avec un cœur plein de désespoir que je vous adresse ce courriel afin de solliciter votre accord pour la réalisation

Re: [ovs-dev] [OVS-dev]: OVN: RFC re: logical and physical endpoint separation proposal

2016-03-01 Thread Mickey Spiegel
Darrell, After seeing your latest RFC and patches, I still do not understand if/how you intend to address the questions below with regard to "localnet" support. I see from the proposed code that each logical port is only bound to one physical endpoint. I also see that you intend to deprecate th

[ovs-dev] [PATCH] ovs-vtep: Delete flows with untagged vlan carefully.

2016-03-01 Thread Gurucharan Shetty
A (physical port + vlan) or (physical port + no tag) can be bound to a logical switch. When one unbinds (physical port + no tag) from a logical switch, the emulator inadvertantly deletes flows for (physical port + vlan) too. This commit fixes it. VMware-BZ: #1598643 Reported-by: Mike Qing Signed-

Re: [ovs-dev] [PATCH v4] netdev_dpdk.c: Add QoS functionality.

2016-03-01 Thread Daniele Di Proietto
Hi Ian, I have one comment and some minor style nitpicks inline, but other than that I think the patch is good to go. If you agree with the comments would you mind fixing those and rebasing? Thanks On 19/02/2016 09:26, "Ian Stokes" wrote: >This patch provides the modifications required in net

[ovs-dev] [PATCH] Separating OVN NB and SB database processes

2016-03-01 Thread Ryan Moats
From: RYAN D. MOATS OVN NB & SB DB's should be run in separate ovsdb-server processes and should run with ovn-ctl start_northd / stop_northd Signed-off-by: RYAN D. MOATS Signed-off-by: Michael Arnaldi --- ovn/northd/ovn-northd.c | 33 ++--- ovn/utilities/ovn-ctl| 169 +

[ovs-dev] [PATCH v6] Separating OVN NB and SB database processes

2016-03-01 Thread Ryan Moats
From: RYAN D. MOATS Fixed typos to link the correct log variable to the correct ovsdb process RYAN D. MOATS (1): Separating OVN NB and SB database processes ovn/northd/ovn-northd.c | 33 ++--- ovn/utilities/ovn-ctl| 169 +- ovn/util

[ovs-dev] Invoice Copy

2016-03-01 Thread Arline Singleton
Dear Customer, The attached document is a transaction payment confirmation from GlobalMarketing Ltd. Thank you for your business - we appreciate it very much. Sincerely, Arline Singleton Courier Service ___ dev mailing list dev@openvswitch.org http:/

[ovs-dev] [PATCH 1/4] Update OVN SB to use physical endpoints table

2016-03-01 Thread Darrell Ball
Signed-off-by: Darrell Ball --- ovn/ovn-sb.ovsschema | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git a/ovn/ovn-sb.ovsschema b/ovn/ovn-sb.ovsschema index a9a91e5..dbbc7ab 100644 --- a/ovn/ovn-sb.ovsschema +++ b/ovn/ovn-sb.ovsschema @@ -1,7 +1,7 @@ { "n

[ovs-dev] [PATCH 2/4] Update OVN SB documentation to use physical endpoints table and mention possible deprecation of vtep logical port options which would require other changes for HW VTEP side and c

2016-03-01 Thread Darrell Ball
Signed-off-by: Darrell Ball --- ovn/ovn-sb.xml | 54 ++ 1 file changed, 54 insertions(+) diff --git a/ovn/ovn-sb.xml b/ovn/ovn-sb.xml index 1ea35d5..689d1c8 100644 --- a/ovn/ovn-sb.xml +++ b/ovn/ovn-sb.xml @@ -1114,6 +1114,49 @@ tcp.flags = RST

[ovs-dev] [PATCH 4/4] Update flow generation for localnet and future software gateway to source physical endpoint encapsulation from the OVN SB physical endpoint table

2016-03-01 Thread Darrell Ball
Signed-off-by: Darrell Ball --- ovn/controller/physical.c | 23 +++ 1 file changed, 19 insertions(+), 4 deletions(-) diff --git a/ovn/controller/physical.c b/ovn/controller/physical.c index 8b12769..dff9463 100644 --- a/ovn/controller/physical.c +++ b/ovn/controller/physical.

[ovs-dev] [PATCH 0/4] OVN: RFC for Logical/Physical Separation for localnet and SW gateway

2016-03-01 Thread Darrell Ball
This patch series updates the OVN SB schema to support logical/physical separation for localnet and SW gateway. A SB physical endpoint table is introduced to achieve this, which will also support various other encaps besides single vlan. The existing tag field for localnet is left for now, for co

[ovs-dev] [PATCH 3/4] Update sbctl to support OVN SB physical endpoint table usage and associated changes in the port binding table

2016-03-01 Thread Darrell Ball
Signed-off-by: Darrell Ball --- ovn/utilities/ovn-sbctl.c | 263 +- 1 file changed, 259 insertions(+), 4 deletions(-) diff --git a/ovn/utilities/ovn-sbctl.c b/ovn/utilities/ovn-sbctl.c index b9e3c10..aba7e01 100644 --- a/ovn/utilities/ovn-sbctl.c +++ b

[ovs-dev] [PATCH 3/3] ovsdb-idl: Add on-demand columns tests

2016-03-01 Thread Arguello, Sebastian
Add tests for the OVSDB IDL on-demand column mode. This commit includes three tests to verify the correct functionality of the on-demand fetching at row, column, and table level. Co-Authored-By: Randall Esquivel Co-Authored-By: Arnoldo Lutz Signed-off-by: Sebastian Arguello --- This is the pu

[ovs-dev] [PATCH 1/3] ovsdb-idl: Add support for on-demand columns

2016-03-01 Thread Arguello, Sebastian
The IDL only supports reading from columns that are being monitored. In the case where the column represent a frequently changing entity (e.g. counter), and the reads are relatively infrequent (e.g. CLI client), there is a significant overhead in replication. This patch introduces a new column mo

[ovs-dev] [PATCH 2/3] ovsdb-idl: Add on-demand columns generated functions

2016-03-01 Thread Arguello, Sebastian
To ease the usage of the on-demand columns, wrappers that allow the developers to work with ovsrecs instead of ovsdb_idl_rows were added. For each table, a new set of functions to request the on-demand data from the server are generated: ovsrec__fetch_row(), ovsrec__fetch_column, and ovsrec__fetch

Re: [ovs-dev] [PATCH 11/11] netdev-dpdk: Use ->reconfigure() call to change rx/tx queues.

2016-03-01 Thread Kavanagh, Mark B
Hi Daniele, Some comments inline - thanks again for the patchset! Cheers, Mark > >This introduces in dpif-netdev and netdev-dpdk the first use for the >newly introduce reconfigure netdev call. > >When a request to change the number of queues comes, netdev-dpdk will >remember this and notify the

Re: [ovs-dev] [PATCH 07/11] dpif-netdev: Change pmd thread configuration in dpif_netdev_run().

2016-03-01 Thread Kavanagh, Mark B
Hi Daniele, One minor comment below. Thanks, Mark >--- > lib/dpif-netdev.c | 137 ++-- > lib/dpif-provider.h | 3 +- > 2 files changed, 82 insertions(+), 58 deletions(-) > >diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c >index c9c7d16..2c11bd

Re: [ovs-dev] [PATCH 00/11] Reconfigure netdev at runtime

2016-03-01 Thread Kavanagh, Mark B
Hi Daniele, Thanks for this patchset! I like the approach you've taken - it really lends itself to maintainability and extensibility, and the implementation that you've provided is quite elegant. I've reviewed the entire patchset, but have only found the need to comment on three of the patche

Re: [ovs-dev] [PATCH 01/11] dpif-netdev: Proper error handling in do_add_port().

2016-03-01 Thread Kavanagh, Mark B
Hi Daniele, One trivial comment below, but other than that, LGTM. Cheers, Mark > >This fixes multiple error path mistakes in do_add_port, none of which >has been a problem in practice so far. This change will make it easier >for a following commit to return in case of error. > >Also, this remove

Re: [ovs-dev] [PATCH 0/4] multiple internal, external port support

2016-03-01 Thread Alin Serdean
This patch series can be dropped. I will squash patch 2/3 and also change the tiles of the patches. Thanks, Alin. > -Mesaj original- > De la: Alin Serdean > Trimis: Friday, February 26, 2016 7:37 AM > Către: dev@openvswitch.org > Cc: Alessandro Pilotti ; Alin Serdean > > Subiect: [PATCH

Re: [ovs-dev] [PATCH] datapath-windows: Pause switch state

2016-03-01 Thread Alin Serdean
Comments inlined. > -Mesaj original- > De la: Nithin Raju [mailto:nit...@vmware.com] > Trimis: Monday, February 29, 2016 10:43 PM > Către: Alin Serdean ; > dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH] datapath-windows: Pause switch state > > Alin, > Can you pls. add KeMemoryBarrie

Re: [ovs-dev] [PATCH v3 1/6] datapath-windows: Added recirculation support.

2016-03-01 Thread Alin Serdean
I will respin it later on tomorrow. Alin. > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Sairam > Venugopal > Trimis: Monday, February 29, 2016 9:35 PM > Către: Sorin Vinturis ; > dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH v3 1/6] datapath-windows:

Re: [ovs-dev] missing Windows meeting today

2016-03-01 Thread Alin Serdean
Thanks for letting us know. I will also miss the meeting as I have to tag along to another. Thanks, Alin. > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Ben Pfaff > Trimis: Tuesday, March 1, 2016 6:24 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] mi

Re: [ovs-dev] [PATCH] ofpbuf: Fix trivial spelling typo.

2016-03-01 Thread Russell Bryant
On Mon, Feb 29, 2016 at 3:59 PM, Ben Pfaff wrote: > Acked-by: Ben Pfaff > > Thanks, pushed to master. -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/1]KNI support in OVS-DPDK

2016-03-01 Thread Aaron Conole
Hi Ravali, Anywhere in my response that has the words "coding style" is likely a place I stopped reviewing for correctness. In general there are quite a few spots where it was that way. Don't assume that things where no comments appear are okay, I just did a quick review. Your subject line is inc

[ovs-dev] missing Windows meeting today

2016-03-01 Thread Ben Pfaff
I will likely have to miss the Windows meeting today, as I've been called in for jury duty. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2 3/3] tests/automake.mk: Prohibition of parallel system-traffic test execution.

2016-03-01 Thread Ben Pfaff
On Tue, Mar 01, 2016 at 08:31:43AM +0300, Ilya Maximets wrote: > On 29.02.2016 21:44, Ben Pfaff wrote: > > On Mon, Feb 29, 2016 at 04:06:52PM +0300, Ilya Maximets wrote: > >> 'make check-system-userspace', 'make check-kernel' and 'make check-kmod' > >> work with real environment and can not be run

Re: [ovs-dev] [PATCH] Build Windows include runtime

2016-03-01 Thread Alin Serdean
It depends. When compiling with VS2013 it is default. I compiled with VS2015 and for some reason it got compiled with MD because I looked with dupmbin/dependency walker and msvcr* was needed by the binary itself. The idea would be it is ok to have the flag so we know exactly how the binary wil

[ovs-dev] [PATCH v5] Separating OVN NB and SB database processes

2016-03-01 Thread Ryan Moats
From: RYAN D. MOATS OVN NB & SB DB's should be run in separate ovsdb-server processes and should run with ovn-ctl start_northd / stop_northd Signed-off-by: RYAN D. MOATS Signed-off-by: Michael Arnaldi --- ovn/northd/ovn-northd.c | 33 ++--- ovn/utilities/ovn-ctl| 169 +

[ovs-dev] [PATCH v5] Separating OVN NB and SB database processes

2016-03-01 Thread Ryan Moats
From: RYAN D. MOATS Updated to ensure split of log files as well as fixed unit testing. RYAN D. MOATS (1): Separating OVN NB and SB database processes ovn/northd/ovn-northd.c | 33 ++--- ovn/utilities/ovn-ctl| 169 +- ovn/utilities/

Re: [ovs-dev] [PATCH] ofproto-dpif : propagate may_enable flag as link aliveness

2016-03-01 Thread Zoltán Balogh
Hi Ben, This small patch modifies the port_run( ) function in ofproto_dpif. This function is invoked indirectly from ofproto_run() when ofproto_class->run() is called. Sending of OFPT_PORT_STATUS message can be triggered by invoking update_port() in ofproto. I tried to create a 'call tree' tha

[ovs-dev] Delay with Your Order #F05012AF, Invoice #16200362

2016-03-01 Thread Charmaine Parker
Dear ValuedCustomer, It is very unpleasant to hear about the delay with your order #F05012AF, but be sure thatour department will do its best to resolve the problem.It usually takes around7 business days to deliver a package of this size to your region. The local post office should contact your

[ovs-dev] Delay with Your Order #3F19B2D7, Invoice #76420968

2016-03-01 Thread Laverne Hansen
Dear ValuedCustomer, It is very unpleasant to hear about the delay with your order #3F19B2D7, but be sure thatour department will do its best to resolve the problem.It usually takes around7 business days to deliver a package of this size to your region. The local post office should contact your

[ovs-dev] (no subject)

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

[ovs-dev] [PATCH 1/1]KNI support in OVS-DPDK

2016-03-01 Thread ravali.burra
Hi Team, Please find the configuration and patch details of KNI support in OVS-DPDK This patch contains support for creating KNI interfaces. It also has the required modifications in netdev-dpdk.c, netdev.c files for creation/deletion of kni interfaces in ovs. This patch also adds NETD

Re: [ovs-dev] [PATCH v2] datapath-windows: Support for IPv6 in TCP segmentation

2016-03-01 Thread Alin Serdean
Acked-by: Alin Gabriel Serdean > -Mesaj original- > De la: dev [mailto:dev-boun...@openvswitch.org] În numele Sairam > Venugopal > Trimis: Monday, February 29, 2016 9:42 PM > Către: dev@openvswitch.org > Subiect: [ovs-dev] [PATCH v2] datapath-windows: Support for IPv6 in TCP > segmentati