Re: [ovs-dev] [PATCH] Update netbsd install doc

2016-12-21 Thread Takashi YAMAMOTO
On Wed, Dec 21, 2016 at 3:25 AM, Hui Kang wrote: > - test ovs on netbsd 7.0.2 > - use gmake to compile and install > > Signed-off-by: Hui Kang > --- > Documentation/intro/install/netbsd.rst | 5 +++-- > 1 file changed, 3 insertions(+), 2

Re: [ovs-dev] [PATCH 1/2] ovn: Document upgrade procedure.

2016-12-21 Thread Ben Pfaff
On Wed, Dec 21, 2016 at 10:56:35PM -0500, Russell Bryant wrote: > On Wed, Dec 21, 2016 at 5:06 PM, Ben Pfaff wrote: > > > On Wed, Dec 07, 2016 at 02:28:13PM -0500, Russell Bryant wrote: > > > Signed-off-by: Russell Bryant > > > > This procedure isn't what I

Re: [ovs-dev] [PATCH v3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-21 Thread Ben Pfaff
On Wed, Dec 21, 2016 at 06:35:43PM -0500, Lance Richardson wrote: > Add support for SSL connections to OVN northbound and/or > southbound databases. > > To improve security, the NB and SB ovsdb daemons no longer > have open ptcp connections by default. This is a change in > behavior from

Re: [ovs-dev] [PATCH 1/2] ovn: Document upgrade procedure.

2016-12-21 Thread Russell Bryant
On Wed, Dec 21, 2016 at 5:06 PM, Ben Pfaff wrote: > On Wed, Dec 07, 2016 at 02:28:13PM -0500, Russell Bryant wrote: > > Signed-off-by: Russell Bryant > > This procedure isn't what I expected. It recommends: > > 1. Upgrade SB/NB databases and northd. >

Re: [ovs-dev] [PATCH v2] odp-execute: Optimize IP header modification in OVS datapath

2016-12-21 Thread Daniele Di Proietto
2016-12-13 9:27 GMT-08:00 Zoltán Balogh : > > Hi Daniele, > >> Have you tried avoiding also computing the new field if the mask is 0? >> For example, if >> mask->ipv4_src is 0, there's not reason to compute new_ip_src, or to >> extract ip_src_nh. > > Yes, I have

Re: [ovs-dev] [RFC-PATCH 2/2] lib/dpdk: No more deferred release

2016-12-21 Thread Daniele Di Proietto
On 21/12/2016 12:06, "Aaron Conole" wrote: >Aaron Conole writes: > >> DPDK documentation is recently updated to reflect that DPDK does not >> hold any references to, nor take ownership of, the argv/argc elements. >> With that understanding, let's

Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: Remove OVS_UNUSED from upcall_unixctl_set_flow_limit() arg.

2016-12-21 Thread Justin Pettit
> On Dec 21, 2016, at 4:06 PM, Joe Stringer wrote: > > On 21 December 2016 at 14:03, Justin Pettit wrote: >> The 'argv' argument is used. >> >> Signed-off-by: Justin Pettit > > Acked-by: Joe Stringer Thanks. I pushed this to

Re: [ovs-dev] [PATCH 1/5] system-traffic: Introduce OVS_START_L7 macro.

2016-12-21 Thread Joe Stringer
On 21 December 2016 at 17:05, Daniele Di Proietto wrote: > 2016-12-20 13:28 GMT-08:00 Joe Stringer : >> All of the commands starting L7 servers duplicate detailed specifics >> which inhibits readability, and makes it difficult to ensure that the >> servers are

Re: [ovs-dev] [PATCH] ovn-trace: Fix small error condition memory leak in trace().

2016-12-21 Thread Justin Pettit
> On Dec 21, 2016, at 2:12 PM, Ben Pfaff wrote: > > On Wed, Dec 21, 2016 at 02:04:32PM -0800, Justin Pettit wrote: >> Signed-off-by: Justin Pettit >> --- >> ovn/utilities/ovn-trace.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >> >> diff --git

Re: [ovs-dev] [PATCH 2/5] system-traffic: Wait for L7 servers to start.

2016-12-21 Thread Joe Stringer
On 21 December 2016 at 17:05, Daniele Di Proietto wrote: > 2016-12-20 13:28 GMT-08:00 Joe Stringer : >> Use OVS_WAIT_UNTIL() with netstat to ensure servers are listening before >> sending requests to them. This allows us to drop --retry-connrefused on >> wget,

Re: [ovs-dev] [PATCH 5/5] lib: Add support for tftp ct helper.

2016-12-21 Thread Daniele Di Proietto
2016-12-20 13:28 GMT-08:00 Joe Stringer : > The kernel datapath provides support for TFTP helpers, so add support > for this ALG to the commandline and OpenFlow encoding/decoding. > > Signed-off-by: Joe Stringer Do you want to add a test to tests/ofp-actions.at to

Re: [ovs-dev] [PATCH 3/5] system-traffic: Add banners for ct sections.

2016-12-21 Thread Daniele Di Proietto
2016-12-20 13:28 GMT-08:00 Joe Stringer : > Signed-off-by: Joe Stringer Acked-by: Daniele Di Proietto > --- > tests/system-ovn.at | 2 ++ > tests/system-traffic.at | 3 +++ > 2 files changed, 5 insertions(+) > > diff --git

Re: [ovs-dev] [PATCH] system-traffic: Fix clone test.

2016-12-21 Thread Joe Stringer
On 21 December 2016 at 14:22, William Tu wrote: > The existing clone test fails the system testsuite. The patch provides > fix, removes the unused at_ns2, and uses "ovs-ofctl monitor" to validate > the packet contents after actions inside a clone. > > Signed-off-by: William

Re: [ovs-dev] [PATCH] ofproto-dpif-upcall: Remove OVS_UNUSED from upcall_unixctl_set_flow_limit() arg.

2016-12-21 Thread Joe Stringer
On 21 December 2016 at 14:03, Justin Pettit wrote: > The 'argv' argument is used. > > Signed-off-by: Justin Pettit Acked-by: Joe Stringer ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [RFC PATCH v6 1/6] Add support for 802.1ad (QinQ tunneling)

2016-12-21 Thread Ben Pfaff
On Mon, Dec 12, 2016 at 11:29:49AM -0500, Eric Garver wrote: > Flow key handling changes: > - Add VLAN header array in struct flow, to record multiple 802.1q VLAN >headers. > - Add dpif multi-VLAN capability probing. If datapath supports >multi-VLAN, increase the maximum depth of nested

Re: [ovs-dev] [ovs-ovn] ovn:When configuring Logical switch port type by" lsp-set-type"command, Not Specify the type of Logical switch port.

2016-12-21 Thread Ben Pfaff
On Mon, Dec 12, 2016 at 11:00:38AM +0800, zhou.huij...@zte.com.cn wrote: > commit 812f44a9cb6862f3b487e23154a9e114040c40a2 > Author: zhaojingjing > Date: Mon Dec 12 10:48:33 2016 +0800 > > Specify the type of Logical switch port > This seems to have limited

[ovs-dev] [PATCH v3] ovn-ctl: add support for SSL nb/sb db connections

2016-12-21 Thread Lance Richardson
Add support for SSL connections to OVN northbound and/or southbound databases. To improve security, the NB and SB ovsdb daemons no longer have open ptcp connections by default. This is a change in behavior from previous versions, users wishing to use TCP connections to the NB/SB daemons can

Re: [ovs-dev] [PATCH] Update netbsd install doc

2016-12-21 Thread Ben Pfaff
On Tue, Dec 20, 2016 at 01:25:07PM -0500, Hui Kang wrote: > - test ovs on netbsd 7.0.2 > - use gmake to compile and install > > Signed-off-by: Hui Kang Thanks, applied to master. ___ dev mailing list d...@openvswitch.org

[ovs-dev] [PATCH v4 1/2] ovn-controller: Tie OpenFlow and logical flows using OpenFlow cookie.

2016-12-21 Thread Ben Pfaff
This makes it easy to find the logical flow that generated a particular OpenFlow flow, by running "ovn-sbctl dump-flows ". Later, this can be refined (and automated for "ofproto/trace"), but this is still a significant advance. Signed-off-by: Ben Pfaff --- lib/uuid.c

[ovs-dev] [PATCH v4 0/2] Clearly relate OVN and OpenFlow flows.

2016-12-21 Thread Ben Pfaff
v1->v2: - Patch 1 in v1 was committed and therefore dropped from the series. - Patch 2 in v2 is new. v2->v3: - Rebase to fix patch rejects. v3->v4: - Rebase to fix patch rejects. Ben Pfaff (2): ovn-controller: Tie OpenFlow and logical flows using OpenFlow cookie. ovn-trace: New

Re: [ovs-dev] [PATCH] ovsdb-data: Add support for integer ranges in database commands

2016-12-21 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 10:39:35PM +0100, Łukasz Rząsik wrote: > I was not worrying about this problem because I assumed it was already > there. A user could provide a very long list of integers and this would > affect memory in similar way. There is a limit for how long the list could > get,

Re: [ovs-dev] [PATCH] Documentation: fix some typos

2016-12-21 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 04:37:16PM -0500, Lance Richardson wrote: > s/deamon/daemon/ > s/dependant/dependent/ > > Signed-off-by: Lance Richardson Thanks, applied to master. ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH 5/5] datapath-windows: Conntrack disable type truncation warning

