Re: [ovs-dev] [PATCH RFC v3] ovn-northd: add default_dhcpvx_options for Logical_Switch

2016-09-19 Thread Zong Kai Li
On Thu, Sep 15, 2016 at 7:48 AM, Ben Pfaff wrote: > On Thu, Sep 01, 2016 at 09:56:30AM +, Zongkai LI wrote: >> This patch adds default_dhcpv4_options and default_dhcpv6_options columns for >> Logical_Switch, which should help CMS not to calculate and set dhcpv4_options >> and dhcpv6_options co

Re: [ovs-dev] Setting pmd-cpu-mask in cluster-on-die mode

2016-09-19 Thread Jeremias Blendin
Hello Mauricio, thank you for the pointer to the bin-to-hex conversion, I knew it looked strange but I could not see why m) In any case, it was just an example, the actual configuration for testing is correct. 2016-09-18 18:25 GMT+02:00 Mauricio Vasquez : > Hello Jeremias, > > > On 09/18/2016 05

Re: [ovs-dev] Good to finally reach you

2016-09-19 Thread george gibson
Hello How are you? i hope all is well with you. I have tried reachiny you without luck. My client is looking for a good business to invest in your country. If you have any good business in mind please get back to me urgently so i can give you more update. We are looking at Real Estate, Spa & fit

[ovs-dev] khgj vjsaglh

2016-09-19 Thread schedule
This message was undeliverable due to the following reason: Your message was not 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 likely there

[ovs-dev] [PATCH for p4c-behavioral 0/2] P4 support in OVS

2016-09-19 Thread Muhammad Shahbaz
Hi all, We are submitting the following patches on this list for your review. These patches contain changes to add support for P4 in OVS. An earlier, proof-of-concept, implementation has been developed and published in SIGCOMM this year: http://pisces.cs.princeton.edu/papers/sigcomm16-pisces.pdf.

[ovs-dev] [PATCH for p4c-behavioral 2/2] Add 'hash.c.h' file to support P4 based hash calculation

2016-09-19 Thread Muhammad Shahbaz
From: Cian Ferriter As a first pass, all standard header fields defined in the P4 input file are hashed. This file uses the bit_width of each field similar to the packets.h.h file to decide between calling the 32bit or 64bit OVS hashing function. This implementation was built around the Ethernet

[ovs-dev] [PATCH for p4c-behavioral 1/2] Adding OVS plugin

2016-09-19 Thread Muhammad Shahbaz
This adds a new plugin, called OVS, in p4c-behavioral. Given a P4 program, the plugin generates a set of C (macro) code which is called at different locations within the OVS codebase. The code under the plugin folder reflects the same directory structure and files in which they are called in the OV

[ovs-dev] [PATCH 2/2] flow: P4 based hash calculation on IP and Ethernet headers

2016-09-19 Thread Muhammad Shahbaz
From: Cian Ferriter Replaces miniflow_hash_5tuple() with miniflow_hash_ntuple() which calls the dynamically generated OVS_HASH_FIELDS macro. This macro is generated at the configure stage of OVS compilation and depends on the standard header fields in the P4 input file. As a first pass, all stand

[ovs-dev] [PATCH 0/2] P4 support in OVS

2016-09-19 Thread Muhammad Shahbaz
Hi all, We are submitting the following patches on this list for your review. These patches contain changes to add support for P4 in OVS. An earlier, proof-of-concept, implementation has been developed and published in SIGCOMM this year: http://pisces.cs.princeton.edu/papers/sigcomm16-pisces.pdf.

[ovs-dev] [PATCH 1/2] adding P4 support

2016-09-19 Thread Muhammad Shahbaz
This adds necessary changes in the OVS codebase to consume the C (macro) code generated by the OVS plugin in p4c-behavioral. It also updates the OVS build system to take a P4 program as input at 'configure' time. --- acinclude.m4 | 17 +++ build-aux/extract-o

