[ovs-dev] [dpdk-ovs] OVS 2.5.0 is incompatible with latest dpdk-16.04

2016-04-11 Thread Wang, Alex
Hi Kevin, Latest official DPDK-16.04 has been released and we're trying to use OVS(v2.5.0) with this version DPDK. However current OVS cannot support dpdk16.04 and we hit the incompatible issue which caused OVS failed to build with dpdk16.04. According to the fail

[ovs-dev] Dev, DON'T FEAR from getting know closer Robinet Bannowsky

2016-04-11 Thread Robinet N . Bannowsky
Ed his feet away on for help.Hi f#ck senseii found your pics on facebook. you are pretty boy .want to h00kup? send me a B00tyCall request so we can meet :-)My screen name is Robinet98 9-)My account is - http://ypmoqxkw。LambdaDating。ru?pid=Robinet98It is me Talk soon! __

[ovs-dev] Eytbmog

2016-04-11 Thread Returned mail
The original message was received at Tue, 12 Apr 2016 12:06:55 +0800 from openvswitch.org [36.190.237.106] - The following addresses had permanent fatal errors - - Transcript of session follows - while talking to openvswitch.org.: >>> MAIL From:"Returned mail" <<< 501 "Returne

Re: [ovs-dev] [PATCH] ovsdb: Force columns that contain weak references to be immutable.

