Re: [ovs-dev] OVS Soft Freeze date for 2.14 release

Hi Maximets, On Thu, Jul 02, 2020 at 08:28:21PM +0200, Ilya Maximets wrote: > Hi Ben, > > We reached the "soft freeze" date yesterday. So, we probably need > to make announcement on a mailing list that soft freeze is started. > > And we need to announce a "feature freeze" (branching) targeted da

Re: [ovs-dev] [PATCH v3 ovn 4/5] ovn-northd: Refactor NAT address parsing.

On Thu, Jul 2, 2020 at 7:53 AM Dumitru Ceara wrote: > > Store NAT entries pointers in ovn_datapath and pre-parse the external IP > addresses. This simplifies the code and makes it easier to reuse the parsed > external IP and solicited-node address without reparsing. > > Signed-off-by: Dumitru Cear

Re: [ovs-dev] [PATCH v3 ovn 3/5] ovn-northd: Refactor ARP/NS responder in router pipeline.

On Thu, Jul 2, 2020 at 7:53 AM Dumitru Ceara wrote: > > Add functions to build the ARP/NS responder flows for table > S_ROUTER_IN_IP_INPUT and use them in all places where responder > flows are created. > > Signed-off-by: Dumitru Ceara Acked-by: Han Zhou ___

Re: [ovs-dev] [PATCH v3 ovn 2/5] ovn-northd: Store ETH address of router inport in xreg0.

On Thu, Jul 2, 2020 at 7:53 AM Dumitru Ceara wrote: > > This helps simplifying logical flows that need to use the port's > configured ETH address: > - ARP responders for owned IPs > - NS responders for owned IPs > > Signed-off-by: Dumitru Ceara > --- > northd/ovn-northd.8.xml | 22 --- > n

Re: [ovs-dev] [PATCH v3 ovn 5/5] ovn-northd: Minimize number of ARP/NS responder flows for DNAT.

Thanks Dumitru. I have 2 comments below. On Thu, Jul 2, 2020 at 7:53 AM Dumitru Ceara wrote: > > Most ARP/NS responder flows can be configured per datapath instead of per > router port. > > The only exception is with distributed gateway router ports which need > special treatment. This patch chan

[ovs-dev] [Engineer] Engineer Jobs T-Shirts Catalogue New Design 2020

*Shop Funny Engineer T-Shirts Online*

Re: [ovs-dev] [PATCH v2 ovn] Throttle the OVS-OVN Global config logs

Bleep bloop. Greetings Ankur Sharma, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Ankur Sharma needs to sign off. WARNING: Unexpected sign-offs from developers wh

Re: [ovs-dev] [PATCH v1 ovn] Throttle the OVS-OVN Global config logs

Bleep bloop. Greetings Ankur Sharma, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: ERROR: Author Ankur Sharma needs to sign off. WARNING: Unexpected sign-offs from developers wh

Re: [ovs-dev] [PATCH] system-dpdk: add negotiation check for userspace-tso

Hi Gowrishankar, Thanks for working on this patch, it's a good idea to test the TSO negotiation. Which dpdk version are you testing? Because I am getting the same results on 19.11.3 or master. See the details below. Sometimes it fails with this in testpmd-dpdkvhostuser0.log: EAL: Detected 4 l

[ovs-dev] [PATCH v2 ovn] Throttle the OVS-OVN Global config logs

From: Ankur Sharma ISSUE: We observed that if ovn-controller is running, while ovn-encap-ip/ ovn-encap-type is not set, then following error gets logged continously: "chassis|INFO|Need to specify an encap type and ip" Above log increased the size of ovn-controller.log to hundreds of GBs before

[ovs-dev] [PATCH v1 ovn] Throttle the OVS-OVN Global config logs

From: Ankur Sharma From: Ankur Sharma ISSUE: We observed that if ovn-controller is running, while ovn-encap-ip/ ovn-encap-type is not set, then following error gets logged continously: "chassis|INFO|Need to specify an encap type and ip" Above log increased the size of ovn-controller.log to hun

[ovs-dev] Mediación de conflictos y prevención de demandas