[ovs-dev] [PATCH v6] ovn-nbctl: Add LB commands.

2016-09-19 Thread nickcooper-zhangtonghao
This patch provides the command line to create a load balancer. You can create a load balancer independently and add it to multiple switches or routers. A single load balancer can have multiple vips. Add a name column for the load balancer. With --add-duplicate, the command really creates a new loa

[ovs-dev] (no subject)

2016-09-19 Thread rbaker
Your message was undeliverable due to the following reason: Your message could not be 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 t

[ovs-dev] [PATCH] netdev-dpdk: Configure flow control only when necessary.

2016-09-19 Thread Ilya Maximets
It is not necessary to touch the physical device each time, if the configuration has not been changed. Also, few style issues fixed. Signed-off-by: Ilya Maximets --- lib/netdev-dpdk.c | 30 +- 1 file changed, 17 insertions(+), 13 deletions(-) diff --git a/lib/netdev-

Re: [ovs-dev] [PATCH v6] ovn-nbctl: Add LB commands.

2016-09-19 Thread Guru Shetty
On 19 September 2016 at 05:16, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can have mul

Re: [ovs-dev] [PATCH] datapath-windows: Add define for last module number

2016-09-19 Thread Nithin Raju
Only comment I had was to nuke the OVS_DBG_RESERVED and treat OVS_DBG_LAST itself as the last bit. Basically, we are restricting this to 32 since ŒovsLogLevel¹ is 32 bits. We can probably do: BUILD_ASSERT(OVS_DBG_LAST < 31) BUILD_ASSERT(OVS_DBG_LAST < sizeof ovsLogLevel). Thanks for the cleanup.

Re: [ovs-dev] backport request for 2.6: enable kernel datapath check in rpms

2016-09-19 Thread Russell Bryant
On Wed, Sep 14, 2016 at 1:50 PM, Ben Pfaff wrote: > On Tue, Sep 13, 2016 at 12:07:25PM -0400, Lance Richardson wrote: > > Requesting backports of these commits to the 2.6 branch: > > > > commit 8ef22bb1ceb7b2841d2e29eb283387f0a3a67ff9 > > rhel: add option to run kernel datapath test w

Re: [ovs-dev] Difference between hmap and cmap

2016-09-19 Thread Ben Pfaff
On Mon, Sep 19, 2016 at 01:53:37AM -0400, Hui Kang wrote: > I am studying the hashing functions in OVS and find these two > implementations: hmap and cmap. I understand that cmap is the improved > cuckoo hashing and cmap supports multiple-reader and single writer. My > questions are: > > 1. What d

Re: [ovs-dev] [PATCH] INSTALL.DPDK-ADVANCED.md: Remove execute permissions.

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 11:25:43AM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > INSTALL.DPDK-ADVANCED.md | 0 > 1 file changed, 0 insertions(+), 0 deletions(-) > mode change 100755 => 100644 INSTALL.DPDK-ADVANCED.md > > diff --git a/INSTALL.DPDK-ADVANCED.md b/INSTALL.DPDK-

Re: [ovs-dev] [PATCH 1/3] jsonrpc: Improve comment for jsonrpc_get_status().

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 11:27:00AM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit I think that this was a one-word change, plus rewrapping. Acked-by: Ben Pfaff ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinf

Re: [ovs-dev] [PATCH 2/3] dpif-provider: Clean up comments around ct functions.

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 11:27:01AM -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/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 11:27:02AM -0700, Justin Pettit wrote: > Signed-off-by: Justin Pettit Please drop the changes from "flow-based" to "flow\-based". The rule for "-" versus "\-" in nroff isn't "put \ before every -", but instead more like "put \ before a literal - in sample syntax". The fu

Re: [ovs-dev] [PATCH v2 rebase] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

