[ovs-dev] [PATCH 1/3 v8] Windows: Add internal switch port per OVS bridge

From: Alin Serdean This patch updates the following commands in the vswitch: ovs-vsctl add-br br-test ovs-vsctl del-br br-test ovs-vsctl add-br br-test: This command will now create an internal port on the MSFT virtual switch using the WMI interface from

[ovs-dev] [PATCH 2/3 v8] datapath-windows: Add multiple switch internal ports

This patch adds multiple internal ports support to the windows datapath. All tunnels types have been updated to accommodate this new functionality. Signed-off-by: Alin Gabriel Serdean Co-authored-by: Sorin Vinturis

[ovs-dev] [PATCH 3/3 v8] Windows: document multiple NIC support setup

From: Alin Serdean This patch updates the documentation on how to set up OVS with multiple NICs. Also update the documentation to show users how new internal ports are created Signed-off-by: Alin Gabriel Serdean Acked-by: Paul

Re: [ovs-dev] [RFC: fedora ovn packaging 0/5] RFC fedora ovn packaging

On Tue, Dec 20, 2016 at 8:27 AM, Guru Shetty wrote: > > > On 15 December 2016 at 05:35, Andy Zhou wrote: > >> The goal of this series is to make it easier to set up OVN in >> a larger setup, i.e. not having all services running on a single host. >> >> Currently

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

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 --- Documentation/intro/install/general.rst | 2 + NEWS| 1 + Vagrantfile

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

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 address of the > device must be specified via the 'dpdk-devargs' option. eg. > > ovs-vsctl add-port br0 my-port > ovs-vsctl set

Re: [ovs-dev] [PATCH 2/3 v8] datapath-windows: Add multiple switch internal ports

Acked-by: Sairam Venugopal On 12/20/16, 11:42 AM, "Alin Serdean" wrote: > >This patch adds multiple internal ports support to the windows datapath. >All tunnels types have been updated to accommodate this new functionality. >

Re: [ovs-dev] OVS2.6+dpdk IPv6 Conntrack support

2016-12-19 21:25 GMT-08:00 Darrell Ball : > > > From: Jarno Rajahalme > Date: Monday, December 19, 2016 at 2:30 PM > To: Darrell Ball > Cc: Sridhar Gaddam , "ovs-dev@openvswitch.org" > , Srikanth

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

Signed-off-by: Joe Stringer --- tests/system-ovn.at | 2 ++ tests/system-traffic.at | 3 +++ 2 files changed, 5 insertions(+) diff --git a/tests/system-ovn.at b/tests/system-ovn.at index 7b35e84a43d2..022dc63f22c2 100644 --- a/tests/system-ovn.at +++ b/tests/system-ovn.at @@

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

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 semantics from the test perspective and hide the details in the macro. A

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

Justin, I'd like to get this feature into OVS 2.7. Will you review these patches? Thanks, Ben. On Tue, Dec 20, 2016 at 08:07:49AM -0800, Ben Pfaff wrote: > v1->v2: > - Patch 1 in v1 was committed and therefore dropped from the series. > - Patch 2 in v2 is new. > > v2->v3: > - Rebase to

Re: [ovs-dev] [PATCH RFC] dpcls: Avoid one 8-byte chunk in subtable mask.

> On Dec 19, 2016, at 2:43 PM, antonio.fische...@intel.com wrote: > > This patch skips the chunk comprising of dp_hash and in_port in the > subtable mask when the dpcls refers to a physical port. This will > slightly speed up the hash computation as one expensive function call > to hash_add64()

[ovs-dev] Compras - 12 Conferencias inéditas

12 conferencias pregrabadas / Durante 3 meses Póliza indispensable para los responsables de COMPRAS 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 área de

[ovs-dev] [PATCH] ovn: Log chassis claiming lport only when changes occur

With recent OVN commits, the logic for a chassis to claim or release a logical port was consolidated. This is a good thing. However, there was a logic change that resulted in VLOG_INFO being generated every time on the ovn-controller. This patch changes the logic so that VLOG_INFO is only

Re: [ovs-dev] [PATCH] ovn-controller: Fix conntrack zone in gateway routers.