Nuestro webinar interactivo le brindará herramientas que le ayudarán para mejorar los procesos en la administración del recurso humano mediante acciones preventivas y de negociación de conflictos para evitar el litigio laboral. El participante dispondrá, con mayor certeza, de información para e

Re: [ovs-dev] [PATCH] dpif-netdev-unixctl.man: document bond-show command

On Fri, Jun 26, 2020 at 03:56:31PM +0200, Adrian Moreno wrote: > Document recently added ovs-appctl command Could you add a Fixes tag to indicate the commit? Thanks fbl > > Cc: Vishal Deep Ajmera > Signed-off-by: Adrian Moreno > --- > lib/dpif-netdev-unixctl.man | 9 + > 1 file change

Re: [ovs-dev] [PATCH v9] AB bonding: Add "primary" interface concept

Another thing, consider a small note in NEWS. On Thu, Jul 02, 2020 at 05:17:59PM -0300, Flavio Leitner wrote: > > Hi Jeff, > > Thanks for following up. > I have a couple comments inline. > > > On Thu, Jun 25, 2020 at 02:03:34PM -0700, Jeff Squyres via dev wrote: > > In AB bonding, if the cur

Re: [ovs-dev] [PATCH v2] odp-execute: Fix length checking while executing check_pkt_len action.

On Fri, Jun 26, 2020 at 12:51:10PM +0200, Ilya Maximets wrote: > If dp-packet contains l2 padding or cutlen was applied to it, size will > be larger than the actual size of a payload and action will work > incorrectly. > > Ex. Padding could be added during miniflow_extract() if detected. > > CC:

Re: [ovs-dev] [PATCH v9] AB bonding: Add "primary" interface concept

Hi Jeff, Thanks for following up. I have a couple comments inline. On Thu, Jun 25, 2020 at 02:03:34PM -0700, Jeff Squyres via dev wrote: > In AB bonding, if the current active slave becomes disabled, a > replacement slave is arbitrarily picked from the remaining set of > enabled slaves. This

[ovs-dev] お支払い方法の情報を更新

お支払い方法の情報を更新してください。Update default card for your membership. マイストア? タイムセール? ギフト券 Amazonプライムをご利用頂きありがとうございます。お客様のAmazonプライム会員資格は、2020/06/09に更新を迎えます。お調べしたところ、会費のお支払いに使用できる有効なクレジットカードがアカウントに登録されていません。クレジットカード情報の更新、新しいクレジットカードの追加については以下の手順をご確認ください。 1. アカウントサービスからAmazonプライム会員情報を管理するにアクセスします。

Re: [ovs-dev] [PATCH v9] AB bonding: Add "primary" interface concept

Gentle nudge. Any further feedback on this one? Thank you! > On Jun 25, 2020, at 5:03 PM, Jeff Squyres wrote: > > In AB bonding, if the current active slave becomes disabled, a > replacement slave is arbitrarily picked from the remaining set of > enabled slaves. This commit adds the concept

Re: [ovs-dev] [PATCH ovn v2] ovn-controller: Configure hwaddr for the integration bridge

Excellent! Thanks, Numan! Acked-by: Mark Michelson On 7/1/20 11:34 AM, num...@ovn.org wrote: From: Numan Siddique When a first non-local port is added to the integration bridge, it results in the recalculation of datapath-id by ovs-vswitchd forcing all active connections to the controllers t

[ovs-dev] OVS Soft Freeze date for 2.14 release

Hi Ben, We reached the "soft freeze" date yesterday. So, we probably need to make announcement on a mailing list that soft freeze is started. And we need to announce a "feature freeze" (branching) targeted date. I think it might be good to announce today or tomorrow and set the branch creation da

[ovs-dev] OVN Meeting Logs 02 July, 2020

Here is the IRC log for the OVN meeting for 02 July, 2020 http://eavesdrop.openstack.org/meetings//ovn_community_development_discussion/2020/ovn_community_development_discussion.2020-07-02-17.19.log.html If you are interested in attending this meeting, it happens every Thursday in the #openvswitc

[ovs-dev] [PATCH v6 0/6] DPCLS Subtable ISA Optimization

