Re: [dpdk-dev] [PATCH v3] vhost: read last used index once

Hi Ling, I will take a look. Thanks for reporting. Regards, Balazs On Sat, May 8, 2021 at 10:44 AM Ling, WeiX wrote: > Hi Balazs Nemeth, > > Your patch cause a bug about virtio-user can't recieve packets when use > packed ring and ring size is 255. > > Please have a look about it. > > https:/

Re: [dpdk-dev] : nvgre inner rss problem in mlx5

Hello Wenxu, Can you please create a new BZ ticket? Looks like this is not handled properly in our pmd and we’ll handle it and update. Regards, Asaf Penso From: wenxu Sent: Monday, May 10, 2021 7:54 AM To: Asaf Penso Cc: dev@dpdk.org Subject: Re:: [dpdk-dev] nvgre inner rss problem in mlx5

Re: [dpdk-dev] [PATCH v3 17/22] net/ena: support SMP for mz alloc counter

On Sun, May 09, 2021 at 02:41:18PM +0100, Ferruh Yigit wrote: > On 5/7/2021 6:18 PM, Stanislaw Kardach wrote: > > On Fri, May 07, 2021 at 05:48:50PM +0100, Ferruh Yigit wrote: > >> On 5/6/2021 3:25 PM, Michal Krawczyk wrote: > >>> From: Stanislaw Kardach > >>> > >>> Introduce a memory area for ENA

Re: [dpdk-dev] [PATCH v3 18/22] net/ena: move default RSS key to shared mem

On Thu, May 06, 2021 at 04:25:22PM +0200, Michal Krawczyk wrote: > From: Stanislaw Kardach > > Make sure the same random generated RSS key is available to all ENA PMD > instances in all processes. > > Signed-off-by: Stanislaw Kardach > Reviewed-by: Michal Krawczyk > Reviewed-by: Igor Chauskin

[dpdk-dev] [PATCH] net/af_xdp: fix build warning for kernels < 5.4

Prior to this change, two implementations of rx_syscall_handler existed although only one was needed (for the zero copy path which is only available from kernel 5.4 and onwards). Remove the second definition from compat.h and move the first definition back to where it is called in the Rx function.

Re: [dpdk-dev] [EXT] [PATCH v3 01/15] drivers: introduce mlx5 crypto PMD

On Sun, May 09, 2021 at 09:32:45AM +, Akhil Goyal wrote: > > > > + > > > > +fmt_name = 'mlx5_crypto' > > > > +deps += ['common_mlx5', 'eal', 'cryptodev'] > > > > > > I don't see a reason for eal dependency. Do you really need it? > > > > See RTE_LOG_REGISTER. > > Compilation works fine even a

Re: [dpdk-dev] [PATCH] raw/ioat: fix parameter shadow warning

On Fri, May 07, 2021 at 05:20:25PM +, Kevin Laatz wrote: > In the function __idxd_completed_ops() we have a parameter shadow warning > due to a local variable having the same name as one of the function > parameters. This is fixed by simply renaming the local variable. > > Fixes: 245efe544d8e

[dpdk-dev] [PATCH] common/cpt: add checks for offset overflow

Add checks to catch overflow of any offsets. Offset control word specifies, 1. 16 bits encryption offset 2. 8 bits IV offset 3. 8 bits auth offset Signed-off-by: Anoob Joseph --- drivers/common/cpt/cpt_ucode.h | 59 ++ 1 file changed, 59 insertions(+) di

Re: [dpdk-dev] [EXT] Re: [PATCH 2/2] eal: fix side effects in ptr align macros

>On Sun, 9 May 2021 22:48:02 +0530 > wrote: > >> From: Pavan Nikhilesh >> >> Avoid expanding parameters inside RTE_*_ALIGN macros. >> Update common_autotest to detect macro side effects. >> Workaround static arrays relying on RTE_ALIGN macros. >> >> Fixes: af75078fece3 ("first public release") >>

[dpdk-dev] [dpdk-dev v2] crypto/snow3g: add support for digest appended ops

This patch enable out-of-place auth-cipher operations where digest should be encrypted among with the rest of raw data. It also adds support for partially encrypted digest when using auth-cipher operations. Fixes: 7c87e2d7b359 ("crypto/snow3g: use IPsec library") Cc: pablo.de.lara.gua...@intel.com

Re: [dpdk-dev] [PATCH] raw/ioat: fix resource leak for directory handle