On Tue, Dec 20, 2016 at 08:21:28AM -0800, Guru Shetty wrote: > On 19 December 2016 at 21:26, Ben Pfaff wrote: > > > -ct->zone_src.field = mf_from_id(MFF_LOG_CT_ZONE); > > > +ct->zone_src.field = ep->is_switch ? mf_from_id(MFF_LOG_CT_ZONE) > > > +:

Re: [ovs-dev] [PATCH 1/3 v8] Windows: Add internal switch port per OVS bridge

Acked-by: Sairam Venugopal On 12/20/16, 11:41 AM, "Alin Serdean" wrote: >From: Alin Serdean > >This patch updates the following commands in the vswitch: > ovs-vsctl add-br br-test > ovs-vsctl del-br br-test

[ovs-dev] [PATCH 2/2] conntrack: Return NEW for IPv6 ND packets without tracking.

The userspace connection tracker treats Neighbor Discovery packets as invalid, because they're not checked against any connection. This in inconsistent with the kernel connection tracker which always returns 'CS_NEW'. Therefore, this commit makes the userspace connection tracker conforming with

[ovs-dev] [PATCH 4/5] system-traffic: Reorder and bannerize ct tests.

Signed-off-by: Joe Stringer --- tests/system-traffic.at | 898 1 file changed, 449 insertions(+), 449 deletions(-) diff --git a/tests/system-traffic.at b/tests/system-traffic.at index 14fbf4518fdf..8e424c56031c 100644 ---

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

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, needed before we can switch to curl. Signed-off-by: Joe Stringer --- tests/system-common-macros.at | 4

Re: [ovs-dev] [PATCH 3/3 v8] Windows: document multiple NIC support setup

On 20 December 2016 at 11:42, Alin Serdean wrote: > From: Alin Serdean > > This patch updates the documentation on how to set up OVS with multiple > NICs. > > Also update the documentation to show users how new internal ports are

Re: [ovs-dev] [PATCH 2/2] conntrack: Return NEW for IPv6 ND packets without tracking.

On 12/20/16, 12:25 PM, "ovs-dev-boun...@openvswitch.org on behalf of Daniele Di Proietto" wrote: The userspace connection tracker treats Neighbor Discovery packets as invalid, because they're not checked against any

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

Yep, I'd be happy to. --Justin > On Dec 20, 2016, at 2:44 PM, Ben Pfaff wrote: > > Justin, I'd like to get this feature into OVS 2.7. Will you review > these patches? > > Thanks, > > Ben. > > On Tue, Dec 20, 2016 at 08:07:49AM -0800, Ben Pfaff wrote: >> v1->v2: >> - Patch 1

[ovs-dev] Competencias del Comprador Exitoso

Póliza de Indispensable para los Responsables de Compras 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 área de Compras; temas especializados que han sido

Re: [ovs-dev] [PATCH 2/2] conntrack: Return NEW for IPv6 ND packets without tracking.

On 20/12/2016 17:14, "Joe Stringer" wrote: >On 20 December 2016 at 14:46, Darrell Ball wrote: >> >> >> On 12/20/16, 12:25 PM, "ovs-dev-boun...@openvswitch.org on behalf of Daniele >> Di Proietto" >

Re: [ovs-dev] [PATCH 2/2] conntrack: Return NEW for IPv6 ND packets without tracking.

On 12/20/16, 5:37 PM, "Daniele Di Proietto" wrote: On 20/12/2016 14:46, "Darrell Ball" wrote: > > >On 12/20/16, 12:25 PM, "ovs-dev-boun...@openvswitch.org on behalf of Daniele Di Proietto"

Re: [ovs-dev] ovn-northd: ovsdb idl question

On Tue, Dec 20, 2016 at 01:59:02PM -0500, Lance Richardson wrote: > I've been investigating a crash in ovn-northd. This is very easy to > reproduce by inserting a row into the NB Logical_Switch_Port table > without setting the "name" column, e.g.: > > ovn-nbctl ls-add ls0 > ovn-nbctl --id=@foo

Re: [ovs-dev] [PATCH] tests: Regenerate atlocal when running tests.

On Tue, Dec 20, 2016 at 03:18:43PM -0800, Joe Stringer wrote: > On 20 December 2016 at 13:16, Ben Pfaff wrote: > > A previous patch fixed double rebuilds when running tests in some cases. > > That patch removed dependencies from targets in tests/automake.mk that > > were redundant

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

