Re: [dpdk-dev] [PATCH v8] app/testpmd: support multi-process

2021-03-29 Thread Min Hu (Connor)
在 2021/3/30 11:11, Ajit Khaparde 写道: On Mon, Mar 29, 2021 at 6:48 PM Min Hu (Connor) wrote: From: Lijun Ou This patch adds multi-process support for testpmd. The test cmd example as follows: the primary cmd: ./dpdk-testpmd -a xxx --proc-type=auto -l 0-1 -- -i \ --rxq=4 --txq=4 --num-procs

[dpdk-dev] [PATCH v5] build: alias default build as generic

2021-03-29 Thread Juraj Linkeš
The current machine='default' build name is not descriptive. The actual default build is machine='native'. Add an alternative string which does the same build and better describes what we're building: machine='generic'. Leave machine='default' for backwards compatibility. Signed-off-by: Juraj Link

Re: [dpdk-dev] [PATCH v8] app/testpmd: support multi-process

2021-03-29 Thread Min Hu (Connor)
在 2021/3/30 10:17, Li, Xiaoyun 写道: Hi -Original Message- From: Min Hu (Connor) Sent: Tuesday, March 30, 2021 09:48 To: dev@dpdk.org Cc: Yigit, Ferruh ; ajit.khapa...@broadcom.com; Li, Xiaoyun Subject: [PATCH v8] app/testpmd: support multi-process From: Lijun Ou This patch adds m

[dpdk-dev] [PATCH v3 3/3] net/i40e: fix segment fault in AVX512

2021-03-29 Thread Wenzhuo Lu
Fix segment fault when failing to get the memory from the pool. Fixes: e6a6a138919f ("net/i40e: add AVX512 vector path") Cc: sta...@dpdk.org Reported-by: David Coyle Signed-off-by: Wenzhuo Lu --- drivers/net/i40e/i40e_rxtx_vec_avx2.c | 117 +-- drivers/net/i40e/i40e_rxtx_vec_

[dpdk-dev] [PATCH v3 2/3] net/ice: fix segment fault in AVX512

2021-03-29 Thread Wenzhuo Lu
Fix segment fault when failing to get the memory from the pool. Fixes: 7f85d5ebcfe1 ("net/ice: add AVX512 vector path") Cc: sta...@dpdk.org Reported-by: David Coyle Signed-off-by: Wenzhuo Lu --- drivers/net/ice/ice_rxtx_vec_avx2.c | 120 +--- drivers/net/ice/ice_rxtx_vec_avx5

[dpdk-dev] [PATCH v3 1/3] net/iavf: fix segment fault in AVX512

2021-03-29 Thread Wenzhuo Lu
Fix segment fault when failing to get the memory from the pool. Fixes: 31737f2b66fb ("net/iavf: enable AVX512 for legacy Rx") Cc: sta...@dpdk.org Reported-by: David Coyle Signed-off-by: Wenzhuo Lu --- drivers/net/iavf/iavf_rxtx_vec_avx2.c | 120 +-- drivers/net/iavf/iavf_rxtx

[dpdk-dev] [PATCH v3 0/3] fix segment fault in avx512 code

2021-03-29 Thread Wenzhuo Lu
Fix no memory segment fault of iavf, ice, i40e. v2: - Drop the duplicate code. v3: - Fix compile error on no-x86 platform. Wenzhuo Lu (3): net/iavf: fix segment fault in AVX512 net/ice: fix segment fault in AVX512 net/i40e: fix segment fault in AVX512 drivers/net/i40e/i40e_rxtx_vec_avx

Re: [dpdk-dev] [PATCH] net/ice: refine RSS configure

2021-03-29 Thread Zhang, Qi Z
> -Original Message- > From: Ding, Xuan > Sent: Tuesday, March 30, 2021 12:31 PM > To: Zhang, Qi Z ; Guo, Jia > Cc: dev@dpdk.org; Yang, Qiming ; Noonan, Gordon > ; Varshney, Shashank1 > > Subject: RE: [PATCH] net/ice: refine RSS configure > > > > -Original Message- > > From:

Re: [dpdk-dev] [PATCH v4] pflock: add phase-fair reader writer locks

2021-03-29 Thread Stephen Hemminger
On Mon, 29 Mar 2021 22:00:47 -0700 Stephen Hemminger wrote: > This is a new type of reader-writer lock that provides better fairness > guarantees which better suited for typical DPDK applications. > A pflock has two ticket pools, one for readers and one > for writers. > > Phase fair reader write

[dpdk-dev] [Bug 666] [dpdk-21.05] unit_test_eal/test_kvargs: dpdk-20.11.0 use dpdk-21.05 lib to run kvargs_autotest failed.

2021-03-29 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=666 lingwei (weix.l...@intel.com) changed: What|Removed |Added Resolution|--- |FIXED Status|IN_PROGRE

[dpdk-dev] [PATCH v4] pflock: add phase-fair reader writer locks

2021-03-29 Thread Stephen Hemminger
This is a new type of reader-writer lock that provides better fairness guarantees which better suited for typical DPDK applications. A pflock has two ticket pools, one for readers and one for writers. Phase fair reader writer locks ensure that neither reader nor writer will be starved. Neither rea

Re: [dpdk-dev] [PATCH v1 0/2] Enhancements to crypto adapter forward mode

2021-03-29 Thread Gujjar, Abhinandan S
> -Original Message- > From: Jerin Jacob > Sent: Tuesday, March 30, 2021 9:34 AM > To: Shijith Thotton > Cc: dpdk-dev ; Thomas Monjalon > ; Jerin Jacob ; Gujjar, > Abhinandan S ; Hemant Agrawal > ; Nipun Gupta ; > sachin.sax...@oss.nxp.com; Anoob Joseph ; Matan > Azrad ; Zhang, Roy Fan

