> -Original Message-
> From: Yang, Qiming
> Sent: Thursday, February 4, 2021 11:23 AM
> To: Wang, Haiyue ; dev@dpdk.org
> Cc: Zhang, Qi Z ; Creeley, Brett
>
> Subject: RE: [PATCH v2] net/ice: fix VLAN strip show in double VLAN mode
>
> Acked-by: Qiming Yang
>
> > -Original Messa
Short option "s" was passed to getopt_long function, while there was
no condition on this option.
Fixes: f5188211c7 ("examples/vhost_crypto: add sample application")
Cc: roy.fan.zh...@intel.com
Signed-off-by: Ibtisam Tariq
---
examples/vhost_crypto/main.c | 2 +-
1 file changed, 1 insertion(+),
> -Original Message-
> From: Zhang, Yuying
> Sent: Thursday, February 4, 2021 2:08 PM
> To: dev@dpdk.org; Zhang, Qi Z ; Wang, Haiyue
>
> Cc: Fu, Qi ; Zhang, Yuying
> Subject: [PATCH v1] net/ice: fix QinQ switch rule input set mask
>
> QinQ switch rule doesn't support ethertype field
> -Original Message-
> From: dev On Behalf Of Xing, Beilei
> Sent: Thursday, February 4, 2021 2:58 PM
> To: Li, Xiaoyun ; Wu, Jingjing ;
> dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] net/iavf: fix VLAN insert issue
>
>
>
> > -Original Message-
> > From: Li, Xiaoyun
> > Se
DPDK v20.11 removed udata64 field, and changed type of the dynfield1 to
uint32_t.
Due to define:
lib/common/pg_compat.h:#define udata64 dynfield1[0]
the copy of udata64 was in fact copying only the first 32 bits.
Using udata64 as an address is ok, as it will point to the beginni
On Thu, Feb 4, 2021 at 9:52 AM Michal Krawczyk wrote:
>
> DPDK v20.11 removed udata64 field, and changed type of the dynfield1 to
> uint32_t.
>
> Due to define:
> lib/common/pg_compat.h:#define udata64 dynfield1[0]
> the copy of udata64 was in fact copying only the first 32 bits.
When ASSERT is enabled for compilation, the 2 usages of assert mechanism
in the driver are failed due to typos.
Fix the typos.
Fixes: f8c97babc9f4 ("compress/mlx5: add data-path functions")
Fixes: 37862dafcbed ("compress/mlx5: support 32-bit systems")
Reported-by: Viacheslav Ovsiienko
Signed-of
czw., 4 lut 2021 o 10:01 David Marchand napisał(a):
>
> On Thu, Feb 4, 2021 at 9:52 AM Michal Krawczyk wrote:
> >
> > DPDK v20.11 removed udata64 field, and changed type of the dynfield1 to
> > uint32_t.
> >
> > Due to define:
> > lib/common/pg_compat.h:#define udata64 dynfield1
Hi,
Sorry your questions are quite confused.
Please start explaining what is the problem you are trying to solve.
In general, closing a process does not mean removing the device,
because it can be used by other processes.
04/02/2021 07:56, oulijun:
> Hi, Thomas Monjalon&Ferruh Yigit and others
On 2/4/2021 7:56 AM, Ido Goshen wrote:
-Original Message-
From: Ferruh Yigit
Sent: Thursday, 4 February 2021 2:13
To: Ido Goshen
Cc: dev@dpdk.org
Subject: Re: [PATCH v3 1/1] net/pcap: imissed stats support
On 2/3/2021 11:07 PM, Ido Goshen wrote:
get value from pcap_stats.ps_drop (s
04/02/2021 10:17, Michał Krawczyk:
> czw., 4 lut 2021 o 10:01 David Marchand
> napisał(a):
> >
> > On Thu, Feb 4, 2021 at 9:52 AM Michal Krawczyk wrote:
> > >
> > > DPDK v20.11 removed udata64 field, and changed type of the dynfield1 to
> > > uint32_t.
> > >
> > > Due to define:
> > > lib/comm
The kni linux module is using a custom target for building, which
doesn't take into account any cross compilation arguments. The arguments
in question are ARCH, CROSS_COMPILE (for gcc, clang) and CC, LD (for
clang). Get those from the cross file and pass them to the custom
target.
The user supplie
On Wed, Feb 03, 2021 at 10:18:18PM +0100, Thomas Monjalon wrote:
> AMD CPU can present a high number of NUMA nodes.
> The default should be 32 for better compatibility.
>
> Signed-off-by: Thomas Monjalon
> ---
Seems reasonable.
Acked-by: Bruce Richardson
> -Original Message-
> From: Ferruh Yigit
> Sent: Thursday, 4 February 2021 11:26
> To: Ido Goshen
> Cc: dev@dpdk.org
> Subject: Re: [PATCH v3 1/1] net/pcap: imissed stats support
>
> On 2/4/2021 7:56 AM, Ido Goshen wrote:
> >
> >
> >> -Original Message-
> >> From: Ferruh Yigit
Shared age action query was implemented as part of flow query,
but was not implemented as part of shared action query.
This patch adds the required implementation.
Fixes: 2f622174bf86 ("app/testpmd: support query of age action")
Cc: sta...@dpdk.org
Signed-off-by: Dekel Peled
Acked-by: Matan Azr
remove Bernard Iremonger
remove Wenzhuo Lu
Signed-off-by: Bernard Iremonger
---
MAINTAINERS | 2 --
1 file changed, 2 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 77a2273..4e9fb0a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1581,9 +1581,7 @@ F: app/test/sample_packet_forward.c
Previous patch just removes the last case of direct inclusion of the
generic/ headers.
Signed-off-by: David Marchand
---
buildtools/chkincs/meson.build | 1 -
1 file changed, 1 deletion(-)
diff --git a/buildtools/chkincs/meson.build b/buildtools/chkincs/meson.build
index f345e87551..f28cfd3cd4
There is no need for the direct inclusion of the generic/ header [1]
now that we don't use the rte_atomic API anymore.
1: https://git.dpdk.org/dpdk/commit/?id=3eb860b08eb7
Fixes: e41d27a68df6 ("mbuf: remove atomic reference counters")
Cc: sta...@dpdk.org
Signed-off-by: David Marchand
Reviewed-b
The C version of pmdinfogen was replaced by a Python implementation.
The license exception can be removed.
Fixes: dd2ea14772f5 ("buildtools: remove C implementation of pmdinfogen")
Reported-by: John McNamara
Signed-off-by: Thomas Monjalon
---
license/exceptions.txt | 1 -
1 file changed, 1 del
On Thu, Feb 04, 2021 at 11:10:15AM +0100, Thomas Monjalon wrote:
> The C version of pmdinfogen was replaced by a Python implementation.
> The license exception can be removed.
>
> Fixes: dd2ea14772f5 ("buildtools: remove C implementation of pmdinfogen")
>
> Reported-by: John McNamara
> Signed-of
On Thu, Feb 04, 2021 at 11:05:20AM +0100, David Marchand wrote:
> Previous patch just removes the last case of direct inclusion of the
> generic/ headers.
>
> Signed-off-by: David Marchand
> ---
Acked-by: Bruce Richardson
On 2/4/2021 10:02 AM, Ido Goshen wrote:
-Original Message-
From: Ferruh Yigit
Sent: Thursday, 4 February 2021 11:26
To: Ido Goshen
Cc: dev@dpdk.org
Subject: Re: [PATCH v3 1/1] net/pcap: imissed stats support
On 2/4/2021 7:56 AM, Ido Goshen wrote:
-Original Message-
From:
On 2/4/2021 10:04 AM, Bernard Iremonger wrote:
remove Bernard Iremonger
remove Wenzhuo Lu
Signed-off-by: Bernard Iremonger
---
MAINTAINERS | 2 --
1 file changed, 2 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 77a2273..4e9fb0a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1581
The C based pmdinfogen tool has been replaced by a Python
based tool with a BSD license. As such, we no longer need
to call out a licence exception for pmdinfogen.
Signed-off-by: John McNamara
---
license/exceptions.txt | 1 -
1 file changed, 1 deletion(-)
diff --git a/license/exceptions.txt b/
04/02/2021 07:19, Jerin Jacob:
> On Thu, Feb 4, 2021 at 2:49 AM Thomas Monjalon wrote:
> >
> > AMD CPU can present a high number of NUMA nodes.
> > The default should be 32 for better compatibility.
>
> The typical configuration is 4 nodes[1] for AMD. Just wondering, Is it
> an exception case? if
get value from pcap_stats.ps_drop (see man pcap_stats)
the value is adjusted in this cases:
- port stop - pcap is closed and will lose count
- stats reset - pcap doesn't provide reset api
- rollover - pcap counter size is u_32 only
Signed-off-by: Ido Goshen
---
v4:
* remove volatile
* line spa
> -Original Message-
> From: Thomas Monjalon
> Sent: Thursday, February 4, 2021 10:10 AM
> To: dev@dpdk.org
> Cc: Mcnamara, John ; Neil Horman
> ; Dmitry Kozlyuk
> Subject: [PATCH] license: remove exception for old pmdinfogen
>
> The C version of pmdinfogen was replaced by a Python imple
> -Original Message-
> From: Dmitry Kozlyuk
> Sent: Wednesday, February 3, 2021 7:08 AM
> To: Rong, Leyi
> Cc: david.march...@redhat.com; Zhang, Qi Z ; Yigit,
> Ferruh ; tho...@monjalon.net; Richardson, Bruce
> ; tal...@nvidia.com; Kadam, Pallavi
> ; Menon, Ranjit ; Xing,
> Beilei ; ac
Acked-by: Hemant Agrawal
Hi,
> -Original Message-
> From: Dey, Souvik
> Sent: Tuesday, February 2, 2021 7:49 PM
> To: Raslan Darawsheh ; Slava Ovsiienko
> ; Matan Azrad ; Shahaf
> Shuler
> Cc: dev@dpdk.org; sta...@dpdk.org; Souvik Dey
> Subject: [PATCH v3] common/mlx5: fix storing the synched MAC to internal
>
HI Ferruh,
On 2/2/2021 5:08 PM, Ferruh Yigit wrote:
On 1/20/2021 2:27 PM, Hemant Agrawal wrote:
This patch introduce a new pmd api, which can help the applications
to configure the max framelen for a given dpdmux interface
Signed-off-by: Hemant Agrawal
<...>
+/**
+ * @warning
+ * @b EXPER
On 2/2/2021 5:11 PM, Ferruh Yigit wrote:
On 1/20/2021 2:27 PM, Hemant Agrawal wrote:
From: Gagandeep Singh
Add basic support for scheduling and shaping on dpaa2
platform.
HW supports 2 level of scheduling and shaping.
However the current patch only support single level.
Signed-off-by: Gaga
>-Original Message-
>From: dev On Behalf Of Thomas Monjalon
>Sent: Wednesday, February 3, 2021 11:18 PM
>To: dev@dpdk.org
>Cc: Bruce Richardson
>Subject: [dpdk-dev] [PATCH] config: increase default maximum number of
>NUMA nodes
>
>AMD CPU can present a high number of NUMA nodes.
>The defa
On Fri, Jan 29, 2021 at 12:46 AM Harman Kalra wrote:
>
> On Tue, Jan 26, 2021 at 02:22:43PM +0530, sk...@marvell.com wrote:
> > From: Sunil Kumar Kori
> >
> > Maximum Rx packet length is getting updated twice which
> > corrupts actual value.
> >
> > Fixes: 3151e6a687a3 ("net/octeontx: support MTU
On Thu, Feb 4, 2021 at 3:58 PM Thomas Monjalon wrote:
>
> 04/02/2021 07:19, Jerin Jacob:
> > On Thu, Feb 4, 2021 at 2:49 AM Thomas Monjalon wrote:
> > >
> > > AMD CPU can present a high number of NUMA nodes.
> > > The default should be 32 for better compatibility.
> >
> > The typical configuratio
在 2021/2/4 17:25, Thomas Monjalon 写道:
Hi,
Sorry your questions are quite confused.
Please start explaining what is the problem you are trying to solve.
Start the master and slave processes at the same time, and then run the
kill -9 command to kill the slave processes.
The slave process shoul
The number of descriptors specified for queue creation
implies the queue should be able to contain the specified
amount of packets being sent. Typically one packet takes
one queue descriptor (WQE) to be handled. If there is inline
data option enabled one packet might require more WQEs to
embrace th
Meeting minutes of 4 February 2021
--
Agenda:
* Release Dates
* -rc2 status
* Subtrees
* LTS
* Bugzilla
* Opens
Participants:
* Arm
* Broadcom
* Debian/Microsoft
* Intel
* Marvell
* Nvidia
* NXP
* Red Hat
Release Dates
-
* v21.02 dates
* -rc2 is r
DPDK v20.11 removed mbuf's udata64 field, and changed type of the
dynfield1 to uint32_t.
Due to pktgen's define for DPDK v20.11+:
lib/common/pg_compat.h:#define udata64 dynfield1[0]
udata64 field was being mapped to the first entry of the dynfield1
array.
Because of that, access
On Thu, Feb 04, 2021 at 07:47:01PM +0800, oulijun wrote:
>
>
> 在 2021/2/4 17:25, Thomas Monjalon 写道:
> > Hi,
> >
> > Sorry your questions are quite confused.
> > Please start explaining what is the problem you are trying to solve.
> Start the master and slave processes at the same time, and then
On Wed, Feb 3, 2021 at 11:42 PM Timothy McDaniel
wrote:
>
> This patch updates the PMD to initialize response fields
> prior to calling into the PF layer.
>
> Fixes the following coverity issues:
>
> Fixes: eb14a3421afd ("event/dlb: add eventdev start")
> Coverity issue: 366205
>
> Fixes: f0073621
04/02/2021 12:47, oulijun:
>
> 在 2021/2/4 17:25, Thomas Monjalon 写道:
> > Hi,
> >
> > Sorry your questions are quite confused.
> > Please start explaining what is the problem you are trying to solve.
> Start the master and slave processes at the same time, and then run the
> kill -9 command to kil
On Tue, Jan 26, 2021 at 10:07 PM Mattias Rönnblom
wrote:
>
> Use C11-style GCC built-in functions for atomic operations.
>
> Signed-off-by: Mattias Rönnblom
> ---
> drivers/event/dsw/dsw_evdev.c | 5 +
> drivers/event/dsw/dsw_evdev.h | 6 +++---
> drivers/event/dsw/dsw_event.c | 37
On 2/4/2021 10:41 AM, Zhang, Qi Z wrote:
-Original Message-
From: Dmitry Kozlyuk
Sent: Wednesday, February 3, 2021 7:08 AM
To: Rong, Leyi
Cc: david.march...@redhat.com; Zhang, Qi Z ; Yigit,
Ferruh ; tho...@monjalon.net; Richardson, Bruce
; tal...@nvidia.com; Kadam, Pallavi
; Menon, R
In Single VLAN Mode, single VLAN filters via ICE_SW_LKUP_VLAN are based
on the inner VLAN ID, so the VLAN TPID (i.e. 0x8100 or 0x888a8) doesn't
matter.
In Double VLAN Mode, outer/single VLAN filters via ICE_SW_LKUP_VLAN are
based on the outer/single VLAN ID + VLAN TPID.
For both modes, adding a V
On 2021-02-04 13:34, Jerin Jacob wrote:
> On Tue, Jan 26, 2021 at 10:07 PM Mattias Rönnblom
> wrote:
>> Use C11-style GCC built-in functions for atomic operations.
>>
>> Signed-off-by: Mattias Rönnblom
>> ---
>> drivers/event/dsw/dsw_evdev.c | 5 +
>> drivers/event/dsw/dsw_evdev.h | 6
On 2/2/2021 5:06 PM, Ferruh Yigit wrote:
On 1/20/2021 2:27 PM, Hemant Agrawal wrote:
From: Youri Querry
since for qbman 5.0 generally, pi == ci, no need for extra checks.
They are causing issue.
It would be nice to document the issues, in case users encounter them,
they will know this pat
On Mon, Jan 25, 2021 at 12:07 PM oulijun wrote:
> >> +static void
> >> +signal_handler(int signal)
> >> +{
> >> + if (signal == SIGINT)
> >> + rte_eal_cleanup();
> >> + exit(0);
> >> +}
> >
> > Calling rte_eal_cleanup from a signal handler is a bad idea.
> > In most cases
04/02/2021 13:36, Ferruh Yigit:
> On 2/4/2021 10:41 AM, Zhang, Qi Z wrote:
> > From: Dmitry Kozlyuk
> >> On Tue, 2 Feb 2021 22:32:58 +0800, Leyi Rong wrote:
> >>> Adds extra cflags '-fno-asynchronous-unwind-tables'
> >>> to avoid the MinGW build error:
> >>> Error: invalid register for .seh_sav
In Single VLAN Mode, single VLAN filters via ICE_SW_LKUP_VLAN are based
on the inner VLAN ID, so the VLAN TPID (i.e. 0x8100 or 0x888a8) doesn't
matter.
In Double VLAN Mode, outer/single VLAN filters via ICE_SW_LKUP_VLAN are
based on the outer/single VLAN ID + VLAN TPID.
For both modes, adding a V
> -Original Message-
> From: Ibtisam Tariq
> Sent: Thursday, February 4, 2021 8:06 AM
> To: maxime.coque...@redhat.com; Xia, Chenbo
> Cc: dev@dpdk.org; Ibtisam Tariq ; Zhang, Roy
> Fan
> Subject: [PATCH] examples/vhost_crypto: remove unused short option
>
> Short option "s" was passed t
> -Original Message-
> From: Ibtisam Tariq
> Sent: Thursday, February 4, 2021 8:06 AM
> To: maxime.coque...@redhat.com; Xia, Chenbo
> Cc: dev@dpdk.org; Ibtisam Tariq ; Zhang, Roy
> Fan
> Subject: [PATCH] examples/vhost_crypto: remove unused short option
>
> Short option "s" was passed t
On 2/3/21 12:01 PM, Olivier Matz wrote:
> When doing a mempool dump or an audit, the application can panic because
> the length of the cache is greater than the flush threshold, which is
> seen as a fatal error. But this can temporarily happen when the mempool
> is in use.
>
> Fix the panic condit
When a target compilation is broken, one way to skip the target is to
uninstall the associated toolchain.
But it is not always possible and you end up with hacking the script to
avoid this target until a fix is ready.
It is also often quicker to check a fix on a failing target before
checking comp
On 2/4/21 1:05 PM, David Marchand wrote:
> There is no need for the direct inclusion of the generic/ header [1]
> now that we don't use the rte_atomic API anymore.
>
> 1: https://git.dpdk.org/dpdk/commit/?id=3eb860b08eb7
>
> Fixes: e41d27a68df6 ("mbuf: remove atomic reference counters")
> Cc: sta
On Thu, Feb 4, 2021 at 1:36 PM Ferruh Yigit wrote:
>
> On 2/4/2021 10:41 AM, Zhang, Qi Z wrote:
> >
> >
> >> -Original Message-
> >> From: Dmitry Kozlyuk
> >> Sent: Wednesday, February 3, 2021 7:08 AM
> >> To: Rong, Leyi
> >> Cc: david.march...@redhat.com; Zhang, Qi Z ; Yigit,
> >> Ferru
> -Original Message-
> From: dev On Behalf Of Matan Azrad
> Sent: Thursday, February 4, 2021 11:06
> To: dev@dpdk.org
> Cc: akhil.go...@nxp.com
> Subject: [dpdk-dev] [PATCH] compress/mlx5: fix assert compilation
>
> When ASSERT is enabled for compilation, the 2 usages of assert mechanism
04/02/2021 14:34, David Marchand:
> When a target compilation is broken, one way to skip the target is to
> uninstall the associated toolchain.
> But it is not always possible and you end up with hacking the script to
> avoid this target until a fix is ready.
>
> It is also often quicker to check
> -Original Message-
> From: Wang, Haiyue
> Sent: Thursday, February 4, 2021 8:50 PM
> To: dev@dpdk.org
> Cc: Yang, Qiming ; Zhang, Qi Z
> ; Wang, Haiyue ; Creeley,
> Brett
> Subject: [PATCH v2] net/ice: fix VLAN 0 adding based on VLAN mode
>
> In Single VLAN Mode, single VLAN filters
Hi,
I'm sorry for asking the questions very late.
Is still this patch set actual and should it be updated and considered?
As I can understand this one optimizes the memory writes in some way using the
instructions with the hints.
Is this specific for some CPU families? Is this more common? I su
>-Original Message-
>From: Andrew Rybchenko
>Sent: Monday, February 1, 2021 4:39 PM
>To: Xueming(Steven) Li
>Cc: dev@dpdk.org; Slava Ovsiienko ; Asaf Penso
>
>Subject: Re: [dpdk-dev] [PATCH v5 0/9] ethdev: support SubFunction representor
>
>On 1/28/21 5:31 PM, Xueming(Steven) Li wrote:
In cryptography, a block cipher is a deterministic algorithm operating
on fixed-length groups of bits, called blocks.
A block cipher consists of two paired algorithms, one for encryption
and the other for decryption. Both algorithms accept two inputs:
an input block of size n bits and a key of siz
Hi,
> -Original Message-
> From: Jiawei Wang
> Sent: Wednesday, February 3, 2021 10:29 AM
> To: Slava Ovsiienko ; Matan Azrad
> ; Ori Kam ; NBU-Contact-Thomas
> Monjalon
> Cc: dev@dpdk.org; Raslan Darawsheh ;
> sta...@dpdk.org
> Subject: [PATCH] net/mlx5: fix E-Switch egress mirror flow
On Thu, Feb 4, 2021 at 2:59 PM Thomas Monjalon wrote:
>
> 04/02/2021 14:34, David Marchand:
> > When a target compilation is broken, one way to skip the target is to
> > uninstall the associated toolchain.
> > But it is not always possible and you end up with hacking the script to
> > avoid this t
There is an 82599 errata that UDP frames with a zero checksum are
incorrectly marked as checksum invalid by the hardware. This was
leading to misleading PKT_RX_L4_CKSUM_BAD flag.
This patch changes the bad UDP checksum to PKT_RX_L4_CKSUM_UNKNOWN,
so the software application will then have to reco
On 2/3/2021 12:23 PM, Lijun Ou wrote:
Based on some suggestions, we reorganize the fixes
and resend the only fixes.
Chengchang Tang (4):
net/hns3: fix stats flip overflow
net/hns3: replace all atomic type with C11 atomic builtins
net/hns3: fix cmdq cleared during firmware process
net
> -Original Message-
> From: Yigit, Ferruh
> Sent: Wednesday 3 February 2021 17:30
> To: Ferriter, Cian
> Cc: Yigit, Ferruh ; dev@dpdk.org; sta...@dpdk.org
> Subject: [PATCH] net/pcap: fix byte stats for drop Tx
>
> Drop Tx path in pcap is Tx that just drops the packets, which is used
04/02/2021 15:48, David Marchand:
> On Thu, Feb 4, 2021 at 2:59 PM Thomas Monjalon wrote:
> >
> > 04/02/2021 14:34, David Marchand:
> > > When a target compilation is broken, one way to skip the target is to
> > > uninstall the associated toolchain.
> > > But it is not always possible and you end
Hi Ferruh,
This fixes the issue I was seeing. Now an error is reported, rather than silent
failure.
I have one piece of feedback about the particular error message below inline
which you can take or leave, I'm happy for you to upstream this fix either way.
Acked-by: Cian Ferriter
> -Orig
CQE compression must be enabled by default unless the rxq_cqe_comp_en
devarg is set to 0 or FW cannot support CQE compression for some reason.
The latest commit that introduced checks for FW capabilities for new
CQE zipping formats disables the CQE compression unless the devarg is 1.
Revert this lo
> -Original Message-
> From: Alexander Kozyrev
> Sent: Thursday, February 4, 2021 18:07
> To: dev@dpdk.org
> Cc: Raslan Darawsheh ; ferruh.yi...@intel.com; Slava
> Ovsiienko
> Subject: [PATCH] net/mlx5: fix default CQE compression config
>
> CQE compression must be enabled by default unl
> -Original Message-
> From: Ferriter, Cian
> Sent: Thursday 4 February 2021 15:48
> To: Yigit, Ferruh
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: RE: [PATCH] net/pcap: fix byte stats for drop Tx
>
>
>
> > -Original Message-
> > From: Yigit, Ferruh
> > Sent: Wednesday 3 F
On 2/4/2021 4:03 PM, Ferriter, Cian wrote:
Hi Ferruh,
This fixes the issue I was seeing. Now an error is reported, rather than silent
failure.
I have one piece of feedback about the particular error message below inline
which you can take or leave, I'm happy for you to upstream this fix eithe
On 2/4/2021 10:04 AM, Dekel Peled wrote:
Shared age action query was implemented as part of flow query,
but was not implemented as part of shared action query.
This patch adds the required implementation.
Fixes: 2f622174bf86 ("app/testpmd: support query of age action")
Cc: sta...@dpdk.org
Sign
Hi,
> -Original Message-
> From: dev On Behalf Of Dekel Peled
> Sent: Wednesday, February 3, 2021 12:09 PM
> To: Matan Azrad ; Shahaf Shuler
> ; Slava Ovsiienko
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: [dpdk-dev] [PATCH] net/mlx5: fix check of shared RSS capability
>
> Existing co
On 2/4/2021 3:47 PM, Ferriter, Cian wrote:
-Original Message-
From: Yigit, Ferruh
Sent: Wednesday 3 February 2021 17:30
To: Ferriter, Cian
Cc: Yigit, Ferruh ; dev@dpdk.org; sta...@dpdk.org
Subject: [PATCH] net/pcap: fix byte stats for drop Tx
Drop Tx path in pcap is Tx that just dro
Hi,
> -Original Message-
> From: dev On Behalf Of Dekel Peled
> Sent: Wednesday, February 3, 2021 12:09 PM
> To: Matan Azrad ; Shahaf Shuler
> ; Slava Ovsiienko
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: [dpdk-dev] [PATCH] net/mlx5: fix validation of hash Rx queue
> pointer
>
> Imple
Hi,
> -Original Message-
> From: Alexander Kozyrev
> Sent: Thursday, February 4, 2021 6:07 PM
> To: dev@dpdk.org
> Cc: Raslan Darawsheh ; ferruh.yi...@intel.com; Slava
> Ovsiienko
> Subject: [PATCH] net/mlx5: fix default CQE compression config
>
> CQE compression must be enabled by defa
Packet forwarding is not working when infinite Rx feature is used with
large .pcap files that has high number of packets.
The problem is number of allocated mbufs are less than the infinite Rx
ring size, and all mbufs consumed to fill the ring, so there is no mbuf
left for forwarding.
Current log
Hi,
> -Original Message-
> From: Viacheslav Ovsiienko
> Sent: Thursday, February 4, 2021 2:04 PM
> To: dev@dpdk.org
> Cc: Raslan Darawsheh ; Matan Azrad
> ; Ori Kam ; NBU-Contact-Thomas
> Monjalon ; sta...@dpdk.org
> Subject: [PATCH] net/mlx5: fix Tx queue size created with DevX
>
> The
> -Original Message-
> From: Ferruh Yigit
> Sent: Thursday 4 February 2021 16:29
> To: Ferriter, Cian
> Cc: dev@dpdk.org; sta...@dpdk.org
> Subject: Re: [PATCH] net/pcap: fix infinite Rx with large files
>
> On 2/4/2021 4:03 PM, Ferriter, Cian wrote:
> > Hi Ferruh,
> >
> > This fixes t
The new error message looks great.
As I've already given my ack, I'm happy for this to be applied.
> -Original Message-
> From: Yigit, Ferruh
> Sent: Thursday 4 February 2021 16:51
> To: Ferriter, Cian
> Cc: Yigit, Ferruh ; dev@dpdk.org; sta...@dpdk.org
> Subject: [PATCH v2] net/pcap: f
In http://doc.dpdk.org/guides/nics/overview.html, table 1.1 lists all
supported features.
It has a single line for "Flow API" that refers to rte_flow support.
rte_flow is composed of many items and actions that are not expressed in
this single line.
The following new tables are suggested:
1. rte_f
On 2/4/2021 5:02 PM, Ferriter, Cian wrote:
The new error message looks great.
As I've already given my ack, I'm happy for this to be applied.
-Original Message-
From: Yigit, Ferruh
Sent: Thursday 4 February 2021 16:51
To: Ferriter, Cian
Cc: Yigit, Ferruh ; dev@dpdk.org; sta...@dpdk.o
On 2/4/2021 4:50 PM, Raslan Darawsheh wrote:
Hi,
-Original Message-
From: Alexander Kozyrev
Sent: Thursday, February 4, 2021 6:07 PM
To: dev@dpdk.org
Cc: Raslan Darawsheh ; ferruh.yi...@intel.com; Slava
Ovsiienko
Subject: [PATCH] net/mlx5: fix default CQE compression config
CQE compr
On Thu, Feb 04, 2021 at 10:51:41AM +0100, Juraj Linkeš wrote:
> The kni linux module is using a custom target for building, which
> doesn't take into account any cross compilation arguments. The arguments
> in question are ARCH, CROSS_COMPILE (for gcc, clang) and CC, LD (for
> clang). Get those fro
> > When ASSERT is enabled for compilation, the 2 usages of assert mechanism
> > in the driver are failed due to typos.
> >
> > Fix the typos.
> >
> > Fixes: f8c97babc9f4 ("compress/mlx5: add data-path functions")
> > Fixes: 37862dafcbed ("compress/mlx5: support 32-bit systems")
> >
> > Reported-by
> Some platforms may not support operations on encrypted digest,
> So in this patch adding a check for such test cases.
>
> Signed-off-by: Gagandeep Singh
> ---
Acked-by: Akhil Goyal
Applied to dpdk-next-crypto
Thanks.
On Mon, Feb 1, 2021 at 6:57 PM David Marchand wrote:
>
> On Mon, Feb 1, 2021 at 2:20 PM Harry van Haaren
> wrote:
> >
> > The PMD info get API has a void return type. Remove the
> > @return 0 Success doxygen comment as it doesn't make sense here.
>
> Fixes: 5223a1f3b8de ("eventdev: define southbo
The following changes since commit f6f2d22401531ba06f5a1c967853453b0cbb8a9d:
devtools: fix examples build test (2021-02-02 23:58:22 +0100)
are available in the Git repository at:
http://dpdk.org/git/next/dpdk-next-eventdev
for you to fetch changes up to b982f0ef3f58e8381fa473c2636ea4725b505
> Subject: [PATCH v2 0/3] Update crypto perf script and doc
>
> This patchset contains some fixes for the crypto perf graphing script.
>
> A release note is added, and some small fixes for the script documentation
> are also included.
> The script config handling is modified to handle EAL and app
On 2/4/2021 10:33 AM, Ido Goshen wrote:
get value from pcap_stats.ps_drop (see man pcap_stats)
the value is adjusted in this cases:
- port stop - pcap is closed and will lose count
- stats reset - pcap doesn't provide reset api
- rollover - pcap counter size is u_32 only
Signed-off-by: Ido
This patch series reorganizes the main datastructure for each
queue, struct ionic_qcq. Its constituent struct ionic_queue and
struct ionic_cq are stripped down first. Then the generic struct
ionic_qcq is stripped down, and a unique struct is created for
each queue type.
The adminq code is consolid
Add Q_NEXT_TO_POST() and Q_NEXT_TO_SRVC() macros.
Use a precomputed size mask.
This will conserve resources.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev.c | 14 +++---
drivers/net/ionic/ionic_dev.h | 14 +++---
drivers/net/ionic/ionic_lif.c | 5 +++--
drivers
The adminq is the only caller of ionic_q_service(), so absorb it
into ionic_adminq_service().
Move all of the adminq code together into ionic_main.c.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic.h | 13 +
drivers/net/ionic/ionic_dev.c | 85 -
Drop the callback part of the object and store only the pointers.
This saves a bit of space and simplifies the code.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev.c | 8 ++--
drivers/net/ionic/ionic_dev.h | 21 +--
drivers/net/ionic/ionic_main.c | 8
This will conserve resources.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev.c | 6 --
drivers/net/ionic/ionic_dev.h | 2 --
drivers/net/ionic/ionic_lif.c | 1 -
drivers/net/ionic/ionic_rxtx.c | 9 +
4 files changed, 5 insertions(+), 13 deletions(-)
diff --git a/driv
This will conserve resources.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 7 ---
drivers/net/ionic/ionic_lif.h | 1 -
2 files changed, 8 deletions(-)
diff --git a/drivers/net/ionic/ionic_lif.c b/drivers/net/ionic/ionic_lif.c
index 1ca6e050ad..462a526935 100644
--- a/driv
This will conserve resources.
Rename ionic_qcq_alloc() arg from 'base' to 'type_name' for clarity.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_dev.c | 14 +++---
drivers/net/ionic/ionic_dev.h | 34 ++-
drivers/net/ionic/ionic_lif.c | 49 +++
Create a unique Q-CQ struct for adminq, notifyq, rxq, and txq to
reduce the size of each object.
Minimize the size of each field to squeeze into as few cachelines
as possible.
Signed-off-by: Andrew Boyer
---
drivers/net/ionic/ionic_lif.c | 153 ++---
drivers/net/ion
1 - 100 of 134 matches
Mail list logo
