Re: [dpdk-dev] [Bug 826] red_autotest random failures

2021-11-18 Thread Thomas Monjalon
18/11/2021 23:10, Liguzinski, WojciechX: > Hi, > > I was trying to reproduce this test failure, but for me RED tests are > passing. > I was running the exact test command like the one described in Bug 826 - > 'red_autotest' on the current main branch. The test is not always failing. There are

RE: [PATCH v1] net/vhost: add queue status check

2021-11-18 Thread Li, Miao
Hi > -Original Message- > From: Maxime Coquelin > Sent: Tuesday, November 16, 2021 5:36 PM > To: Li, Miao ; dev@dpdk.org > Cc: Xia, Chenbo > Subject: Re: [PATCH v1] net/vhost: add queue status check > > > > On 11/16/21 10:34, Maxime Coquelin wrote: > > > > > > On 11/16/21 17:44, Miao

Re: [PATCH] net/bnxt: fail init when mbuf allocation fails

2021-11-18 Thread Ajit Khaparde
On Thu, Nov 18, 2021 at 9:40 PM Ajit Khaparde wrote: > > Fix driver init when Rx mbuf allocation fails. > If we continue to use the driver with whatever rings were > created successfully, it can cause unexpected behavior. > > Signed-off-by: Ajit Khaparde > Reviewed-by: Somnath Kotur > Reviewed-b

Re: [PATCH] net/bnxt: fix crash caused by error recovery

2021-11-18 Thread Ajit Khaparde
On Thu, Nov 18, 2021 at 7:57 PM Somnath Kotur wrote: > > bnxt_stop_rxtx() does not stop data path processing as intended > as it does not update the recently introduced fast-path pointers > '(struct rte_eth_fp_ops)->rx_pkt_burst'. Since both the burst routines > only use the fast-path pointer, the

[PATCH] net/bnxt: fail init when mbuf allocation fails

2021-11-18 Thread Ajit Khaparde
Fix driver init when Rx mbuf allocation fails. If we continue to use the driver with whatever rings were created successfully, it can cause unexpected behavior. Signed-off-by: Ajit Khaparde Reviewed-by: Somnath Kotur Reviewed-by: Kalesh AP --- drivers/net/bnxt/bnxt_hwrm.c | 41

[dpdk-dev] [PATCH] doc: fix memif driver acronyms

2021-11-18 Thread jerinj
From: Jerin Jacob The commit d250589d5702 ("net/memif: replace master/slave arguments") replaced master/slave terms to server/client terms. Fix the documentation to reflect the same. Fixes: d250589d5702 ("net/memif: replace master/slave arguments") Cc: sta...@dpdk.org Signed-off-by: Jerin Jacob

Re: [dpdk-dev] Minutes of Technical Board Meeting, 2021-Oct-27

2021-11-18 Thread Jerin Jacob
On Fri, Nov 19, 2021 at 1:39 AM Honnappa Nagarahalli wrote: > > > > > > > > > > > There was a comment to remove the TLV length. I will do that > > > > > > next version with implementation. > > > > > > > > > > > > Identified the following set of work for this. > > > > > > > > > > > > 1) Common cod

[PATCH] net/bnxt: fix crash caused by error recovery

2021-11-18 Thread Somnath Kotur
bnxt_stop_rxtx() does not stop data path processing as intended as it does not update the recently introduced fast-path pointers '(struct rte_eth_fp_ops)->rx_pkt_burst'. Since both the burst routines only use the fast-path pointer, the real burst routines get invoked instead of the dummy ones set b

RE: [dpdk-dev] [Bug 826] red_autotest random failures

2021-11-18 Thread Liguzinski, WojciechX
Hi, I was trying to reproduce this test failure, but for me RED tests are passing. I was running the exact test command like the one described in Bug 826 - 'red_autotest' on the current main branch. Here is an example when DPDK is build without RTE_SCHED_CMAN enabled, but with this flag set to

Re: [PATCH v1] gpudev: return EINVAL if invalid input pointer for free and unregister

2021-11-18 Thread Tyler Retzlaff
On Thu, Nov 18, 2021 at 07:28:02PM +, eagost...@nvidia.com wrote: > From: Elena Agostini > > Signed-off-by: Elena Agostini > --- > lib/gpudev/gpudev.c | 10 ++ > lib/gpudev/rte_gpudev.h | 2 ++ > 2 files changed, 12 insertions(+) > > diff --git a/lib/gpudev/gpudev.c b/lib/gpud

