Re: [ovs-dev] [PATCH 0/1] Fix Rhel spec file for kmod RPM

2016-01-28 Thread Thomas F Herbert
On 1/28/16 9:54 AM, Russell Bryant wrote: On 01/28/2016 08:48 AM, Thomas F Herbert wrote: In the process of building RPM's in OVS-OPNFV project, it was discovered that the Rhel spec file for building the kernel module RPM was broken in such a way to break depmod on kernel module installs.

Re: [ovs-dev] [PATCH 1/1] Openvswitch: RHEL: Fix RHEL Spec file for kmod RPM install.

2016-01-28 Thread Guru Shetty
On 28 January 2016 at 05:48, Thomas F Herbert wrote: > Signed-off-by: Thomas F Herbert > The changes that this commit reverts were initially added as part of commit 9c12d2ec7569a12d9. So this commit will break the original fix. > --- >

Re: [ovs-dev] [PATCH 1/3] ovn: Add port_security proposal

2016-01-28 Thread Russell Bryant
On 01/28/2016 10:11 AM, Numan Siddique wrote: > On 01/28/2016 08:04 PM, Russell Bryant wrote: >> You raised another thread about the proposed syntax here: >> >> http://openvswitch.org/pipermail/dev/2016-January/064921.html >> >> Let's make sure we agree on that before proceeding. In that thread,

Re: [ovs-dev] [PATCH 1/1] Openvswitch: RHEL: Fix RHEL Spec file for kmod RPM install.

