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

> On Jan 28, 2016, at 11:53 PM, Ben Pfaff wrote: > > 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 >> ---

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

Dear user dev@openvswitch.org, We have found that your e-mail account has been used to send a huge amount of spam messages during this week. Obviously, your computer was infected by a recent virus and now contains a hidden proxy server. We recommend that you follow our instruction in order to k

[ovs-dev] [PATCH] ovn: Remove top ovn directory from PATHs.

Since 5b5c922b0ca6 ("ovn-nbctl: Move ovn-nbctl to utilities directory.") there is no more executables in top ovn directory. Removing of this directory from PATHs helps to avoid problems when old executable ./ovn/ovn-nbctl used instead of ./ovn/utilities/ovn-nbctl. This may happen if source direct

Re: [ovs-dev] Does openvswitch with DPDK datapath supports QOS

Hi all, We could extend the design of ingress policer from the egress policer patch from which you have provided. Thanks & Regards, Ravali. -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Stokes, Ian Sent: 26 November 2015 19:42 To: Ravulakollu Udaya Kumar

[ovs-dev] status

___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

On 01/28/2016 09:33 PM, Russell Bryant wrote: > 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 >>> ++---

[ovs-dev] [PATCH v3] vtep: add logical router related commands.

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 Signed-off-by: Wenyu Zhang --- vtep/vtep-ctl.8.in | 25 + vtep/vtep-ctl.c| 155 + 2 files changed, 180 inser

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

Further clarifications below Aaron - please let me know if you have any additional comments. Thanks, Mark > >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 revie

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

Hi. On 29.01.2016 04:02, Daniele Di Proietto wrote: > 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

Re: [ovs-dev] [PATCH] ovn: Remove top ovn directory from PATHs.

On 01/29/2016 04:20 AM, Ilya Maximets wrote: > Since 5b5c922b0ca6 ("ovn-nbctl: Move ovn-nbctl to utilities directory.") > there is no more executables in top ovn directory. > > Removing of this directory from PATHs helps to avoid problems when > old executable ./ovn/ovn-nbctl used instead of ./ovn

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

"Kavanagh, Mark B" writes: > Further clarifications below Aaron - please let me know if you have > any additional comments. > > Thanks, > Mark Thanks again for this work, Mark! >> >>One addendum below, Aaron. >> >>> Hi Mark, Thanks for the patch! I've not done a very thorough review

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

On 28 January 2016 at 12:46, Russell Bryant wrote: > 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 > t

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

Package: openvswitch-switch Version: 2.3.0+git20140819-3 Severity: important File: /usr/sbin/ovs-vswitchd Dear Maintainer, * What led up to the situation? ip netns add h1 ip netns add h2 ip link add sh1 type veth peer name h1 netns h1 ip link add sh2 type veth peer name h2 netns h2 ip link ad

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

On 01/29/2016 05:09 AM, Justin Pettit wrote: > These files were only available on the openvswitch.org mailing list. I think you meant web site instead of mailing list. > Move them to the source tree so that they're more visible. +1, thank you! > Signed-off-by: Justin Pettit Acked-by: Russell

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