Re: [dpdk-dev] [PATCH v3] pflock: implementation of phase-fair reader writer locks

2021-03-29 Thread Stephen Hemminger
On Tue, 30 Mar 2021 00:18:40 + Honnappa Nagarahalli wrote: > > That won't work because the implementation of trylock requires > > compare/exchange of the whole structure as an atomic operation. > I meant, placing 'rd' and 'wr' on separate cache lines. It might help in the > reader-writer c

Re: [dpdk-dev] [PATCH] net/ice: refine RSS configure

2021-03-29 Thread Ding, Xuan
> -Original Message- > From: Zhang, Qi Z > Sent: Tuesday, March 30, 2021 11:30 AM > To: Ding, Xuan ; Guo, Jia > Cc: dev@dpdk.org; Yang, Qiming ; Noonan, Gordon > ; Varshney, Shashank1 > > Subject: RE: [PATCH] net/ice: refine RSS configure > > > > > -Original Message- > > Fro

Re: [dpdk-dev] [PATCH v1 0/2] Enhancements to crypto adapter forward mode

2021-03-29 Thread Jerin Jacob
On Fri, Mar 26, 2021 at 2:43 PM Shijith Thotton wrote: > > This series proposes a new event device enqueue operation if crypto > adapter forward mode is supported. Second patch in the series is the > implementation of the same in PMD. Test application changes for the > usage of new API is yet to a

Re: [dpdk-dev] [PATCH] net/ice: refine RSS configure

2021-03-29 Thread Zhang, Qi Z
> -Original Message- > From: Ding, Xuan > Sent: Tuesday, March 30, 2021 11:00 AM > To: Zhang, Qi Z ; Guo, Jia > Cc: dev@dpdk.org; Yang, Qiming ; Noonan, Gordon > ; Varshney, Shashank1 > > Subject: RE: [PATCH] net/ice: refine RSS configure > > Hi Qi, > > Replies inline. > > > -O

Re: [dpdk-dev] [PATCH v2 3/3] net/ice: support FDIR for IP fragment packet

2021-03-29 Thread Xu, Ting
Hi, Jeff > -Original Message- > From: Guo, Jia > Sent: Wednesday, March 24, 2021 9:54 PM > To: Yang, Qiming ; Zhang, Qi Z > > Cc: Xu, Ting ; dev@dpdk.org; Guo, Jia > Subject: [PATCH v2 3/3] net/ice: support FDIR for IP fragment packet > > New FDIR parsing are added to handle the fragme

[dpdk-dev] [PATCH v2 3/3] net/i40e: fix segment fault in AVX512

2021-03-29 Thread Wenzhuo Lu
Fix segment fault when failing to get the memory from the pool. Fixes: e6a6a138919f ("net/i40e: add AVX512 vector path") Cc: sta...@dpdk.org Reported-by: David Coyle Signed-off-by: Wenzhuo Lu --- drivers/net/i40e/i40e_rxtx_vec_avx2.c | 117 +-- drivers/net/i40e/i40e_rxtx_vec_

[dpdk-dev] [PATCH v2 2/3] net/ice: fix segment fault in AVX512

2021-03-29 Thread Wenzhuo Lu
Fix segment fault when failing to get the memory from the pool. Fixes: 7f85d5ebcfe1 ("net/ice: add AVX512 vector path") Cc: sta...@dpdk.org Reported-by: David Coyle Signed-off-by: Wenzhuo Lu --- drivers/net/ice/ice_rxtx_vec_avx2.c | 120 +--- drivers/net/ice/ice_rxtx_vec_avx5

[dpdk-dev] [PATCH v2 1/3] net/iavf: fix segment fault in AVX512

2021-03-29 Thread Wenzhuo Lu
Fix segment fault when failing to get the memory from the pool. Fixes: 31737f2b66fb ("net/iavf: enable AVX512 for legacy Rx") Cc: sta...@dpdk.org Reported-by: David Coyle Signed-off-by: Wenzhuo Lu --- drivers/net/iavf/iavf_rxtx_vec_avx2.c | 120 +-- drivers/net/iavf/iavf_rxtx

[dpdk-dev] [PATCH v2 0/3] fix segment fault in avx512

2021-03-29 Thread Wenzhuo Lu
Fix no memory segment fault of iavf, ice, i40e. v2: - Drop the duplicate code. Wenzhuo Lu (3): net/iavf: fix segment fault in AVX512 net/ice: fix segment fault in AVX512 net/i40e: fix segment fault in AVX512 drivers/net/i40e/i40e_rxtx_vec_avx2.c | 117 +-- drivers/net/i

Re: [dpdk-dev] [PATCH] net/ice: refine RSS configure

2021-03-29 Thread Ding, Xuan
Hi Qi, Replies inline. > -Original Message- > From: Zhang, Qi Z > Sent: Thursday, March 25, 2021 8:43 PM > To: Guo, Jia ; Ding, Xuan > Cc: dev@dpdk.org; Yang, Qiming ; Noonan, Gordon > ; Varshney, Shashank1 > ; Zhang, Qi Z > Subject: [PATCH] net/ice: refine RSS configure > > The ICE_R

Re: [dpdk-dev] [RFC 3/5] eal: lcore state FINISHED is not required

2021-03-29 Thread Honnappa Nagarahalli
> > Hi everyone, Thanks Konstantin for the review. > > > > > > > > > > > > > > > Subject: [RFC 3/5] eal: lcore state FINISHED is not required > > > > > > > > > > > > FINISHED state seems to be used to indicate that the worker's > > > > > > update of the 'state' is not visible to other threads

Re: [dpdk-dev] [PATCH v8] app/testpmd: support multi-process

