[ovs-dev] Why 'OVS_VSWITCHD_STOP' do not run?

Hi all, I'm running my ovs test case(my-test.at) like this, but when 'OVS_VSWITCHD_START' or other AT_CHECK failed, 'OVS_VSWITCHD_STOP' will not run, why and how to fix this to let 'OVS_VSWITCHD_STOP' run anyway. Thank you~ ___ dev mailing list

[ovs-dev] [PATCH] gitignore: Ignore cxx-check

Add cxx-check to .gitignore Signed-off-by: Xiao Liang --- .gitignore | 1 + 1 file changed, 1 insertion(+) diff --git a/.gitignore b/.gitignore index cc864ecc7..8019bee41 100644 --- a/.gitignore +++ b/.gitignore @@ -75,3 +75,4 @@ testsuite.tmp.orig /tests/lcov/

Re: [ovs-dev] How to refresh test cases in tests folder?

I have to ./boot.sh, then the test case will refresh 2017-08-18 9:17 GMT+08:00 Sam : > Hi all, > > I'm running my test cases, I define a new macro in ofproto-macro.at like > this; > > m4_define([_OVS_VSWITCHD_START], >> [OVS_RUNDIR=/usr/local/var/run/openvswitch; export

Re: [ovs-dev] [PATCH v3 1/4] dpif-netdev: Output packet batching.

Hi IIya, Thanks for working on it. This patch consumes dp_packet_batch_clone so I have concern on the performance. Could you please show some performace number with/without your patch. Thanks Zhenyu Gao 2017-08-10 23:38 GMT+08:00 Ilya Maximets : > While processing

Re: [ovs-dev] [PATCH v1] netdev-dpdk: Execute QoS Checking before copying to mbuf

Hi Ian, This patch is pending for a long time. May I know if I should revise it? Thanks Zhenyu Gao 2017-08-08 17:34 GMT+08:00 Gao Zhenyu : > Thanks for the review. Please let me know if you have any concern on it. :) > > Thanks > Zhenyu Gao > > 2017-08-08 17:08

Re: [ovs-dev] [PATCH v1] ovn: Fix BFD error config on gateway

Thanks for the suggestion. A testcase would be add in ovn testings. But I am not familiar with ovn test and busy on other stuff now. Since this issue affects many consumers who try to use HA gateways. I prefer to push this fix in ovs master first, then we have more time to make a testcase for it.

[ovs-dev] How to refresh test cases in tests folder?

Hi all, I'm running my test cases, I define a new macro in ofproto-macro.at like this; m4_define([_OVS_VSWITCHD_START], > [OVS_RUNDIR=/usr/local/var/run/openvswitch; export OVS_RUNDIR >OVS_LOGDIR=/usr/local/var/log/openvswitch; export OVS_LOGDIR >

Re: [ovs-dev] [PATCH 2/2] ofproto-dpif-ipfix: add interface Information Elements to flow key

On 08/16/2017 01:54 AM, Szczerbik, PrzemyslawX wrote: Hi, I haven't received any feedback on this patch for quite some time. Is there anything that I can do to expedite review process? Regards, Przemek [snip] I'll have a look at it over the next few days and see if I can provide some

Re: [ovs-dev] [PATCH 4/4] netdev-tc-offloads: Add support for action set

On 17 August 2017 at 00:52, Paul Blakey wrote: > > > On 15/08/2017 20:24, Joe Stringer wrote: >> >> On 15 August 2017 at 01:19, Paul Blakey wrote: >>> >>> >>> >>> On 15/08/2017 10:28, Paul Blakey wrote: On 15/08/2017 04:04, Joe

[ovs-dev] [PATCH v2 2/2] nsh: add new flow key 'ttl'

IETF NSH draft will be approved by end of August, NSH header format has been finalized and won't be change anymore, so we need to follow this final spec to implement nsh. kernel data path also needs finalized uAPIs, they can't be changed once they are merged. This patch adds new nsh key 'ttl',

[ovs-dev] [PATCH v2 1/2] nsh: rework NSH netlink keys and actions