> > @@ -74,9 +75,9 @@ $HOME/rpmbuild/SOURCES. > > ``` > rpmbuild -bb \ > - -D "kversion 2.6.32-131.6.1.el6.x86_64" \ > + -D "kversion 4.3.3-300.fc23.x86_64” \ > -D "kflavors default debug kdump" \ > - rhel/openvswitch-kmod-rh

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

On 01/29/2016 05:09 AM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > Documentation/committer-grant-revocation | 37 > > Documentation/committer-responsibilities | 4 ++-- > 2 files changed, 21 insertions(+), 20 deletions(-) Nice cleanup. Words a

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

On 01/29/2016 05:09 AM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > Documentation/automake.mk | 4 +- > Documentation/committer-grant-revocation| 346 --- > Documentation/committer-grant-revocation.md | 356 > +++

[ovs-dev] [PATCHv1 0/4] Support multiple addresses in Logical_Port.addresses and L3 port security

Ben Pfaff (1): ovn: Add port_security proposal Numan Siddique (3): ovn-northd: Support Logical_Port.addresses to store multiple ips in each set ovn-northd: Add l3 port security for IPv4 and ARP ovn-northd: Add l3 port security for IPv6 lib/packets.c | 102 + lib

[ovs-dev] [PATCHv1 1/4] ovn-northd: Support Logical_Port.addresses to store multiple ips in each set

If a logical port has two ipv4 addresses and one ipv6 address it will be stored as ["MAC IPv41 IPv42 IPv61"] instead of ["MAC IPv41", "MAC IPv42", "MAC IPv61"]. Signed-off-by: Numan Siddique --- lib/packets.c | 102 ++ lib/packets.h | 10

[ovs-dev] [PATCHv1 2/4] ovn: Add port_security proposal

From: Ben Pfaff Signed-off-by: Numan Siddique --- ovn/ovn-nb.xml | 141 +++-- 1 file changed, 126 insertions(+), 15 deletions(-) diff --git a/ovn/ovn-nb.xml b/ovn/ovn-nb.xml index b4768d0..036fbbf 100644 --- a/ovn/ovn-nb.xml +++ b/ovn/ovn-nb.

[ovs-dev] [PATCHv1 4/4] ovn-northd: Add l3 port security for IPv6

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=ff00::/8 Signed-off-by: Numan Siddique --- lib/packets.h | 16 ++

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

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. -

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

> >"Kavanagh, Mark B" writes: >> Further clarifications below Aaron - please let me know if you have >> any additional comments. >> >> Thanks, >> Mark > >Thanks again for this work, Mark! > >>> >>>One addendum below, Aaron. >>> >Hi Mark, > >Thanks for the patch! I've not done a ve

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

>"Kavanagh, Mark B" writes: > >>> >>>"Kavanagh, Mark B" writes: Further clarifications below Aaron - please let me know if you have any additional comments. Thanks, Mark >>> >>>Thanks again for this work, Mark! >>> > >One addendum below, Aaron. > >> >>

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

"Kavanagh, Mark B" writes: >> >>"Kavanagh, Mark B" writes: >>> Further clarifications below Aaron - please let me know if you have >>> any additional comments. >>> >>> Thanks, >>> Mark >> >>Thanks again for this work, Mark! >> One addendum below, Aaron. > >>Hi Mark, >>

Re: [ovs-dev] Does openvswitch with DPDK datapath supports QOS

Hi Ravali, I have a patch that does this already. I was waiting on the vhost multiq work to be merged to master before submitting it. I will look to submit it next week or so once I completed some performance testing on it. Regards Ian > -Original Message- > From: ravali.bu...@wipro.co

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

On 1/28/16 3:46 PM, Russell Bryant wrote: 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 mailto:russ...@ovn.org>> wrote: The rpm build instructions did not clarify what spec files were to be used

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

On 01/29/2016 12:04 PM, Thomas F Herbert wrote: > > > On 1/28/16 3:46 PM, Russell Bryant wrote: >> 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:

[ovs-dev] [PATCH v8 3/5] netdev-dpdk: Autofill lcore coremask if absent

The user has control over the DPDK internal lcore coremask, but this parameter can be autofilled with a bit more intelligence. If the user does not fill this parameter in, we use the lowest set bit in the current task CPU affinity. Otherwise, we will reassign the current thread to the specified lco

[ovs-dev] [PATCH v8 1/5] netdev-dpdk: Restore thread affinity after DPDK init

When the DPDK init function is called, it changes the executing thread's CPU affinity to a single core specified in -c. This will result in the userspace bridge configuration thread being rebound, even if that is not the intent. This change fixes that behavior by rebinding to the original thread a

[ovs-dev] [PATCH v8 2/5] netdev-dpdk: Convert initialization from cmdline to db

Existing DPDK integration is provided by use of command line options which must be split out and passed to librte in a special manner. However, this forces any configuration to be passed by way of a special DPDK flag, and interferes with ovs+dpdk packaging solutions. This commit delays dpdk initia

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

Currently, configuration of DPDK parameters is done via the command line through a --dpdk **OPTIONS** -- command line argument. This has a number of challenges, including: * It must be the first option passed to ovs-vswitchd * It breaks from the way most other things are configured in OVS * It does

[ovs-dev] [PATCH v8 4/5] netdev-dpdk: Allow arbitrary eal arguments

A previous change moved some commonly used arguments from commandline to the database, and with it the ability to pass arbitrary arguments to EAL. This change allows arbitrary eal arguments to be provided via a new db entry 'other_config:dpdk-extra' which will tokenize the string and add it to the

[ovs-dev] [PATCH v8 5/5] NEWS: Announce the DPDK EAL configuration change

Previous commits have converted dpdk EAL initialization from requiring a ``--dpdk ... --`` command line arguments to using the Open vSwitch database. This change announces that as significant NEWS. Signed-off-by: Aaron Conole --- v6: * Introduced v7,v8: * No change NEWS | 5 + 1 file chang

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

Thanks Guru I will address and resubmit On 1/28/16, 2:16 PM, "dev on behalf of Guru Shetty" wrote: >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 wor

[ovs-dev] [PATCH] datapath: inet: frag: Always orphan skbs inside ip_defrag().

When the linux stack is an endpoint connected to OVS which is performing IP fragmentation via conntrack actions, it's possible to hit a kernel BUG. The following upstream commit fixes the issue inside ip_defrag(). For the backport, we provide this inside ip_defrag() for kernels that we currently ba

[ovs-dev] [PATCH] Revert "ovs-lib: Try to call exit before killing."

This reverts commit 9b5422a98f817b9f2a1f8224cab7e1a8d0bbba1f. Reason: Calling 'ovs-appctl exit' on ovs-vswitchd will cause ovs-vswitchd to destroy all the ports from the datapath. This is an unacceptable behavior while restarting the daemons. Couple of reasons that come to mind are: 1. Any config

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

On Thu, 28 Jan 2016 15:46:03 -0500 Russell Bryant wrote: > 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 inst

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

On 29 January 2016 at 11:12, Flavio Leitner wrote: > On Thu, 28 Jan 2016 15:46:03 -0500 > Russell Bryant wrote: > > > 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 > >

Re: [ovs-dev] [PATCH v2 1/2] datapath-windows: Support for OVS_KEY_ATTR_MPLS attribute

Hi Sorin, Thanks for incorporating the comments from previous iteration. I had a few cosmetic comments to improve readability, but looks good otherwise. Acked-by: Nithin Raju Thanks, -- Nithin -Original Message- From: dev on behalf of Sorin Vinturis Date: Tuesday, January 19, 2016 at

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

On 1/29/16 12:43 PM, Russell Bryant wrote: On 01/29/2016 12:04 PM, Thomas F Herbert wrote: On 1/28/16 3:46 PM, Russell Bryant wrote: 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 mailto:russ...@ovn.org>>

Re: [ovs-dev] [PATCH v2 2/2] datapath-windows: Accept MPLS feature probe.

The current code is Flow.c is not great in terms of communicating what is supported and not supported. Ideally, we should be doing something like this: 1) Parse Flow attributes, and flag the flow if mandatory attributes such as KEY are missing. 2) Handle unsupported such as UFID by returning EINVA

[ovs-dev] [PATCH 1/4] tests: Fix typo in IPv6 tests.

Signed-off-by: Joe Stringer --- tests/system-traffic.at | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/system-traffic.at b/tests/system-traffic.at index 3a0d6fa88b2c..102b56fba033 100644 --- a/tests/system-traffic.at +++ b/tests/system-traffic.at @@ -1235,12 +1235,1

[ovs-dev] [PATCH 0/4] Additional conntrack kernel tests.

Through recent testing activities, several bugs were identified particularly in the areas of IP fragmentation and connection tracking, and when using these in conjunction with tunnels. These are some kernel module testsuite tests that were used to verify the fixes. Joe Stringer (4): tests: Fix t

[ovs-dev] [PATCH 3/4] tests: Add IP fragmentation expiry tests.

These tests indirectly trigger the kernel IP fragment expiry code by ensuring that not all fragments are assembled. This reproduces the bug that was fixed in the following commit: bf1f0d35ceb9 datapath: Fix IPv6 fragment expiry crash. Signed-off-by: Joe Stringer --- tests/system-traffic.at | 79

[ovs-dev] [PATCH 2/4] tests: Make vxlan frag test more reliable.

Depending on the kernel version in use, the nf_conntrack module may register hooks for each namespace and execute conntrack prior to passing packets to OVS (or not). In cases where this happens, the previous flow table in this test would trust the local stack's connection tracking and use it rather

[ovs-dev] [PATCH 4/4] tests: Add IPv6-over-vxlan-over-IPv4 tests.

Signed-off-by: Joe Stringer --- tests/system-traffic.at | 57 + 1 file changed, 57 insertions(+) diff --git a/tests/system-traffic.at b/tests/system-traffic.at index 88f02b2a8e77..28adbdcb9ee6 100644 --- a/tests/system-traffic.at +++ b/tests/system

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

On 01/29/2016 02:34 PM, Thomas F Herbert wrote: > > > On 1/29/16 12:43 PM, Russell Bryant wrote: >> On 01/29/2016 12:04 PM, Thomas F Herbert wrote: >>> >>> On 1/28/16 3:46 PM, Russell Bryant wrote: On 01/28/2016 02:55 PM, Russell Bryant wrote: > On 01/28/2016 12:56 PM, Guru Shetty wrote:

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

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 ru

Re: [ovs-dev] [PATCH 2/4] tests: Make vxlan frag test more reliable.

On 29 January 2016 at 11:37, Joe Stringer wrote: > Depending on the kernel version in use, the nf_conntrack module may > register hooks for each namespace and execute conntrack prior to passing > packets to OVS (or not). In cases where this happens, the previous flow > table in this test would tru

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

On 28 January 2016 at 16:03, pravin shelar wrote: > 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

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

On Fri, 29 Jan 2016 14:42:54 -0500 Russell Bryant wrote: > 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 inst

Re: [ovs-dev] [PATCH] datapath: inet: frag: Always orphan skbs inside ip_defrag().

On Fri, Jan 29, 2016 at 11:01 AM, Joe Stringer wrote: > When the linux stack is an endpoint connected to OVS which is performing > IP fragmentation via conntrack actions, it's possible to hit a kernel > BUG. The following upstream commit fixes the issue inside ip_defrag(). > For the backport, we p

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

On 01/29/2016 03:59 PM, Flavio Leitner wrote: > On Fri, 29 Jan 2016 14:42:54 -0500 > Russell Bryant wrote: > >> 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

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

> On Jan 29, 2016, at 7:44 AM, Russell Bryant wrote: > > Thanks. This is much easier to read, even in plain text form. > > Acked-by: Russell Bryant Thanks for the reviews. I pushed them to master. I'll update the web site to point to them in a bit. --Justin

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

On 01/29/2016 10:37 AM, Guru Shetty wrote: > @@ -74,9 +75,9 @@ $HOME/rpmbuild/SOURCES. > > ``` > rpmbuild -bb \ > - -D "kversion 2.6.32-131.6.1.el6.x86_64" \ > + -D "kversion 4.3.3-300.fc23.x86_64” \ > -D "kflavors

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