2021-03-29 Thread Li, Xiaoyun
Hi > -Original Message- > From: Min Hu (Connor) > Sent: Tuesday, March 30, 2021 09:48 > To: dev@dpdk.org > Cc: Yigit, Ferruh ; ajit.khapa...@broadcom.com; Li, > Xiaoyun > Subject: [PATCH v8] app/testpmd: support multi-process > > From: Lijun Ou > > This patch adds multi-process suppor

[dpdk-dev] [PATCH] mbuf: Fix illegal pointer access to mempool members

2021-03-29 Thread wenwux . ma
From: wenwu ma Before accessing the private data of mempool in function rte_pktmbuf_priv_size() and rte_pktmbuf_data_room_size(), it is necessary to determine whether the private data exists, otherwise it will cause null pointer access. Signed-off-by: wenwu ma --- lib/librte_mbuf/rte_mbuf.h |

Re: [dpdk-dev] [PATCH v7] app/testpmd: support multi-process

2021-03-29 Thread Min Hu (Connor)
在 2021/3/29 15:51, Li, Xiaoyun 写道: Hi -Original Message- From: Min Hu (Connor) Sent: Friday, March 26, 2021 16:53 To: dev@dpdk.org Cc: Yigit, Ferruh ; ajit.khapa...@broadcom.com; Li, Xiaoyun Subject: [PATCH v7] app/testpmd: support multi-process From: Lijun Ou This patch adds mu

[dpdk-dev] [PATCH v8] app/testpmd: support multi-process

2021-03-29 Thread Min Hu (Connor)
From: Lijun Ou This patch adds multi-process support for testpmd. The test cmd example as follows: the primary cmd: ./dpdk-testpmd -a xxx --proc-type=auto -l 0-1 -- -i \ --rxq=4 --txq=4 --num-procs=2 --proc-id=0 the secondary cmd: ./dpdk-testpmd -a xxx --proc-type=auto -l 2-3 -- -i \ --rxq=4 --t

Re: [dpdk-dev] [PATCH v4 6/6] net/txgbe: add FFE parameters for user debugging

2021-03-29 Thread Jiawen Wu
On March 29, 2021 11:51 PM, Ferruh Yigit wrote: > On 3/29/2021 4:17 AM, Jiawen Wu wrote: > > Support to set PHY link mode by user defined. > > And update release notes 21.05 for txgbe. > > > > During merge, release notes update moved to 3/6 where looks more > appropriate, since this patch adds FFE

[dpdk-dev] [PATCH v3 4/4] regex/mlx5: prevent wrong calculation of free sqs in umr mode

2021-03-29 Thread Suanming Mou
From: John Hurley A recent change adds support for scattered mbuf and UMR support for regex. Part of this commit makes the pi and ci counters of the regex_sq a quarter of the length in non umr mode, effectively moving them from 16 bits to 14. The new get_free method casts the difference in pi and

[dpdk-dev] [PATCH v3 2/4] regex/mlx5: add data path scattered mbuf process

2021-03-29 Thread Suanming Mou
UMR WQE can convert multiple mkey's memory sapce to contiguous space. Take advantage of the UMR WQE, scattered mbuf in one operation can be converted to an indirect mkey. The RegEx which only accepts one mkey can now process the whole scattered mbuf. The maximum scattered mbuf can be supported in

[dpdk-dev] [PATCH v3 3/4] app/test-regex: support scattered mbuf input

2021-03-29 Thread Suanming Mou
This commits adds the scattered mbuf input support. Signed-off-by: Suanming Mou Acked-by: Ori Kam --- app/test-regex/main.c | 134 +++-- doc/guides/tools/testregex.rst | 3 + 2 files changed, 112 insertions(+), 25 deletions(-) diff --git a/app/test-regex/

[dpdk-dev] [PATCH v3 1/4] common/mlx5: add user memory registration bits

2021-03-29 Thread Suanming Mou
This commit adds the UMR capability bits. Signed-off-by: Suanming Mou Acked-by: Ori Kam --- drivers/common/mlx5/linux/meson.build | 2 ++ drivers/common/mlx5/mlx5_devx_cmds.c | 5 + drivers/common/mlx5/mlx5_devx_cmds.h | 3 +++ 3 files changed, 10 insertions(+) diff --git a/drivers/commo

[dpdk-dev] [PATCH v3 0/4] regex/mlx5: support scattered mbuf

2021-03-29 Thread Suanming Mou
The scattered mbuf was not supported in mlx5 RegEx driver. This patch set adds the support of scattered mbuf by UMR WQE. UMR WQE can convert multiple mkey's memory sapce to contiguous space. Take advantage of the UMR WQE, scattered mbuf in one operation can be converted to an indirect mkey. The Re

Re: [dpdk-dev] [PATCH 3/4] vhost: avoid deadlock on async register

2021-03-29 Thread Hu, Jiayu
Hi Maxime, > -Original Message- > From: Maxime Coquelin > Sent: Monday, March 29, 2021 11:19 PM > To: Hu, Jiayu ; dev@dpdk.org > Cc: Xia, Chenbo ; Wang, Yinan > ; Jiang, Cheng1 ; Pai G, > Sunil > Subject: Re: [PATCH 3/4] vhost: avoid deadlock on async register > > > > On 3/17/21 1:56

Re: [dpdk-dev] [PATCH v16 1/3] build: disable/enable drivers in Arm builds

2021-03-29 Thread Honnappa Nagarahalli
> > > > > > > > > > > > > > The blocklist is, I think, agreed upon by everyone. > > > > > > > > > > The question is whether we want to support the > > > > > > > > > > allowlist alongside it and there seem to be enough reasons > > > > > > > > > > to do > that. > > > > > > > > > Sorry, may be thi

Re: [dpdk-dev] [PATCH v3] pflock: implementation of phase-fair reader writer locks

