[ovs-dev] [PATCH v3] dpdk: add support for v2.1.0

2015-09-04 Thread Timo Puha
Update relevant artifacts to add support for DPDK v2.1.0 - INSTALL.DPDK.md - acinclude.m4: Change DPDK library name - netdev-dpdk: Limit minimum mbuf size to to adapt to DPDK bug fix that changes the treatment of the requested mbuf size - build.sh: Change DPDK version number Note that this

[ovs-dev] [PATCH] INSTALL.DPDK: Fix a typo.

2015-09-04 Thread Russell Bryant
I started by just fixing s/expect/expects/ and made a couple more tweaks to the short paragraph while I was in there. Signed-off-by: Russell Bryant --- INSTALL.DPDK.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/INSTALL.DPDK.md b/INSTALL.DPDK.md index 35dd9a0..20bd1

Re: [ovs-dev] [PATCH 1/1] netdev-linux: Don't set ethtool flags if flag is already set on netdev

2015-09-04 Thread Anoob Soman
My bad, Thanks. -Anoob On 03/09/15 20:54, Jesse Gross wrote: On Thu, Sep 3, 2015 at 6:53 AM, Anoob Soman wrote: Check if ethtool flags is already set on a netdev, before trying to set it. This patch works around issues with some older verison of ethernet drivers, which tend to reset the NIC

Re: [ovs-dev] [PATCH] INSTALL.DPDK: Fix a typo.

2015-09-04 Thread Flavio Leitner
On Fri, Sep 04, 2015 at 10:46:58AM -0400, Russell Bryant wrote: > I started by just fixing s/expect/expects/ and made a couple more > tweaks to the short paragraph while I was in there. > > Signed-off-by: Russell Bryant > --- LGTM Acked-by: Flavio Leitner

Re: [ovs-dev] Fwd: [Dpdk-ovs] dpdk ovs inter VM communication

2015-09-04 Thread Flavio Leitner
> > -Original Message- > > From: Dpdk-ovs [mailto:dpdk-ovs-boun...@lists.01.org] On Behalf Of > > Gayathri > > Sent: Wednesday, September 2, 2015 6:13 PM > > To: dpdk-...@lists.01.org > > Subject: [Dpdk-ovs] dpdk ovs inter VM communication > > > > Hi *, > > > > I have dpdk 2.0 and ovs 2.4 w

Re: [ovs-dev] [PATCH 1/4] tests: Fix banner name for ovn-sbctl.

2015-09-04 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 02:46:59PM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH 2/2] ovs-router: include ovs-router.h as first header

2015-09-04 Thread Thadeu Lima de Souza Cascardo
Signed-off-by: Thadeu Lima de Souza Cascardo --- lib/ovs-router.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lib/ovs-router.c b/lib/ovs-router.c index d4a5c35..7493c79 100644 --- a/lib/ovs-router.c +++ b/lib/ovs-router.c @@ -14,6 +14,8 @@ * limitations under the Licen

[ovs-dev] [PATCH 1/2] ovs-router: add missing ds_destroy after ds was put

2015-09-04 Thread Thadeu Lima de Souza Cascardo
Signed-off-by: Thadeu Lima de Souza Cascardo --- lib/ovs-router.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/ovs-router.c b/lib/ovs-router.c index df55bb4..d4a5c35 100644 --- a/lib/ovs-router.c +++ b/lib/ovs-router.c @@ -278,6 +278,7 @@ ovs_router_lookup_cmd(struct unixctl_conn *conn

[ovs-dev] [PATCH] ovn: Document supported predicates.

2015-09-04 Thread Russell Bryant
ovn-sb(5) discussed that predicates are supported and described what they were. It did not list the predicates that are defined for OVN matches. This patch lists the supported predicates and what they expand to. Signed-off-by: Russell Bryant --- ovn/ovn-sb.xml | 21 + 1 fil

Re: [ovs-dev] [PATCH 2/4] tests: Group the OVN tests together.

2015-09-04 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 02:47:00PM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 3/4] ovn-nbctl: Sort output of most commands.

2015-09-04 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 02:47:01PM -0700, Justin Pettit wrote: > This will be helpful for unit tests. > > Signed-off-by: Justin Pettit I don't understand why this commit changes do_lport_get_macs() and do_lport_get_port_security(), since the changes appear to be no-ops. Did you omit the code to

Re: [ovs-dev] [PATCH 4/4] tests: Add unit tests for ovn-nbctl.