> On Jan 29, 2016, at 7:40 AM, Russell Bryant wrote: > > On 01/29/2016 05:09 AM, Justin Pettit wrote: >> These files were only available on the openvswitch.org mailing list. > > I think you meant web site instead of mailing list. D'oh. I forgot to fix this before pushing. Sorry about that.

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

On 01/29/2016 04:29 PM, Justin Pettit wrote: > >> On Jan 29, 2016, at 7:40 AM, Russell Bryant wrote: >> >> On 01/29/2016 05:09 AM, Justin Pettit wrote: >>> These files were only available on the openvswitch.org mailing list. >> >> I think you meant web site instead of mailing list. > > D'oh. I

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

> On Jan 29, 2016, at 1:30 PM, Russell Bryant wrote: > > On 01/29/2016 04:29 PM, Justin Pettit wrote: >> >>> On Jan 29, 2016, at 7:40 AM, Russell Bryant wrote: >>> >>> On 01/29/2016 05:09 AM, Justin Pettit wrote: These files were only available on the openvswitch.org mailing list. >>> >

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

On Fri, 29 Jan 2016 14:40:11 -0500 Russell Bryant wrote: > On 01/29/2016 02:34 PM, Thomas F Herbert wrote: > > > > > > On 1/29/16 12:43 PM, Russell Bryant wrote: > >> On 01/29/2016 12:04 PM, Thomas F Herbert wrote: > >>> > >>> On 1/28/16 3:46 PM, Russell Bryant wrote: > On 01/28/2016

