Re: [dpdk-dev] [EXTERNAL] [PATCH v2 7/7] examples/cmdline: build on Windows

2020-09-25 Thread Narcisa Ana Maria Vasile
Adding Khoa -Original Message- From: Dmitry Kozlyuk Sent: Thursday, July 30, 2020 2:07 PM To: dev@dpdk.org Cc: Dmitry Malloy (MESHCHANINOV) ; Narcisa Ana Maria Vasile ; Fady Bader ; Tal Shnaiderman ; Kadam, Pallavi ; Dmitry Kozlyuk ; Olivier Matz Subject: [EXTERNAL] [PATCH v2 7/7] ex

Re: [dpdk-dev] [EXTERNAL] [PATCH] eal/windows: use bundled getopt with MinGW

2020-09-25 Thread Khoa To
> -Original Message- > From: Dmitry Kozlyuk > Sent: Thursday, September 24, 2020 4:17 PM > To: dev@dpdk.org > Cc: Dmitry Kozlyuk ; sta...@dpdk.org; Khoa To > ; Tal Shnaiderman ; Narcisa Ana > Maria Vasile ; Dmitry Malloy > (MESHCHANINOV) ; Pallavi Kadam > > Subject: [EXTERNAL] [PATCH] e

Re: [dpdk-dev] [PATCH] net/bnxt: fix vector mode for 32-bit x86

2020-09-25 Thread Ajit Khaparde
On Fri, Sep 25, 2020 at 2:40 PM Lance Richardson wrote: > > Descriptor valid mask should be 64-bit all ones, use > appropriate initializer type (unsigned long long) to > obtain correct value in 32-bit mode. > > Fixes: deae85145c64 ("net/bnxt: handle multiple packets per loop in vector > Rx") > Si

[dpdk-dev] [PATCH] net/bnxt: fix vector mode for 32-bit x86

2020-09-25 Thread Lance Richardson
Descriptor valid mask should be 64-bit all ones, use appropriate initializer type (unsigned long long) to obtain correct value in 32-bit mode. Fixes: deae85145c64 ("net/bnxt: handle multiple packets per loop in vector Rx") Signed-off-by: Lance Richardson Reviewed-by: Ajit Kumar Khaparde --- dri

Re: [dpdk-dev] [PATCH] net/vdev_netvsc: handle removal of associated pci device

2020-09-25 Thread Long Li
HI Matan, While troubleshooting a failure in DPDK on device removal when VF device briefly disappears and comes back, I notice the failsafe driver is trying repeatedly to start a sub device (after this sub device has been successfully configured, but later hot removed from the kernel). This is

Re: [dpdk-dev] [PATCH v2 1/2] eal/windows: add interrupt thread skeleton

2020-09-25 Thread Narcisa Ana Maria Vasile
On Sat, Sep 26, 2020 at 02:32:42AM +0300, Dmitry Kozlyuk wrote: > Windows interrupt support is based on IO completion ports (IOCP). > Interrupt thread would send the devices requests to notify about > interrupts and then wait for any request completion. Add skeleton code > of this model without any

Re: [dpdk-dev] [PATCH v2 2/2] eal/windows: implement alarm API

2020-09-25 Thread Narcisa Ana Maria Vasile
On Sat, Sep 26, 2020 at 02:32:43AM +0300, Dmitry Kozlyuk wrote: > Implementation is based on waitable timers Win32 API. When timer is set, > a callback and its argument are supplied to the OS, while timer handle > is stored in EAL alarm list. When timer expires, OS wakes up the > interrupt thread a

[dpdk-dev] [PATCH v2 2/2] eal/windows: implement alarm API

2020-09-25 Thread Dmitry Kozlyuk
Implementation is based on waitable timers Win32 API. When timer is set, a callback and its argument are supplied to the OS, while timer handle is stored in EAL alarm list. When timer expires, OS wakes up the interrupt thread and runs the callback. Upon completion it removes the alarm. Waitable ti

[dpdk-dev] [PATCH v2 1/2] eal/windows: add interrupt thread skeleton

