The configuration of FDIR flex mask and flex pit should not be set
during flow validate. It should be set when flow create.
Fixes: 6ced3dd72f5f ("net/i40e: support flexible payload parsing for FDIR")
Cc: sta...@dpdk.org
Signed-off-by: Chenxu Di
Tested-by: Zhou, Jun
Acked-by: Jeff Guo
---
v4:
-
> From: Stephen Hemminger [mailto:step...@networkplumber.org]
> Sent: Monday, November 9, 2020 7:05 PM
>
> On Mon, 9 Nov 2020 11:21:02 +0100
> Morten Brørup wrote:
>
> > > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce
> Richardson
> > > Sent: Monday, November 9, 2020 11:06 AM
> > >
> -Original Message-
> From: Chenxu Di
> Sent: Tuesday, November 10, 2020 1:43 PM
> To: dev@dpdk.org
> Cc: Xing, Beilei ; Guo, Jia ; Wang,
> Haiyue ; Zhou, JunX W ;
> Di, ChenxuX ; sta...@dpdk.org
> Subject: [PATCH v3] net/i40e: fix incorrect FDIR flex configuration
>
> The configuration
The return value of rte_mempool_default_cache should be
checked as it can be NULL.
Fixes: 9ab9514c150e ("net/iavf: enable AVX512 for Tx")
Reported-by: Konstantin Ananyev
Signed-off-by: Wenzhuo Lu
---
drivers/net/iavf/iavf_rxtx_vec_avx512.c | 8 +++-
1 file changed, 7 insertions(+), 1 delet
The return value of rte_mempool_default_cache should be
checked as it can be NULL.
Fixes: a4e480de268e ("net/ice: optimize Tx by using AVX512")
Reported-by: Konstantin Ananyev
Signed-off-by: Wenzhuo Lu
---
drivers/net/ice/ice_rxtx_vec_avx512.c | 8 +++-
1 file changed, 7 insertions(+), 1 d
Hello David,
IMHO, it cannot be moved to read_uint16 parser.
If we do, we can't verify that the user input value is greater than
UINT16 MAX or not on the overflow data.
> > + if (data_room_size == 0 ||
> > + data_room_size > UINT16_MAX) {
The configuration of FDIR flex mask and flex pit should not be set
during flow validate. It should be set when flow create.
Fixes: 6ced3dd72f5f ("net/i40e: support flexible payload parsing for FDIR")
Cc: sta...@dpdk.org
Signed-off-by: Chenxu Di
Tested-by: Zhou, Jun
---
v3:
-rebased the patch, f
> -Original Message-
> From: Steve Yang
> Sent: Monday, November 9, 2020 2:52 PM
> To: dev@dpdk.org
> Cc: Yang, Qiming ; Xing, Beilei
> ; Wu, Jingjing ; Yang, SteveX
>
> Subject: [PATCH v2 1/2] net/iavf: fix reset VF doesn't notify PF to reset
>
> When VF closed, the VF should notify
As shared RSS action will be shared by multiple flows, the action
is created as global standalone action and managed only by the
relevant shared action management functions.
Currently, hrxqs will be created by shared RSS action or general
queue action. For hrxqs created by shared RSS action, they
Hi,
The replies are inline.
> -Original Message-
> From: David Marchand
> Sent: Monday, November 9, 2020 8:40 PM
> To: Maxime Coquelin ; Jiang, Cheng1
>
> Cc: Xia, Chenbo ; dev ; Fu, Patrick
> ; Yang, YvonneX ;
> Thomas Monjalon ; Yigit, Ferruh
>
> Subject: Re: [dpdk-dev] [PATCH v10 0/4
Tested-by: Xie,WeiX < weix@intel.com>
Regards,
Xie Wei
> -Original Message-
> From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Simei Su
> Sent: Tuesday, November 10, 2020 10:02 AM
> To: Zhang, Qi Z
> Cc: dev@dpdk.org; Wang, Haiyue ; Ding, Xuan
> ; Su, Simei
> Subject: [dpdk-de
> -Original Message-
> From: Su, Simei
> Sent: Tuesday, November 10, 2020 10:02 AM
> To: Zhang, Qi Z
> Cc: dev@dpdk.org; Wang, Haiyue ; Ding, Xuan
> ; Su, Simei
> Subject: [PATCH] net/ice: fix core dump when device reset
>
> When device resets, it should not implememt ACL initializat
> -Original Message-
> From: Guo, Jia
> Sent: Monday, November 9, 2020 2:05 PM
> To: Wu, Jingjing ; Zhang, Qi Z ;
> Xing, Beilei
> Cc: dev@dpdk.org; Guo, Jia
> Subject: [PATCH v1] net/iavf: fix rxdid configure
>
> When configure rx queue by virtchnnl, the rxdid should be configured o
> -Original Message-
> From: Xu, Ting
> Sent: Tuesday, November 10, 2020 9:56 AM
> To: dev@dpdk.org
> Cc: Zhang, Qi Z ; Xing, Beilei ;
> Wu, Jingjing ; Xu, Ting ;
> sta...@dpdk.org
> Subject: [PATCH v2] doc: fix missing release note for large VF
>
> Update release note for large VF, suppo
From: Yi Yang
Many NICs can't offload VXLAN UFO, so it is very important
to do VXLAN UDP GSO by software to improve VM-to-VM UDP
performance, especially for the case that VM MTU is just
1500 but not 9000.
With this enabled in DPDK, OVS DPDK can leverage it to
improve VM-to-VM UDP performance, pe
When device resets, it should not implememt ACL initialization
which is only executed in DCF. This patch disable ACL init and
uninit when in DPDK PF only mode.
Fixes: 40d466fa9f76 ("net/ice: support ACL filter in DCF")
Signed-off-by: Simei Su
---
drivers/net/ice/ice_acl_filter.c | 12 --
On Tuesday, November 10, 2020 3:19 AM, Ferruh Yigit wrote:
> On 11/9/2020 7:43 AM, Jiawen Wu wrote:
> > Remove direct use of compiler attribute.
> >
> > Signed-off-by: Jiawen Wu
> > ---
> > drivers/net/txgbe/base/txgbe_dummy.h | 2 +-
> > drivers/net/txgbe/base/txgbe_osdep.h | 6 ++
> > 2
Update release note for large VF, supporting up to 256 queue pairs per
VF.
Fixes: e436cd43835b ("net/iavf: negotiate large VF and request more queues")
Cc: sta...@dpdk.org
Signed-off-by: Ting Xu
---
doc/guides/rel_notes/release_20_11.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/doc/g
> -Original Message-
> From: Ferruh Yigit
> Sent: Tuesday, November 10, 2020 2:17 AM
> To: Suanming Mou ; Slava Ovsiienko
> ; Matan Azrad
> Cc: Raslan Darawsheh ; dev@dpdk.org
> Subject: Re: [dpdk-dev] [PATCH] net/mlx5: fix shared RSS action release
>
> On 11/6/2020 3:08 AM, Suanming M
> -Original Message-
> From: Maxime Coquelin
> Sent: Monday, November 9, 2020 8:16 PM
> To: dev@dpdk.org; Ding, Xuan ;
> step...@networkplumber.org; tho...@monjalon.net; sta...@dpdk.org; Xia,
> Chenbo
> Cc: Maxime Coquelin
> Subject: [PATCH v3 2/3] vhost: fix fd leak in dirty logging set
Replace the maintainers for OcteonTx2 DMA and EP drivers.
Signed-off-by: Radha Mohan Chintakuntla
---
MAINTAINERS | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index a720cf672e..214515060a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1258,1
> > > Change formatting so that it's more consistent and readable,
> > > add/modify comments/stdout messages, move configuration options to
> > > more appropriate places and make the order consistent according to
> > > these
> > rules:
> > > 1. First list generic configuration options, then list
> >
> > >
> > > Add Arm SoC configuration to Arm meson.build and add a meson option
> > > to enable those options for native builds. This is preferable to
> > > specifying a cross file when doing aarch64 -> aarch64 builds, since
> > > the cross file specifies the toolchain as well.
> > >
> > > Si
> >
> > >
> > > Some Arm SoCs are not NUMA systems. Add the capability to disable
> > > NUMA for cross build and disabled NUMA in Arm cross files.
> > >
> > > Signed-off-by: Juraj Linkeš
> > > ---
> > > config/arm/arm64_armada_linux_gcc| 1 +
> > > config/arm/arm64_armv8_linux_gcc | 1
> >
> > >
> > > A few options that disabled drivers in the old makefiles were
> > > improperly ported to the meson build system. Fix this by adding a to
> > > the list of disabled
> > The fixes need to be separated into a different commit as they need to
> > be backported.
> > It would be good to
> >
> > >
> > > Use dictionary lookup instead of checking for existing variables,
> > > iterating over all elements in the list or checking lists for
> > > optional configuration. Move variable contents into the dictionary
> > > for variables that would be referenced only once.
> > > Fallback to
If xmedata mode 1 enabled and create a flow with RSS and mark action,
there was an error that rdma-core failed to create RQT due to wrong
queue definition. This was due to mixed flow creation in thread specific
flow workspace.
This patch introduces nested flow workspace(context data), each flow
us
> > >
> > > 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 compatibi
>
> Add an option to automatically discover the host's numa and cpu counts and
> use those values for a non cross-build.
> Give users the option to override the per-arch default values or values from
> cross files by specifying them on the command line with -Dmax_lcores and -
> Dmax_numa_nodes.
> >
> > >
> > > [2339/2407] Compiling C object app/test/dpdk-
> > > test.p/test_ring_mt_peek_stress_zc.c.o
> > > FAILED: app/test/dpdk-test.p/test_ring_mt_peek_stress_zc.c.o
> > > gcc -Iapp/test/dpdk-test.p -Iapp/test -I../app/test -Ilib/librte_acl
> > > - I../lib/librte_acl -I. -I.. -Iconfig -I.
According to the Technical Board decision
(http://mails.dpdk.org/archives/dev/2020-November/191859.html),
the mempool pointer in the mbuf struct is moved
from the second to the first half.
It may increase performance in some cases
on systems having 64-byte cache line, i.e. mbuf split in two cache l
The drivers for OCTEON TX2 are not supported in 32-bit mode.
Suggested-by: Jerin Jacob
Signed-off-by: Thomas Monjalon
---
drivers/common/octeontx2/meson.build | 18 ++
drivers/crypto/octeontx2/meson.build | 17 +++--
drivers/event/octeontx2/meson.build | 18 +
After some discussions, it seems the best option is still to use
the remaining space in the first half to move the pool pointer.
The fields move has an impact on octeontx2 in 32-bit build.
v2: disable octeontx2 drivers in 32-bit build.
Thomas Monjalon (2):
drivers: disable OCTEON TX2 in 32-bit
09/11/2020 21:35, David Christensen:
> The SPAPR IOMMU requires that a DMA window size be defined before memory
> can be mapped for DMA. Current code dynamically modifies the DMA window
> size in response to every new memory allocation which is potentially
> dangerous because all existing mappings
09/11/2020 18:44, Thomas Monjalon:
> 09/11/2020 18:20, Thomas Monjalon:
> > 09/11/2020 18:15, Ferruh Yigit:
> > > On 11/9/2020 5:01 PM, Thomas Monjalon wrote:
> > > > 09/11/2020 17:48, Ferruh Yigit:
> > > >> I can send a new version to reset 'CUSTOM_PKG_CONFIG_PATH',
> > > >> what is the intention/
PKG_CONFIG_PATH is specific to each target, so it must be empty
before configuring each build from the file according to DPDK_TARGET.
Inheriting a default PKG_CONFIG_PATH for all targets does not make sense
and is prone to confusion.
DPDK_MESON_OPTIONS might take a global initial value from enviro
The SPAPR v2 IOMMU used on bare-metal PowerNV systems requires that a DMA
window be defined before mapping/unmapping memory. The current VFIO code
dynamically resizes this DMA window every time a new memory request is
made, which requires that all existing memory be unmapped/remapped.
While this s
The SPAPR IOMMU requires that a DMA window size be defined before memory
can be mapped for DMA. Current code dynamically modifies the DMA window
size in response to every new memory allocation which is potentially
dangerous because all existing mappings need to be unmapped/remapped in
order to resi
09/11/2020 19:02, Bruce Richardson:
> On Mon, Nov 09, 2020 at 06:09:51PM +0100, Thomas Monjalon wrote:
> > 27/10/2020 18:38, Bruce Richardson:
> > > To test the installation process of DPDK using "ninja install"
> > > test-meson-builds.sh builds a subset of the examples using "make". To
> > > allo
Hi Akhil,
Heads up on this doc update still to be applied.
> -Original Message-
> From: Tom Rix
> Sent: Tuesday, November 3, 2020 12:00 PM
> To: Chautru, Nicolas ; dev@dpdk.org;
> akhil.go...@nxp.com; tho...@monjalon.net
> Cc: Mcnamara, John ;
> david.march...@redhat.com; Richardson, Br
On 11/3/2020 10:07 AM, Jiawen Wu wrote:
Add the remaining part of txgbe PMD.
Support include flow API, traffic manager, macsec and ipsec.
Jiawen Wu (37):
net/txgbe: add ntuple filter init and uninit
net/txgbe: support ntuple filter add and delete
net/txgbe: add ntuple parse rule
net/
On 11/9/2020 7:43 AM, Jiawen Wu wrote:
Remove direct use of compiler attribute.
Signed-off-by: Jiawen Wu
---
drivers/net/txgbe/base/txgbe_dummy.h | 2 +-
drivers/net/txgbe/base/txgbe_osdep.h | 6 ++
2 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/drivers/net/txgbe/base/tx
On 11/9/2020 6:37 PM, Ferruh Yigit wrote:
On 11/6/2020 2:47 PM, Maxime Coquelin wrote:
This patches fixes virtqueue initialization issue causing
segfault or file descriptor being closed unexpectedly.
The wrong index was passed to init_vring_queue() by
alloc_vring_queue() when a hole in the virt
On 11/9/20 7:37 PM, Ferruh Yigit wrote:
> On 11/6/2020 2:47 PM, Maxime Coquelin wrote:
>> This patches fixes virtqueue initialization issue causing
>> segfault or file descriptor being closed unexpectedly.
>>
>> The wrong index was passed to init_vring_queue() by
>> alloc_vring_queue() when a ho
On 11/9/2020 5:09 AM, Jiang, YuX wrote:
Tested-by: JiangYuX
Please don't top post.
Also tag should be: "Tested-by: Yu Jiang "
-Original Message-
From: Maxime Coquelin [mailto:maxime.coque...@redhat.com]
Sent: Friday, November 6, 2020 10:48 PM
To: dev@dpdk.org; Xia, Chenbo ;
amore
On 11/6/2020 2:47 PM, Maxime Coquelin wrote:
This patches fixes virtqueue initialization issue causing
segfault or file descriptor being closed unexpectedly.
The wrong index was passed to init_vring_queue() by
alloc_vring_queue() when a hole in the virtqueue array was
met.
Fixes: 8acd7c213353 (
On 11/6/2020 3:08 AM, Suanming Mou wrote:
As shared RSS action is global standalone action, the shared action
should be maintained only by the shared action management functions.
Currently, the shared RSS action hrxq uses the same fate action type
with existed general queues, and implicitly uses
On Mon, 9 Nov 2020 11:21:02 +0100
Morten Brørup wrote:
> > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Bruce Richardson
> > Sent: Monday, November 9, 2020 11:06 AM
> >
> > On Mon, Nov 09, 2020 at 09:16:27AM +0100, Morten Brørup wrote:
> > > +CC techboard
> > >
> > > > From: Jerin Ja
On Mon, Nov 09, 2020 at 06:09:51PM +0100, Thomas Monjalon wrote:
> 27/10/2020 18:38, Bruce Richardson:
> > To test the installation process of DPDK using "ninja install"
> > test-meson-builds.sh builds a subset of the examples using "make". To allow
> > more flexibility for people testing, allow th
>
> 'uint64_t' is used to hold the pointer, for 32-bits build this
> assumption is wrong and giving following build error:
>
> rte_eth_af_xdp.c: In function ‘xdp_umem_configure’:
> rte_eth_af_xdp.c:970:15:
> error: cast to pointer from integer of different size
>[-Werror=int-to-po
09/11/2020 18:20, Thomas Monjalon:
> 09/11/2020 18:15, Ferruh Yigit:
> > On 11/9/2020 5:01 PM, Thomas Monjalon wrote:
> > > 09/11/2020 17:48, Ferruh Yigit:
> > >> I can send a new version to reset 'CUSTOM_PKG_CONFIG_PATH',
> > >> what is the intention/plan with the config file?
> > >
> > > You sho
On 11/9/2020 9:49 AM, Zhang, Qi Z wrote:
-Original Message-
From: dev On Behalf Of Guo, Jia
Sent: Monday, November 9, 2020 2:24 PM
To: Prateek Agarwal
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH] Made i40 header CPP compatible using extern
"C". Library headers work directly in cp
> From: Honnappa Nagarahalli
> Sent: Monday 9 November 2020 17:24
> To: Walsh, Conor ; Ananyev, Konstantin
>
> Cc: dev@dpdk.org; Chen, LingliX ; nd
> ; Honnappa Nagarahalli ;
> nd
> Subject: RE: [PATCH v1] app/test: zcd needs to be initialised to fix gcc
> compile
> error
>
>
>
> > > >
> > >
> > >
> > > Hi Honnappa,
> > >
> > > Sorry about the delayed reply.
> > No problem, thank you.
> >
> > > The build error trace is below, the build command used was: CC=gcc
> > > meson
> > --
> > > optimization=1 --werror build1; ninja -C build1 I am not sure how I
> > > could fix this issue witho
09/11/2020 18:15, Ferruh Yigit:
> On 11/9/2020 5:01 PM, Thomas Monjalon wrote:
> > 09/11/2020 17:48, Ferruh Yigit:
> >> On 11/9/2020 4:19 PM, Bruce Richardson wrote:
> >>> On Mon, Nov 09, 2020 at 05:14:24PM +0100, Thomas Monjalon wrote:
> 09/11/2020 16:44, Bruce Richardson:
> > On Mon, Nov
On 11/9/2020 5:01 PM, Thomas Monjalon wrote:
09/11/2020 17:48, Ferruh Yigit:
On 11/9/2020 4:19 PM, Bruce Richardson wrote:
On Mon, Nov 09, 2020 at 05:14:24PM +0100, Thomas Monjalon wrote:
09/11/2020 16:44, Bruce Richardson:
On Mon, Nov 09, 2020 at 04:28:16PM +0100, Thomas Monjalon wrote:
09/
27/10/2020 18:38, Bruce Richardson:
> To test the installation process of DPDK using "ninja install"
> test-meson-builds.sh builds a subset of the examples using "make". To allow
> more flexibility for people testing, allow the set of examples chosen for
> this make test to be overridden using vari
Hi Honnappa,
> From: Honnappa Nagarahalli
> Sent: Monday 9 November 2020 16:45
> To: Walsh, Conor ; Ananyev, Konstantin
>
> Cc: dev@dpdk.org; Chen, LingliX ; nd
> ; Honnappa Nagarahalli ;
> nd
> Subject: RE: [PATCH v1] app/test: zcd needs to be initialised to fix gcc
> compile
> error
>
>
>
09/11/2020 17:48, Ferruh Yigit:
> On 11/9/2020 4:19 PM, Bruce Richardson wrote:
> > On Mon, Nov 09, 2020 at 05:14:24PM +0100, Thomas Monjalon wrote:
> >> 09/11/2020 16:44, Bruce Richardson:
> >>> On Mon, Nov 09, 2020 at 04:28:16PM +0100, Thomas Monjalon wrote:
> 09/11/2020 15:55, Ferruh Yigit:
On 11/9/2020 4:19 PM, Bruce Richardson wrote:
On Mon, Nov 09, 2020 at 05:14:24PM +0100, Thomas Monjalon wrote:
09/11/2020 16:44, Bruce Richardson:
On Mon, Nov 09, 2020 at 04:28:16PM +0100, Thomas Monjalon wrote:
09/11/2020 15:55, Ferruh Yigit:
If the 'PKG_CONFIG_PATH' is set in the environmen
>
> Hi Honnappa,
>
> Sorry about the delayed reply.
No problem, thank you.
> The build error trace is below, the build command used was: CC=gcc meson --
> optimization=1 --werror build1; ninja -C build1 I am not sure how I could fix
> this issue without initialising zcd.
>
> [2339/2407] Compi
On 11/3/20 6:35 AM, Patrick Fu wrote:
> Async inflight packet counter should take failed packets into account.
> Failed packets will be deducted in the error handling logic.
>
> Fixes: 6b3c81db8bb7 ("vhost: simplify async copy completion")
> Fixes: cd6760da1076 ("vhost: introduce async enqueue
On 11/2/20 12:38 PM, Ibtisam Tariq wrote:
> Add new line character(\n) in the usage of vhost_crypto example for
> better readability
>
> Fixes: 709521f4c2c ("examples/vhost_crypto: support multi-core")
> Cc: roy.fan.zh...@intel.com
>
> Signed-off-by: Ibtisam Tariq
> ---
> examples/vhost_cryp
On 11/6/20 3:47 PM, Maxime Coquelin wrote:
> This patches fixes virtqueue initialization issue causing
> segfault or file descriptor being closed unexpectedly.
>
> The wrong index was passed to init_vring_queue() by
> alloc_vring_queue() when a hole in the virtqueue array was
> met.
>
> Fixes:
On Mon, 9 Nov 2020 16:10:45 +
Bruce Richardson wrote:
> On Mon, Nov 09, 2020 at 05:07:16PM +0100, David Marchand wrote:
> > On Mon, Nov 9, 2020 at 5:06 PM Bruce Richardson
> > wrote:
> > >
> > > On Mon, Nov 09, 2020 at 05:03:06PM +0100, David Marchand wrote:
> > > > On Mon, Nov 9, 2020 a
Hi Honnappa,
Sorry about the delayed reply.
The build error trace is below, the build command used was: CC=gcc meson
--optimization=1 --werror build1; ninja -C build1
I am not sure how I could fix this issue without initialising zcd.
[2339/2407] Compiling C object
app/test/dpdk-test.p/test_ring
On Mon, Nov 09, 2020 at 05:14:24PM +0100, Thomas Monjalon wrote:
> 09/11/2020 16:44, Bruce Richardson:
> > On Mon, Nov 09, 2020 at 04:28:16PM +0100, Thomas Monjalon wrote:
> > > 09/11/2020 15:55, Ferruh Yigit:
> > > > If the 'PKG_CONFIG_PATH' is set in the environment before script run,
> > > > 32
09/11/2020 16:44, Bruce Richardson:
> On Mon, Nov 09, 2020 at 04:28:16PM +0100, Thomas Monjalon wrote:
> > 09/11/2020 15:55, Ferruh Yigit:
> > > If the 'PKG_CONFIG_PATH' is set in the environment before script run,
> > > 32 bit still uses that value for 64-bits libraries.
> > >
> > > From the buil
On Mon, Nov 09, 2020 at 05:07:16PM +0100, David Marchand wrote:
> On Mon, Nov 9, 2020 at 5:06 PM Bruce Richardson
> wrote:
> >
> > On Mon, Nov 09, 2020 at 05:03:06PM +0100, David Marchand wrote:
> > > On Mon, Nov 9, 2020 at 4:55 PM Stephen Hemminger
> > > wrote:
> > > > > This patch causes build
On Mon, Nov 9, 2020 at 5:06 PM Bruce Richardson
wrote:
>
> On Mon, Nov 09, 2020 at 05:03:06PM +0100, David Marchand wrote:
> > On Mon, Nov 9, 2020 at 4:55 PM Stephen Hemminger
> > wrote:
> > > > This patch causes build failures which persist until patch 3, because of
> > > > these deprecations. I
On Mon, Nov 09, 2020 at 05:03:06PM +0100, David Marchand wrote:
> On Mon, Nov 9, 2020 at 4:55 PM Stephen Hemminger
> wrote:
> > > This patch causes build failures which persist until patch 3, because of
> > > these deprecations. I think you need to define the backward-compatibility
> > > macros he
On Mon, Nov 09, 2020 at 07:54:40AM -0800, Stephen Hemminger wrote:
> On Mon, 9 Nov 2020 13:49:31 +
> Bruce Richardson wrote:
>
> > On Thu, Nov 05, 2020 at 02:35:57PM -0800, Stephen Hemminger wrote:
> > > This patch renames the enum values in the EAL include files.
> > > As a backward compatib
On Mon, Nov 9, 2020 at 4:55 PM Stephen Hemminger
wrote:
> > This patch causes build failures which persist until patch 3, because of
> > these deprecations. I think you need to define the backward-compatibility
> > macros here, but only mark them deprecated at the end of the set.
> >
> > /Bruce
>
On Mon, 9 Nov 2020 13:49:31 +
Bruce Richardson wrote:
> On Thu, Nov 05, 2020 at 02:35:57PM -0800, Stephen Hemminger wrote:
> > This patch renames the enum values in the EAL include files.
> > As a backward compatible temporary migration tool, define
> > a replacement mapping for old values.
>
Any EAL memory allocation often goes through eal_get_virtual_area()
function, which will print a warning whenever the resulting allocation
didn't match the specified address requirements. This is useful for
when we have requested a specific base virtual address, to let the user
know that the mappin
On Mon, Nov 09, 2020 at 04:28:16PM +0100, Thomas Monjalon wrote:
> 09/11/2020 15:55, Ferruh Yigit:
> > If the 'PKG_CONFIG_PATH' is set in the environment before script run,
> > 32 bit still uses that value for 64-bits libraries.
> >
> > From the build log:
> >
> > "
> > Using DPDK_TARGET i386-pc-
Hi,
> -Original Message-
> From: Viacheslav Ovsiienko
> Sent: Friday, November 6, 2020 6:58 PM
> To: dev@dpdk.org
> Cc: Raslan Darawsheh ; Alexander Kozyrev
> ; Matan Azrad ;
> sta...@dpdk.org
> Subject: [PATCH] net/mlx5: fix Tx queue completions on stop
>
> The Tx queue completion produ
09/11/2020 15:55, Ferruh Yigit:
> If the 'PKG_CONFIG_PATH' is set in the environment before script run,
> 32 bit still uses that value for 64-bits libraries.
>
> From the build log:
>
> "
> Using DPDK_TARGET i386-pc-linux-gnu
> meson -Dexamples=l3fwd --buildtype=debugoptimized --werror
> -Dc_arg
Any EAL memory allocation often goes through eal_get_virtual_area()
function, which will print a warning whenever the resulting allocation
didn't match the specified address requirements. This is useful for
when we have requested a specific base virtual address, to let the user
know that the mappin
If the 'PKG_CONFIG_PATH' is set in the environment before script run,
32 bit still uses that value for 64-bits libraries.
>From the build log:
"
Using DPDK_TARGET i386-pc-linux-gnu
meson -Dexamples=l3fwd --buildtype=debugoptimized --werror
-Dc_args=-m32 -Dc_link_args=-m32
/tmp/dpdk_maintain/self
On Mon, Nov 9, 2020 at 8:12 PM Thomas Monjalon wrote:
>
> 09/11/2020 15:08, Jerin Jacob:
> > On Mon, Nov 9, 2020 at 7:32 PM Thomas Monjalon wrote:
> > > 09/11/2020 14:35, Jerin Jacob:
> > > > Are you facing the issue with 32bit? Could you share the steps to
> > > > reproduce and gcc version?
> >
09/11/2020 15:08, Jerin Jacob:
> On Mon, Nov 9, 2020 at 7:32 PM Thomas Monjalon wrote:
> > 09/11/2020 14:35, Jerin Jacob:
> > > Are you facing the issue with 32bit? Could you share the steps to
> > > reproduce and gcc version?
> >
> > Oh you're right, the issue was with 32-bit build,
>
> Thanks
>
Hi,
> -Original Message-
> From: Viacheslav Ovsiienko
> Sent: Friday, November 6, 2020 7:16 PM
> To: dev@dpdk.org
> Cc: Raslan Darawsheh ; Alexander Kozyrev
> ; Matan Azrad ;
> sta...@dpdk.org
> Subject: [PATCH] net/mlx5: fix Rx queues completion index consistency
>
> The Rx queue comple
From: Hongbo Zheng
In current driver print log function, some print format
symbols does not match with the actual variable types.
Signed-off-by: Hongbo Zheng
Signed-off-by: Lijun Ou
---
drivers/net/hns3/hns3_dcb.c | 10 +-
drivers/net/hns3/hns3_ethdev.c| 32 -
According to the rule of the static check tools
that arrange blank lines properly to keep the
code compact, here remove some unncessary blank
line to fix the above rule warning.
Signed-off-by: Lijun Ou
---
drivers/net/hns3/hns3_cmd.c| 1 -
drivers/net/hns3/hns3_ethdev.c | 6 +-
drivers
From: Chengchang Tang
FLR operation will reset the queue enabling state and
the driver needs to restore the state after reset. As
a result, it will result in unpredictable behavior with
reseted when user start or stop queue by calling the
relatived function if the driver does not do this.
This p
From: Hongbo Zheng
According to the rule of the used static check tool,
each member is initialized on a separate lines when
struct and union members are initialized, here is
tempting to adjust some code lines in order to remove
the warning.
Signed-off-by: Hongbo Zheng
Signed-off-by: Lijun Ou
-
Here series include four coverity tool warnings fixes
and a bug fixes.
V2->V3:
1. fix some checkpatch warnings for patch[3]
2. split patch[3/6] into two independent patchse[3/6]
and patch[6/6] according to Ferruh Yigit's advice
3. rewrite commit log for patch[5/5]
4. rewrite the patch title for
From: Hongbo Zheng
According to bit operator reliability style, variables in
the right expression participating int bit operation
must be an unsigned type.
Signed-off-by: Hongbo Zheng
Signed-off-by: Lijun Ou
---
V2->V3:
- fix commit log for correct descriptions
- fix unnecessary variable decla
Here adds a check for the return value when calling
rte_pci_write_config.
Coverity issue: 363714
Fixes: cea37e513329 ("net/hns3: fix FLR reset")
Cc: sta...@dpdk.org
Signed-off-by: Lijun Ou
---
V2->V3:
- drop the un-related change
V1->V2:
- no fix
---
drivers/net/hns3/hns3_ethdev_vf.c | 8 +
在 2020/11/7 0:57, Ferruh Yigit 写道:
On 11/6/2020 3:51 AM, Lijun Ou wrote:
From: Chengchang Tang
The FLR will resets the queue enabling status. In the
current code, the queue enabling status is not restored
after the reset. Therefore, if upper layer users have
called queue start/stop function
Unlike TCP, UDP checksums are optional and may be zero to indicate "not
set" [RFC 768] (except for IPv6, where this prohibited [RFC 8200]). Add
this special case to the checksum offload emulation in net/tap.
Signed-off-by: Michael Pfeiffer
---
drivers/net/tap/rte_eth_tap.c | 13 +++--
1
On Mon, Nov 9, 2020 at 7:32 PM Thomas Monjalon wrote:
>
> 09/11/2020 14:35, Jerin Jacob:
> > On Mon, Nov 9, 2020 at 6:29 PM Thomas Monjalon wrote:
> > >
> > > 09/11/2020 13:01, Jerin Jacob:
> > > > Hi @Thomas Monjalon
> > > >
> > > > Any specific reason why you removed the static assert from octe
09/11/2020 14:35, Jerin Jacob:
> On Mon, Nov 9, 2020 at 6:29 PM Thomas Monjalon wrote:
> >
> > 09/11/2020 13:01, Jerin Jacob:
> > > Hi @Thomas Monjalon
> > >
> > > Any specific reason why you removed the static assert from octeontx2.
> >
> > I have a build failure when cross-compiling for octeontx
09/11/2020 14:35, Bruce Richardson:
> On Mon, Nov 09, 2020 at 01:24:28PM +, Ferruh Yigit wrote:
> > On 11/9/2020 1:19 PM, Thomas Monjalon wrote:
> > > 09/11/2020 14:00, Ferruh Yigit:
> > > > 32 bit still uses the pkgconfig file for 64-bits libraries, from the
> > > > build log:
> > > >
> > > >
On Thu, Nov 05, 2020 at 02:35:57PM -0800, Stephen Hemminger wrote:
> This patch renames the enum values in the EAL include files.
> As a backward compatible temporary migration tool, define
> a replacement mapping for old values.
>
> The old names relating to blacklist and whitelist are replaced
>
在 2020/11/7 0:45, Ferruh Yigit 写道:
On 11/6/2020 3:51 AM, Lijun Ou wrote:
Here adds a check for the return value when calling
rte_pci_write_config.
Coverity issue: 363714
Fixes: cea37e513329 ("net/hns3: fix FLR reset")
Cc: sta...@dpdk.org
Signed-off-by: Lijun Ou
---
V1->V2:
- rte_pci_wirte_
Device name parsing is done on all buses during device iterations at
either EAL or ethdev levels.
When a bus implements device name parsing slowly, all iterations are
impacted. Efficient implementation is important.
The DPAA bus device name parsing has two issues: it allocates dynamic
memory and
On 09/11/20 12:58 +0100, David Marchand wrote:
> On Mon, Nov 9, 2020 at 12:32 PM Gaetan Rivet wrote:
> > + delta = 0;
> > + if (strncmp(name, "dpaa_bus:", 9) == 0) {
> > + delta = 9;
> > + } else if (strncmp(name, "name=", 5) == 0) {
> > + delta = 5;
>
On Mon, Nov 9, 2020 at 6:29 PM Thomas Monjalon wrote:
>
> 09/11/2020 13:01, Jerin Jacob:
> > Hi @Thomas Monjalon
> >
> > Any specific reason why you removed the static assert from octeontx2.
>
> I have a build failure when cross-compiling for octeontx2.
I am trying the below command, I am not abl
1 - 100 of 167 matches
Mail list logo