Hi Joe, Thanks I will take a look. William On Tue, Dec 20, 2016 at 10:25 AM, Joe Stringer wrote: > On 18 December 2016 at 00:18, Ben Pfaff wrote: >> From: William Tu >> >> This patch adds OpenFlow clone action with syntax as below: >>

Re: [ovs-dev] [PATCH] ovn: Log chassis claiming lport only when changes occur

On Tue, Dec 20, 2016 at 01:23:46PM -0800, Mickey Spiegel wrote: > With recent OVN commits, the logic for a chassis to claim > or release a logical port was consolidated. This is a > good thing. However, there was a logic change that > resulted in VLOG_INFO being generated every time on the >

[ovs-dev] [PATCH] ovsdb-idlc: Initialize nonnull string columns for inserted rows.

When a schema column has type "exactly one string", the corresponding struct member has type "char *" and the documented and expected behavior is that the string should always be nonnull. (The code generator even adds a comment /* Always nonnull. */ in the struct definition.) In the case where a

Re: [ovs-dev] [PATCH] tests: Regenerate atlocal when running tests.

On 20 December 2016 at 16:28, Ben Pfaff wrote: > On Tue, Dec 20, 2016 at 03:18:43PM -0800, Joe Stringer wrote: >> On 20 December 2016 at 13:16, Ben Pfaff wrote: >> > A previous patch fixed double rebuilds when running tests in some cases. >> > That patch removed

[ovs-dev] [PATCH] ovs-advanced.rst: Add missing \ to a few examples.

This makes these examples easier to cut and paste into a terminal, and makes them consistent with the other examples. Signed-off-by: Ben Pfaff --- Documentation/tutorials/ovs-advanced.rst | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

[ovs-dev] pmd->flow_table

(changed the subj) Could anybody help confirm about pmd->flow_table below? -Thanks- -- Forwarded message -- From: Joo Kim Date: Mon, Dec 19, 2016 at 4:35 PM Subject: Re: [ovs-dev] relationship between dpcls_rule and dp_netdev_flow To: Jarno Rajahalme

Re: [ovs-dev] [PATCH 2/2] conntrack: Return NEW for IPv6 ND packets without tracking.

On 20 December 2016 at 14:46, Darrell Ball wrote: > > > On 12/20/16, 12:25 PM, "ovs-dev-boun...@openvswitch.org on behalf of Daniele > Di Proietto" diproiet...@vmware.com> wrote: > @@ -864,25 +896,32 @@

Re: [ovs-dev] pmd->flow_table

2016-12-20 16:49 GMT-08:00 Joo Kim : > (changed the subj) > > Could anybody help confirm about pmd->flow_table below? -Thanks- > > -- Forwarded message -- > From: Joo Kim > Date: Mon, Dec 19, 2016 at 4:35 PM > Subject: Re: [ovs-dev]

Re: [ovs-dev] [PATCH] ovsdb-idlc: Initialize nonnull string columns for inserted rows.

On Tue, Dec 20, 2016 at 07:32:58PM -0500, Lance Richardson wrote: > > > - Original Message - > > From: "Ben Pfaff" > > To: d...@openvswitch.org > > Cc: "Ben Pfaff" , "Lance Richardson" > > Sent: Tuesday, December 20, 2016 7:17:31 PM > >

Re: [ovs-dev] [PATCH] ovs-vsctl: Print error when add-port fails.

On Tue, Dec 20, 2016 at 06:49:24PM -0800, Daniele Di Proietto wrote: > When the add-port command fails, vsctl reports the failure and just > suggests to check the logs for more details. > > ovs-vswitchd fills the error column in the Interface table with a > description of the error, so it might

[ovs-dev] [PATCH v2 3/4] ofproto-dpif: Break trace functionality into a separate source file.

An upcoming commit will rewrite much of the ofproto/trace functionality. As the mechanism behind this grows and evolves, it makes sense to move it into its own file, especially since ofproto-dpif.c is too big anyway. Signed-off-by: Ben Pfaff --- ofproto/automake.mk | 2

Re: [ovs-dev] [PATCH 4/4] [RFC] ofproto-dpif: Make ofproto/trace output easier to read.

I posted a new version of this series that should be ready for review: https://mail.openvswitch.org/pipermail/ovs-dev/2016-December/326549.html ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] OVS2.6+dpdk IPv6 Conntrack support

