RE: [dpdk-dev] [PATCH v1] drivers: remove octeontx2 drivers

2021-12-07 Thread Akhil Goyal
> As per the deprecation notice, In the view of enabling unified driver > for octeontx2(cn9k)/octeontx3(cn10k), removing drivers/octeontx2 > drivers and replace with drivers/cnxk/ which > supports both octeontx2(cn9k) and octeontx3(cn10k) SoCs. > > This patch does the following > > - Replace dri

DPDK Release Status Meeting 2021-12-02

2021-12-07 Thread Mcnamara, John
Release status meeting minutes 2021-12-02 = Agenda: * Release Dates * Subtrees * Roadmaps * LTS * Defects * Opens Participants: * ARM * Canonical * Intel * Marvell * Nvidia * Red Hat Release Dates - The discussion this week was mainly around

[PATCH] net/i40e: enable max frame size at port level

2021-12-07 Thread dapengx . yu
From: Dapeng Yu Currently max frame size is set at queue level, which makes the values of the following counters wrong when a jumbo frame is received. The expected value: rx_good_bytes: 0 rx_errors: 1 rx_oversize_errors: 1 The actual value: rx_good_bytes: 1626 rx_errors: 0 rx_oversize_errors: 0

[Bug 894] [dpdk-20.11.4] lib/librte_eal.a.p/librte_eal_common_rte_random.c.obj build failed on WIN10 with clang8.0.0

2021-12-07 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=894 Bug ID: 894 Summary: [dpdk-20.11.4] lib/librte_eal.a.p/librte_eal_common_rte_random.c.obj build failed on WIN10 with clang8.0.0 Product: DPDK Version: 20.11 Hard

[Bug 895] dpdk 18.11 ip_pipeline :One to one forwarding processing mode appear CPU 0 is 100% of interrupted

2021-12-07 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=895 Bug ID: 895 Summary: dpdk 18.11 ip_pipeline :One to one forwarding processing mode appear CPU 0 is 100% of interrupted Product: DPDK Version: 18.11 Hardware: x86 OS:

Re: [dpdk-dev] [RFC PATCH] ethdev: mtr: enhance input color table features

2021-12-07 Thread Jerin Jacob
On Wed, Nov 17, 2021 at 5:30 PM Jerin Jacob wrote: > > On Mon, Oct 11, 2021 at 8:53 PM Dumitrescu, Cristian > wrote: > > > > Hi Jerin, > Hi Cristian, Could you share your feedback so that I can send the v1? > > > > > > > -Original Message- > > > From: jer...@marvell.com > > > Sent: Fr

Re: [dpdk-dev] [PATCH v1] drivers: remove octeontx2 drivers

2021-12-07 Thread Ferruh Yigit
On 12/7/2021 7:39 AM, Jerin Jacob wrote: On Mon, Dec 6, 2021 at 7:05 PM Ferruh Yigit wrote: On 12/6/2021 8:35 AM, jer...@marvell.com wrote: From: Jerin Jacob As per the deprecation notice, In the view of enabling unified driver for octeontx2(cn9k)/octeontx3(cn10k), removing drivers/octeontx

[PATCH v2 0/4] arm64 cross docs improvements/fixes

2021-12-07 Thread Juraj Linkeš
The docs are a big outdated and in need of updates. Using CFLAGS and LDFLAGS is an artefact the old build system, Meson doesn't support those well. Numaclt can't be cross compiled with clang. The names of downloaded binaried have an extra -none- in them. And to make these changes more easily un

[PATCH v2 1/4] doc: arm64 cross build CFLAGS/LDFLAGS alternatives

2021-12-07 Thread Juraj Linkeš
Remove CFLAGS and LDFLAGS since Meson doesn't support them well enough. Add Meson alternatives: -Dc_args and -Dc_link_args on the command line and in cross files. Signed-off-by: Juraj Linkeš --- .../linux_gsg/cross_build_dpdk_for_arm64.rst | 25 --- 1 file changed, 21 insertions

[PATCH v2 2/4] doc: arm64 cross build binary names update

2021-12-07 Thread Juraj Linkeš
The newer version have an extra -none- in the name. Signed-off-by: Juraj Linkeš --- doc/guides/linux_gsg/cross_build_dpdk_for_arm64.rst | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/doc/guides/linux_gsg/cross_build_dpdk_for_arm64.rst b/doc/guides/linux_gsg/cros

[PATCH v2 3/4] doc: arm64 cross build numactl compilers

2021-12-07 Thread Juraj Linkeš
Numactl cross compilation doesn't work with clang, remove it and fix the gcc cross compiler executable name. Signed-off-by: Juraj Linkeš --- doc/guides/linux_gsg/cross_build_dpdk_for_arm64.rst | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/doc/guides/linux_gsg/cross_b