2015-09-04 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 02:47:02PM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit Thanks for writing tests! Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] INSTALL.DPDK: Fix a typo.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 10:46:58AM -0400, Russell Bryant wrote: > I started by just fixing s/expect/expects/ and made a couple more > tweaks to the short paragraph while I was in there. > > Signed-off-by: Russell Bryant Applied to master, thanks! ___ d

Re: [ovs-dev] [PATCH] datapath-windows: check vport attribute before access

2015-09-04 Thread Sairam Venugopal
Acked-by: Sairam Venugopal On 9/3/15, 7:01 AM, "Nithin Raju" wrote: >OVS_VPORT_ATTR_OPTIONS being an optional attribute should be >preceded by a check for valid value before access. > >Signed-off-by: Nithin Raju >--- > datapath-windows/ovsext/Vport.c | 10 ++ > 1 file changed, 6 insert

Re: [ovs-dev] [PATCH] ovn: Document supported predicates.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 01:46:30PM -0400, Russell Bryant wrote: > ovn-sb(5) discussed that predicates are supported and described what > they were. It did not list the predicates that are defined for OVN > matches. This patch lists the supported predicates and what they > expand to. > > Signed-o

Re: [ovs-dev] [PATCH] datapath-windows: check vport attribute before access

2015-09-04 Thread Ben Pfaff
Thanks Nithin and Sairam, applied to master and branch-2.4. On Fri, Sep 04, 2015 at 06:01:19PM +, Sairam Venugopal wrote: > Acked-by: Sairam Venugopal > > > On 9/3/15, 7:01 AM, "Nithin Raju" wrote: > > >OVS_VPORT_ATTR_OPTIONS being an optional attribute should be > >preceded by a check fo

Re: [ovs-dev] [PATCH] FAQ: mention about plotnetcfg tool

2015-09-04 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 11:53:17AM -0300, Flavio Leitner wrote: > The plotnetcfg is an open source tool to visualy represent > relationship between network interfaces on a single host. > > It helps to understand the path of a packet on a host. > > Signed-off-by: Flavio Leitner Applied to master

Re: [ovs-dev] [PATCH] sandbox: Include vtep dir into the sandbox path

2015-09-04 Thread Ben Pfaff
On Thu, Sep 03, 2015 at 01:08:13PM -0400, Russell Bryant wrote: > On 09/03/2015 01:02 PM, majop...@redhat.com wrote: > > From: Miguel Angel Ajo > > > > Otherwise the built vtep-ctl is not available from the > > sandbox command line. > > > > Signed-off-by: Miguel Angel Ajo > > Thanks! > > Acke

Re: [ovs-dev] [PATCH] datapath-windows: Tunnel filter initialization check

2015-09-04 Thread Sorin Vinturis
Hi Ben, Could you please apply this patch to master and branch-2.4? Thanks, Sorin -Original Message- From: Nithin Raju [mailto:nit...@vmware.com] Sent: Thursday, 3 September, 2015 17:18 To: Sorin Vinturis Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH] datapath-windows: Tunnel fi

Re: [ovs-dev] [PATCH v2] datapath-windows: Add corresponding tag for tunnel context allocations

2015-09-04 Thread Sorin Vinturis
Hi Ben, Could you please apply this patch to master and branch-2.4? Thanks, Sorin -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sorin Vinturis Sent: Wednesday, 2 September, 2015 19:45 To: dev@openvswitch.org Subject: [ovs-dev] [PATCH v2] datapath-windows:

Re: [ovs-dev] [PATCH v2] datapath-windows: New tag for tunnel filter pool allocations

2015-09-04 Thread Sorin Vinturis
Hi Ben, Could you please apply this patch to master and branch-2.4? Thanks, Sorin -Original Message- From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sorin Vinturis Sent: Wednesday, 2 September, 2015 19:44 To: dev@openvswitch.org Subject: [ovs-dev] [PATCH v2] datapath-windows:

Re: [ovs-dev] [PATCH 3/4] ovn-nbctl: Sort output of most commands.

2015-09-04 Thread Justin Pettit
> On Sep 4, 2015, at 10:54 AM, Ben Pfaff wrote: > > On Thu, Sep 03, 2015 at 02:47:01PM -0700, Justin Pettit wrote: >> This will be helpful for unit tests. >> >> Signed-off-by: Justin Pettit > > I don't understand why this commit changes do_lport_get_macs() and > do_lport_get_port_security(),

[ovs-dev] [PATCHv2] ovn-nbctl: Sort output of most commands.

