[ovs-dev] [PATCH] utilities/ovs-ofctl.8.in: Added clarifications and examples to ovs-ofctl man page

Added a few examples and clarifications to the man page to explain points I had difficulty figuring out without reading code/tests. Signed-off-by: Ori Shoshan --- 1 file changed, 22 insertions(+), 2 deletions(-) diff --git a/utilities/ovs-ofctl.8.in

Re: [ovs-dev] [PATCH V3] ovs-vtep: vtep-ctl and ovs-vtep support of adding explicit tunnel key

Darrell, To aviod lengthy response, i'll address the issues directly. The code in ovs-vtep should not be Neutron L2 border gateway specific. Please remove references to so-called relay and mesh ports. The relay vteps are a second level of hierachy of vteps - I have more detail --agreed

Re: [ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

On Thu, Aug 11, 2016 at 7:10 PM, Ryan Moats wrote: > "dev" wrote on 08/11/2016 07:20:34 PM: > >> From: Jesse Gross >> To: dev@openvswitch.org >> Date: 08/11/2016 07:22 PM >> Subject: [ovs-dev] [PATCH rebase 3/3] ovn-controller:

Re: [ovs-dev] [PATCH rebase 3/3] ovn-controller: Use UDP checksums when creating Geneve tunnels.

On Fri, Aug 12, 2016 at 1:06 PM, Ben Pfaff wrote: > On Thu, Aug 11, 2016 at 05:20:34PM -0700, Jesse Gross wrote: >> Currently metadata transmitted by OVN over Geneve tunnels is >> unprotected by any checksum other than the one provided by the link >> layer - this includes both the

[ovs-dev] [CudaMailTagged] Bonjour

Bonjour, Acceptez et tolérez cette intrusion peu décente dans votre messagerie. Je sais que mon message sera d'une grande surprise quand t-il vous parviendra. En effet, je suis Françoise Marcelle Clémence RABILLARD, chrétienne et j'ai 68 ans. J'ai le cœur serin vu que je suis touchée par une

[ovs-dev] Returned mail: see transcript for details

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

Re: [ovs-dev] [PATCH] utilities/ovs-ofctl.8.in: Added clarifications and examples to ovs-ofctl man page

On Sun, Aug 14, 2016 at 12:49:00PM +0300, Ori Shoshan wrote: > Added a few examples and clarifications to the man page to explain points I > had difficulty figuring out without reading code/tests. > > Signed-off-by: Ori Shoshan Thanks a lot, applied to master.

Re: [ovs-dev] Is linear search a searching method for vswitchd flow table?

[adding back ovs-dev] On Sun, Aug 14, 2016 at 09:57:37PM +0900, Nam Bong Ha wrote: > 1. PDF 'SDN Switches: Architecture abd Design' 20 page > URL : >

Re: [ovs-dev] [PATCH rebase 2/3] ovn-controller: Make encap processing more robust against changes.

On Sat, Aug 13, 2016 at 11:12:18PM -0700, Jesse Gross wrote: > I think the code is reasonably robust at this point and certainly it > fixes a number of existing bugs. However, the truth is that I wasn't > really all that happy about it. It took a fairly long time to work > through the various

Re: [ovs-dev] [PATCH v5 2/2] ovn-northd: Add logical flows to support DHCPv6

On Fri, Aug 05, 2016 at 09:36:39AM +0530, Numan Siddique wrote: > OVN implements native DHCPv6. DHCPv6 options are stored > in the 'DHCP_Options' NB table and logical ports refer to this > table to configure the DHCPv6 options. > > For each logical port configured with DHCPv6 Options following

Re: [ovs-dev] [PATCH 3/3] acinclude: Use SSE4.2 instruction set

Thanks for looking in to this Daniele, My comments inline. >-Original Message- >From: Daniele Di Proietto [mailto:diproiet...@ovn.org] >Sent: Sunday, August 14, 2016 9:13 PM >To: Bodireddy, Bhanuprakash >Cc: dev@openvswitch.org >Subject: Re: [ovs-dev]

Re: [ovs-dev] [PATCH v5 1/2] ovsdb: Add/use partial set updates.

On Sat, Aug 06, 2016 at 05:46:29PM -0500, Ryan Moats wrote: > This patchset mimics the changes introduced in > > f199df26 (ovsdb-idl: Add partial map updates functionality.) > 010fe7ae (ovsdb-idlc.in: Autogenerate partial map updates functions.) > 7251075c (tests: Add test for partial map

[ovs-dev] [PATCH] ovs-ctl: Properly handle shell quoting in os-release.

Until now, this code did not strip "" or '' from variable assignments in os-release. This fixes the problem. CC: Matt Mulsow Fixes: c60d6b096436 ("ovs-ctl: support populating system info from /etc/os-release") Signed-off-by: Ben Pfaff --- I have not tested

Re: [ovs-dev] [PATCH v4 0/8] ovn-trace: New utility for testing and debugging OVN

On Sat, Aug 13, 2016 at 09:30:22AM -0700, Ben Pfaff wrote: > The exciting part is the final commit. I recommend reading the > documentation for the new program. > > Also available at https://github.com/blp/ovs-reviews/tree/ovnacts > > v1->v2: First 9 commits applied. Rebased. > v2->v3: First

[ovs-dev] [PATCH v5 8/8] ovn-trace: New utility.

This new utility is intended to fulfill for OVN the purpose that "ofproto/trace" has for Open vSwitch. First, it's meant to be a useful tool for troubleshooting and diagnosis and in general for improving one's understanding of the emergent properties of a flow table. Second, it simplifies and

[ovs-dev] [PATCH v5 6/8] expr: New function expr_evaluate().

An upcoming commit will need to evaluate individual expressions outside the context of a classifier. test-ovn already had a function to do this but it wasn't general-purpose, so this commit makes a general-purpose version and adopts it for use in test-ovn as well. Signed-off-by: Ben Pfaff

Re: [ovs-dev] [PATCH v5 2/2] python: Add support for partial map and partial set updates

On Sat, Aug 06, 2016 at 05:46:30PM -0500, Ryan Moats wrote: > Allow the python IDL to use mutate operations more freely > by mimicing the partial map and partial set operations now > available in the C IDL. > > Unit tests for both of these types of operations are included. > They are not carbon

Re: [ovs-dev] Fix copyright statements from commit f1ab6e06

On Sun, Aug 14, 2016 at 06:43:06PM -0500, Ryan Moats wrote: > Commit f1ab6e06 ("Add/user partial set updates.) incorrectly > did not include HPE attribution for derived files > lib/ovsdb-set-op.[ch]. Add the attribution to correct this. > > Signed-off-by: Ryan Moats Wow,

Re: [ovs-dev] [PATCH v4] ofproto: Reduce bundle memory use.

On Mon, Aug 08, 2016 at 11:26:30AM -0700, Jarno Rajahalme wrote: > Instead of storing the (big) struct ofputil_flow_mod, create the new > rule and/or create the rule criteria for matching at bundle message > insert time. This change reduces the size of a bundle flow mod from > 3.5kb to 272 bytes,

Re: [ovs-dev] [RFC] Revert "netdev: do not allow devices to be opened with conflicting types"

Thanks for following up on this. I think we should apply this to master, as restart is broken if an internal interface has an IP address. Could you provide a signoff for the patch, please? Thanks, Daniele 2016-08-01 13:14 GMT-07:00 Thadeu Lima de Souza Cascardo < casca...@redhat.com>: > This

Re: [ovs-dev] [PATCH 1/3] INSTALL.DPDK-ADVANCED: Fix number and indentation in jumbo frames

Applied to master, thanks! 2016-08-14 11:35 GMT-07:00 Bhanuprakash Bodireddy < bhanuprakash.bodire...@intel.com>: > Fix numbering in jumbo frames subsection and indentation for markdown > editor viewer. > > Signed-off-by: Bhanuprakash Bodireddy > --- >

Re: [ovs-dev] [PATCH 3/3] acinclude: Use SSE4.2 instruction set

By default we do not want to build Open vSwitch for a particular CPU, we want to create a build that can be distributed to every CPU of the architecture (e.g. amd64). Unfortunately, this not entirely possible for with DPDK, because rte_memcpy (included by the headers in netdev-dpdk) requires

Re: [ovs-dev] [PATCH] ovs-ctl: Properly handle shell quoting inos-release.

"dev" wrote on 08/14/2016 02:39:56 PM: > From: Ben Pfaff > To: dev@openvswitch.org > Cc: Matthew Mulsow/Austin/IBM@IBMUS, Ben Pfaff > Date: 08/14/2016 02:40 PM > Subject: [ovs-dev] [PATCH] ovs-ctl: Properly handle shell quoting in >

[ovs-dev] [PATCH 3/3] acinclude: Use SSE4.2 instruction set

On processors with SSE4.2 instruction set support, CRC32 intrinsics can be used for efficient hash computation. Update the m4_translit to convert '.' to '_' that otherwise cause 'bad substitution' error when configuring OVS DPDK with msse4.2 support. ./configure: line 21027:

[ovs-dev] [PATCH 2/3] FAQ: Add question on Packet loss with RFC2544 tests

Signed-off-by: Bhanuprakash Bodireddy --- FAQ.md | 13 + 1 file changed, 13 insertions(+) diff --git a/FAQ.md b/FAQ.md index 2f50b96..22345c3 100644 --- a/FAQ.md +++ b/FAQ.md @@ -1051,6 +1051,19 @@ A: The short answer is that this is a misuse of a

[ovs-dev] [PATCH 1/3] INSTALL.DPDK-ADVANCED: Fix number and indentation in jumbo frames

Fix numbering in jumbo frames subsection and indentation for markdown editor viewer. Signed-off-by: Bhanuprakash Bodireddy --- INSTALL.DPDK-ADVANCED.md | 38 +++--- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git

Re: [ovs-dev] [PATCH v5 1/2] ovn-controller: Add 'put_dhcpv6_opts' action in ovn-controller

On Fri, Aug 05, 2016 at 09:36:07AM +0530, Numan Siddique wrote: > This patch adds a new OVN action 'put_dhcpv6_opts' to support native > DHCPv6 in OVN. > > ovn-controller parses this action and adds a NXT_PACKET_IN2 > OF flow with 'pause' flag set and the DHCPv6 options stored in > 'userdata'

Re: [ovs-dev] [PATCH] ovn: Add second ACL stage

On Sat, Aug 13, 2016 at 10:02 PM, Ben Pfaff wrote: > On Fri, Jul 29, 2016 at 05:28:26PM +, Mickey Spiegel wrote: > > Could you expand on why priorities in a single stage aren't enough to > > satisfy the use case? > > > > > > If two features are configured independently with a

Re: [ovs-dev] [PATCH v2 1/2] Add dry-run option to ovs-dpctl and ovs-ofctl commands.

Ben Pfaff wrote on 08/14/2016 12:36:31 AM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 08/14/2016 12:36 AM > Subject: Re: [ovs-dev] [PATCH v2 1/2] Add dry-run option to ovs- > dpctl and ovs-ofctl commands. > > On Thu, Aug 04,

[ovs-dev] [PATCH v5 3/8] lex: Integrate error handling into struct lexer.

The actions and expr modules had each developed their own error handling code that were very similar. Upcoming code needs similar error handling, so rather than duplicating it again, integrate it into the lexer itself. Signed-off-by: Ben Pfaff Acked-by: Ryan Moats

[ovs-dev] [PATCH v5 5/8] ovn-northd: Copy name in logical datapath southbound representations.

This makes it easier to debug based on the southbound database without looking at the northbound representation. This commit adds the name to "ovn-sbctl dump-flows" output and it will be even more useful in an upcoming commit. Signed-off-by: Ben Pfaff Acked-by: Ryan Moats

[ovs-dev] [PATCH v5 7/8] expr: New function expr_parse_microflow().

This allows "ovstest test-ovn evaluate-expr" work with arbitrary microflows rather than just a few restricted variables, but the main point is to enable the upcoming "ovn-trace" utility to accept arbitrary microflows in a format that seems reasonable for OVN. Signed-off-by: Ben Pfaff

[ovs-dev] [PATCH v5 2/8] ovn-util: Add common functions for the default NB and SB databases.

Signed-off-by: Ben Pfaff Acked-by: Ryan Moats --- ovn/controller-vtep/ovn-controller-vtep.c | 5 +++-- ovn/lib/ovn-util.c| 27 + ovn/lib/ovn-util.h| 4 ovn/northd/ovn-northd.c

[ovs-dev] [PATCH v5 4/8] meta-flow: New functions mf_subfield_copy() and mf_subfield_swap().

The function nxm_execute_reg_move() was almost a general-purpose function for manipulating subfields, except for its awkward interface that took a struct ofpact_reg_move instead of a plain source and destination. This commit introduces a general-purpose function in meta-flow that corrects this

[ovs-dev] [PATCH v5 0/8] ovn-trace: New utility for testing and debugging OVN

The exciting part is the final commit. I recommend reading the documentation for the new program. Also available at https://github.com/blp/ovs-reviews/tree/ovnacts v1->v2: First 9 commits applied. Rebased. v2->v3: First 12 commits applied. Fixed bad %-specifier in patch 2. v3->v4: First

Re: [ovs-dev] [PATCH v5 1/2] ovsdb: Add/use partial set updates.

Ben Pfaff wrote on 08/14/2016 06:33:45 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: d...@openvswitch.com > Date: 08/14/2016 06:33 PM > Subject: Re: [ovs-dev] [PATCH v5 1/2] ovsdb: Add/use partial set updates. > > On Sat, Aug 06, 2016 at 05:46:29PM

[ovs-dev] [PATCH] Fix copyright statements from commit f1ab6e06

Commit f1ab6e06 ("Add/user partial set updates.) incorrectly did not include HPE attribution for derived files lib/ovsdb-set-op.[ch]. Add the attribution to correct this. Signed-off-by: Ryan Moats --- lib/ovsdb-set-op.c | 3 ++- lib/ovsdb-set-op.h | 3 ++- 2 files changed, 4

[ovs-dev] [PATCH v3 1/2] Add read-only option to ovs-dpctl and ovs-ofctl commands.

ovs-dpctl and ovs-ofctl lack a read-only option to prevent running of commands that perform read-write operations. Add it and the necessary scaffolding to each. Signed-off-by: Ryan Moats --- v1->v2: Fixed typo in usage string lib/command-line.c | 50

[ovs-dev] [PATCH v3 2/2] Add wrapper scripts for *ctl commands

This commit creates wrapper scripts for the *ctl commands to use --dry-run or --read-only as appropriate, and to allow for log level setting vi ovs-appctl without allowing full access to ovs-appctl. The debian install files for ovn-common, openvswitch-common and openvswitch-vswitch have been

[ovs-dev] [PATCH v3 0/2] Read only versions of the *ctl commands

This patch series adds read-only wrapper scripts around the various *ctl commands that leverage the --dry-run command line option for scripts that modify a database. For commands that don't have the --dry-run option (ovs-dpctl and ovs-ofctl) a new --read-only option is added to prevent commands

Re: [ovs-dev] [PATCH] ovs-ctl: Properly handle shell quoting inos-release.

On Sun, Aug 14, 2016 at 06:57:54PM -0500, Ryan Moats wrote: > "dev" wrote on 08/14/2016 02:39:56 PM: > > > From: Ben Pfaff > > To: dev@openvswitch.org > > Cc: Matthew Mulsow/Austin/IBM@IBMUS, Ben Pfaff > > Date: 08/14/2016 02:40 PM > >

[ovs-dev] [PATCH] Simplify partial map Py3 IDL test added by commit a59912a0

Commit a59912a0 ("python: Add support for partial map and partial set updates") added unit tests for the partial map function for the python IDL. However, because Python3 doesn't order dictionaries consistently, this test is a crap shoot for systems that support Python3. As a short term fix, do

Re: [ovs-dev] [PATCH] ovs-ctl: Properly handle shell quoting inos-release.

Ben Pfaff wrote on 08/14/2016 07:38:08 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org, Matthew Mulsow/Austin/IBM@IBMUS > Date: 08/14/2016 07:38 PM > Subject: Re: [ovs-dev] [PATCH] ovs-ctl: Properly handle shell > quoting

Re: [ovs-dev] [PATCH] sandbox: launch SB backup server when running in OVN mode

On Tue, Aug 09, 2016 at 04:49:54PM -0700, Andy Zhou wrote: > Automatically launch backup server for OVN SB database that replicates > all transactions of the active server. This can be handy for > experimenting with the newly added replication feature. > > Signed-off-by: Andy Zhou

[ovs-dev] 开发国外买家的方案

我已邀请您填写以下表单： 开发国外买家的方案 要填写此表单，请访问： https://docs.google.com/forms/d/e/1FAIpQLSeWhXRmpb-9DHMq3k1cn4xQEKHXVskNO3TtDWNzEfndh_eMqQ/viewform?c=0w=1usp=mail_form_link Google表单：创建调查问卷并分析调查结果。 ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH v8 3/3] tests: Add a new MTU test.

On Fri, Aug 12, 2016 at 07:56:55PM -0700, Daniele Di Proietto wrote: > Also, netdev-dummy needs to call netdev_change_seq_changed() in > set_mtu(). > > Signed-off-by: Daniele Di Proietto This introduces a novel, and smart, usage of the ovs-vsctl "wait-until" command.

Re: [ovs-dev] [PATCH v8 2/3] netdev-dummy: Add dummy-internal class.

On Fri, Aug 12, 2016 at 07:56:54PM -0700, Daniele Di Proietto wrote: > "internal" netdevs are treated specially in OVS (e.g. for MTU), but > the dummy datapath remaps both "system" and "internal" devices to the > same "dummy" netdev class, so there's no way to discern those in tests. > > This

[ovs-dev] [PATCH] ovn: ARP reply security vulnerability.

The the logical routers check only the "arp.op == 2" for ARP replies and then use ARP replies to populate the logical router's ARP table. If we continue to send ARP replies, which have different "arp.spa" and "arp.sha", to logical router, the MAC_Binding table will continue to increase. That may

Re: [ovs-dev] [ovs-dev, v3, 1/2] Add read-only option to ovs-dpctl and ovs-ofctl commands.

On Mon, Aug 15, 2016 at 12:17:34AM +, Ryan Moats wrote: > ovs-dpctl and ovs-ofctl lack a read-only option to prevent > running of commands that perform read-write operations. Add > it and the necessary scaffolding to each. > > Signed-off-by: Ryan Moats > --- > > v1->v2:

[ovs-dev] [PATCH v3] ovn-controller: Remove flows created for now deleted SB database rows.

Ensure that rows created for deleted port binding and multicast group rows are cleared when doing full processing. Signed-off-by: Ryan Moats --- v2->v3: - correctly clear old hmap (that's what I get for not reading the hmap_clear code first) v1->v2: - replace use of

Re: [ovs-dev] [ovs-dev, v2] ovn-controller: Remove flows created for now deleted SB database rows.

Ben Pfaff wrote on 08/14/2016 10:06:41 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 08/14/2016 10:06 PM > Subject: Re: [ovs-dev, v2] ovn-controller: Remove flows created for > now deleted SB database rows. > > On Fri, Aug

Re: [ovs-dev] [PATCH v2 1/2] Add dry-run option to ovs-dpctl and ovs-ofctl commands.

On Sun, Aug 14, 2016 at 06:59:09PM -0500, Ryan Moats wrote: > On a side note - I'm starting to see consistent failures in the bfd - decay > case... > Is this just me, or are others also seeing it? We've had at least one report, but I haven't been seeing it myself.

[ovs-dev] Message could not be delivered

The original message was received at Mon, 15 Aug 2016 08:01:33 +0700 from hocevar.net [67.153.54.186] - The following addresses had permanent fatal errors - dev@openvswitch.org - Transcript of session follows - ... while talking to host 97.243.75.53: 554 ...

[ovs-dev] [PATCH v4 2/2] Add wrapper scripts for *ctl commands

This commit creates wrapper scripts for the *ctl commands to use --dry-run or --read-only as appropriate, and to allow for log level setting vi ovs-appctl without allowing full access to ovs-appctl. The debian install files for ovn-common, openvswitch-common and openvswitch-vswitch have been

Re: [ovs-dev] [ovs-dev, v2] ovn-controller: Remove flows created for now deleted SB database rows.

On Fri, Aug 12, 2016 at 09:35:07AM -0500, Ryan Moats wrote: > Ensure that rows created for deleted port binding and > multicast group rows are cleared when doing full processing. > > Signed-off-by: Ryan Moats > --- > v1->v2: > - replace use of ssets for storing UUIDs as

Re: [ovs-dev] Simplify partial map Py3 IDL test added by commit a59912a0

On Sun, Aug 14, 2016 at 07:48:24PM -0500, Ryan Moats wrote: > Commit a59912a0 ("python: Add support for partial map > and partial set updates") added unit tests for the partial > map function for the python IDL. However, because Python3 > doesn't order dictionaries consistently, this > test is a

[ovs-dev] [PATCH v4 1/2] Add read-only option to ovs-dpctl and ovs-ofctl commands.

ovs-dpctl and ovs-ofctl lack a read-only option to prevent running of commands that perform read-write operations. Add it and the necessary scaffolding to each. Signed-off-by: Ryan Moats --- v1->v2: Fixed typo in usage string lib/command-line.c | 50

[ovs-dev] [PATCH v4 0/2] Read only versions of the *ctl commands

This patch series adds read-only wrapper scripts around the various *ctl commands that leverage the --dry-run command line option for scripts that modify a database. For commands that don't have the --dry-run option (ovs-dpctl and ovs-ofctl) a new --read-only option is added to prevent commands

Re: [ovs-dev] [ovs-dev, v3, 1/2] Add read-only option to ovs-dpctl and ovs-ofctl commands.

Ben Pfaff wrote on 08/14/2016 10:22:33 PM: > From: Ben Pfaff > To: Ryan Moats/Omaha/IBM@IBMUS > Cc: dev@openvswitch.org > Date: 08/14/2016 10:22 PM > Subject: Re: [ovs-dev, v3, 1/2] Add read-only option to ovs-dpctl > and ovs-ofctl commands. > > On Mon, Aug 15, 2016

Re: [ovs-dev] [PATCH v2 1/2] ovn: Ability to set multiple load balancers.

On Fri, Aug 12, 2016 at 02:48:12AM -0700, Gurucharan Shetty wrote: > The schema prevented one from setting more than one load balancer > to a switch. This removes that anomaly. > > (The test has a slightly unrelated change where it increases the > number of connections to get rid of some

Re: [ovs-dev] [PATCH v8 1/3] netdev: Pass 'netdev_class' to ->run() and ->wait().

On Fri, Aug 12, 2016 at 07:56:53PM -0700, Daniele Di Proietto wrote: > This will allow run() and wait() methods to be shared between different > classes and still perform class-specific work. > > Signed-off-by: Daniele Di Proietto Acked-by: Ben Pfaff