2016-01-28 Thread Guru Shetty
It should ideally work. If not, it is a bug. Just to make sure that you are doing the right thing, have you read the following piece of instruction in INSTALL.RHEL.md carefully? """ On RHEL 6, to build the Open vSwitch kernel module, copy rhel/openvswitch-kmod.files into the RPM sources

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-28 Thread Ansis Atteka
On 26 January 2016 at 11:07, Christian Ehrhardt < christian.ehrha...@canonical.com> wrote: > Hi Ansis, > the links I referred were older and yes some had apparmor/selinux issues. > I didn't refer so much to the issues they had, but more to the fact that > they usually ended up running vswitchd as

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-28 Thread Christian Ehrhardt
On Tue, Jan 26, 2016 at 8:06 PM, Aaron Conole wrote: > I should be on the discuss mailing list. Let me just state a big _YES_ I > am working on this problem from multiple facets. > > Ansis Atteka writes: > [...] > > The link you posted seems to mention

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-28 Thread Christian Ehrhardt
On Wed, Jan 27, 2016 at 9:29 AM, Ansis Atteka wrote: > > > On 26 January 2016 at 11:07, Christian Ehrhardt < > christian.ehrha...@canonical.com> wrote: > >> Hi Ansis, >> the links I referred were older and yes some had apparmor/selinux issues. >> I didn't refer so much to

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-28 Thread Ansis Atteka
On 27 January 2016 at 02:30, Christian Ehrhardt < christian.ehrha...@canonical.com> wrote: > > On Wed, Jan 27, 2016 at 9:29 AM, Ansis Atteka > wrote: > >> >> >> On 26 January 2016 at 11:07, Christian Ehrhardt < >> christian.ehrha...@canonical.com> wrote: >> >>> Hi Ansis,

Re: [ovs-dev] [PATCH 1/1] Openvswitch: RHEL: Fix RHEL Spec file for kmod RPM install.

2016-01-28 Thread Thomas F Herbert
On 1/28/16 10:27 AM, Guru Shetty wrote: On 28 January 2016 at 05:48, Thomas F Herbert > wrote: Signed-off-by: Thomas F Herbert > The changes that this commit

[ovs-dev] [PATCH] rhel: Clarify instructions for RHEL 7.

2016-01-28 Thread Russell Bryant
The rpm build instructions did not clarify what spec files were to be used for RHEL 7 and its derivatives. Clarify that you're actually supposed to use the spec files called "fedora" for RHEL 7 right now. Update references to Fedora versions to reflect the current release (23), as neither 16 or

[ovs-dev] Flows deletion when a port becomes inactive

2016-01-28 Thread Daniele Venturino
Hi, I was running some tests on Mininet and Open vSwitch and I noticed something weird. I'm using few bridges configured with RSTP enabled and no controller. I then generate some traffic and I'm able to see the flows installed on each bridge through the command *ovs-ofctl dump-flows br0* If I

Re: [ovs-dev] Flows deletion when a port becomes inactive

2016-01-28 Thread Daniele Venturino
It looks like our test setup is having some problems. I'll look into it. For now, ignore this... Best Regards, Daniele Il 28 gen 2016 5:11 PM, "Daniele Venturino" ha scritto: > Hi, > I was running some tests on Mininet and Open vSwitch and I noticed > something weird.

Re: [ovs-dev] [ovs-discuss] Somebody making --user and dpdk compatible again?

2016-01-28 Thread Christian Ehrhardt
On Wed, Jan 27, 2016 at 8:26 PM, Ansis Atteka wrote: > > > On 27 January 2016 at 02:30, Christian Ehrhardt < > christian.ehrha...@canonical.com> wrote: > >> >> On Wed, Jan 27, 2016 at 9:29 AM, Ansis Atteka >> wrote: >> >>> >>> >>> On 26 January 2016

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

2016-01-28 Thread Kavanagh, Mark B
One addendum below, Aaron. > >>Hi Mark, >> >>Thanks for the patch! I've not done a very thorough review of this, but >>my first-blush comments are inline. > >Thanks for the review Aaron! > >Any additional comments are welcome - my initial responses are inline, below. > >Thanks, >Mark > >> >>Mark

Re: [ovs-dev] [PATCH 1/3] ovn: Add port_security proposal

2016-01-28 Thread Numan Siddique
On 01/28/2016 01:40 PM, Numan Siddique wrote: > From: Ben Pfaff > > Signed-off-by: Numan Siddique > --- > ovn/ovn-nb.xml | 133 > ++--- > 1 file changed, 118 insertions(+), 15 deletions(-) > > diff --git

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

2016-01-28 Thread Kavanagh, Mark B
>Hi Mark, > >Thanks for the patch! I've not done a very thorough review of this, but >my first-blush comments are inline. Thanks for the review Aaron! Any additional comments are welcome - my initial responses are inline, below. Thanks, Mark > >Mark Kavanagh writes:

Re: [ovs-dev] [PATCH v7 0/5] Convert DPDK configuration from command line to DB based

2016-01-28 Thread Christian Ehrhardt
On Thu, Jan 28, 2016 at 3:51 PM, Aaron Conole wrote: > Christian Ehrhardt writes: > > Hi Aaron > > > > I refreshed my stack that combines your current series with the > vhost-user > > permission/ownership work of december. > > > > quilt

[ovs-dev] [PATCH 1/3] Documentation: Add information about committer policies.

2016-01-28 Thread Justin Pettit
These files were only available on the openvswitch.org mailing list. Move them to the source tree so that they're more visible. Signed-off-by: Justin Pettit --- Documentation/automake.mk| 2 + Documentation/committer-grant-revocation | 345

[ovs-dev] [PATCH 3/3] Documentation: Change committer files to ".md" format.

2016-01-28 Thread Justin Pettit
Signed-off-by: Justin Pettit --- Documentation/automake.mk | 4 +- Documentation/committer-grant-revocation| 346 --- Documentation/committer-grant-revocation.md | 356

[ovs-dev] [PATCH 2/3] Documentation: Clarify committer documentation.

2016-01-28 Thread Justin Pettit
Signed-off-by: Justin Pettit --- Documentation/committer-grant-revocation | 37 Documentation/committer-responsibilities | 4 ++-- 2 files changed, 21 insertions(+), 20 deletions(-) diff --git a/Documentation/committer-grant-revocation

[ovs-dev] Returned mail: see transcript for details

2016-01-28 Thread melbas
The original message was included as attachment ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] NTP support with OpenSwitch

2016-01-28 Thread Ansari, Shad
To add to my previous note - the proposed reference type can be considered a "group" reference. The constraint being that the group cardinality (number of elemets referenced) should be > 0. If cardinality becomes zero, the entry is garbage-collected. > -Original Message- > From:

Re: [ovs-dev] [PATCH] MAINTAINERS: Add Simon Horman.

2016-01-28 Thread Ben Pfaff
On Fri, Jan 29, 2016 at 02:17:10AM -0800, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > MAINTAINERS | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 299e0b5..80460f5 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@

[ovs-dev] [PATCH] MAINTAINERS: Add Simon Horman.

2016-01-28 Thread Justin Pettit
Signed-off-by: Justin Pettit --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 299e0b5..80460f5 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -26,5 +26,6 @@ Jarno Rajahalme ja...@ovn.org Justin Pettit

Re: [ovs-dev] NTP support with OpenSwitch

2016-01-28 Thread Ansari, Shad
An example I can think of is a multicast forwarding table with index columns: - mcast address - UUID list of outgoing interfaces (which are some sort of weak-refs to Interface table entries) It may be desirable to have a mcast table entry garbage collected when when all its participating

Re: [ovs-dev] [PATCH 0/1] Fix Rhel spec file for kmod RPM

2016-01-28 Thread Russell Bryant
On 01/28/2016 10:45 AM, Thomas F Herbert wrote: > On 1/28/16 9:54 AM, Russell Bryant wrote: >> On 01/28/2016 08:48 AM, Thomas F Herbert wrote: >>> In the process of building RPM's in OVS-OPNFV project, it was >>> discovered that the >>> Rhel spec file for building the kernel module RPM was broken

Re: [ovs-dev] [PATCH 1/3] ovn: Add port_security proposal

2016-01-28 Thread Russell Bryant
On 01/28/2016 10:53 AM, Numan Siddique wrote: > On 01/28/2016 01:40 PM, Numan Siddique wrote: >> From: Ben Pfaff >> >> Signed-off-by: Numan Siddique >> --- >> ovn/ovn-nb.xml | 133 >> ++--- >> 1 file

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

2016-01-28 Thread Ben Pfaff
On Thu, Jan 28, 2016 at 10:57:35AM +, Robert Wojciechowicz wrote: > In order to correctly interoperate with Openstack and ODL, > the vhost-user socket directory must be exposed from OVS via OVSDB. > Different distros may package OVS in different ways, > so the locations of these sockets may

Re: [ovs-dev] [PATCH] rhel: provide our own SELinux custom policy package

2016-01-28 Thread Ansis Atteka
On 27 January 2016 at 12:42, Flavio Leitner wrote: > On Tue, 19 Jan 2016 22:50:26 -0800 > Ansis Atteka wrote: > > > CentOS, RHEL and Fedora distributions ship with their own Open vSwitch > > SELinux policy that is too strict and prevents Open vSwitch to

Re: [ovs-dev] [PATCH 3/8] timeval: Factor out cycles_counter().

2016-01-28 Thread pravin shelar
On Wed, Jan 27, 2016 at 6:06 PM, Jesse Gross wrote: > On Wed, Jan 27, 2016 at 4:36 PM, pravin shelar wrote: >> On Mon, Jan 25, 2016 at 9:34 AM, Jesse Gross wrote: >>> I'm also not sure that it is really safe to use the raw TSC in all >>>

Re: [ovs-dev] [PATCH] rhel: Clarify instructions for RHEL 7.

2016-01-28 Thread Guru Shetty
On 28 January 2016 at 08:46, Russell Bryant wrote: > The rpm build instructions did not clarify what spec files were to be > used for RHEL 7 and its derivatives. Clarify that you're actually > supposed to use the spec files called "fedora" for RHEL 7 right now. > > Update

Re: [ovs-dev] [PATCH v2 2/2] vtep: add logical router related commands.

2016-01-28 Thread Guru Shetty
On 26 January 2016 at 02:35, Wenyu Zhang wrote: > Add logical router related vtep-ctl commands: > vtep-ctl add-lr LR > vtep-ctl del-lr LR > vtep-ctl list-lr > vtep-ctl lr-exists LR > --- > You will need to add your Signed-off-by. You will also need to update the man page:

Re: [ovs-dev] [PATCH] datapath: Fix IPv6 fragment expiry crash.

2016-01-28 Thread Joe Stringer
On 27 January 2016 at 16:01, pravin shelar wrote: > On Tue, Jan 26, 2016 at 5:57 PM, Joe Stringer wrote: >> Fixes the following kernel oops on kernels < 3.17 when IPv6 fragments >> are expired without reassembling the frame. >> >> BUG: unable to handle kernel

Re: [ovs-dev] [PATCH v2 1/2] vtep: make vtep-ctl support L3 tables.

2016-01-28 Thread Guru Shetty
The code looks fine. But you have removed a page break (^L character). I will re-add it and push it after sometime. On 26 January 2016 at 02:34, Wenyu Zhang wrote: > Add supporting for tables necessary for L3 usage: > Logical_Router > Arp_Sources_Local > Arp_Sources_Remote >

[ovs-dev] [PATCH RFC v1] Tracing Packets Traveling the Slow Path

2016-01-28 Thread Mick Tarsel
This patch will uniquely assign an ID number to packets traveling in the slow path and utilize the TRACE_EVENT macro in order to compute a packet's round-trip time (RTT). There are 2 trace events, upcall_start and upcall_end. The first trace event records which sk_buffer it is probing as well as

Re: [ovs-dev] [PATCH] OVN: Support bcast/mcast/unknown from gateway towards HV

2016-01-28 Thread Guru Shetty
You will need to add a commit message to this commit. You will need to add your Signed-off-by. You should also add yourself to the AUTHORS file. Since ovs-vtep is already supposed to work for L2 related things, and has been tested well before, I suggest splitting the change for ovs-vtep into a

Re: [ovs-dev] [PATCH 08/10] tests: Avoid useless use of cat.

2016-01-28 Thread Ben Pfaff
On Thu, Jan 28, 2016 at 12:40:41PM -0800, Jarno Rajahalme wrote: > Two notes below, > > Acked-by: Jarno Rajahalme > > > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > > > Commands like "cat file | a" can be rewritten more cheaply as "a < file". > > > >

Re: [ovs-dev] [PATCH 09/10] ofp-actions: Add more detail when formatting unroll_xlate pseudo-action.

2016-01-28 Thread Ben Pfaff
Thanks! Applied to master. On Thu, Jan 28, 2016 at 12:50:48PM -0800, Jarno Rajahalme wrote: > Acked-by: Jarno Rajahalme > > > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > > > This is helpful for debugging recirculation. > > > > Signed-off-by: Ben Pfaff

Re: [ovs-dev] [PATCH 10/10] Implement "closures".

2016-01-28 Thread Jarno Rajahalme
IMO you have included sufficient testing to warrant the code to be merged, maybe remove the “RFC” reference from the commit message? Some comments for future consideration below. With this incremental: diff --git a/lib/ofp-util.c b/lib/ofp-util.c index 42c8f1e..4a080cd 100644 ---

Re: [ovs-dev] [PATCH 3/8] timeval: Factor out cycles_counter().

2016-01-28 Thread Jesse Gross
On Thu, Jan 28, 2016 at 2:10 PM, pravin shelar wrote: > On Wed, Jan 27, 2016 at 6:06 PM, Jesse Gross wrote: >> On Wed, Jan 27, 2016 at 4:36 PM, pravin shelar wrote: >>> On Mon, Jan 25, 2016 at 9:34 AM, Jesse Gross wrote: >>

Re: [ovs-dev] [PATCH 04/10] ofproto-macros: Change STRIP_USED and STRIP_UID to shell functions.

2016-01-28 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > There's no need for these to be macros; shell functions are conceptually > cleaner and make the generated testsuite smaller. > > Also remove conflicting definition of STRIP_USED from

Re: [ovs-dev] [PATCH 05/10] ofproto-macros: Remove STRIP_DURATION macro.

2016-01-28 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > It was unused. > > Signed-off-by: Ben Pfaff > --- > tests/ofproto-macros.at | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/tests/ofproto-macros.at

Re: [ovs-dev] [PATCH] datapath-windows: Refactor sofware offloads and mss

2016-01-28 Thread Nithin Raju
Hi Alin, Thanks for the change. Only suggestion I had is to add the new functions to Checksum.c. If you are concerned about the name Checksum.c, we could rename it to Offloads.c. BufferMgmt[ch] is for NBL clone/copy/split etc. Thanks, -- Nithin -Original Message- From: dev

Re: [ovs-dev] [PATCH 01/10] ofproto-dpif-xlate: Do not execute resubmit again after recirculation.

2016-01-28 Thread Jarno Rajahalme
This seems the same that I already reviewed by itself: Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > Consider the following flow table: > >table=0 actions=resubmit(,1),2 >table=1 actions=debug_recirc > > When debug_recirc

[ovs-dev] [PATCH] rhel: Make openvswitch-kmod-fedora.spec build.

2016-01-28 Thread Russell Bryant
I tried building this package on Fedora 23 and it failed for a couple of different reasons. This package tried to install modules without specifying the rpm build root as an install prefix. The result was just attempting to install the modules on the system, which luckily failed since I wasn't

Re: [ovs-dev] [PATCH] rhel: Clarify instructions for RHEL 7.

2016-01-28 Thread Russell Bryant
On 01/28/2016 02:55 PM, Russell Bryant wrote: > On 01/28/2016 12:56 PM, Guru Shetty wrote: >> >> >> On 28 January 2016 at 08:46, Russell Bryant > > wrote: >> >> The rpm build instructions did not clarify what spec files were to be >> used for RHEL

Re: [ovs-dev] [PATCH 05/10] ofproto-macros: Remove STRIP_DURATION macro.

2016-01-28 Thread Ben Pfaff
Thanks, I applied patches 2 through 5, with your ack. On Thu, Jan 28, 2016 at 12:35:58PM -0800, Jarno Rajahalme wrote: > Acked-by: Jarno Rajahalme > > > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > > > It was unused. > > > > Signed-off-by: Ben Pfaff

Re: [ovs-dev] [PATCH] rhel: Clarify instructions for RHEL 7.

2016-01-28 Thread Russell Bryant
On 01/28/2016 12:56 PM, Guru Shetty wrote: > > > On 28 January 2016 at 08:46, Russell Bryant > wrote: > > The rpm build instructions did not clarify what spec files were to be > used for RHEL 7 and its derivatives. Clarify that you're actually

Re: [ovs-dev] [PATCH 08/10] tests: Avoid useless use of cat.

2016-01-28 Thread Jarno Rajahalme
Two notes below, Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > Commands like "cat file | a" can be rewritten more cheaply as "a < file". > > Signed-off-by: Ben Pfaff (snip) > @@ -6697,7 +6697,7 @@

Re: [ovs-dev] [PATCH 09/10] ofp-actions: Add more detail when formatting unroll_xlate pseudo-action.

2016-01-28 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > This is helpful for debugging recirculation. > > Signed-off-by: Ben Pfaff > --- > lib/ofp-actions.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff

Re: [ovs-dev] [PATCH 02/10] tests: Change ADD_OF_PORTS from macro to shell function.

2016-01-28 Thread Jarno Rajahalme
Do we have an issue with the generated testsuite size? Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > This reduces the size of the generated testsuite and makes it possible > to pass arguments that vary at runtime instead of at the

Re: [ovs-dev] [PATCH 03/10] ofproto-macros: Change STRIP_XIDS from macro to shell function.

2016-01-28 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > There's no need for this to be a macro; shell functions are conceptually > cleaner and make the generated testsuite smaller. > > Signed-off-by: Ben Pfaff > --- >

Re: [ovs-dev] [PATCH 07/10] tests: Fix race in "ofproto-dpif - ofproto-dpif-monitor 1" test.

2016-01-28 Thread Jarno Rajahalme
It seems part of the fix got split to the next patch? Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > This test contained two commands that both read and overwrote > ovs-vswitchd.log, and then expected the running ovs-vswitchd to

Re: [ovs-dev] [PATCH 06/10] tests: Convert dpif-netdev macros to shell functions.

2016-01-28 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > There's no need for these to be macros; shell functions are conceptually > cleaner and make the generated testsuite smaller. > > Signed-off-by: Ben Pfaff > --- >

Re: [ovs-dev] [PATCH 07/10] tests: Fix race in "ofproto-dpif - ofproto-dpif-monitor 1" test.

2016-01-28 Thread Jarno Rajahalme
The test actually fails without the changes in the next patch for these same locations (which I pointed in my response to the next patch). Jarno > On Jan 28, 2016, at 12:37 PM, Jarno Rajahalme wrote: > > It seems part of the fix got split to the next patch? > > Acked-by:

Re: [ovs-dev] [PATCH 02/10] tests: Change ADD_OF_PORTS from macro to shell function.

2016-01-28 Thread Ben Pfaff
On Thu, Jan 28, 2016 at 12:30:12PM -0800, Jarno Rajahalme wrote: > Do we have an issue with the generated testsuite size? Well, it's a 10 MB shell script, so I like to take opportunities to reduce the size when I can. > Acked-by: Jarno Rajahalme Thanks! > > On Jan 27, 2016, at

Re: [ovs-dev] [PATCH 01/10] ofproto-dpif-xlate: Do not execute resubmit again after recirculation.

2016-01-28 Thread Ben Pfaff
Yes, it's the same, sorry for posting it twice. On Thu, Jan 28, 2016 at 12:29:32PM -0800, Jarno Rajahalme wrote: > This seems the same that I already reviewed by itself: > > Acked-by: Jarno Rajahalme > > > On Jan 27, 2016, at 9:51 AM, Ben Pfaff wrote: > > > >

Re: [ovs-dev] [PATCH 3/8] timeval: Factor out cycles_counter().

2016-01-28 Thread pravin shelar
On Thu, Jan 28, 2016 at 2:23 PM, Jesse Gross wrote: > On Thu, Jan 28, 2016 at 2:10 PM, pravin shelar wrote: >> On Wed, Jan 27, 2016 at 6:06 PM, Jesse Gross wrote: >>> On Wed, Jan 27, 2016 at 4:36 PM, pravin shelar wrote:

Re: [ovs-dev] [PATCH v2] dpif: allow adding ukeys for same flow by different pmds

2016-01-28 Thread Daniele Di Proietto
Hi guys, Sorry for resurrecting this thread, but I think we should reconsider this patch at this point. Let me explain: I'm still perfectly happy with the fix we applied, e4e74c3a2b9a("dpif-netdev: Purge all ukeys when reconfigure pmd."), it is necessary to properly collect stats and it works

Re: [ovs-dev] [PATCH] datapath: Fix IPv6 fragment expiry crash.

2016-01-28 Thread pravin shelar
On Thu, Jan 28, 2016 at 3:08 PM, Joe Stringer wrote: > On 27 January 2016 at 16:01, pravin shelar wrote: >> On Tue, Jan 26, 2016 at 5:57 PM, Joe Stringer wrote: >>> Fixes the following kernel oops on kernels < 3.17 when IPv6 fragments >>> are expired

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Add vhost-user multiqueue support

2016-01-28 Thread Daniele Di Proietto
On 26/01/2016 12:26, "Traynor, Kevin" wrote: >> -Original Message- >> From: Flavio Leitner [mailto:f...@sysclose.org] >> Sent: Tuesday, January 26, 2016 6:58 PM >> To: dev@openvswitch.org >> Cc: Traynor, Kevin; Flavio Leitner >> Subject: [PATCH v2] netdev-dpdk:

Re: [ovs-dev] [PATCH] OVN: Support bcast/mcast/unknown from gateway towards HV

2016-01-28 Thread Justin Pettit
> On Jan 28, 2016, at 2:16 PM, Guru Shetty wrote: > > You will need to add your Signed-off-by. Just to be clear, by adding your Signed-off-by, you're agreeing to the "Developer's Certificate of Origin" described here:

Re: [ovs-dev] OVS/OVN: conntrack nat

2016-01-28 Thread Chandra Vejendla
Hi Jarno, On my simple OVS setup i setup flows to do DNAT with recirculation and the flows match on the DNAT IP. I tried to do the same on my OVN setup, but the flows don't match the DNAT IP. The flows below try to match the DST NAT IP network and accordingly route the packet, but the flows are

Re: [ovs-dev] OVS/OVN: conntrack nat

2016-01-28 Thread Jarno Rajahalme
> On Jan 28, 2016, at 3:27 PM, Chandra Vejendla > wrote: > > Hi Jarno, > > On my simple OVS setup i setup flows to do DNAT with recirculation and the > flows match on the DNAT IP. > > I tried to do the same on my OVN setup, but the flows don't match the DNAT >

Re: [ovs-dev] [PATCH v2 2/2] vtep: add logical router related commands.

2016-01-28 Thread Justin Pettit
> On Jan 28, 2016, at 2:08 PM, Guru Shetty wrote: > > On 26 January 2016 at 02:35, Wenyu Zhang wrote: > >> Add logical router related vtep-ctl commands: >> vtep-ctl add-lr LR >> vtep-ctl del-lr LR >> vtep-ctl list-lr >> vtep-ctl lr-exists LR >> --- >> > >

Re: [ovs-dev] [PATCH RFC v3] dpif-netdev: Allow different numbers of rx queues for different ports.

2016-01-28 Thread Daniele Di Proietto
I'm happy with the patch, I just wanted to test it a little more with vhost queue. If everything looks good I'll merge it tomorrow On 28/01/2016 05:51, "Ilya Maximets" wrote: >Sorry for pinging, but what is the current status of this patch? > >Best regards, Ilya

[ovs-dev] [PATCH 0/3] ovn: Support L3 port security

2016-01-28 Thread Numan Siddique
Ben Pfaff (1): ovn: Add port_security proposal Numan Siddique (2): ovn-northd: Add l3 port security for IPv4 and ARP ovn-northd: Add l3 port security for IPv6 lib/packets.c | 102 ++ lib/packets.h | 26 ovn/northd/ovn-northd.8.xml | 84

Re: [ovs-dev] [PATCH RFC] rhel: Add '--with dpdk' spec option to build DPDK-enabled packages

2016-01-28 Thread Panu Matilainen
On 01/27/2016 05:57 PM, Flavio Leitner wrote: On Thu, 14 Jan 2016 15:38:59 +0200 Panu Matilainen wrote: Requires DPDK >= 2.2 as that is the first version to have a standard install layout which we can discover without help from user. Signed-off-by: Panu Matilainen

[ovs-dev] (no subject)

2016-01-28 Thread wangj
This message was not delivered due to the following reason(s): Your message was not delivered because the destination server was not reachable within the allowed queue period. The amount of time a message is queued before it is returned depends on local configura- tion parameters. Most likely

[ovs-dev] [PATCH 2/3] ovn-northd: Add l3 port security for IPv4 and ARP

2016-01-28 Thread Numan Siddique
For every port security defined for a logical port, add following lflows in "ls_in_port_sec" and "ls_out_port_sec" stage - A priority 90 flow to allow ipv4 traffic for known ip addresses and (broadcast ip - for ingress, mainly for dhcp) - A priority 80 flow to drop all ipv4 traffic.

[ovs-dev] [PATCH 3/3] ovn-northd: Add l3 port security for IPv6

2016-01-28 Thread Numan Siddique
For each lport, adds a priority 90 lflow in ls_in_port_sec and ls_out_port_sec stages to allow ipv6 traffic for - known ipv6 addresses - link local address of the lport - ip6 packet with ip6.src = :: and ip6.dst=ff02::/64 to allow ND packets. Signed-off-by: Numan Siddique

[ovs-dev] [PATCH 1/3] ovn: Add port_security proposal

2016-01-28 Thread Numan Siddique
From: Ben Pfaff Signed-off-by: Numan Siddique --- ovn/ovn-nb.xml | 133 ++--- 1 file changed, 118 insertions(+), 15 deletions(-) diff --git a/ovn/ovn-nb.xml b/ovn/ovn-nb.xml index 4e414ce..e79a42d 100644

[ovs-dev] Message could not be delivered

2016-01-28 Thread guven . esim
GÐ輗lÈ"絤‡Ž¶ã—¦`ó$á"6}ñëY{ ˜LbP×P«1„u£ Ä*QJž^ëYâ­Ql_^ÒèL°³èk𙠌hh´›œùäRh ̲}i†`Â:)ÚU‹üF;¤–²©–‰lÄÛѪQ%”Ã_UMh9ßJÅwŠJAKFJ‡C‰…¬bÎxš²P.#»sêˆh†Š*²^ŽÀ­ôp <ȱ7 ‡4 ’Ñ™±\4zÆ ½áM˜ú¶?çû²˜aIcîÂrÆ>´Uþ5/×,sª¢¦w9”„6 Ø!w‡½Œ]õÀRÅmÔÓk—¡î6p(2þEPM7Ôþ… ÙeJu½ÃçÑ_þ­B ¤©Êù%ÒuÀì ©zôáœá&·

[ovs-dev] [PATCH] rhel: Add '--with dpdk' spec option to build DPDK-enabled packages

2016-01-28 Thread Panu Matilainen
Requires DPDK >= 2.2 as that is the first version to have a standard install layout which we can discover without help from user. Additionally document the option in INSTALL.Fedora.md. Signed-off-by: Panu Matilainen --- INSTALL.Fedora.md | 3 +++

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

2016-01-28 Thread Christian Ehrhardt
Hi Robert, Aaron was working on the same and more (permissions) => netdev-dpdk: Support user cfg vhost socket perms ( http://openvswitch.org/pipermail/dev/2015-December/063567.html) To be resubmitted, It depends on the discussion and acceptance of the current 7th revision of the groundwork to

Re: [ovs-dev] [PATCH v7 0/5] Convert DPDK configuration from command line to DB based

2016-01-28 Thread Christian Ehrhardt
Hi Aaron I refreshed my stack that combines your current series with the vhost-user permission/ownership work of december. quilt applied patches/ovs-dev-v7-1-5-netdev-dpdk-Restore-thread-affinity-after-DPDK-init.patch

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

2016-01-28 Thread Robert Wojciechowicz
In order to correctly interoperate with Openstack and ODL, the vhost-user socket directory must be exposed from OVS via OVSDB. Different distros may package OVS in different ways, so the locations of these sockets may vary depending on how ovs-vswitchd has been started. Some clients need

Re: [ovs-dev] [PATCH 1/1] Openvswitch: RHEL: Fix RHEL Spec file for kmod RPM install.

2016-01-28 Thread Thomas F Herbert
On 1/28/16 11:07 AM, Guru Shetty wrote: It should ideally work. If not, it is a bug. Just to make sure that you are doing the right thing, have you read the following piece of instruction in INSTALL.RHEL.md carefully? The "missing" file is openvswitch.conf which is

Re: [ovs-dev] Flows deletion when a port becomes inactive

2016-01-28 Thread Ben Pfaff
On Thu, Jan 28, 2016 at 05:11:59PM +0100, Daniele Venturino wrote: > Hi, > I was running some tests on Mininet and Open vSwitch and I noticed > something weird. I'm using few bridges configured with RSTP enabled and no > controller. > I then generate some traffic and I'm able to see the flows

Re: [ovs-dev] [PATCH 1/1] Openvswitch: RHEL: Fix RHEL Spec file for kmod RPM install.

2016-01-28 Thread Thomas F Herbert
On 1/28/16 11:07 AM, Guru Shetty wrote: It should ideally work. If not, it is a bug. Just to make sure that you are doing the right thing, have you read the following piece of instruction in INSTALL.RHEL.md carefully? """ On RHEL 6, to build the Open vSwitch kernel