2016-04-11 Thread Ben Pfaff
Thanks. I applied this patch to master. On Tue, Apr 12, 2016 at 02:37:41AM +, Elluru, Krishna Mohan wrote: > HI Ben, > > I would like you to make the patch, if there is no counter proposal at the > moment. > > Thanks > Krishna Mohan > > -Original Message- > From: Ben Pfaff [mailto

Re: [ovs-dev] A proposal of ovn-scale-test tool

2016-04-11 Thread Ben Pfaff
OK, I'm happy with that. I created a "Scale Test" team within the Open vSwitch organization on github. I made you the owner of this team. I believe that means that, if you want to transfer the ovn-scale-test repo into the Open vSwitch organization, you can now do it. On Tue, Apr 12, 2016 at 10:

Re: [ovs-dev] [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router

2016-04-11 Thread Na Zhu
Hi Russel, I use my gmail to send the review again, now it is available in below place. https://patchwork.ozlabs.org/patch/609231/ Regards, Juno Zhu IBM China Development Labs (CDL) Cloud IaaS Lab Email: na...@cn.ibm.com 5F, Building 10, 399 Keyuan Road, Zhangjiang Hi-Tech Park, Pudong New Di

Re: [ovs-dev] A proposal of ovn-scale-test tool

2016-04-11 Thread Ben Pfaff
On Mon, Apr 11, 2016 at 06:58:42PM -0700, Han Zhou wrote: > On Mon, Apr 11, 2016 at 3:57 PM, Ben Pfaff wrote: > > > > [dropping some CCs for people I know to be on ovs-dev] > > > > On Sun, Apr 10, 2016 at 08:45:38PM -0700, Han Zhou wrote: > > > As requested by several folks and also mentioned in l

[ovs-dev] [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router

2016-04-11 Thread Na Zhu
This patch add column "enabled" to table Logical_Router for setting router administrative state. The type of "enabled" is bool. If the administrative state is false, delete all the flows relevant to the logical router from table Logical_Flow. Signed-off-by: Na Zhu --- ovn/northd/ovn-northd.8.x

Re: [ovs-dev] [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router

2016-04-11 Thread Na Zhu
Hi Russel, If i use "git send-mail", can i still use IBM mail client, or should i change to use gmail client? Regards, Juno Zhu IBM China Development Labs (CDL) Cloud IaaS Lab Email: na...@cn.ibm.com 5F, Building 10, 399 Keyuan Road, Zhangjiang Hi-Tech Park, Pudong New District, Shanghai, Chi

Re: [ovs-dev] [PATCH v4] acinclude: Autodetect DPDK location when configuring OVS

2016-04-11 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 08:03:05PM +0100, Bhanuprakash Bodireddy wrote: > When using DPDK datapath, the OVS configure script requires the DPDK > build directory passed on --with-dpdk. This can be avoided if DPDK > library, headers are in standard compiler search paths. > > This patch fixes the pro

Re: [ovs-dev] [PATCH] ovsdb: Force columns that contain weak references to be immutable.

2016-04-11 Thread Elluru, Krishna Mohan
HI Ben, I would like you to make the patch, if there is no counter proposal at the moment. Thanks Krishna Mohan -Original Message- From: Ben Pfaff [mailto:b...@ovn.org] Sent: Monday, April 11, 2016 8:46 PM To: Elluru, Krishna Mohan Cc: dev@openvswitch.org Subject: Re: [ovs-dev] [PATC

Re: [ovs-dev] [PATCH v2] ofp-actions: Add a new action to truncate a packet.

2016-04-11 Thread William Tu
Hi Pravin and Ben, Thanks for the feedback and yes, it's better to make truncate without associating to the output action. I will make the modification and resubmit v3 patch. Regards, William On Mon, Apr 11, 2016 at 6:49 PM, Ben Pfaff wrote: > On Mon, Apr 11, 2016 at 06:46:27PM -0700, pravin s

Re: [ovs-dev] A proposal of ovn-scale-test tool

2016-04-11 Thread Lei Huang
Hi Ben, In my opinion, put the scale test in a separate repo under https://github.com/openvswitch/ is a better choice. As you said OVS is quickly evolving and the review process is heavyweight and slow, workload is already high, put scale test into it can cause more overhead. The review & merge p

Re: [ovs-dev] A proposal of ovn-scale-test tool

2016-04-11 Thread Han Zhou
On Mon, Apr 11, 2016 at 3:57 PM, Ben Pfaff wrote: > > [dropping some CCs for people I know to be on ovs-dev] > > On Sun, Apr 10, 2016 at 08:45:38PM -0700, Han Zhou wrote: > > As requested by several folks and also mentioned in last week's ovn > > meeting, we would like this repo to be hosted publi

Re: [ovs-dev] [PATCH v2] ofp-actions: Add a new action to truncate a packet.

2016-04-11 Thread Ben Pfaff
On Mon, Apr 11, 2016 at 06:46:27PM -0700, pravin shelar wrote: > On Mon, Apr 11, 2016 at 5:58 PM, Ben Pfaff wrote: > > On Mon, Apr 11, 2016 at 01:14:26PM -0700, pravin shelar wrote: > >> The datapath truncate action should not be associated with output > >> action. It should be general truncate ac

[ovs-dev] [PATCH 1/2] physical: Add local vars to make physical_run() easier to understand.

2016-04-11 Thread Ben Pfaff
'binding->datapath->tunnel_key' and 'binding->tunnel_key' were used repeatedly and in my opinion having local variables for each of these makes the code a little easier to understand. Signed-off-by: Ben Pfaff --- ovn/controller/physical.c | 46 -- 1 fi

[ovs-dev] [PATCH 2/2] ovn-controller: Introduce helpers for looking up datapaths.

2016-04-11 Thread Ben Pfaff
The new helpers get_local_datapath() and get_patched_datapath() make code a little shorter and easier to read. They also avoid a pitfall that was present in at least a few of the instances: CONTAINER_OF is not safe on a null pointer, because it does a raw pointer subtraction and will change NULL t

Re: [ovs-dev] [PATCH v2] ofp-actions: Add a new action to truncate a packet.

2016-04-11 Thread pravin shelar
On Mon, Apr 11, 2016 at 5:58 PM, Ben Pfaff wrote: > On Mon, Apr 11, 2016 at 01:14:26PM -0700, pravin shelar wrote: >> The datapath truncate action should not be associated with output >> action. It should be general truncate action with truncate length >> parameter. This way we can apply it to act

Re: [ovs-dev] [PATCH v1 2/2] ovn: Send garp on localnet

2016-04-11 Thread Ben Pfaff
On Sun, Apr 03, 2016 at 09:49:04PM -0400, Ramu Ramamurthy wrote: > In some usecases such as VM migration or when VMs reuse IP addresses, > VMs become unreachable externally because > external switches/routers on localnet > have stale port-mac or arp caches. The problem resolves > after some time wh

Re: [ovs-dev] [PATCH v1 1/2] ovn: Move extract_lport_addresses

2016-04-11 Thread Ben Pfaff
On Sun, Apr 03, 2016 at 09:49:03PM -0400, Ramu Ramamurthy wrote: > Move the function extract_lport_addresses to a file > in ovn/lib since that function can be used by ovn-controller also > to parse addresses stored in the mac column of the > port_binding table. Currently that function is used only

Re: [ovs-dev] [PATCH v2] ofp-actions: Add a new action to truncate a packet.

2016-04-11 Thread Ben Pfaff
On Mon, Apr 11, 2016 at 01:14:26PM -0700, pravin shelar wrote: > The datapath truncate action should not be associated with output > action. It should be general truncate action with truncate length > parameter. This way we can apply it to actions other than output. This may require more care than

Re: [ovs-dev] [PATCH V3] Configure ovn-controller SB probe_timer on the fly and disable other unix domain socket based connections

2016-04-11 Thread Ben Pfaff
Please limit the subject to about 65 characters. This is mentioned in CONTRIBUTING.md. Please don't indent the commit message. On Fri, Apr 01, 2016 at 03:28:11PM -0700, ngh...@us.ibm.com wrote: > Configure ovn-controller SB probe_timer on the fly and disable other unix > domain socket based

Re: [ovs-dev] [PATCH v4 2/2] ovn: Apply ACL changes to existing connections.

2016-04-11 Thread Ben Pfaff
On Fri, Apr 01, 2016 at 11:20:21AM -0700, Russell Bryant wrote: > Prior to this commit, once a connection had been committed to the > connection tracker, the connection would continue to be allowed, even > if the policy defined in the ACL table changed. This patch changes > the implementation so t

Re: [ovs-dev] [PATCH v4 1/2] ovn: Add ct_commit(ct_mark=INT, ct_label=INT); action.

2016-04-11 Thread Ben Pfaff
On Fri, Apr 01, 2016 at 11:20:20AM -0700, Russell Bryant wrote: > Update the "ct_commit;" logical flow action to optionally take > one or two parameters, setting the value of "ct_mark" or "ct_label". > Supported ct_commit syntax now includes: > > ct_commit; > ct_commit(); > ct_commit(c

Re: [ovs-dev] [PATCH] ovn-northd: Add support for static_routes.

2016-04-11 Thread Guru Shetty
On 11 April 2016 at 16:52, Mickey Spiegel wrote: > -Gurucharan Shetty wrote: - > > >To: dev@openvswitch.org > >From: Gurucharan Shetty > >Sent by: "dev" > >Date: 04/11/2016 07:46AM > >Cc: Gurucharan Shetty > >Subject: [ovs-dev] [PATCH] ovn-northd: Add support for static_routes. > > > >s

Re: [ovs-dev] [PATCH] ovn-northd: Add support for static_routes.

2016-04-11 Thread Mickey Spiegel
Gurucharan Shetty wrote: >To: dev@openvswitch.org >From: Gurucharan Shetty >Sent by: "dev" >Date: 04/11/2016 07:46AM >Cc: Gurucharan Shetty >Subject: [ovs-dev] [PATCH] ovn-northd: Add support for static_routes. > >static routes are useful when connecting multiple >routers with each other. Yes

[ovs-dev] Report

2016-04-11 Thread MAILER-DAEMON
Your message was not delivered due to the following reason(s): Your message could not be 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 likel

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-11 Thread Aaron Conole
Hi Ben, Ben Pfaff writes: > (Yow, that's a lot of CCs.) Lots of cooks in the kitchen on this one. > On Fri, Apr 01, 2016 at 11:31:31AM -0400, Aaron Conole wrote: >> This commit adds a new function (ovs_realpath) to perform the role of >> realpath on various operating systems. The purpose is to

Re: [ovs-dev] [PATCH v11 2/8] util: Add a path canonicalizer

2016-04-11 Thread Ben Pfaff
(Yow, that's a lot of CCs.) On Fri, Apr 01, 2016 at 11:31:31AM -0400, Aaron Conole wrote: > This commit adds a new function (ovs_realpath) to perform the role of > realpath on various operating systems. The purpose is to ensure that a > given path to file exists, and to return a completely resolve

Re: [ovs-dev] [ovs-dev, v4] Dynamically reconnect ovn-controller if ovn-remote value changes

2016-04-11 Thread Ben Pfaff
On Tue, Apr 05, 2016 at 10:22:48AM -0500, Ryan Moats wrote: > From: RYAN D. MOATS > > Allows for auto detection and reconnect if the ovn-remote needs > to change. Ovn-controller test case updated to include testing > this code. > > Signed-off-by: RYAN D. MOATS Thanks for the patch! I think t

Re: [ovs-dev] [ovs-dev, v13, 4/8] Add incremental proessing to lflow_run

2016-04-11 Thread Ben Pfaff
s/proessing/processing/ in subject. On Thu, Mar 31, 2016 at 10:05:08AM -0500, Ryan Moats wrote: > From: RYAN D. MOATS > > This code changes lflow_run to do incremental process of the > logical flow table rather than processing the full table each run. > > Signed-off-by: RYAN D. MOATS It's beg

Re: [ovs-dev] [ovs-dev, v13, 3/8] Persist local_datapaths and patched_datapaths

2016-04-11 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 10:05:07AM -0500, Ryan Moats wrote: > From: RYAN D. MOATS > > Persist local_datapaths and patch_datapaths across runs so > that changes to either can be used as a trigger to reset > incremental flow processing. > > Signed-off-by: RYAN D. MOATS New compiler errors:

Re: [ovs-dev] [ovs-dev,v13,2/8] Persist lport and mcgroup indexes

2016-04-11 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 10:05:06AM -0500, Ryan Moats wrote: > From: RYAN D. MOATS > > Persisting these entries is a pre-requisite for incremental > processing. > > Signed-off-by: RYAN D. MOATS I get lots of new errors here: ../ovn/controller/lport.c:48:39: error: passing 'const struct hmap_no

Re: [ovs-dev] [ovs-dev, v13, 1/8] Make flow table persistent in ovn controller

2016-04-11 Thread Ben Pfaff
On Thu, Mar 31, 2016 at 10:05:05AM -0500, Ryan Moats wrote: > From: RYAN D. MOATS > > This is a prerequisite for incremental processing. > > Signed-off-by: RYAN D. MOATS I'm really happy with the easier-to-understand use of sequence numbers in this version of the patch. I think that it can be

Re: [ovs-dev] [PATCH 1/1] ovsdb-idl: Retain column values of deleted rows until change-track is cleared.

2016-04-11 Thread Ben Pfaff
On Fri, Apr 01, 2016 at 10:15:42AM -0600, Ryan Moats wrote: > It looks like the calls to SBREC_*_EACH_TRACKED only show you the > incremental changes since tracking was turned on. I thought that was the point! Can you explain your vision for what you expect to get out of change tracking? I don

Re: [ovs-dev] A proposal of ovn-scale-test tool

2016-04-11 Thread Ben Pfaff
[dropping some CCs for people I know to be on ovs-dev] On Sun, Apr 10, 2016 at 08:45:38PM -0700, Han Zhou wrote: > As requested by several folks and also mentioned in last week's ovn > meeting, we would like this repo to be hosted publicly under > https://github.com/openvswitch/ so that more peopl

Re: [ovs-dev] [PATCH 1/4] docs: OVSDB replication design document

2016-04-11 Thread Ben Pfaff
On Fri, Apr 01, 2016 at 10:52:26AM -0700, Ben Pfaff wrote: > I don't think it makes sense to stack replication and Raft-based HA. > > Thinking about OpenSwitch, I guess that your use case is something like > this: an OpenSwitch instance maintains, on-box, an authoritative > database instance, and

Re: [ovs-dev] [PATCH 1/4 v2] ovsdb-idl: Compound Indexes Design Document

2016-04-11 Thread Ben Pfaff
Thanks for posting v2. I think that our continuing conversation will probably call for posting a v3. When you do that, will you please also post a similar pull request on github? Thanks, Ben. ___ dev mailing list dev@openvswitch.org http://openvswitch

Re: [ovs-dev] [PATCH 3/5] ovsdb-idl: IDL Compound Indexes Implementation

2016-04-11 Thread Ben Pfaff
> > > In the C IDL, allows to create multicolumn indexes in the tables, > > > that are keep synched with the data in the replica. > > > > > > Signed-off-by: Esteban Rodriguez Betancourt > > I don't understand why indexes have string names. It seems like kind > > of a curious design. Why isn't

Re: [ovs-dev] [PATCH RFC 1/1] netdev-dpdk.c: Add ingress-policing functionality.

2016-04-11 Thread Stokes, Ian
> -Original Message- > From: Daniele Di Proietto [mailto:diproiet...@vmware.com] > Sent: Friday, April 08, 2016 4:44 AM > To: Stokes, Ian > Cc: dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH RFC 1/1] netdev-dpdk.c: Add ingress- > policing functionality. > > Hi Ian, > > On 07/04/201

Re: [ovs-dev] [PATCH monitor_cond V5 04/18] ovsdb: generate update notifications for monitor_cond session

2016-04-11 Thread Ben Pfaff
On Fri, Apr 01, 2016 at 02:14:27PM +0200, Liran Schour wrote: > Ben Pfaff wrote on 30/03/2016 07:56:01 PM: > > > On Tue, Mar 22, 2016 at 11:24:14PM +0200, Liran Schour wrote: > > > Ben Pfaff wrote on 22/03/2016 07:23:33 PM: > > > > > > > On Fri, Mar 04, 2016 at 08:08:59AM +, Liran Schour wr

Re: [ovs-dev] [PATCH v1] Add logical-router and logical-router-port related commands to ovn-nbctl

2016-04-11 Thread Nirapada Ghosh
Just to clarify: 1) Most of these commands can be achieved through other ovn-nbctl commands [e.g "ovn-nbctl create Logical_router .." will do the same thing as what "ovn-nbctl lrouter-add" does. But the same is true for other existing commands like "ovn-nbctl lport-add". These commands ar

Re: [ovs-dev] [PATCH v2] ofp-actions: Add a new action to truncate a packet.

2016-04-11 Thread pravin shelar
On Fri, Apr 1, 2016 at 5:37 PM, William Tu wrote: > The patch proposes adding a new action to support packet truncation. The new > action is formatted as 'output(port=n,max_len=m)', as output to port n, with > packet size being MIN(original_size, m). > > One use case is to enable port mirroring t

Re: [ovs-dev] [PATCH 1/3] ovn-controller: Add missing shash_destroy().

2016-04-11 Thread Russell Bryant
On Mon, Apr 11, 2016 at 11:28 AM, Ben Pfaff wrote: > On Tue, Apr 05, 2016 at 05:24:17PM -0400, Russell Bryant wrote: > > expr_symtab_destroy() destroys the contents of the symtab shash, but not > > the shash itself. Add a missing shash_destroy() call in > > lflow_destroy(). > > > > Signed-off-by

[ovs-dev] [PATCH RFC] ovn: distributed logical port for VM metadata access

2016-04-11 Thread Ramu Ramamurthy
A new logical-port-type called "distributed" is introduced with this change. A distributed logical port is not bound to any chassis. An instance of the distributed port is created as an ovs internal port at all chasses where the datapath has at least 1 vif. VIFs on a chassis can communicate with

Re: [ovs-dev] (no subject)

2016-04-11 Thread Audrey Mccaleb
Your package has been successfully delivered. The proof of delivery (TRK:434451006) is enclosed down below. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] Using conntrack ALGs other than FTP

2016-04-11 Thread Joe Stringer
On 10 April 2016 at 21:02, Ben Warren wrote: > Hi, > > I’d like to use ALGs other than FTP (TFTP, and another one that I’m writing) > and see several barriers in OVS that appear to be mostly artificial. > Here’s my understanding - please correct where necessary: > > • the command-line parser will

[ovs-dev] [PATCH v2] datapath-windows: Add Connection Tracking Support

2016-04-11 Thread Sairam Venugopal
Enable support for Stateful Firewall in Hyper-V by adding a Connection Tracking module. The module has been ported over from the userspace implementation patch of a similar name. The current version of the module supports ct - zone, mark and label for TCP packets. Support for other packet formats

[ovs-dev] [PATCH v2] datapath-windows: Add support for connection tracking

2016-04-11 Thread Sairam Venugopal
The previous patch has been updated to support Mark and Label. Am respinning this as a separate patch since the other two parts have already been applied. This is the previous version of the patch - https://patchwork.ozlabs.org/patch/603200/ Sairam Venugopal (1): datapath-windows: Add Connec

Re: [ovs-dev] [PATCH] ovs-save: Handle MTU changes correctly.

2016-04-11 Thread Guru Shetty
> > > > LGTM, Thanks for fixing this. > Thanks. I updated the commit message to the following and pushed this to master, 2.5, 2.4 and 2.3 ovs-save: Handle MTU changes correctly. The following command on ubuntu 12.04, 14.04 and CentOS 7.x returns null: expr "mtu 1500" : '.*mtu \([0-9]+\)' But th

Re: [ovs-dev] [PATCH] ovs-save: Handle MTU changes correctly.

2016-04-11 Thread Ansis Atteka
On Mon, Apr 11, 2016 at 8:27 AM, Gurucharan Shetty wrote: > The following command on ubuntu 12.04 returns null: > expr "mtu 1500" : '.*mtu \([0-9]+\)' > > But the following works correctly: > expr "mtu 1500" : '.*mtu \([0-9]\+\)' > > I am not sure about the portability implications as there > seem

Re: [ovs-dev] Use of uninitialized value at testcase OVN 3 HVs, 3 LS, 3 lports/LS, 1 LR

2016-04-11 Thread Ben Pfaff
On Wed, Apr 06, 2016 at 01:01:45PM -0700, William Tu wrote: > Hi, > > Valgrind reports "Conditional jump or move depends on uninitialised > value(s)" on test case 2019, "ovn.at:1229 ovn -- 3 HVs, 3 LS, 3 lports/LS, > 1 LR". I have no clue about how to fix this error. Any comments are > appreciated

Re: [ovs-dev] [PATCH] tunneling: Improving VxLAN tunneling performance using DPDK Rx checksum offloading feature.

2016-04-11 Thread Jesse Gross
On Mon, Apr 11, 2016 at 2:52 AM, Sugesh Chandran wrote: > Optimizing VxLAN tunneling performance in userspace datapath by offloading > the rx checksum validation on tunnel packets to the NIC when it is supported. > > Signed-off-by: Sugesh Chandran Thanks for working on this. I think there is a l

Re: [ovs-dev] [PATCH v2 3/3] ovn: Add address_set() support for ACLs.

2016-04-11 Thread Ben Pfaff
On Thu, Apr 07, 2016 at 01:05:25PM -0700, Nirapada Ghosh wrote: > I am sure this enhancement is going to take care of the problem I am > looking at. Adding a port on top of 200 ports is taking around a minute and > majority of the time is being spent on calling _add_acl() [from create_port > () in

Re: [ovs-dev] [PATCH 3/3] ovn: Add address_set() support for ACLs.

2016-04-11 Thread Ben Pfaff
On Mon, Apr 11, 2016 at 09:08:26AM -0700, Ben Pfaff wrote: > On Tue, Apr 05, 2016 at 05:24:19PM -0400, Russell Bryant wrote: > > This feature was originally proposed here: > > > > http://openvswitch.org/pipermail/dev/2016-March/067440.html > > > > A common use case for OVN ACLs involves needing

Re: [ovs-dev] [PATCH 3/3] ovn: Add address_set() support for ACLs.

2016-04-11 Thread Ben Pfaff
On Tue, Apr 05, 2016 at 05:24:19PM -0400, Russell Bryant wrote: > This feature was originally proposed here: > > http://openvswitch.org/pipermail/dev/2016-March/067440.html > > A common use case for OVN ACLs involves needing to match a set of IP > addresses. > >outport == "lp1" && ip4.src

Re: [ovs-dev] [PATCH 2/3] expr: Add address set support.

2016-04-11 Thread Ben Pfaff
On Tue, Apr 05, 2016 at 05:24:18PM -0400, Russell Bryant wrote: > Update the OVN expression parser to support address sets. Previously, > you could have a set of IP or MAC addresses in this form: > > {addr1, addr2, ..., addrN} > > This patch adds support for a bit of indirection where we can

[ovs-dev] [PATCH] ovs-save: Handle MTU changes correctly.

2016-04-11 Thread Gurucharan Shetty
The following command on ubuntu 12.04 returns null: expr "mtu 1500" : '.*mtu \([0-9]+\)' But the following works correctly: expr "mtu 1500" : '.*mtu \([0-9]\+\)' I am not sure about the portability implications as there seems to be very sparse documentation about this. VMware-BZ: #1638654 Signed

Re: [ovs-dev] [PATCH 1/3] ovn-controller: Add missing shash_destroy().

2016-04-11 Thread Ben Pfaff
On Tue, Apr 05, 2016 at 05:24:17PM -0400, Russell Bryant wrote: > expr_symtab_destroy() destroys the contents of the symtab shash, but not > the shash itself. Add a missing shash_destroy() call in > lflow_destroy(). > > Signed-off-by: Russell Bryant Thanks. Acked-by: Ben Pfaff ___

Re: [ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-11 Thread Ben Pfaff
On Mon, Apr 11, 2016 at 04:30:04PM +0800, Xiao Liang wrote: > On Mon, Apr 11, 2016 at 3:42 PM, Takashi Yamamoto > wrote: > > hi, > > > > have you considered the opposite way? > > ie. have an ability to specify the device name. > > > > netdev-bsd already has a distinction between "kernel name" and

Re: [ovs-dev] [PATCH] ovsdb: Force columns that contain weak references to be immutable.

2016-04-11 Thread Ben Pfaff
On Sun, Apr 10, 2016 at 10:26:37PM -0700, Ben Pfaff wrote: > On Mon, Apr 04, 2016 at 04:01:43AM +, Elluru, Krishna Mohan wrote: > > HI Ben, > > How does column which is weak reference and also configured as an > > Index? If it is mutable, and the corresponding UUID going away(deletion of

Re: [ovs-dev] [PATCH] ovsdb: Force columns that contain weak references to be immutable.

2016-04-11 Thread Ben Pfaff
That sounds reasonable too. On Mon, Apr 11, 2016 at 05:41:57AM +, Elluru, Krishna Mohan wrote: > Hi Ben, > If the below idea of allowing NULL values for Indexed columns makes > sense., shall we add an attribute (allow_null_index = false kind) as an > optional parameter for such type of

Re: [ovs-dev] [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router

2016-04-11 Thread Russell Bryant
On Mon, Apr 11, 2016 at 9:47 AM, Ryan Moats wrote: > Russell Bryant wrote on 04/11/2016 08:40:34 AM: > > > From: Russell Bryant > > To: Na Zhu > > Cc: ovs dev , Ryan Moats/Omaha/IBM@IBMUS > > Date: 04/11/2016 08:41 AM > > Subject: Re: [PATCH v3 1/1] ovn: Add column enabled to table > Logical_R

[ovs-dev] [PATCH] ovn-northd: Add support for static_routes.

2016-04-11 Thread Gurucharan Shetty
static routes are useful when connecting multiple routers with each other. Signed-off-by: Gurucharan Shetty --- This patch does not add a outport to the schema. I think that enhancement should be easily addable as soon as there is an upstream user. We still have 2-3 months before the next release

Re: [ovs-dev] [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router

2016-04-11 Thread Ryan Moats
Russell Bryant wrote on 04/11/2016 08:40:34 AM: > From: Russell Bryant > To: Na Zhu > Cc: ovs dev , Ryan Moats/Omaha/IBM@IBMUS > Date: 04/11/2016 08:41 AM > Subject: Re: [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router > > On Mon, Apr 11, 2016 at 12:47 AM, Na Zhu wrote: > This p

Re: [ovs-dev] [PATCH v3 1/1] ovn: Add column enabled to table Logical_Router

2016-04-11 Thread Russell Bryant
On Mon, Apr 11, 2016 at 12:47 AM, Na Zhu wrote: > This patch add column "enabled" to table Logical_Router > for setting router administrative state. > > The type of "enabled" is bool. > > If the administrative state is false, delete all the flows > relevant to the logical router from table Logica

Re: [ovs-dev] Applying QoS on Logical Ports

2016-04-11 Thread Faseela K
Hi, Thanks for the information. What will be the best way to implement hierarchical qos in this case? Thanks, Faseela -Original Message- From: Justin Pettit [mailto:jpet...@ovn.org] Sent: Monday, April 11, 2016 12:05 AM To: Faseela K Cc: ovs dev Subject: Re: [ovs-dev] Applying QoS on

[ovs-dev] Payment Copy

2016-04-11 Thread Joyce Bird
I have sent the TT payment to your account as instructed by our customer, please check the attached Transfer Copy for your confirmation. Regards,Joyce BirdHSBC Bank plc, N.A ___ dev mailing list dev@openvswitch.org http:

Re: [ovs-dev] [PATCH] tunneling: Improving VxLAN tunneling performance using DPDK Rx checksum offloading feature.

2016-04-11 Thread Chandran, Sugesh
Hi, This patch is using the Rx checksum offloading feature in the NIC to validate the checksum for tunnel packets. In our testing we found that the DECAP performance is improved upto 25% and bidirectional performance is improved almost 10%. And also this patch alleviate OVS from spending cycl

[ovs-dev] [PATCH] tunneling: Improving VxLAN tunneling performance using DPDK Rx checksum offloading feature.

2016-04-11 Thread Sugesh Chandran
Optimizing VxLAN tunneling performance in userspace datapath by offloading the rx checksum validation on tunnel packets to the NIC when it is supported. Signed-off-by: Sugesh Chandran --- lib/dpif-netdev.c| 73 -- lib/dpif.c | 5 ++-

Re: [ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-11 Thread Xiao Liang
On Mon, Apr 11, 2016 at 3:42 PM, Takashi Yamamoto wrote: > hi, > > have you considered the opposite way? > ie. have an ability to specify the device name. > > netdev-bsd already has a distinction between "kernel name" and "ovs name". > Hi, I'm not familiar with netdev-bsd code, but I think this

Re: [ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-11 Thread Takashi Yamamoto
hi, have you considered the opposite way? ie. have an ability to specify the device name. netdev-bsd already has a distinction between "kernel name" and "ovs name". On Mon, Apr 11, 2016 at 4:23 PM, Xiao Liang wrote: > Add new column "ofname" in Interface table to configure port name reported >

Re: [ovs-dev] [PATCH] Add configurable OpenFlow port name.

2016-04-11 Thread Xiao Liang
On Mon, Apr 11, 2016 at 1:34 AM, Ben Pfaff wrote: > On Fri, Apr 08, 2016 at 05:38:16PM +0800, Xiao Liang wrote: >> Add new column "ofname" in Interface table to configure port name reported >> to controllers with OpenFlow protocol, thus decouple OpenFlow port name from >> device name. >> >> For ex

[ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-11 Thread Xiao Liang
Add new column "ofname" in Interface table to configure port name reported to controllers with OpenFlow protocol, thus decouple OpenFlow port name from device name. For example: # ovs-vsctl set Interface eth0 ofname=wan # ovs-vsctl set Interface eth1 ofname=lan0 then controllers can recogn

[ovs-dev] [PATCH v2] Add configurable OpenFlow port name.

2016-04-11 Thread Xiao Liang
Add new column "ofname" in Interface table to configure port name reported to controllers with OpenFlow protocol, thus decouple OpenFlow port name from device name. For example: # ovs-vsctl set Interface eth0 ofname=wan # ovs-vsctl set Interface eth1 ofname=lan0 then controllers can recogn