v6 work done: - Fix as --64 unrecognized option - Fix build issues with avx512 library changes - Fix files left in build dir after distclean - Fix CPU arch dependant RTE_CPUFLAG_ usage Thanks William & Ian for review & help on v5. All known issues are fixed and working here. Hi All, This patchs

[ovs-dev] [PATCH v6 3/6] dpif-netdev: add subtable-lookup-prio-get command.

This commit adds ia new command, "dpif-netdev/subtable-lookup-prio-get" which prints the available sutable lookup functions in this OVS binary. Example output from the command: Available lookup functions (priority : name) 0 : autovalidator 1 : generic Signed-off-by: Harry van Haar

[ovs-dev] [PATCH v6 6/6] docs/dpdk/bridge: add datapath performance section.

This commit adds a section to the dpdk/bridge.rst netdev documentation, detailing the added DPCLS functionality. The newly added commands are documented, and sample output is provided. Running the DPCLS autovalidator with unit tests by default is possible through re-compiling the autovalidator to

[ovs-dev] [PATCH v6 4/6] dpdk: enable cpu feature detection.

This commit implements a method to retrieve the CPU ISA capabilities. These ISA capabilities can be used in OVS to at runtime select a function implementation to make the best use of the available ISA on the CPU. Signed-off-by: Harry van Haaren --- v6: - Add #if around RTE_CPUFLAG_AVX512F as it

[ovs-dev] [PATCH v6 5/6] dpif-lookup: add avx512 gather implementation.

This commit adds an AVX-512 dpcls lookup implementation. It uses the AVX-512 SIMD ISA to perform multiple miniflow operations in parallel. To run this implementation, the "avx512f" and "bmi2" ISAs are required. These ISA checks are performed at runtime while probing the subtable implementation. If

[ovs-dev] [PATCH v6 1/6] dpif-netdev: implement subtable lookup validation.

This commit refactors the existing dpif subtable function pointer infrastructure, and implements an autovalidator component. The refactoring of the existing dpcls subtable lookup function handling, making it more generic, and cleaning up how to enable more implementations in future. In order to e

[ovs-dev] [PATCH v6 2/6] dpif-netdev: add subtable lookup prio set command.

This commit adds a command for the dpif-netdev to set a specific lookup function to a particular priority level. The command enables runtime switching of the dpcls subtable lookup implementation. Selection is performed based on a priority. Higher priorities take precedence, eg; priotity 5 will be

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

On 7/2/20 5:25 PM, Pai G, Sunil wrote: >> >>> + >>> if [ "${DPDK_VER##refs/*/}" != "${DPDK_VER}" ]; then >>> # Avoid using cache for git tree build. >>> rm -rf dpdk-dir >>> @@ -108,7 +112,8 @@ function install_dpdk() >>> if [ -f "${VERSION_FILE}" ]; then >>>

Re: [ovs-dev] [PATCH] dpdk: Refuse running on cpu >= RTE_MAX_LCORE.

On Thu, Jul 2, 2020 at 5:53 PM Ilya Maximets wrote: > > pmd threads will never run on any core >= RTE_MAX_LCORE ok, it works. > > > > Though, it binds OVS core id and DPDK lcore id. > > Should OVS really care about this? > > I think this is better than random abort() in production environment. > A

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