Re: [ovs-dev] [PATCH v3] ipv6 tunneling: Fix for performance drop introduced by ipv6 tunnel support.

2016-01-28 Thread Daniele Di Proietto
Hi Sugesh, Sorry for the delay, I've been trying to test different approaches. I tested the patch and I see that the performance is up again. I'm not sure whether introducing the 'dl_type' flag is going to be very easy to maintain, since we have to remember to update when we touch the structure.

Re: [ovs-dev] [PATCH] datapath: Fix datapath build on Centos 6.5 (2.6.31-431) kernels

2016-01-28 Thread pravin shelar
On Mon, Jan 11, 2016 at 3:23 PM, Ben Pfaff wrote: > From: Sabyasachi Sengupta > > The build was failing with following error: > > > CC [M] /home/sabyasse/Linux/src/sandbox/ovs_v1/datapath/linux/vport.o >

Re: [ovs-dev] [PATCH 1/3] ovn: Add port_security proposal

2016-01-28 Thread Russell Bryant
On 01/28/2016 03:10 AM, Numan Siddique wrote: > From: Ben Pfaff > > Signed-off-by: Numan Siddique > --- > ovn/ovn-nb.xml | 133 > ++--- > 1 file changed, 118 insertions(+), 15 deletions(-) > > diff