2021-03-29 Thread Honnappa Nagarahalli
> Subject: Re: [PATCH v3] pflock: implementation of phase-fair reader writer > locks > > Meta question: is implementing trylock worth it? > The original did not have it. If there is no use for it currently, I suggest not to add. If someone sees a need, they can always add. I am ok if you add a

[dpdk-dev] [PATCH v5 05/10] eal: implement thread priority management functions

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Add function for setting the priority for a thread. Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/rte_thread.c | 25 ++ lib/librte_eal/include/rte_thread.h | 17 +++ lib/librte_eal/windows/rte_thread.c | 76 + 3 files chan

[dpdk-dev] [PATCH v5 07/10] eal: implement functions for mutex management

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Add functions for mutex init, destroy, lock, unlock. Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/rte_thread.c| 24 + lib/librte_eal/include/rte_thread.h | 53 +++ lib/librte_eal/include/rte_thread_types.h | 3 +

[dpdk-dev] [PATCH v5 08/10] eal: implement functions for thread barrier management

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Add functions for barrier init, destroy, wait. Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/rte_thread.c| 16 +++ lib/librte_eal/include/rte_thread.h | 46 +++ lib/librte_eal/include/rte_thread_types.h | 2 + .../in

[dpdk-dev] [PATCH v5 09/10] eal: add EAL argument for setting thread priority

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Allow the user to choose the thread priority through an EAL command line argument. The user can select the thread priority to be either 'normal' or 'critical': --thread-prio normal --thread-prio realtime Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/eal_common_o

[dpdk-dev] [PATCH v5 06/10] eal: add thread lifetime management

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Add function for thread creation, join, canceling. Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/rte_thread.c | 110 lib/librte_eal/include/rte_thread.h | 53 lib/librte_eal/windows/rte_thread.c | 125 ++

[dpdk-dev] [PATCH v5 03/10] windows/eal: translate Windows errors to errno-style errors

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Add function to translate Windows error codes to errno-style error codes. Signed-off-by: Narcisa Vasile --- lib/librte_eal/windows/rte_thread.c | 65 ++--- 1 file changed, 50 insertions(+), 15 deletions(-) diff --git a/lib/librte_eal/windows/rte_th

[dpdk-dev] [PATCH v5 04/10] eal: implement functions for thread affinity management

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Implement functions for getting/setting thread affinity. Signed-off-by: Narcisa Vasile Signed-off-by: Dmitry Malloy --- lib/librte_eal/common/rte_thread.c | 13 ++ lib/librte_eal/include/rte_thread.h | 41 +++ lib/librte_eal/windows/eal_lcore.c | 170 ++

[dpdk-dev] [PATCH v5 01/10] eal: add thread id and simple thread functions

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Add the thread identifier type. Add functions for comparing thread ids and obtaining the thread id for the current thread. Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/rte_thread.c| 101 ++ lib/librte_eal/include/rte_thread.h

[dpdk-dev] [PATCH v5 00/10] eal: Add new API for threading

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile EAL thread API **Problem Statement** DPDK currently uses the pthread interface to create and manage threads. Windows does not support the POSIX thread programming model, so it currently relies on a header file that hides the Windows calls under pthread matched interfaces. Gi

[dpdk-dev] [PATCH v5 02/10] eal: add thread attributes

2021-03-29 Thread Narcisa Ana Maria Vasile
From: Narcisa Vasile Implement thread attributes for: * thread affinity * thread priority Implement functions for managing thread attributes. Signed-off-by: Narcisa Vasile --- lib/librte_eal/common/rte_thread.c| 53 lib/librte_eal/include/rte_thread.h | 82 +

Re: [dpdk-dev] [PATCH 2/6] net/hns3: fix compiling error for using SVE algorithm

2021-03-29 Thread Honnappa Nagarahalli
> Subject: Re: [PATCH 2/6] net/hns3: fix compiling error for using SVE algorithm > > On Mon, Mar 29, 2021 at 8:32 PM Aaron Conole > wrote: > > Ferruh Yigit writes: > > > This is a very obvious build error, I am concerned how this is > > > released. Do you have any internal testing? > > > > > >

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

