Re: [ovs-dev] [PATCH 5/5] lib/netdev-dpdk: copy large packet to multi-segment mbufs

2016-11-01 Thread Michael Qiu
2016/10/28 19:17, Kavanagh, Mark B : Currently, one packet is only copied to one segment in function dpdk_do_tx_copy(), this could be an issue when a jumboframe comes, especially for multipile segments. Typos - 'jumboframe', 'multipile' This patch doesn't apply cleanly - change 'mbufs' to

Re: [ovs-dev] [PATCH 5/5] lib/netdev-dpdk: copy large packet to multi-segment mbufs

2016-11-01 Thread Michael Qiu
2016/10/28 19:17, Kavanagh, Mark B : Currently, one packet is only copied to one segment in function dpdk_do_tx_copy(), this could be an issue when a jumboframe comes, especially for multipile segments. Typos - 'jumboframe', 'multipile' This patch doesn't apply cleanly - change 'mbufs' to

Re: [ovs-dev] [PATCH] release: Propose a shorter release cycle for 2.7.

2016-11-01 Thread Russell Bryant
On Mon, Oct 31, 2016 at 10:53 PM, Justin Pettit wrote: > > > On Oct 31, 2016, at 12:22 PM, Russell Bryant wrote: > > > > > > > > On Mon, Oct 31, 2016 at 5:23 PM, Justin Pettit wrote: > > > >> > On Oct 29, 2016, at 9:19 AM, Russell Bryant

Re: [ovs-dev] [PATCH v2] doc: fix duplicated colons in rst documents

2016-11-01 Thread Russell Bryant
On Tue, Nov 1, 2016 at 5:12 AM, Mauricio Vasquez B < mauricio.vasq...@polito.it> wrote: > Signed-off-by: Mauricio Vasquez B > --- > FAQ.rst | 106 +- > > INSTALL.DPDK.rst | 74

[ovs-dev] [PATCH ovs V1 2/9] dpif-hw-acc: Add tc interface

2016-11-01 Thread Paul Blakey
Add tc interface in order to send/recv data from/to the HW. The kerenl side of tc will pass the tc messages to the driver and back. Signed-off-by: Paul Blakey Signed-off-by: Shahar Klein --- lib/automake.mk | 2 + lib/tc.c| 906

[ovs-dev] [PATCH ovs V1 7/9] dpif-hw-acc: operate implementation

2016-11-01 Thread Paul Blakey
added operate implemenation using tc for flow offload, supporting flow get, flow put, and flow del. Signed-off-by: Paul Blakey Signed-off-by: Shahar Klein --- lib/dpif-hw-acc.c | 821 +-

[ovs-dev] [PATCH ovs V1 3/9] dpif-hw-acc: Add new hash maps ufid <-> tc flow and ovs port <-> netdev

2016-11-01 Thread Paul Blakey
A flow in ovs can be identified by its ufid, and in tc by its flow handle + priority. We use the flow protocol number as the priority when offloading to tc. So we save a map that maps from ovs ufid <-> tc flow handle + protocol. Also added a map to map ovs in_port to/from netdevice to be used when

[ovs-dev] [PATCH ovs V1 5/9] dpif-hw-acc: converting a tc flow back to ovs flow

2016-11-01 Thread Paul Blakey
converts a dumped tc flow back to dpif-flow, for use in dumping/getting flows. Signed-off-by: Paul Blakey Signed-off-by: Shahar Klein --- lib/dpif-hw-acc.c | 200 ++ 1 file changed, 200 insertions(+)

[ovs-dev] [PATCH ovs V1 0/9] Introducing HW offload support for openvswitch

2016-11-01 Thread Paul Blakey
Openvswitch currently configures the kerenel datapath via netlink over an internal ovs protocol. This patch series offers a new provider: dpif-netlink-tc that uses the tc flower protocol to offload ovs rules into HW data-path through netdevices that e.g represent NIC e-switch ports. The user

Re: [ovs-dev] [PATCH] ofproto-dpif: Log warning when ct action or its variants are not supported.

2016-11-01 Thread Ben Pfaff
On Mon, Oct 31, 2016 at 05:23:25PM -0700, Joe Stringer wrote: > On 31 October 2016 at 14:33, Ben Pfaff wrote: > > On Mon, Oct 31, 2016 at 02:16:05PM -0700, Joe Stringer wrote: > >> On 31 October 2016 at 13:23, Ben Pfaff wrote: > >> > Some datapaths do not support the

[ovs-dev] [PATCH ovs V1 4/9] dpif-hw-acc: add a user offload policy framework

2016-11-01 Thread Paul Blakey
Added a new framework so user can specify which flows to try and offload. A new map was added to save each flow's policy. Signed-off-by: Paul Blakey Signed-off-by: Shahar Klein --- lib/automake.mk | 2 ++ lib/dpif-hw-acc.c | 2 ++

