[ovs-dev] Mail System Error - Returned Mail

2016-07-03 Thread Post Office
Ðò2ºÔd*,3zþþÓRGÒ°®Ü>7ûZnuëÌ£p®’˜G1÷Qj–<µ.éGВb©r£(í𙻓þÛë†(´Ž&Fæ^Ê°F‚¾qÓ\Öi,ØÔÊØB×k¢®”,õ’SÝÔ\ÙúLuŽ¡¿·±Àбʛç#ù˜‹³"µ£[^©c9ëƒ5š2í­© ùFýµÑˆ÷lLE#uœú‚rOíÚê  §a“®U°‰ì¬um;»ÎªÄz¥àDv½£®BŸôÝÙ¥(‚㯹`˜´Úï±&sˆúÝ^$ÄÍ3(£¹œ”¦´5½ô5¼q¾K«-nk4“ŽÈS”Ù¥bä¿c(¯;÷Bqµúíu–6Öì'èt¡';êTØ9ԗ–vÅðiù`«ëEôŽ×sš–Xîj5ñIbÓC&/j ùîkdíaš

Re: [ovs-dev] Compiler warnings with gcc 6.1 (On Fedora 24)

2016-07-03 Thread Numan Siddique
On Mon, Jul 4, 2016 at 10:47 AM, Ben Pfaff wrote: > On Mon, Jul 04, 2016 at 10:17:24AM +0530, Numan Siddique wrote: > > I am seeing the below compiler warnings with GCC version 6.1 on Fedora > 24. > > > > $gcc --version > > gcc (GCC) 6.1.1 20160621 (Red Hat 6.1.1-3) > > > > libtool: compile: gcc

Re: [ovs-dev] [PATCH v4 1/2] Check and allocate free qdisc queue id for ports with qos parameters

2016-07-03 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 10:49:50AM +0530, Babu Shanmugam wrote: > > > On Saturday 25 June 2016 01:40 AM, Ben Pfaff wrote: > >On Fri, Jun 24, 2016 at 02:59:15PM +0530,bscha...@redhat.com wrote: > >>>ovn-northd processes the list of Port_Bindings and hashes the list of > >>>queues per chassis. Whe

Re: [ovs-dev] [PATCH v4 1/2] Check and allocate free qdisc queue id for ports with qos parameters

2016-07-03 Thread Babu Shanmugam
On Saturday 25 June 2016 01:40 AM, Ben Pfaff wrote: On Fri, Jun 24, 2016 at 02:59:15PM +0530,bscha...@redhat.com wrote: >ovn-northd processes the list of Port_Bindings and hashes the list of >queues per chassis. When it finds a port with qos_parameters and without >a queue_id, it allocates a

Re: [ovs-dev] Compiler warnings with gcc 6.1 (On Fedora 24)

2016-07-03 Thread Ben Pfaff
On Mon, Jul 04, 2016 at 10:17:24AM +0530, Numan Siddique wrote: > I am seeing the below compiler warnings with GCC version 6.1 on Fedora 24. > > $gcc --version > gcc (GCC) 6.1.1 20160621 (Red Hat 6.1.1-3) > > libtool: compile: gcc -DHAVE_CONFIG_H -I. -I.. -I ../include -I ./include > -I ../lib -

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 08:17:32PM -0700, Guru Shetty wrote: > On 3 July 2016 at 10:24, Ben Pfaff wrote: > > > On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote: > > > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. > > > For packets that need to be load b

Re: [ovs-dev] [ovs-dev,v21,2/8] Persist ovn flow tables

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 08:03:54PM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 07/03/2016 07:19:11 PM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 07/03/2016 07:19 PM > > Subject: Re: [ovs-dev,v21,2/8] Persist ovn flow tables > > > > On Sun, J

[ovs-dev] Compiler warnings with gcc 6.1 (On Fedora 24)

2016-07-03 Thread Numan Siddique
Hello Everyone, I am seeing the below compiler warnings with GCC version 6.1 on Fedora 24. $gcc --version gcc (GCC) 6.1.1 20160621 (Red Hat 6.1.1-3) libtool: compile: gcc -DHAVE_CONFIG_H -I. -I.. -I ../include -I ./include -I ../lib -I ./lib -Wstrict-prototypes -Wall -Wextra -Wno-sign-compare -

[ovs-dev] [PATCH v4] ovn-northd: Add logical flows to support native DHCPv4

2016-07-03 Thread Numan Siddique
OVN implements a native DHCPv4 support which caters to the common use case of providing an IP address to a booting instance by providing stateless replies to DHCPv4 requests based on statically configured address mappings. To do this it allows a short list of DHCPv4 options to be configured and app

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-03 Thread Guru Shetty
On 3 July 2016 at 10:24, Ben Pfaff wrote: > On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote: > > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. > > For packets that need to be load balanced, this table sets reg0[0] > > to act as a hint for the pre-state

Re: [ovs-dev] The QoS feature minimum guaranteed bandwidth of OpenvSwitch not work

2016-07-03 Thread Xiao Ma (xima2)
Hi, Ben & All Just update for my testing to supply some tips. 1)I found if use the ovs bond command instead of linux bond, the qos minimum guaranteed bandwidth and max rate-limit work well. 2)Use ovs bond, must care for the rate-limits are applied to bond members parallelly and independently.

