[dpdk-dev] [PATCH v4] net/bonding: change the state machine to defaulted

2020-07-17 Thread Weifeng Li
A dpdk bonding 802.3ad network as follows: +--++---+ |dpdk lacp |bond1.1 <--> bond2.1|switch lacp| | |bond1.2 <--> bond2.2| | +--++---+ If a fiber optic go wrong about single pass during norma

[dpdk-dev] [PATCH v3] net/bonding: change the state machine to defaulted

2020-07-17 Thread Weifeng Li
A dpdk bonding 802.3ad network as follows: +--++---+ |dpdk lacp |bond1.1 <--> bond2.1|switch lacp| | |bond1.2 <--> bond2.2| | +--++---+ If a fiber optic go wrong about single pass during norma

Re: [dpdk-dev] [PATCH] net/i40e: fix incorrect hash look up table

2020-07-17 Thread Jeff Guo
hi, shougang On 7/15/2020 5:07 PM, Chen, BoX C wrote: Tested-by: zhang,xi Regards, Chen Bo -Original Message- From: dev On Behalf Of Shougang Wang Sent: July 15, 2020 14:35 To: dev@dpdk.org Cc: Xing, Beilei ; Guo, Jia ; Wang, ShougangX ; sta...@dpdk.org Subject: [dpdk-dev] [PATCH] n

Re: [dpdk-dev] [PATCH] net/ice: fix issue for GTPU and GTPU extension packet

2020-07-17 Thread Zhang, Qi Z
> -Original Message- > From: Su, Simei > Sent: Friday, July 17, 2020 11:28 AM > To: Zhang, Qi Z > Cc: dev@dpdk.org; Guo, Jia ; Su, Simei > > Subject: [PATCH] net/ice: fix issue for GTPU and GTPU extension packet > > Because of incomplete protocol header fields, GTPU_INNER_IPV4_UDP an

Re: [dpdk-dev] ixgbe vector rx does not conform torte_eth_rx_burst() API

2020-07-17 Thread Zhao1, Wei
> -Original Message- > From: dev On Behalf Of Zhao1, Wei > Sent: Saturday, July 18, 2020 11:32 AM > To: Morten Brørup ; Guo, Jia > > Cc: dev@dpdk.org; Thomas Monjalon ; Yigit, Ferruh > ; Andrew Rybchenko > Subject: Re: [dpdk-dev] ixgbe vector rx does not conform torte_eth_rx_burst() >

Re: [dpdk-dev] ixgbe vector rx does not conform torte_eth_rx_burst() API

2020-07-17 Thread Zhao1, Wei
HI, > -Original Message- > From: Morten Brørup > Sent: Thursday, July 16, 2020 5:09 PM > To: Zhao1, Wei ; Guo, Jia > Cc: dev@dpdk.org; Thomas Monjalon ; Yigit, Ferruh > ; Andrew Rybchenko > Subject: RE: [dpdk-dev] ixgbe vector rx does not conform torte_eth_rx_burst() > API > > > From:

[dpdk-dev] ?????? [dpdk-stable] [PATCH V2] net/bonding: delete redundant code

2020-07-17 Thread ??????????
Ok, my signature is :       Dongyang Pan <197020...@qq.com> Thanks! --  -- ??: "Ferruh Yigit"

Re: [dpdk-dev] [PATCH v4 26/27] memif: replace master/slave with server/client

2020-07-17 Thread Stephen Hemminger
On Fri, 17 Jul 2020 14:01:46 +0100 "Burakov, Anatoly" wrote: > On 01-Jul-20 9:23 PM, Stephen Hemminger wrote: > > The names master and slave have no place inside this driver. > > It is really a traditional client server model, so use those names. > > > > Signed-off-by: Stephen Hemminger > > ---

Re: [dpdk-dev] [PATCH v4 25/27] eal: mark old naming as deprecated

2020-07-17 Thread Stephen Hemminger
On Fri, 17 Jul 2020 16:21:37 +0100 "Burakov, Anatoly" wrote: > On 16-Jul-20 11:04 PM, Stephen Hemminger wrote: > > On Thu, 16 Jul 2020 14:41:41 +0100 > > "Burakov, Anatoly" wrote: > > > >> On 15-Jul-20 9:29 PM, Stephen Hemminger wrote: > >>> On Wed, 15 Jul 2020 14:28:17 +0100 > >>> "Burakov

Re: [dpdk-dev] [PATCH v8 2/3] devtools: prevent use of rte atomic APIs in future patches

2020-07-17 Thread Stephen Hemminger
On Fri, 17 Jul 2020 22:48:04 + Honnappa Nagarahalli wrote: > > > > Don't we want to convert all libs? > > > The goal is to convert all the libs. > > > > > > > If we are adding one more rte_atomic in a lib, we should ask the > > > > question why not converting to C11, no? > > > Agree, I

Re: [dpdk-dev] [PATCH] lib/librte_eal: support SVE flag on ARM64

2020-07-17 Thread Wei Hu (Xavier)
Hi, Honnappa Nagarahalli On 2020/7/18 7:04, Honnappa Nagarahalli wrote: Hi, Thanks for the patch. Has this been tested on any platform (simulator, emulator etc)? Do you plan to add more code using SVE? We have tested it on FPGA board and will upstream some code using SVE later. Thank

Re: [dpdk-dev] [PATCH] lib/librte_eal: support SVE flag on ARM64

2020-07-17 Thread Honnappa Nagarahalli
Hi, Thanks for the patch. Has this been tested on any platform (simulator, emulator etc)? Do you plan to add more code using SVE? Thank you, Honnappa > -Original Message- > From: dev On Behalf Of Wei Hu (Xavier) > Sent: Friday, July 17, 2020 7:08 AM > To: dev@dpdk.org > Cc: xavi

