Re: [ovs-dev] OVN implementation of the NAT/FIP architecture

2016-02-12 Thread Michael
Hi, I would also like to express my concerns. I agree that FIP and in general NAT traversal on a single node are bottlenecks and single points of failure. In general even supporting HA using VRRP save from the single point of failure but doesn’t really solve the bottleneck issue. We should also

[ovs-dev] LAST CALL: Income TAX Return Preparations - Feb. 24

2016-02-12 Thread Jhie Tejada
I. USE OF

Re: [ovs-dev] [PATCH RFC] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-12 Thread Flavio Leitner
On Fri, 12 Feb 2016 09:00:48 +0300 Ilya Maximets wrote: > Hi, Flavio. > > Comment inlined. > > On 12.02.2016 07:44, Flavio Leitner wrote: > > > > Hi Ilya, > > > > On Thu, 11 Feb 2016 16:04:12 +0300 > > Ilya Maximets wrote: > > > >> Currently virtio driver in guest operating system have to

[ovs-dev] Bug#813116: /usr/sbin/ovs-vswitchd: segfault when push_mpls flow action is applied

2016-02-12 Thread Jan Moskyto Matejka
>* What led up to the situation? Sending any MPLS packet to any bridge from TAP device. Other devices not tested. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH V3] netdev-dpdk: Add Jumbo Frame Support.

2016-02-12 Thread Flavio Leitner
On Thu, 11 Feb 2016 02:55:43 + Daniele Di Proietto wrote: > On 08/02/2016 03:23, "Kavanagh, Mark B" wrote: > > >> > >>Hi Mark, > >> > > > >Hi Daniele, > > > >Thanks for the review! Responses inline below. > > > >Cheers, > >Mark > > > >>This patch besides adding Jumbo Frame support also

Re: [ovs-dev] [PATCH V3] netdev-dpdk: Add Jumbo Frame Support.

2016-02-12 Thread Flavio Leitner
On Mon, 8 Feb 2016 10:54:08 + "Kavanagh, Mark B" wrote: > > > >Hi, > > > >Some comments below. > > > > Hi Flavio, > > Thanks for your review! > > I've responded to your comments inline - I've also addressed the > cosmetic changes that you suggested in V3 of the patch. > > Thanks again,

Re: [ovs-dev] [PATCH RFC v2] netdev-dpdk: vhost-user: Fix sending packets to queues not enabled by guest.

2016-02-12 Thread Flavio Leitner
On Fri, 12 Feb 2016 09:21:12 +0300 Ilya Maximets wrote: > Currently virtio driver in guest operating system have to be configured > to use exactly same number of queues. If number of queues will be less, > some packets will get stuck in queues unused by guest and will not be > received. > > Fix

Re: [ovs-dev] [PATCH V3] netdev-dpdk: Add Jumbo Frame Support.

2016-02-12 Thread Kavanagh, Mark B
>> > >> >Hi, >> > >> >Some comments below. >> > >> >> Hi Flavio, >> >> Thanks for your review! >> >> I've responded to your comments inline - I've also addressed the >> cosmetic changes that you suggested in V3 of the patch. >> >> Thanks again, >> Mark >> >> > >> >On Mon, 1 Feb 2016 10:18:54 +

Re: [ovs-dev] [PATCH] build windows: Update OpenSSL download link

2016-02-12 Thread Guru Shetty
On 10 February 2016 at 17:12, Alin Serdean wrote: > This patch updates the appveyor dowload link for OpenSSL prebuilt binaries. > > Signed-off-by: Alin Gabriel Serdean > Applied, thanks! > --- > appveyor.yml | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/appve

Re: [ovs-dev] [PATCH V3] netdev-dpdk: Add Jumbo Frame Support.

2016-02-12 Thread Flavio Leitner
On Fri, 12 Feb 2016 14:36:14 + "Kavanagh, Mark B" wrote: > >> > > >> >Hi, > >> > > >> >Some comments below. > >> > > >> > >> Hi Flavio, > >> > >> Thanks for your review! > >> > >> I've responded to your comments inline - I've also addressed the > >> cosmetic changes that you suggested in V3 o