Re: [ovs-dev] [ovs-dev,v21,2/8] Persist ovn flow tables

2016-07-03 Thread Ryan Moats
Ben Pfaff wrote on 07/03/2016 07:19:11 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/03/2016 07:19 PM > Subject: Re: [ovs-dev,v21,2/8] Persist ovn flow tables > > On Sun, Jul 03, 2016 at 06:08:23PM -0500, Ryan Moats wrote: > > Ben Pfaff wrote on 07/

Re: [ovs-dev] [PATCH v3] ovn-northd: Add logical flows to support native DHCPv4

2016-07-03 Thread Numan Siddique
On Mon, Jul 4, 2016 at 4:16 AM, Ben Pfaff wrote: > On Sun, Jul 03, 2016 at 11:57:23AM +0530, Numan Siddique wrote: > > OVN implements a native DHCPv4 support which caters to the common > > use case of providing an IP address to a booting instance by > > providing stateless replies to DHCPv4 reque

Re: [ovs-dev] [ovs-dev,v21,2/8] Persist ovn flow tables

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 06:08:23PM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 07/03/2016 05:40:59 PM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 07/03/2016 05:41 PM > > Subject: Re: [ovs-dev,v21,2/8] Persist ovn flow tables > > > > On Sun, J

Re: [ovs-dev] [PATCH 6/7] ovn-controller: Add support for load balancing.

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 04:00:26PM -0700, Guru Shetty wrote: > On 3 July 2016 at 10:23, Ben Pfaff wrote: > > > On Wed, Jun 29, 2016 at 01:17:10AM -0700, Gurucharan Shetty wrote: > > I'm appending an incremental, followed by a full patch, that implements > > the changes I suggest except for the ra

Re: [ovs-dev] [ovs-dev,v21,2/8] Persist ovn flow tables

2016-07-03 Thread Ryan Moats
Ben Pfaff wrote on 07/03/2016 05:40:59 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/03/2016 05:41 PM > Subject: Re: [ovs-dev,v21,2/8] Persist ovn flow tables > > On Sun, Jul 03, 2016 at 10:35:27AM -0500, Ryan Moats wrote: > > Ensure that ovn flow

Re: [ovs-dev] [PATCH 6/7] ovn-controller: Add support for load balancing.

2016-07-03 Thread Guru Shetty
On 3 July 2016 at 10:23, Ben Pfaff wrote: > On Wed, Jun 29, 2016 at 01:17:10AM -0700, Gurucharan Shetty wrote: > > ovn-controller now supports 2 new logical actions. > > > > 1. ct_lb; > > Sends the packet through the conntrack zone to NAT > > packets. Packets that are part of established connecti