Re: [dpdk-dev] [PATCH v8 2/3] devtools: prevent use of rte atomic APIs in future patches

2020-07-17 Thread Honnappa Nagarahalli
> > > > > > > > > > > Subject: Re: [dpdk-dev] [PATCH v8 2/3] devtools: prevent use of > > > > > rte atomic APIs in future patches > > > > > > > > > > 16/07/2020 12:48, David Marchand: > > > > > > On Thu, Jul 16, 2020 at 6:58 AM Phil Yang > wrote: > > > > > > > check_forbidden_additions() { # >

Re: [dpdk-dev] Random failure in service_autotest

2020-07-17 Thread Honnappa Nagarahalli
> Subject: Re: [dpdk-dev] Random failure in service_autotest > > Lukasz Wojciechowski writes: > > > W dniu 17.07.2020 o 17:19, David Marchand pisze: > >> On Fri, Jul 17, 2020 at 10:56 AM David Marchand > >> wrote: > >>> On Wed, Jul 15, 2020 at 12:41 PM Ferruh Yigit > wrote: > On 7/15/20

Re: [dpdk-dev] Random failure in service_autotest

2020-07-17 Thread Aaron Conole
Lukasz Wojciechowski writes: > W dniu 17.07.2020 o 17:19, David Marchand pisze: >> On Fri, Jul 17, 2020 at 10:56 AM David Marchand >> wrote: >>> On Wed, Jul 15, 2020 at 12:41 PM Ferruh Yigit >>> wrote: On 7/15/2020 11:14 AM, David Marchand wrote: > Hello Harry and guys who touched the

Re: [dpdk-dev] [PATCH v2 3/3] eal/windows: librte_net build on Windows

2020-07-17 Thread Narcisa Ana Maria Vasile
On Sun, Jun 28, 2020 at 09:48:25AM +, Fady Bader wrote: > > Subject: Re: [PATCH v2 3/3] eal/windows: librte_net build on Windows > > > > > diff --git a/lib/librte_eal/common/meson.build > > > b/lib/librte_eal/common/meson.build > > > index e1bdaf024..05e40a26b 100644 > > > --- a/lib/librte_eal

Re: [dpdk-dev] [PATCH 0/7] cmdline: support Windows

2020-07-17 Thread Narcisa Ana Maria Vasile
On Sun, Jun 21, 2020 at 12:05:03AM +0300, Dmitry Kozlyuk wrote: > This patchset enables librte_cmdline on Windows. To do that, it creates > a number of wrappers for OS-dependent terminal handling and I/O. > Considered alternative was to revive [1] and use libedit (Unix-only) > for terminal handling

[dpdk-dev] [Bug 510] Virtio driver breaks all apps which use ETH_MQ_RX_RSS

2020-07-17 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=510 Bug ID: 510 Summary: Virtio driver breaks all apps which use ETH_MQ_RX_RSS Product: DPDK Version: 19.08 Hardware: All OS: All Status: UNCONFIRMED Severity: normal

Re: [dpdk-dev] Random failure in service_autotest

2020-07-17 Thread Lukasz Wojciechowski
W dniu 17.07.2020 o 17:19, David Marchand pisze: > On Fri, Jul 17, 2020 at 10:56 AM David Marchand > wrote: >> On Wed, Jul 15, 2020 at 12:41 PM Ferruh Yigit wrote: >>> On 7/15/2020 11:14 AM, David Marchand wrote: Hello Harry and guys who touched the service code recently :-) I sp

[dpdk-dev] [PATCH] bus/vmbus: use SMP memory barrier for signaling read

2020-07-17 Thread Long Li
From: Long Li rte_smp_mb() uses the same locked ADD as the in-kernel vmbus driver, and it has slightly performance improvement over rte_mb(). Signed-off-by: Long Li --- drivers/bus/vmbus/vmbus_channel.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/bus/vmbus/vmbus

Re: [dpdk-dev] [PATCH v1 2/2] crypto/aesni_mb: improve security instance setup

2020-07-17 Thread De Lara Guarch, Pablo
Hi David, > -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:36 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De > Lara Guarch, Pablo ; Trahe, Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, > Mairtin ; Coyle, David > Subject: [PATCH v1 2/2] crypto/ae

Re: [dpdk-dev] [PATCH v1 2/2] crypto/aesni_mb: improve DOCSIS session creation

2020-07-17 Thread De Lara Guarch, Pablo
Hi David, > -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:32 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De > Lara Guarch, Pablo ; Trahe, Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, > Mairtin ; Coyle, David > Subject: [PATCH v1 2/2] crypto/ae

Re: [dpdk-dev] [PATCH v1] app/crypto-perf: set mbuf lengths correctly for DOCSIS tests

2020-07-17 Thread De Lara Guarch, Pablo
Hi David, > -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:31 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De > Lara Guarch, Pablo ; Trahe, Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, > Mairtin ; Coyle, David > Subject: [PATCH v1] app/crypto-pe

Re: [dpdk-dev] [PATCH] doc: update QAT driver version for D15xx

2020-07-17 Thread Trahe, Fiona
> -Original Message- > From: Dybkowski, AdamX > Sent: Thursday, July 16, 2020 12:41 PM > To: dev@dpdk.org; Trahe, Fiona ; akhil.go...@nxp.com > Cc: Dybkowski, AdamX > Subject: [PATCH] doc: update QAT driver version for D15xx > > This patch updates the minimum driver version that suppo

Re: [dpdk-dev] [PATCH v1 1/2] crypto/qat: improve DOCSIS session creation