2016-09-19 Thread Ben Pfaff
On Thu, Sep 15, 2016 at 09:28:37AM -0700, Jesse Gross wrote: > When using tunnel TLVs (at the moment, this means Geneve options), a > controller must first map the class and type onto an appropriate OXM > field so that it can be used in OVS flow operations. This table is > managed using OpenFlow ex

[ovs-dev] OVS 2.6 release proposal

2016-09-19 Thread Ben Pfaff
I suggest that we plan to release OVS 2.6.0 this Thursday, Sept. 22, unless significant new bugs appear before then. Comments? ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] Code complexity survey

2016-09-19 Thread Vard Antinyan
Dear openvswitch developers, We have undertaken a task to assess code complexity triggers and generate recommendations for developing simple and understandable code. Our intension is to share the results with you, developers, so everyone can learn the triggers behind complex software. We n

Re: [ovs-dev] [PATCH 1/3] jsonrpc: Improve comment for jsonrpc_get_status().

2016-09-19 Thread Justin Pettit
> On Sep 19, 2016, at 8:19 AM, Ben Pfaff wrote: > > On Fri, Sep 16, 2016 at 11:27:00AM -0700, Justin Pettit wrote: >> Signed-off-by: Justin Pettit > > I think that this was a one-word change, plus rewrapping. Yes, it added a word that then increased the line length of that line past 80 colum

Re: [ovs-dev] [PATCH 3/3] ovs-ofctl.8.in: Fix minor formatting issues.

2016-09-19 Thread Justin Pettit
> On Sep 19, 2016, at 8:26 AM, Ben Pfaff wrote: > > On Fri, Sep 16, 2016 at 11:27:02AM -0700, Justin Pettit wrote: >> Signed-off-by: Justin Pettit > > Please drop the changes from "flow-based" to "flow\-based". The rule > for "-" versus "\-" in nroff isn't "put \ before every -", but instead

[ovs-dev] [RFC v2] ovn-northd: support IPAM with externally specified MAC

2016-09-19 Thread Lance Richardson
The current IPAM implementation allocates both a MAC address and an IPv4 address when dynamic address allocation is requested. This patch adds the ability to specify a fixed MAC address for use with dynamic IPv4 address allocation. Example: ovn-nbctl lsp-set-addresses p1 "00:01:02:03:04:05 dyna

[ovs-dev] [PATCH] ovn-ctl: Initialize the databases.

2016-09-19 Thread Gurucharan Shetty
Without initializing the databases, commands of the form 'ovn-nbctl --wait=sb ls-add ls0' will simply hang. Signed-off-by: Gurucharan Shetty --- ovn/utilities/ovn-ctl | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ovn/utilities/ovn-ctl b/ovn/utilities/ovn-ctl index 07bff8a..8544d14 100755

Re: [ovs-dev] [PATCH v6] ovn-nbctl: Add LB commands.

2016-09-19 Thread nickcooper-zhangtonghao
> On Sep 19, 2016, at 10:40 PM, Guru Shetty wrote: > > On 19 September 2016 at 05:16, nickcooper-zhangtonghao > > wrote: > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to

[ovs-dev] [PATCH v7] ovn-nbctl: Add LB commands.

2016-09-19 Thread nickcooper-zhangtonghao
This patch provides the command line to create a load balancer. You can create a load balancer independently and add it to multiple switches or routers. A single load balancer can have multiple vips. Add a name column for the load balancer. With --add-duplicate, the command really creates a new loa

Re: [ovs-dev] [PATCH v2 rebase] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

2016-09-19 Thread Jesse Gross
On Mon, Sep 19, 2016 at 8:48 AM, Ben Pfaff wrote: > On Thu, Sep 15, 2016 at 09:28:37AM -0700, Jesse Gross wrote: >> When using tunnel TLVs (at the moment, this means Geneve options), a >> controller must first map the class and type onto an appropriate OXM >> field so that it can be used in OVS fl

Re: [ovs-dev] [PATCH] ovn-ctl: Initialize the databases.