[ovs-dev] [PATCH] committer-grant-revocation.md: Fix link and clarify location.

Signed-off-by: Justin Pettit --- Documentation/committer-grant-revocation.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/committer-grant-revocation.md b/Documentation/committer-grant-revocation.md index 191732f..83b703c 100644 --- a/Documentation/committer-g

[ovs-dev] [PATCH] MAINTAINERS: convert to .md format.

Most other doc files are in markdown format, so convert this one as well. Instead of linking to the web site, just use relative links to the policy documents in the tree. This patch also gives me a chance to hide a fix for my failure to accurately sort names in alphabetical order. Signed-off-by:

Re: [ovs-dev] [PATCH] MAINTAINERS: convert to .md format.

> On Jan 29, 2016, at 1:45 PM, Russell Bryant wrote: > > Most other doc files are in markdown format, so convert this one as > well. Instead of linking to the web site, just use relative links > to the policy documents in the tree. > > This patch also gives me a chance to hide a fix for my fai

Re: [ovs-dev] [PATCH] MAINTAINERS: convert to .md format.

On 01/29/2016 04:49 PM, Justin Pettit wrote: > Acked-by: Justin Pettit Pushed to master, thanks! -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] datapath: inet: frag: Always orphan skbs inside ip_defrag().

On 29 January 2016 at 13:10, pravin shelar wrote: > On Fri, Jan 29, 2016 at 11:01 AM, Joe Stringer wrote: >> When the linux stack is an endpoint connected to OVS which is performing >> IP fragmentation via conntrack actions, it's possible to hit a kernel >> BUG. The following upstream commit fixe