RE: [dpdk-dev] Minutes of Technical Board Meeting, 2021-Oct-27

2021-11-18 Thread Honnappa Nagarahalli
> > > > > > > There was a comment to remove the TLV length. I will do that > > > > > next version with implementation. > > > > > > > > > > Identified the following set of work for this. > > > > > > > > > > 1) Common code at lib/dwa/ > > > > > 2) Marvell DPU based driver at drivers/dwa/cnxk/ > > >

[PATCH] net/memif: allow stopping and closing device

2021-11-18 Thread Junxiao Shi
Bugzilla ID: 888 Fixes: febc855b358e ("ethdev: forbid closing started device") Signed-off-by: Junxiao Shi --- drivers/net/memif/rte_eth_memif.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/memif/rte_eth_memif.c b/drivers/net/memif/rte_eth_memif.c i

[Bug 888] cannot close memif port

2021-11-18 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=888 Bug ID: 888 Summary: cannot close memif port Product: DPDK Version: 21.11 Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: Normal

Re: [dpdk-dev] Minutes of Technical Board Meeting, 2021-Oct-27

2021-11-18 Thread Jerin Jacob
On Thu, Nov 18, 2021 at 8:44 PM Honnappa Nagarahalli wrote: > > > > > > > There was a comment to remove the TLV length. I will do that next > > > > version with implementation. > > > > > > > > Identified the following set of work for this. > > > > > > > > 1) Common code at lib/dwa/ > > > > 2) Mar

Re: [PATCH v1] gpudev: return EINVAL if invalid input pointer for free and unregister

2021-11-18 Thread Elena Agostini
> From: Stephen Hemminger > Date: Thursday, 18 November 2021 at 17:21 > To: Elena Agostini > Cc: dev@dpdk.org > Subject: Re: [PATCH v1] gpudev: return EINVAL if invalid input pointer for > free and unregister > External email: Use caution opening links or attachments> > > On Thu, 18 Nov 2021 19

Re: [PATCH v1] gpudev: return EINVAL if invalid input pointer for free and unregister

2021-11-18 Thread Stephen Hemminger
On Thu, 18 Nov 2021 19:28:02 + wrote: > diff --git a/lib/gpudev/gpudev.c b/lib/gpudev/gpudev.c > index 2b174d8bd5..97575ed979 100644 > --- a/lib/gpudev/gpudev.c > +++ b/lib/gpudev/gpudev.c > @@ -576,6 +576,11 @@ rte_gpu_mem_free(int16_t dev_id, void *ptr) > return -rte_errno; >

[PATCH v2] common/mlx5: fix mempool registration

2021-11-18 Thread Dmitry Kozlyuk
Mempool registration was not correctly processing mempools with RTE_PKTMBUF_F_PINEND_EXT_BUF flag set ("pinned mempools" for short), because it is not known at registration time whether the mempool is a pktmbuf one, and its elements may not yet be initialized to analyze them. Attempts had been made

RE: [EXT] [PATCH -v1] bbdev: update num_ops type to be uint32_t in rte_bbdev_enc_op_alloc_bulk

2021-11-18 Thread Chautru, Nicolas
Hi Akhil, The change below from Mingshan is not for 21.11, this is targeting following release. It should have been mentioned in cover letter. Thanks Nic > -Original Message- > From: Akhil Goyal > Sent: Wednesday, November 17, 2021 11:21 PM > To: Zhang, Mingshan ; dev@dpdk.org; > akhi

RE: [dpdk-dev] Minutes of Technical Board Meeting, 2021-Oct-27

2021-11-18 Thread Honnappa Nagarahalli
> > > There was a comment to remove the TLV length. I will do that next > > > version with implementation. > > > > > > Identified the following set of work for this. > > > > > > 1) Common code at lib/dwa/ > > > 2) Marvell DPU based driver at drivers/dwa/cnxk/ > > > 3) Test application at app/test

[PATCH v1 1/3] fix PMD wording typo

2021-11-18 Thread Sean Morrissey
Removing the use of driver following PMD as its unnecessary. Signed-off-by: Sean Morrissey Signed-off-by: Conor Fogarty --- app/test-pmd/cmdline.c| 4 +-- doc/guides/bbdevs/turbo_sw.rst| 2 +- doc/guides/cryptodevs/virtio.rst | 2 +- doc/g