2016-09-19 Thread Ben Pfaff
On Sun, Sep 18, 2016 at 11:26:52PM -0700, Gurucharan Shetty wrote: > Without initializing the databases, commands of the form > 'ovn-nbctl --wait=sb ls-add ls0' will simply hang. > > Signed-off-by: Gurucharan Shetty It's better if the separate initialization step is optional. I think that ovn-n

Re: [ovs-dev] [PATCH] ovsdb: monitor - Replace tabs with spaces

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 12:51:49AM +, Alin Serdean wrote: > Found by inspection. > > Signed-off-by: Alin Gabriel Serdean Thanks, applied to master. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH] NEWS: Add known issues for v2.6.0.

2016-09-19 Thread Ben Pfaff
On Thu, Sep 15, 2016 at 04:26:05PM -0700, Joe Stringer wrote: > Signed-off-by: Joe Stringer ... > + * When using the openvswitch module distributed with OVS against kernel > + versions 4.4-4.6, the openvswitch module cannot be loaded or used at > the > + same time as "ip_gre".

Re: [ovs-dev] [PATCH] ovn-ctl: Initialize the databases.

2016-09-19 Thread Guru Shetty
On 19 September 2016 at 11:08, Ben Pfaff wrote: > On Sun, Sep 18, 2016 at 11:26:52PM -0700, Gurucharan Shetty wrote: > > Without initializing the databases, commands of the form > > 'ovn-nbctl --wait=sb ls-add ls0' will simply hang. > > > > Signed-off-by: Gurucharan Shetty > > It's better if the

[ovs-dev] Registration Open for OVS Fall 2016 Conference

2016-09-19 Thread Justin Pettit
The Open vSwitch team will host our third annual conference focused on Open vSwitch and OVN on Nov. 7 and 8, 2016, to be held at the San Jose Doubletree. The Call for Participation is still open, but talks will likely include: - New stateful services for OVS - Deploying OVN at s

Re: [ovs-dev] [PATCH] ovn-ctl: Initialize the databases.

2016-09-19 Thread Ben Pfaff
On Mon, Sep 19, 2016 at 11:18:20AM -0700, Guru Shetty wrote: > On 19 September 2016 at 11:08, Ben Pfaff wrote: > > > On Sun, Sep 18, 2016 at 11:26:52PM -0700, Gurucharan Shetty wrote: > > > Without initializing the databases, commands of the form > > > 'ovn-nbctl --wait=sb ls-add ls0' will simply

[ovs-dev] FW: payment transfer

2016-09-19 Thread Mega Exchange service
Good day Transfer has been made to you on request of our customer. Find attached bank slip. :15A SENDER'S REFERENCE 11044TRF12008119 :21B BANK OPERATION CODE CRED :32A CURRENCY/INTERBANK SETTLEMENT 140927 US53,744,64 :33B CURRENCY/USD INSTRUCTED :001 ORDERING CUSTOMER /AL3621311044009

[ovs-dev] [PATCH 2/2] vagrant: use RECHECK=yes for system checks

2016-09-19 Thread Thadeu Lima de Souza Cascardo
Use RECHECK=yes for both kernel and userspace datapath tests. Signed-off-by: Thadeu Lima de Souza Cascardo --- Vagrantfile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Vagrantfile b/Vagrantfile index 11bd048..91f943d 100644 --- a/Vagrantfile +++ b/Vagrantfile @@ -47,7

[ovs-dev] [PATCH 1/2] vagrant: cleanup before building

2016-09-19 Thread Thadeu Lima de Souza Cascardo
Clean the source directory before building, otherwise, build might fail if it has been configured already. Signed-off-by: Thadeu Lima de Souza Cascardo --- Vagrantfile | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Vagrantfile b/Vagrantfile index 72f224c..11bd048 100644 --- a/Vagrantfile

Re: [ovs-dev] [PATCH v7] ovn-nbctl: Add LB commands.

2016-09-19 Thread Guru Shetty
On 19 September 2016 at 09:38, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can have mul