2016-12-21 Thread Guru Shetty
On 5 December 2016 at 07:39, Alin Serdean wrote: > Compiling with the WDK 10 gave the following warning: > Warning C4311 'type cast': pointer truncation from 'POVS_CT_ENTRY' to > 'UINT32' > ovsext (OVSExt\ovsext) Conntrack.c 1139 > > This patch disables

Re: [ovs-dev] [PATCH 2/5] datapath-windows: Add Windows 10 family to solution

2016-12-21 Thread Guru Shetty
On 5 December 2016 at 07:39, Alin Serdean wrote: > This patch adds two more compiling targets: > - one for Windows 10 release > - one for Windows 10 Debug > > The new targets are flagged properly to use the new Windows 10 kernel mode > driver and its

Re: [ovs-dev] [PATCH] ovn-northd: fix monitor process naming

2016-12-21 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 11:16:53AM -0500, Lance Richardson wrote: > Currently the ovn-northd monitor process and the ovn-northd process > have the same name, e.g. ps -ef | grep northd shows (edited for space): > > ... ovn-northd --detach --monitor --log-file=ovn-northd.log --pidfile > ...

Re: [ovs-dev] [PATCH v2 0/3] ovn: support ssl connections to nb/sb dbs

2016-12-21 Thread Ben Pfaff
On Thu, Dec 08, 2016 at 01:26:41PM -0500, Lance Richardson wrote: > > From: "Lance Richardson" > > To: d...@openvswitch.org, b...@ovn.org, russ...@ovn.org, nusid...@redhat.com > > Sent: Thursday, December 8, 2016 1:12:22 PM > > Subject: [ovs-dev] [PATCH v2 0/3] ovn: support