[ovs-dev] [PATCH ovs V1 6/9] dpif-hw-acc: using port maps on init and changes

2016-11-01 Thread Paul Blakey
updating the port maps on dpif init and when add/removing a port. Signed-off-by: Paul Blakey Signed-off-by: Shahar Klein --- lib/dpif-hw-acc.c | 49 + 1 file changed, 45 insertions(+), 4 deletions(-)

[ovs-dev] [PATCH ovs V1 1/9] dpif-hw-acc: New dpif provider

2016-11-01 Thread Paul Blakey
Added infrastructure for a new provider that will be able to send some flows to supporting HW for offloading. Signed-off-by: Paul Blakey Signed-off-by: Shahar Klein --- lib/automake.mk | 2 + lib/dpif-hw-acc.c | 444

Re: [ovs-dev] [PATCH] datapath: geneve: Handle vlan tag

2016-11-01 Thread Joe Stringer
On 31 October 2016 at 22:00, Pravin B Shelar wrote: > The compat vlan code ignores vlan tag for inner packet > on egress path. Following patch fixes this by inserting the > tag for inner packet before tunnel encapsulation. > > Signed-off-by: Pravin B Shelar Is

Re: [ovs-dev] [PATCH] datapath: geneve: Handle vlan tag

2016-11-01 Thread Pravin Shelar
On Tue, Nov 1, 2016 at 10:30 AM, Joe Stringer wrote: > On 31 October 2016 at 22:00, Pravin B Shelar wrote: >> The compat vlan code ignores vlan tag for inner packet >> on egress path. Following patch fixes this by inserting the >> tag for inner packet before tunnel

Re: [ovs-dev] [PATCH 1/2] ovsdb: implement read-only remote connection type

2016-11-01 Thread Ben Pfaff
On Tue, Oct 25, 2016 at 12:38:48PM -0400, Lance Richardson wrote: > Adds a new "read_only" column for remote connections. > > Operations that would alter the state of the database are not > permitted on connections for which the "read_only" column is set > to "true". > > Signed-off-by: Lance

Re: [ovs-dev] [PATCH 2/2] ovn-southbound: implement read-only connection support

2016-11-01 Thread Ben Pfaff
On Tue, Oct 25, 2016 at 04:31:15PM -0400, Lance Richardson wrote: > > From: "Lance Richardson" > > To: dev@openvswitch.org > > Sent: Tuesday, October 25, 2016 12:38:49 PM > > Subject: [ovs-dev] [PATCH 2/2] ovn-southbound: implement read-only > > connection support > > > >

Re: [ovs-dev] [PATCH v2] datapath: geneve: Handle vlan tag

2016-11-01 Thread Pravin Shelar
On Tue, Nov 1, 2016 at 11:41 AM, Joe Stringer wrote: > On 1 November 2016 at 11:35, Pravin B Shelar wrote: >> The compat vlan code ignores vlan tag for inner packet >> on egress path. Following patch fixes this by inserting the >> tag for inner packet before tunnel

[ovs-dev] [PATCH v2] datapath: geneve: Handle vlan tag

2016-11-01 Thread Pravin B Shelar
The compat vlan code ignores vlan tag for inner packet on egress path. Following patch fixes this by inserting the tag for inner packet before tunnel encapsulation. Signed-off-by: Pravin B Shelar --- datapath/linux/compat/geneve.c | 33 +++-- 1 file

[ovs-dev] [PATCH] ovn: fix ovn-northd leak in build_acls

2016-11-01 Thread Flavio Fernandes
Avoid leak due to missing ds_destroy in build_acl for non-stateful drop action. Found via inspection. I do not know that this fixes a common leak. Signed-off-by: Flavio Fernandes --- ovn/northd/ovn-northd.c | 1 + 1 file changed, 1 insertion(+) diff --git

[ovs-dev] [PATCH 1/2] Set release date for 2.6.1.

2016-11-01 Thread Justin Pettit
Signed-off-by: Justin Pettit --- NEWS | 3 ++- debian/changelog | 6 -- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/NEWS b/NEWS index 4326107..4c1f391 100644 --- a/NEWS +++ b/NEWS @@ -1,7 +1,8 @@ -v2.6.1 - xx xxx +v2.6.1 - 01 Nov 2016

[ovs-dev] [PATCH 2/2] Prepare for 2.6.2.

2016-11-01 Thread Justin Pettit
Signed-off-by: Justin Pettit --- NEWS | 4 configure.ac | 2 +- debian/changelog | 8 3 files changed, 13 insertions(+), 1 deletion(-) diff --git a/NEWS b/NEWS index 4c1f391..9970297 100644 --- a/NEWS +++ b/NEWS @@ -1,3 +1,7 @@ +v2.6.2 - xx xxx