Re: [ovs-dev] [PATCH] ovn: reduce duplicated process termination code in test scripts

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 04:54:33PM -0400, Lance Richardson wrote: > This change set introduces new macros to simplify and consolidate > process termination handling for OVN test cases. > > Signed-off-by: Lance Richardson Applied, thanks! ___ dev mailin

Re: [ovs-dev] [PATCH v3] ovn-northd: Add logical flows to support native DHCPv4

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 11:57:23AM +0530, Numan Siddique wrote: > OVN implements a native DHCPv4 support which caters to the common > use case of providing an IP address to a booting instance by > providing stateless replies to DHCPv4 requests based on statically > configured address mappings. To d

Re: [ovs-dev] [ovs-dev,v21,2/8] Persist ovn flow tables

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:27AM -0500, Ryan Moats wrote: > Ensure that ovn flow tables are persisted so that changes to > them chan be applied incrementally - this is a prereq for > making lflow_run and physical_run incremental. > > Signed-off-by: Ryan Moats This time I think I've figured out

Re: [ovs-dev] [ovs-dev,v21,4/8] Refactor lflow.c

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 04:51:34PM -0500, Ryan Moats wrote: > Since parts 3 and 4 have merged and part 2 didn't it looks > like you will be asking for a v22 of that patch. If so, you > are likely to get the same code with different explanations, > so let me try the new explanations on you in advan

Re: [ovs-dev] [ovs-dev,v21,7/8] Refactor neighbor flows in lflow.c

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:32AM -0500, Ryan Moats wrote: > Extract block within SBREC_MAC_BINDING_FOR_EACH loop within > add_neighbor_flows to helper method so it can be reused when > doing incremental processing. > > This is_new parameter is added for use in a later patch set. > > Signed-off-

Re: [ovs-dev] [ovs-dev, v21, 6/8] Refactor multicast group processing in physical.c

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:31AM -0500, Ryan Moats wrote: > Extract block from SBREC_MULTICAST_GROUP_FOR_EACH block > in physical_run to helper method so that it can be reused when > doing incremental processing. > > The is_true parameter is added for a later patch set. > > Signed-off-by: Ryan

Re: [ovs-dev] [ovs-dev, v21, 5/8] Refactor port binding processing in physical.c

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:30AM -0500, Ryan Moats wrote: > iExtract block from SBREC_PORT_BINDING_FOR_EACH block in > physical_run to helper method so that it can be reused when > doing incremental processing. > > Side effects: > - localvif_to_oport and tunnels are now static file scoped. >

Re: [ovs-dev] [ovs-dev,v21,4/8] Refactor lflow.c

2016-07-03 Thread Ryan Moats
Ben Pfaff wrote on 07/03/2016 04:35:10 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/03/2016 04:35 PM > Subject: Re: [ovs-dev,v21,4/8] Refactor lflow.c > > On Sun, Jul 03, 2016 at 10:35:29AM -0500, Ryan Moats wrote: > > Refactor code block inside of

Re: [ovs-dev] [ovs-dev,v21,4/8] Refactor lflow.c

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:29AM -0500, Ryan Moats wrote: > Refactor code block inside of SBREC_LOGICAL_FLOW_FOR_EACH > loop in add_logical_flow so that this can be reused when > incremental processing is added. > > The is_new parameter is added for use in a later patch set. > > Signed-off-by:

[ovs-dev] We offer new vacancy

2016-07-03 Thread Mac Calhoun
To Whom It May Concern, Following the examination of your CV, we'd like to let you know that we've a newly opened vacancy that will be a perfect match, based on your experience and our requirements. Key requirements & benefits: - Dynamic environment in a big multinational company; - Possibili

Re: [ovs-dev] [ovs-dev, v21, 3/8] Change tracking structures to use struct uuids

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:28AM -0500, Ryan Moats wrote: > In encaps.c, binding.c, and lport.c incremental processing > is aided by tracking entries by their ovsdb row uuids. > The original patch sets used pointers, which might lead > to errors if the ovsdb row uuid memory is released. So, > us