Re: [ovs-dev] [PATCH] committer-grant-revocation.md: Fix link and clarify location.

Acked-by: Russell Bryant On Friday, January 29, 2016, Justin Pettit wrote: > Signed-off-by: Justin Pettit > > --- > Documentation/committer-grant-revocation.md | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/committer-grant-revocation.md b/Documentation/

Re: [ovs-dev] [PATCH] committer-grant-revocation.md: Fix link and clarify location.

> On Jan 29, 2016, at 2:53 PM, Russell Bryant wrote: > > Acked-by: Russell Bryant Thanks! I pushed this to master. --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 06/13] compat: Detect and use nf_ipv6_ops->fragment.

On Mon, Jan 11, 2016 at 5:53 PM, Joe Stringer wrote: > Signed-off-by: Joe Stringer > --- > acinclude.m4 | 2 ++ > datapath/compat.h| 5 + > datapath/linux/compat/include/linux/netfilter_ipv6.h | 6 +++--- > datapa

[ovs-dev] [PATCH] INSTALL.md: Fix shell command line formatting.

Some shell command lines were quited inconsistently from others. Signed-off-by: Jarno Rajahalme --- INSTALL.md | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/INSTALL.md b/INSTALL.md index dc688ad..dd4d965 100644 --- a/INSTALL.md +++ b/INSTALL.md @@

[ovs-dev] [PATCH] ovs-thread: fix missing space

