[ovs-dev] Brfendshyfoedv

This message was not delivered 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] Unable to hit group table entry

Hi All, I want to add some flow rules with actions as a group entry. The group and flow rule have been installed successfully. Outputs below are obtained by the ovs-ofctl command dump-flows, dump-groups and dump-group-stats. According to the stats for the flow, the flow rule was hit, but the

[ovs-dev] Meter implementation in openvswitch

Hi Ben, I want to implement and contribute B.19.13-Meter action (EXT-379) of OF 1.5.1 to openvswitch. As per my understanding, meter support in openvswitch datapath is not present as stated in the FAQs. Some patches[1] for meter implementation have been submitted but not applied to ovs

Re: [ovs-dev] [PATCH] system-traffic: Fix packet-in format for tests.

Thanks for fixing this! Acked-by: Daniele Di Proietto On 06/04/2016 15:07, "Joe Stringer" wrote: >Since continuations were introduced, the system-traffic tests which use >OpenFlow monitors to check the results of datapath execution have been >failing,

Re: [ovs-dev] [PATCH 1/1] Add Static route to logical router

> > > > Steve My understanding of last point, > 1. How to static routes? > format: "10.0.0.0/24 nexthop 10.1.1.1" or "10.0.0.0/24 nexthop 10.1.1.1 > dev port1" > For the latter format, it need the networking-ovn to lookup logical router > port table. > It will do what this patch did in

Re: [ovs-dev] [PATCH v2] dp-packet: Fix use of uninitialised value at emc_lookup.

Thanks for the fix! I've applied this to master and branch-2.5 2016-04-06 16:28 GMT-07:00 William Tu : > Valgrind reports "Conditional jump or move depends on uninitialised value" > and "Use of uninitialised value" at case 2016 ovn -- 3 HVs, 1 LS, 3 > lports/HV. It is caused

Re: [ovs-dev] [PATCH 1/1] Add Static route to logical router

On Wed, Apr 6, 2016 at 5:58 PM, Guru Shetty wrote: > On 6 April 2016 at 16:55, Mickey Spiegel wrote: > > > Steve and Guru, > > > > I am not all that concerned about the "valid" column, but I do think that > > we will need a different additional column in the

Re: [ovs-dev] [PATCH 1/1] Add Static route to logical router

On 6 April 2016 at 16:55, Mickey Spiegel wrote: > Steve and Guru, > > I am not all that concerned about the "valid" column, but I do think that > we will need a different additional column in the near future for output > port. > > There are three different motivations for

[ovs-dev] [PATCH] ovn-docker: Update overlay mode for new tcp ports.

There were changes made recently wherein 2 ovsdb-server is started for northbound and southbound databases with tcp ports 6641 and 6642. This breaks Docker integration. This commit fixes it. Signed-off-by: Gurucharan Shetty --- INSTALL.Docker.md | 10

[ovs-dev] (no subject)

This message was undeliverable due to the following reason(s): Your message could not be delivered because the destination server 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

Re: [ovs-dev] [PATCH 1/1] Add Static route to logical router

Steve and Guru, I am not all that concerned about the "valid" column, but I do think that we will need a different additional column in the near future for output port. There are three different motivations for allowing output port to be specified in the static route: 1) In order to support

[ovs-dev] [PATCH v2] dp-packet: Fix use of uninitialised value at emc_lookup.

Valgrind reports "Conditional jump or move depends on uninitialised value" and "Use of uninitialised value" at case 2016 ovn -- 3 HVs, 1 LS, 3 lports/HV. It is caused by 1) assigning an uninitialized value to 'key->hash' at emc_processing(). Due to uninit rss_hash_valid, dp_packet_rss_valid()

Re: [ovs-dev] [PATCH 3/3] ovs-dpctl: Document conntrack "zone" arguments in help output.

> On Apr 6, 2016, at 8:29 AM, Russell Bryant wrote: > > > On Mon, Mar 28, 2016 at 11:35 PM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > utilities/ovs-dpctl.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > >

[ovs-dev] [PATCH] system-traffic: Fix packet-in format for tests.