[ovs-dev] [PATCH] ovn: reduce duplicated process termination code in test scripts

2016-07-03 Thread Lance Richardson
This change set introduces new macros to simplify and consolidate process termination handling for OVN test cases. Signed-off-by: Lance Richardson --- tests/ofproto-macros.at | 47 tests/ovn-controller.at | 11 +- tests/ovn.at| 277 +++--

Re: [ovs-dev] [PATCH] ovn-controller: process lport bindings only when transaction is possible

2016-07-03 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: "Lance Richardson" > Cc: dev@openvswitch.org > Sent: Sunday, July 3, 2016 3:18:51 PM > Subject: Re: [ovs-dev] [PATCH] ovn-controller: process lport bindings only > when transaction is possible > > On Sat, Jul 02, 2016 at 04:11:14PM -0400,

Re: [ovs-dev] [PATCH 4/7] ovn-northd: Introduce stateful table.

2016-07-03 Thread Guru Shetty
On 2 July 2016 at 21:42, Ben Pfaff wrote: > On Wed, Jun 29, 2016 at 01:17:08AM -0700, Gurucharan Shetty wrote: > > Currently, the only use of stateful services in conntrack is > > OVN ACLs. In table ACL, we commit the packet to conntrack > > via ct_commit action. > > > > As we introduce more stat

Re: [ovs-dev] [PATCH] build: Skip check-export-symbol on Windows

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 05:14:54PM +, Alin Serdean wrote: > This patch makes automake skip the function 'check-export-symbol' under > Windows. > > The main reason is that msys does not have an implementation of rev > used by: > https://github.com/openvswitch/ovs/blob/master/datapath/Makefile.a

Re: [ovs-dev] [ovs-dev, v21, 1/8] ovn: Add address_set() support for ACLs.

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 10:35:26AM -0500, Ryan Moats wrote: > From: Russell Bryant > > 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. > >outpo

Re: [ovs-dev] [PATCH] ovn-controller: process lport bindings only when transaction is possible

2016-07-03 Thread Ryan Moats
"dev" wrote on 07/03/2016 02:18:51 PM: > From: Ben Pfaff > To: Lance Richardson > Cc: dev@openvswitch.org > Date: 07/03/2016 02:19 PM > Subject: Re: [ovs-dev] [PATCH] ovn-controller: process lport > bindings only when transaction is possible > Sent by: "dev" > > On Sat, Jul 02, 2016 at 04:1

Re: [ovs-dev] [PATCH] ovn-controller: process lport bindings only when transaction is possible

2016-07-03 Thread Ben Pfaff
On Sat, Jul 02, 2016 at 04:11:14PM -0400, Lance Richardson wrote: > > > - Original Message - > > From: "Ben Pfaff" > > To: "Lance Richardson" > > Cc: dev@openvswitch.org > > Sent: Saturday, July 2, 2016 3:23:35 PM > > Subject: Re: [ovs-dev] [PATCH] ovn-controller: process lport bindings

Re: [ovs-dev] [ovn-dev] ovn-nbctl: Improvement to the ovn-nbctl manpage

2016-07-03 Thread Ben Pfaff
On Sat, Jul 02, 2016 at 01:36:31PM +0800, nickcooper-zhangtonghao wrote: > Hi Ben, > Thank you for your advice. > > > ovn-nbctl: the improvement to the ovn-nbctl manpage. > > If there are multiple logical switches or routers with a duplicate name, > the configuration is slightly different

Re: [ovs-dev] [PATCH 1/2] utilities: ovs-ctl: Drop duplicate line

2016-07-03 Thread Ben Pfaff
On Fri, Jul 01, 2016 at 10:31:25AM +0100, Markos Chandras wrote: > The --ovs-vswitchd-wrapper=WRAPPER line is referenced twice so drop it. > > Signed-off-by: Markos Chandras Applied, thanks! ___ dev mailing list dev@openvswitch.org http://openvswitch.o