Re: [ovs-dev] [PATCH] datapath-windows: Remove dead code from PacketIO

2016-12-21 Thread Ben Pfaff
On Thu, Dec 08, 2016 at 05:57:19PM +, Alin Serdean wrote: > Assigning value to 'nativeNbls' has no effect outside the function and > the variable is not used inside the function. > > Signed-off-by: Alin Gabriel Serdean Applied, thanks!

Re: [ovs-dev] [PATCH] nx-match: Only store significant bytes to stack.

2016-12-21 Thread Ben Pfaff
On Wed, Dec 07, 2016 at 04:49:00PM -0800, Jarno Rajahalme wrote: > Always storing the maximum mf_value size wastes about 120 bytes for > each stack entry. This patch changes the stack from an mf_value array > to a string of value-length pairs. > > The lenght is stored after the value so that the

Re: [ovs-dev] [PATCH] odp: Use struct in6_addr for IPv6 addresses.

2016-12-21 Thread Ben Pfaff
On Wed, Dec 07, 2016 at 04:48:23PM -0800, Jarno Rajahalme wrote: > Code is simplified when the ODP keys use the same type as the struct > flow for the IPv6 addresses. As the change is facilitated by > extract-odp-netlink-h, this change only affects the userspace. We > already do the same for the

[ovs-dev] [PATCH] system-traffic: Fix clone test.

