[ovs-dev] [PATCH] python: Send non-zero flag for a SSL socket

2022-08-07 Thread Miro Tomaska
Python std library SSLSocket.send does not allow non-zero value for the optional flag. pyOpenSSL was recently switched for the Python standard library ssl module commit 68543dd523bd00f53fa7b91777b962ccb22ce679 (python: Replace pyOpenSSL with ssl). Python SSLsocket.send() does not allow non-zero

Re: [ovs-dev] [PATCH ovn] run CI for every commit when merging several commits, not for the latest.

2022-08-07 Thread Igor Zhukov
Great! I see it works: https://github.com/ovn-org/ovn/commits/main > On Fri, Aug 5, 2022 at 1:05 AM Igor Zhukov wrote: > >> From: Igor Zhukov >> >> Docs: >> https://docs.github.com/en/actions/using-jobs/using-concurrency#example-using-a-fallback-value >> >> Test: https://github.com/fsb4000/ov

Re: [ovs-dev] [PATCH ovn] controller: physical: fix regression for container ports

2022-08-07 Thread Numan Siddique
On Fri, Jul 29, 2022 at 12:04 AM Ihar Hrachyshka wrote: > > Oops, sorry about that. This should be backported to 22.06. > > Acked-By: Ihar Hrachyshka > On Thu, Jul 28, 2022 at 9:55 AM Lorenzo Bianconi > wrote: > > > > After commit 'd07e5f99d ("Introduce match_outport_dp_and_port_keys > > in phys

Re: [ovs-dev] [PATCH ovn]OVN-CI: ovn unit tests run in parallel jobs.

2022-08-07 Thread Numan Siddique
On Fri, Aug 5, 2022 at 9:18 PM Dumitru Ceara wrote: > > On 8/4/22 17:32, Mohammad Heib wrote: > > Ovn unit tests supported matrix size has been increased > > after adding support to monitor_all and northd_parallelization > > options recently, and that increased the execution time of the ovn-ci job

Re: [ovs-dev] [PATCH ovn] pinctrl: fix bug of monitor status in health check

2022-08-07 Thread Numan Siddique
On Mon, Aug 1, 2022 at 11:06 PM wangchuanlei wrote: > > when vm of backend do not send reply packet, > monitor state should change from waiting to offline, > this patch is to fix it! > > Signed-off-by: wangchuanlei This patch breaks the service monitor functionality for UDP. For UDP protocol,

Re: [ovs-dev] [PATCH ovn] run CI for every commit when merging several commits, not for the latest.

2022-08-07 Thread Numan Siddique
On Fri, Aug 5, 2022 at 1:05 AM Igor Zhukov wrote: > > From: Igor Zhukov > > Docs: > https://docs.github.com/en/actions/using-jobs/using-concurrency#example-using-a-fallback-value > > Test: https://github.com/fsb4000/ovn/actions and https://imgur.com/a/mUBQWSO > > Signed-off-by: Igor Zhukov Tha

Re: [ovs-dev] [PATCH ovn] tests: enable and fix vif-provider test scenario

2022-08-07 Thread Numan Siddique
On Thu, Jul 28, 2022 at 4:44 PM Frode Nordahl wrote: > > Hello, Ihar, > > On Thu, Jul 28, 2022 at 1:28 AM Ihar Hrachyshka wrote: > > > > Signed-off-by: Ihar Hrachyshka > > --- > > controller/test-vif-plug.c | 2 +- > > tests/ovn-vif-plug.at | 2 +- > > tests/testsuite.at | 1 + > >

Re: [ovs-dev] [PATCH ovn] northd: add support to make l3dgw ports fully distributed

2022-08-07 Thread Numan Siddique
On Wed, Jul 20, 2022 at 12:15 AM Vladislav Odintsov wrote: > > This is used when traffic from HW VTEP goes to > routable networks and logical switch to which VTEP > logical port is attached also needs to support > distributed routing features such as NAT and others. > > Signed-off-by: Vladislav Od

Re: [ovs-dev] [PATCH ovn 2/2] release-process: Add 2023 calendar.

2022-08-07 Thread Numan Siddique
On Thu, Jul 28, 2022 at 12:28 AM Mark Michelson wrote: > > This also removes the 2021 calendar since that has come and gone. > > Signed-off-by: Mark Michelson Acked-by: Numan Siddique Numan > --- > Documentation/internals/release-process.rst | 20 ++-- > 1 file changed, 10 in

Re: [ovs-dev] [PATCH ovn 1/2] release-process: Change master to main.

2022-08-07 Thread Numan Siddique
On Thu, Jul 28, 2022 at 12:28 AM Mark Michelson wrote: > > The document makes reference to the "master" branch throughout, but it > has been renamed "main" for some time now. > > Signed-off-by: Mark Michelson Acked-by: Numan Siddique Numan > --- > Documentation/internals/release-process.rst