Re: [ovs-dev] [PATCH] ovn: fix ovn-northd leak in build_acls

2016-11-01 Thread Ben Pfaff
On Tue, Nov 01, 2016 at 04:19:59PM -0500, Flavio Fernandes wrote: > Avoid leak due to missing ds_destroy in build_acl for non-stateful > drop action. > > Found via inspection. I do not know that this fixes a common leak. > > Signed-off-by: Flavio Fernandes Thanks! Applied

Re: [ovs-dev] [PATCH v1] debian: Include ovn-trace in ovn-common package

2016-11-01 Thread Ben Pfaff
On Fri, Oct 21, 2016 at 07:00:22PM -0500, Flavio Fernandes wrote: > Include the ovn-trace binary and it's man page in ovn-common package > > Fixes: 4acd1e87a8bf ("ovn-trace: New utility.") > Fixes: f448e5a4bb1f ("rhel: Include ovn-trace in ovn-common package") > Signed-off-by: Flavio Fernandes

Re: [ovs-dev] OVS 2.6.1 release?

2016-11-01 Thread Pravin Shelar
On Mon, Oct 31, 2016 at 1:30 PM, Justin Pettit wrote: > >> On Oct 31, 2016, at 6:10 AM, Richard Theis wrote: >> >> > So, it turns out my fix does not introduce any new problems. I have >> > just posted >> > it, so consider if you want to wait to apply it

Re: [ovs-dev] [PATCH 1/2] Set release date for 2.6.1.

2016-11-01 Thread Ben Pfaff
On Tue, Nov 01, 2016 at 03:44:12PM -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

[ovs-dev] Final Reminder for OVS Fall 2016 Conference

2016-11-01 Thread Justin Pettit
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 schedule is finalized and available here: https://events.linuxfoundation.org/events/open-vswitch-2016-fall-conference/program/agenda

Re: [ovs-dev] [PATCH 1/2] ovsdb windows: Allow online compacting

2016-11-01 Thread Ben Pfaff
On Thu, Oct 27, 2016 at 09:45:42PM +, Alin Serdean wrote: > This patch allows online compacting to be done under Windows. > > To achieve the above we need to close all file handles before trying to > rename the file, switch from rename to MoveFileEx (because rename/MoveFile > fails if the

Re: [ovs-dev] [PATCH 1/2] ovsdb windows: Allow online compacting

2016-11-01 Thread Ben Pfaff
On Tue, Nov 01, 2016 at 08:26:06PM -0700, Ben Pfaff wrote: > +/* Replace original file by the temporary file. > + * > + * We support two strategies: > + * > + * - The preferred strategy is to rename the temporary file over the > + * original one in-place, then

Re: [ovs-dev] [patch_v5 0/3] ovn: arp responder fixes and documentation.

2016-11-01 Thread Ben Pfaff
On Fri, Oct 21, 2016 at 01:36:36PM -0700, Darrell Ball wrote: > The first patch fixes arping from vtep-gw physical ports > and the code change was done by Ramu Ramamurthy. > > The second patch adds additional comments regarding arp responders > in ovn. This is intended to aid in understanding

Re: [ovs-dev] OVS 2.6.1 release?

2016-11-01 Thread Justin Pettit
> On Nov 1, 2016, at 12:09 PM, Pravin Shelar wrote: > > On Mon, Oct 31, 2016 at 1:30 PM, Justin Pettit wrote: >> >>> On Oct 31, 2016, at 6:10 AM, Richard Theis wrote: >>> So, it turns out my fix does not introduce any new problems. I

Re: [ovs-dev] [PATCH 2/2] tests: windows ovsdb online compact

2016-11-01 Thread Ben Pfaff
On Thu, Oct 27, 2016 at 09:45:42PM +, Alin Serdean wrote: > Skip symlinks checks on Windows. > > Signed-off-by: Alin Gabriel Serdean Applied, thanks! ___ dev mailing list dev@openvswitch.org

Re: [ovs-dev] [PATCH] datapath: geneve: Handle vlan tag

2016-11-01 Thread Joe Stringer
On 1 November 2016 at 10:48, Pravin Shelar wrote: > On Tue, Nov 1, 2016 at 10:30 AM, Joe Stringer wrote: >> On 31 October 2016 at 22:00, Pravin B Shelar wrote: >>> The compat vlan code ignores vlan tag for inner packet >>> on egress path.

Re: [ovs-dev] [PATCH] datapath: geneve: Handle vlan tag

2016-11-01 Thread Pravin Shelar
On Tue, Nov 1, 2016 at 11:06 AM, Joe Stringer wrote: > On 1 November 2016 at 10:48, Pravin Shelar wrote: >> On Tue, Nov 1, 2016 at 10:30 AM, Joe Stringer wrote: >>> On 31 October 2016 at 22:00, Pravin B Shelar wrote: The compat