06/05/2021 13:19, Bruce Richardson: > When reading the /dev directory as part of the bus scan for DSA devices, > the directory handle from opendir was not freed on function return, > leading to a resource leak. > > Coverity issue: 370588 > Fixes: b7aaf417f936 ("raw/ioat: add bus driver for device

Re: [dpdk-dev] [PATCH] raw/ioat: fix bus requiring virtual addressing when no devs

06/05/2021 17:09, Bruce Richardson: > +static unsigned int devcount = 0; ERROR:INITIALISED_STATIC: do not initialise statics to 0 I will fix (remove = 0) if you agree.

Re: [dpdk-dev] [PATCH] raw/ioat: fix bus requiring virtual addressing when no devs

06/05/2021 17:34, Bruce Richardson: > On Thu, May 06, 2021 at 04:27:03PM +0100, Burakov, Anatoly wrote: > > On 06-May-21 4:09 PM, Bruce Richardson wrote: > > > If after a bus scan, there are no devices using a particular bus, then > > > that bus should not be taken into account when deciding whethe

[dpdk-dev] [PATCH] crypto/qat: fix NULL authentication request

This patch fixes the NULL auth generation case where the request shouldn't contain the authentication result address. Allows to run ipsec_autotest with a QAT device. Fixes: 65beb9abca6d ("crypto/qat: fix null auth when using VFIO") Signed-off-by: Adam Dybkowski --- drivers/crypto/qat/qat_sym.c

Re: [dpdk-dev] [PATCH] raw/ioat: fix bus requiring virtual addressing when no devs

On Mon, May 10, 2021 at 12:13:43PM +0200, Thomas Monjalon wrote: > 06/05/2021 17:09, Bruce Richardson: > > +static unsigned int devcount = 0; > > ERROR:INITIALISED_STATIC: do not initialise statics to 0 > > I will fix (remove = 0) if you agree. > Sure. Thanks.

Re: [dpdk-dev] [PATCH] raw/ioat: fix bus requiring virtual addressing when no devs

07/05/2021 14:30, Walsh, Conor: > > > > > Reported-by: Harry van Haaren > > > > Signed-off-by: Bruce Richardson > > > > --- > > > > > > Tested-by: Anatoly Burakov > > > > Tested-by: Harry van Haaren > > I was having an issue while running l3fwd after rc2 where I had to force > IOVA mode to P

Re: [dpdk-dev] [PATCH] raw/ioat: fix parameter shadow warning

> On Fri, May 07, 2021 at 05:20:25PM +, Kevin Laatz wrote: > > In the function __idxd_completed_ops() we have a parameter shadow > warning > > due to a local variable having the same name as one of the function > > parameters. This is fixed by simply renaming the local variable. > > > > Fixes

Re: [dpdk-dev] [PATCH] raw/ioat: fix bus requiring virtual addressing when no devs

On Mon, May 10, 2021 at 12:18:50PM +0200, Thomas Monjalon wrote: > 06/05/2021 17:34, Bruce Richardson: > > On Thu, May 06, 2021 at 04:27:03PM +0100, Burakov, Anatoly wrote: > > > On 06-May-21 4:09 PM, Bruce Richardson wrote: > > > > If after a bus scan, there are no devices using a particular bus,

Re: [dpdk-dev] [PATCH 2/2] net: fix header include order for FreeBSD

Hello Ben, On Fri, May 7, 2021 at 9:13 PM Ben Pfaff wrote: > > > > It would be interesting to know exact issue this solves for OVS. > > > > Referenced commit only says FreeBSD "insists" on this include order, > > > > but DPDK and standalone files with these includes build either way. > > > > > >

Re: [dpdk-dev] [PATCH] crypto/qat: fix NULL authentication request

> -Original Message- > From: Dybkowski, AdamX > Sent: Monday, May 10, 2021 11:20 AM > To: dev@dpdk.org; Zhang, Roy Fan ; Kusztal, > ArkadiuszX ; gak...@marvell.com > Cc: Dybkowski, AdamX > Subject: [PATCH] crypto/qat: fix NULL authentication request > > This patch fixes the NULL auth gen

Re: [dpdk-dev] [PATCH v3 00/22] net/ena: update ENA PMD to v2.3.0

pt., 7 maj 2021 o 16:59 Ferruh Yigit napisał(a): > > On 5/6/2021 3:25 PM, Michal Krawczyk wrote: > > Hi, > > > > this version updates the driver to version 2.3.0, which fixes multiple > > bugs, contains part of the work on making the ENA PMD fully MP aware, and > > also updates HAL to the latest v

Re: [dpdk-dev] [PATCH] raw/ioat: fix bus requiring virtual addressing when no devs

10/05/2021 13:20, Bruce Richardson: > On Mon, May 10, 2021 at 12:18:50PM +0200, Thomas Monjalon wrote: > > 06/05/2021 17:34, Bruce Richardson: > > > On Thu, May 06, 2021 at 04:27:03PM +0100, Burakov, Anatoly wrote: > > > > On 06-May-21 4:09 PM, Bruce Richardson wrote: > > > > > If after a bus scan,

[dpdk-dev] [PATCH 0/2] bugfix for mlx4 and mlx5

This patch set contains two bugfixes for mlx4 and mlx5. Chengwen Feng (2): net/mlx4: fix memory barrier incorrectly placed net/mlx5: fix memory barrier incorrectly placed drivers/net/mlx4/mlx4_mp.c | 2 +- drivers/net/mlx5/linux/mlx5_mp_os.c | 2 +- 2 files changed, 2 insertions(+),

[dpdk-dev] [PATCH 2/2] net/mlx5: fix memory barrier incorrectly placed

The memory barrier is used to ensure that the response is returned only after the Tx/Rx function is set, it should place after the Rx/Tx function is set. Fixes: 2aac5b5d119f ("net/mlx5: sync stop/start with secondary process") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- drivers/net/mlx

[dpdk-dev] [PATCH 1/2] net/mlx4: fix memory barrier incorrectly placed

The memory barrier is used to ensure that the response is returned only after the Tx/Rx function is set, it should place after the Rx/Tx function is set. Fixes: 0203d33a1059 ("net/mlx4: support secondary process") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- drivers/net/mlx4/mlx4_mp.c |

Re: [dpdk-dev] [PATCH 0/2] bugfix for mlx4 and mlx5

10/05/2021 14:06, Chengwen Feng: > This patch set contains two bugfixes for mlx4 and mlx5. > > Chengwen Feng (2): > net/mlx4: fix memory barrier incorrectly placed > net/mlx5: fix memory barrier incorrectly placed Please could you use the option "--cc-cmd devtools/get-maintainer.sh" when send

[dpdk-dev] [PATCH 1/3] net/i40e: fix return status for unsupported VF message

This patch modifies the return status for unsupported VF messages, in order to make it the same as the return status of the kernel driver. Fixes: 4861cde46116 ("i40e: new poll mode driver") Cc: sta...@dpdk.org Signed-off-by: Alvin Zhang --- drivers/net/i40e/i40e_pf.c | 4 ++-- 1 file changed, 2

[dpdk-dev] [PATCH] net/iavf: fix error logs

Remove some useless logs which are duplicated of the lower-level function logs and may confuse users when running VF without some features supportted by PF. Fixes: 5a038d19962d ("net/iavf: fix RSS configuration on i40e VF") Fixes: 95f2f0e9fc2a ("net/iavf: improve default RSS") Signed-off-by: Alvi

[dpdk-dev] [PATCH 2/3] common/iavf: fix V-channel status

Add VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED to enum virtchnl_status_code. Fixes: e5b2a9e957e7 ("net/avf/base: add base code for avf PMD") Cc: sta...@dpdk.org Signed-off-by: Alvin Zhang --- drivers/common/iavf/virtchnl.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/common/iavf/virtchn

Re: [dpdk-dev] [PATCH] net/mlx5/linux: fix missing firmware version copying

Hi, > -Original Message- > From: dev On Behalf Of Kamil Vojanec > Sent: Friday, February 5, 2021 11:01 AM > To: dev@dpdk.org > Cc: vikto...@cesnet.cz; Matan Azrad ; Shahaf Shuler > ; Slava Ovsiienko > Subject: [dpdk-dev] [PATCH] net/mlx5/linux: fix missing firmware version > copying > >

[dpdk-dev] [PATCH 3/3] net/iavf: fix V-channel status

Add support of VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED for facilitating compatibility with PF. Chang the error log-level from ERROR to WARNING when a V-channel message is not supported by PF, because the VF may still be able to run without certain features which not supported by PF. Fixes: 0c35eecfe8

Re: [dpdk-dev] [PATCH v3 00/22] net/ena: update ENA PMD to v2.3.0

On 5/10/2021 1:02 PM, Michał Krawczyk wrote: > pt., 7 maj 2021 o 16:59 Ferruh Yigit napisał(a): >> >> On 5/6/2021 3:25 PM, Michal Krawczyk wrote: >>> Hi, >>> >>> this version updates the driver to version 2.3.0, which fixes multiple >>> bugs, contains part of the work on making the ENA PMD fully M

Re: [dpdk-dev] [PATCH v4] net/mlx5: fix tunnel offload private items location

Hello Ferruh, The patch has failed ci/Intel-compilation test. (http://mails.dpdk.org/archives/test-report/2021-May/193327.html) I've checked that with Thomas. He confirmed that was a known fault and should not affect the current patch. Can we proceed with the patch integration ? Thank you. Rega

Re: [dpdk-dev] [PATCH v3] eal: fix memory mapping for 32-bit targets

+Cc Anatoly 08/05/2021 16:27, Lance Richardson: > For 32-bit targets, size_t is normally a 32-bit type and > does not have sufficient range to represent 64-bit offsets > that can are needed when mapping PCI addresses. Use uint64_t > instead. > > Found when attempting to run 32-bit Linux dpdk-test

[dpdk-dev] [PATCH v2] raw/ioat: fix parameter shadow warning

In the function __idxd_completed_ops() we have a parameter shadow warning due to a local variable having the same name as one of the function parameters. This issue is fixed by simply renaming the local variable. This warning was discovered during an OVS build with DPDK 21.05-rc2. The OVS build pa

Re: [dpdk-dev] [PATCH] config/arm: add ability to express arch extensions

05/05/2021 14:14, pbhagavat...@marvell.com: > From: Pavan Nikhilesh > > The ARM architecture allows SoCs to have extensions in addition > to base profiles such as Large System Extension (LSE), CRC etc. > > Add ability to declare SoC specific extensions. > > Signed-off-by: Pavan Nikhilesh I do

[dpdk-dev] [PATCH v2] net/mlx5: fix LAG representor probe on PF1 PCI

In case of bonding, orchestrator wants to use same devargs for LAG and non-LAG scenario to probe representor on PF1 using PF1 PCI address like ",representor=pf1vf[0-3]". This patch changes PCI address check policy to allow PF1 PCI address for representors on PF1. Note: detaching PF0 device can't

Re: [dpdk-dev] [PATCH] eal: fix leak in shared lib mode detection

06/05/2021 12:40, Bruce Richardson: > On Thu, May 06, 2021 at 12:06:37PM +0200, David Marchand wrote: > > This is reported by our internal covscan: > > > > 1. dpdk-20.11/lib/librte_eal/common/eal_common_options.c:508: alloc_fn: > > Storage is returned from allocation function "dlopen". > > 6. dpdk

Re: [dpdk-dev] [PATCH v3] hash: fix tuple adjustment

06/05/2021 13:40, Vladimir Medvedkin: > rte_thash_adjust_tuple() uses random to generate a new subtuple if > fn() callback reports about collision. In some cases random changes > the subtuple in a way that after complementary bits are applied the > original tuple is obtained. This patch replaces ra

Re: [dpdk-dev] [PATCH v2 1/2] graph: fix memory leak

On Thu, May 6, 2021 at 9:16 AM Min Hu (Connor) wrote: > > From: HongBo Zheng > > Fix function 'stats_mem_populate' return without > free dynamic memory referenced by 'stats'. > > Fixes: af1ae8b6a32c ("graph: implement stats") > Cc: sta...@dpdk.org > > Signed-off-by: HongBo Zheng > Signed-off-by:

Re: [dpdk-dev] [PATCH v2 2/2] graph: fix dereferencing null pointer

On Thu, May 6, 2021 at 9:16 AM Min Hu (Connor) wrote: > > From: HongBo Zheng > > In function 'stats_mem_init', pointer 'stats' should > be confirmed not null before memset it. > > Fixes: af1ae8b6a32c ("graph: implement stats") > Cc: sta...@dpdk.org > > Signed-off-by: HongBo Zheng > Signed-off-by

Re: [dpdk-dev] [dpdk-stable] [PATCH] bus/fslmc: remove unused macro

06/05/2021 10:41, David Marchand: > Fixes: ce9efbf5bb09 ("bus/fslmc: support dynamic logging") > Cc: sta...@dpdk.org > > Signed-off-by: David Marchand Acked-by: Hemant Agrawal Applied, thanks

Re: [dpdk-dev] [EXT] Re: [PATCH 2/2] eal: fix side effects in ptr align macros

Hello Pavan, On Mon, May 10, 2021 at 11:50 AM Pavan Nikhilesh Bhagavatula wrote: > > >On Sun, 9 May 2021 22:48:02 +0530 > > wrote: > > > >> From: Pavan Nikhilesh > >> > >> Avoid expanding parameters inside RTE_*_ALIGN macros. > >> Update common_autotest to detect macro side effects. > >> Workaro

Re: [dpdk-dev] [PATCH v2] raw/ioat: fix parameter shadow warning

On Mon, May 10, 2021 at 12:55:14PM +, Kevin Laatz wrote: > In the function __idxd_completed_ops() we have a parameter shadow warning > due to a local variable having the same name as one of the function > parameters. This issue is fixed by simply renaming the local variable. > > This warning w

[dpdk-dev] [PATCH 4/4] net/hns3: return fail if FEC mode not support

From: Chengwen Feng If the FEC mode was not supported, it should return error code. This patch also adds a space when log error info. Fixes: 9bf2ea8dbc65 ("net/hns3: support FEC") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_ethde

[dpdk-dev] [PATCH 0/4] bugfix for hns3 PMD

This patch set contains four bugfix for hns3 PMD Chengwen Feng (4): net/hns3: fix unsupported mbx msg id log error net/hns3: fix secondary process request start/stop Rx/Tx net/hns3: fix memory barrier incorrectly placed net/hns3: return fail if FEC mode not support drivers/net/hns3/hns3_

[dpdk-dev] [PATCH 2/4] net/hns3: fix secondary process request start/stop Rx/Tx

From: Chengwen Feng This secondary process should not send request to start/stop Rx/Tx, this patch fixes it. Fixes: 23d4b61fee5d ("net/hns3: support multiple process") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_mp.c | 2 +- 1 fil

[dpdk-dev] [PATCH 3/4] net/hns3: fix memory barrier incorrectly placed

From: Chengwen Feng The memory barrier is used to ensure that the response is returned only after the Tx/Rx function is set, it should place after the Rx/Tx function is set. Fixes: 23d4b61fee5d ("net/hns3: support multiple process") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-b

[dpdk-dev] [PATCH 1/4] net/hns3: fix unsupported mbx msg id log error

From: Chengwen Feng The mailbox message id is uint8_t, but the unsupported mailbox message id was logged by uint16. Fixes: 463e748964f5 ("net/hns3: support mailbox") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Min Hu (Connor) --- drivers/net/hns3/hns3_mbx.c | 2 +- 1 file

[dpdk-dev] [RFC v2] bus/auxiliary: introduce auxiliary bus

Auxiliary [1] provides a way to split function into child-devices representing sub-domains of functionality. Each auxiliary_device represents a part of its parent functionality. Auxiliary device is identified by unique device name, sysfs path: /sys/bus/auxiliary/devices/ [1] kernel auxiliary bu

[dpdk-dev] [PATCH v2 1/2] eal: fix side effects in align mul macros

From: Pavan Nikhilesh Avoid expanding parameters inside the macro multiple times. For example: RTE_ALIGN_MUL_NEAR(rte_rdtsc() - start, CYC_PER_10MHZ); Here rte_rdtsc() call is expanded multiple times in the macro causing it to return different values that leads to unintended side effects.

[dpdk-dev] [PATCH v2 2/2] eal: fix side effects in ptr align macros

From: Pavan Nikhilesh Avoid expanding parameters inside RTE_*_ALIGN macros. Update common_autotest to detect macro side effects. Workaround static arrays relying on RTE_ALIGN macros. Fixes: af75078fece3 ("first public release") Cc: sta...@dpdk.org Signed-off-by: Pavan Nikhilesh Signed-off-by:

Re: [dpdk-dev] [PATCH v2] raw/ioat: fix parameter shadow warning

On Mon, May 10, 2021 at 2:55 PM Kevin Laatz wrote: > > In the function __idxd_completed_ops() we have a parameter shadow warning > due to a local variable having the same name as one of the function > parameters. This issue is fixed by simply renaming the local variable. > > This warning was disco

Re: [dpdk-dev] [PATCH v2] test/timer: fix memzone reserve failure check

04/05/2021 03:07, Min Hu (Connor): > Segmentation fault may occur without checking if memzone > reserves succeed or not. The sentence is confusing. Please try to make it more logical. Something like "It was potentially dereferencing a null pointer. It is fixed by checking the pointer before derefe

Re: [dpdk-dev] [PATCH v2] raw/ioat: fix parameter shadow warning

On Mon, May 10, 2021 at 04:06:00PM +0200, David Marchand wrote: > On Mon, May 10, 2021 at 2:55 PM Kevin Laatz wrote: > > > > In the function __idxd_completed_ops() we have a parameter shadow warning > > due to a local variable having the same name as one of the function > > parameters. This issue

[dpdk-dev] [PATCH 1/4] net/bnx2x: fix build with gcc11

Reproduced with '--buildtype=debugoptimized' config, compiler version: gcc (GCC) 12.0.0 20210509 (experimental) Build error: In file included from ../drivers/net/bnx2x/bnx2x_rxtx.c:8: ../drivers/net/bnx2x/bnx2x_rxtx.c: In function ‘bnx2x_upd_rx_prod_fast’: ../drivers/net/bnx2x/bnx2x.h:1528:35:

[dpdk-dev] [PATCH 2/4] net/bnx2x: fix build with gcc11

Reproduced with '--buildtype=debugoptimized' config, compiler version: gcc (GCC) 12.0.0 20210509 (experimental) Build error: In file included from ../drivers/net/bnx2x/bnx2x.c:16: ../drivers/net/bnx2x/bnx2x.c: In function ‘bnx2x_hc_ack_sb’: ../drivers/net/bnx2x/bnx2x.h:1528:35: warning: ‘

[dpdk-dev] [PATCH 3/4] net/ice/base: fix build with gcc11

Reproduced with '--buildtype=debugoptimized' config, compiler version: gcc (GCC) 12.0.0 20210509 (experimental) There are multiple build errors, like: ../drivers/net/ice/base/ice_switch.c: In function ‘ice_add_marker_act’: ../drivers/net/ice/base/ice_switch.c:3727:15: warning: array subscr

[dpdk-dev] [PATCH 4/4] net/tap: fix build with gcc11

Reproduced with '--buildtype=debugoptimized' config, compiler version: gcc (GCC) 12.0.0 20210509 (experimental) There are multiple build errors, like: In file included from ../drivers/net/tap/tap_flow.c:13: In function ‘rte_jhash_2hashes’, inlined from ‘rte_jhash’ at ../lib/hash/rte_jhash.h:28

Re: [dpdk-dev] [PATCH v3] eal: fix memory mapping for 32-bit targets

On 10-May-21 1:42 PM, Thomas Monjalon wrote: +Cc Anatoly 08/05/2021 16:27, Lance Richardson: For 32-bit targets, size_t is normally a 32-bit type and does not have sufficient range to represent 64-bit offsets that can are needed when mapping PCI addresses. Use uint64_t instead. Found when atte

Re: [dpdk-dev] [PATCH v3] eal: fix memory mapping for 32-bit targets

10/05/2021 17:31, Burakov, Anatoly: > On 10-May-21 1:42 PM, Thomas Monjalon wrote: > > +Cc Anatoly > > > > 08/05/2021 16:27, Lance Richardson: > >> For 32-bit targets, size_t is normally a 32-bit type and > >> does not have sufficient range to represent 64-bit offsets > >> that can are needed when

Re: [dpdk-dev] [PATCH v4] net/mlx5: fix tunnel offload private items location

On 5/10/2021 1:29 PM, Gregory Etelson wrote: > Hello Ferruh, > > The patch has failed ci/Intel-compilation test. > (http://mails.dpdk.org/archives/test-report/2021-May/193327.html) > I've checked that with Thomas. He confirmed that was a known fault and > should not affect the current patch. > Ca

[dpdk-dev] [PATCH 0/3] app/flow-perf: support meter policy API

This patch set contains three updates for flow perf application: 1. Adds the option 'policy-mtr' to indicate if create a meter with policy; 2. Adds the option 'meter-cir' to configure the CIR parameter; 3. Adds the option 'packet-mode' to enable the meter PPS. Jiawei Wang (3): app/flow-perf: sup

[dpdk-dev] [PATCH 1/3] app/flow-perf: support meter policy API

Add option "policy-mtr" to indicate if meter creation will include policy or not. Meter creation will keep same without it. With "policy-mtr", policy is introduced. API create_meter_policy is to create a policy. API create_meter_rule will use it to create meter. Signed-off-by: Haifei Luo Signed-

[dpdk-dev] [PATCH 2/3] app/flow-perf: add new meter CIR Configuration

Add the new meter CIR configuration parameter, user can set the different value for committed information rate(CIR) parameter. The usage as below: --meter-cir=N, default count is 125. Signed-off-by: Jiawei Wang Acked-by: Wisam Jaddo --- app/test-flow-perf/main.c | 13 +++-- do

[dpdk-dev] [PATCH 3/3] app/flow-perf: add the supports for meter PPS

The flow perf application used the srtcm_rfc2697 as meter profile while do the meter testing. This patch adds the support new configuration parameter '--packet-mode' to generate the meter flows with the packet mode. Signed-off-by: Jiawei Wang Acked-by: Wisam Jaddo --- app/test-flow-perf/main.c

Re: [dpdk-dev] [PATCH 1/3] app/flow-perf: support meter policy API

Hi guys, I have some issues w/ this patch, please exclude my ack for now. Can you please check my comments on the this previous patch in ML? BRs, Wisam Jaddo Get Outlook for iOS From: Jiawei(Jonny) Wang Sent: Monday, May 10, 2021 7:17:21

[dpdk-dev] [PATCH] net/nfp: fix reporting of RSS capabilities

Before this change the dev_infos callback always reported RSS capabilities regardless of whether the capability is supported by the device or not. First check the capabilities field in the BAR of the device and advertise RSS functionality accordingly. Fixes: 8b945a7f7d ("drivers/net: update Rx RSS

[dpdk-dev] [PATCH] examples/l3fwd: change mq-mode on single queue devices

From: "Chaoyong.He" Set the Rx multi-queue mode to NONE when configuring a port that is associated with hardware that only supports a single Rx queue. Signed-off-by: Chaoyong He Signed-off-by: Heinrich Kuhn Signed-off-by: Simon Horman --- examples/l3fwd/main.c | 4 1 file changed, 4 ins

Re: [dpdk-dev] [PATCH 3/4] net/ice/base: fix build with gcc11

> -Original Message- > From: dev On Behalf Of Ferruh Yigit > Sent: Monday, May 10, 2021 23:03 > To: Yang, Qiming ; Zhang, Qi Z ; > Stillwell Jr, Paul M > ; Lu, Wenzhuo ; Rong, > Leyi ; > Shukla, Shivanshu > Cc: Yigit, Ferruh ; dev@dpdk.org; sta...@dpdk.org; > Kevin Traynor > ; Ajit Kha

Re: [dpdk-dev] [EXT] Re: [PATCH] config/arm: add ability to express arch extensions

>05/05/2021 14:14, pbhagavat...@marvell.com: >> From: Pavan Nikhilesh >> >> The ARM architecture allows SoCs to have extensions in addition >> to base profiles such as Large System Extension (LSE), CRC etc. >> >> Add ability to declare SoC specific extensions. >> >> Signed-off-by: Pavan Nikhilesh

Re: [dpdk-dev] [PATCH 3/4] net/ice/base: fix build with gcc11

> -Original Message- > From: Wang, Haiyue > Sent: Tuesday, May 11, 2021 01:05 > To: Ferruh Yigit ; Yang, Qiming > ; Zhang, Qi Z > ; Stillwell Jr, Paul M ; > Lu, Wenzhuo > ; Rong, Leyi ; Shukla, Shivanshu > > Cc: Yigit, Ferruh ; dev@dpdk.org; sta...@dpdk.org; > Kevin Traynor > ; Ajit Kha

Re: [dpdk-dev] [EXT] Re: [PATCH] config/arm: add ability to express arch extensions

> > >05/05/2021 14:14, pbhagavat...@marvell.com: > >> From: Pavan Nikhilesh > >> > >> The ARM architecture allows SoCs to have extensions in addition to > >> base profiles such as Large System Extension (LSE), CRC etc. > >> > >> Add ability to declare SoC specific extensions. > >> > >> Signed-of

Re: [dpdk-dev] [PATCH 3/4] net/ice/base: fix build with gcc11

On 5/10/2021 6:04 PM, Wang, Haiyue wrote: >> -Original Message- >> From: dev On Behalf Of Ferruh Yigit >> Sent: Monday, May 10, 2021 23:03 >> To: Yang, Qiming ; Zhang, Qi Z >> ; Stillwell Jr, Paul M >> ; Lu, Wenzhuo ; Rong, >> Leyi ; >> Shukla, Shivanshu >> Cc: Yigit, Ferruh ; dev@dpdk.

Re: [dpdk-dev] [PATCH 3/4] net/ice/base: fix build with gcc11

On 5/10/2021 6:13 PM, Wang, Haiyue wrote: >> -Original Message- >> From: Wang, Haiyue >> Sent: Tuesday, May 11, 2021 01:05 >> To: Ferruh Yigit ; Yang, Qiming >> ; Zhang, Qi Z >> ; Stillwell Jr, Paul M >> ; Lu, Wenzhuo >> ; Rong, Leyi ; Shukla, Shivanshu >> >> Cc: Yigit, Ferruh ; dev@dpdk

Re: [dpdk-dev] [EXT] Re: [PATCH] config/arm: add ability to express arch extensions

>-Original Message- >From: dev On Behalf Of Honnappa >Nagarahalli >Sent: Monday, May 10, 2021 10:51 PM >To: Pavan Nikhilesh Bhagavatula ; >tho...@monjalon.net >Cc: Jerin Jacob Kollanukkaran ; >juraj.lin...@pantheon.tech; Jan Viktorin ; >Ruifeng Wang ; Bruce Richardson >; dev@dpdk.org; n

[dpdk-dev] [PATCH v3 1/2] eal: fix side effects in align mul macros

From: Pavan Nikhilesh Avoid expanding parameters inside the macro multiple times. For example: RTE_ALIGN_MUL_NEAR(rte_rdtsc() - start, CYC_PER_10MHZ); Here rte_rdtsc() call is expanded multiple times in the macro causing it to return different values that leads to unintended side effects.

[dpdk-dev] [PATCH v3 2/2] eal: fix side effects in ptr align macros

From: Pavan Nikhilesh Avoid expanding parameters inside RTE_*_ALIGN macros. Update common_autotest to detect macro side effects. Workaround static arrays relying on RTE_ALIGN macros. Fixes: af75078fece3 ("first public release") Cc: sta...@dpdk.org Signed-off-by: Pavan Nikhilesh Signed-off-by:

Re: [dpdk-dev] [PATCH v3] eal: fix memory mapping for 32-bit targets

On Mon, May 10, 2021 at 11:55 AM Thomas Monjalon wrote: > 10/05/2021 17:31, Burakov, Anatoly: > > On 10-May-21 1:42 PM, Thomas Monjalon wrote: > > > +Cc Anatoly > > > > > > 08/05/2021 16:27, Lance Richardson: > > >> For 32-bit targets, size_t is normally a 32-bit type and > > >> does not have suf

[dpdk-dev] [PATCH] doc: fix build with Sphinx 4

Sphinx 4.0 became stricter with permalink configuration: " html_add_permalinks has been deprecated since v3.5.0. Please use html_permalinks and html_permalinks_icon instead. " The new variable is used while keeping compatibility with older Sphinx versions. Signed-off-by: Thomas Monjalon --- doc

Re: [dpdk-dev] [EXT] Re: [PATCH] config/arm: add ability to express arch extensions

> >> > >> >05/05/2021 14:14, pbhagavat...@marvell.com: > >> >> From: Pavan Nikhilesh > >> >> > >> >> The ARM architecture allows SoCs to have extensions in addition to > >> >> base profiles such as Large System Extension (LSE), CRC etc. > >> >> > >> >> Add ability to declare SoC specific extensio

Re: [dpdk-dev] [EXT] Re: [PATCH] config/arm: add ability to express arch extensions

> > > > > The patch still holds true for CRC though as it is listed separately > > below > > https://developer.arm.com/architectures/cpu-architecture/a- > > profile/exploration-tools/feature-names-for-a-profile CRC is mandatory starting in V8.1, refer to Arm-ARM document. > > > > Also, looks lik

[dpdk-dev] [PATCH v3] test/timer: fix memzone reserve failure check

Segmentation fault may occur without checking if memzone reserves succeed or not. This patch fixed it. Fixes: 50247fe03fe0 ("test/timer: exercise new APIs in secondary process") Cc: sta...@dpdk.org Signed-off-by: Min Hu (Connor) --- v3: * do not split error messages. v2: * use TEST_ASSERT_NOT_

Re: [dpdk-dev] [PATCH v2] test/timer: fix memzone reserve failure check

在 2021/5/10 22:13, Thomas Monjalon 写道: 04/05/2021 03:07, Min Hu (Connor): Segmentation fault may occur without checking if memzone reserves succeed or not. The sentence is confusing. Please try to make it more logical. Something like "It was potentially dereferencing a null pointer. It is f

[dpdk-dev] [PATCH v2 1/3] net/i40e: fix return status for unsupported VF message

This patch modifies the return status for unsupported VF messages, in order to make it the same as the return status of the kernel driver. Fixes: 4861cde46116 ("i40e: new poll mode driver") Cc: sta...@dpdk.org Signed-off-by: Alvin Zhang --- drivers/net/i40e/i40e_pf.c | 4 ++-- 1 file changed, 2

[dpdk-dev] [PATCH v2 2/3] common/iavf: fix V-channel status

Add VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED to enum virtchnl_status_code. Fixes: e5b2a9e957e7 ("net/avf/base: add base code for avf PMD") Cc: sta...@dpdk.org Signed-off-by: Alvin Zhang --- drivers/common/iavf/virtchnl.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/common/iavf/virtchn

[dpdk-dev] [PATCH v2 3/3] net/iavf: fix V-channel message status

Add support of VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED for facilitating compatibility with PF. Chang the error log-level from ERROR to WARNING when a V-channel message is not supported by PF, because the VF may still be able to run without certain features which not supported by PF. Fixes: 0c35eecfe8

Re: [dpdk-dev] [PATCH 3/4] net/ice/base: fix build with gcc11

> -Original Message- > From: Yigit, Ferruh > Sent: Tuesday, May 11, 2021 01:28 > To: Wang, Haiyue ; Yang, Qiming > ; Zhang, Qi Z > ; Stillwell Jr, Paul M ; > Lu, Wenzhuo > ; Rong, Leyi ; Shukla, Shivanshu > > Cc: dev@dpdk.org; sta...@dpdk.org; Kevin Traynor ; Ajit > Khaparde > > Subje

[dpdk-dev] [PATCH v3 1/3] net/i40e: fix return status for unsupported VF message

This patch modifies the return status for unsupported VF messages, in order to make it the same as the return status of the kernel driver. Fixes: 4861cde46116 ("i40e: new poll mode driver") Cc: sta...@dpdk.org Signed-off-by: Alvin Zhang --- drivers/net/i40e/i40e_pf.c | 4 ++-- 1 file changed, 2

[dpdk-dev] [PATCH v3 2/3] common/iavf: fix V-channel status

Add VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED to enum virtchnl_status_code. Fixes: e5b2a9e957e7 ("net/avf/base: add base code for avf PMD") Cc: sta...@dpdk.org Signed-off-by: Alvin Zhang --- drivers/common/iavf/virtchnl.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/common/iavf/virtchn

[dpdk-dev] [PATCH v3 3/3] net/iavf: fix V-channel message status

Add support of VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED for facilitating compatibility with PF. Change the error log-level from ERROR to WARNING when a V-channel message is not supported by PF, because the VF may still be able to run without certain features which not supported by PF. Fixes: 0c35eecfe

[dpdk-dev] [PATCH v2] net/iavf: fix error logs

Remove some useless logs which are duplicated of the lower-level function logs and may confuse users when running VF without some features that not supported by PF. Fixes: 5a038d19962d ("net/iavf: fix RSS configuration on i40e VF") Fixes: 95f2f0e9fc2a ("net/iavf: improve default RSS") Signed-off-

Re: [dpdk-dev] [PATCH v3 2/2] eal: fix side effects in ptr align macros

> -Original Message- > From: pbhagavat...@marvell.com > Sent: Tuesday, May 11, 2021 03:40 > To: david.march...@redhat.com; Wang, Haiyue ; Matan > Azrad ; > Shahaf Shuler ; Viacheslav Ovsiienko > ; Jiawen Wu > ; Jian Wang ; Chautru, > Nicolas > ; Thomas Monjalon ; Yigit, > Ferruh > ; An

[dpdk-dev] [PATCH v2] net/i40e: fix offload flag checking in simple Tx datapath

Tx offload flags 'PKT_TX_IPV6, PKT_TX_IPV4, PKT_TX_OUTER_IPV6, PKT_TX_OUTER_IPV4' are supported in simple datapath. This patch removes these offload flags from packet checking in simple Tx datapath and defines 2 macro I40E_TX_OFFLOAD_SIMPLE_SUP_MASK and I40E_TX_OFFLOAD_SIMPLE_NOTSUP_MASK. Fixes:

Re: [dpdk-dev] [PATCH v3 2/3] common/iavf: fix V-channel status

> -Original Message- > From: Zhang, AlvinX > Sent: Tuesday, May 11, 2021 10:03 AM > To: Xing, Beilei ; Xu, Ting > Cc: dev@dpdk.org; Zhang, AlvinX ; > sta...@dpdk.org > Subject: [PATCH v3 2/3] common/iavf: fix V-channel status > > Add VIRTCHNL_STATUS_ERR_NOT_IMPLEMENTED to enum > virtc

Re: [dpdk-dev] : nvgre inner rss problem in mlx5

Will do. Thanks BR wenxu 发件人：Asaf Penso 发送日期：2021-05-10 16:05:54 收件人：wenxu 抄送人："dev@dpdk.org" 主题：RE: Re:: [dpdk-dev] nvgre inner rss problem in mlx5 Hello Wenxu, Can you please create a new BZ ticket? Looks like this is not handled properly in our pmd and we’ll handle it and update.

[dpdk-dev] [PATCH 1/2] net/mlx5: fix the checking for age action

Current the ASO age action was supported in the non-root table, and the counter based age action was be used in the root table. The FDB table skips group 0 on MLX5 PMD by adding implicit rule that jump to non-root table, but PMD code use the original group value for checking. This patch uses the

[dpdk-dev] [PATCH 2/2] net/mlx5: fix ASO age null context issue

The flow context in the rte_flow_action_age structure was set by user, and reported by the MLX5 PMD while calling rte_flow_get_aged_flow API. If the flow context was NULL while create ASO age action, while flow aged, the PMD report the NULL context to user. This patch adds the checking if context

[dpdk-dev] DTS improvement WG - Debate/triag inputs

Hello, We will continuing debating/triaging the inputs from [1]. It would be helpful if everyone can review and come prepared for the meeting. I will send out the MoM for the last meeting soon. Thank you, Honnappa [1] https://docs.google.com/document/d/1c5S0_mZzFvzZfYkq

[dpdk-dev] [PATCH v4 00/19] net/ena: update ENA PMD to v2.3.0

Hi, this version updates the driver to version 2.3.0, which fixes multiple bugs, contains part of the work on making the ENA PMD fully MP aware, and also updates HAL to the latest version. More detailed list of changes: * memcpy mapping to the dpdk-optimized version. * ena_com (HAL) update to

  1   2   >