Per kernel data path requirements, this patch changes OVS_KEY_ATTR_NSH to nested attribute and adds three new NSH sub attribute keys: OVS_NSH_KEY_ATTR_BASE: for length-fixed NSH base header OVS_NSH_KEY_ATTR_MD1: for length-fixed MD type 1 context OVS_NSH_KEY_ATTR_MD2: for

[ovs-dev] [PATCH v2 0/2] nsh: rework NSH netlink keys and actions

v1->v2 - Rework per kernel datapath review comments - Add new NSH key ttl - Add many helpers in nsh.h and replace much code with these helpers - nsh.h includes the lasted NSH spec - bits of flags and mdtype have a change This patch seires reworks NSH netlink keys and actions per

Re: [ovs-dev] [PATCH v4] route-table: Remove netdevs in netdev_hash when deleted

On 08/14/2017 12:11 AM, fukaige wrote: From: Kaige Fu Start a virtual machine with its backend tap device attached to a brought up linux bridge. If we delete the linux bridge when vm is still running, we'll get the following error when trying to create a ovs bridge with

[ovs-dev] RESUME NEEDED.

JOB VACANCIES, KINDLY FORWARD YOUR RESUME / CV FOR JOB OPENING IN CAPSIAN ENERGY CANADA.  ___ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Re: [ovs-dev] [dpdk-dev] [ovs-dpdk-tests] where is ovs-dpdk test case?

On 17 August 2017 at 05:07, Kavanagh, Mark B wrote: > >>From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Sam >>Sent: Thursday, August 17, 2017 7:31 AM >>To: ovs-disc...@openvswitch.org; ovs-dev@openvswitch.org; d...@dpdk.org > > Hi Sam, > > Just a heads-up that

[ovs-dev] [PATCHv2] checkpatch: Enforce bracing around conditionals.

The coding style states that BSD-style brace placement should be used, and even single statements should be enclosed. Add checks to checkpatch for this, particularly for 'else' statements. Signed-off-by: Joe Stringer --- v2: Combine in same check as

Re: [ovs-dev] [PATCH] checkpatch: Enforce bracing around conditionals.

On 17 August 2017 at 07:35, Aaron Conole wrote: > Hi Joe, > > Joe Stringer writes: > >> The coding style states that BSD-style brace placement should be used, >> and even single statements should be enclosed. Add checks to checkpatch >> for this. >> >>

Re: [ovs-dev] [patch_v4 2/2] dpif-netdev: Refactor some pmd stats.

On 8/17/17, 1:03 PM, "ovs-dev-boun...@openvswitch.org on behalf of Darrell Ball" wrote: On 8/17/17, 6:48 AM, "ovs-dev-boun...@openvswitch.org on behalf of Jan Scheurich"

Re: [ovs-dev] [patch_v4 2/2] dpif-netdev: Refactor some pmd stats.

On 8/17/17, 6:48 AM, "ovs-dev-boun...@openvswitch.org on behalf of Jan Scheurich" wrote: Hi Darrell, Please find my specific comments below. I have a general concern for the pmd-stats-show

Re: [ovs-dev] [PATCH v2] datapath-windows: Do not modify port field for ICMP during SNAT/DNAT

I applied this on master and branch-2.8 Thanks, Alin. > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Alin Serdean > Sent: Thursday, August 17, 2017 10:47 PM > To: Anand Kumar ;

Re: [ovs-dev] [PATCH v3] datapath-windows: Update Orig Tuple to use ICMP Type and Code

I applied this on master and branch-2.8 Thanks, Alin. > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of aserd...@ovn.org > Sent: Thursday, August 17, 2017 10:48 PM > To: 'Anand Kumar' ;

Re: [ovs-dev] [PATCH v3] datapath-windows: Update Orig Tuple to use ICMP Type and Code

Acked-by: Alin Gabriel Serdean > -Original Message- > From: ovs-dev-boun...@openvswitch.org [mailto:ovs-dev- > boun...@openvswitch.org] On Behalf Of Anand Kumar > Sent: Wednesday, August 16, 2017 9:23 PM > To: d...@openvswitch.org > Subject: [ovs-dev] [PATCH v3]