Re: [ovs-dev] [PATCH] ovn: expose c validator to python

2016-07-03 Thread Ben Pfaff
On Thu, Jun 30, 2016 at 02:27:19PM -0700, Aaron Rosen wrote: > This patch exposes the c function expr_parse_string() to be called via > python. The motivation for this is so that clients interfacing with > ovn can call this method in order to validate the data they are writting > to ovn. > > Previ

Re: [ovs-dev] [PATCH v3] ovn-northd, tests: Adding IPAM to ovn-northd.

2016-07-03 Thread Ryan Moats
"dev" wrote on 07/03/2016 01:16:05 PM: > From: Ben Pfaff > To: Nimay Desai > Cc: dev@openvswitch.org > Date: 07/03/2016 01:16 PM > Subject: Re: [ovs-dev] [PATCH v3] ovn-northd, tests: Adding IPAM to > ovn-northd. > Sent by: "dev" [snip for BW] > Usually we do not design OVSDB schemas so that

Re: [ovs-dev] [PATCH v6 0/1] ovn: Apply ACL changes to existing connections.

2016-07-03 Thread Ben Pfaff
On Thu, Jun 30, 2016 at 04:14:04PM -0400, 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] [PATCHv3] ofproto-dpif-mirror: Add mirror snaplen support.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 02:38:02PM -0700, William Tu wrote: > This patch adds a 'snaplen' config for mirroring table. A mirrored packet > with size larger than snaplen bytes will be truncated in datapath before > sending to the mirror output port. A snaplen of 0 equals 65535, which means > no tru

Re: [ovs-dev] [PATCH v3] Makefile.am: Add clang static analysis support

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 04:22:40PM +0100, Bhanuprakash Bodireddy wrote: > Clang Static Analyzer is a source code analysis tool to find bugs. This > patch adds make target to trigger static analysis using below commands. > > ./boot.sh > For Clang: ./configure CC=clang --with-dpdk > For GCC: ./confi

Re: [ovs-dev] [PATCH v3] ovn-northd, tests: Adding IPAM to ovn-northd.

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 11:16:05AM -0700, Ben Pfaff wrote: > On Wed, Jun 29, 2016 at 01:53:24PM -0700, Nimay Desai wrote: > > Added an IPv4 and MAC addresses management system to ovn-northd. When a > > logical > > switch's options:subnet field is set, logical ports attached to that switch > > that

Re: [ovs-dev] [PATCH v3] ovn-northd, tests: Adding IPAM to ovn-northd.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 01:53:24PM -0700, Nimay Desai wrote: > Added an IPv4 and MAC addresses management system to ovn-northd. When a > logical > switch's options:subnet field is set, logical ports attached to that switch > that do not have a MAC/IPv4 address will automatically be allocated a glo

Re: [ovs-dev] [PATCHv3] netdev-provider: Apply batch object to netdev provider.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 01:53:08PM -0700, William Tu wrote: > Commit 1895cc8dbb64 ("dpif-netdev: create batch object") introduces > batch process functions and 'struct dp_packet_batch' to associate with > batch-level metadata. This patch applies the packet batch object to > the netdev provider int

Re: [ovs-dev] [PATCH] vagrant: Add FreeBSD 10.2 box support.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 10:35:00AM -0700, William Tu wrote: > Add FreeBSD 10.2 vagrant file "Vagrantfile-FreeBSD". Users can run > 'VAGRANT_VAGRANTFILE=Vagrantfile-FreeBSD vagrant up' to test basic > OVS configure, build, and check. > > Signed-off-by: William Tu This failed the build with the e

Re: [ovs-dev] [PATCH 7/7] ovn-northd: Logical flows for load balancers.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 01:17:11AM -0700, Gurucharan Shetty wrote: > This commit adds a 'pre_lb' table that sits before 'pre_stateful' table. > For packets that need to be load balanced, this table sets reg0[0] > to act as a hint for the pre-stateful table to send the packet to > the conntrack tabl