Since continuations were introduced, the system-traffic tests which use OpenFlow monitors to check the results of datapath execution have been failing, because the new PACKET_IN2 format is used rather than PACKET_IN. Switch the expected output over to PACKET_IN2. Signed-off-by: Joe Stringer

Re: [ovs-dev] [PATCH] ovn-northd: Handle IPv4 addresses with prefixes in lport port security

I think you might be able to write a slightly simpler patch by using ip_format_masked() like the following: -=-=-=-=-=-=-=-=-=- diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 4b1d611..890b17c 100644 --- a/ovn/northd/ovn-northd.c +++ b/ovn/northd/ovn-northd.c @@ -1179,8

Re: [ovs-dev] [PATCH v2] ovn: Add software l2 gateway.

On Wed, Apr 6, 2016 at 5:43 PM, Ramu Ramamurthy wrote: > On Mon, Apr 4, 2016 at 5:58 AM, Russell Bryant wrote: > > This patch implements one approach to using ovn-controller to implement > > a software l2 gateway between logical and physical networks.

Re: [ovs-dev] [PATCH v2] ovn: Add software l2 gateway.

On Mon, Apr 4, 2016 at 5:58 AM, Russell Bryant wrote: > This patch implements one approach to using ovn-controller to implement > a software l2 gateway between logical and physical networks. > > A new logical port type called "gateway" is introduced here. It is very > close to

Re: [ovs-dev] [PATCHv2] checkpatch: Don't enforce char limit on tests.

On Wed, Apr 6, 2016 at 3:46 PM, Joe Stringer wrote: > Although tests ideally also stick to shorter line lengths, it is very > common for fixed text blocks like flows or large packets to be specified > within tests. Checkpatch shouldn't complain about cases like these. > >

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

In the work made in our projects, it was found the need to have a faster access to the rows contained in tables in the replica, as well to have the possibility to loop over a subset of rows that meet some specified criteria. Those needs lead us to design and implement a functionality that

[ovs-dev] [PATCH 3/4 v2] ovsdb-idl: IDL Compound Indexes Implementation

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 --- lib/ovsdb-idl-provider.h | 30 lib/ovsdb-idl.c | 378

[ovs-dev] [PATCH 4/4 v2] ovsdb-idl: Autogenerated functions for compound indexes

From: Arnoldo Lutz Guevara Generates and fill the default comparators for columns with type int, real, string. Also creates the macros that allow to iterate over the contents of the index, and perform queries. Signed-off-by: Arnoldo Lutz Guevara

[ovs-dev] [PATCH 2/4 v2] lib:Data structures: Skiplist implementation

Skiplist implementation intended for the IDL compound indexes feature. Signed-off-by: Esteban Rodriguez Betancourt --- lib/automake.mk | 2 + lib/skiplist.c| 313 ++ lib/skiplist.h| 54 +

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

> From: Ben Pfaff [mailto:b...@ovn.org] > Sent: martes, 22 de marzo de 2016 14:55 > To: Rodriguez Betancourt, Esteban > Cc: dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH 3/5] ovsdb-idl: IDL Compound Indexes > Implementation > > On Wed, Mar 09, 2016 at 12:03:54AM +,

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

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. At the end of the message, valgrind reports "Uninitialised value was

[ovs-dev] [PATCHv2] checkpatch: Don't enforce char limit on tests.

Although tests ideally also stick to shorter line lengths, it is very common for fixed text blocks like flows or large packets to be specified within tests. Checkpatch shouldn't complain about cases like these. Signed-off-by: Joe Stringer --- v2: Broaden the set of blacklisted

Re: [ovs-dev] [PATCH] checkpatch: Don't enforce char limit on tests.

On 5 April 2016 at 11:37, Russell Bryant wrote: > > > On Tue, Apr 5, 2016 at 2:17 PM, Joe Stringer wrote: >> >> Although tests ideally also stick to shorter line lengths, it is very >> common for fixed text blocks like flows or large packets to be specified >>

Re: [ovs-dev] [PATCH] checkpatch: Don't enforce char limit on tests.

On 6 April 2016 at 07:33, Aaron Conole wrote: > Russell Bryant writes: > >> On Tue, Apr 5, 2016 at 2:17 PM, Joe Stringer wrote: >> >>> Although tests ideally also stick to shorter line lengths, it is very >>> common for fixed text blocks like

