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

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.

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

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:

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

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 +++

[ovs-dev] (no subject)

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

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

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(-)

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

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

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

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

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

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:

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

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

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

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

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

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

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

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 v2 rebase] tun-metadata: Manage tunnel TLV mapping table on a per-bridge basis.

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

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

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

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

> 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

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

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

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

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

[ovs-dev] OVS 2.6 release proposal

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

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

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

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

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

> 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

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

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 >

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

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

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

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

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

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

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

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

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

> 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. >

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

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

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

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

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

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

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

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

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

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

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

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

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

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

[ovs-dev] FW: payment transfer

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

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

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

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

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 +++

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

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

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

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

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

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

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

> 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

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

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

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

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

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

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

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

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

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

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

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

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

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

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:

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

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

[ovs-dev]

___ 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.

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

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

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

[ovs-dev]

___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

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

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

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

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 >--- >

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

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. > > > >

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

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

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

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. > >

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

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

[ovs-dev] khgj vjsaglh

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 v2] datapath-windows: Add define for last module number

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

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

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"

[ovs-dev] Possible Spam :

___ 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

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

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

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:

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

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

[ovs-dev]

___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

[ovs-dev]

___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

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

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