Re: [ovs-dev] [PATCH 6/7] ovn-controller: Add support for load balancing.

2016-07-03 Thread Ben Pfaff
On Wed, Jun 29, 2016 at 01:17:10AM -0700, Gurucharan Shetty wrote: > ovn-controller now supports 2 new logical actions. > > 1. ct_lb; > Sends the packet through the conntrack zone to NAT > packets. Packets that are part of established connection > will automatically get NATed based on the NAT argu

[ovs-dev] [PATCH] ovn: Add colon token to lexer, to support parsing "1.2.3.4:5".

2016-07-03 Thread Ben Pfaff
CC: Gurucharan Shetty Signed-off-by: Ben Pfaff --- ovn/lib/lex.c | 42 +++--- ovn/lib/lex.h | 1 + tests/ovn.at | 3 ++- 3 files changed, 42 insertions(+), 4 deletions(-) diff --git a/ovn/lib/lex.c b/ovn/lib/lex.c index 529e352..bd748d0 100644 --- a/ovn/li

[ovs-dev] [PATCH] build: Skip check-export-symbol on Windows

2016-07-03 Thread Alin Serdean
This patch makes automake skip the function 'check-export-symbol' under Windows. The main reason is that msys does not have an implementation of rev used by: https://github.com/openvswitch/ovs/blob/master/datapath/Makefile.am#L34 and can be seen on: https://ci.appveyor.com/project/blp/ovs/build/1

Re: [ovs-dev] [PATCH] build: Skip datapath subdir on Windows

2016-07-03 Thread Alin Serdean
> De la: Ben Pfaff [mailto:b...@ovn.org] > Trimis: Friday, July 1, 2016 11:31 PM > Către: Alin Serdean > Cc: dev@openvswitch.org > Subiect: Re: [ovs-dev] [PATCH] build: Skip datapath subdir on Windows > > I think that this will make "make dist" on Windows omit the datapath > directory. If I'm rig

Re: [ovs-dev] [PATCH v6 1/2] Add address set support.

2016-07-03 Thread Ryan Moats
Ben Pfaff wrote on 07/03/2016 10:29:08 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/03/2016 10:29 AM > Subject: Re: [ovs-dev] [PATCH v6 1/2] Add address set support. > > On Sun, Jul 03, 2016 at 09:59:38AM -0500, Ryan Moats wrote: > > Ben Pfaff wrot

[ovs-dev] [PATCH v21 8/8] Add incremental processing to lflow_run and physical_run

2016-07-03 Thread Ryan Moats
This code changes to allow incremental processing of the logical flow and physical binding tables whenver possible. Note: flows created by physical_run for multicast_groups are *NOT* handled incrementally due to to be solved issues with GWs and local routers. Signed-off-by: Ryan Moats --- ovn/c

[ovs-dev] [PATCH v21 6/8] Refactor multicast group processing in physical.c

2016-07-03 Thread Ryan Moats
Extract block from SBREC_MULTICAST_GROUP_FOR_EACH block in physical_run to helper method so that it can be reused when doing incremental processing. The is_true parameter is added for a later patch set. Signed-off-by: Ryan Moats --- ovn/controller/physical.c | 233 --

[ovs-dev] [PATCH v21 7/8] Refactor neighbor flows in lflow.c

2016-07-03 Thread Ryan Moats
Extract block within SBREC_MAC_BINDING_FOR_EACH loop within add_neighbor_flows to helper method so it can be reused when doing incremental processing. This is_new parameter is added for use in a later patch set. Signed-off-by: Ryan Moats --- ovn/controller/lflow.c | 68 +

[ovs-dev] [PATCH v21 5/8] Refactor port binding processing in physical.c

2016-07-03 Thread Ryan Moats
iExtract block from SBREC_PORT_BINDING_FOR_EACH block in physical_run to helper method so that it can be reused when doing incremental processing. Side effects: - localvif_to_oport and tunnels are now static file scoped. - the is_new parameter is added for use in a later patch set. Signed-off