Re: [ovs-dev] [PATCH v7] ovn-nbctl: Add LB commands.

2016-09-19 Thread Guru Shetty
On 19 September 2016 at 09:38, nickcooper-zhangtonghao < nickcooper-zhangtong...@opencloud.tech> wrote: > This patch provides the command line to create a load balancer. > You can create a load balancer independently and add it to multiple > switches or routers. A single load balancer can have mul

[ovs-dev] [PATCH v2] ovn: Initialize the databases.

2016-09-19 Thread Gurucharan Shetty
Without initializing the databases, commands of the form 'ovn-nbctl --wait=sb ls-add ls0' will simply hang. Signed-off-by: Gurucharan Shetty --- ovn/northd/ovn-northd.c | 10 +++--- ovn/utilities/ovn-ctl | 2 ++ 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/ovn/northd/ovn

Re: [ovs-dev] [PATCH] ovn-ctl: Initialize the databases.

2016-09-19 Thread Guru Shetty
On 19 September 2016 at 11:33, Ben Pfaff wrote: > On Mon, Sep 19, 2016 at 11:18:20AM -0700, Guru Shetty wrote: > > On 19 September 2016 at 11:08, Ben Pfaff wrote: > > > > > On Sun, Sep 18, 2016 at 11:26:52PM -0700, Gurucharan Shetty wrote: > > > > Without initializing the databases, commands of

Re: [ovs-dev] [PATCH v2] netdev-dpdk: Add new 'dpdkvhostuserclient' port type

2016-09-19 Thread Daniele Di Proietto
Apologies for the delay, applied to master and branch-2.6 Thanks, Daniele 2016-09-15 6:53 GMT-07:00 Mooney, Sean K : > Hi I just wanted to follow up on the status of this patch. > > Without this patch support for vhost user reconnect will be blocked in > openstack. > > At this point It is now t

[ovs-dev] [PATCH] ovn-vtep: fix arping from vtep-gw physical port

2016-09-19 Thread Ramu Ramamurthy
Currently, arping from a vetp-gw physical-switch port to a VIF IP address does not work. When a physical-switch-port arps for an IP address of a VIF, that arp packet comes into the VIF hypervisor via a vxlan tunnel. That arp packet must not be responded-to by the arp responder table. Its treatment

Re: [ovs-dev] [PATCH 1/3] netdev-dpdk: Fix memory leak in dpdk_mp_{get, put}().

2016-09-19 Thread Daniele Di Proietto
Thanks, I simplified it slightly and applied this to master and branch-2.6 2016-09-05 6:36 GMT-07:00 Ilya Maximets : > 'dmp' should be freed on failure and on put. > > Fixes: 8a9562d21a40 ("dpif-netdev: Add DPDK netdev.") > Fixes: 8d38823bdf8b ("netdev-dpdk: fix memory leak") > Signed-off-by: Ily

Re: [ovs-dev] [PATCH 3/3] netdev-dpdk: Don't fail vHost port creation if server mode is unavailable.

2016-09-19 Thread Daniele Di Proietto
I believe that this is not necessary anymore, since Ciara's change (which I just applied) 2d24d165d6a5("netdev-dpdk: Add new 'dpdkvhostuserclient' port type"). Thanks, Daniele 2016-09-05 6:36 GMT-07:00 Ilya Maximets : > Currently vHost client mode feature is broken in case where QEMU and OVS >

[ovs-dev] [PATCH] ovsdb-idl: Update comment.

2016-09-19 Thread Ben Pfaff
TXN_AGAIN_WAIT and TXN_AGAIN_NOW were combined into a single return code TXN_TRY_AGAIN a long time ago, but this comment was not updated. Reported-by: Justin Pettit Signed-off-by: Ben Pfaff --- lib/ovsdb-idl.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/lib/ovsdb-idl.

Re: [ovs-dev] [PATCH] ovsdb-idl: Update comment.

