Re: [ovs-dev] [PATCH net v2 2/3] geneve: Relax MTU constraints

2016-02-09 Thread Tom Herbert
On Tue, Feb 9, 2016 at 5:47 PM, David Wragg wrote: > Allow the MTU of geneve devices to be set to large values, in order to > exploit underlying networks with larger frame sizes. > > GENEVE does not have a fixed encapsulation overhead (an openvswitch > rule can add variable length options), so the

Re: [ovs-dev] [PATCH] CONTRIBUTING: Strengthen testing guidelines

2016-02-09 Thread Justin Pettit
> On Feb 8, 2016, at 5:38 PM, Russell Bryant wrote: > > Somewhat related - I'd like to be more proactive at keeping an eye on > travis-ci failures. I just signed up for the build mailing list. It > looks like travis-ci is configured to post there, but I don't see any > posts from travis-ci on

Re: [ovs-dev] [PATCH] test-ovn: Add help output for parse-actions command.

2016-02-09 Thread Justin Pettit
> On Feb 9, 2016, at 9:19 PM, Ben Pfaff wrote: > > Signed-off-by: Ben Pfaff Acked-by: Justin Pettit --Justin ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH v3 7/7] Implement "closures".

2016-02-09 Thread Ben Pfaff
One purpose of OpenFlow packet-in messages is to allow a controller to interpose on the path of a packet through the flow tables. If, for example, the controller needs to modify a packet in some way that the switch doesn't directly support, the controller should be able to program the switch to se

[ovs-dev] [PATCH v3 6/7] ofproto-dpif-xlate: Distinguish "freezing" from "recirculation".