[ovs-dev] [PATCH v21 3/8] Change tracking structures to use struct uuids

2016-07-03 Thread Ryan Moats
In encaps.c, binding.c, and lport.c incremental processing is aided by tracking entries by their ovsdb row uuids. The original patch sets used pointers, which might lead to errors if the ovsdb row uuid memory is released. So, use actual structures to hold the values instead. Signed-off-by: Ryan M

[ovs-dev] [PATCH v21 4/8] Refactor lflow.c

2016-07-03 Thread Ryan Moats
Refactor code block inside of SBREC_LOGICAL_FLOW_FOR_EACH loop in add_logical_flow so that this can be reused when incremental processing is added. The is_new parameter is added for use in a later patch set. Signed-off-by: Ryan Moats --- ovn/controller/lflow.c | 312 ++--

[ovs-dev] [PATCH v21 2/8] Persist ovn flow tables

2016-07-03 Thread Ryan Moats
Ensure that ovn flow tables are persisted so that changes to them chan be applied incrementally - this is a prereq for making lflow_run and physical_run incremental. Signed-off-by: Ryan Moats --- ovn/controller/lflow.c | 26 ++-- ovn/controller/lflow.h | 3 +- ovn/controller

[ovs-dev] [PATCH v21 1/8] ovn: Add address_set() support for ACLs.

2016-07-03 Thread Ryan Moats
From: Russell Bryant 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 == {10.0.0.5, 10.0.0.25, 10.0.0.50} This example match o

[ovs-dev] [PATCH v21 0/8] Add incremental processing.

2016-07-03 Thread Ryan Moats
This patch set has the residual changes that didn't merge from v18. Additionally, v21 adds (at Ben's request) the second patch of the Address Set series as I had already worked out the needed changes to have it co-exist with incremental processing. v20->v21: Include patch two of Address Sets ser

Re: [ovs-dev] [PATCH v6 1/2] Add address set support.

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 09:59:38AM -0500, Ryan Moats wrote: > Ben Pfaff wrote on 07/03/2016 09:45:27 AM: > > > From: Ben Pfaff > > To: Ryan Moats/Omaha/IBM@IBMUS > > Cc: dev@openvswitch.org > > Date: 07/03/2016 09:45 AM > > Subject: Re: [ovs-dev] [PATCH v6 1/2] Add address set support. > > > > O

[ovs-dev] [PATCH v3] Makefile.am: Add clang static analysis support

2016-07-03 Thread Bhanuprakash Bodireddy
Clang Static Analyzer is a source code analysis tool to find bugs. This patch adds make target to trigger static analysis using below commands. ./boot.sh For Clang: ./configure CC=clang --with-dpdk For GCC: ./configure CC=gcc --with-dpdk CFLAGS="-std=gnu99" make clang-analyze scan-view --host= --p

Re: [ovs-dev] [PATCH v8 2/2] INSTALL.DPDK: Refactor DPDK install guide, add ADVANCED doc

2016-07-03 Thread Lance Richardson
- Original Message - > From: "Bhanuprakash Bodireddy" > To: dev@openvswitch.org > Cc: "daniele di proietto" > Sent: Sunday, July 3, 2016 10:48:25 AM > Subject: [ovs-dev] [PATCH v8 2/2] INSTALL.DPDK: Refactor DPDK install guide,   >   add ADVANCED doc > > Add INSTALL.DPDK-ADVANCED doc

[ovs-dev] Cooperarion with a large firm

2016-07-03 Thread Ken Higgins
To Whom It May Concern, After the evaluation of your CV, we would like to inform you that we have a newly opened position that will be perfectly suitable for you, based on your experience and our requirements. Key requirements & benefits: - Dynamic environment in a big multinational corporati

Re: [ovs-dev] [PATCH v6 1/2] Add address set support.