2016-09-19 Thread Justin Pettit
> On Sep 19, 2016, at 2:54 PM, Ben Pfaff wrote: > > TXN_AGAIN_WAIT and TXN_AGAIN_NOW were combined into a single return code > TXN_TRY_AGAIN a long time ago, but this comment was not updated. > > Reported-by: Justin Pettit > Signed-off-by: Ben Pfaff > --- > lib/ovsdb-idl.c | 4 +--- > 1 file c

[ovs-dev] [PATCH] datapath-windows: Add define for last module number

2016-09-19 Thread Shashank Ram
Adds a define for the last defined module number. Change-Id: Id3b278d261c982d407b123b6fded46758c3ebe88 Signed-off-by: Shashank Ram --- datapath-windows/ovsext/Debug.c | 1 + datapath-windows/ovsext/Debug.h | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/datapath-windows/ovsex

[ovs-dev] [PATCH v2] datapath-windows: Add define for last module number

2016-09-19 Thread Shashank Ram
Adds a define for the last defined module number. Change-Id: I1ea9230317a849e911900cf69e96ed85a65d3a8c Signed-off-by: Shashank Ram --- datapath-windows/ovsext/Debug.c | 1 + datapath-windows/ovsext/Debug.h | 5 ++--- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/datapath-windows

Re: [ovs-dev] [PATCH v2] datapath-windows: Add define for last module number

2016-09-19 Thread Nithin Raju
Much better. Thanks for addressing the comments. Acked-by: Nithin Raju -Original Message- From: dev on behalf of Shashank Ram Date: Monday, September 19, 2016 at 3:24 PM To: "dev@openvswitch.org" Cc: Shashank Ram Subject: [ovs-dev] [PATCH v2] datapath-windows: Add define for last mo

[ovs-dev] Possible Spam :

2016-09-19 Thread Maureen cawley
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2] datapath-windows: Add define for last module number

2016-09-19 Thread Sairam Venugopal
Hi Shashank, Looks like the patch is missing ancestor history. Can you resend this patch after fixing this? Thanks, Sairam On 9/19/16, 3:24 PM, "Shashank Ram" wrote: >Adds a define for the last defined module number. > >Change-Id: I1ea9230317a849e911900cf69e96ed85a65d3a8c >Signed-off-by: Shash

[ovs-dev]

2016-09-19 Thread Leann borthwick
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH v3] datapath-windows: Add define for last module number

2016-09-19 Thread Shashank Ram
Adds a define for the last defined module number. Change-Id: I1ea9230317a849e911900cf69e96ed85a65d3a8c Signed-off-by: Shashank Ram --- datapath-windows/ovsext/Debug.c | 1 + datapath-windows/ovsext/Debug.h | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/datapath-windows/

Re: [ovs-dev] [PATCH v2] datapath-windows: Add define for last module number

2016-09-19 Thread Shashank Ram
Thanks Sai, latest patch should have this resolved. From: Sairam Venugopal Sent: Monday, September 19, 2016 3:55:19 PM To: Shashank Ram; dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH v2] datapath-windows: Add define for last module number Hi Shashank, Looks

Re: [ovs-dev] [PATCH v3] datapath-windows: Add define for last module number

2016-09-19 Thread Sairam Venugopal
Thanks for fixing this. Acked-by: Sairam Venugopal On 9/19/16, 4:16 PM, "Shashank Ram" wrote: >Adds a define for the last defined module number. > >Change-Id: I1ea9230317a849e911900cf69e96ed85a65d3a8c >Signed-off-by: Shashank Ram >--- > datapath-windows/ovsext/Debug.c | 1 + > datapath-windo

Re: [ovs-dev] [PATCH] ovsdb-idl: Update comment.