2016-12-21 Thread William Tu
The existing clone test fails the system testsuite. The patch provides fix, removes the unused at_ns2, and uses "ovs-ofctl monitor" to validate the packet contents after actions inside a clone. Signed-off-by: William Tu --- tests/system-common-macros.at | 6 ++

Re: [ovs-dev] [PATCH] ovn-trace: Fix small error condition memory leak in trace().

2016-12-21 Thread Ben Pfaff
On Wed, Dec 21, 2016 at 02:04:32PM -0800, Justin Pettit wrote: > Signed-off-by: Justin Pettit > --- > ovn/utilities/ovn-trace.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/ovn/utilities/ovn-trace.c b/ovn/utilities/ovn-trace.c > index

Re: [ovs-dev] [PATCH 2/2] ovn: Move a couple files into ovn/docs/.

2016-12-21 Thread Ben Pfaff
On Wed, Dec 07, 2016 at 02:28:14PM -0500, Russell Bryant wrote: > Signed-off-by: Russell Bryant Acked-by: Ben Pfaff ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

[ovs-dev] [PATCH] ovn-trace: Fix small error condition memory leak in trace().

2016-12-21 Thread Justin Pettit
Signed-off-by: Justin Pettit --- ovn/utilities/ovn-trace.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ovn/utilities/ovn-trace.c b/ovn/utilities/ovn-trace.c index 71798f8..6d8e514 100644 --- a/ovn/utilities/ovn-trace.c +++ b/ovn/utilities/ovn-trace.c

[ovs-dev] [PATCH] ofproto-dpif-upcall: Remove OVS_UNUSED from upcall_unixctl_set_flow_limit() arg.

2016-12-21 Thread Justin Pettit
The 'argv' argument is used. Signed-off-by: Justin Pettit --- ofproto/ofproto-dpif-upcall.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ofproto/ofproto-dpif-upcall.c b/ofproto/ofproto-dpif-upcall.c index df12882..32c7730 100644 ---

Re: [ovs-dev] splitting commits in Git (was: Re: Sync on PTAP, EXT-382 and NSH)

2016-12-21 Thread Ben Pfaff
On Wed, Dec 21, 2016 at 10:06:29AM -0800, Jarno Rajahalme wrote: > > > On Dec 21, 2016, at 9:30 AM, Ben Pfaff wrote: > > > > There was a question about splitting up large commits in Git. I do this > > routinely and I volunteered to find some resources. > > > > The "git-rebase"

Re: [ovs-dev] [PATCH v2 0/4] doc: Rework DPDK guide

2016-12-21 Thread Ben Pfaff
On Wed, Dec 14, 2016 at 10:07:56AM +, Stephen Finucane wrote: > As promised, this series reworks the largest of the installation guides > to suit the new structure of the documentation. A couple of related > patches are included to build upon this. > > Changes since v2: > - Rebase onto master

Re: [ovs-dev] [PATCH v3] ovn-controller: Fix duplicated flow add attempts in table 32.

2016-12-21 Thread Darrell Ball
There is another comment about the test below From: Han Zhou Date: Wednesday, December 21, 2016 at 1:50 PM To: Darrell Ball Cc: "d...@openvswitch.org" Subject: Re: [ovs-dev] [PATCH v3] ovn-controller: Fix duplicated flow add attempts

Re: [ovs-dev] [PATCH 1/5] doc: Add vlan cookbook from website

2016-12-21 Thread Ben Pfaff
On Sat, Dec 17, 2016 at 10:24:04PM +, Stephen Finucane wrote: > This is mostly copied verbatim. > > Signed-off-by: Stephen Finucane Thanks, I applied all of this series to master. ___ dev mailing list d...@openvswitch.org

Re: [ovs-dev] [PATCH 3/6] doc: Add info on building documentation

2016-12-21 Thread Ben Pfaff
On Sat, Dec 17, 2016 at 10:23:39PM +, Stephen Finucane wrote: > I know how to do this, but does anyone else? Let's make it obvious and > ease the cognitive load on the great folks writing docs. > > Links to the various packaging guides, previously missing, are included > on the main page. >

Re: [ovs-dev] [PATCH 1/6] doc: Add some useful tools for doc editing

2016-12-21 Thread Ben Pfaff
On Sat, Dec 17, 2016 at 10:23:37PM +, Stephen Finucane wrote: > This has come up on the mailing list. Let's document it! > > Signed-off-by: Stephen Finucane Thank you very much! I applied this series to master. ___ dev mailing