[PATCH v1 3/3] doc: capitalise PMD

2021-11-18 Thread Sean Morrissey
The doc's contain references to pmd but the proper use is to use PMD. Signed-off-by: Sean Morrissey --- doc/guides/cryptodevs/ccp.rst | 2 +- doc/guides/cryptodevs/openssl.rst | 2 +- doc/guides/cryptodevs/overview.rst | 2 +- doc/guides/eventdevs/opdl.rst | 2 +- doc/

[PATCH v1 2/3] fix 'the the' typo

2021-11-18 Thread Sean Morrissey
Remove the use of double "the" as it does not make sense. Signed-off-by: Sean Morrissey Signed-off-by: Conor Fogarty --- app/test/test_barrier.c | 2 +- doc/guides/sample_app_ug/keep_alive.rst | 2 +- drivers/bus/vmbus/rte_bus_vmbus.h| 2 +- drivers/common/cpt/cpt_hw_t

[PATCH v1 0/3] Fix typo's and capitalise PMD

2021-11-18 Thread Sean Morrissey
This patchset cleans up the use of the phrase PMD throughout dpdk by capitalising pmd and also removing the use of the word driver following PMD. This patchset also removes some unnecessary duplication of the word "the" in comments and docs. Sean Morrissey (3): fix PMD wording typo fix 'the t

[PATCH] doc: support IPsec Multi-buffer lib v1.1

2021-11-18 Thread Pablo de Lara
Updated AESNI MB and AESNI GCM, KASUMI, ZUC and SNOW3G PMD documentation guides with information about the latest Intel IPSec Multi-buffer library supported. Signed-off-by: Pablo de Lara --- doc/guides/cryptodevs/aesni_gcm.rst | 8 doc/guides/cryptodevs/aesni_mb.rst | 8 doc/g

Re: [PATCH] common/mlx5: decrease log level for hlist creation

2021-11-18 Thread David Marchand
Hi Slava, On Wed, Nov 17, 2021 at 3:46 PM David Marchand wrote: > > On Wed, Nov 17, 2021 at 2:28 PM Slava Ovsiienko > wrote: > > > > I've re-checked the mlx5_hlist_create() call tree. > > > > And it seems all the calls are done with hardcoded const values for "size" > > argument, > > and all

Re: [PATCH] ci: add Fedora 35 container in GHA

2021-11-18 Thread Aaron Conole
David Marchand writes: > On Tue, Nov 16, 2021 at 3:44 PM Aaron Conole wrote: >> > Build DPDK with Fedora 35 containers. >> > Differences with the Ubuntu GHA vm images: >> > - tasks are run as root in containers, no need for sudo, >> > - compiler must be explicitly installed: clang is not in cont

Re: [dpdk-dev] [PATCH] config/x86: add support for AMD platform

2021-11-18 Thread Thomas Monjalon
18/11/2021 14:52, Bruce Richardson: > On Thu, Nov 18, 2021 at 01:25:38PM +0100, Thomas Monjalon wrote: > > I request a techboard decision for this patch. > > > > > > 02/11/2021 20:04, Thomas Monjalon: > > > 02/11/2021 19:45, David Marchand: > > > > On Tue, Nov 2, 2021 at 3:53 PM Aman Kumar > >

Re: [dpdk-dev] [PATCH] config/x86: add support for AMD platform

2021-11-18 Thread Bruce Richardson
On Thu, Nov 18, 2021 at 01:25:38PM +0100, Thomas Monjalon wrote: > I request a techboard decision for this patch. > > > 02/11/2021 20:04, Thomas Monjalon: > > 02/11/2021 19:45, David Marchand: > > > On Tue, Nov 2, 2021 at 3:53 PM Aman Kumar wrote: > > > > > > > > -Dcpu_instruction_set=znverX mes

[PATCH v1] gpu/cuda: properly set rte_errno

2021-11-18 Thread eagostini
From: Elena Agostini Signed-off-by: Elena Agostini --- drivers/gpu/cuda/cuda.c | 229 +++- 1 file changed, 153 insertions(+), 76 deletions(-) diff --git a/drivers/gpu/cuda/cuda.c b/drivers/gpu/cuda/cuda.c index 24ae630d04..9991f9b9f1 100644 --- a/drivers/gpu

Re: [dpdk-dev] [PATCH] config/x86: add support for AMD platform