2020-07-17 Thread Trahe, Fiona
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:32 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De Lara > Guarch, Pablo > ; Trahe, Fiona > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, Mairtin > ; Coyle, David > Subject: [PATCH v1 1/2] crypto/qat: impr

Re: [dpdk-dev] [PATCH v1] crypto/qat: check for multi-segment buffers for DOCSIS

2020-07-17 Thread Trahe, Fiona
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:34 PM > To: akhil.go...@nxp.com; Doherty, Declan ; Trahe, > Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, Mairtin > ; De Lara Guarch, Pablo > ; Coyle, David > > Subject: [PATCH v1] crypto/qat: che

[dpdk-dev] [PATCH 14/14] net/bnxt: remove unnecessary debug log

2020-07-17 Thread Somnath Kotur
remove a debug log for now Fixes: 75f9197c0ded ("net/bnxt: add support for identifier search and ref count") Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_mapper.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/net/bnxt/tf_ulp/ulp_mapper.c b/drivers/net/bnxt/tf_u

[dpdk-dev] [PATCH 12/14] net/bnxt: update cfa truflow resource allocation numbers

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha The truflow session open allocation parameters are updated to support NAT records, L2 context regions, engress encap features. Signed-off-by: Kishore Padmanabha Reviewed-by: Somnath Kotur Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 20 ++

[dpdk-dev] [PATCH 09/14] net/bnxt: change header to SPDX-License

2020-07-17 Thread Somnath Kotur
From: Randy Schacher Update include file header Signed-off-by: Randy Schacher Reviewed-by: Ajit Kumar Khaparde Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/cfa_resource_types.h | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/drivers/net/bnxt/tf

[dpdk-dev] [PATCH 11/14] net/bnxt: default egress rule changes

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha The default egress rule should include buffer descriptor action record only if the VF representor is enabled. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 13 +

[dpdk-dev] [PATCH 07/14] net/bnxt: replace NAT IPv4 action SRC/DEST

2020-07-17 Thread Somnath Kotur
From: Jay Ding Use NAT IPv4 instead of NAT IPv4 SRC and DST. Signed-off-by: Jay Ding Reviewed-by: Kishore Padmanabha Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_core.h | 4 +--- drivers/net/bnxt/tf_core/tf_device_p4.h | 1 - drivers/net/bnxt/tf_core/tf_util.c | 6 +

[dpdk-dev] [PATCH 10/14] net/bnxt: fix dereference of a null pointer

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha Avoid dereferencing a null pointer. Fixes: 313ac35ac701 ("net/bnxt: support ULP session manager init") Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c | 8 +--- 1 file changed,

Re: [dpdk-dev] [PATCH v1] crypto/qat: use better variable names in CRC functions

2020-07-17 Thread Trahe, Fiona
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:35 PM > To: akhil.go...@nxp.com; Doherty, Declan ; Trahe, > Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, Mairtin > ; De Lara Guarch, Pablo > ; Coyle, David > > Subject: [PATCH v1] crypto/qat: use

[dpdk-dev] [PATCH 06/14] net/bnxt: fix em message size

2020-07-17 Thread Somnath Kotur
From: Farah Smith Fix incorrect EM message size fix for insert_em_internal. Fixes: 98487d729b4a ("net/bnxt: cleanup and refactor session management") Signed-off-by: Farah Smith Reviewed-by: Randy Schacher Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_msg.c | 2 +- 1 file chan

[dpdk-dev] [PATCH 08/14] net/bnxt: clean up em message definition

2020-07-17 Thread Somnath Kotur
From: Randy Schacher Remove unused define Signed-off-by: Randy Schacher Reviewed-by: Randy Schacher Reviewed-by: Shahaji Bhosle Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_msg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/bnxt/tf_core/tf_msg.c b/drivers/

[dpdk-dev] [PATCH 05/14] net/bnxt: add protocol header info based on proto field data

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha The protocol header are implicitly matched based on the proto field data. For instance, if ether type is set as 0x800 in the ether header then ipv4 protocol header is assumed to be present for template matching even if ipv4 header is not present in the given flow pattern.

[dpdk-dev] [PATCH 03/14] net/bnxt: enable default flows in truflow mode

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha Removed the check to enable default flows only when VF representor are enabled. It should be enabled all the time in truflow mode. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/bnxt_ethdev.c | 5 ++---

[dpdk-dev] [PATCH 02/14] net/bnxt: initialize table scope rm parameters