Re: [ovs-dev] [PATCH V3] netdev-dpdk: Add Jumbo Frame Support.

2016-02-12 Thread Kavanagh, Mark B
> >> >> > >> >> >Hi, >> >> > >> >> >Some comments below. >> >> > >> >> >> >> Hi Flavio, >> >> >> >> Thanks for your review! >> >> >> >> I've responded to your comments inline - I've also addressed the >> >> cosmetic changes that you suggested in V3 of the patch. >> >> >> >> Thanks again, >> >> Mark

Re: [ovs-dev] [PATCH v4] ovn: Connect to remote lports through localnet port.

2016-02-12 Thread Russell Bryant
On 02/12/2016 12:36 AM, Han Zhou wrote: > Before this patch, inter-chassis communication between VIFs of same > lswitch will always go through tunnel, which end up of modeling a > single physical network with many lswitches and pairs of lports, and > complexity in CMS like OpenStack neutron to mana

Re: [ovs-dev] [PATCH v2] tunneling: Disable IPv6 tunnel

2016-02-12 Thread Thadeu Lima de Souza Cascardo
On Thu, Feb 11, 2016 at 01:05:16AM -0800, Pravin B Shelar wrote: > There are multiple issues in IPv6 userspace tunnel > implementation. Even the kernel module that ships with > 2.5 does not support IPv6 tunneling. There is not > enough time to get all fixes in branch-2.5. So it make > sense to disa

Re: [ovs-dev] [PATCH v3] ovn-controller: Assign zone-id consistently

2016-02-12 Thread Russell Bryant
On 02/11/2016 08:18 PM, Ramu Ramamurthy wrote: > Currently, ovn-controller does not record the > lport->zoneid map, and so, after ovn-controller restart, > zone-ids may get set inconsistently on lports, resulting > in possible hits to already established connections. > > Set zone-id as an external

Re: [ovs-dev] [PATCH v3] ovn-controller: Assign zone-id consistently

2016-02-12 Thread Russell Bryant
On 02/12/2016 12:31 PM, Russell Bryant wrote: > On 02/11/2016 08:18 PM, Ramu Ramamurthy wrote: >> Currently, ovn-controller does not record the >> lport->zoneid map, and so, after ovn-controller restart, >> zone-ids may get set inconsistently on lports, resulting >> in possible hits to already esta

Re: [ovs-dev] [PATCH v3] ovn-controller: Assign zone-id consistently

2016-02-12 Thread Suryanarayan Ramamurthy
thanks !, I will make these updates in the next version From: Justin Pettit To: Ramu Ramamurthy Cc: dev@openvswitch.org, Badri Ramaswamy/Santa Clara/IBM@IBMUS Date: 02/11/2016 11:22 PM Subject:Re: [ovs-dev] [PATCH v3] ovn-controller: Assign zone-id consistently Sent by:

[ovs-dev] [PATCH] lib: Remove unused function prototypes