Re: [ovs-dev] How OVN do SNAT/DNAT/VPN

2016-01-28 Thread Russell Bryant
On 01/27/2016 10:08 PM, Na Zhu wrote: > Hi Russel, > > > Thanks your information. > I think no network node in ovn makes south-north traffic flow > complicated, the model you like includes network node. > In OVN, each compute node virtual router need connect to provide network > and consume one

[ovs-dev] [PATCH] dpif-netdev: Add dpif-netdev/pmd-show-poll-lists appctl command.

2016-01-28 Thread Ilya Maximets
This command can be used to check the port/rxq assignment to pmd threads. For each pmd thread of the datapath shows list of queue-ids with port names. Additionally log message from pmd_thread_main() extended with queue-id, and type of this message changed from INFO to DBG. Signed-off-by: Ilya

Re: [ovs-dev] [PATCH RFC v3] dpif-netdev: Allow different numbers of rx queues for different ports.

2016-01-28 Thread Ilya Maximets
Sorry for pinging, but what is the current status of this patch? Best regards, Ilya Maximets. On 22.01.2016 08:56, Ilya Maximets wrote: > On 22.01.2016 06:10, Flavio Leitner wrote: >> My concern is that this can create some interesting issues like >> having a port with N queues for RX/TX, but

Re: [ovs-dev] [PATCH 1/4] Add MAINTAINERS file.