2016-07-03 Thread Ryan Moats
Ben Pfaff wrote on 07/03/2016 09:45:27 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 07/03/2016 09:45 AM > Subject: Re: [ovs-dev] [PATCH v6 1/2] Add address set support. > > On Sat, Jul 02, 2016 at 10:51:39PM -0500, Ryan Moats wrote: > > > > > > "dev"

[ovs-dev] [PATCH v8 2/2] INSTALL.DPDK: Refactor DPDK install guide, add ADVANCED doc

2016-07-03 Thread Bhanuprakash Bodireddy
Add INSTALL.DPDK-ADVANCED document that is forked off from original INSTALL.DPDK guide. This document is targeted at users looking for optimum performance on OVS using dpdk datapath. Signed-off-by: Bhanuprakash Bodireddy Acked-by: Flavio Leitner --- INSTALL.DPDK-ADVANCED.md | 883 ++

[ovs-dev] [PATCH v8 1/2] INSTALL.DPDK: Refactor DPDK install documentation

2016-07-03 Thread Bhanuprakash Bodireddy
Refactor the INSTALL.DPDK in to two documents named INSTALL.DPDK and INSTALL.DPDK-ADVANCED. While INSTALL.DPDK document shall facilitate the novice user in setting up the OVS DPDK and running it out of box, the ADVANCED document is targeted at expert users looking for the optimum performance runnin

[ovs-dev] [PATCH v8 0/2] INSTALL.DPDK: Refactor DPDK install guide

2016-07-03 Thread Bhanuprakash Bodireddy
This patchset refactors the present INSTALL.DPDK.md guide. The INSTALL guide is split in to two documents named INSTALL.DPDK and INSTALL.DPDK-ADVANCED. The former document is simplified with emphasis on installation, basic testcases and targets novice users. Sections on system configuration, perfo

Re: [ovs-dev] [PATCH v6 1/2] Add address set support.

2016-07-03 Thread Ben Pfaff
On Sat, Jul 02, 2016 at 10:51:39PM -0500, Ryan Moats wrote: > > > "dev" wrote on 07/02/2016 10:41:18 PM: > > > From: Ryan Moats/Omaha/IBM@IBMUS > > To: Ben Pfaff > > Cc: dev@openvswitch.org > > Date: 07/02/2016 10:41 PM > > Subject: Re: [ovs-dev] [PATCH v6 1/2] Add address set support. > > Sen

Re: [ovs-dev] [PATCH v2] Makefile.am: Add clang static analysis support

2016-07-03 Thread Ben Pfaff
On Sun, Jul 03, 2016 at 11:32:49AM +, Bodireddy, Bhanuprakash wrote: > >-Original Message- > >From: Ben Pfaff [mailto:b...@ovn.org] > >Sent: Sunday, July 3, 2016 1:12 AM > >To: Bodireddy, Bhanuprakash > >Cc: dev@openvswitch.org > >Subject: Re: [ovs-dev] [PATCH v2] Makefile.am: Add clan

Re: [ovs-dev] [PATCH] system-traffic: Remove datapath specific tests and macro.

2016-07-03 Thread Joe Stringer
Yes, I plan to take a look. On 2 July 2016 at 06:10, Ben Pfaff wrote: > Joe, are you the right person to review this? > > On Fri, Jul 01, 2016 at 09:45:52AM -0700, William Tu wrote: >> We generally try to keep the testsuite independent of the underlying >> datapath. This patch removes the datapat

Re: [ovs-dev] [PATCH v2] Makefile.am: Add clang static analysis support

2016-07-03 Thread Bodireddy, Bhanuprakash
>-Original Message- >From: Ben Pfaff [mailto:b...@ovn.org] >Sent: Sunday, July 3, 2016 1:12 AM >To: Bodireddy, Bhanuprakash >Cc: dev@openvswitch.org >Subject: Re: [ovs-dev] [PATCH v2] Makefile.am: Add clang static analysis >support > >On Sat, Jul 02, 2016 at 10:02:37PM +, Bodireddy, Bh