2020-09-25 Thread Dmitry Kozlyuk
Windows interrupt support is based on IO completion ports (IOCP). Interrupt thread would send the devices requests to notify about interrupts and then wait for any request completion. Add skeleton code of this model without any hardware support. Another way to wake up the interrupt thread is APC (

[dpdk-dev] [PATCH v2 0/2] eal/windows: implement alarms

2020-09-25 Thread Dmitry Kozlyuk
This patchset provides EAL alarm support for Windows. Basic interrupt thread code is added to monitor alarm events. It doesn't include callback management, because Windows alarms, unlike Unix EALs, rely on the OS for callback execution scheduling. v2: rebase on ToT to resolve conflicts. Dmitry Ko

Re: [dpdk-dev] [PATCH v3 0/8] fix distributor synchronization issues

2020-09-25 Thread Lukasz Wojciechowski
Hello David, Thank you for your review. W dniu 25.09.2020 o 14:31, David Marchand pisze: > Hello Lukasz, > > On Wed, Sep 23, 2020 at 3:25 PM Lukasz Wojciechowski > wrote: >> During review and verification of the patch created by Sarosh Arif: >> "test_distributor: prevent memory leakages from the

[dpdk-dev] [PATCH v4 7/8] distributor: fix handshake deadlock

2020-09-25 Thread Lukasz Wojciechowski
Synchronization of data exchange between distributor and worker cores is based on 2 handshakes: retptr64 for returning mbufs from workers to distributor and bufptr64 for passing mbufs to workers. Without proper order of verifying those 2 handshakes a deadlock may occur. This can happen when worker

[dpdk-dev] [PATCH v4 6/8] distributor: fix missing handshake synchronization

2020-09-25 Thread Lukasz Wojciechowski
rte_distributor_return_pkt function which is run on worker cores must wait for distributor core to clear handshake on retptr64 before using those buffers. While the handshake is set distributor core controls buffers and any operations on worker side might overwrite buffers which are unread yet. Sam

[dpdk-dev] [PATCH v4 8/8] distributor: align API documentation with code

2020-09-25 Thread Lukasz Wojciechowski
After introducing burst API there were some artefacts in the API documentation from legacy single API. Also the rte_distributor_poll_pkt() function return values mismatched the implementation. Fixes: c0de0eb82e40 ("distributor: switch over to new API") Cc: david.h...@intel.com Cc: sta...@dpdk.org

[dpdk-dev] [PATCH v4 4/8] test/distributor: fix freeing mbufs

2020-09-25 Thread Lukasz Wojciechowski
Sanity tests with mbuf alloc and shutdown tests assume that mbufs passed to worker cores are freed in handlers. Such packets should not be returned to the distributor's main core. The only packets that should be returned are the packets send after completion of the tests in quit_workers function.

[dpdk-dev] [PATCH v4 3/8] distributor: do not use oldpkt when not needed

2020-09-25 Thread Lukasz Wojciechowski
rte_distributor_request_pkt and rte_distributor_get_pkt dereferenced oldpkt parameter when in RTE_DIST_ALG_SINGLE even if number of returned buffers from worker to distributor was 0. This patch passes NULL to the legacy API when number of returned buffers is 0. This allows passing NULL as oldpkt p

[dpdk-dev] [PATCH v4 5/8] test/distributor: collect return mbufs

2020-09-25 Thread Lukasz Wojciechowski
During quit_workers function distributor's main core processes some packets to wake up pending worker cores so they can quit. As quit_workers acts also as a cleanup procedure for next test case it should also collect these packages returned by workers' handlers, so the cyclic buffer with returned p

[dpdk-dev] [PATCH v4 2/8] test/distributor: synchronize lcores statistics

2020-09-25 Thread Lukasz Wojciechowski
Statistics of handled packets are cleared and read on main lcore, while they are increased in workers handlers on different lcores. Without synchronization occasionally showed invalid values. This patch uses atomic acquire/release mechanisms to synchronize. Fixes: c3eabff124e6 ("distributor: add

[dpdk-dev] [PATCH v4 0/8] fix distributor synchronization issues

2020-09-25 Thread Lukasz Wojciechowski
During review and verification of the patch created by Sarosh Arif: "test_distributor: prevent memory leakages from the pool" I found out that running distributor unit tests multiple times in a row causes fails. So I investigated all the issues I found. There are few synchronization issues that mi

[dpdk-dev] [PATCH v4 1/8] test/distributor: fix deadlock with freezed worker

2020-09-25 Thread Lukasz Wojciechowski
The sanity test with worker shutdown delegates all bufs to be processed by a single lcore worker, then it freezes one of the lcore workers and continues to send more bufs. Problem occurred if freezed lcore is the same as the one that is processing the mbufs. The lcore processing mbufs might be dif

Re: [dpdk-dev] [PATCH V12 1/4] ethdev: introduce FEC API

2020-09-25 Thread Ajit Khaparde
On Fri, Sep 25, 2020 at 1:47 AM Min Hu (Connor) wrote: > > HI,Andrew, > I fix it in V13 according to your advice. > Thanks for your patient review. Please check it out again. > > By the way, there is always a coding check warning, like this: > > WARNING:PREFER_FALLTHROUGH: Prefer

Re: [dpdk-dev] [EXT] Re: [PATCH] net/bnx2x: enhance old ethdev close behavior

2020-09-25 Thread Rasesh Mody
Hi Thomas, >From: Thomas Monjalon >Sent: Friday, September 25, 2020 12:11 AM > >25/09/2020 06:17, Rasesh Mody: >> Set RTE_ETH_DEV_CLOSE_REMOVE upon probe so all the private resources >> for the port can be freed by rte_eth_dev_close(). With this change the >> private port resources are released i

Re: [dpdk-dev] [PATCH] build: support i686 target on x86 64-bit hosts

2020-09-25 Thread Lance Richardson
On Fri, Sep 25, 2020 at 9:44 AM Bruce Richardson wrote: > > On Fri, Sep 25, 2020 at 09:27:25AM -0400, Lance Richardson wrote: > > On Fri, Sep 25, 2020 at 5:13 AM Bruce Richardson > > wrote: > > > > > > On Thu, Sep 24, 2020 at 12:37:42PM -0400, Lance Richardson wrote: > > > > Add meson cross files

Re: [dpdk-dev] [PATCH v2 5/5] lib/stack: remove pop cas release ordering

2020-09-25 Thread Eads, Gage
> -Original Message- > From: Steven Lariau > Sent: Friday, September 25, 2020 12:44 PM > To: Eads, Gage ; Olivier Matz > Cc: dev@dpdk.org; n...@arm.com; Steven Lariau > Subject: [PATCH v2 5/5] lib/stack: remove pop cas release ordering > > Replace the store-release by relaxed for the

[dpdk-dev] [PATCH v2 4/5] lib/stack: reload head when pop fails

2020-09-25 Thread Steven Lariau
List head must be loaded right before continue (when failed to find the new head). Without this, one thread might keep trying and failing to pop items without ever loading the new correct head. Fixes: 7e6e609939a8 ("stack: add C11 atomic implementation") Cc: gage.e...@intel.com Cc: sta...@dpdk.org

[dpdk-dev] [PATCH v2 5/5] lib/stack: remove pop cas release ordering

2020-09-25 Thread Steven Lariau
Replace the store-release by relaxed for the CAS success at the end of pop. Release isn't needed, because there is not write to data that need to be synchronized. The only preceding write is when the length is decreased, but the length CAS loop already ensures the right synchronization. The situati

[dpdk-dev] [PATCH v2 1/5] lib/stack: fix inconsistent weak / strong cas

2020-09-25 Thread Steven Lariau
Fix cmpexchange usage of weak / strong. The generated code is the same on x86 and ARM (there is no weak cmpexchange), but the old usage was inconsistent. For push and pop update size, weak is used because cmpexchange is inside a loop. For pop update root, strong is used even though cmpexchange is i

[dpdk-dev] [PATCH v2 2/5] lib/stack: remove push acquire fence

2020-09-25 Thread Steven Lariau
An acquire fence is used to make sure loads after the fence can observe all store operations before a specific store-release. But push doesn't read any data, except for the head which is part of a CAS operation (the items on the list are not read). So there is no need for the acquire barrier. Sign

[dpdk-dev] [PATCH v2 3/5] lib/stack: remove redundant orderings for list->len

2020-09-25 Thread Steven Lariau
The load-acquire of list->len on pop function is redundant. Only the CAS success needs to be load-acquire. It synchronizes with the store release in push, to ensure that the updated head is visible when the new length is visible. Without this, one thread in pop could see the increased length but th

[dpdk-dev] [PATCH v2 0/5] lib/stack: improve lockfree C11 implementation

2020-09-25 Thread Steven Lariau
One implementation of the DPDK stack library is lockfree, based on C11 memory model for atomics. Some of these atomic operations use unnecessary memory orders, that can be relaxed. This patch relax some of these operations in order to improve the performance of the stack library. The patch was tes

Re: [dpdk-dev] [PATCH] build: support i686 target on x86 64-bit hosts

2020-09-25 Thread Lance Richardson
On Fri, Sep 25, 2020 at 5:13 AM Bruce Richardson wrote: > > On Thu, Sep 24, 2020 at 12:37:42PM -0400, Lance Richardson wrote: > > Add meson cross files for building i686 targets using gcc on x86_64 > > linux hosts. > > > > Uusage example: > > > > meson --cross-file config/x86/cross-i686-linux-

Re: [dpdk-dev] [PATCH] checkpatch: suppress fallthrough warning

2020-09-25 Thread Stephen Hemminger
On Fri, 25 Sep 2020 18:41:40 +0200 David Marchand wrote: > On Fri, Sep 25, 2020 at 6:14 PM Stephen Hemminger > wrote: > > > > The DPDK does not have (or want to use) the same fallthrough > > wrapper as the Linux kernel. Therefore silence the warning. > > > > Signed-off-by: Stephen Hemminger N

Re: [dpdk-dev] [PATCH] eal/windows: fix incorrect free condition in getopt implementation

2020-09-25 Thread Tal Shnaiderman
> From: Dmitry Kozlyuk > On Wed, 23 Sep 2020 17:15:38 +0300, Tal Shnaiderman wrote: > > In the Windows getopt_internal function the condition freeing the > > memory allocated by _dupenv_s is correct only for the first call to > > the function. > > Hi Tal, > > a few days back Khoa To and me priva

Re: [dpdk-dev] [PATCH] checkpatch: suppress fallthrough warning

2020-09-25 Thread David Marchand
On Fri, Sep 25, 2020 at 6:14 PM Stephen Hemminger wrote: > > The DPDK does not have (or want to use) the same fallthrough > wrapper as the Linux kernel. Therefore silence the warning. > > Signed-off-by: Stephen Hemminger > --- > devtools/checkpatches.sh | 2 +- > 1 file changed, 1 insertion(+),

Re: [dpdk-dev] [PATCH V12 1/4] ethdev: introduce FEC API

2020-09-25 Thread Ferruh Yigit
On 9/25/2020 5:12 PM, Stephen Hemminger wrote: On Fri, 25 Sep 2020 08:36:06 -0700 Ajit Khaparde wrote: WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #264: FILE: drivers/net/hns3/hns3_ethdev.c:5601: + /* fallthrough */ Try /* FALLTHROUGH */ That won't work

[dpdk-dev] [PATCH] checkpatch: suppress fallthrough warning

2020-09-25 Thread Stephen Hemminger
The DPDK does not have (or want to use) the same fallthrough wrapper as the Linux kernel. Therefore silence the warning. Signed-off-by: Stephen Hemminger --- devtools/checkpatches.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/devtools/checkpatches.sh b/devtools/checkpat

Re: [dpdk-dev] [PATCH V12 1/4] ethdev: introduce FEC API

2020-09-25 Thread Stephen Hemminger
On Fri, 25 Sep 2020 08:36:06 -0700 Ajit Khaparde wrote: > > > > WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment > > #264: FILE: drivers/net/hns3/hns3_ethdev.c:5601: > > + /* fallthrough */ > Try /* FALLTHROUGH */ That won't work either. This is a new warning f

Re: [dpdk-dev] [v3 PATCH] test_distributor: prevent memory leakages from the pool

2020-09-25 Thread David Hunt
Hi David, On 25/9/2020 3:22 PM, David Marchand wrote: On Tue, Sep 8, 2020 at 12:22 PM Sarosh Arif wrote: rte_mempool_get_bulk is used to get bufs/many_bufs from the pool, but at some locations when test fails the bufs/many_bufs are not returned back to the pool. Due to this, multiple execution

Re: [dpdk-dev] [v3 PATCH] test_distributor: prevent memory leakages from the pool

2020-09-25 Thread David Hunt
On 8/9/2020 11:22 AM, Sarosh Arif wrote: rte_mempool_get_bulk is used to get bufs/many_bufs from the pool, but at some locations when test fails the bufs/many_bufs are not returned back to the pool. Due to this, multiple executions of distributor_autotest gives the following error message: Erro

Re: [dpdk-dev] Question about “PCI memorymmap”

2020-09-25 Thread Burakov, Anatoly
On 02-Jul-20 3:49 AM, 陈毅强 wrote: Hi,ALL We fail to start dpdk in a newly deployed machine.Failed progress works in proc-type=secondary mode. (DPDK version 16.04).Some logs are shown below = Primary prog

Re: [dpdk-dev] [PATCH v7 10/10] buildtools: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Cc: Dmitry Kozlyuk > Cc: Narcisa Ana Maria Vasile > Cc: Dmitry Malloy > Cc: Pallavi Kadam > > Signed-off-by: Kevin Laatz Acked-by: Rob

Re: [dpdk-dev] [PATCH v7 09/10] app/test-bbdev: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Cc: Nicolas Chautru > > Signed-off-by: Louise Kilheeney Acked-by: Robin Jarry

Re: [dpdk-dev] [PATCH v7 08/10] config/arm: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2. > > Cc: Thomas Monjalon > > Signed-off-by: Louise Kilheeney Acked-by: Robin Jarry

Re: [dpdk-dev] [PATCH v7 06/10] app/test: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Signed-off-by: Louise Kilheeney > Signed-off-by: Kevin Laatz Acked-by: Robin Jarry

Re: [dpdk-dev] [PATCH v7 07/10] devtools: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Cc: Neil Horman > Cc: Ray Kinsella > > Signed-off-by: Louise Kilheeney > Acked-by: Ray Kinsella > Acked-by:

Re: [dpdk-dev] [PATCH v7 05/10] app/test-cmdline: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Cc: Olivier Matz > > Signed-off-by: Louise Kilheeney > Signed-off-by: Kevin Laatz > Reviewed-by: Bruce Richardson [snip] > @@ -43,7 +38,

Re: [dpdk-dev] [PATCH v7 04/10] usertools/cpu_layout: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Signed-off-by: Louise Kilheeney > Reviewed-by: Bruce Richardson Acked-by: Robin Jarry

Re: [dpdk-dev] [PATCH v7 03/10] usertools/dpdk-pmdinfo: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Cc: Neil Horman > > Signed-off-by: Louise Kilheeney > Reviewed-by: Bruce Richardson > Acked-by: Neil Horman

Re: [dpdk-dev] [PATCH 5/8] net/bnxt: add a null ptr check in bnxt PCI probe

2020-09-25 Thread Somnath Kotur
On Fri, Sep 25, 2020 at 4:16 PM Somnath Kotur wrote: > > On Fri, Sep 25, 2020 at 2:12 PM Ferruh Yigit wrote: > > > > On 9/25/2020 3:04 AM, Somnath Kotur wrote: > > > On Thu, Sep 24, 2020 at 8:17 PM Ferruh Yigit > > > wrote: > > >> > > >> On 9/22/2020 8:06 AM, Somnath Kotur wrote: > > >>> Check

Re: [dpdk-dev] [PATCH v7 02/10] usertools/dpdk-devbind: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Signed-off-by: Louise Kilheeney > Reviewed-by: Bruce Richardson Acked-by: Robin Jarry

Re: [dpdk-dev] [PATCH] build: support i686 target on x86 64-bit hosts

2020-09-25 Thread Ferruh Yigit
On 9/25/2020 3:28 PM, Bruce Richardson wrote: On Fri, Sep 25, 2020 at 10:08:35AM -0400, Lance Richardson wrote: On Fri, Sep 25, 2020 at 9:44 AM Bruce Richardson wrote: On Fri, Sep 25, 2020 at 09:27:25AM -0400, Lance Richardson wrote: On Fri, Sep 25, 2020 at 5:13 AM Bruce Richardson wrote:

Re: [dpdk-dev] [PATCH 1/2] net/virtio: set UNKNOWN as default speed

2020-09-25 Thread David Marchand
On Fri, Sep 25, 2020 at 4:36 PM Maxime Coquelin wrote: > On 9/22/20 11:18 PM, Ivan Dyukov wrote: > > rte_ethdev states new rule for NICs: they should return UNKNOWN > > speed if speed is unknown and interface is up, in case of down > > interface, NONE speed should be returned. > > > > Signed-off-b

[dpdk-dev] [PATCH v2] net/bnxt: add a check in bnxt PCI probe

2020-09-25 Thread Somnath Kotur
Check for num_rep before invoking rep port probe. Fixes: 6dc83230b43b ("net/bnxt: support port representor data path") Signed-off-by: Somnath Kotur Reviewed-by: Venkat Duvvuru --- v2: Check against num_rep instead of devargs ptr drivers/net/bnxt/bnxt_ethdev.c | 4 1 file changed, 4 insert

Re: [dpdk-dev] [PATCH 5/8] net/bnxt: add a null ptr check in bnxt PCI probe

2020-09-25 Thread Somnath Kotur
On Fri, Sep 25, 2020 at 2:12 PM Ferruh Yigit wrote: > > On 9/25/2020 3:04 AM, Somnath Kotur wrote: > > On Thu, Sep 24, 2020 at 8:17 PM Ferruh Yigit wrote: > >> > >> On 9/22/2020 8:06 AM, Somnath Kotur wrote: > >>> Check for devargs before invoking rep port probe. > >>> > >>> Fixes: 6dc83230b43b (

[dpdk-dev] [PATCH v2] net/bnxt: add a check in bnxt PCI probe

2020-09-25 Thread Somnath Kotur
Check for num_rep before invoking rep port probe. Fixes: 6dc83230b43b ("net/bnxt: support port representor data path") Signed-off-by: Somnath Kotur Reviewed-by: Venkat Duvvuru --- v2: Check against num_rep instead of devargs ptr drivers/net/bnxt/bnxt_ethdev.c | 4 1 file changed, 4 insert

Re: [dpdk-dev] [PATCH 1/2] net/virtio: set UNKNOWN as default speed

2020-09-25 Thread Maxime Coquelin
Hi Ivan, On 9/22/20 11:18 PM, Ivan Dyukov wrote: > rte_ethdev states new rule for NICs: they should return UNKNOWN > speed if speed is unknown and interface is up, in case of down > interface, NONE speed should be returned. > > Signed-off-by: Ivan Dyukov > --- > doc/guides/nics/virtio.rst

Re: [dpdk-dev] [PATCH 2/2] net/virtio: sync a virtio speed capa with ethdev

2020-09-25 Thread Maxime Coquelin
On 9/22/20 11:18 PM, Ivan Dyukov wrote: > ethdev library was updated with new speed 200G > > Add 200G speed capa to virtio device > > Signed-off-by: Ivan Dyukov > --- > drivers/net/virtio/virtio_ethdev.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/net/virtio/virtio_et

Re: [dpdk-dev] [PATCH v7 01/10] usertools/dpdk-telemetry-client: support python3 only

2020-09-25 Thread Robin Jarry
Hi Kevin, all, 2020-09-21, Kevin Laatz: > From: Louise Kilheeney > > Changed script to explicitly use python3 only to avoid > maintaining python 2 and removed deprecation notice. > > Cc: Kevin Laatz > > Signed-off-by: Louise Kilheeney > Acked-by: Bruce Richardson [snip] > -s

Re: [dpdk-dev] [PATCH v2 0/2] Use WFE for spinlock and ring

2020-09-25 Thread David Marchand
Hello Honnappa, On Sun, May 3, 2020 at 11:32 PM Honnappa Nagarahalli wrote: > > > > > Subject: Re: [PATCH v2 0/2] Use WFE for spinlock and ring > > > > On Sun, Apr 26, 2020 at 10:39 AM Gavin Hu wrote: > > > > > > The rte_wait_until_equal_xxx APIs abstract the functionality of > > > 'polling for

Re: [dpdk-dev] [PATCH] build: support i686 target on x86 64-bit hosts

2020-09-25 Thread Bruce Richardson
On Fri, Sep 25, 2020 at 10:08:35AM -0400, Lance Richardson wrote: > On Fri, Sep 25, 2020 at 9:44 AM Bruce Richardson > wrote: > > > > On Fri, Sep 25, 2020 at 09:27:25AM -0400, Lance Richardson wrote: > > > On Fri, Sep 25, 2020 at 5:13 AM Bruce Richardson > > > wrote: > > > > > > > > On Thu, Sep 2

Re: [dpdk-dev] [PATCH 5/5] lib/stack: remove pop cas release ordering

2020-09-25 Thread David Marchand
Hello Steven, On Mon, Sep 21, 2020 at 7:17 PM Eads, Gage wrote: > > -Original Message- > > From: Steven Lariau > > Sent: Friday, September 11, 2020 10:30 AM > > To: Eads, Gage ; Olivier Matz > > Cc: dev@dpdk.org; n...@arm.com; dharmik.thak...@arm.com; Steven Lariau > > > > Subject: [PA

Re: [dpdk-dev] [v3 PATCH] test_distributor: prevent memory leakages from the pool

2020-09-25 Thread David Marchand
On Tue, Sep 8, 2020 at 12:22 PM Sarosh Arif wrote: > > rte_mempool_get_bulk is used to get bufs/many_bufs from the pool, > but at some locations when test fails the bufs/many_bufs are > not returned back to the pool. > Due to this, multiple executions of distributor_autotest gives the > following

Re: [dpdk-dev] [PATCH 0/4] remove experimental tags from APIs

2020-09-25 Thread Ananyev, Konstantin
> On Wed, Sep 16, 2020 at 7:09 PM David Marchand > wrote: > > > > On Fri, Sep 11, 2020 at 7:46 AM Joyce Kong wrote: > > > > > > Experimental tags can be removed from APIs in module: ticket lock, > > > MCS lock, pause (wait until equal APIs), and RCU (except for defer > > > queue APIs) as they ha

Re: [dpdk-dev] [PATCH 0/4] remove experimental tags from APIs

2020-09-25 Thread David Marchand
On Wed, Sep 16, 2020 at 7:09 PM David Marchand wrote: > > On Fri, Sep 11, 2020 at 7:46 AM Joyce Kong wrote: > > > > Experimental tags can be removed from APIs in module: ticket lock, > > MCS lock, pause (wait until equal APIs), and RCU (except for defer > > queue APIs) as they have been aroud fou

Re: [dpdk-dev] [PATCH v3 0/4] use C11 atomic builtins for libs

2020-09-25 Thread David Marchand
On Thu, Sep 24, 2020 at 7:40 AM Phil Yang wrote: > > Since rte_atomicXX APIs are not allowed to be used[1][2], use C11 atomic > builtins instead in eal, bbdev, power, and ethdev libs. > > [1] > http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecation.rst#L87 > [2] http://code.dpdk

Re: [dpdk-dev] [PATCH] build: support i686 target on x86 64-bit hosts

2020-09-25 Thread Bruce Richardson
On Fri, Sep 25, 2020 at 09:27:25AM -0400, Lance Richardson wrote: > On Fri, Sep 25, 2020 at 5:13 AM Bruce Richardson > wrote: > > > > On Thu, Sep 24, 2020 at 12:37:42PM -0400, Lance Richardson wrote: > > > Add meson cross files for building i686 targets using gcc on x86_64 > > > linux hosts. > > >

Re: [dpdk-dev] [PATCH V4 0/2] change data type in TC queue

2020-09-25 Thread Ferruh Yigit
On 9/25/2020 1:51 PM, Min Hu (Connor) wrote: From: Huisong Li This series change data type in TC rxq and TC txq, fix compiling errors for per-queue statistics. Huisong Li (2): dpdk: resolve compiling errors for per-queue stats ethdev: change data type in TC rxq and TC txq app/proc-inf

Re: [dpdk-dev] [PATCH 10/20] net/null: release port upon close

2020-09-25 Thread Thomas Monjalon
25/09/2020 10:52, Ferruh Yigit: > On 9/24/2020 10:58 PM, Thomas Monjalon wrote: > > 23/09/2020 22:47, Thomas Monjalon: > >> 23/09/2020 18:44, Ferruh Yigit: > >>> On 9/13/2020 11:07 PM, Thomas Monjalon wrote: > The flag RTE_ETH_DEV_CLOSE_REMOVE is set so all port resources > can be freed b

[dpdk-dev] [PATCH V4 0/2] change data type in TC queue

2020-09-25 Thread Min Hu (Connor)
From: Huisong Li This series change data type in TC rxq and TC txq, fix compiling errors for per-queue statistics. Huisong Li (2): dpdk: resolve compiling errors for per-queue stats ethdev: change data type in TC rxq and TC txq app/proc-info/main.c | 2 +- app/test-pmd/cm

[dpdk-dev] [PATCH V4 2/2] ethdev: change data type in TC rxq and TC txq

2020-09-25 Thread Min Hu (Connor)
From: Huisong Li Currently, base and nb_queue in the tc_rxq and tc_txq information of queue and TC mapping on both TX and RX paths are uint8_t. However, these data will be truncated when queue number under a TC is greater than 256. So it is necessary for base and nb_queue to change from uint8_t t

[dpdk-dev] [PATCH V4 1/2] dpdk: resolve compiling errors for per-queue stats

2020-09-25 Thread Min Hu (Connor)
From: Huisong Li Currently, only statistics of rx/tx queues with queue_id less than RTE_ETHDEV_QUEUE_STAT_CNTRS can be displayed. If there is a certain application scenario that it needs to use 256 or more than 256 queues and display all statistics of rx/tx queue. At this moment, we have to chang

[dpdk-dev] [PATCH v4 6/6] app/testpmd: fix displaying Rx Tx queues information

2020-09-25 Thread Wei Hu (Xavier)
From: Huisong Li Currently, the information of Rx/Tx queues from PMD driver is not displayed exactly in the rxtx_config_display function. Because "ports[pid].rx_conf" and "ports[pid].tx_conf" maintained in testpmd application may be not the value actually used by PMD driver. For instance, user do

[dpdk-dev] [PATCH v4 3/6] app/testpmd: remove restriction on txpkts set

2020-09-25 Thread Wei Hu (Xavier)
From: Chengchang Tang Currently, if nb_txd is not set, the txpkts is not allowed to be set because the nb_txd is used to avoid the numer of segments exceed the Tx ring size and the default value of nb_txd is 0. And there is a bug that nb_txd is the global configuration for Tx ring size and the ri

[dpdk-dev] [PATCH v4 0/6] minor fixes for testpmd

2020-09-25 Thread Wei Hu (Xavier)
This series are minor fixes for testpmd application. Chengchang Tang (5): app/testpmd: fix missing verification of port id app/testpmd: fix VLAN offload configuration when config fail app/testpmd: remove restriction on txpkts set app/testpmd: fix packet header in txonly mode app/testpmd:

[dpdk-dev] [PATCH v4 1/6] app/testpmd: fix missing verification of port id

2020-09-25 Thread Wei Hu (Xavier)
From: Chengchang Tang To set Tx vlan offloads, it is required to stop port firstly. But before checking whether the port is stopped, the port id entered by the user is not checked for validity. When the port id is illegal, it would lead to a segmentation fault since it attempts to access a member

[dpdk-dev] [PATCH v4 4/6] app/testpmd: fix packet header in txonly mode

2020-09-25 Thread Wei Hu (Xavier)
From: Chengchang Tang In txonly forward mode, the packet header is fixed by the initial setting, including the packet length and checksum. So when the packets varies, this may cause a packet header error. Currently, there are two methods in txonly mode to randomly change the packets. 1. Set txspl

[dpdk-dev] [PATCH v4 2/6] app/testpmd: fix VLAN offload configuration when config fail

2020-09-25 Thread Wei Hu (Xavier)
From: Chengchang Tang When failing to configure VLAN offloads after the port was started, there is no need to update the port configuration. Currently, when user configure an unsupported VLAN offloads and fails, and then restart the port, it will fails since the configuration has been refreshed.

[dpdk-dev] [PATCH v4 5/6] app/testpmd: fix valid desc id check

2020-09-25 Thread Wei Hu (Xavier)
From: Chengchang Tang The number of desc is a per queue configuration. But in the check function, nb_txd & nb_rxd are used to check whether the desc_id is valid. nb_txd & nb_rxd are the global configuration of number of desc. If the queue configuration is changed by cmdline liks: "port config xx

Re: [dpdk-dev] [PATCH] net/ring: fix unchecked return value

2020-09-25 Thread Ferruh Yigit
On 9/22/2020 6:20 PM, Kevin Laatz wrote: Add a check for the return value of the sscanf call in parse_internal_args(), returning an error if we don't get the expected result. Coverity issue: 362049 Fixes: 96cb19521147 ("net/ring: use EAL APIs in PMD specific API") Cc: sta...@dpdk.org Signed-off

[dpdk-dev] [PATCH v2 7/8] net/virtio: split virtio-user start

2020-09-25 Thread Maxime Coquelin
Move feature bit settings in device start out as an standalone function, so that feature bit could be negotiated at device feature_ok status. Signed-off-by: Maxime Coquelin --- .../net/virtio/virtio_user/virtio_user_dev.c | 50 --- .../net/virtio/virtio_user/virtio_user_dev.h |

[dpdk-dev] [PATCH v2 6/8] net/virtio: adapt Virtio-user status size

2020-09-25 Thread Maxime Coquelin
Set proper payload size for set/get status message. The payload size varies according to backend types. Signed-off-by: Maxime Coquelin Signed-off-by: Patrick Fu --- .../net/virtio/virtio_user/virtio_user_dev.c | 34 +-- 1 file changed, 23 insertions(+), 11 deletions(-) diff --

[dpdk-dev] [PATCH v2 8/8] net/virtio: introduce Vhost-vDPA backend

2020-09-25 Thread Maxime Coquelin
vhost-vDPA is a new virtio backend type introduced by vDPA kernel framework, which provides abstruction to the vDPA devices and exposes an unified control interface through a char dev. This patch adds support to the vhost-vDPA backend. As similar to the existing vhost kernel backend, a set of virt

Re: [dpdk-dev] [PATCH v3 0/8] fix distributor synchronization issues

2020-09-25 Thread David Marchand
Hello Lukasz, On Wed, Sep 23, 2020 at 3:25 PM Lukasz Wojciechowski wrote: > > During review and verification of the patch created by Sarosh Arif: > "test_distributor: prevent memory leakages from the pool" I found out > that running distributor unit tests multiple times in a row causes fails. > S

[dpdk-dev] [PATCH v2 5/8] net/virtio: check protocol feature in user backend

2020-09-25 Thread Maxime Coquelin
When sending set status message, move protocol feature check to vhost_user to be compatible with different backend types. Signed-off-by: Maxime Coquelin --- drivers/net/virtio/virtio_user/vhost_user.c | 6 +- drivers/net/virtio/virtio_user/virtio_user_dev.c | 6 -- 2 files changed,

[dpdk-dev] [PATCH v2 4/8] net/virtio: introduce Vhost-vDPA backend type

2020-09-25 Thread Maxime Coquelin
Backend type is determined by checking char-device major numbers Signed-off-by: Maxime Coquelin Signed-off-by: Adrian Moreno --- .../net/virtio/virtio_user/virtio_user_dev.h | 1 + drivers/net/virtio/virtio_user_ethdev.c | 48 +-- 2 files changed, 45 insertions(+), 4 del

[dpdk-dev] [PATCH v2 0/8] virtio-user: introduce vhost-vdpa backend

2020-09-25 Thread Maxime Coquelin
vhost-vDPA is a new vhost backend type introduced by vDPA kernel framework, which provides abstruction to the vDPA devices and exposes to userspace a unified control interface through char devs. This patch set adds vhost-vdpa backend type to the virtio_user. A set of vhost-vdpa specific ops callba

[dpdk-dev] [PATCH v2 3/8] net/virtio: move backend type selection to ethdev

2020-09-25 Thread Maxime Coquelin
From: Adrian Moreno This is a preparation patch with no functional change. Use an enum instead of a boolean for the backend type. Move the detection logic to the ethdev layer (where it is needed for the first time). The virtio_user_dev stores the backend type in the virtio_user_dev struct so the

[dpdk-dev] [PATCH v2 2/8] net/virtio: introduce DMA ops

2020-09-25 Thread Maxime Coquelin
Add DMA map/unmap callbacks to the virtio_user pmd, which could be leveraged by vdev bus driver to map memory for backend devices with DMA capability. Signed-off-by: Maxime Coquelin --- drivers/net/virtio/virtio_user/vhost.h | 4 ++ drivers/net/virtio/virtio_user_ethdev.c | 54

[dpdk-dev] [PATCH v2 1/8] bus/vdev: add DMA mapping ops

2020-09-25 Thread Maxime Coquelin
Add DMA map/unmap operation callbacks to the vdev bus, which could be used by DMA capable vdev drivers. Signed-off-by: Maxime Coquelin --- drivers/bus/vdev/rte_bus_vdev.h | 46 +++-- drivers/bus/vdev/vdev.c | 52 + 2 files changed,

[dpdk-dev] IPSEC-SECGW application

2020-09-25 Thread Rohit kamble
HI Guys , I am trying to use a DPDK based *ipsec-secgw application*. Can anyone please provide me with all the configurations, setup/topology and about the NIC drivers. I already went through the documentation of dpdk, but I am seeing several issues regarding ipsec-secgw applications. Thanks, Ro

[dpdk-dev] [PATCH v6 6/6] net/qede: add VF FLR support

2020-09-25 Thread Manish Chopra
This patch adds required bit to handle VF FLR indication from Management FW (MFW) of the device With that VFs were able to load in VM (VF attached as PCI passthrough to the guest VM) followed by FLR successfully Updated the docs/guides with the feature support Signed-off-by: Manish Chopra Signe

[dpdk-dev] [PATCH v6 4/6] net/qede: add infrastructure support for VF load

2020-09-25 Thread Manish Chopra
This patch adds necessary infrastructure support (required to handle messages from VF and sending ramrod on behalf of VF's configuration request from alarm handler context) to start/load the VF-PMD driver instance on top of PF-PMD driver instance. Signed-off-by: Manish Chopra Signed-off-by: Igor

[dpdk-dev] [PATCH v6 5/6] net/qede: initialize VF MAC and link

2020-09-25 Thread Manish Chopra
This patch configures VFs with random mac if no MAC is provided by the PF/bulletin. This also adds required bulletin APIs by PF-PMD driver to communicate LINK properties/changes to the VFs through bulletin update mechanism. With these changes, VF-PMD instance is able to run fastpath over PF-PMD dr

[dpdk-dev] [PATCH v6 2/6] net/qede: define PCI config space specific osals

2020-09-25 Thread Manish Chopra
This patch defines various PCI config space access APIs in order to read and find IOV specific PCI capabilities. With these definitions implemented, it enables the base driver to do SR-IOV specific initialization and HW specific configuration required from PF-PMD driver instance. Signed-off-by: M

Re: [dpdk-dev] [PATCH v2] net/bnxt: add a check in bnxt PCI probe

2020-09-25 Thread Ferruh Yigit
On 9/25/2020 11:40 AM, Somnath Kotur wrote: Check for num_rep before invoking rep port probe. Fixes: 6dc83230b43b ("net/bnxt: support port representor data path") Signed-off-by: Somnath Kotur Reviewed-by: Venkat Duvvuru Reviewed-by: Ferruh Yigit Applied to dpdk-next-net/main, thanks.

[dpdk-dev] [PATCH v6 1/6] drivers: add generic API to find PCI extended cap

2020-09-25 Thread Manish Chopra
By adding generic API, this patch removes individual functions/defines implemented by drivers to find extended PCI capabilities. Signed-off-by: Manish Chopra Signed-off-by: Igor Russkikh Reviewed-by: Gaetan Rivet --- drivers/bus/pci/pci_common.c | 43 ++ drivers/b

[dpdk-dev] [PATCH v6 3/6] net/qede: configure VFs on hardware

2020-09-25 Thread Manish Chopra
Based on number of VFs enabled at PCI, PF-PMD driver instance enables/configures those VFs from hardware perspective, such that in later patches they could get required HW access to communicate with PFs for slowpath configuration and run the fastpath themsleves. This patch also add two new qede IO

[dpdk-dev] [PATCH v6 0/6] qede: SR-IOV PF driver support

2020-09-25 Thread Manish Chopra
Hi, This series adds SR-IOV PF pmd driver support to have VF pmd driver work over PF pmd driver instances in order to run the adapter completely under DPDK environment for one of the use cases like ovs-dpdk. This is very initial bring-up with following testing covered - * Enable/Disable SR-IOV V

Re: [dpdk-dev] [PATCH] bus/pci: fix mapping BAR containing MSI-X table

2020-09-25 Thread Burakov, Anatoly
On 25-Sep-20 3:14 AM, Hyong Youb Kim wrote: When the BAR contains MSI-X table, pci_vfio_mmap_bar() tries to skip the table and map the rest. "map around it" is the phrase used in the source. The function splits the BAR into two regions: the region before the table (first part or memreg[0]) and th

[dpdk-dev] [PATCH] examples/vhost_blk: fix unchecked return value

2020-09-25 Thread wangyunjian
From: Yunjian Wang This checks the return value from the function rte_vhost_driver_start. Coverity issue: 362027 Fixes: c19beb3f38cd ("examples/vhost_blk: introduce vhost storage sample") Cc: sta...@dpdk.org Signed-off-by: Yunjian Wang --- examples/vhost_blk/vhost_blk.c | 6 +- 1 file cha

  1   2   >