2016-02-09 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif-xlate.c | 267 +++ 1 file changed, 143 insertions(+), 124 deletions(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index f48c5ac..009d8e5 100644 --- a/ofproto/ofproto-dpif-xlate

[ovs-dev] [PATCH v3 5/7] ofproto-dpif-xlate: Break recirculation actions out from action_set.

2016-02-09 Thread Ben Pfaff
In my opinion, this is less confusing in multiple ways. I now understand the code better myself. Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif-xlate.c | 128 +-- 1 file changed, 51 insertions(+), 77 deletions(-) diff --git a/ofproto/ofproto-dpif-xlat

[ovs-dev] [PATCH v3 2/7] ofpbuf: Make ofpbuf_put_hex() accept period between hex bytes also.

2016-02-09 Thread Ben Pfaff
This makes it accept the same syntax as parse_NOTE(), so that that function can be simplified. In an upcoming commit a second action will also be able to take advantage of the same feature. Signed-off-by: Ben Pfaff --- lib/ofp-actions.c | 30 ++ lib/ofpbuf.c | 1

[ovs-dev] [PATCH v3 3/7] ofproto-dpif-xlate: Improve recirc_unroll_actions() interface.

2016-02-09 Thread Ben Pfaff
This interface is still straightforward and easier for the client. Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif-xlate.c | 21 +++-- 1 file changed, 7 insertions(+), 14 deletions(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index a6ea067..3d7dc

[ovs-dev] [PATCH v3 1/7] dynamic-string: Make ds_chomp() return true if it removed a character.

2016-02-09 Thread Ben Pfaff
This will be used in an upcoming commit. Signed-off-by: Ben Pfaff --- lib/dynamic-string.c | 7 +-- lib/dynamic-string.h | 4 ++-- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/lib/dynamic-string.c b/lib/dynamic-string.c index a6c8f6c..a5a3460 100644 --- a/lib/dynamic-string

[ovs-dev] [PATCH v3 4/7] ofproto-dpif-xlate: New helper function ctx_cancel_recirculation().

2016-02-09 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- ofproto/ofproto-dpif-xlate.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/ofproto/ofproto-dpif-xlate.c b/ofproto/ofproto-dpif-xlate.c index 3d7dc08..aa10217 100644 --- a/ofproto/ofproto-dpif-xlate.c +++ b/ofproto/ofp

[ovs-dev] [PATCH v3 0/7] another try at closures

2016-02-09 Thread Ben Pfaff
This is the third revision of my "closures" patch. The first version was: http://openvswitch.org/pipermail/dev/2016-January/064607.html The second version, unfortunately not marked v2, was: https://patchwork.ozlabs.org/patch/574946/ and the v1->v2 changes were described at: http://openvswitch.org

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

2016-02-09 Thread Ben Pfaff
On Thu, Jan 28, 2016 at 02:15:29PM -0800, Jarno Rajahalme wrote: > IMO you have included sufficient testing to warrant the code to be > merged, maybe remove the “RFC” reference from the commit message? Yes, that was a mistake, I forgot to remove it from the commit message before posting. > Some c

[ovs-dev] [PATCH v2] ovn: OVN changes for QOS settings

2016-02-09 Thread Babu Shanmugam
From: Ben Pfaff Signed-off-by: Ben Pfaff Signed-off-by: Babu Shanmugam --- ovn/controller/binding.c| 46 - ovn/controller/ovn-controller.c | 4 ovn/ovn-nb.xml | 17 +++ ovn/ovn-sb.xml | 17 +

[ovs-dev] [PATCH] test-ovn: Add help output for parse-actions command.

2016-02-09 Thread Ben Pfaff
Signed-off-by: Ben Pfaff --- tests/test-ovn.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/test-ovn.c b/tests/test-ovn.c index 0a51369..ae2787c 100644 --- a/tests/test-ovn.c +++ b/tests/test-ovn.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2015 Nicira, Inc. + * Copyri

Re: [ovs-dev] [PATCH v2] : ovn-controller - Assign zone-ids consistently

2016-02-09 Thread Ben Pfaff
On Wed, Feb 10, 2016 at 01:46:24AM +, Suryanarayan Ramamurthy wrote: > Currently, conntrack zone-id is assigned to lport by ovn-controller, > but the ovn-controller does not record this map (lport->zoneid) > > So, after ovn-controller restart, zone-ids may get set > inconsisten

[ovs-dev] [PATCH v2] : ovn-controller - Assign zone-ids consistently

2016-02-09 Thread Suryanarayan Ramamurthy
Currently, conntrack zone-id is assigned to lport by ovn-controller, but the ovn-controller does not record this map (lport->zoneid) So, after ovn-controller restart, zone-ids may get set inconsistently on lports, resulting in possible hits to already established connections.

[ovs-dev] [PATCH net v3 1/3] vxlan: Relax MTU constraints

2016-02-09 Thread David Wragg
Allow the MTU of vxlan devices without an underlying device to be set to larger values (up to a maximum based on IP packet limits and vxlan overhead). Previously, their MTUs could not be set to higher than the conventional ethernet value of 1500. This is a very arbitrary value in the context of v

[ovs-dev] [PATCH net v3 3/3] vxlan, gre, geneve: Set a large MTU on ovs-created tunnel devices

2016-02-09 Thread David Wragg
Prior to 4.3, openvswitch tunnel vports (vxlan, gre and geneve) could transmit vxlan packets of any size, constrained only by the ability to send out the resulting packets. 4.3 introduced netdevs corresponding to tunnel vports. These netdevs have an MTU, which limits the size of a packet that can

[ovs-dev] [PATCH net v3 2/3] geneve: Relax MTU constraints

2016-02-09 Thread David Wragg
Allow the MTU of geneve devices to be set to large values, in order to exploit underlying networks with larger frame sizes. GENEVE does not have a fixed encapsulation overhead (an openvswitch rule can add variable length options), so there is no relevant maximum MTU to enforce. A maximum of IP_MA

[ovs-dev] [PATCH net v2 1/3] vxlan: Relax the MTU constraints

2016-02-09 Thread David Wragg
Allow the MTU of vxlan devices without an underlying device to be set to larger values (up to a maximum based on IP packet limits and vxlan overhead). Previously, their MTUs could not be set to higher than the conventional ethernet value of 1500. This is a very arbitrary value in the context of v

[ovs-dev] [PATCH net v3 0/3] Set a large MTU on ovs-created tunnel devices

2016-02-09 Thread David Wragg
Prior to 4.3, openvswitch tunnel vports (vxlan, gre and geneve) could transmit vxlan packets of any size, constrained only by the ability to send out the resulting packets. 4.3 introduced netdevs corresponding to tunnel vports. These netdevs have an MTU, which limits the size of a packet that can

[ovs-dev] [PATCH] ovsdb-client: Fix memory leak reported by valgind.

2016-02-09 Thread William Tu
Testcase 1429: ovsdb-server/add-db and remove-db. xmemdup0 (util.c:142) main (ovsdb-client.c:133) Signed-off-by: William Tu --- Comments: This testcase happens to exercise the error and exit case, thus valgrind complains about memory leak. --- ovsdb/ovsdb-client.c | 1 + 1 file changed,

[ovs-dev] [PATCH] test-ovsdb: Fix memory leak reported by valgrind.

2016-02-09 Thread William Tu
Testcase 1314: UUID-distinct queries on scalars. Call stacks: allocate_row (row.c:37) ovsdb_row_clone (row.c:67) do_query_distinct (test-ovsdb.c:1232) ovs_cmdl_run_command (command-line.c:121) main (test-ovsdb.c:72) Signed-off-by: William Tu --- tests/test-ovsdb.c | 4 1

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

2016-02-09 Thread Russell Bryant
On 02/09/2016 03:58 PM, Suryanarayan Ramamurthy wrote: > > +update_local_zone_ids(const struct ovsrec_bridge *br_int, struct simap > *ct_zones, > + struct controller_ctx *ctx) > { > int i; > +struct smap new; > +int zone_id; > +char *zone; > + > +if (!ct

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

2016-02-09 Thread Kyle Mestery
On Tue, Feb 9, 2016 at 3:21 PM, Michael wrote: > The main point here as I currently get it is: > "the usage we are doing in OVN of ovsdb is completely different from the > usage done in ovs” > > I put this on quotes because it is not a reason not to follow the standard > but I just think that pu

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

2016-02-09 Thread Michael
The main point here as I currently get it is: "the usage we are doing in OVN of ovsdb is completely different from the usage done in ovs” I put this on quotes because it is not a reason not to follow the standard but I just think that putting code in the start_daemon to accept this case will ad

[ovs-dev] [PATCH 2/2] travis: Automatically recheck failed tests.

2016-02-09 Thread Ben Pfaff
This should make the automatic testsuite more reliable on Travis. It's better to fix tests to be more reliable, of course, but in practie it's difficult to make all of them 100% reliable. Signed-off-by: Ben Pfaff --- .travis/build.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -

[ovs-dev] [PATCH 1/2] tests: Add ability to automatically rerun failed tests.

2016-02-09 Thread Ben Pfaff
A lot of packaging was doing this already, so this simplifies their implementation. Signed-off-by: Ben Pfaff --- INSTALL.md| 5 - debian/rules | 3 +-- rhel/openvswitch-fedora.spec.in | 3 +-- rhel/openvswitch.spec.in | 3 +-- tests/aut

Re: [ovs-dev] [PATCH monitor_cond V3 01/10] ovsdb: create column index mapping between ovsdb row to monitor row

2016-02-09 Thread Andy Zhou
On Tue, Feb 9, 2016 at 6:41 AM, Liran Schour wrote: > Liran Schour/Haifa/IBM wrote on 09/02/2016 08:19:45 AM: > > > From: Liran Schour/Haifa/IBM > > To: Andy Zhou > > Cc: "" > > Date: 09/02/2016 08:19 AM > > Subject: Re: [ovs-dev] [PATCH monitor_cond V3 01/10] ovsdb: create > > column index mapp

[ovs-dev] potential branch-2.5 GRE tunnel recreate issue

2016-02-09 Thread Sabyasachi Sengupta
Hi Pravin, I'm trying out GRE tunnel configure/deconfigure with branch-2.5 and master and saw that it was not able to configure the tunnel if it is recreated with openvswitch-dkms. My environment is Centos 6.6 (2.6.32-504 kernel). I noticed that as part of commit e23775, you added ifdef guard

Re: [ovs-dev] [PATCH monitor_cond V3 03/10] ovsdb: allow non-monitored columns for condition evaluation

2016-02-09 Thread Andy Zhou
On Tue, Feb 9, 2016 at 12:06 AM, Liran Schour wrote: > Andy Zhou wrote on 08/02/2016 09:16:18 PM: > > > On Mon, Feb 8, 2016 at 2:41 AM, Liran Schour wrote: > > I see that my reply was blocked again at the mailing list... > > > > Andy Zhou wrote on 05/02/2016 10:10:59 AM: > > > > On Wed, Feb 3,

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

2016-02-09 Thread Russell Bryant
On 02/09/2016 01:21 PM, Kyle Mestery wrote: > On Tue, Feb 9, 2016 at 12:18 PM, Russell Bryant wrote: >> On 02/09/2016 01:12 PM, Kyle Mestery wrote: >>> On Tue, Feb 9, 2016 at 5:24 AM, Michael >>> wrote: Options for running ovsdb separately is done and also described in the help file.

Re: [ovs-dev] [PATCH net v2 2/3] geneve: Relax MTU constraints

2016-02-09 Thread David Wragg
Sergei Shtylyov writes: >The networking code formats comments: > > /* Like > * this. > */ Thanks. And I noticed another silly mistake. Will respin. David ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

2016-02-09 Thread Kyle Mestery
On Tue, Feb 9, 2016 at 12:18 PM, Russell Bryant wrote: > On 02/09/2016 01:12 PM, Kyle Mestery wrote: >> On Tue, Feb 9, 2016 at 5:24 AM, Michael >> wrote: >>> Options for running ovsdb separately is done and also described in the help >>> file. >>> >>> NEWS file is updated. >>> >>> I just have t

Re: [ovs-dev] [PATCH net v2 2/3] geneve: Relax MTU constraints

2016-02-09 Thread Sergei Shtylyov
On 02/09/2016 07:47 PM, David Wragg wrote: Allow the MTU of geneve devices to be set to large values, in order to exploit underlying networks with larger frame sizes. GENEVE does not have a fixed encapsulation overhead (an openvswitch rule can add variable length options), so there is no releva

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

2016-02-09 Thread Russell Bryant
On 02/09/2016 01:12 PM, Kyle Mestery wrote: > On Tue, Feb 9, 2016 at 5:24 AM, Michael wrote: >> Options for running ovsdb separately is done and also described in the help >> file. >> >> NEWS file is updated. >> >> I just have to rise a few points: >> >> - the way thing are running with start/sto

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

2016-02-09 Thread Kyle Mestery
On Tue, Feb 9, 2016 at 5:24 AM, Michael wrote: > Options for running ovsdb separately is done and also described in the help > file. > > NEWS file is updated. > > I just have to rise a few points: > > - the way thing are running with start/stop daemon from ova-lib require > the daemon to have a p

Re: [ovs-dev] [PATCH 2/2] dist-docs: Fix relative links.

2016-02-09 Thread Russell Bryant
On 02/09/2016 12:56 PM, Ben Pfaff wrote: > On Tue, Feb 09, 2016 at 12:50:45PM -0500, Russell Bryant wrote: >> On 02/09/2016 12:36 PM, Ben Pfaff wrote: >>> On Tue, Feb 09, 2016 at 11:24:22AM -0500, Russell Bryant wrote: On 02/09/2016 11:09 AM, Ben Pfaff wrote: > I didn't get the original pa

Re: [ovs-dev] [PATCH] dist-docs: Convert tabs to spaces.

2016-02-09 Thread Russell Bryant
On 02/09/2016 12:59 PM, Ben Pfaff wrote: > On Tue, Feb 09, 2016 at 12:54:19PM -0500, Russell Bryant wrote: >> This file used mixed indentation. Convert the tabs to spaces for >> consistency. >> >> Signed-off-by: Russell Bryant > > Acked-by: Ben Pfaff > Thanks, pushed to master. -- Russell B

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread Ben Pfaff
Yes. The documentation for this in the GNU make manual is really sketchy. On Tue, Feb 09, 2016 at 10:03:30AM -0800, William Tu wrote: > Thank you. Now I understand. > > A simple example is below: > [root@vm-dev tmp]# cat Makefile > test: > echo ${MAKEFLAGS};cd subdir; ${MAKE} > > test2: >

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread William Tu
Thank you. Now I understand. A simple example is below: [root@vm-dev tmp]# cat Makefile test: echo ${MAKEFLAGS};cd subdir; ${MAKE} test2: echo ${MAKEFLAGS};cd subdir; make [root@vm-dev tmp]# cat subdir/Makefile sub-make: echo "this is sub-make"; echo ${MAKEFLAGS} === if we use make

Re: [ovs-dev] [PATCH] dist-docs: Convert tabs to spaces.

2016-02-09 Thread Ben Pfaff
On Tue, Feb 09, 2016 at 12:54:19PM -0500, Russell Bryant wrote: > This file used mixed indentation. Convert the tabs to spaces for > consistency. > > Signed-off-by: Russell Bryant Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://op

Re: [ovs-dev] [PATCH 2/2] dist-docs: Fix relative links.

2016-02-09 Thread Ben Pfaff
On Tue, Feb 09, 2016 at 12:50:45PM -0500, Russell Bryant wrote: > On 02/09/2016 12:36 PM, Ben Pfaff wrote: > > On Tue, Feb 09, 2016 at 11:24:22AM -0500, Russell Bryant wrote: > >> On 02/09/2016 11:09 AM, Ben Pfaff wrote: > >>> I didn't get the original patch, for whatever reason. > >> > >> I'm havi

[ovs-dev] [PATCH] dist-docs: Convert tabs to spaces.

2016-02-09 Thread Russell Bryant
This file used mixed indentation. Convert the tabs to spaces for consistency. Signed-off-by: Russell Bryant --- build-aux/dist-docs | 32 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/build-aux/dist-docs b/build-aux/dist-docs index 73c85ed..a81

Re: [ovs-dev] [PATCH 2/2] dist-docs: Fix relative links.

2016-02-09 Thread Russell Bryant
On 02/09/2016 12:36 PM, Ben Pfaff wrote: > On Tue, Feb 09, 2016 at 11:24:22AM -0500, Russell Bryant wrote: >> On 02/09/2016 11:09 AM, Ben Pfaff wrote: >>> I didn't get the original patch, for whatever reason. >> >> I'm having the same problem... >> >>> Which links does this fix? >> >> MAINTAINERS.m

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

2016-02-09 Thread Traynor, Kevin
> -Original Message- > From: Aaron Conole [mailto:acon...@redhat.com] > Sent: Friday, January 29, 2016 5:57 PM > To: dev@openvswitch.org > Cc: Flavio Leitner ; Panu Matilainen ; > Traynor, Kevin ; Zoltan Kiss > ; Christian Ehrhardt > > Subject: [PATCH v8 0/5] Convert DPDK configuration f

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread Ben Pfaff
Make automatically passes the flags through the environment in the MAKEFLAGS variable. On Tue, Feb 09, 2016 at 09:37:34AM -0800, William Tu wrote: > I'm also running "make -j2 dist-docs" but could find -j2 optioned passed > in. Is the correct way of using this is like this: > > # MAKE="make -j2"

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread Ben Pfaff
On Tue, Feb 09, 2016 at 11:59:53AM -0500, Russell Bryant wrote: > On 02/09/2016 11:16 AM, Ben Pfaff wrote: > > When GNU make sees that a command to be executed contains the string > > $(MAKE), it makes the jobserver that limits parallelism available to > > the command. Otherwise, any sub-make that

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread Russell Bryant
Based on the explanation in the commit message, you're not going to see it reflected in $(MAKE). On 02/09/2016 12:37 PM, William Tu wrote: > Hi Ben and Russell, > > I'm also running "make -j2 dist-docs" but could find -j2 optioned passed > in. Is the correct way of using this is like this: > > #

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread William Tu
Hi Ben and Russell, I'm also running "make -j2 dist-docs" but could find -j2 optioned passed in. Is the correct way of using this is like this: # MAKE="make -j2" make dist-docs Thank you William On Tue, Feb 9, 2016 at 8:59 AM, Russell Bryant wrote: > On 02/09/2016 11:16 AM, Ben Pfaff wrote: >

Re: [ovs-dev] [PATCH 2/2] dist-docs: Fix relative links.

2016-02-09 Thread Ben Pfaff
On Tue, Feb 09, 2016 at 11:24:22AM -0500, Russell Bryant wrote: > On 02/09/2016 11:09 AM, Ben Pfaff wrote: > > I didn't get the original patch, for whatever reason. > > I'm having the same problem... > > > Which links does this fix? > > MAINTAINERS.md links to .md files in Documentation/. I und

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

2016-02-09 Thread Ryan Moats
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 rechecking of rows skipped due to a logical_datapath not exist

[ovs-dev] [PATCH v2 1/1] [ovn-controller] Add logical flow incremental processing

2016-02-09 Thread Ryan Moats
Changed over from trying to work with logical datapaths to actually working with logical flows. A test with deploying 400 copies of this pattern in openstack (n1 -- r1 -- n2) results in 22083 flows in the logical flow table with the peak number of flows being worked in a single lflow_run pass bei

Re: [ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread Russell Bryant
On 02/09/2016 11:16 AM, Ben Pfaff wrote: > When GNU make sees that a command to be executed contains the string > $(MAKE), it makes the jobserver that limits parallelism available to > the command. Otherwise, any sub-make that executes sees that > parallelism is enabled but does not have access to

[ovs-dev] [PATCH net v2 1/3] vxlan: Relax the MTU constraints

2016-02-09 Thread David Wragg
Allow the MTU of vxlan devices without an underlying device to be set to larger values (up to a maximum based on IP packet limits and vxlan overhead). Previously, their MTUs could not be set to higher than the conventional ethernet value of 1500. This is a very arbitrary value in the context of v

[ovs-dev] [PATCH net v2 2/3] geneve: Relax MTU constraints

2016-02-09 Thread David Wragg
Allow the MTU of geneve devices to be set to large values, in order to exploit underlying networks with larger frame sizes. GENEVE does not have a fixed encapsulation overhead (an openvswitch rule can add variable length options), so there is no relevant maximum MTU to enforce. A maximum of IP_MA

[ovs-dev] [PATCH net v2 3/3] vxlan, gre, geneve: Set a large MTU on ovs-created tunnel devices

2016-02-09 Thread David Wragg
Prior to 4.3, tunnel vports (vxlan, gre and geneve) could transmit vxlan packets of any size, constrained only by the ability to send out the resulting packets. 4.3 introduced netdevs corresponding to tunnel vports. These netdevs have an MTU, which limits the size of a packet that can be successf

[ovs-dev] [PATCH net v2 0/3] Set a large MTU on ovs-created tunnel devices

2016-02-09 Thread David Wragg
Prior to 4.3, tunnel vports (vxlan, gre and geneve) could transmit vxlan packets of any size, constrained only by the ability to send out the resulting packets. 4.3 introduced netdevs corresponding to tunnel vports. These netdevs have an MTU, which limits the size of a packet that can be successf

Re: [ovs-dev] [PATCH 2/2] dist-docs: Fix relative links.

2016-02-09 Thread Russell Bryant
On 02/09/2016 11:09 AM, Ben Pfaff wrote: > I didn't get the original patch, for whatever reason. I'm having the same problem... > Which links does this fix? MAINTAINERS.md links to .md files in Documentation/. The files in Documentation/ had similar issues, I think, but that may have been fixed

Re: [ovs-dev] [PATCH 1/2] dist-docs: Properly pass "make" options into sub-make.

2016-02-09 Thread Ben Pfaff
On Tue, Feb 09, 2016 at 08:46:56AM -0500, Russell Bryant wrote: > On 02/08/2016 11:18 PM, Ben Pfaff wrote: > > In particular this ensures that the sub-make that dist-docs runs has the > > benefit of any parallelism flags passed into the top-level make. > > > > Signed-off-by: Ben Pfaff > > diff -

[ovs-dev] [PATCH v2] dist-docs: Make GNU make aware of sub-make.

2016-02-09 Thread Ben Pfaff
When GNU make sees that a command to be executed contains the string $(MAKE), it makes the jobserver that limits parallelism available to the command. Otherwise, any sub-make that executes sees that parallelism is enabled but does not have access to the jobserver, so it prints a warning and turns

Re: [ovs-dev] [PATCH 2/2] dist-docs: Fix relative links.

2016-02-09 Thread Ben Pfaff
I didn't get the original patch, for whatever reason. Which links does this fix? ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] How to set my own ttl value to mpls flows.

2016-02-09 Thread Ben Pfaff
On Tue, Feb 09, 2016 at 01:04:28PM +, Pynbiang Hadem wrote: > Dear Ben,I have tried to make the changes to openvswitch to introduce the > set_field action for MPLS TTL field as below:  > sudo ovs-ofctl -O OpenFlow13 add-flow s1 > "table=0,in_port=1,eth_type=0x800,actions=push_mpls:0x8847,set_

Re: [ovs-dev] [PATCH v1] Basic GTP-U tunnel implementation in ovs

2016-02-09 Thread Gray, Mark D
> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Niti Rohilla > Sent: Friday, February 5, 2016 10:48 AM > To: dev@openvswitch.org > Cc: deepankar.gu...@tcs.com; partha.da...@tcs.com > Subject: [ovs-dev] [PATCH v1] Basic GTP-U tunnel implementation in ovs > > GPRS Tunneling Protocol (G

[ovs-dev] [PATCH v2] INSTALL.DPDK: Update details of XL710 restrictions for DPDK 2.2.

2016-02-09 Thread Ian Stokes
DPDK 2.2 removes restrictions related to maximum number of TX queues for XL710 devices. Update documentation to reflect these changes. Signed-off-by: Ian Stokes --- INSTALL.DPDK.md | 22 +++--- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/INSTALL.DPDK.md b/I

Re: [ovs-dev] [PATCH monitor_cond V3 01/10] ovsdb: create column index mapping between ovsdb row to monitor row

2016-02-09 Thread Liran Schour
Liran Schour/Haifa/IBM wrote on 09/02/2016 08:19:45 AM: > From: Liran Schour/Haifa/IBM > To: Andy Zhou > Cc: "" > Date: 09/02/2016 08:19 AM > Subject: Re: [ovs-dev] [PATCH monitor_cond V3 01/10] ovsdb: create > column index mapping between ovsdb row to monitor row > > Andy Zhou wrote on 09/02/

[ovs-dev] [PATCH v2] INSTALL.DPDK: Add notes regarding vhost multiq configuration.

2016-02-09 Thread Ian Stokes
Linux kernel network devices in a guest should have the number of multi-purpose channels configured when used with DPDK multiqueue on the host. This commit adds an example of how this can be done. Also add QEMU 2.5 requirements for multiqueue with DPDK in NEWS. Signed-off-by: Ian Stokes --- INST

Re: [ovs-dev] [ovs-dev, v1] INSTALL.DPDK: Add notes regarding vhost multiq configuration.

2016-02-09 Thread Stokes, Ian
Thanks for the review Ilya, much appreciated, comments inline. > Hi. > There is one issue with patch itself: > Applying: INSTALL.DPDK: Add notes regarding vhost multiq > configuration. > warning: 3 lines add whitespace errors. > Noted, I'll clean these up for the v2. > And one comment

Re: [ovs-dev] [PATCH 2/2] dist-docs: Install plaintext files properly.

2016-02-09 Thread Russell Bryant
On 02/08/2016 11:18 PM, Ben Pfaff wrote: > We shouldn't try to treat every file as Markdown. > > Signed-off-by: Ben Pfaff Acked-by: Russell Bryant -- Russell Bryant ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH 1/2] dist-docs: Properly pass "make" options into sub-make.

2016-02-09 Thread Russell Bryant
On 02/08/2016 11:18 PM, Ben Pfaff wrote: > In particular this ensures that the sub-make that dist-docs runs has the > benefit of any parallelism flags passed into the top-level make. > > Signed-off-by: Ben Pfaff > --- > Makefile.am | 2 +- > build-aux/dist-docs | 2 +- > 2 files changed,

Re: [ovs-dev] [ovs-dev, v1] INSTALL.DPDK: Add notes regarding vhost multiq configuration.

2016-02-09 Thread Ilya Maximets
Hi. There is one issue with patch itself: Applying: INSTALL.DPDK: Add notes regarding vhost multiq configuration. warning: 3 lines add whitespace errors. And one comment inlined. Best regards, Ilya Maximets. On 05.02.2016 19:55, Ian Stokes wrote: > Kernel network devices in a gue

Re: [ovs-dev] [PATCH 1/2] dist-docs: Include Documentation/ in dist-docs.

2016-02-09 Thread Russell Bryant
On 02/08/2016 11:21 PM, Ben Pfaff wrote: > On Mon, Feb 08, 2016 at 12:25:13PM -0500, Russell Bryant wrote: >> Signed-off-by: Russell Bryant > > Thanks. I noticed two other issues while testing this, so I sent > patches for those too: > https://patchwork.ozlabs.org/patch/580625/ >

Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: vhost-user: Note about number of queues inside guest OS.

2016-02-09 Thread Ilya Maximets
Sorry. Missed your patch. I'll reply my comments to it. Best regards, Ilya Maximets. On 09.02.2016 13:02, Stokes, Ian wrote: > Hi Ilya, > > I've submitted a patch for this last week which is waiting on review if you'd > like to provide feedback for it > > http://openvswitch.org/pipermail/dev/2

Re: [ovs-dev] How to set my own ttl value to mpls flows.

2016-02-09 Thread Pynbiang Hadem
Dear Ben,I have tried to make the changes to openvswitch to introduce the set_field action for MPLS TTL field as below:  sudo ovs-ofctl -O OpenFlow13 add-flow s1 "table=0,in_port=1,eth_type=0x800,actions=push_mpls:0x8847,set_field:100->mpls_label,set_field:60->mpls_ttl,output:2"mininet@mininet-vm

[ovs-dev] [PATCH V2] Separating ovn nb&sb databases

2016-02-09 Thread Michael
Options for running ovsdb separately is done and also described in the help file. NEWS file is updated. I just have to rise a few points: - the way thing are running with start/stop daemon from ova-lib require the daemon to have a pidfile named daemon.pid and this is impossible to use in our ca

Re: [ovs-dev] [PATCH] INSTALL.DPDK.md: vhost-user: Note about number of queues inside guest OS.

2016-02-09 Thread Stokes, Ian
Hi Ilya, I've submitted a patch for this last week which is waiting on review if you'd like to provide feedback for it http://openvswitch.org/pipermail/dev/2016-February/065739.html Thaks Ian > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Ilya > Maxi

[ovs-dev] [PATCH] INSTALL.DPDK.md: vhost-user: Note about number of queues inside guest OS.

2016-02-09 Thread Ilya Maximets
Signed-off-by: Ilya Maximets --- INSTALL.DPDK.md | 9 + 1 file changed, 9 insertions(+) diff --git a/INSTALL.DPDK.md b/INSTALL.DPDK.md index d892788..250ca7a 100644 --- a/INSTALL.DPDK.md +++ b/INSTALL.DPDK.md @@ -586,6 +586,15 @@ Follow the steps below to attach vhost-user port(s) to a

[ovs-dev] [PATCH V5] Add Passive TCP connection to IDL.

2016-02-09 Thread ofer . benyacov
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 schema). In addition, it enables IDL to go back to "listen m

[ovs-dev] [PATCH 1/2] packets: Fix UDP over IPv6 checksum

2016-02-09 Thread Thadeu Lima de Souza Cascardo
Fix the IPv6 pseudoheader checksum. Requires that the payload length is given to the function. Signed-off-by: Thadeu Lima de Souza Cascardo --- lib/netdev-vport.c | 14 +- lib/packets.c | 8 lib/packets.h | 2 +- 3 files changed, 14 insertions(+), 10 deletions(-)

[ovs-dev] [PATCH 2/2] packets: fix compose_nd to not crash vswitchd

2016-02-09 Thread Thadeu Lima de Souza Cascardo
packet_set_ipv6 and packet_set_nd assumed that valid packets were already in place, so they were not a good fit for composing new packets. In fact, packet_set_ipv6 didn't even set the IP version or set L4 offset, causing crashes when compose_nd tried to access L4. This patch introduces ipv6_compos

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

2016-02-09 Thread Mail Delivery Subsystem
The message was not delivered due to the following reason: Your message was not delivered because the destination computer was unreachable 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 there i