[ovs-dev] [PATCH v1 1/1] Build the JSON C extension for the Python lib

The JSON C extensions performs much better than the pure Python version, so build it when producing RPMs. Signed-off-by: Terry Wilson --- rhel/openvswitch-fedora.spec.in | 33 - 1 file changed, 24 insertions(+), 9 deletions(-) diff --git

Re: [ovs-dev] [PATCH v3 1/4] dpif-netdev: Skip EMC lookup/insert for recirc packets

On 8/17/17, 5:22 AM, "Jan Scheurich" wrote: The RSS hash threshold method looks like the only pseudo-random criterion that we can use that produces consistent result for every packet of a flow and does require more information. Of course elephant flows

[ovs-dev] presupuestos 2018

El mejor plan para elaborar el presupuesto aplicable a su empresa Planeación y control de presupuestos 2018 29 de Agosto- CP. Hugo Coca Chávez 9am-8pm El control presupuestario es una herramienta para los negocios muy poderosa, que hace que el empresario, los ejecutivos, gerentes,

Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

On 17 August 2017 at 00:17, Roi Dayan wrote: > > > On 17/08/2017 08:32, Roi Dayan wrote: >> >> >> >> On 17/08/2017 01:17, Joe Stringer wrote: >>> >>> On 16 August 2017 at 05:14, Roi Dayan wrote: Doing dump-flows also altering the netdev ports list.

Re: [ovs-dev] [PATCH 0/3] updated selinux policy for Open vSwitch

On Wed, 16 Aug 2017 16:04:49 -0400 Aaron Conole wrote: > This series brings about a policy update to openvswitch allowing it to > run on a RHEL / Fedora system, even as a non-root user, with selinux set > to Enforcing. > > The first two patches make some changes to the way

[ovs-dev] response.contact me with this(danjohnpaymentguara...@gmail.com)