[PATCH v2 4/4] docs: add an example arm64 cross file

2021-12-07 Thread Juraj Linkeš
The docs mention modifications and additions to the cross file, but there is no demonstration of how those should look like. Signed-off-by: Juraj Linkeš --- .../linux_gsg/cross_build_dpdk_for_arm64.rst | 26 +++ 1 file changed, 26 insertions(+) diff --git a/doc/guides/linux_gsg

Re: [dpdk-dev] [PATCH v1] drivers: remove octeontx2 drivers

2021-12-07 Thread Kevin Traynor
On 07/12/2021 11:01, Ferruh Yigit wrote: On 12/7/2021 7:39 AM, Jerin Jacob wrote: On Mon, Dec 6, 2021 at 7:05 PM Ferruh Yigit wrote: On 12/6/2021 8:35 AM, jer...@marvell.com wrote: From: Jerin Jacob As per the deprecation notice, In the view of enabling unified driver for octeontx2(cn9k)/o

Re: [PATCH 1/3] common/mlx5: add min WQE size for striding RQ

2021-12-07 Thread Ferruh Yigit
On 11/23/2021 6:38 PM, michae...@nvidia.com wrote: From: Michael Baum Some devices have a WQE size limit for striding RQ. On some newer devices, this limitation is smaller and information on its size is provided by the firmware. This patch adds the attribute query from firmware: the minimum req

Re: [PATCH 2/3] net/mlx5: improve stride parameter names

2021-12-07 Thread Ferruh Yigit
On 11/23/2021 6:38 PM, michae...@nvidia.com wrote: From: Michael Baum In the striding RQ management there are two important parameters, the size of the single stride in bytes and the number of strides. Both the data-path structure and config structure keep the log of the above parameters. Howev

Re: [PATCH 3/3] net/mlx5: fix missing adjustment MPRQ stride devargs

2021-12-07 Thread Ferruh Yigit
On 11/23/2021 6:38 PM, michae...@nvidia.com wrote: From: Michael Baum In Multy-Packet RQ creation, the user can choose the number of strides Multi-Packet ? and their size in bytes. The user updates it using specific devargs for both of these parameters. The above two parameters determine the

Re: [PATCH] net/mlx5: fix metadata endianness in modify field action

2021-12-07 Thread Ferruh Yigit
On 11/29/2021 12:32 PM, Viacheslav Ovsiienko wrote: As modify field action immediate source parameter the metadata should follow the CPU endianness (according to SET_META action structure format), and mlx5 PMD wrongly handled the immediate parameter metadata buffer as big-endian, resulting in wro

The dpdk 18.11 version is on the i40e network card and the overcurrent 1G execution results in a high occupation of 0 cores

2021-12-07 Thread ????
Hello, I used the ip_pipeline program of 18.11 dpdk and made it flow 1GB/s. I found that the 0-core CPU occupies very high (usleep has been added in the for loop of the ip_pipeline main function, which should not occupy very high CPU).Use perf to view and find that it is occupied in dw_readl.

RE: [dpdk-dev] [RFC PATCH] ethdev: mtr: enhance input color table features

2021-12-07 Thread Dumitrescu, Cristian
HI Jerin, Sorry for my delay. I am currently in vacation until the beginning on January 2022, so my response is slower than usual. > -Original Message- > From: Jerin Jacob > Sent: Tuesday, December 7, 2021 9:55 AM > To: Dumitrescu, Cristian > Cc: jer...@marvell.com; Thomas Monjalon ; Y

[PATCH v4 0/4] regex/cn9k: use cnxk infrastructure

2021-12-07 Thread lironh
From: Liron Himi 3 patches add support for REE into cnkx infrastructure. the last patch change the octeontx2 driver to use the new cnxk code. in addition all references to octeontx2/otx2 were replaced with cn9k. v4: - squashed the 4th patch v3: - fix documentation issues v2: - fix review comme

[PATCH v4 1/4] common/cnxk: add REE HW definitions

2021-12-07 Thread lironh
From: Liron Himi adding REE (Regular Expression Engine) HW definitions Signed-off-by: Liron Himi --- drivers/common/cnxk/hw/ree.h | 126 +++ drivers/common/cnxk/hw/rvu.h | 5 ++ 2 files changed, 131 insertions(+) create mode 100644 drivers/common/cnxk/hw/ree.

[PATCH v4 2/4] common/cnxk: add REE mbox definitions

2021-12-07 Thread lironh
From: Liron Himi add REE mbox definitions Signed-off-by: Liron Himi --- drivers/common/cnxk/roc_mbox.h | 100 + 1 file changed, 100 insertions(+) diff --git a/drivers/common/cnxk/roc_mbox.h b/drivers/common/cnxk/roc_mbox.h index b63fe108c9..e97d93e261 100644 --

[PATCH v4 3/4] common/cnxk: add REE support