Re: [ovs-dev] [PATCH] ovn-northd: Handle IPv4 addresses with prefixes in lport port security

"dev" wrote on 04/06/2016 10:18:57 AM: > From: Numan Siddique > To: ovs dev > Date: 04/06/2016 10:19 AM > Subject: [ovs-dev] [PATCH] ovn-northd: Handle IPv4 addresses with > prefixes in lport port security > Sent by: "dev"

[ovs-dev] [PATCH v4] datapath-windows: Hot add CPU support.

Hot add CPU is the ability to dynamically add CPUs to a running system. Adding CPUs can occur physically by adding new hardware, logically by online hardware partitioning, or virtually through a virtualization layer. This patch add support to reallocate any per-cpu resources, in case a new

Re: [ovs-dev] [PATCH] dp-packet: Fix use of uninitialised value at emc_lookup.

Hi Darrell and Daniele, Thanks for the comments! I will use dp_packet_rss_invalidate() instead and send v2 patch. Regards, William On Wed, Apr 6, 2016 at 11:26 AM, Daniele Di Proietto wrote: > 2016-04-06 10:09 GMT-07:00 Darrell Ball : > > On Wed, Apr 6,

Re: [ovs-dev] [PATCH] dp-packet: Fix use of uninitialised value at emc_lookup.

2016-04-06 10:09 GMT-07:00 Darrell Ball : > On Wed, Apr 6, 2016 at 9:37 AM, William Tu wrote: > >> Valgrind reports "Conditional jump or move depends on uninitialised value" >> and "Use of uninitialised value" at case 2016 ovn -- 3 HVs, 1 LS, 3 >> lports/HV.

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

On 1 April 2016 at 17:37, 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

[ovs-dev] Advertencia buzón ha superado el límite de cuota

Querido usuario, Su buzón ha superado el límite de la cuota establecida por el administrador, usted no será capaz de enviar o recibir correo hasta que vuelve a validar su cuenta. Por favor, haga clic en el siguiente enlace o copiar y pegar en tu navegador para validar su buzón de correo.

Re: [ovs-dev] [PATCH] dp-packet: Fix use of uninitialised value at emc_lookup.

On Wed, Apr 6, 2016 at 9:37 AM, William Tu wrote: > Valgrind reports "Conditional jump or move depends on uninitialised value" > and "Use of uninitialised value" at case 2016 ovn -- 3 HVs, 1 LS, 3 > lports/HV. It is caused by reading uninitialized 'key->hash' at >

[ovs-dev] [PATCH] dp-packet: Fix use of uninitialised value at emc_lookup.

Valgrind reports "Conditional jump or move depends on uninitialised value" and "Use of uninitialised value" at case 2016 ovn -- 3 HVs, 1 LS, 3 lports/HV. It is caused by reading uninitialized 'key->hash' at emc_lookup() and 'rss_hash_valid' from dp_packet_rss_valid(). At emc_processing(), the

Re: [ovs-dev] [PATCH 1/3] ovn-northd: Limit line length to under 80 columns.

On Mon, Mar 28, 2016 at 11:35 PM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > ovn/northd/ovn-northd.c | 34 +++--- > 1 file changed, 19 insertions(+), 15 deletions(-) > Acked-by: Russell Bryant

Re: [ovs-dev] [PATCH 2/3] dpctl.man: Fix bolding for flush-conntrack command.

On Mon, Mar 28, 2016 at 11:35 PM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > lib/dpctl.man | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Acked-by: Russell Bryant -- Russell Bryant

Re: [ovs-dev] [PATCH 3/3] ovs-dpctl: Document conntrack "zone" arguments in help output.

On Mon, Mar 28, 2016 at 11:35 PM, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > utilities/ovs-dpctl.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > Acked-by: Russell Bryant -- Russell Bryant

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