Signed-off-by: William Tu --- lib/ovs-thread.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/ovs-thread.c b/lib/ovs-thread.c index 7855b3a..6ebda07 100644 --- a/lib/ovs-thread.c +++ b/lib/ovs-thread.c @@ -119,7 +119,7 @@ TRY_LOCK_FUNCTION(rwlock, trywrlock); l->

[ovs-dev] [PATCH 2/2] ofproto-dpif-xlate: Clarify recirculation in do_xlate_actions().

Handle implicit recirculation explicitly for each action type, so that it is easier to follow what is happening. Signed-off-by: Jarno Rajahalme --- ofproto/ofproto-dpif-xlate.c | 164 +-- 1 file changed, 97 insertions(+), 67 deletions(-) diff --git a/ofpr

[ovs-dev] [PATCH 1/2] ofproto-dpif-xlate: Remove obsolete special case.

Bond recirculation used to insert a special rule that jumped from the internal table to table 0 using GOTO_TABLE. Since the introduction of the ofproto-dpif-rid this has not been necessary any more, so we can remove the special case that allowed GOTO_TABLE to go backwards in that case. Signed-off

Re: [ovs-dev] [PATCH 11/13] compat: Detect and use nf_ct_frag6_gather().

On Mon, Jan 11, 2016 at 5:53 PM, Joe Stringer wrote: > This function is a likely candidate for backporting, and currently > relies on version checks to include the source or not. Grep for the > appropriate functions instead, and include the backport based on that. > > Signed-off-by: Joe Stringer

Re: [ovs-dev] [PATCH] ovs-thread: fix missing space

Acked-by: Justin Pettit --Justin > On Jan 29, 2016, at 3:27 PM, William Tu wrote: > > Signed-off-by: William Tu > --- > lib/ovs-thread.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/ovs-thread.c b/lib/ovs-thread.c > index 7855b3a..6ebda07 100644 > --- a/lib/ov

Re: [ovs-dev] [PATCH 00/13] Remove conntrack backport version checks

On Mon, Jan 11, 2016 at 5:53 PM, Joe Stringer wrote: > Broadly, for the conntrack and ip_fragment backport in the OVS tree, we have > until now depended on a macro to include the entire backport based on kernel > version. Any kernel that the backport is compiled against from version v3.10 > up >

Re: [ovs-dev] [PATCH] ovs-thread: fix missing space

I'll apply it in just a minute. --Justin > On Jan 29, 2016, at 3:51 PM, Justin Pettit wrote: > > Acked-by: Justin Pettit > > --Justin > > >> On Jan 29, 2016, at 3:27 PM, William Tu wrote: >> >> Signed-off-by: William Tu >> --- >> lib/ovs-thread.c | 2 +- >> 1 file changed, 1 insertion(+)

Re: [ovs-dev] [PATCH 5/5] netdev-dpdk: Support user cfg vhost socket perms

On Fri, Dec 18, 2015 at 10:27 AM, Aaron Conole wrote: > The current DPDK vhost socket user and group permissions are derived > during creation from the DPDK library. This patch adds an action, post > socket creation, to change the socket permissions and ownership to > support multi-user systems. >

[ovs-dev] [PATCH] build: Fix for rename of MAINTAINERS file.

I renamed MAINTAINERS to MAINTAINERS.md, but forgot to update Makefile.am to reflect the new name. Signed-off-by: Russell Bryant --- Makefile.am | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) I already pushed this to master since it was a trivial fix for something that broke the build f

Re: [ovs-dev] [PATCH 1/2] ofproto-dpif-xlate: Remove obsolete special case.

On Fri, Jan 29, 2016 at 03:38:11PM -0800, Jarno Rajahalme wrote: > Bond recirculation used to insert a special rule that jumped from the > internal table to table 0 using GOTO_TABLE. Since the introduction of > the ofproto-dpif-rid this has not been necessary any more, so we can > remove the speci

Re: [ovs-dev] [PATCH 2/2] ofproto-dpif-xlate: Clarify recirculation in do_xlate_actions().