2015-09-04 Thread Justin Pettit
This will be helpful for unit tests. Signed-off-by: Justin Pettit --- v1->v2: Actually sort the macs and port security addresses. --- ovn/utilities/ovn-nbctl.c | 51 ++-- 1 files changed, 44 insertions(+), 7 deletions(-) diff --git a/ovn/utilities/ovn-n

[ovs-dev] [PATCH] .travis.yml: Direct email notifications to ovs-build without encryption.

2015-09-04 Thread Ben Pfaff
Maybe it'll work, this way. Signed-off-by: Ben Pfaff --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index 70cc14b..d14f786 100644 --- a/.travis.yml +++ b/.travis.yml @@ -28,4 +28,4 @@ script: ./.travis/build.sh $OPTS notifications:

Re: [ovs-dev] [PATCHv2] ovn-nbctl: Sort output of most commands.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 01:21:59PM -0700, Justin Pettit wrote: > This will be helpful for unit tests. > > Signed-off-by: Justin Pettit > --- > v1->v2: Actually sort the macs and port security addresses. Acked-by: Ben Pfaff ___ dev mailing list dev@ope

Re: [ovs-dev] [PATCH] tnl-ports: Add destination IP and MAC address to the match.

2015-09-04 Thread Ben Pfaff
On Wed, Sep 02, 2015 at 09:03:15PM -0700, Pravin B Shelar wrote: > Currently tnl-port table wildcard destination ip and mac addresses > for given tunnel packet. That could result accepting tunnel > packets destined for other hosts. Following patch adds > support for matching for ip and mac addres

[ovs-dev] [PATCH 0/3] Fix a number of compilation errors

2015-09-04 Thread Aaron Conole
This change should fix a few compilation errors that recently cropped up on the Travis-CI builds, making the Travis-CI builds successful again. See the travis build at: https://travis-ci.org/orgcandman/ovs/builds/78823232 Aaron Conole (3): Very pleased to make this change request. Commit 74ff

[ovs-dev] [PATCH 1/3] Very pleased to make this change request.

2015-09-04 Thread Aaron Conole
Signed-off-by: Aaron Conole --- AUTHORS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/AUTHORS b/AUTHORS index 6c0989a..c532788 100644 --- a/AUTHORS +++ b/AUTHORS @@ -1,7 +1,7 @@ The following people authored or signed off on commits in the Open vSwitch source code or webpa

[ovs-dev] [PATCH 3/3] The netdev-dpdk uses the struct ether_addr rather than struct eth_addr internal ovs datatype.

2015-09-04 Thread Aaron Conole
To facilitate using either the .ea OR the struct ether_addr.addr_bytes argument for printing/logging, add a new ETH_ADDR_BYTES_ARG() define. Signed-off-by: Aaron Conole --- lib/netdev-dpdk.c | 4 ++-- lib/packets.h | 4 +++- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/

[ovs-dev] [PATCH 2/3] Commit 74ff3298c880 introduced a compilation issue due to a bad unsigned 64-bit constant, as well as an implicit narrow.

2015-09-04 Thread Aaron Conole
This commit uses the C99 ULL suffix to tell the compiler to treat the constant as 64-bits, and also masks portions of the uint64_t argument to the htons() calls to avoid compiler errors. Signed-off-by: Aaron Conole --- lib/packets.h | 4 ++-- tests/test-aa.c | 2 +- 2 files changed, 3 insertio

Re: [ovs-dev] [PATCH] tnl-ports: Add destination IP and MAC address to the match.

2015-09-04 Thread Pravin Shelar
On Fri, Sep 4, 2015 at 1:50 PM, Ben Pfaff wrote: > On Wed, Sep 02, 2015 at 09:03:15PM -0700, Pravin B Shelar wrote: >> Currently tnl-port table wildcard destination ip and mac addresses >> for given tunnel packet. That could result accepting tunnel >> packets destined for other hosts. Following

[ovs-dev] Example of Neutron security groups as OVN ACLs

2015-09-04 Thread Russell Bryant
I've been working on Neutron security groups for OVN a bit this week and have the first rough cut "working" (it does something, at least). Right now it only creates ACLs on neutron port creation. I have to go back and add cleanup, handle when ports or security groups get updated after they are cr

Re: [ovs-dev] [PATCHv2] ovn-nbctl: Sort output of most commands.

2015-09-04 Thread Justin Pettit
> On Sep 4, 2015, at 1:28 PM, Ben Pfaff wrote: > > On Fri, Sep 04, 2015 at 01:21:59PM -0700, Justin Pettit wrote: >> This will be helpful for unit tests. >> >> Signed-off-by: Justin Pettit >> --- >> v1->v2: Actually sort the macs and port security addresses. > > Acked-by: Ben Pfaff Thanks.