On Wednesday, April 6, 2016, Russell Bryant > wrote: > > > On Tue, Apr 5, 2016 at 10:03 PM, Han Zhou wrote: > >> >> >> On Tue, Apr 5, 2016 at 2:24 PM, Russell Bryant wrote: >> >> +/* Return

[ovs-dev] [PATCH] ovn-northd: Handle IPv4 addresses with prefixes in lport port security

Initial implementation of port security, missed out this feature. Reported-by: Na Zhu Reported-at: https://bugs.launchpad.net/networking-ovn/+bug/1564414 Signed-off-by: Numan Siddique --- ovn/northd/ovn-northd.c | 31 ---

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

On Tue, Apr 5, 2016 at 5:24 PM, 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" &&

Re: [ovs-dev] [PATCH v2] ovn: Add software l2 gateway.

On Wed, Apr 6, 2016 at 10:49 AM, Han Zhou wrote: > > > On Wednesday, April 6, 2016, Russell Bryant wrote: > >> >> >> On Wed, Apr 6, 2016 at 3:10 AM, Han Zhou wrote: >> >>> >>> On Mon, Apr 4, 2016 at 5:58 AM, Russell Bryant

Re: [ovs-dev] [PATCH v2] ovn: Add software l2 gateway.

On Wed, Apr 6, 2016 at 3:10 AM, Han Zhou wrote: > > On Mon, Apr 4, 2016 at 5:58 AM, Russell Bryant wrote: > > - Each localnet logical port is implemented as a > pair of > > - patch ports, one in the integration bridge, one in a different > >

[ovs-dev] [PATCH v2] ovn: Add software l2 gateway.

On Wednesday, April 6, 2016, Russell Bryant > wrote: > > > On Wed, Apr 6, 2016 at 3:10 AM, Han Zhou wrote: > >> >> On Mon, Apr 4, 2016 at 5:58 AM, Russell Bryant wrote: >> > - Each

Re: [ovs-dev] [PATCH] checkpatch: Don't enforce char limit on tests.

Russell Bryant writes: > On Tue, Apr 5, 2016 at 2:17 PM, Joe Stringer wrote: > >> Although tests ideally also stick to shorter line lengths, it is very >> common for fixed text blocks like flows or large packets to be specified >> within tests. Checkpatch

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

On Tue, Apr 5, 2016 at 10:03 PM, Han Zhou wrote: > > > On Tue, Apr 5, 2016 at 2:24 PM, Russell Bryant wrote: > >> +/* Return true if the address sets match, false otherwise. */ > > +static bool > > +address_sets_match(struct address_set *addr_set, > > +

Re: [ovs-dev] quick question about: dev7.thedataarchive.com

Hi there, Just wanted to check back in to see if you received my email. Are there any questions I can help answer? Kind regards, Ryan On Thu, Mar 31, 2016 at 12:09 PM, Ryan Brody wrote: I really

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

"dev" wrote on 04/05/2016 04:24:18 PM: > From: Russell Bryant > To: dev@openvswitch.org > Date: 04/05/2016 04:25 PM > Subject: [ovs-dev] [PATCH 2/3] expr: Add address set support. > Sent by: "dev" > > Update the OVN

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

"dev" wrote on 04/05/2016 04:24:17 PM: > From: Russell Bryant > To: dev@openvswitch.org > Date: 04/05/2016 04:24 PM > Subject: [ovs-dev] [PATCH 1/3] ovn-controller: Add missing shash_destroy (). > Sent by: "dev" > >

[ovs-dev] Document(1)

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

[ovs-dev] [PATCH] dpif-netdev: report numa node number on pmd thread create failure

Since PMD threads are placed on the NUMA node of the port regardless of a possible pmd-cpu-mask setting, this can lead to a somewhat confusing "out of unpinned cores" message - there might be plenty of available cores in the mask but they cannot be used if the port is on different NUMA node than

[ovs-dev] Mail System Error - Returned Mail

The original message was received at Wed, 6 Apr 2016 15:57:03 +0530 from 130.237.117.135 - The following addresses had permanent fatal errors - ___ dev mailing list dev@openvswitch.org

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

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

Re: [ovs-dev] [PATCH v2] ovn: Add software l2 gateway.

On Mon, Apr 4, 2016 at 5:58 AM, Russell Bryant wrote: > - Each localnet logical port is implemented as a pair of > - patch ports, one in the integration bridge, one in a different > - bridge, with the same external-ids:ovn-localnet-port > -