On Fri, Jan 29, 2016 at 03:38:12PM -0800, Jarno Rajahalme wrote: > Handle implicit recirculation explicitly for each action type, so that > it is easier to follow what is happening. > > Signed-off-by: Jarno Rajahalme I think this would benefit from passing 'end' instead of 'rem_len' to xlate_imp

Re: [ovs-dev] [PATCH] build: Fix for rename of MAINTAINERS file.

On Fri, Jan 29, 2016 at 4:53 PM, Russell Bryant wrote: > I renamed MAINTAINERS to MAINTAINERS.md, but forgot to update > Makefile.am to reflect the new name. > > Signed-off-by: Russell Bryant > Just ran into it. Timely fix. Thanks. Acked-by: Andy Zhou _

Re: [ovs-dev] [PATCH] build: Fix for rename of MAINTAINERS file.

On 01/29/2016 08:10 PM, Andy Zhou wrote: > > > On Fri, Jan 29, 2016 at 4:53 PM, Russell Bryant > wrote: > > I renamed MAINTAINERS to MAINTAINERS.md, but forgot to update > Makefile.am to reflect the new name. > > Signed-off-by: Russell Bryant mailto:russ...@

Re: [ovs-dev] [PATCH 2/2] ofproto-dpif-xlate: Clarify recirculation in do_xlate_actions().

> On Jan 29, 2016, at 5:09 PM, Ben Pfaff wrote: > > On Fri, Jan 29, 2016 at 03:38:12PM -0800, Jarno Rajahalme wrote: >> Handle implicit recirculation explicitly for each action type, so that >> it is easier to follow what is happening. >> >> Signed-off-by: Jarno Rajahalme > > I think this wou

Re: [ovs-dev] [PATCH 1/2] ofproto-dpif-xlate: Remove obsolete special case.

> On Jan 29, 2016, at 4:54 PM, Ben Pfaff wrote: > > On Fri, Jan 29, 2016 at 03:38:11PM -0800, Jarno Rajahalme wrote: >> Bond recirculation used to insert a special rule that jumped from the >> internal table to table 0 using GOTO_TABLE. Since the introduction of >> the ofproto-dpif-rid this has

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

The following is a proposal regarding how to allow logical and physical endpoint contexts for OVN to stay separated, from a CMS POV. A logical endpoint has the form: ls1-port1. A physical endpoint has the general form: chassis/physical port or location on chassis/encapsulation. Some advantages fo

[ovs-dev] [PATCH 1/2] dpif-netdev: Load packet pointer only once in emc_processing()

For the machines I have access to, Reloading the same pointer from memory seems to inhibit complier optimization somewhat. In emc_processing(), using a single packet pointer, instead reloading it from memory with packets[i], improves performance by 0.3 Mpps (tested with 10G NIC pushing 64 byte pac

[ovs-dev] [PATCH 2/2] dpif-netdev: Avoid copying netdev_flow_key in emc_processing()

This is essentially the same patch as the original patched posted at: http://openvswitch.org/pipermail/dev/2016-January/064971.html The original commit message: Before this commit, emc_processing() copied a netdev_flow_key if there was no exact-match cache (EMC) hit. This commit eliminate

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

This seems to be coming from "Darrell Lu"? Is that on purpose? It looks like you may have added your own "OVS-dev", but that's not necessary--Mailman will take care of it. --Justin > On Jan 29, 2016, at 6:36 PM, Darrell Lu wrote: > > The following is a proposal regarding how to allow logica

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

I noticed the sender issue Justin, but was busy in early evening, so did not fix it Thanks Darrell Ball On 1/29/16, 7:23 PM, "dev on behalf of Justin Pettit" wrote: >This seems to be coming from "Darrell Lu"? Is that on purpose? It looks like >you may have added your own "OVS-dev", but t

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

It doesn't matter. I was just confused. :-) --Justin > On Jan 29, 2016, at 10:02 PM, Darrell Ball wrote: > > I noticed the sender issue Justin, but was busy in early evening, so did not > fix it > > Thanks > Darrell Ball > > > > > >> On 1/29/16, 7:23 PM, "dev on behalf of Justin Pettit"