[ovs-dev] 答复: RE: 答复: Re: 答复: Re: 答复: Re: [PATCH] ovn-controller: Support vxlan tunnel in ovn

ovn-controller-vtep could only be used to forwards traffic between networks managed by openstack and physical network openstack not managed. We want to use ovn in the scenary that ovs-computer node and sriov computer node all managed by openstack. Waiting for your suggestions. Thanks

Re: [ovs-dev] [PATCH 1/2] Revert "tunneling: Avoid recirculation on datapath."

On 8 May 2017 at 17:35, William Tu wrote: > Hi Joe and Greg, > > Maybe it's better I put this revert tunneling patch (1/2) and its > tunnel-tests (2/2) in one patch, so the "make check" can pass? They can be separate. It's currently broken, the revert will fix it. The test

Re: [ovs-dev] 答复: Re: 答复: Re: 答复: Re: [PATCH] ovn-controller: Support vxlan tunnel in ovn

Hi Qianyu, Did you try the ovn-controller-vtep before? You can use the ovn-controller-vtep with HW switch to setup the vxlan tunnel with host which run ovn-controller. -Original Message- From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] On Behalf Of

Re: [ovs-dev] [PATCH 1/2] Revert "tunneling: Avoid recirculation on datapath."

On Mon, 2017-05-08 at 17:35 -0700, William Tu wrote: > Hi Joe and Greg, > > Maybe it's better I put this revert tunneling patch (1/2) and its > tunnel-tests (2/2) in one patch, so the "make check" can pass? > > Regards, > William I don't understand... the revert has to be it's own separate

Re: [ovs-dev] [PATCHv2] tunnel-tests: Add test to match tunnel traffic.

If you add this second incremental, it is explicit that you want to match on the outer header as part of this test (to check the present breakage). It also makes it clear which packets are involved and needed in this test. diff --git a/tests/tunnel-push-pop.at b/tests/tunnel-push-pop.at index

[ovs-dev] 答复: Re: 答复: Re: 答复: Re: [PATCH] ovn-controller: Support vxlan tunnel in ovn

Hi Ben and Mikey, thank you for your review and analysis. As we discribed below, vxlan is used very common, and the use case that we mentioned below is typical architecture of telecom operators' networks. So, we think it is very necessary to support vxlan between ovs and HW switch. If ovn does