Re: [ovs-dev] [PATCH] .travis.yml: Direct email notifications to ovs-build without encryption.

2015-09-04 Thread Justin Pettit
Sure... Acked-by: Justin Pettit --Justin > On Sep 4, 2015, at 1:27 PM, Ben Pfaff wrote: > > Maybe it'll work, this way. > > Signed-off-by: Ben Pfaff > --- > .travis.yml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/.travis.yml b/.travis.yml > index 70cc14b..d14f7

Re: [ovs-dev] [PATCH] tnl-ports: Add destination IP and MAC address to the match.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 02:02:56PM -0700, Pravin Shelar wrote: > On Fri, Sep 4, 2015 at 1:50 PM, Ben Pfaff wrote: > > On Wed, Sep 02, 2015 at 09:03:15PM -0700, Pravin B Shelar wrote: > >> Currently tnl-port table wildcard destination ip and mac addresses > >> for given tunnel packet. That could r

Re: [ovs-dev] [PATCH] tnl-ports: Add destination IP and MAC address to the match.

2015-09-04 Thread Pravin Shelar
On Fri, Sep 4, 2015 at 4:09 PM, Ben Pfaff wrote: > On Fri, Sep 04, 2015 at 02:02:56PM -0700, Pravin Shelar wrote: >> On Fri, Sep 4, 2015 at 1:50 PM, Ben Pfaff wrote: >> > On Wed, Sep 02, 2015 at 09:03:15PM -0700, Pravin B Shelar wrote: >> >> Currently tnl-port table wildcard destination ip and ma

Re: [ovs-dev] [PATCH] tnl-ports: Add destination IP and MAC address to the match.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 04:19:52PM -0700, Pravin Shelar wrote: > On Fri, Sep 4, 2015 at 4:09 PM, Ben Pfaff wrote: > > On Fri, Sep 04, 2015 at 02:02:56PM -0700, Pravin Shelar wrote: > >> On Fri, Sep 4, 2015 at 1:50 PM, Ben Pfaff wrote: > >> > On Wed, Sep 02, 2015 at 09:03:15PM -0700, Pravin B Shel

Re: [ovs-dev] [PATCH 1/3] Very pleased to make this change request.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 04:53:28PM -0400, Aaron Conole wrote: > Signed-off-by: Aaron Conole Applied, thanks! ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 2/3] Commit 74ff3298c880 introduced a compilation issue due to a bad unsigned 64-bit constant, as well as an implicit narrow.

2015-09-04 Thread Ben Pfaff
On Fri, Sep 04, 2015 at 04:53:29PM -0400, Aaron Conole wrote: > This commit uses the C99 ULL suffix to tell the compiler to treat the > constant as 64-bits, and also masks portions of the uint64_t argument to > the htons() calls to avoid compiler errors. > > Signed-off-by: Aaron Conole Applied.

Re: [ovs-dev] [PATCH 3/3] The netdev-dpdk uses the struct ether_addr rather than struct eth_addr internal ovs datatype.

2015-09-04 Thread Ben Pfaff
Would one of you mind reviewing this? Thanks, Ben. On Fri, Sep 04, 2015 at 04:53:30PM -0400, Aaron Conole wrote: > To facilitate using either the .ea OR the struct ether_addr.addr_bytes > argument for printing/logging, add a new ETH_ADDR_BYTES_ARG() define. > > Signed-off-by: Aaron Conole > --

Re: [ovs-dev] [PATCH v2] datapath-windows: New tag for tunnel filter pool allocations

2015-09-04 Thread Ben Pfaff
Done. On Fri, Sep 04, 2015 at 07:02:15PM +, Sorin Vinturis wrote: > Hi Ben, > > Could you please apply this patch to master and branch-2.4? > > Thanks, > Sorin > > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sorin Vinturis > Sent: Wednesday, 2 Se

Re: [ovs-dev] [PATCH v2] datapath-windows: Add corresponding tag for tunnel context allocations

2015-09-04 Thread Ben Pfaff
Done. On Fri, Sep 04, 2015 at 07:01:54PM +, Sorin Vinturis wrote: > Hi Ben, > > Could you please apply this patch to master and branch-2.4? > > Thanks, > Sorin > > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Sorin Vinturis > Sent: Wednesday, 2 Se

Re: [ovs-dev] [PATCH] datapath-windows: Tunnel filter initialization check