2016-02-12 Thread Andy Zhou
Found by inspection. Signed-off-by: Andy Zhou --- lib/stream.h | 4 1 file changed, 4 deletions(-) diff --git a/lib/stream.h b/lib/stream.h index 0db50da..f8e1891 100644 --- a/lib/stream.h +++ b/lib/stream.h @@ -37,10 +37,6 @@ int stream_open(const char *name, struct stream **, uint8_t ds

Re: [ovs-dev] [ovs-dev,V5] Add Passive TCP connection to IDL.

2016-02-12 Thread Russell Bryant
On 02/09/2016 04:25 AM, Ofer Ben-Yacov wrote: > From: Ofer Ben-Yacov > > Currently the IDL does not support passive TCP connection, > i.e. when the OVSDB connects to its manager. > > This patch enables IDL to use an already-open session > (the one which was previously used for retrieving the db

Re: [ovs-dev] [PATCH] lib: Remove unused function prototypes

2016-02-12 Thread Russell Bryant
On 02/12/2016 02:04 PM, Andy Zhou wrote: > Found by inspection. > > Signed-off-by: Andy Zhou > --- > lib/stream.h | 4 > 1 file changed, 4 deletions(-) > > diff --git a/lib/stream.h b/lib/stream.h > index 0db50da..f8e1891 100644 > --- a/lib/stream.h > +++ b/lib/stream.h > @@ -37,10 +37,6 @

[ovs-dev] [PATCH/RFC] openvswitch: loosen restriction of output of MPLS to tunnel vports

2016-02-12 Thread Simon Horman
If an skb was not MPLS initially then it may be GSO and in that case if it became MPLS then GSO can't be performed because both MPLS and tunnels make use of the inner_protocol field of struct skbuff in order to allow GSO to be performed in the inner packet. On the other hand if an skb was MPLS ini

Re: [ovs-dev] [PATCH V3] Separating ovn nb&sb databases

2016-02-12 Thread Russell Bryant
On 02/10/2016 09:56 AM, Michael wrote: > Added a check on pid in start_northd > Added ssl params option cloning the conf db of normal openvswitch db (in this > way ssl should follow exactly the same behaviour as in normal ovsdb) > Added configuration options for logging Please line wrap the commi

Re: [ovs-dev] [PATCH] [ovn-controller] Add logical flow incremental processing

2016-02-12 Thread Russell Bryant
On 02/10/2016 05:14 PM, Ryan Moats wrote: > From: RYAN D. MOATS > > Add incremental processing of logical flows in ovn-controller by > tracking changes to the match column of the Logical_Flow > OVN SB table. Code is included to properly handle the order of > checked sequence numbers and rechecki

Re: [ovs-dev] [PATCH] lib: Remove unused function prototypes

2016-02-12 Thread Andy Zhou
On Fri, Feb 12, 2016 at 11:05 AM, Russell Bryant wrote: > On 02/12/2016 02:04 PM, Andy Zhou wrote: > > Found by inspection. > > > > Signed-off-by: Andy Zhou > > --- > > lib/stream.h | 4 > > 1 file changed, 4 deletions(-) > > > > diff --git a/lib/stream.h b/lib/stream.h > > index 0db50da..

[ovs-dev] [PATCH 2/3] Enable some existing OVN BUM tests

2016-02-12 Thread Darrell Ball
Signed-off-by: Darrell Ball --- tests/ovn.at | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/ovn.at b/tests/ovn.at index e00674d..a60ab56 100644 --- a/tests/ovn.at +++ b/tests/ovn.at @@ -870,10 +870,8 @@ for s in 1 2 3; do done # Broadcast and multicast

[ovs-dev] [PATCH 3/3] Handle tunnel set changes for BUM traffic in the vtep emulator

2016-02-12 Thread Darrell Ball
Signed-off-by: Darrell Ball --- vtep/ovs-vtep | 20 ++-- 1 file changed, 18 insertions(+), 2 deletions(-) diff --git a/vtep/ovs-vtep b/vtep/ovs-vtep index 46a5692..54bf0db 100755 --- a/vtep/ovs-vtep +++ b/vtep/ovs-vtep @@ -135,15 +135,31 @@ class Logical_Switch(object):

[ovs-dev] [PATCH 0/3] VTEP BUM Support Changes

2016-02-12 Thread Darrell Ball
This patch series 1) adds VTEP schema mcast macs remote table update support 2) enables some existing BUM tests for OVN 3) adds tunnel set changes for BUM handling in the VTEP emulator Darrell Ball (3): VTEP Schema - Support mcast macs remote table updates Enable some existing OVN BUM tes

[ovs-dev] [PATCH 1/3] VTEP Schema - Support mcast macs remote table updates

2016-02-12 Thread Darrell Ball
Signed-off-by: Darrell Ball --- AUTHORS| 1 + ovn/controller-vtep/vtep.c | 207 - 2 files changed, 186 insertions(+), 22 deletions(-) diff --git a/AUTHORS b/AUTHORS index 5c3643a..b709482 100644 --- a/AUTHORS +++ b/AUTHORS @@ -417