Re: [ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

On 5/8/17, 5:24 PM, "William Tu" wrote: [snip] > -dnl Check ARP Snoop > +dnl Use arp reply to achieve tunnel next hop mac binding > AT_CHECK([ovs-appctl netdev-dummy/receive br0

Re: [ovs-dev] [PATCH 1/2] Revert "tunneling: Avoid recirculation on datapath."

Hi Joe and Greg, Maybe it's better I put this revert tunneling patch (1/2) and its tunnel-tests (2/2) in one patch, so the "make check" can pass? Regards, William On Mon, May 8, 2017 at 11:34 AM, Greg Rose wrote: > On Mon, 2017-05-08 at 11:15 -0700, Joe Stringer wrote: >>

[ovs-dev] [PATCHv2] tunnel-tests: Add test to match tunnel traffic.

This test highlights a bug that was affecting master up until the previous patch. Put simply, we have two bridges: an integration bridge which contains a tunnel, and a physical bridge for underlay network connectivity. This test simulates putting UDP traffic through the integration bridge, with

Re: [ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

[snip] > -dnl Check ARP Snoop > +dnl Use arp reply to achieve tunnel next hop mac binding > AT_CHECK([ovs-appctl netdev-dummy/receive br0 > 'recirc_id(0),in_port(100),eth(src=f8:bc:12:44:34:b6,dst=ff:ff:ff:ff:ff:ff),eth_type(0x0806),arp(sip=1.1.2.92,ti > > AT_CHECK([ovs-appctl tnl/neigh/show |

Re: [ovs-dev] [PATCH v6 3/4] datapath-windows: NAT integration with conntrack

Hi Sai, Thanks for the feedback! When I removed "static __inline", I meant to make the function public. It's not a matter of coding standard or style, but a matter of feature. Please note the changes I made to Actions.h as well. The functions I made public are utility functions. It's not

Re: [ovs-dev] [PATCH] tunnel-tests: Add test to match tunnel traffic.

I sent out a suggested incremental Thanks Darrell On 5/8/17, 5:09 PM, "ovs-dev-boun...@openvswitch.org on behalf of William Tu" wrote: This test highlights a bug that was affecting master up until the previous patch. Put

Re: [ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

I added in the following incremental which eliminates unnecessary/misleading parts, fixes some misleading/incorrect comments and removes redundant parts. dball@ubuntu:~/ovs$ git diff tests/tunnel-push-pop.at diff --git a/tests/tunnel-push-pop.at b/tests/tunnel-push-pop.at index c772467..654e622

[ovs-dev] [PATCH] tunnel-tests: Add test to match tunnel traffic.

This test highlights a bug that was affecting master up until the previous patch. Put simply, we have two bridges: an integration bridge which contains a tunnel, and a physical bridge for underlay network connectivity. This test simulates putting UDP traffic through the integration bridge, with

Re: [ovs-dev] [PATCH v6 3/4] datapath-windows: NAT integration with conntrack

Hi Yin, Thanks for the patches. I had some immediate review comments. - OvsUpdateAddressAndPort does not have a return type - Refrain from changing other code that your patch does not require. Please send out separate incremental patch. - This is the correct way to define static inline

Re: [ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

Thanks. I will re-submit this test. William On Mon, May 8, 2017 at 3:56 PM, Joe Stringer wrote: > On 8 May 2017 at 11:35, Greg Rose wrote: >> On Mon, 2017-05-08 at 11:15 -0700, Joe Stringer wrote: >>> From: William Tu >>> >>> This test

Re: [ovs-dev] [PATCH v6 4/4] Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com>

Yin, Can you rebase and rebuild your changes? OvsTcpSegmentNBL arguments have changed and this patch breaks compilation. Can you add commit msgs to your patches? This commit title says: "Signed-off-by:Alin Gabriel…” Can you fix this? Thanks, Sairam On 5/8/17, 3:38 PM,

Re: [ovs-dev] [PATCH v6 2/4] datapath-windows: Add NAT module in conntrack

Yin, thanks for the patches. Please briefly describe in the commit message the scope of this patch. Same applies to other patches in this series. Thanks, Shashank From: ovs-dev-boun...@openvswitch.org on behalf of Yin Lin

Re: [ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

On 8 May 2017 at 11:35, Greg Rose wrote: > On Mon, 2017-05-08 at 11:15 -0700, Joe Stringer wrote: >> From: William Tu >> >> This test highlights a bug that was affecting master up until the >> previous patch. Put simply, we have two bridges: an

[ovs-dev] [PATCH v6 4/4] Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com>

From: Alin Gabriel Serdean --- datapath-windows/ovsext/Actions.c | 38 ++ 1 file changed, 38 insertions(+) diff --git a/datapath-windows/ovsext/Actions.c b/datapath-windows/ovsext/Actions.c index d1938f3..bb1e6ea 100644 ---

[ovs-dev] [PATCH v6 3/4] datapath-windows: NAT integration with conntrack

Signed-off-by: Yin Lin --- datapath-windows/automake.mk | 4 +- datapath-windows/ovsext/Actions.c | 118 - datapath-windows/ovsext/Actions.h | 20 datapath-windows/ovsext/Conntrack.c| 187 +

[ovs-dev] [PATCH v6 2/4] datapath-windows: Add NAT module in conntrack

Signed-off-by: Yin Lin Issue: # Change-Id: I6f37360c36525548b343f0016304015fec8aba7d --- datapath-windows/automake.mk| 2 + datapath-windows/ovsext/Conntrack-nat.c | 424 datapath-windows/ovsext/Conntrack-nat.h | 39 +++ 3 files

[ovs-dev] [PATCH v6 1/4] datapath-windows: Add support for NAT in conntrack

From: Anand Kumar Add support for parsing netlink attributes related to NAT in conntrack. Co-Authored-by: Anand Kumar Co-Authored-by: Darrell Ball Signed-off-by: Yin Lin --- datapath-windows/ovsext/Conntrack.c

Re: [ovs-dev] [PATCH] ovn-openstack.rst: Fix typo.

On Mon, May 08, 2017 at 02:51:21PM -0700, Andy Zhou wrote: > On Mon, May 8, 2017 at 1:53 PM, Ben Pfaff wrote: > > The text here was inconsistent: it referred to port 4 in the text just > > above but the example used port 5 in one place. This fixes the issue. > > > > Signed-off-by:

Re: [ovs-dev] [PATCH] ovn-openstack.rst: Fix typo.

On Mon, May 8, 2017 at 1:53 PM, Ben Pfaff wrote: > The text here was inconsistent: it referred to port 4 in the text just > above but the example used port 5 in one place. This fixes the issue. > > Signed-off-by: Ben Pfaff Acked-by: Andy Zhou I must

Re: [ovs-dev] [PATCH v5] tunneling: Avoid recirculation on datapath by computing the recirculate actions at translate time.

On Mon, May 8, 2017 at 11:29 AM, Zoltán Balogh wrote: > Hi William, > > The reason of 'incorrect' n_bytes stats could be due to the mechanism truncate > and tunneling with clone action do work. As you wrote, the packet size is > changed when the output action is

Re: [ovs-dev] [PATCH 0/3] Support OpenFlow 1.5 packet-out

On Mon, May 08, 2017 at 10:24:16AM -0700, Yi-Hung Wei wrote: > Hi Ben, > > Thanks for your valuable review. Yes, it makes sense to use 'struct flow' > instead of 'struct match' to represent metadata. > > As for the "pipeline fields", I briefly look at ovs-fields (7), and I think > the > patch

[ovs-dev] [PATCH] ovn-openstack.rst: Fix typo.

The text here was inconsistent: it referred to port 4 in the text just above but the example used port 5 in one place. This fixes the issue. Signed-off-by: Ben Pfaff --- Documentation/tutorials/ovn-openstack.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [ovs-dev] [PATCH 4/6] ovsdb: add support for role-based access controls

On Mon, May 08, 2017 at 01:35:54PM -0400, Lance Richardson wrote: > > From: "Ben Pfaff" > > To: "Lance Richardson" > > Cc: d...@openvswitch.org > > > As a high-level comment, it looks to me like documentation is missing > > for the ways that this affects the

Re: [ovs-dev] [PATCH] checkpatch: Fix inconsistencies skipping datapath files.

On Mon, May 08, 2017 at 03:16:55PM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > The code in checkpatch inconsistently stripped "a/" or "b/" from the > > beginning of a file name, and the check for "datapath" only worked when > > the prefix was not stripped. This fixes the

Re: [ovs-dev] [PATCH] checkpatch: Fix inconsistencies skipping datapath files.

Ben Pfaff writes: > The code in checkpatch inconsistently stripped "a/" or "b/" from the > beginning of a file name, and the check for "datapath" only worked when > the prefix was not stripped. This fixes the problem. > > CC: Aaron Conole > Signed-off-by: Ben

Re: [ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

On Mon, 2017-05-08 at 11:15 -0700, Joe Stringer wrote: > From: William Tu > > This test highlights a bug that was affecting master up until the > previous patch. Put simply, we have two bridges: an integration bridge > which contains a tunnel, and a physical bridge for

Re: [ovs-dev] [PATCH 1/2] Revert "tunneling: Avoid recirculation on datapath."

On Mon, 2017-05-08 at 11:15 -0700, Joe Stringer wrote: > This reverts commit f1dac5128ce6db2e493f0d1c7a8b53fb9f34476f. When this > commit was introduced, it broke the 'make check-system-userspace' > testsuite. It appears that the new translation fails to modify the flow > in a way that would

Re: [ovs-dev] [PATCH v5] tunneling: Avoid recirculation on datapath by computing the recirculate actions at translate time.

Hi William, The reason of 'incorrect' n_bytes stats could be due to the mechanism truncate and tunneling with clone action do work. As you wrote, the packet size is changed when the output action is applied. Let's say, I have the config below: ns1 | +--o-+ |

[ovs-dev] [PATCH 2/2] tunnel-tests: Add test to match tunnel traffic.

From: William Tu This test highlights a bug that was affecting master up until the previous patch. Put simply, we have two bridges: an integration bridge which contains a tunnel, and a physical bridge for underlay network connectivity. This test simulates putting UDP traffic

[ovs-dev] [PATCH 1/2] Revert "tunneling: Avoid recirculation on datapath."

This reverts commit f1dac5128ce6db2e493f0d1c7a8b53fb9f34476f. When this commit was introduced, it broke the 'make check-system-userspace' testsuite. It appears that the new translation fails to modify the flow in a way that would represent the flow as an encapsulated flow when the traffic is

[ovs-dev] [PATCH 0/2] Revert recirculation on datapath.

Currently, the native (userspace) tunneling functionality is not appropriately applying tunnel headers during translation. In a typical 2-bridge configuration with an integration bridge which connects to a tunnel port, plus an underlay bridge which provides connectivity to the physical network,

Re: [ovs-dev] [PATCH 4/6] ovsdb: add support for role-based access controls

> From: "Ben Pfaff" > To: "Lance Richardson" > Cc: d...@openvswitch.org > As a high-level comment, it looks to me like documentation is missing > for the ways that this affects the schema and the wire protocol. We try > to document those kinds of changes,

Re: [ovs-dev] [PATCH 0/3] Support OpenFlow 1.5 packet-out

Hi Jan, Thanks for letting me know, and sorry that I did not aware of this patch [1]. I take a look at patch [1], and it seems like the common part of [1] and [2] are 1) Defining OF 1.5 packet out format 2) Decoding and encoding OF 1.5 packet out message On the other hand, [1] focuses on

Re: [ovs-dev] [PATCH 0/3] Support OpenFlow 1.5 packet-out

Hi Ben, Thanks for your valuable review. Yes, it makes sense to use 'struct flow' instead of 'struct match' to represent metadata. As for the "pipeline fields", I briefly look at ovs-fields (7), and I think the patch series should be update to include at least the following fields. * Tunnel

Re: [ovs-dev] [PATCH branch-2.7] docs: Add dpdk stable release to DPDK install docs.

On 5/8/17, 9:21 AM, "Stokes, Ian" wrote: > Ian > > This patch does not apply to 2.7; could you respin ? > > Thanks Darrell Hi Darrell, Thanks for looking at this, I think this patch was superseded by a later patch, checking branch

Re: [ovs-dev] [PATCH branch-2.7] docs: Add dpdk stable release to DPDK install docs.

> Ian > > This patch does not apply to 2.7; could you respin ? > > Thanks Darrell Hi Darrell, Thanks for looking at this, I think this patch was superseded by a later patch, checking branch 2.7 it already has a commit that makes these changes 3648ff4f614d6e50f896cb28e234d7d0ab33d167 Thanks

Re: [ovs-dev] RFC: ovs-dump-flows utility

Hi Ben, Thanks for the look! Ben Pfaff writes: > On Fri, Apr 28, 2017 at 04:44:32PM -0400, Aaron Conole wrote: >> Greetings dev, >> >> I have whipped up a quick little utility (find below), that I've done a >> bit of debugging with and it seems to have made working with

Re: [ovs-dev] [RFC PATCH 0/6] Change dpdk rxq scheduling to incorporate rxq processing cycles.

On Mon, 2017-05-08 at 11:49 +0100, Kevin Traynor wrote: > On 05/06/2017 02:01 AM, Greg Rose wrote: > > On Fri, May 5, 2017 at 9:34 AM, Kevin Traynor wrote: > >> Rxqs are scheduled to be handled across available pmds in round robin > >> order with no weight or priority. > >> >

Re: [ovs-dev] [PATCH v5] tunneling: Avoid recirculation on datapath by computing the recirculate actions at translate time.

Hi Zoltan, Yes, we disallow truncate followed by patch port as output on purpose. The reason is that truncate action sets the packet's new length in its metadata instead of immediately change the size. Actual change of the packet size happens when we see the output action. Carrying this

Re: [ovs-dev] checkpatch name checking

On Mon, May 08, 2017 at 10:33:19AM -0400, Aaron Conole wrote: > Ben Pfaff writes: > > > Hi Aaron, checkpatch currently tries to ignores files in the "datapath" > > directories but it's not entirely successful. I think that's because, > > in the "parse == 1" case, it doesn't strip

[ovs-dev] [PATCH] checkpatch: Fix inconsistencies skipping datapath files.

The code in checkpatch inconsistently stripped "a/" or "b/" from the beginning of a file name, and the check for "datapath" only worked when the prefix was not stripped. This fixes the problem. CC: Aaron Conole Signed-off-by: Ben Pfaff ---

Re: [ovs-dev] RFC: ovs-dump-flows utility

On Fri, Apr 28, 2017 at 04:44:32PM -0400, Aaron Conole wrote: > Greetings dev, > > I have whipped up a quick little utility (find below), that I've done a > bit of debugging with and it seems to have made working with dump-flows > from ovs-ofctl a little easier to use. > > If you think it's

[ovs-dev] [PATCH] docs: Improve formatting for daemon options in a few manpages.

daemon.man is meant to have a heading above it, but in a few manpages its text was running directly into the previous documentation because this had been overlooked. By adding .PP to daemon.man, we make this problem less severe if the heading is similarly omitted in future manpages, since at

Re: [ovs-dev] [PATCH v5 1/6] userspace: Support for push_eth and pop_eth actions

On Sat, May 06, 2017 at 03:49:43PM +, Zoltán Balogh wrote: > From: Jan Scheurich > > Add support for actions push_eth and pop_eth to the netdev datapath and > the supporting libraries. This patch relies on the support for these actions > in the kernel datapath to

Re: [ovs-dev] [PATCH 2/2] rhel: delete transient ports on boot when starting ovsdb-server

On 05/05/2017 08:41 PM, Aaron Conole wrote: [...] >> @@ -534,7 +538,7 @@ fi >> %doc COPYING NOTICE README.rst NEWS rhel/README.RHEL.rst >> /var/lib/openvswitch >> /var/log/openvswitch >> -%ghost %attr(755,root,root) %{_rundir}/openvswitch >> +%dir %{_rundir}/openvswitch > > Doesn't this still

Re: [ovs-dev] checkpatch name checking

Ben Pfaff writes: > Hi Aaron, checkpatch currently tries to ignores files in the "datapath" > directories but it's not entirely successful. I think that's because, > in the "parse == 1" case, it doesn't strip a leading "a/" or "b/" from > filenames: > current_file

Re: [ovs-dev] [PATCH v3] Windows: Secure the namedpipe implementation

Thanks for figuring this out! I applied this to all of those branches. On Sun, May 07, 2017 at 03:38:18PM +, Alin Serdean wrote: > Thanks a lot for the patch! Me and Sai talked offline and we will add the > creator (owner) user to be added in another incremental. > > Could you please apply

Re: [ovs-dev] [PATCH v8 5/5] datapath-windows: Fragment NBL based on MRU size

Thanks a lot Anand and Alin! I applied all of these to master. On Sat, May 06, 2017 at 01:45:11AM +, Alin Serdean wrote: > Acked-by: Alin Gabriel Serdean > > > -Original Message- > > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > >

[ovs-dev] checkpatch name checking

Hi Aaron, checkpatch currently tries to ignores files in the "datapath" directories but it's not entirely successful. I think that's because, in the "parse == 1" case, it doesn't strip a leading "a/" or "b/" from filenames: current_file = match.group(2) whereas in the "parse ==

[ovs-dev] [PATCH] NEWS: Reorganize and edit recent items for clarity.

Signed-off-by: Ben Pfaff --- NEWS | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git a/NEWS b/NEWS index 7a2b185bbd84..abfd13cd93f4 100644 --- a/NEWS +++ b/NEWS @@ -1,20 +1,16 @@ Post-v2.7.0 - - - Tunnels: - *

Re: [ovs-dev] [PATCH 1/2] sparse: Add rte_memcpy.h replacement header.

Darrell, do you plan to review patch 2? On Sun, May 07, 2017 at 08:58:35AM -0400, Ben Pfaff wrote: > OK, I added that it affects i386 and applied this to master. Thank you! > > On Sat, May 06, 2017 at 08:27:47PM +, Darrell Ball wrote: > > I am not able to reproduce this using 64bit arch.,

Re: [ovs-dev] Package/service name on Ubuntu vs RedHat

On Mon, May 08, 2017 at 12:57:43PM +, Balazs Nemeth wrote: > Currently the deb package and service name is 'openvswitch-switch' on Ubuntu > according to debian/rules, but the rpm package and service name is > 'openvswitch' on Red Hat according to rhel/openvswitch.spec.in. > Why is this

Re: [ovs-dev] [PATCH v2] ofproto-dpif-ipfix: total counters.

On Thu, Apr 20, 2017 at 03:25:13PM +0100, mweglicx wrote: > Implementation of IPFix counters which hold > total values measured since metering process startup. > > v2: Patch is reapplied for a review because > it hasn't been correctly marked in patchwork. > > Signed-off-by: Michal Weglicki

Re: [ovs-dev] [PATCH 6/6] ovn-sbctl: support setting rbac role for remote connections

On Mon, May 01, 2017 at 10:13:32AM -0400, Lance Richardson wrote: > Add support for specifying rbac "role" when setting remote > connection configuration in the southbound database. > > Prior to this change, usage examples included: > > ovn-sbctl set-connection ptcp:6642 > ovn-sbctl

Re: [ovs-dev] [PATCH v2] ofproto-dpif-ipfix: total counters.

On Mon, May 08, 2017 at 09:22:42AM +, Weglicki, MichalX wrote: > > -Original Message- > > From: Ben Pfaff [mailto:b...@ovn.org] > > Sent: Monday, May 1, 2017 7:36 PM > > To: Weglicki, MichalX > > Cc: d...@openvswitch.org > > Subject: Re: [ovs-dev] [PATCH

[ovs-dev] Package/service name on Ubuntu vs RedHat

Hi, Currently the deb package and service name is 'openvswitch-switch' on Ubuntu according to debian/rules, but the rpm package and service name is 'openvswitch' on Red Hat according to rhel/openvswitch.spec.in. Why is this difference exist? Is it possible to use 'openvswitch-switch' on every

Re: [ovs-dev] [PATCH ovs V8 07/26] netdev-tc-offloads: Implement netdev flow flush using tc interface

On Wed, May 03, 2017 at 06:07:58PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman This does not appear to address Flavio

Re: [ovs-dev] [PATCH ovs V8 06/26] dpif-netlink: Flush added ports using netdev flow api

On Wed, May 03, 2017 at 06:07:57PM +0300, Roi Dayan wrote: > From: Paul Blakey > > If netdev flow offloading is enabled, flush all > added ports using netdev flow api. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan >

Re: [ovs-dev] [PATCH ovs V8 05/26] dpif: Save added ports in a port map for netdev flow api use

On Wed, May 03, 2017 at 06:07:56PM +0300, Roi Dayan wrote: > From: Paul Blakey > > To use netdev flow offloading api, dpifs needs to iterate over > added ports. This addition inserts the added dpif ports in a hash map, > The map will also be used to translate dpif ports to

Re: [ovs-dev] [PATCH ovs V8 04/26] other-config: Add tc-policy switch to control tc flower flag

On Wed, May 03, 2017 at 06:07:55PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Add a new configuration tc-policy option that controls tc > flower flag. Possible options are none, skip_sw, skip_hw. > The default is none which is to insert the rule both to sw and hw. > This

Re: [ovs-dev] [PATCH ovs V8 03/26] other-config: Add hw-offload switch to control netdev flow offloading

On Wed, May 03, 2017 at 06:07:54PM +0300, Roi Dayan wrote: > From: Paul Blakey > > Add a new configuration option - hw-offload that enables netdev > flow api. Enabling this option will allow offloading flows > using netdev implementation instead of the kernel datapath. > This

Re: [ovs-dev] [PATCH ovs V8 02/26] netdev: Adding a new netdev api to be used for offloading flows

On Wed, May 03, 2017 at 06:07:53PM +0300, Roi Dayan wrote: > From: Paul Blakey Please add some text to the changelog. > > Signed-off-by: Paul Blakey > Reviewed-by: Roi Dayan > Reviewed-by: Simon Horman

Re: [ovs-dev] [PATCH 0/3] Support OpenFlow 1.5 packet-out

Hi Yi-Hung, We have already started to add support for the OpenFlow 1.5 Packet Out message in our patch series for "packet type-aware pipeline: https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/331023.html https://mail.openvswitch.org/pipermail/ovs-dev/2017-April/331032.html Our focus

[ovs-dev] Liebes Kind Gottes,

Liebes Kind Gottes, Grüße 'ich bin. Frau Eunice Ton, eine Witwe zu spät Herr Fabrice Ton. Ich bin 75 Jahre alt und leide an Bauchspeicheldrüsenkrebs. Mein Zustand ist wirklich  schlecht und es ist ganz offensichtlich, dass ich nicht mehr als drei Monate nach meinen Ärzten leben. Ich bin

Re: [ovs-dev] [PATCH v5] tunneling: Avoid recirculation on datapath by computing the recirculate actions at translate time.

Hi, I looked into the code of truncate, I saw that patch ports are not handled. On the other hand I saw that "Avoid recirculation" commit should not be affected by this fact. I verified that packets are truncated correctly with my last patch I sent you before, but flow stats are not correct on

Re: [ovs-dev] [RFC PATCH 0/6] Change dpdk rxq scheduling to incorporate rxq processing cycles.

On 05/06/2017 02:01 AM, Greg Rose wrote: > On Fri, May 5, 2017 at 9:34 AM, Kevin Traynor wrote: >> Rxqs are scheduled to be handled across available pmds in round robin >> order with no weight or priority. >> >> It can happen that some very busy queues are handled by one pmd

[ovs-dev] [RFC PATCH 1/1] netdev-dpdk: enable multi-segment jumbo frames

Currently, jumbo frame support for OvS-DPDK is implemented by increasing the size of mbufs within a mempool, such that each mbuf within the pool is large enough to contain an entire jumbo frame of a user-defined size. Typically, for each user-defined MTU 'requested_mtu', a new mempool is created,

Re: [ovs-dev] [PATCH 0/1] netdev-dpdk: multi-segment mbuf jumbo frame support

Please disregard this patch - incorrect subject line. I'll submit the updated version promptly. Thanks, Mark >-Original Message- >From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev-boun...@openvswitch.org] >On Behalf Of >Mark Kavanagh >Sent: Monday, May 8, 2017 10:59 AM >To:

[ovs-dev] [RFC PATCH 0/1] netdev-dpdk: multi-segment mbuf jumbo frame support

This RFC introduces an approach for implementing jumbo frame support for OvS-DPDK with multi-segment mbufs. == Overview == Currently, jumbo frame support for OvS-DPDK is implemented by increasing the size of mbufs within a mempool, such that each mbuf within the pool is large enough to contain

[ovs-dev] [PATCH 1/1] netdev-dpdk: enable multi-segment jumbo frames

Currently, jumbo frame support for OvS-DPDK is implemented by increasing the size of mbufs within a mempool, such that each mbuf within the pool is large enough to contain an entire jumbo frame of a user-defined size. Typically, for each user-defined MTU 'requested_mtu', a new mempool is created,

[ovs-dev] [PATCH 0/1] netdev-dpdk: multi-segment mbuf jumbo frame support

This RFC introduces an approach for implementing jumbo frame support for OvS-DPDK with multi-segment mbufs. == Overview == Currently, jumbo frame support for OvS-DPDK is implemented by increasing the size of mbufs within a mempool, such that each mbuf within the pool is large enough to contain

Re: [ovs-dev] [PATCH v2] ofproto-dpif-ipfix: total counters.

> -Original Message- > From: Ben Pfaff [mailto:b...@ovn.org] > Sent: Monday, May 1, 2017 7:36 PM > To: Weglicki, MichalX > Cc: d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH v2] ofproto-dpif-ipfix: total counters. > > On Thu, Apr 20, 2017 at 03:25:13PM