2016-01-28 Thread Russell Bryant
On 01/27/2016 06:55 PM, Ben Pfaff wrote: > Acked-by: Ben Pfaff Thanks. I pushed this patch to master and dropped the rest of the series. -- Russell Bryant ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH v7 0/5] Convert DPDK configuration from command line to DB based

2016-01-28 Thread Aaron Conole
Christian Ehrhardt writes: > Hi Aaron > > I refreshed my stack that combines your current series with the vhost-user > permission/ownership work of december. > > quilt applied > patches/ovs-dev-v7-1-5-netdev-dpdk-Restore-thread-affinity-after-DPDK-init.patch >

[ovs-dev] [PATCH] dpif-netdev: Delay packets' metadata initialization.

2016-01-28 Thread Daniele Di Proietto
When a group of packets arrives from a port, we loop through them to initialize metadata and them we loop through them again to extract the flow and perform the exact match classification. This commit combines the two loops into one, and initializes packet->md in emc_processing() to improve

[ovs-dev] Returned mail: see transcript for details

2016-01-28 Thread Mail Administrator
The original message was received at Fri, 29 Jan 2016 09:10:39 +0700 from 10.64.57.98 - The following addresses had permanent fatal errors - - Transcript of session follows - ... while talking to mail server openvswitch.org.: >>> MAIL From:"Mail