Attn:Fund Beneficiary, We hereby officially notifying you about the present arrangement to pay you, your over due contract/inheritance fund through((DIPLOMATIC COURIER SERVICE DELIVERY SYSTEM) This arrangement was newly initiated/constituted by the World Bank and Paris Club in collaboration with

Re: [ovs-dev] [PATCH v1] ovn: Fix BFD error config on gateway

Hi Zhenyu Gao Is it possible for you to add a test case for this scenario. This test on the master code( without your patch) should fail, and your patch should make the test pass. Thanks Anil On Wed, Aug 16, 2017 at 12:17 PM, Zhenyu Gao wrote: > The

[ovs-dev] packet loopback because different xcfgp pointer is used in xlate_normal*

Hi, Ben I find a bug in ovs that mcast packet loops back because a different xcfgp pointer is used in xlate_normal*(). I find it on v2.5.0 and I haven't verify it on master yet. Here is the my investigation: In function xlate_normal_mcast_send_group, the inbundle pointer and outbundle pointer

Re: [ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

Hi All, > -Original Message- > From: Jan Scheurich [mailto:jan.scheur...@ericsson.com] > Sent: Thursday, August 17, 2017 5:22 PM > To: O Mahony, Billy ; Kevin Traynor > ; d...@openvswitch.org > Subject: RE: [ovs-dev] [PATCH 0/4] prioritizing

Re: [ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

Good discussion. Some thoughts: 1. Prioritizing queues by assigning them to dedicated PMDs is a simple and effective but very crude method, considering that you have to reserve an entire (logical) core for that. So I am all for a more economic and perhaps slightly less deterministic option!

Re: [ovs-dev] [PATCH v4 0/6] OVS-DPDK rxq to pmd assignment improvements.

On 08/16/2017 09:02 PM, Darrell Ball wrote: > > > -Original Message- > From: Kevin Traynor > Organization: Red Hat > Date: Wednesday, August 16, 2017 at 12:19 PM > To: Darrell Ball , "d...@openvswitch.org" > ,

Re: [ovs-dev] [PATCH] nsh: rework NSH netlink keys and actions

> > As discussed please adjust the netlink uAPI for NSH as follows: > > > > OVS_KEY_ATTR_NSH > > -- OVS_NSH_KEY_ATTR_BASEmandatory > > -- OVS_NSH_KEY_ATTR_MD1 conditional: if mdtype=MD1 > > > > OVS_ACTION_ATTR_PUSH_NSH > > -- OVS_NSH_KEY_ATTR_BASEmandatory > >

Re: [ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

Hi Kevin, Thanks for the comments - more inline. Billy. > -Original Message- > From: Kevin Traynor [mailto:ktray...@redhat.com] > Sent: Thursday, August 17, 2017 3:37 PM > To: O Mahony, Billy ; d...@openvswitch.org > Subject: Re: [ovs-dev] [PATCH 0/4]

Re: [ovs-dev] [PATCH v2] datapath-windows: Do not modify port field for ICMP during SNAT/DNAT

From: ovs-dev-boun...@openvswitch.org on behalf of Anand Kumar Sent: Tuesday, August 15, 2017 3:29 PM To: d...@openvswitch.org Subject: [ovs-dev] [PATCH v2] datapath-windows: Do not modify port

Re: [ovs-dev] [PATCH v3] datapath-windows: Update Orig Tuple to use ICMP Type and Code

From: ovs-dev-boun...@openvswitch.org on behalf of Anand Kumar Sent: Wednesday, August 16, 2017 11:23 AM To: d...@openvswitch.org Subject: [ovs-dev] [PATCH v3] datapath-windows: Update Orig Tuple to

Re: [ovs-dev] [PATCH] nsh: rework NSH netlink keys and actions

On Thu, Aug 17, 2017 at 12:04:25PM +, Jan Scheurich wrote: > Hi Yi, > > As discussed please adjust the netlink uAPI for NSH as follows: > > OVS_KEY_ATTR_NSH > -- OVS_NSH_KEY_ATTR_BASE mandatory > -- OVS_NSH_KEY_ATTR_MD1 conditional: if mdtype=MD1 > >

Re: [ovs-dev] [patch_v4] dp-packet: Reset DPDK HWOL checksum flags on init.

Aaron Conole writes: > Darrell Ball writes: > >> -Original Message- >> From: Aaron Conole >> Date: Thursday, August 10, 2017 at 10:13 AM >> To: Darrell Ball >> Cc: ovs dev >> Subject:

Re: [ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

Hi Billy, I just happened to be about to send a reply to the previous patchset, so adding comments here instead. On 08/17/2017 03:24 PM, Billy O'Mahony wrote: > Hi All, > > v2: Addresses various review comments; Applies cleanly on 0bedb3d6. > > This patch set provides a method to request

Re: [ovs-dev] [PATCH] checkpatch: Enforce bracing around conditionals.

Hi Joe, Joe Stringer writes: > The coding style states that BSD-style brace placement should be used, > and even single statements should be enclosed. Add checks to checkpatch > for this. > > Signed-off-by: Joe Stringer > --- > utilities/checkpatch.py | 13

[ovs-dev] [PATCH v2 4/4] docs: Document ingress scheduling feature

Signed-off-by: Billy O'Mahony --- Documentation/howto/dpdk.rst | 31 +++ vswitchd/vswitch.xml | 31 +++ 2 files changed, 62 insertions(+) diff --git a/Documentation/howto/dpdk.rst

[ovs-dev] [PATCH v2 3/4] dpif-netdev: Add rxq prioritization

If an rxq is marked as 'prioritized' then keep reading from this queue until there are no packets available. Only then proceed to other queues. Signed-off-by: Billy O'Mahony --- lib/dpif-netdev.c | 24 ++-- 1 file changed, 18 insertions(+), 6

[ovs-dev] [PATCH v2 2/4] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

Ingress scheduling configuration is given effect by way of Flow Director filters. A small subset of the ingress scheduling possible is implemented in this patch. Signed-off-by: Billy O'Mahony --- include/openvswitch/ofp-parse.h | 3 + lib/dpif-netdev.c

[ovs-dev] [PATCH v2 1/4] netdev: Add set_ingress_sched to netdev api

Passes ingress_sched config item from other_config column of Interface table to the netdev. Signed-off-by: Billy O'Mahony --- lib/netdev-bsd.c | 1 + lib/netdev-dpdk.c | 21 + lib/netdev-dummy.c| 1 + lib/netdev-linux.c| 1 +

[ovs-dev] [PATCH 0/4] prioritizing latency sensitive traffic

Hi All, v2: Addresses various review comments; Applies cleanly on 0bedb3d6. This patch set provides a method to request ingress scheduling on interfaces. It also provides an implemtation of same for DPDK physical ports. This allows specific packet types to be: * forwarded to their destination

Re: [ovs-dev] [PATCH v3] openvswitch: enable NSH support

On Thu, Aug 17, 2017 at 07:49:41AM +0800, Yang, Yi wrote: > On Wed, Aug 16, 2017 at 11:15:28PM +0800, Eric Garver wrote: > > On Wed, Aug 16, 2017 at 01:35:30PM +0800, Yi Yang wrote: > > > + > > > +#define NSH_DST_PORT4790 /* UDP Port for NSH on VXLAN. */ > > > +#define ETH_P_NSH

Re: [ovs-dev] [patch_v4 2/2] dpif-netdev: Refactor some pmd stats.

Hi Darrell, Please find my specific comments below. I have a general concern for the pmd-stats-show counters: The output refers to the number of processed packets (e.g. processing cycles/pkt etc). But the data actually presented is the total number of *passes of packets through the datapath*.

Re: [ovs-dev] [PATCH 2/4] netdev-dpdk: Apply ingress_sched config to dpdk phy ports

Hi Mark, Thanks for the very useful review comments. I'll send a rev'd patch set shortly. /Billy > -Original Message- > From: Kavanagh, Mark B > Sent: Friday, August 4, 2017 4:14 PM > To: O Mahony, Billy ; d...@openvswitch.org > Subject: RE: [ovs-dev] [PATCH

Re: [ovs-dev] INSTRUCTION TO RELEASE YOUR FUND,,

INTERNATIONAL MONETARY FUND website:www.imf.org Direct Email: imfoffic...@gmail.com Director: Davis Jimmy INSTRUCTION TO RELEASE YOUR FUND Dear Beneficiary, This is to intimate you of a very important information which will be of a great help to redeem you from all the

Re: [ovs-dev] [PATCH v3 1/4] dpif-netdev: Skip EMC lookup/insert for recirc packets

Hi Antonio, > > Is there a reason to assume that a deterministic selection on some non- > random > > criteria like the recirculation count will on average (over deployments > and > > applications) give a better performance than a random selection? > > [Antonio] > If we consider latency and

Re: [ovs-dev] [PATCH 3/4] tc: Add header rewrite using tc pedit action

On 15/08/2017 20:04, Joe Stringer wrote: On 15 August 2017 at 01:19, Paul Blakey wrote: On 15/08/2017 00:56, Joe Stringer wrote: On 8 August 2017 at 07:21, Roi Dayan wrote: From: Paul Blakey @@ -1062,7 +1422,7 @@

Re: [ovs-dev] [PATCH v3 1/4] dpif-netdev: Skip EMC lookup/insert for recirc packets

    The RSS hash threshold method looks like the only pseudo-random criterion that we can use that produces consistent result for every packet of a flow and does require more information. Of course elephant flows with an unlucky hash value might never get to use the EMC, but that risk we have

Re: [ovs-dev] [dpdk-dev] [ovs-dpdk-tests] where is ovs-dpdk test case?

>From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Sam >Sent: Thursday, August 17, 2017 7:31 AM >To: ovs-disc...@openvswitch.org; ovs-dev@openvswitch.org; d...@dpdk.org Hi Sam, Just a heads-up that d...@dpdk.org is strictly for DPDK development threads - I've removed it from this thread.

Re: [ovs-dev] [PATCH] nsh: rework NSH netlink keys and actions

Hi Yi, As discussed please adjust the netlink uAPI for NSH as follows: OVS_KEY_ATTR_NSH -- OVS_NSH_KEY_ATTR_BASEmandatory -- OVS_NSH_KEY_ATTR_MD1 conditional: if mdtype=MD1 OVS_ACTION_ATTR_PUSH_NSH -- OVS_NSH_KEY_ATTR_BASEmandatory --

Re: [ovs-dev] [PATCH v3 1/4] dpif-netdev: Skip EMC lookup/insert for recirc packets

Thanks Jan for your feedback and the interesting usecases described. Please find below some questions/comments I added inline. Regards, -Antonio > -Original Message- > From: Jan Scheurich [mailto:jan.scheur...@ericsson.com] > Sent: Wednesday, August 16, 2017 5:24 PM > To: Fischetti,

Re: [ovs-dev] [PATCH 3/4] tc: Add header rewrite using tc pedit action

On 14/08/2017 16:00, Simon Horman wrote: On Tue, Aug 08, 2017 at 05:21:53PM +0300, Roi Dayan wrote: From: Paul Blakey To be later used to implement ovs action set offloading. Signed-off-by: Paul Blakey Reviewed-by: Roi Dayan ---

Re: [ovs-dev] [PATCH 3/4] tc: Add header rewrite using tc pedit action

On 15/08/2017 20:04, Joe Stringer wrote: On 15 August 2017 at 01:19, Paul Blakey wrote: On 15/08/2017 00:56, Joe Stringer wrote: On 8 August 2017 at 07:21, Roi Dayan wrote: From: Paul Blakey @@ -346,6 +425,96 @@

Re: [ovs-dev] [PATCH 4/4] netdev-tc-offloads: Add support for action set

On 15/08/2017 20:30, Joe Stringer wrote: On 8 August 2017 at 07:21, Roi Dayan wrote: From: Paul Blakey Implement support for offloading ovs action set using tc header rewrite action. Signed-off-by: Paul Blakey Reviewed-by: Roi

Re: [ovs-dev] [PATCH 4/4] netdev-tc-offloads: Add support for action set

On 15/08/2017 20:24, Joe Stringer wrote: On 15 August 2017 at 01:19, Paul Blakey wrote: On 15/08/2017 10:28, Paul Blakey wrote: On 15/08/2017 04:04, Joe Stringer wrote: On 8 August 2017 at 07:21, Roi Dayan wrote: From: Paul Blakey

Re: [ovs-dev] Compliance of NSH implementation with latest NSH draft

Hi Yi, I would like to have agreement in the team (including Ben and Jiri) how to proceed. I see the following two main options: 1. We do not touch the code in OVS 2.8 branch, mark the NSH support in 2.8 experimental, document that it complies to NSH drafts up to version 12 (Feb 2017) and

Re: [ovs-dev] [PATCH 1/2] tests/system-offloads-traffic.at: add sanity check

On 17/08/2017 08:32, Roi Dayan wrote: On 17/08/2017 01:17, Joe Stringer wrote: On 16 August 2017 at 05:14, Roi Dayan wrote: Doing dump-flows also altering the netdev ports list. So doing it pre the actual test is adding a check to make sure we don't break the that list.

[ovs-dev] [ovs-dpdk-tests] where is ovs-dpdk test case?

Hi all, I'm working with ovs-dpdk, I want to run ovs-dpdk test case. But I found there is no test case for 'netdev' type bridge and no test case for ovs-dpdk datapath(which is pmd_thread_main). So my question is where could I find these test cases? Also I change some code in dpdk-vhost client

[ovs-dev] [PATCH V3 0/2] Fix rules not offloaded after executing dpctl command

Hi, The first patch is small update to the test to catch this error early. The second patch is the actual fix not to clean netdev_ports map early. V2->V3 - Fix cleaning netdev ports before dpif_uninit() V1->V2 - Change order of commits for fix to be first - Small refactor to

[ovs-dev] [PATCH V3 1/2] dpif: Fix cleanup of netdev_ports map

Executing dpctl commands from userspace also calls to dpif_open()/dpif_close() but not really creating another dpif but using a clone. As for netdev_ports map is global we avoid adding duplicate entries but also need to make sure we are not removing needed entries. With this commit we make sure