On 7/2/20 6:10 PM, Pai G, Sunil wrote: >> -Original Message- >> From: Richardson, Bruce >> Sent: Thursday, July 2, 2020 9:32 PM >> To: Pai G, Sunil ; Ilya Maximets >> ; d...@openvswitch.org >> Cc: Stokes, Ian ; david.march...@redhat.com; >> Tummala, Sivaprasad >> Subject: RE: [PATCH RFC d

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

> -Original Message- > From: Richardson, Bruce > Sent: Thursday, July 2, 2020 9:32 PM > To: Pai G, Sunil ; Ilya Maximets > ; d...@openvswitch.org > Cc: Stokes, Ian ; david.march...@redhat.com; > Tummala, Sivaprasad > Subject: RE: [PATCH RFC dpdk-latest] build: Add support for DPDK meson >

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

> -Original Message- > From: Pai G, Sunil > Sent: Thursday, July 2, 2020 4:25 PM > To: Ilya Maximets ; d...@openvswitch.org > Cc: Stokes, Ian ; david.march...@redhat.com; > Richardson, Bruce ; Tummala, Sivaprasad > > Subject: RE: [PATCH RFC dpdk-latest] build: Add support for DPDK meso

Re: [ovs-dev] [PATCH] dpdk: Refuse running on cpu >= RTE_MAX_LCORE.

On 7/2/20 3:56 PM, David Marchand wrote: > On Thu, Jul 2, 2020 at 3:09 PM Ilya Maximets wrote: >> >> On 6/26/20 2:27 PM, David Marchand wrote: >>> DPDK lcores range is [0, RTE_MAX_LCORE[. >>> Trying to use a lcore out of this range expose OVS to crashes in DPDK >>> mempool >>> local cache array.

Re: [ovs-dev] [PATCH] dpdk: Add commands to configure log levels.

On Thu, Jul 2, 2020 at 12:04 PM Ilya Maximets wrote: > > On 7/2/20 10:25 AM, David Marchand wrote: > > On Thu, Jul 2, 2020 at 12:05 AM Ilya Maximets wrote: > >> On 6/26/20 2:27 PM, David Marchand wrote: > >>> Enabling debug logs in dpdk can be a challenge to be sure of what is > >>> actually enab

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

> -Original Message- > From: Richardson, Bruce > Sent: Thursday, July 2, 2020 7:29 PM > To: Pai G, Sunil ; d...@openvswitch.org > Cc: Stokes, Ian ; i.maxim...@ovn.org; > david.march...@redhat.com; Tummala, Sivaprasad > > Subject: RE: [PATCH RFC dpdk-latest] build: Add support for DPDK mes

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

> -Original Message- > From: Ilya Maximets > Sent: Thursday, July 2, 2020 7:26 PM > To: Pai G, Sunil ; d...@openvswitch.org > Cc: Stokes, Ian ; i.maxim...@ovn.org; > david.march...@redhat.com; Richardson, Bruce > ; Tummala, Sivaprasad > ; i.maxim...@ovn.org > Subject: Re: [PATCH RFC dpdk-l

Re: [ovs-dev] [PATCH v5 0/6] DPCLS Subtable ISA Optimization

> -Original Message- > From: William Tu > Sent: Wednesday, July 1, 2020 10:53 PM > To: Van Haaren, Harry > Cc: ovs-dev ; Stokes, Ian ; > Ilya > Maximets ; Federico Iezzi > Subject: Re: [PATCH v5 0/6] DPCLS Subtable ISA Optimization > > On Wed, Jul 1, 2020 at 7:22 AM Harry van Haaren >

[ovs-dev] [PATCH v3 ovn 3/5] ovn-northd: Refactor ARP/NS responder in router pipeline.

Add functions to build the ARP/NS responder flows for table S_ROUTER_IN_IP_INPUT and use them in all places where responder flows are created. Signed-off-by: Dumitru Ceara --- northd/ovn-northd.8.xml |8 + northd/ovn-northd.c | 314 +-- tests/

[ovs-dev] [PATCH v3 ovn 4/5] ovn-northd: Refactor NAT address parsing.

Store NAT entries pointers in ovn_datapath and pre-parse the external IP addresses. This simplifies the code and makes it easier to reuse the parsed external IP and solicited-node address without reparsing. Signed-off-by: Dumitru Ceara --- northd/ovn-northd.c | 115 +

[ovs-dev] [PATCH v3 ovn 5/5] ovn-northd: Minimize number of ARP/NS responder flows for DNAT.

Most ARP/NS responder flows can be configured per datapath instead of per router port. The only exception is with distributed gateway router ports which need special treatment. This patch changes the ARP/NS responder behavior and adds: - Priority 92 flows to reply to ARP requests on distributed ga

[ovs-dev] [PATCH v3 ovn 1/5] ovn-northd: Document OVS register usage in logical flows.

Also, use macros instead of bare references to register names. Acked-by: Han Zhou Signed-off-by: Dumitru Ceara --- northd/ovn-northd.c | 158 --- 1 file changed, 110 insertions(+), 48 deletions(-) diff --git a/northd/ovn-northd.c b/northd/ovn-no

[ovs-dev] [PATCH v3 ovn 2/5] ovn-northd: Store ETH address of router inport in xreg0.

This helps simplifying logical flows that need to use the port's configured ETH address: - ARP responders for owned IPs - NS responders for owned IPs Signed-off-by: Dumitru Ceara --- northd/ovn-northd.8.xml | 22 --- northd/ovn-northd.c | 148 ++

[ovs-dev] [PATCH v3 ovn 0/5] Reduce number of flows in IN_IP_INPUT table for DNAT.

Patch 1 documents and refactors the usage of OVS registers in logical flows. Patches 2-4 refactor the ARP/NS responder code for logical routers in order to make it easier for patch 5 to configure the flows with different priorities depending on logical port type. Suggested-by: Han Zhou Reported-

[ovs-dev] [PATCH v8] ovsdb-idl: Force IDL retry when missing updates encountered.

Adds a generic recovery mechanism which triggers an IDL retry with fast resync disabled in case the IDL has detected that it ended up in an inconsistent state due to other bugs in the ovsdb-server/ovsdb-idl implementation. Additionally, this commit also: - bumps IDL semantic error logs to level ER

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

> -Original Message- > From: Pai G, Sunil > Sent: Thursday, July 2, 2020 2:14 PM > To: d...@openvswitch.org > Cc: Stokes, Ian ; i.maxim...@ovn.org; > david.march...@redhat.com; Richardson, Bruce ; > Pai G, Sunil ; Tummala, Sivaprasad > > Subject: [PATCH RFC dpdk-latest] build: Add supp

Re: [ovs-dev] [PATCH] dpdk: Refuse running on cpu >= RTE_MAX_LCORE.

On Thu, Jul 2, 2020 at 3:09 PM Ilya Maximets wrote: > > On 6/26/20 2:27 PM, David Marchand wrote: > > DPDK lcores range is [0, RTE_MAX_LCORE[. > > Trying to use a lcore out of this range expose OVS to crashes in DPDK > > mempool > > local cache array. > > Prefer a "clean" abort so that users know

Re: [ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

On 7/2/20 3:13 PM, Sunil Pai G wrote: > Make based build is deprecated in DPDK. Meson based > build to be used for future DPDK releases. > > This updates travis, configure script and documentation > for using DPDK Meson with OVS. > > Signed-off-by: Sunil Pai G Thanks for working on this! Not a

Re: [ovs-dev] 10-25 packet drops every few (10-50) seconds TCP (iperf3)

Thanks Yanqin, I am not seeing any context switches beyond 40usec in our do nothing loop test. But when OvS packets multiple rings(queues) on the same CPU and the number of packet it starts batching (MAX_BURST_SIZE) the toops will will take more time, I can see rings getting getting filled up. And

[ovs-dev] [PATCH RFC dpdk-latest] build: Add support for DPDK meson build.

Make based build is deprecated in DPDK. Meson based build to be used for future DPDK releases. This updates travis, configure script and documentation for using DPDK Meson with OVS. Signed-off-by: Sunil Pai G --- .travis.yml | 3 ++ .travis/linux-build.sh

Re: [ovs-dev] [PATCH] dpdk: Refuse running on cpu >= RTE_MAX_LCORE.

On 6/26/20 2:27 PM, David Marchand wrote: > DPDK lcores range is [0, RTE_MAX_LCORE[. > Trying to use a lcore out of this range expose OVS to crashes in DPDK mempool > local cache array. > Prefer a "clean" abort so that users know that something is wrong with > their configuration. > > Signed-off-b

Re: [ovs-dev] [PATCH V2] dpif-netdev: Do RCU synchronization at fixed interval in PMD main loop.

On 11/12/19 7:38 AM, Nitin Katiyar wrote: > Hi Guys, > Can you please review the patch? It has been in mailing list for long time. Hi. Really sorry for so long delay. I lost track of this change somehow during my relocation and job changes. The patch looks good to me in general. I'll take anot

Re: [ovs-dev] Vhost-user dequeue zero copy removal

On 7/2/20 1:56 PM, Kevin Traynor wrote: > On 02/07/2020 12:18, Stokes, Ian wrote: >> >> >> On 7/1/2020 3:08 PM, Kevin Traynor wrote: >>> On 01/07/2020 13:46, Ilya Maximets wrote: On 7/1/20 1:46 PM, Kevin Traynor wrote: > On 01/07/2020 11:28, Stokes, Ian wrote: >> Hi All, >> >>

Re: [ovs-dev] Vhost-user dequeue zero copy removal

On 02/07/2020 12:18, Stokes, Ian wrote: > > > On 7/1/2020 3:08 PM, Kevin Traynor wrote: >> On 01/07/2020 13:46, Ilya Maximets wrote: >>> On 7/1/20 1:46 PM, Kevin Traynor wrote: On 01/07/2020 11:28, Stokes, Ian wrote: > Hi All, > > While completing validation work for DPDK 18.11.9

Re: [ovs-dev] [PATCH] netdev-offload-tc: Add drop action support.

On Wed, Jul 1, 2020 at 12:30 AM William Tu wrote: > > On Tue, Jun 30, 2020 at 9:13 AM Simon Horman > wrote: > > > > On Tue, Jun 30, 2020 at 09:06:40AM -0700, William Tu wrote: > > > On Tue, Jun 30, 2020 at 9:01 AM Simon Horman > > > wrote: > > > > > > > > On Tue, Jun 30, 2020 at 08:30:45AM -07

Re: [ovs-dev] Vhost-user dequeue zero copy removal

On 7/1/2020 3:08 PM, Kevin Traynor wrote: On 01/07/2020 13:46, Ilya Maximets wrote: On 7/1/20 1:46 PM, Kevin Traynor wrote: On 01/07/2020 11:28, Stokes, Ian wrote: Hi All, While completing validation work for DPDK 18.11.9 and 19.11.3 it was found that zero-copy for vhostuserclient devices

[ovs-dev] Urgent From Hospital

Hello My Beloved this is Ms Marilis Mannik from Estonia writing from the hospital here in Ivory Coast;Dear I want you to know that I'm dying here in this hospital right now which i don't know if i will see some few days to come. My Beloved, i was informed by my doctor that i got poisoned and

Re: [ovs-dev] [PATCH] dpdk: Add commands to configure log levels.

On 7/2/20 10:25 AM, David Marchand wrote: > On Thu, Jul 2, 2020 at 12:05 AM Ilya Maximets wrote: >> On 6/26/20 2:27 PM, David Marchand wrote: >>> Enabling debug logs in dpdk can be a challenge to be sure of what is >>> actually enabled, add commands to list and change those log levels. >> >> Could

Re: [ovs-dev] 10-25 packet drops every few (10-50) seconds TCP (iperf3)

Hi Shahaji, IIUC, 1Hz time tick cannot be disabled even if full dynticks, right? But I have no idea of why it caused packet loss because it should be only a small overhead when rcu_nocbs is enabled . Best Regards, Wei Yanqin === From: Shahaji Bhosle Sent: Thursday, July 2, 2020 6:11

Re: [ovs-dev] [PATCH] dpdk: Add commands to configure log levels.

On Thu, Jul 2, 2020 at 12:05 AM Ilya Maximets wrote: > On 6/26/20 2:27 PM, David Marchand wrote: > > Enabling debug logs in dpdk can be a challenge to be sure of what is > > actually enabled, add commands to list and change those log levels. > > Could you, please, provide some usage examples? > Ma

Re: [ovs-dev] [PATCH v2 ovn 1/5] ovn-northd: Document OVS register usage in logical flows.

On 7/1/20 8:57 PM, Han Zhou wrote: > > > On Tue, Jun 30, 2020 at 2:41 PM Dumitru Ceara > wrote: >> >> Also, use macros instead of bare references to register names. >> >> Signed-off-by: Dumitru Ceara > >> --- >>  northd/ovn-northd.c |  166 > +