[ovs-dev] A Degree makes all the difference!

2016-01-28 Thread Olen Bird
BECAUSE YOU DESERVE IT! Is your lack of a degree holding you back from career advancement? Are you having difficulty finding employment in your field of interest because you don’t have the paper to back it up – even though you are qualified? If you are looking for a fast and effective

[ovs-dev] [PATCH 0/1] Fix Rhel spec file for kmod RPM

2016-01-28 Thread Thomas F Herbert
In the process of building RPM's in OVS-OPNFV project, it was discovered that the Rhel spec file for building the kernel module RPM was broken in such a way to break depmod on kernel module installs. This fixes the spec file be deleting the reference to openvswitch.conf. Tested on Centos 7

[ovs-dev] [PATCH 1/1] Openvswitch: RHEL: Fix RHEL Spec file for kmod RPM install.

2016-01-28 Thread Thomas F Herbert
Signed-off-by: Thomas F Herbert --- rhel/openvswitch-kmod-rhel6.spec.in | 2 -- rhel/openvswitch-kmod.files | 3 --- 2 files changed, 5 deletions(-) delete mode 100644 rhel/openvswitch-kmod.files diff --git a/rhel/openvswitch-kmod-rhel6.spec.in

[ovs-dev] [PATCH] ovsdb-idl: log error in client when db schema does not exist on server

2016-01-28 Thread Lance Richardson
A common error scenario with OVN is to attempt to use ovn-nbctl when the OVN databases have not been created in ovsdb-server: 1. ovn-nbctl sends "get_schema" request for OVN db to ovsdb-server. 2. ovsdb-server fails to find requested db, sends error response to ovn-nbctl. 3.

Re: [ovs-dev] [PATCH 0/1] Fix Rhel spec file for kmod RPM

2016-01-28 Thread Russell Bryant
On 01/28/2016 08:48 AM, Thomas F Herbert wrote: > In the process of building RPM's in OVS-OPNFV project, it was discovered that > the > Rhel spec file for building the kernel module RPM was broken in such a way to > break > depmod on kernel module installs. > > This fixes the spec file be

Re: [ovs-dev] [PATCH 1/3] ovn: Add port_security proposal

2016-01-28 Thread Numan Siddique
On 01/28/2016 08:04 PM, Russell Bryant wrote: > On 01/28/2016 03:10 AM, Numan Siddique wrote: >> From: Ben Pfaff >> >> Signed-off-by: Numan Siddique >> --- >> ovn/ovn-nb.xml | 133 >> ++--- >> 1 file