Re: [ovs-dev] [PATCH v2 1/4] ofproto: Update access rules for 'flow_cookie'.

2016-12-21 Thread Ben Pfaff
Thanks! I applied this patch to master. On Wed, Dec 21, 2016 at 10:40:23AM -0800, Jarno Rajahalme wrote: > Acked-by: Jarno Rajahalme > > > On Dec 20, 2016, at 11:03 PM, Ben Pfaff wrote: > > > > The 'flow_cookie' member of struct rule is read during flow

Re: [ovs-dev] [PATCH v2] ovn-controller: Fix duplicated flows in table 32.

2016-12-21 Thread Han Zhou
Hi Darrell, Sorry for late response. I posted v3 just now. On Wed, Dec 14, 2016 at 12:19 PM, Darrell Ball wrote: > > ERROR log level is not appropriate since we always filter adding the extra flows anyways > > and it amount to extra processing and filtering. > > We anyways

[ovs-dev] [PATCH 2/3] doc: Misc Windows doc formatting fixes

2016-12-21 Thread Stephen Finucane
There are a couple of minor issues with this document: - Some commands intended to be run in the MinGW shell are prefixed with '>', suggesting they are in fact PowerShell commands - PowerShell syntax highlighting is not enabled - Indentation is off for a couple of blocks Resolve all of these

[ovs-dev] [PATCH 1/3] doc: Resolve issues with Windows guide

2016-12-21 Thread Stephen Finucane
The formatting of this file was broken in a recent commit. Resolve this issue. Signed-off-by: Stephen Finucane Fixes: a0c03adff6c2 ("Windows: document multiple NIC support setup") Cc: Alin Gabriel Serdean ---

[ovs-dev] [PATCH 3/3] doc: Prefer use of 'code-block' directive

2016-12-21 Thread Stephen Finucane
The '::' element previously preferred is shorter, but does not allow for non-Python syntax highlighting. New documentation should make use of syntax highlighting wherever possible. Signed-off-by: Stephen Finucane ---

[ovs-dev] [PATCH v3] ovn-controller: Fix duplicated flow add attempts in table 32.

2016-12-21 Thread Han Zhou
In commit 475f0a2c it introduced a priority 150 flow for filtering the sending of traffic received from vxlan tunnels back out tunnels. However, it added the flow for every remote port processing, which results in continuous logs about duplicated flows. We only need to install this flow once per

Re: [ovs-dev] [PATCH v2 0/4] make ofproto/trace output easier to read

2016-12-21 Thread Lance Richardson
- Original Message - > From: "Ben Pfaff" > To: d...@openvswitch.org > Cc: "Ben Pfaff" > Sent: Wednesday, December 21, 2016 2:03:05 AM > Subject: [ovs-dev] [PATCH v2 0/4] make ofproto/trace output easier to read > > This series seeks to make the output of