2021-03-29 Thread Thomas Monjalon
25/09/2020 14:31, Maxime Coquelin: > Add DMA map/unmap operation callbacks to the vdev bus, which > could be used by DMA capable vdev drivers. [...] > @@ -551,6 +601,8 @@ static struct rte_bus rte_vdev_bus = { > .plug = vdev_plug, > .unplug = vdev_unplug, > .parse = vdev_parse, >

Re: [dpdk-dev] [PATCH 2/2] net/mlx5: fix missing shared RSS hash types

2021-03-29 Thread Matan Azrad
From: Xiaoyu Min > Shared RSS action create all supported RSS hash combination in advance and > lookup the right hash TIR when flow is actually applied by comparing hash > field > value. > > Unfortunately some hash combination is missed, for example, UDP/TCP dest > port only, L3-src-only, etc.

Re: [dpdk-dev] [PATCH 1/2] net/mlx5: fix shared inner RSS

2021-03-29 Thread Matan Azrad
From: Xiaoyu Min > The shared RSS action use the _tunnel_ information which is derived from flow > items to decide whether need to do inner RSS or not. > However, inner RSS should be decided by RSS level (>1) in configuration and > then to create TIR with 'IBV_RX_HASH_INNER' hash bit set. > > A

Re: [dpdk-dev] [PATCH v3 1/2] ethdev: replace callback getting filter operations

2021-03-29 Thread Matan Azrad
From: Thomas Monjalon > Since rte_flow is the only API for filtering operations, > the legacy driver interface filter_ctrl was too much complicated > for the simple task of getting the struct rte_flow_ops. > > The filter type RTE_ETH_FILTER_GENERIC and > the filter operarion RTE_ETH_FILTER_GET

Re: [dpdk-dev] [PATCH 1/2] timer: clarify subsystem_init return value

2021-03-29 Thread Carrillo, Erik G
> -Original Message- > From: Michal Krawczyk > Sent: Friday, March 26, 2021 5:48 AM > To: dev@dpdk.org; Robert Sanford ; Carrillo, Erik G > > Cc: Stanislaw Kardach ; Michal Krawczyk > > Subject: [PATCH 1/2] timer: clarify subsystem_init return value > > From: Stanislaw Kardach > > rte

Re: [dpdk-dev] [PATCH 1/2] [RFC]: ethdev: add pre-defined meter policy API

2021-03-29 Thread Matan Azrad
From: Dumitrescu, Cristian > Hi Matan, > > > -Original Message- > > From: Matan Azrad > > Sent: Thursday, March 25, 2021 6:57 AM > > To: Dumitrescu, Cristian ; Li Zhang > > ; Dekel Peled ; Ori Kam > > ; Slava Ovsiienko ; Shahaf > > Shuler ; lir...@marvell.com; Singh, Jasvinder > > ; NBU

Re: [dpdk-dev] [PATCH 1/2] [RFC]: ethdev: add pre-defined meter policy API

2021-03-29 Thread Matan Azrad
Hi Jerin Thanks for the review. PSB From: Jerin Jacob > On Thu, Mar 18, 2021 at 2:28 PM Li Zhang wrote: > > > > Currently, the flow meter policy does not support multiple actions per > > color; also the allowed action types per color are very limited. > > In addition, the policy cannot be pre-de

Re: [dpdk-dev] [PATCH 2/2] [RFC]: ethdev: manage meter API object handles by the drivers

2021-03-29 Thread Matan Azrad
Hi Jerin Thanks for the review. PSB From: Jerin Jacob > On Thu, Mar 25, 2021 at 1:51 PM Matan Azrad wrote: > > > > Hi Cristian > > > > From: Dumitrescu, Cristian > > > Hi Li and Matan, > > > > > > > -Original Message- > > > > From: Li Zhang > > > > Sent: Thursday, March 18, 2021 8:58 AM

Re: [dpdk-dev] [PATCH v3] pflock: implementation of phase-fair reader writer locks

2021-03-29 Thread Stephen Hemminger
Meta question: is implementing trylock worth it? The original did not have it. There are tradeoffs about number of readers and added complexity in the code? > > diff --git a/lib/librte_eal/include/generic/rte_pflock.h > > b/lib/librte_eal/include/generic/rte_pflock.h > > new file mode 100644 > >

Re: [dpdk-dev] [PATCH 2/2] [RFC]: ethdev: manage meter API object handles by the drivers

2021-03-29 Thread Matan Azrad
Hi Ajit Looks like you agree with this RFC to move meter objects to be managed by PMD pointers. Thanks for the review! From: Ajit Khaparde > On Thu, Mar 25, 2021 at 1:21 AM Matan Azrad wrote: > > > > Hi Cristian > > > > From: Dumitrescu, Cristian > > > Hi Li and Matan, > > > > > > > -Origi

Re: [dpdk-dev] [PATCH 2/6] net/hns3: fix compiling error for using SVE algorithm

2021-03-29 Thread David Marchand
On Mon, Mar 29, 2021 at 8:32 PM Aaron Conole wrote: > Ferruh Yigit writes: > > This is a very obvious build error, I am concerned how this is > > released. Do you have any internal testing? > > > > + Aaron & Honnappa, > > > > If we can have a build test in our public CI with SVE? > > Maybe it's p

Re: [dpdk-dev] [PATCH v2] ethdev: introduce enable_driver_sdk to install driver headers

2021-03-29 Thread Thomas Monjalon
29/03/2021 17:23, Ferruh Yigit: > On 3/29/2021 1:10 PM, Thomas Monjalon wrote: > > 29/03/2021 11:43, Ferruh Yigit: > >> On 3/26/2021 8:52 PM, Tyler Retzlaff wrote: > >>> On Fri, Mar 26, 2021 at 12:02:55PM +, Ferruh Yigit wrote: > On 3/24/2021 4:24 PM, Tyler Retzlaff wrote: > > On Wed,

Re: [dpdk-dev] [PATCH 2/6] net/hns3: fix compiling error for using SVE algorithm

2021-03-29 Thread Aaron Conole
Ferruh Yigit writes: > On 3/23/2021 1:45 PM, Min Hu (Connor) wrote: >> From: Huisong Li >> >> The 'queue_full_cnt' stats have been encapsulated in 'dfx_stats'. >> However, the modification in the SVE algorithm is omitted. >> As a result, the driver fails to be compiled when the SVE >> algorithm

Re: [dpdk-dev] Questions about reporting auto-negotiation capability

2021-03-29 Thread Ajit Khaparde
On Mon, Mar 29, 2021 at 12:19 AM Thomas Monjalon wrote: > > 29/03/2021 09:03, Thomas Monjalon: > > 29/03/2021 06:02, Huisong Li: > > > 'speed_capa' in struct rte_eth_dev_info is defined as follows: > > > > > > uint32_t speed_capa; /**< Supported speeds bitmap (ETH_LINK_SPEED_). */ > > >

Re: [dpdk-dev] [PATCH] drivers/net: remove useless autoneg capability

2021-03-29 Thread Ajit Khaparde
On Mon, Mar 29, 2021 at 12:29 AM Thomas Monjalon wrote: > > The flag ETH_LINK_SPEED_AUTONEG is 0, > so it cannot be used in a capability bitmap. > > Having 0 in speed capability means all speeds are accepted. > > Signed-off-by: Thomas Monjalon > --- > drivers/net/bnxt/bnxt_ethdev.c | 2

Re: [dpdk-dev] [PATCH v3] pflock: implementation of phase-fair reader writer locks

2021-03-29 Thread Honnappa Nagarahalli
> > > > > Subject: [PATCH v3] pflock: implementation of phase-fair reader > > > writer locks > > > > > > This is a new type of reader-writer lock that provides better > > > fairness guarantees which makes it better for typical DPDK applications. > > > They lock internally uses two ticket pools, o

Re: [dpdk-dev] rte_flow ageing

2021-03-29 Thread Asaf Penso
Hello David, Thanks for reaching out, I'll try to answer as best as I know and I added Matan who will be able to provide further info during next week. First, according to our pmd documentation (http://doc.dpdk.org/guides/nics/mlx5.html#supported-hardware-offloads) we recommend using DPDK20.11

Re: [dpdk-dev] [PATCH v3] pflock: implementation of phase-fair reader writer locks

2021-03-29 Thread Stephen Hemminger
On Mon, 29 Mar 2021 03:14:29 + Honnappa Nagarahalli wrote: > > > > Subject: [PATCH v3] pflock: implementation of phase-fair reader writer locks > > > > This is a new type of reader-writer lock that provides better fairness > > guarantees which makes it better for typical DPDK applications.

[dpdk-dev] [PATCH] doc: remove obsolete future considerations in flow guide

2021-03-29 Thread Thomas Monjalon
After 4 years, rte_flow has evolved enough to not require special notes about what could be added in future. Part of the removed plans were obsolete anyway. Signed-off-by: Thomas Monjalon --- doc/guides/prog_guide/rte_flow.rst | 38 +- 1 file changed, 1 insertion(+),

Re: [dpdk-dev] [PATCH 1/2] [RFC]: ethdev: add pre-defined meter policy API

2021-03-29 Thread Dumitrescu, Cristian
Hi Ori, > -Original Message- > From: Ori Kam > Sent: Monday, March 29, 2021 10:23 AM > To: Matan Azrad ; Dumitrescu, Cristian > ; Li Zhang ; Dekel Peled > ; Slava Ovsiienko ; Shahaf > Shuler ; lir...@marvell.com; Singh, Jasvinder > ; NBU-Contact-Thomas Monjalon > ; Yigit, Ferruh ; Andrew

Re: [dpdk-dev] [PATCH 2/6] net/hns3: fix compiling error for using SVE algorithm

2021-03-29 Thread Ferruh Yigit
On 3/23/2021 1:45 PM, Min Hu (Connor) wrote: From: Huisong Li The 'queue_full_cnt' stats have been encapsulated in 'dfx_stats'. However, the modification in the SVE algorithm is omitted. As a result, the driver fails to be compiled when the SVE algorithm is used. Fixes: 9b77f1fe303f ("net/hns3

Re: [dpdk-dev] [PATCH 1/2] [RFC]: ethdev: add pre-defined meter policy API

2021-03-29 Thread Dumitrescu, Cristian
Hi Matan, > -Original Message- > From: Matan Azrad > Sent: Thursday, March 25, 2021 6:57 AM > To: Dumitrescu, Cristian ; Li Zhang > ; Dekel Peled ; Ori Kam > ; Slava Ovsiienko ; Shahaf > Shuler ; lir...@marvell.com; Singh, Jasvinder > ; NBU-Contact-Thomas Monjalon > ; Yigit, Ferruh ; Andr

Re: [dpdk-dev] [PATCH 01/25] event/dlb2: add dlb v2.5 probe

2021-03-29 Thread McDaniel, Timothy
> -Original Message- > From: Jerin Jacob > Sent: Monday, March 29, 2021 10:51 AM > To: McDaniel, Timothy > Cc: dpdk-dev ; Jerin Jacob ; Van Haaren, > Harry ; Ray Kinsella ; Neil > Horman ; Rao, Nikhil ; > Carrillo, Erik G ; Gujjar, Abhinandan S > ; Pavan Nikhilesh > ; Hemant Agrawal ; >

Re: [dpdk-dev] [PATCH v4 0/6] txgbe backplane AN training

2021-03-29 Thread Ferruh Yigit
On 3/29/2021 4:17 AM, Jiawen Wu wrote: This series update link process for backplane NICs. And support to control AN training. v4: - Convert BP_LOG to dynamic log. v3: - Add devargs gradually in the patches they are used. - Update 21.05 relaese notes. v2: - Convert compile time flags to runtime

Re: [dpdk-dev] [PATCH 01/25] event/dlb2: add dlb v2.5 probe

2021-03-29 Thread Jerin Jacob
On Mon, Mar 29, 2021 at 8:30 PM McDaniel, Timothy wrote: > > > > > -Original Message- > > From: Jerin Jacob > > Sent: Friday, March 26, 2021 9:33 AM > > To: McDaniel, Timothy > > Cc: dpdk-dev ; Jerin Jacob ; Van Haaren, > > Harry ; Ray Kinsella ; Neil > > Horman ; Rao, Nikhil ; > > Carri

Re: [dpdk-dev] [PATCH v4 6/6] net/txgbe: add FFE parameters for user debugging

2021-03-29 Thread Ferruh Yigit
On 3/29/2021 4:17 AM, Jiawen Wu wrote: Support to set PHY link mode by user defined. And update release notes 21.05 for txgbe. During merge, release notes update moved to 3/6 where looks more appropriate, since this patch adds FFE parameters but note is about AN training. But please let me k

Re: [dpdk-dev] [PATCH v2] ethdev: introduce enable_driver_sdk to install driver headers

2021-03-29 Thread Ferruh Yigit
On 3/29/2021 1:10 PM, Thomas Monjalon wrote: 29/03/2021 11:43, Ferruh Yigit: On 3/26/2021 8:52 PM, Tyler Retzlaff wrote: On Fri, Mar 26, 2021 at 12:02:55PM +, Ferruh Yigit wrote: On 3/24/2021 4:24 PM, Tyler Retzlaff wrote: On Wed, Mar 24, 2021 at 12:30:36PM +0100, Thomas Monjalon wrote:

Re: [dpdk-dev] [PATCH 3/4] vhost: avoid deadlock on async register

2021-03-29 Thread Maxime Coquelin
On 3/17/21 1:56 PM, Jiayu Hu wrote: > Users register async copy device when vhost queue is enabled. > However, if VHOST_USER_F_PROTOCOL_FEATURES is not supported, > a deadlock occurs inside rte_vhost_async_channel_register(), > as vhost_user_msg_handler() already takes vq->access_lock > before p

[dpdk-dev] [PATCH v2 2/2] event/octeontx2: support crypto adapter forward mode

2021-03-29 Thread Shijith Thotton
Advertise crypto adapter forward mode capability and set crypto adapter enqueue function in driver. Signed-off-by: Shijith Thotton --- drivers/crypto/octeontx2/otx2_cryptodev_ops.c | 42 ++ drivers/event/octeontx2/otx2_evdev.c | 5 +- .../event/octeontx2/otx2_evdev_crypto_adptr

[dpdk-dev] [PATCH v2 1/2] eventdev: introduce crypto adapter enqueue API

2021-03-29 Thread Shijith Thotton
From: Akhil Goyal In case an event from a previous stage is required to be forwarded to a crypto adapter and PMD supports internal event port in crypto adapter, exposed via capability RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_OP_FWD, we do not have a way to check in the API rte_event_enqueue_bur

[dpdk-dev] [PATCH v2 0/2] Enhancements to crypto adapter forward mode

2021-03-29 Thread Shijith Thotton
This series proposes a new event device enqueue operation if crypto adapter forward mode is supported. Second patch in the series is the implementation of the same in PMD. Test application changes for the usage of new API is yet to add. v2: - Updated release notes. - Made use of RTE_EVENTDEV_VALID

Re: [dpdk-dev] [PATCH 2/4] vhost: remove unnecessary free

2021-03-29 Thread Maxime Coquelin
On 3/17/21 1:56 PM, Jiayu Hu wrote: > This patch removes unnecessary rte_free() for async_pkts_info > and async_descs_split. > > Signed-off-by: Jiayu Hu > --- > lib/librte_vhost/vhost_user.c | 7 --- > 1 file changed, 7 deletions(-) > Reviewed-by: Maxime Coquelin Thanks, Maxime

Re: [dpdk-dev] [PATCH 01/25] event/dlb2: add dlb v2.5 probe

2021-03-29 Thread McDaniel, Timothy
> -Original Message- > From: Jerin Jacob > Sent: Friday, March 26, 2021 9:33 AM > To: McDaniel, Timothy > Cc: dpdk-dev ; Jerin Jacob ; Van Haaren, > Harry ; Ray Kinsella ; Neil > Horman ; Rao, Nikhil ; > Carrillo, Erik G ; Gujjar, Abhinandan S > ; Pavan Nikhilesh > ; Hemant Agrawal ; >

[dpdk-dev] [PATCH v5 3/3] kni: fix kernel deadlock when using mlx devices

2021-03-29 Thread Ferruh Yigit
KNI runs userspace callback with rtnl lock held, this is not working fine with some devices that needs to interact with kernel interface in the callback, like Mellanox devices. The solution is releasing the rtnl lock before calling the userspace callback. But it requires two consideration: 1. The

[dpdk-dev] [PATCH v5 2/3] kni: support async user request

2021-03-29 Thread Ferruh Yigit
Adding async userspace requests which don't wait for the userspace response and always return success. This is preparation to address a regression in KNI. Signed-off-by: Elad Nachman Signed-off-by: Ferruh Yigit --- kernel/linux/kni/kni_net.c | 9 + lib/librte_kni/rte_kni.c|

[dpdk-dev] [PATCH v5 1/3] kni: refactor user request processing

2021-03-29 Thread Ferruh Yigit
From: Elad Nachman Refactor the parameter kni_net_process_request() gets, this is preparation for addressing a user request processing deadlock problem. Signed-off-by: Stephen Hemminger Signed-off-by: Elad Nachman --- kernel/linux/kni/kni_net.c | 25 + 1 file changed,

Re: [dpdk-dev] [PATCH] vdpa/mlx5: improve interrupt management

2021-03-29 Thread Maxime Coquelin
On 3/18/21 10:13 AM, Matan Azrad wrote: > The driver should notify the guest for each traffic burst detected by CQ > polling. > > The CQ polling trigger is defined by `event_mode` device argument, > either by busy polling on all the CQs or by blocked call to HW > completion event using DevX cha

[dpdk-dev] [PATCH 8/8] net/ice/base: add RSS support for PPPoL2TPv2oUDP

2021-03-29 Thread Qi Zhang
Add support for PPPoL2TPv2oUDP RSS hash. L2TPv2 and PPP ptypes and flow headers are added. Prot id for PPP is added. Signed-off-by: Ting Xu Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_flow.c | 65 ++-- drivers/net/ice/base/ice_flow.h | 2 + driver

[dpdk-dev] [PATCH 7/8] net/ice/base: enable I2C read/write commands

2021-03-29 Thread Qi Zhang
Enable I2C read/write AQ commands. They are now required for controlling the external physical connectors via external I2C port expander on E810-T adapters. Signed-off-by: Maciej Machnikowski Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_adminq_cmd.h | 29 + drivers/net/ice/base/

[dpdk-dev] [PATCH 6/8] net/ice/base: change protocol ID for VLAN in case of DVM

2021-03-29 Thread Qi Zhang
Protocol id for first vlan in Double VLAN Mode (DVM) should be ICE_VLAN_OF_HW = 16, but for Single VLAN Mode (SVM) this should be ICE_VLAN_OL_HW = 17. Change protocol id in type to id translation array for outer vlan to 17 when DVM is enabled, which means the driver, package, and firmware support

[dpdk-dev] [PATCH 5/8] net/ice/base: set MAC type for E823C device

2021-03-29 Thread Qi Zhang
Set E823C device's MAC type as generic. Signed-off-by: Anirudh Venkataramanan Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/ice/base/ice_common.c b/drivers/net/ice/base/ice_common.c index 5c21d71b70..356a8b4d0

[dpdk-dev] [PATCH 4/8] net/ice/base: remove unused ptype field in PTT definition

2021-03-29 Thread Qi Zhang
Remove the unused ptype entry, and use the gcc extension for ranged initializers in arrays for Linux, and explicitly target each table entry by index when initializing under Linux. Signed-off-by: Jesse Brandeburg Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_lan_tx_rx.h | 10 --

[dpdk-dev] [PATCH 3/8] net/ice/base: print link configure error

2021-03-29 Thread Qi Zhang
Newer NVMs return link_cfg_err for get_link_status AQ. Print it for debug use. Signed-off-by: Anirudh Venkataramanan Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ice/base/ice_common.c b/drivers/net/ice/base/ice_co

[dpdk-dev] [PATCH 2/8] net/ice/base: support removing VSI from flow profile

2021-03-29 Thread Qi Zhang
Adding a function ice_flow_rem_vsi_prof() to remove flow entries associated to the sw vsi handle. Once complete clear the vsi index from the flow profile bitmap. This will ensure that the a vsi once removed can be re-added and the package block rules will be added again. Signed-off-by: Vignesh Sri

[dpdk-dev] [PATCH 1/8] net/ice/base: code cleanup

2021-03-29 Thread Qi Zhang
1. There are a lots of function header mismatch its function name. 2. remove unnecessary header file include. 3. remove unnecessary macro. 4. remove unnecessary comment. Signed-off-by: Dave Ertman Signed-off-by: Qi Zhang --- drivers/net/ice/base/ice_common.c| 4 ++-- drivers/net/ice/base/i

[dpdk-dev] [PATCH 0/8] ice: base code update batch 2

2021-03-29 Thread Qi Zhang
Qi Zhang (8): net/ice/base: code cleanup net/ice/base: support removing VSI from flow profile net/ice/base: print link configure error net/ice/base: remove unused ptype field in PTT definition net/ice/base: set MAC type for E823C device net/ice/base: change protocol ID for VLAN in case

[dpdk-dev] [PATCH v4] build: add platform meson option

2021-03-29 Thread Juraj Linkeš
The current meson option 'machine' should only specify the ISA, which is not sufficient for Arm, where setting ISA implies other setting as well. Add a new meson option, 'platform', which differentiates the type of the build (native/generic) and sets machine accordingly, unless the user chooses to

Re: [dpdk-dev] [PATCH v3] net/ixgbe: fix RSS RETA be reset after port start

2021-03-29 Thread Wang, Haiyue
> -Original Message- > From: Yang, MurphyX > Sent: Monday, March 29, 2021 16:29 > To: dev@dpdk.org > Cc: Yang, Qiming ; Wang, Haiyue > ; Guo, Jia > ; Yang, SteveX ; Zhang, RobinX > ; > Yang, MurphyX > Subject: [PATCH v3] net/ixgbe: fix RSS RETA be reset after port start > > If one call

[dpdk-dev] rte_flow ageing

2021-03-29 Thread David Bouyeure
Hi, I've found out the pretty useful experimental brand new flow ageing API implemented in the mlx5 PMD. I'm trying it (rte_eth_dev_callback_register(RTE_ETH_EVENT_FLOW_AGED), RTE_FLOW_ACTION_TYPE_AGE) to recover any flow that I previously offloaded. The DPDK version is 20.08 and Mellanox(

Re: [dpdk-dev] rte_flow ageing

2021-03-29 Thread David Bouyeure
Sorry for the below topic confusion(in-reply-to). On 3/29/21 9:50 AM, David Bouyeure wrote: Hi, I've found out the pretty useful experimental brand new flow ageing API implemented in the mlx5 PMD. I'm trying it (rte_eth_dev_callback_register(RTE_ETH_EVENT_FLOW_AGED), RTE_FLOW_ACTION_TYPE_A

Re: [dpdk-dev] [PATCH v2] vhost: add support for packed ring in async vhost

2021-03-29 Thread Jiang, Cheng1
Hi, > -Original Message- > From: Liu, Yong > Sent: Wednesday, March 24, 2021 5:19 PM > To: Jiang, Cheng1 ; maxime.coque...@redhat.com; > Xia, Chenbo > Cc: dev@dpdk.org; Hu, Jiayu ; Yang, YvonneX > ; Wang, Yinan ; Jiang, > Cheng1 > Subject: RE: [dpdk-dev] [PATCH v2] vhost: add support fo

Re: [dpdk-dev] [PATCH v2] ethdev: introduce enable_driver_sdk to install driver headers

2021-03-29 Thread Thomas Monjalon
29/03/2021 11:43, Ferruh Yigit: > On 3/26/2021 8:52 PM, Tyler Retzlaff wrote: > > On Fri, Mar 26, 2021 at 12:02:55PM +, Ferruh Yigit wrote: > >> On 3/24/2021 4:24 PM, Tyler Retzlaff wrote: > >>> On Wed, Mar 24, 2021 at 12:30:36PM +0100, Thomas Monjalon wrote: > 24/03/2021 12:27, Ferruh Yig

Re: [dpdk-dev] [EXTERNAL] Re: Handling missing export functions in MSVC linkage

2021-03-29 Thread Tyler Retzlaff
Unless I misunderstood the last status of this issue was that it was desired empty stubs that fail at runtime as opposed to conditionally generating exports. Is this no longer the case? -Original Message- From: Thomas Monjalon Sent: Friday, March 26, 2021 1:40 AM To: Dmitry Kozlyuk ; r

  1   2   >