2021-12-07 Thread lironh
From: Liron Himi extend cnxk infrastructure to support REE Signed-off-by: Liron Himi --- drivers/common/cnxk/meson.build | 1 + drivers/common/cnxk/roc_api.h | 4 + drivers/common/cnxk/roc_constants.h | 2 + drivers/common/cnxk/roc_platform.c | 1 + drivers/common/cnxk/roc_p

[PATCH v4 4/4] regex/cn9k: use cnxk infrastructure

2021-12-07 Thread lironh
From: Liron Himi update driver to use the REE cnxk code replace octeontx2/otx2 with cn9k Signed-off-by: Liron Himi --- MAINTAINERS | 8 +- doc/guides/platform/cnxk.rst | 3 + doc/guides/platform/octeontx2.rst | 3 - .../regex

[PATCH v12 01/11] eal/windows: return ENOTSUP for not supported API

2021-12-07 Thread Jie Zhou
UT memory_autotest on Windows has 2 failed cases on eal APIs eal_memalloc_get_seg_fd and eal_memalloc_get_seg_fd_offset. These 2 APIs are not supported on Windows yet. Should return ENOTSUP such that in test_memory.c these 2 ENOTSUP cases will not be marked as failures, same as other ENOTSUP cases.

[PATCH v12 00/11] app/test: enable subset of tests on Windows

2021-12-07 Thread Jie Zhou
The goal of this patchset is to enable unit tests in CI for Windows. It mainly contains: - Replace POSIX specific codes - Fix some lib and tests per failures investigation - Add test stubs for not yet supported ones on Windows - Replace .sh script with .py script for meson.build - Enable build and

[PATCH v12 04/11] app/test: skip interrupt tests on Windows

2021-12-07 Thread Jie Zhou
Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_interrupts.c | 10 +++

[PATCH v12 03/11] app/test: fix incorrect errno variable

2021-12-07 Thread Jie Zhou
Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_memory.c | 2 +- 1 file changed, 1 insertion(+), 1 d

[PATCH v12 06/11] app/test: differentiate a strerror on different OS

2021-12-07 Thread Jie Zhou
On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" failure. Signed-off-by: Jie Zhou

[PATCH v12 07/11] app/test: remove two alarm_autotest cases

2021-12-07 Thread Jie Zhou
Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a

[PATCH v12 08/11] app/test: resolve name collision

2021-12-07 Thread Jie Zhou
Add OP_ prefix to resolve name collision to enable hash_perf test Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_hash_perf.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c index

[PATCH v12 05/11] app/test: skip two logs_autotest cases on Windows

2021-12-07 Thread Jie Zhou
DPDK logs_autotest on Windows failed at "dynamic log types" tests. The failures are on 2 test cases for rte_log_set_level_regexp API, due to regular expression is not supported on Windows in DPDK yet and regcomp/regexec are just stubs on Windows (in regex.h). In app\test\test_logs.c, ifndef these

[PATCH v12 10/11] app/test: replace .sh script with .py script

2021-12-07 Thread Jie Zhou
- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26 +

[PATCH v12 11/11] app/test: enable unit test for Windows

2021-12-07 Thread Jie Zhou
Enable a subset of unit tests for Windows CI - For driver tests, it's better for driver owners to enable each. For example, the cryptodev tests will be enabled by "patch-18949: app/test: enable crypto unit tests on Windows" (which depends on this patchset to be merged). - For dump tests, curre

[PATCH v12 09/11] app/test: add test stubs for not supported ones

2021-12-07 Thread Jie Zhou
Add test stubs for tests which are not yet supported for Windows: - The libraries the tests depend on are not enabled on Windows yet - The tests can compile but with issue still under investigation * test_func_reentrancy: Windows EAL has no protection against repeated calls. * test_lc

[PATCH v12 02/11] app/test: remove POSIX-specific code

2021-12-07 Thread Jie Zhou
- Replace POSIX-specific code with DPDK equivalents or conditional platform specific code, or conditionally disable it on Windows - Use NUL on Windows as /dev/null for Unix - Exclude tests not supported on Windows yet * multi-process * PMD performance statistics display on signal Signed-off-

[PATCH v2] doc/dlb2: update dlb2 documentation

2021-12-07 Thread Rashmi Shetty
Number of direct credits, atomic inflight and history list are updated to DLB2.0 supported sizes. As DLB2.0 does not provide dev arg to override the default per-queue atomic inflight allocation, it is removed from the documentation. Signed-off-by: Rashmi Shetty --- doc/guides/eventdevs/dlb2.rst

[PATCH v13 01/11] eal/windows: return ENOTSUP for not supported API