2021-11-18 Thread Thomas Monjalon
I request a techboard decision for this patch. 02/11/2021 20:04, Thomas Monjalon: > 02/11/2021 19:45, David Marchand: > > On Tue, Nov 2, 2021 at 3:53 PM Aman Kumar wrote: > > > > > > -Dcpu_instruction_set=znverX meson option can be used > > > to build dpdk for AMD platforms. Supported options ar

[PATCH v2] gpudev: free and unregister return gracefully if input pointer is NULL

2021-11-18 Thread eagostini
From: Elena Agostini Signed-off-by: Elena Agostini --- drivers/gpu/cuda/cuda.c | 4 ++-- lib/gpudev/gpudev.c | 6 ++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/cuda/cuda.c b/drivers/gpu/cuda/cuda.c index 24ae630d04..f68e2b20b9 100644 --- a/drivers/gpu/cud

[PATCH v1] gpudev: return EINVAL if invalid input pointer for free and unregister

2021-11-18 Thread eagostini
From: Elena Agostini Signed-off-by: Elena Agostini --- lib/gpudev/gpudev.c | 10 ++ lib/gpudev/rte_gpudev.h | 2 ++ 2 files changed, 12 insertions(+) diff --git a/lib/gpudev/gpudev.c b/lib/gpudev/gpudev.c index 2b174d8bd5..97575ed979 100644 --- a/lib/gpudev/gpudev.c +++ b/lib/gpud

[PATCH v1] config/arm: add armv7 native config

2021-11-18 Thread Juraj Linkeš
Arvm7 native build fails with this error: ../config/meson.build:364:1: ERROR: Problem encountered: Number of CPU cores not specified. This is because RTE_MAX_LCORE is not set. We also need to set RTE_MAX_NUMA_NODES in armv7 native builds. Fixes: 8ef09fdc506b ("build: add optional NUMA and CPU cou

[PATCH v2 1/1] app/test-gpudev: introduce ethdev to rx/tx packets using GPU memory

2021-11-18 Thread eagostini
From: Elena Agostini This patch introduces ethdev in test-gpudev app to provide: - an example to show how GPU memory can be used to send and receive packets - an useful tool to measure network metrics when using GPU memory with io forwarding With this feature test-gpudev can: - RX packets in CPU

[PATCH v2 0/1] app/test-gpudev: introduce ethdev to rx/tx packets using GPU memory

2021-11-18 Thread eagostini
From: Elena Agostini This patch introduces ethdev in test-gpudev app to provide: - an example to show how GPU memory can be used to send and receive packets - an useful tool to measure network metrics when using GPU memory with io forwarding With this feature test-gpudev can: - RX packets in CPU

RE: Probing the expected state/support of DPDK@armhf

2021-11-18 Thread Juraj Linkeš
> > > > > What I see when building DPDK 21.11 is > > > > > 2973 ../config/meson.build:364:1: ERROR: Problem encountered: > > > > > Number of CPU cores not specified. > > > > > > > > > > Right now this seems to be broken the same everywhere - Suse > > > > > [1], fedora [2], Debian/Ubuntu [3] > > > >

Re: [PATCH v1 1/1] app/test-gpudev: introduce ethdev to rx/tx packets using GPU memory

2021-11-18 Thread Elena Agostini
> From: Jerin Jacob > Date: Thursday, 18 November 2021 at 07:17 > To: Elena Agostini > Cc: dpdk-dev > Subject: Re: [PATCH v1 1/1] app/test-gpudev: introduce ethdev to rx/tx > packets using GPU memory > External email: Use caution opening links or attachments> > > On Thu, Nov 18, 2021 at 12:28

[PATCH v1] doc: arm64 cross docs improvements/fixes

2021-11-18 Thread Juraj Linkeš
Numactl cross compilation doesn't work with clang, remove it and fix the gcc cross compiler executable name. Remove CFLAGS and LDFLAGS since Meson doesn't support them well enough. Add alternatives. The names of the downloaded gcc binaries differ from those in cross files, so point this out in do

[PATCH] net/i40e: add outer VLAN processing

2021-11-18 Thread Robin Zhang
Outer VLAN processing is supported after firmware v8.4, kernel driver also change the default behavior to support this feature. To align with kernel driver, add support for outer VLAN processing in DPDK. This will not impact on an old firmware. Signed-off-by: Robin Zhang --- drivers/net/i40e/i40