2015-09-04 Thread Ben Pfaff
Done. On Fri, Sep 04, 2015 at 07:01:08PM +, Sorin Vinturis wrote: > Hi Ben, > > Could you please apply this patch to master and branch-2.4? > > Thanks, > Sorin > > -Original Message- > From: Nithin Raju [mailto:nit...@vmware.com] > Sent: Thursday, 3 September, 2015 17:18 > To: Sori

Re: [ovs-dev] [PATCH] ovsdb: Update _version more accurately in transaction commit.

2015-09-04 Thread Ben Pfaff
Thanks Andy, I applied this to master, branch-2.4, and branch-2.3. On Wed, Sep 02, 2015 at 11:45:40AM -0700, Andy Zhou wrote: > Acked-by: Andy Zhou > > On Mon, Aug 31, 2015 at 9:52 AM, Ben Pfaff wrote: > > The _version column in each OVSDB row is supposed to be updated whenever > > any other co

Re: [ovs-dev] Query with respect to OVSDB and vswitchd

2015-09-04 Thread Ben Pfaff
Thanks RishiRaj! I applied this to master, branch-2.4, and branch-2.3. On Wed, Sep 02, 2015 at 09:16:52AM +0530, RishiRaj Maulick wrote: > Hi Ben, > > I have tested the patch , it works , now I don't see an update with just > the _version change. > > Thanks for the quick response . > > Best Re

Re: [ovs-dev] [PATCH] lib: meta-flow.h Remove restriction on modifying ICMP packets.

2015-09-04 Thread Ben Pfaff
On Tue, Sep 01, 2015 at 08:28:04PM -0300, Marcelo Amaral wrote: > Signed-off-by: Marcelo Amaral > > When OpenStack create a virtual router using OpenDaylight, > the default gateway does not respond ping, since it does not > implement the full network stack. The solution is the reply the > gateway

[ovs-dev] [PATCH 2/2] ovn-nbctl: Add ACL commands.

2015-09-04 Thread Justin Pettit
Signed-off-by: Justin Pettit --- ovn/utilities/ovn-nbctl.8.xml | 28 + ovn/utilities/ovn-nbctl.c | 242 + tests/ovn-nbctl.at| 49 3 files changed, 319 insertions(+), 0 deletions(-) diff --git a/ovn/utilities/ovn-nbctl.8.xm

[ovs-dev] [PATCH 1/2] ovn-nb: Add direction and reduce max priority for ACLs.

2015-09-04 Thread Justin Pettit
Introduce a new "direction" column to the ACL table that accepts the values "to-lport" and "from-lport". Also reserve the ACL priority 65535 for return traffic associated with the "allow-related" action. Signed-off-by: Justin Pettit --- ovn/ovn-nb.ovsschema |4 ++- ovn/ovn-nb.xml |

[ovs-dev] [RFC PATCH] ovn: Add stateful ACL support.

2015-09-04 Thread Justin Pettit
Add support for the "allow-related" ACL action. This is dependent on the OVS conntrack functionality, which is not available on all platforms or kernel versions. Here is a sample policy that will allow all tenants in logical switch "ls0" to SSH to each other. Anyone can make an HTTP request to "

[ovs-dev] daywjujtovjcy

2015-09-04 Thread Automatic Email Delivery Software
Message could not be delivered ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Even a hero needs reliable sword!

2015-09-04 Thread Trent
Get tips for better enhancing. style2 { text-align: center; color: #D90003; font-family: Cambria, 'Hoefler Text', 'Liberation Serif', Times, 'Times New Roman', serif; font-size: 20px; font-weight: bold; } style4 {border: 1px solid #707070; } style5 {text-align: center;

Re: [ovs-dev] Example of Neutron security groups as OVN ACLs

2015-09-04 Thread Russell Bryant
On 09/04/2015 05:09 PM, Russell Bryant wrote: > I've been working on Neutron security groups for OVN a bit this week and > have the first rough cut "working" (it does something, at least). Right > now it only creates ACLs on neutron port creation. I have to go back > and add cleanup, handle when

Re: [ovs-dev] Example of Neutron security groups as OVN ACLs

2015-09-04 Thread Justin Pettit
> On Sep 4, 2015, at 8:34 PM, Russell Bryant wrote: > > On 09/04/2015 05:09 PM, Russell Bryant wrote: >> I've been working on Neutron security groups for OVN a bit this week and >> have the first rough cut "working" (it does something, at least). Right >> now it only creates ACLs on neutron por