2021-12-07 Thread Jie Zhou
UT memory_autotest on Windows has 2 failed cases on eal APIs eal_memalloc_get_seg_fd and eal_memalloc_get_seg_fd_offset. These 2 APIs are not supported on Windows yet. Should return ENOTSUP such that in test_memory.c these 2 ENOTSUP cases will not be marked as failures, same as other ENOTSUP cases.

[PATCH v13 03/11] app/test: fix incorrect errno variable

2021-12-07 Thread Jie Zhou
Fix incorrect errno variable used in memory autotest. Use rte_errno instead. Fixes: 086d426406bd ("app/test: fix memory autotests on FreeBSD") Cc: bruce.richard...@intel.com Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_memory.c | 2 +- 1 file changed, 1 insertion(+), 1 d

[PATCH v13 04/11] app/test: skip interrupt tests on Windows

2021-12-07 Thread Jie Zhou
Even though test_interrupts.c can compile on Windows, skip interrupt tests for now since majority of eal_interrupt on Windows are stubs. Will remove the skip after interrupt being fully enabled on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_interrupts.c | 10 +++

[PATCH v13 00/11] app/test: enable subset of tests on Windows

2021-12-07 Thread Jie Zhou
The goal of this patchset is to enable unit tests in CI for Windows. It mainly contains: - Replace POSIX specific codes - Fix some lib and tests per failures investigation - Add test stubs for not yet supported ones on Windows - Replace .sh script with .py script for meson.build - Enable build and

[PATCH v13 02/11] app/test: remove POSIX-specific code

2021-12-07 Thread Jie Zhou
- Replace POSIX-specific code with DPDK equivalents or conditionally disable it on Windows - Use NUL on Windows as /dev/null for Unix - Exclude tests not supported on Windows yet * multi-process * PMD performance statistics display on signal Signed-off-by: Jie Zhou Signed-off-by: Dmitry Koz

[PATCH v13 05/11] app/test: skip two logs_autotest cases on Windows

2021-12-07 Thread Jie Zhou
DPDK logs_autotest on Windows failed at "dynamic log types" tests. The failures are on 2 test cases for rte_log_set_level_regexp API, due to regular expression is not supported on Windows in DPDK yet and regcomp/regexec are just stubs on Windows (in regex.h). In app\test\test_logs.c, ifndef these

[PATCH v13 07/11] app/test: remove two alarm_autotest cases

2021-12-07 Thread Jie Zhou
Remove two alarm_autotest test cases which do bogus range check on Windows. Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/test_alarm.c | 4 1 file changed, 4 insertions(+) diff --git a/app/test/test_alarm.c b/app/test/test_alarm.c index b4034339b8..70e97a3109 100644 --- a

[PATCH v13 10/11] app/test: replace .sh script with .py script

2021-12-07 Thread Jie Zhou
- Add python script to check if system supports hugepages - Remove corresponding .sh script - Replace calling of .sh with corresponding .py in meson.build Signed-off-by: Jie Zhou Acked-by: Dmitry Kozlyuk --- app/test/has-hugepage.sh | 11 --- app/test/has_hugepage.py | 26 +

[PATCH v13 08/11] app/test: resolve name collision

2021-12-07 Thread Jie Zhou
Add OP_ prefix to resolve name collision to enable hash_perf test Signed-off-by: Jie Zhou Signed-off-by: Dmitry Kozlyuk --- app/test/test_hash_perf.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/app/test/test_hash_perf.c b/app/test/test_hash_perf.c index

[PATCH v13 09/11] app/test: add test stubs for not supported ones

2021-12-07 Thread Jie Zhou
Add test stubs for tests which are not yet supported for Windows: - The libraries the tests depend on are not enabled on Windows yet - The tests can compile but with issue still under investigation * test_func_reentrancy: Windows EAL has no protection against repeated calls. * test_lc

[PATCH v13 11/11] app/test: enable unit test on Windows

2021-12-07 Thread Jie Zhou
Enable a subset of unit tests for Windows CI - For driver tests, driver owners should enable corresponding tests when enabling driver for Windows. For example, the cryptodev tests will be enabled by "patch-18949: app/test: enable crypto unit tests on Windows" (which depends on this patchset

[PATCH v13 06/11] app/test: differentiate a strerror on different OS

2021-12-07 Thread Jie Zhou
On Windows, strerror returns just "Unknown error" for errnum greater than MAX_ERRNO, while linux and freebsd returns "Unknown error ", which is the current expectation for errno_autotest. Differentiate the error string on Windows to remove a "duplicate error code" failure. Signed-off-by: Jie Zhou

RE: [PATCH v4 0/2] Add functions to calculate UDP/TCP cksum in mbuf

2021-12-07 Thread Pai G, Sunil
Tested-by: Sunil Pai G Hi , Tested the series with OVS - with both TCP and UDP packets by printing out the csum before sending and after reception of packets from testpmd. Getting expected csum values with this patch. Thanks and regards Sunil