Re: [ovs-dev] [PATCH v3 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-12-21 Thread Daniele Di Proietto
2016-12-21 10:18 GMT-08:00 Kevin Traynor : > On 12/21/2016 03:02 AM, Daniele Di Proietto wrote: >> 2016-12-20 14:08 GMT-08:00 Kevin Traynor : >>> On 12/15/2016 11:54 AM, Ciara Loftus wrote: 'dpdk' ports no longer have naming restrictions. Now, instead

Re: [ovs-dev] [PATCH v3 05/16] ofp-actions: Add clone action.

2016-12-21 Thread Joe Stringer
On 21 December 2016 at 10:28, William Tu wrote: > sorry, I forgot to add priority. Below lower the priority of the NORMAL > action. > > diff --git a/tests/system-traffic.at b/tests/system-traffic.at > index d70c5c3..321a901 100644 > --- a/tests/system-traffic.at > +++

Re: [ovs-dev] [PATCH v2 1/4] ofproto: Update access rules for 'flow_cookie'.

2016-12-21 Thread Jarno Rajahalme
Acked-by: Jarno Rajahalme > On Dec 20, 2016, at 11:03 PM, Ben Pfaff wrote: > > The 'flow_cookie' member of struct rule is read during flow translation > without holding a mutex, breaking the documented OVS_GUARDED access rule. > However, the flow_cookie member is

Re: [ovs-dev] [PATCH v3 05/16] ofp-actions: Add clone action.

2016-12-21 Thread William Tu
sorry, I forgot to add priority. Below lower the priority of the NORMAL action. diff --git a/tests/system-traffic.at b/tests/system-traffic.at index d70c5c3..321a901 100644 --- a/tests/system-traffic.at +++ b/tests/system-traffic.at @@ -340,6 +340,7 @@ AT_CLEANUP AT_SETUP([datapath - clone

Re: [ovs-dev] [PATCH v3 05/16] ofp-actions: Add clone action.

2016-12-21 Thread Joe Stringer
On 21 December 2016 at 07:45, William Tu wrote: > Hi Joe, > > I think we're missing the normal action for handling the non-IP > traffic. Can you try this patch on top of the clone? > > --- a/tests/system-traffic.at > +++ b/tests/system-traffic.at > @@ -340,6 +340,7 @@

Re: [ovs-dev] [PATCH v3 2/3] netdev-dpdk: Arbitrary 'dpdk' port naming

2016-12-21 Thread Kevin Traynor
On 12/21/2016 03:02 AM, Daniele Di Proietto wrote: > 2016-12-20 14:08 GMT-08:00 Kevin Traynor : >> On 12/15/2016 11:54 AM, Ciara Loftus wrote: >>> 'dpdk' ports no longer have naming restrictions. Now, instead of >>> specifying the dpdk port ID as part of the name, the PCI

Re: [ovs-dev] [PATCH 1/5] system-traffic: Introduce OVS_START_L7 macro.

2016-12-21 Thread Joe Stringer
On 20 December 2016 at 13:28, Joe Stringer wrote: > All of the commands starting L7 servers duplicate detailed specifics > which inhibits readability, and makes it difficult to ensure that the > servers are ready before the test proceeds. Add a new macro that > provides simpler

Re: [ovs-dev] splitting commits in Git (was: Re: Sync on PTAP, EXT-382 and NSH)

2016-12-21 Thread Jarno Rajahalme
> On Dec 21, 2016, at 9:30 AM, Ben Pfaff wrote: > > There was a question about splitting up large commits in Git. I do this > routinely and I volunteered to find some resources. > > The "git-rebase" manpage has a section titled "SPLITTING COMMITS" that > is a good place to

[ovs-dev] please help me for senario

2016-12-21 Thread Reza Safaei
hi I attached the used to create the queue but does not work according to my wishes. please guide me Thanks ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] reg: OpenVswitch Fields Implementation

2016-12-21 Thread Ben Pfaff
I'm trying to understand you here. You're saying that you added some code to Open vSwitch. It didn't work. Now, you're asking what's wrong with it. How can we know? We didn't write the code. On Wed, Dec 21, 2016 at 10:54:17PM +0530, Satyavalli Rama wrote: > > Could you please let us

[ovs-dev] splitting commits in Git (was: Re: Sync on PTAP, EXT-382 and NSH)

2016-12-21 Thread Ben Pfaff
There was a question about splitting up large commits in Git. I do this routinely and I volunteered to find some resources. The "git-rebase" manpage has a section titled "SPLITTING COMMITS" that is a good place to start. It does not cover very well how to commit parts of a file rather than a

Re: [ovs-dev] [PATCH] lacp: Select a may-enable IF as the lead IF

2016-12-21 Thread Ben Pfaff
On Thu, Dec 01, 2016 at 02:01:59PM +0100, Torgny Lindberg wrote: > A reboot of one switch in an MC-LAG bond makes all bond links > to go down, causing a total connectivity loss for 3 seconds. > > Packet capture shows that spurious LACP PDUs are sent to OVS with > a different MAC address (partner

[ovs-dev] [PATCH v2] lacp: Select a may-enable IF as the lead IF

2016-12-21 Thread Ben Pfaff
A reboot of one switch in an MC-LAG bond makes all bond links to go down, causing a total connectivity loss for 3 seconds. Packet capture shows that spurious LACP PDUs are sent to OVS with a different MAC address (partner system id) during the final stages of the MC-LAG switch reboot. The

Re: [ovs-dev] reg: OpenVswitch Fields Implementation

2016-12-21 Thread Ben Pfaff
Why are you telling us this? On Wed, Dec 21, 2016 at 10:38:08PM +0530, Satyavalli Rama wrote: > Hi Ben, > > Currently we are implementing OXS as an independent infrastructure. > While sending OXS Flow Stat Request, we are facing the below issue. > We have dumped and verified the packet is in

Re: [ovs-dev] reg: OpenVswitch Fields Implementation

2016-12-21 Thread Satyavalli Rama
Hi Ben, Currently we are implementing OXS as an independent infrastructure. While sending OXS Flow Stat Request, we are facing the below issue. We have dumped and verified the packet is in proper format only, but we are not receiving the Reply.

Re: [ovs-dev] [PATCH] hash: Update murmurhash repo link in comments

2016-12-21 Thread Ben Pfaff
On Wed, Dec 21, 2016 at 04:39:16PM +, Cian Ferriter wrote: > The MurmurHash code repo has moved from code.google to github. Update > the link to reflect this. > > Signed-off-by: Cian Ferriter Thanks, applied to master. ___

[ovs-dev] [PATCH] hash: Update murmurhash repo link in comments

2016-12-21 Thread Cian Ferriter
The MurmurHash code repo has moved from code.google to github. Update the link to reflect this. Signed-off-by: Cian Ferriter --- lib/hash.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/lib/hash.h b/lib/hash.h index f2dd510..d68ed75 100644

[ovs-dev] Recursos Humanos - 12 Temas

2016-12-21 Thread Compras - 12 Conferencias inéditas
12 conferencias pregrabadas / Durante 3 meses Póliza Indispensable para los Responsables de RECURSOS HUMANOS 12 conferencias en cada póliza, pregrabadas, inéditas, para capacitar a todo su personal Al adquirir la Póliza de Capacitación, usted obtiene acceso a 12 temas enfocados al

Re: [ovs-dev] [RFC v2] lib/table: add new flexible formatting

2016-12-21 Thread Ben Pfaff
On Wed, Dec 21, 2016 at 10:07:51AM -0500, Aaron Conole wrote: > Ben Pfaff writes: > > > On Tue, Dec 13, 2016 at 04:43:32PM -0500, Aaron Conole wrote: > >> Ben Pfaff writes: > >> > >> > On Mon, Dec 12, 2016 at 08:11:16PM -0500, Aaron Conole wrote: > >> >> Ben Pfaff

[ovs-dev] [PATCH v8] netdev-dpdk: Increase pmd thread priority

2016-12-21 Thread Bhanuprakash Bodireddy
Increase the DPDK pmd thread scheduling priority by lowering the nice value. This will advise the kernel scheduler to prioritize pmd thread over other processes. Signed-off-by: Bhanuprakash Bodireddy --- v7->v8: * Rebase * Update the documentation file

Re: [ovs-dev] [PATCH v3 05/16] ofp-actions: Add clone action.

2016-12-21 Thread William Tu
Hi Joe, I think we're missing the normal action for handling the non-IP traffic. Can you try this patch on top of the clone? --- a/tests/system-traffic.at +++ b/tests/system-traffic.at @@ -340,6 +340,7 @@ AT_CLEANUP AT_SETUP([datapath - clone action]) OVS_TRAFFIC_VSWITCHD_START()

[ovs-dev] [RFC PATCH] netdev-dpdk: Add OVS DPDK keep-alive functionality.

2016-12-21 Thread Bhanuprakash Bodireddy
This patch is aimed at achieving Fastpath Service Assurance in OVS-DPDK deployments. This commit adds support for monitoring the packet processing cores(pmd thread cores) by dispatching heartbeats at regular intervals. Incase of heartbeat miss the failure shall be detected & reported to higher

Re: [ovs-dev] [RFC v2] lib/table: add new flexible formatting

2016-12-21 Thread Aaron Conole
Ben Pfaff writes: > On Tue, Dec 13, 2016 at 04:43:32PM -0500, Aaron Conole wrote: >> Ben Pfaff writes: >> >> > On Mon, Dec 12, 2016 at 08:11:16PM -0500, Aaron Conole wrote: >> >> Ben Pfaff writes: >> >> >> >> > On Wed, Dec 07, 2016 at 02:15:12PM