2020-07-17 Thread Somnath Kotur
From: Farah Smith Clear out rm_is_allocated parms before calling as base_index was added and is not used in this instance. Signed-off-by: Farah Smith Reviewed-by: Kishore Padmanabha Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_em_common.c | 2 +- 1 file changed, 1 insertion(+

[dpdk-dev] [PATCH 04/14] net/bnxt: fix to avoid accumulation of flow counters

2020-07-17 Thread Somnath Kotur
OVS-DPDK is accumulating the flow counters that are returned as part of the flow_query API and it is being issued at least 3 times every second. So there is no need to accumulate the counts internally in the driver. Fixes: 306c2d28e247 ("net/bnxt: add support for flow query with action_type COUNT

[dpdk-dev] [PATCH v2 00/14] bnxt patches

2020-07-17 Thread Somnath Kotur
From: Venkat Duvvuru Some changes, cleanups/fixes in the TF-ULP layer Farah Smith (2): net/bnxt: initialize table scope rm parameters net/bnxt: fix em message size Jay Ding (1): net/bnxt: replace NAT IPv4 action SRC/DEST Kishore Padmanabha (6): net/bnxt: enable default flows in truflow

[dpdk-dev] [PATCH 01/14] net/bnxt: changes to support new RM and multi-region

2020-07-17 Thread Somnath Kotur
From: Peter Spreadborough - Updated CFA resource types - Updated references - Add High/low Signed-off-by: Peter Spreadborough Reviewed-by: Farah Smith Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/cfa_resource_types.h | 194 drivers/net/bnxt/tf_core/tf_

Re: [dpdk-dev] [PATCH v1 1/2] crypto/qat: improve security instance setup

2020-07-17 Thread Trahe, Fiona
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:36 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De Lara > Guarch, Pablo > ; Trahe, Fiona > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, Mairtin > ; Coyle, David > Subject: [PATCH v1 1/2] crypto/qat: impr

Re: [dpdk-dev] [PATCH v1 2/2] app/crypto-perf: enable security feature for security tests

2020-07-17 Thread De Lara Guarch, Pablo
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:30 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De > Lara Guarch, Pablo ; Trahe, Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, > Mairtin ; Coyle, David > Subject: [PATCH v1 2/2] app/crypto-perf:

Re: [dpdk-dev] [PATCH v1 1/2] test/crypto: enable security feature for security tests

2020-07-17 Thread De Lara Guarch, Pablo
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:30 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De > Lara Guarch, Pablo ; Trahe, Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, > Mairtin ; Coyle, David > Subject: [PATCH v1 1/2] test/crypto: enab

[dpdk-dev] [PATCH 03/14] net/bnxt: enable default flows in truflow mode

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha Removed the check to enable default flows only when VF representor are enabled. It should be enabled all the time in truflow mode. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/bnxt_ethdev.c | 5 ++---

[dpdk-dev] [PATCH 00/14] bnxt patches

2020-07-17 Thread Somnath Kotur
From: Venkat Duvvuru Some changes, cleanups/fixes in the TF-ULP layer Farah Smith (2): net/bnxt: initialize table scope rm parameters net/bnxt: fix em message size Jay Ding (1): net/bnxt: replace NAT IPv4 action SRC/DEST Kishore Padmanabha (6): net/bnxt: enable default flows in truflow

[dpdk-dev] [PATCH 01/14] net/bnxt: changes to support new RM and multi-region

2020-07-17 Thread Somnath Kotur
From: Peter Spreadborough - Updated CFA resource types - Updated references - Add High/low Signed-off-by: Peter Spreadborough Reviewed-by: Farah Smith Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/cfa_resource_types.h | 194 drivers/net/bnxt/tf_core/tf_

[dpdk-dev] [PATCH 02/14] net/bnxt: initialize table scope rm parameters

2020-07-17 Thread Somnath Kotur
From: Farah Smith Clear out rm_is_allocated parms before calling as base_index was added and is not used in this instance. Signed-off-by: Farah Smith Reviewed-by: Kishore Padmanabha Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_em_common.c | 2 +- 1 file changed, 1 insertion(+

Re: [dpdk-dev] [PATCH v1] test/crypto: fix AESNI-MB CPU tests

2020-07-17 Thread De Lara Guarch, Pablo
> -Original Message- > From: Coyle, David > Sent: Thursday, July 16, 2020 4:29 PM > To: akhil.go...@nxp.com; Doherty, Declan ; De > Lara Guarch, Pablo ; Trahe, Fiona > > Cc: dev@dpdk.org; Ryan, Brendan ; O'loingsigh, > Mairtin ; Coyle, David > Subject: [PATCH v1] test/crypto: fix AESN

Re: [dpdk-dev] [PATCH v2] lpm: fix unchecked return value

2020-07-17 Thread Medvedkin, Vladimir
Hi Ruifeng, On 16/07/2020 16:49, Ruifeng Wang wrote: Coverity complains about unchecked return value of rte_rcu_qsbr_dq_enqueue. By default, defer queue size is big enough to hold all tbl8 groups. When enqueue fails, return error to the user to indicate system issue. Coverity issue: 360832 Fixe

Re: [dpdk-dev] [PATCH v3 0/7] net/mlx5: add sys_mem_en devarg

2020-07-17 Thread Raslan Darawsheh
Hi > -Original Message- > From: Suanming Mou > Sent: Friday, July 17, 2020 4:51 PM > To: Slava Ovsiienko ; Matan Azrad > > Cc: Ori Kam ; Raslan Darawsheh > ; dev@dpdk.org > Subject: [PATCH v3 0/7] net/mlx5: add sys_mem_en devarg > > Currently, for MLX5 PMD, once millions of flows create

Re: [dpdk-dev] [PATCH] doc: add more detail to telemetry guides

2020-07-17 Thread Bruce Richardson
On Tue, Jul 14, 2020 at 11:14:15AM +0100, Ciara Power wrote: > This patch adds examples to the Telemetry HowTo guide, to demonstrate > commands that use parameters. The programmer's guide is also modified to > include details on writing a callback function for a new command. > > Signed-off-by: Cia

Re: [dpdk-dev] [PATCH v6 0/8] fib: implement AVX512 vector lookup

2020-07-17 Thread Richardson, Bruce
> -Original Message- > From: Thomas Monjalon > Sent: Wednesday, July 15, 2020 1:45 PM > To: Medvedkin, Vladimir > Cc: Kinsella, Ray ; Stephen Hemminger > ; dev@dpdk.org; david.march...@redhat.com; > jer...@marvell.com; Ananyev, Konstantin ; > Richardson, Bruce ; Mcnamara, John > ; O'Dr

Re: [dpdk-dev] [PATCH v8 3/4] common/qat: use WC store to update queue tail registers

2020-07-17 Thread Trahe, Fiona
> -Original Message- > From: Nicolau, Radu > Sent: Friday, July 17, 2020 11:50 AM > To: dev@dpdk.org > Cc: Xing, Beilei ; Guo, Jia ; > Richardson, Bruce > ; Ananyev, Konstantin > ; > jerinjac...@gmail.com; david.march...@redhat.com; Trahe, Fiona > ; Zhao1, > Wei ; ruifeng.w...@arm.co

Re: [dpdk-dev] [PATCH 0/9] python2 deprecation notice

2020-07-17 Thread Bruce Richardson
On Fri, Jul 17, 2020 at 08:06:23AM -0700, Stephen Hemminger wrote: > On Fri, 17 Jul 2020 13:42:09 +0100 > Bruce Richardson wrote: > > > On Thu, Jul 16, 2020 at 02:01:21PM -0700, Stephen Hemminger wrote: > > > On Thu, 16 Jul 2020 16:44:29 +0200 > > > Robin Jarry wrote: > > > > > > > 2020-07-13

Re: [dpdk-dev] [PATCH v5 2/2] doc: announce deprecation of refcnt atomic member

2020-07-17 Thread Bruce Richardson
On Fri, Jul 17, 2020 at 04:35:56PM +0200, David Marchand wrote: > On Fri, Jul 17, 2020 at 4:32 PM David Marchand > wrote: > > > > On Fri, Jul 17, 2020 at 6:37 AM Phil Yang wrote: > > > > > > refcnt_atomic member in structures rte_mbuf and rte_mbuf_ext_shared_info > > > will be deprecated in 20.11

Re: [dpdk-dev] [PATCH v5 2/2] doc: announce deprecation of refcnt atomic member

2020-07-17 Thread Ananyev, Konstantin
Hi David, > On Fri, Jul 17, 2020 at 4:32 PM David Marchand > wrote: > > > > On Fri, Jul 17, 2020 at 6:37 AM Phil Yang wrote: > > > > > > refcnt_atomic member in structures rte_mbuf and rte_mbuf_ext_shared_info > > > will be deprecated in 20.11 release. > > > > > > Suggested-by: Honnappa Nagarah

Re: [dpdk-dev] [PATCH 3/3] common/mlx5: fix DevX register access opcode

2020-07-17 Thread Thomas Monjalon
17/07/2020 17:23, Slava Ovsiienko: > > > -Original Message- > > From: Thomas Monjalon > > Sent: Friday, July 17, 2020 18:19 > > To: Slava Ovsiienko > > Cc: dev@dpdk.org; Matan Azrad ; Raslan > > Darawsheh > > Subject: Re: [PATCH 3/3] common/mlx5: fix DevX register access opcode > > > >

Re: [dpdk-dev] [PATCH v4 25/27] eal: mark old naming as deprecated

2020-07-17 Thread Burakov, Anatoly
On 17-Jul-20 4:43 PM, Burakov, Anatoly wrote: On 17-Jul-20 4:35 PM, Dmitry Kozliuk wrote: On Fri, Jul 17, 2020, 18:21 Burakov, Anatoly mailto:anatoly.bura...@intel.com>> wrote:     On 16-Jul-20 11:04 PM, Stephen Hemminger wrote: > On Thu, 16 Jul 2020 14:41:41 +0100 > "Burakov, Anatol

Re: [dpdk-dev] [PATCH v4 25/27] eal: mark old naming as deprecated

2020-07-17 Thread Burakov, Anatoly
On 17-Jul-20 4:35 PM, Dmitry Kozliuk wrote: On Fri, Jul 17, 2020, 18:21 Burakov, Anatoly > wrote: On 16-Jul-20 11:04 PM, Stephen Hemminger wrote: > On Thu, 16 Jul 2020 14:41:41 +0100 > "Burakov, Anatoly" mailto:anatoly.bura...@intel.com>> wrote:

Re: [dpdk-dev] [PATCH v4 25/27] eal: mark old naming as deprecated

2020-07-17 Thread Dmitry Kozliuk
On Fri, Jul 17, 2020, 18:21 Burakov, Anatoly wrote: > On 16-Jul-20 11:04 PM, Stephen Hemminger wrote: > > On Thu, 16 Jul 2020 14:41:41 +0100 > > "Burakov, Anatoly" wrote: > > > >> On 15-Jul-20 9:29 PM, Stephen Hemminger wrote: > >>> On Wed, 15 Jul 2020 14:28:17 +0100 > >>> "Burakov, Anatoly" wr

Re: [dpdk-dev] [PATCH 3/3] common/mlx5: fix DevX register access opcode

2020-07-17 Thread Slava Ovsiienko
> -Original Message- > From: Thomas Monjalon > Sent: Friday, July 17, 2020 18:19 > To: Slava Ovsiienko > Cc: dev@dpdk.org; Matan Azrad ; Raslan > Darawsheh > Subject: Re: [PATCH 3/3] common/mlx5: fix DevX register access opcode > > 17/07/2020 17:11, Slava Ovsiienko: > > From: Thomas M

Re: [dpdk-dev] [PATCH v4 25/27] eal: mark old naming as deprecated

2020-07-17 Thread Burakov, Anatoly
On 16-Jul-20 11:04 PM, Stephen Hemminger wrote: On Thu, 16 Jul 2020 14:41:41 +0100 "Burakov, Anatoly" wrote: On 15-Jul-20 9:29 PM, Stephen Hemminger wrote: On Wed, 15 Jul 2020 14:28:17 +0100 "Burakov, Anatoly" wrote: -#define SKIP_MASTERSKIP_INITIAL -#define CALL_MASTERCALL_INIT

Re: [dpdk-dev] Random failure in service_autotest

2020-07-17 Thread David Marchand
On Fri, Jul 17, 2020 at 10:56 AM David Marchand wrote: > > On Wed, Jul 15, 2020 at 12:41 PM Ferruh Yigit wrote: > > > > On 7/15/2020 11:14 AM, David Marchand wrote: > > > Hello Harry and guys who touched the service code recently :-) > > > > > > I spotted a failure for the service UT in Travis: >

Re: [dpdk-dev] [PATCH 3/3] common/mlx5: fix DevX register access opcode

2020-07-17 Thread Thomas Monjalon
17/07/2020 17:11, Slava Ovsiienko: > From: Thomas Monjalon > > 17/07/2020 16:28, Viacheslav Ovsiienko: > > > The dedicated MLX5_CMD_OP_ACCESS_REGISTER_USER opcode must be > > used to > > > read hardware register content from unprotected mode. > > > > Otherwise? What was broken? > > Otherwise the

Re: [dpdk-dev] [PATCH 2/3] net/mlx5: fix compilation issue with atomic128 exchange

2020-07-17 Thread Slava Ovsiienko
> -Original Message- > From: Thomas Monjalon > Sent: Friday, July 17, 2020 18:08 > To: Slava Ovsiienko > Cc: dev@dpdk.org; Matan Azrad ; Raslan > Darawsheh > Subject: Re: [PATCH 2/3] net/mlx5: fix compilation issue with atomic128 > exchange > > 17/07/2020 16:28, Viacheslav Ovsiienko: >

Re: [dpdk-dev] [PATCH 3/3] common/mlx5: fix DevX register access opcode

2020-07-17 Thread Slava Ovsiienko
> -Original Message- > From: Thomas Monjalon > Sent: Friday, July 17, 2020 18:06 > To: Slava Ovsiienko > Cc: dev@dpdk.org; Matan Azrad ; Raslan > Darawsheh > Subject: Re: [PATCH 3/3] common/mlx5: fix DevX register access opcode > > 17/07/2020 16:28, Viacheslav Ovsiienko: > > The dedicat

Re: [dpdk-dev] [PATCH 2/3] net/mlx5: fix compilation issue with atomic128 exchange

2020-07-17 Thread Thomas Monjalon
17/07/2020 16:28, Viacheslav Ovsiienko: > For naw the rte_atomic128_cmp_exchange() is available on x86-64 Typo: now > and ARM64 architectures. The patch fixes the compilation condition > for the code using this atomic transaction. What is fixed exactly? How "not (ppc or 32)" is different of "x86

Re: [dpdk-dev] [PATCH 0/9] python2 deprecation notice

2020-07-17 Thread Stephen Hemminger
On Fri, 17 Jul 2020 13:42:09 +0100 Bruce Richardson wrote: > On Thu, Jul 16, 2020 at 02:01:21PM -0700, Stephen Hemminger wrote: > > On Thu, 16 Jul 2020 16:44:29 +0200 > > Robin Jarry wrote: > > > > > 2020-07-13, Bruce Richardson: > > > > In the absense of a "proper" solution, is the simples

Re: [dpdk-dev] [PATCH 3/3] common/mlx5: fix DevX register access opcode

2020-07-17 Thread Thomas Monjalon
17/07/2020 16:28, Viacheslav Ovsiienko: > The dedicated MLX5_CMD_OP_ACCESS_REGISTER_USER opcode > must be used to read hardware register content from > unprotected mode. Otherwise? What was broken? > > Fixes: 737f44a25d97 ("common/mlx5: add register access DevX routine")

Re: [dpdk-dev] [PATCH] net/mlx5: add queue start and stop feature

2020-07-17 Thread Thomas Monjalon
17/07/2020 16:37, Viacheslav Ovsiienko: > The mlx5 PMD does not support queue_start and queue_stop eth_dev API > routines, queue can't be suspended and resumed during device operation. Please use past tense to describe the status before the patch, and present for the following explanations. > >

Re: [dpdk-dev] [PATCH v3 6/7] app/proc-info: provide way to request info on owned ports

2020-07-17 Thread Thomas Monjalon
15/07/2020 23:22, Stephen Hemminger: > --- a/app/proc-info/Makefile > +++ b/app/proc-info/Makefile > +CFLAGS += -DALLOW_EXPERIMENTAL_API not needed in app/ > +CFLAGS += -O3 > CFLAGS += $(WERROR_FLAGS) > +CFLAGS += -Wno-deprecated-declarations Which deprecated function is used? We must not use d

Re: [dpdk-dev] [PATCH] lib/librte_eal/common/rte_malloc: remove redundant check for size and alignment

2020-07-17 Thread Burakov, Anatoly
On 17-Jul-20 3:40 PM, Burakov, Anatoly wrote: On 15-Jul-20 2:31 PM, Sarosh Arif wrote: Since mallock_socket() always calls malloc_heap_alloc() and this check is present inside malloc_heap_alloc() so there is no need to place it in mallock_socket(). Signed-off-by: Sarosh Arif --- Technically

Re: [dpdk-dev] [PATCH v3] net/i40e: fix fdir cannot receive rx writeback issue

2020-07-17 Thread Xing, Beilei
> -Original Message- > From: Sun, Chenmin > Sent: Friday, July 17, 2020 10:58 PM > To: Zhang, Qi Z ; Xing, Beilei > Cc: dev@dpdk.org; Sun, Chenmin > Subject: [PATCH v3] net/i40e: fix fdir cannot receive rx writeback issue > > From: Chenmin Sun > > This patch fixes the fdir cannot r

Re: [dpdk-dev] [PATCH] lib/librte_eal/common/rte_malloc: remove redundant check for size and alignment

2020-07-17 Thread Burakov, Anatoly
On 15-Jul-20 2:31 PM, Sarosh Arif wrote: Since mallock_socket() always calls malloc_heap_alloc() and this check is present inside malloc_heap_alloc() so there is no need to place it in mallock_socket(). Signed-off-by: Sarosh Arif --- Technically not a bug, so i don't think backporting this is

[dpdk-dev] [PATCH] net/mlx5: add queue start and stop feature

2020-07-17 Thread Viacheslav Ovsiienko
The mlx5 PMD does not support queue_start and queue_stop eth_dev API routines, queue can't be suspended and resumed during device operation. There is the use case when this feature is crucial for applications: - there is the secondary process handling the queue - secondary process crashed/aborted

Re: [dpdk-dev] [PATCH v5 2/2] doc: announce deprecation of refcnt atomic member

2020-07-17 Thread David Marchand
On Fri, Jul 17, 2020 at 4:32 PM David Marchand wrote: > > On Fri, Jul 17, 2020 at 6:37 AM Phil Yang wrote: > > > > refcnt_atomic member in structures rte_mbuf and rte_mbuf_ext_shared_info > > will be deprecated in 20.11 release. > > > > Suggested-by: Honnappa Nagarahalli > > Signed-off-by: Phil

Re: [dpdk-dev] [PATCH 01/14] net/bnxt: changes to support new RM and multi-region

2020-07-17 Thread Thomas Monjalon
17/07/2020 16:14, Somnath Kotur: > From: Peter Spreadborough > > - Updated CFA resource types > - Updated references > - Add High/low > > Signed-off-by: Peter Spreadborough > Reviewed-by: Farah Smith > Signed-off-by: Somnath Kotur Regarding the title, what RM means? Resource Manager? It is n

Re: [dpdk-dev] [PATCH v5 2/2] doc: announce deprecation of refcnt atomic member

2020-07-17 Thread David Marchand
On Fri, Jul 17, 2020 at 6:37 AM Phil Yang wrote: > > refcnt_atomic member in structures rte_mbuf and rte_mbuf_ext_shared_info > will be deprecated in 20.11 release. > > Suggested-by: Honnappa Nagarahalli > Signed-off-by: Phil Yang > Acked-by: Konstantin Ananyev > --- > doc/guides/rel_notes/dep

[dpdk-dev] [PATCH 2/3] net/mlx5: fix compilation issue with atomic128 exchange

2020-07-17 Thread Viacheslav Ovsiienko
For naw the rte_atomic128_cmp_exchange() is available on x86-64 and ARM64 architectures. The patch fixes the compilation condition for the code using this atomic transaction. Fixes: 244788055199 ("net/mlx5: introduce clock queue service routine") Signed-off-by: Viacheslav Ovsiienko --- drivers/

Re: [dpdk-dev] [PATCH 00/14] bnxt patches

2020-07-17 Thread Thomas Monjalon
17/07/2020 16:17, Somnath Kotur: > On Fri, Jul 17, 2020 at 7:46 PM Somnath Kotur > wrote: > > > > From: Venkat Duvvuru > > > > Some changes, cleanups/fixes in the TF-ULP layer > > [...] > Please ignore, missed the 'reply-to' Yes, and you missed the version number and the changelog.

[dpdk-dev] [PATCH 1/3] net/mlx5: fix compilation issue with missing DevX event

2020-07-17 Thread Viacheslav Ovsiienko
If there is no support for DevX event the mlx5_txpp_gather_timestamp() is reported as "not used" causing the compilation warning/error. This patch moves the routine to the conditional block. Fixes: 244788055199 ("net/mlx5: introduce clock queue service routine") Signed-off-by: Viacheslav Ovsiienk

[dpdk-dev] [PATCH 3/3] common/mlx5: fix DevX register access opcode

2020-07-17 Thread Viacheslav Ovsiienko
The dedicated MLX5_CMD_OP_ACCESS_REGISTER_USER opcode must be used to read hardware register content from unprotected mode. Fixes: 737f44a25d97 ("common/mlx5: add register access DevX routine") Signed-off-by: Viacheslav Ovsiienko --- drivers/common/mlx5/mlx5_devx_cmds.c | 3 ++- drivers/common/

Re: [dpdk-dev] [PATCH 1/1] eal/linux: do not create user mem map repeatedly when it exists

2020-07-17 Thread Burakov, Anatoly
On 17-Jul-20 3:19 PM, Burakov, Anatoly wrote: On 16-Jul-20 2:38 PM, wangyunjian wrote: From: Yunjian Wang Currently, we will create new user mem map entry for the same memory segment, but in fact it has already been added to the user mem maps. It's not necessary to create it twice. Fixes: 0cb

Re: [dpdk-dev] [PATCH 1/1] eal/linux: do not create user mem map repeatedly when it exists

2020-07-17 Thread Burakov, Anatoly
On 16-Jul-20 2:38 PM, wangyunjian wrote: From: Yunjian Wang Currently, we will create new user mem map entry for the same memory segment, but in fact it has already been added to the user mem maps. It's not necessary to create it twice. Fixes: 0cbce3a167f1 ("vfio: skip DMA map failure if alrea

Re: [dpdk-dev] [PATCH 00/14] bnxt patches

2020-07-17 Thread Somnath Kotur
On Fri, Jul 17, 2020 at 7:46 PM Somnath Kotur wrote: > > From: Venkat Duvvuru > > Some changes, cleanups/fixes in the TF-ULP layer > > Farah Smith (2): > net/bnxt: initialize table scope rm parameters > net/bnxt: fix em message size > > Jay Ding (1): > net/bnxt: replace NAT IPv4 action SRC/

Re: [dpdk-dev] [PATCH v4 01/27] eal: rename terms used for DPDK lcores

2020-07-17 Thread Burakov, Anatoly
On 01-Jul-20 9:23 PM, Stephen Hemminger wrote: Replace the old use of master/slave lcore with more inclusive name of initial/secondary lcore. The old visible API will stay for now. Change master2slave to new init2worker and vice-versa. This patch breaks the expected practice for new API's. The

Re: [dpdk-dev] [PATCH v3 2/7] app/proc-info: eliminate useless borders

2020-07-17 Thread Thomas Monjalon
15/07/2020 23:22, Stephen Hemminger: > Printing extra borders does not improve readability, and is just > unnecessary. Putting TSC hz in header also makes no sense here. The CPU frequency on headers! OK to remove :)

Re: [dpdk-dev] [PATCH v3 3/7] app/proc-info: hide EAL info messages

2020-07-17 Thread Thomas Monjalon
15/07/2020 23:22, Stephen Hemminger: > The DPDK EAL info messages at the start of a diagnostic application > are not helpful to end user. Suppress them by setting log-level > by default. [...] > + char log_flag[] = "--log-level=6"; Instead of disabling all logs, we could just raise the level o

Re: [dpdk-dev] [PATCH v4 22/27] doc: update references to master/slave lcore in documentation

2020-07-17 Thread Burakov, Anatoly
On 01-Jul-20 9:23 PM, Stephen Hemminger wrote: New terms are initial and worker lcores. Signed-off-by: Stephen Hemminger --- +* **Renamed master lcore to initial lcore.** + + The name given to the first thread in DPDK is changed from master lcore to initial lcore. Removed Items -

Re: [dpdk-dev] [PATCH v10 0/3] generic rte atomic APIs deprecate proposal

2020-07-17 Thread David Marchand
On Fri, Jul 17, 2020 at 12:15 PM Phil Yang wrote: > > DPDK provides generic rte_atomic APIs to do several atomic operations. > These APIs are using the deprecated __sync builtins and enforce full > memory barriers on aarch64. However, full barriers are not necessary > in many use cases. In order t

[dpdk-dev] [PATCH 09/14] net/bnxt: change header to SPDX-License

2020-07-17 Thread Somnath Kotur
From: Randy Schacher Update include file header Signed-off-by: Randy Schacher Reviewed-by: Ajit Kumar Khaparde Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/cfa_resource_types.h | 13 +++-- 1 file changed, 3 insertions(+), 10 deletions(-) diff --git a/drivers/net/bnxt/tf

[dpdk-dev] [PATCH 12/14] net/bnxt: update cfa truflow resource allocation numbers

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha The truflow session open allocation parameters are updated to support NAT records, L2 context regions, engress encap features. Signed-off-by: Kishore Padmanabha Reviewed-by: Somnath Kotur Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 20 ++

[dpdk-dev] [PATCH 10/14] net/bnxt: fix dereference of a null pointer

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha Avoid dereferencing a null pointer. Fixes: 313ac35ac701 ("net/bnxt: support ULP session manager init") Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/ulp_mark_mgr.c | 8 +--- 1 file changed,

[dpdk-dev] [PATCH 14/14] net/bnxt: add debug logs to the TF-ULP layer

2020-07-17 Thread Somnath Kotur
Adding debug logs to the TF-ULP, introducing new debug files for the same Signed-off-by: Somnath Kotur --- drivers/net/bnxt/meson.build |2 + drivers/net/bnxt/tf_ulp/Makefile | 10 + drivers/net/bnxt/tf_ulp/bnxt_tf_common.h |6 + driver

[dpdk-dev] [PATCH 11/14] net/bnxt: default egress rule changes

2020-07-17 Thread Somnath Kotur
From: Kishore Padmanabha The default egress rule should include buffer descriptor action record only if the VF representor is enabled. Signed-off-by: Kishore Padmanabha Reviewed-by: Michael Baucom Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_ulp/bnxt_ulp.c | 13 +

[dpdk-dev] [PATCH 08/14] net/bnxt: clean up em message definition

2020-07-17 Thread Somnath Kotur
From: Randy Schacher Remove unused define Signed-off-by: Randy Schacher Reviewed-by: Randy Schacher Reviewed-by: Shahaji Bhosle Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_msg.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/net/bnxt/tf_core/tf_msg.c b/drivers/

[dpdk-dev] [PATCH 07/14] net/bnxt: replace NAT IPv4 action SRC/DEST

2020-07-17 Thread Somnath Kotur
From: Jay Ding Use NAT IPv4 instead of NAT IPv4 SRC and DST. Signed-off-by: Jay Ding Reviewed-by: Kishore Padmanabha Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_core.h | 4 +--- drivers/net/bnxt/tf_core/tf_device_p4.h | 1 - drivers/net/bnxt/tf_core/tf_util.c | 6 +

[dpdk-dev] [PATCH 04/14] net/bnxt: fix to avoid accumulation of flow counters

2020-07-17 Thread Somnath Kotur
OVS-DPDK is accumulating the flow counters that are returned as part of the flow_query API and it is being issued atleast 3 times every second. So there is no need to accumulate the counts internally in the driver. Fixes: 306c2d28e247 ("net/bnxt: add support for flow query with action_type COUNT"

[dpdk-dev] [PATCH 06/14] net/bnxt: fix em message size

2020-07-17 Thread Somnath Kotur
From: Farah Smith Fix incorrect EM message size fix for insert_em_internal. Fixes: 98487d729b4a ("net/bnxt: cleanup and refactor session management") Signed-off-by: Farah Smith Reviewed-by: Randy Schacher Signed-off-by: Somnath Kotur --- drivers/net/bnxt/tf_core/tf_msg.c | 2 +- 1 file chan

  1   2   3   >