Thanks Daniele for addressing the issue and fixing the inconsistency. Best Regards, --Sridhar. On Wed, Dec 21, 2016 at 2:05 AM, Daniele Di Proietto wrote: > 2016-12-19 21:25 GMT-08:00 Darrell Ball : > > > > > > From: Jarno Rajahalme > >

[ovs-dev] [PATCH v2 2/4] ofproto-dpif: Unhide structure contents.

Until now, ofproto-dpif.c has hidden the definitions of several structures, such as struct ofproto_dpif and struct rule_dpif. This kind of information hiding is often beneficial, because it forces code outside the file with the definition to use the documented interfaces. However, in this case

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

This series seeks to make the output of ofproto/trace easier to read, especially in complicated situations. v1->v2: - Patch 4 extended and revised to make it suitable for review and no longer RFC. I hope that this series pleases many users. Ben Pfaff (4): ofproto: Update access

Re: [ovs-dev] [PATCH] ovsdb-idl: Properly handle conditional monitor update error

I have posted an updated version of this patch at: https://patchwork.ozlabs.org/patch/707398/ On Fri, Dec 16, 2016 at 4:55 PM, Andy Zhou wrote: > When generating conditional monitoring update request, current code > failed to update idl's 'request-id'. This bug causes the reply

Re: [ovs-dev] [PATCH] netdev-dpdk: Use intermediate queue during packet transmission.

On 19.12.2016 21:05, Bodireddy, Bhanuprakash wrote: > Thanks Ilya and Aaron for reviewing this patch and providing your comments, > my reply inline. > >> -Original Message- >> From: Ilya Maximets [mailto:i.maxim...@samsung.com] >> Sent: Monday, December 19, 2016 8:41 AM >> To: Aaron

[ovs-dev] [PATCH 3/3] ovsdb-idl: Enhance conditional monitoring API

To allow client to know when the conditional monitoring changes has been accepted by the OVSDB server and the 'idl' contents has been updated to match the new conditions. Signed-off-by: Andy Zhou --- lib/ovsdb-idl.c| 49 ++---

[ovs-dev] [PATCH 2/3] ovsdb-idl: Properly handle conditional monitor update error

From: andy zhou When generating conditional monitoring update request, current code failed to update idl's 'request-id'. This bug causes the reply message of the update request, regardless an ACK or a NACK, be logged as an unexpected message at the debug level and ignored by the

Re: [ovs-dev] [PATCH] netdev-dpdk: Use intermediate queue during packet transmission.

On 20.12.2016 15:19, Bodireddy, Bhanuprakash wrote: >> -Original Message- >> From: Ilya Maximets [mailto:i.maxim...@samsung.com] >> Sent: Tuesday, December 20, 2016 8:09 AM >> To: Bodireddy, Bhanuprakash ; Aaron >> Conole >> Cc:

Re: [ovs-dev] [PATCH v3 3/4] datapath-windows: Conntrack - Introduce support for tracking related connections

Acked-by: Alin Gabriel Serdean > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Sairam Venugopal > Sent: Saturday, December 17, 2016 12:28 AM > To: d...@openvswitch.org > Subject:

Re: [ovs-dev] [PATCH v3 0/4] datapath-windows: Enable support for tracking FTP connections

Thanks for the series! > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Sairam Venugopal > Sent: Saturday, December 17, 2016 12:28 AM > To: d...@openvswitch.org > Subject: [ovs-dev] [PATCH v3 0/4] datapath-windows:

Re: [ovs-dev] [PATCH] ovn-controller: Fix conntrack zone in gateway routers.

On 19 December 2016 at 21:26, Ben Pfaff wrote: > On Mon, Dec 19, 2016 at 04:12:17AM -0800, Gurucharan Shetty wrote: > > The gateway router was using the ct_next action to > > reassemble packets. But ct_next action by default would > > use the zone allocated for a logical port and

Re: [ovs-dev] [RFC: fedora ovn packaging 0/5] RFC fedora ovn packaging

On 15 December 2016 at 05:35, Andy Zhou wrote: > The goal of this series is to make it easier to set up OVN in > a larger setup, i.e. not having all services running on a single host. > > Currently ovn-central package starts and stop all three services > northbound-db, northd and