[ovs-dev] [PATCH v4 9/9] mfex-avx512: Add support for tunnel packets in avx512 mfex.

2022-08-07 Thread Kumar Amber
This patch adds the necessary support to avx512 mfex to support handling of tunnel packet type. Signed-off-by: Kumar Amber --- lib/dpif-netdev-avx512.c | 16 ++-- lib/dpif-netdev-extract-avx512.c | 146 +- lib/dpif-netdev-private-extract.c | 4 +- 3 files

[ovs-dev] [PATCH v4 8/9] mfex-study: Modify study func to select outer and inner mfex funcs.

2022-08-07 Thread Kumar Amber
The Mfex study function is split into outer and inner to allow for independent selection and studying of packets in outer and inner flows to different ISA optimized Mfexs. Signed-off-by: Kumar Amber --- lib/dpif-netdev-extract-study.c | 126 +--- 1 file changed, 83 in

[ovs-dev] [PATCH v4 7/9] dpif-mfex: Change mfex fn pointer prototype to include md_is_valid.

2022-08-07 Thread Kumar Amber
The md_is_valid parameter is passed from DPIF to MFEX to allow mfex functions to detect the tunneling and decide the processing of Inner packets in static predictable branches. Signed-off-by: Kumar Amber --- lib/dpif-netdev-avx512.c | 3 ++- lib/dpif-netdev-extract-avx512.c | 9 +

[ovs-dev] [PATCH v4 6/9] dpif-mfex: Modify set/get mfex commands to include inner.

2022-08-07 Thread Kumar Amber
The set command in MFEX is changed as to allow the user to select different optimized mfex ISA for processing Inner packets in case of tunneling. $ ovs-appctl dpif-netdev/miniflow-parser-set -pmd 3 study 1024 -inner The get command is modified to indcitate both inner and Outer MFEXs in use. Sign

[ovs-dev] [PATCH v4 5/9] dpif-netdev: Add function pointer for dpif re-circulate.

2022-08-07 Thread Kumar Amber
The patch adds and re-uses the dpif set command to set the function pointers to be used to switch between different inner dpifs. Signed-off-by: Kumar Amber Signed-off-by: Cian Ferriter Co-authored-by: Cian Ferriter --- v3: - Add description to the dpif recirc function. - Fix use of return val

[ovs-dev] [PATCH v4 4/9] dpif-netdev-avx512: Add inner packet handling to dpif.

2022-08-07 Thread Kumar Amber
This patch adds the necessary changes required to support tunnel packet types in avx512 dpif. Signed-off-by: Kumar Amber Signed-off-by: Cian Ferriter Co-authored-by: Cian Ferriter --- v4: - Rebase onto Simple match. v3: - Apply in_port optimization suggested by Harry. --- --- lib/dpif-netdev-

[ovs-dev] [PATCH v4 3/9] dpif-netdev-avx512: Refactor avx512 dpif and create new APIs.

2022-08-07 Thread Kumar Amber
Create new APIs for the avx512 DPIF, enabling one baseline common code to be specialized into DPIF implementations for "outer" processing, and "recirc" processing. Signed-off-by: Kumar Amber Signed-off-by: Cian Ferriter Co-authored-by: Cian Ferriter --- v4: - Rebase onto master. v3: - Fix comm

[ovs-dev] [PATCH v4 2/9] dpif-netdev: Refactor hash function to own header.

2022-08-07 Thread Kumar Amber
The refactor allows us to use hash function accross multiple files which was earlier restricted to dpif-netdev.c only. Signed-off-by: Kumar Amber Signed-off-by: Cian Ferriter Co-authored-by: Cian Ferriter --- v3: - Fix minor comments from Harry. --- --- lib/dpif-netdev-private-dpcls.h | 23 ++

[ovs-dev] [PATCH v4 1/9] dpif-netdev: Refactor per thread recirc data allocation.

2022-08-07 Thread Kumar Amber
The refactor allows us to use *recirc_depth_get() to obtain the depth across ovs which was previously limited to only dpif-netdev.c. Signed-off-by: Kumar Amber Signed-off-by: Cian Ferriter Co-authored-by: Cian Ferriter Acked-by: Harry van Haaren --- lib/dpif-netdev-private-dpif.c | 2 ++ lib/

[ovs-dev] [PATCH v4 0/9] DPIF + MFEX Inner Vxlan AVX512

2022-08-07 Thread Kumar Amber
This Series of Patchsets introduce the Optimizations for supporting Vxlan tunneled packets in DPIF and MFEX. Along with the optimization various refactoring of scalar path is done to be used accross without duplication. Over the Tests we have observed a gain of approximate 20~25% gain in performan

Re: [ovs-dev] [ovs-dev v2 1/4] ofproto-dpif-upcall: fix push_dp_ops

2022-08-07 Thread Peng He
Hello, can anyone have a look at those patches? 0-day Robot 于2022年6月4日周六 23:38写道: > Bleep bloop. Greetings Peng He, 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: > E