2016-09-19 Thread Ben Pfaff
On Mon, Sep 19, 2016 at 02:56:31PM -0700, Justin Pettit wrote: > > > On Sep 19, 2016, at 2:54 PM, Ben Pfaff wrote: > > > > TXN_AGAIN_WAIT and TXN_AGAIN_NOW were combined into a single return code > > TXN_TRY_AGAIN a long time ago, but this comment was not updated. > > > > Reported-by: Justin Pe

Re: [ovs-dev] [PATCH v3] datapath-windows: Add define for last module number

2016-09-19 Thread Nithin Raju
Acked-by: Nithin Raju -Original Message- From: dev on behalf of Sairam Venugopal Date: Monday, September 19, 2016 at 4:21 PM To: Shashank Ram , "dev@openvswitch.org" Subject: Re: [ovs-dev] [PATCH v3] datapath-windows: Add define for last module number >Thanks for fixing this. > >Acked

Re: [ovs-dev] [PATCH v2 2/5] ofproto-dpif-xlate: Adjust generated mask for fragments.

2016-09-19 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme Jarno > On Aug 30, 2016, at 6:47 PM, Daniele Di Proietto > wrote: > > It's possible to install an OpenFlow flow that matches on udp source and > destination ports without matching on fragments. If the subtable where > such flow stays is visited during translation

[ovs-dev]

2016-09-19 Thread Iva lapthorne
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [PATCH v2 3/5] tnl-neigh-cache: Unwildcard flow members before inspecting them.

2016-09-19 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme IMO this and the earlier (2/5) one are bug fixes that should be cherry-picked to branches 2.6 and 2.5, if applicable. Jarno > On Aug 30, 2016, at 6:47 PM, Daniele Di Proietto > wrote: > > tnl_neigh_snoop() is part of the translation. During translation we > have

Re: [ovs-dev] [PATCH v2 4/5] treewide: Use WC_MASK_FIELD() and FLOW_WC_GET_AND_MASK_WC().

2016-09-19 Thread Jarno Rajahalme
Unfortunately this and the last patch of the series does not apply to master, so you’d need to rebase these. Jarno > On Aug 30, 2016, at 6:47 PM, Daniele Di Proietto > wrote: > > I think it makes the code more readable. No functional change. > > Signed-off-by: Daniele Di Proietto > --- >

Re: [ovs-dev] [PATCH 2/3] token-bucket: Add token_bucket_wait_at().

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 04:10:50PM -0700, Jarno Rajahalme wrote: > Having the caller of token_bucket_wait() indicated in the log messages > makes debugging easier. > > Signed-off-by: Jarno Rajahalme Acked-by: Ben Pfaff ___ dev mailing list dev@openvsw

[ovs-dev]

2016-09-19 Thread Deirdre battie
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev]

2016-09-19 Thread Evelyn hodshon
___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev] [PATCH] ovn-controller: Fix possible null pointer dereference.

2016-09-19 Thread Justin Pettit
The code dereferences "chassis", which could be null if chassis_run() returns null. "chassis" will always be null if "chassis_id" is null, so checking "chassis" is sufficient to check both. Found by inspection. Signed-off-by: Justin Pettit --- ovn/controller/ovn-controller.c | 2 +- 1 file cha

Re: [ovs-dev] [PATCH] ovn-vtep: fix arping from vtep-gw physical port

2016-09-19 Thread Darrell Ball
On Mon, Sep 19, 2016 at 2:32 PM, Ramu Ramamurthy wrote: > Currently, arping from a vetp-gw physical-switch port to > a VIF IP address does not work. > > When a physical-switch-port arps for an IP address > of a VIF, that arp packet comes into the VIF hypervisor via a > vxlan tunnel. That arp pack

Re: [ovs-dev] [PATCH 3/3] seq: Add support for initial delay before waking up.

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 04:10:51PM -0700, Jarno Rajahalme wrote: > The execution time of 'ovs-ofctl add-flows' with a large number of > flows can be more than halved if revalidators are not running after > each flow mod separately. This was first suspected when it was found > that 'ovs-ofctl --bun