On Thu, Mar 27, 2025 at 8:57 AM Yang Ming wrote:
>
> DPDK detect vfio container according the existence of vfio
> module. But for container with non-privileged mode, there is
> possibility that no VFIO_DIR(/dev/vfio) mapping from host to
> container when host have both Intel NIC and Mellanox NIC b
Hi Patrick,
I will produce the patches with necessary changes to build on Grace for 22.11
and 23.11.
From: Patrick Robb
Sent: Thursday, May 15, 2025 3:59 PM
To: Wathsala Wathawana Vithanage
Cc: c...@dpdk.org; dev ; Honnappa Nagarahalli
; sta...@dpdk.org; Doug Foster
; nd
Subject: Re: meson
Signed-off-by: Yang Ming
---
.mailmap | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/.mailmap b/.mailmap
index d8439b79ce..e63e1af4fa 100644
--- a/.mailmap
+++ b/.mailmap
@@ -1734,7 +1734,7 @@ Yahui Cao
Yajun Wu
Yangchao Zhou
Yanglong Wu
-Yang Ming
+Yang Ming
Yang
#
May 15, 2025
Attendees
1. Patrick Robb
2. Paul Szczepanek
3. Luca Vizzarro
4. Manit Mahajan
5. Aaron Conole
#
Minutes
=
Thanks Wathsala,
How can we help on the UNH side? Do you want us to produce the 22.11 and
23.11 diffs/patches, which would be based on d007038, with some
modifications based on what you had said to Doug?
I'm not sure what the process looks like exactly for providing backports to
LTS maintainers.
Remove capability requirement from checksum offload test suite
that was not required to run the test cases.
Signed-off-by: Dean Marx
Reviewed-by: Luca Vizzarro
Reviewed-by: Nicholas Pratte
---
dts/tests/TestSuite_checksum_offload.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/dts/tests/
Looks good!
Reviewed-by: Dean Marx
Reviewed-by: Patrick Robb
On Wed, Apr 23, 2025 at 3:40 PM Nicholas Pratte wrote:
> Provide functional performance method to run performance tests using a
> user-supplied performance traffic generator. The single core performance
> test is included, with some basic statistics checks verifying TG packet
> transmission rates
On Wed, Apr 23, 2025 at 3:40 PM Nicholas Pratte wrote:
> Implement the TREX traffic generator for use in the DTS framework. The
> provided implementation leverages TREX's stateless API automation
> library, via use of a Python shell. As such, version control of TREX may
> be needed. The DTS conte
On Wed, Apr 23, 2025 at 3:40 PM Nicholas Pratte wrote:
>
> +
> def send_packets_and_capture(
> self,
> packets: list[Packet],
> diff --git
> a/dts/framework/testbed_model/traffic_generator/performance_traffic_generator.py
> b/dts/framework/testbed_model/traffic_generator/pe
Change the unified packet testing suite to use layer 4
destination ports as packet verification IDs.
Signed-off-by: Dean Marx
Reviewed-by: Luca Vizzarro
Reviewed-by: Nicholas Pratte
---
dts/tests/TestSuite_uni_pkt.py | 106 -
1 file changed, 51 insertions(+), 55
Add a field to the TestPmdVerbosePacket text parser
that stores destination port values for TCP/UDP packets.
Signed-off-by: Dean Marx
Reviewed-by: Luca Vizzarro
Reviewed-by: Nicholas Pratte
---
dts/framework/remote_session/testpmd_shell.py | 5 +
1 file changed, 5 insertions(+)
diff --git
Change the checksum offload testing suite to use layer 4
destination ports as packet verification IDs.
Signed-off-by: Dean Marx
Reviewed-by: Luca Vizzarro
Reviewed-by: Nicholas Pratte
---
dts/tests/TestSuite_checksum_offload.py | 130 ++--
1 file changed, 77 insertions(+),
> +
> +int zsda_sym_dev_create(struct zsda_pci_device *zsda_pci_dev);
> +
> +void zsda_sym_dev_destroy(struct zsda_pci_device *zsda_pci_dev);
> +
sym->crypto missed here.
Please fix CI.
https://mails.dpdk.org/archives/test-report/2025-May/877907.html
>
> v3:
> - Modify the code according to the advise
>
> v2:
> - Modify the errors about cryptodevs/zsda.rst.
> - Modify the errors about author.
>
> v1:
> - Add zsda cryptodev driver.
>
>
> Hanxiao Li (9):
> cryp
When the power_intel_uncore_autotest unit test is run as an unprivileged
user which cannot init the power library, it crashes the unit test
binary due to calling "rte_power_uncore_exit" after the first test case
(initialization) fails. This crash is due to trying to write to NULL
file handles.
Fix
Hi all,
I've submitted an RFC for an initial Flow test suite here:
https://patches.dpdk.org/project/dpdk/patch/20250515163809.184720-2-dm...@iol.unh.edu/
It covers most of the functionality discussed in this thread, besides
jump, RSS, and represented port actions which will be added in the
near
If the initialization of the uncore power management fails, mark the
test as skipped rather than a failure. This prevents false failures in
the case where we are running as a normal user, who doesn't have
permissions to edit uncore power states.
Signed-off-by: Bruce Richardson
---
app/test/test_
Add an RTE Flow API testing suite, which covers some basic
synchronous Flow API rules that should be supported across PMDs.
This suite will be added to over time, as the Flow API is too large
to cover all in one suite, and sending one monolithic series
would be impossible.
Signed-off-by: Dean Marx
This patch adds an initial Flow API test suite, which can
be expanded upon as Flow coverage becomes more widely supported.
Rather than creating one suite to test all possible Flow
rules, which would be statistically impossible due to the number
of possible combinations, this suite creates a framewo
On Thu, May 15, 2025 at 05:22:00PM +0200, Morten Brørup wrote:
> Bruce,
>
> I found another gap in the documentation of CPU affinities.
>
> Let's assume a 6-core system.
>
> If a DPDK application is run using:
> taskset --cpu-list 0-2 dpdk-app --lcores 3-5
>
> Lcores 3-5 are EAL cores, and each
Bruce,
I found another gap in the documentation of CPU affinities.
Let's assume a 6-core system.
If a DPDK application is run using:
taskset --cpu-list 0-2 dpdk-app --lcores 3-5
Lcores 3-5 are EAL cores, and each of the three lcores is bound to one cpu,
respectively cpu 3, 4 and 5?
Lcore 3 is
On Tue, May 13, 2025 at 05:17:07PM +0100, Bruce Richardson wrote:
> This v4 scales back some of the work done in the previous versions, since
> more options need to be looked at for the core handling. For now, then
> this patchset is reduced to some basic cleanup before any more serious work
> is d
On Thu, May 15, 2025 at 11:51:30AM +0530, Shaiq Wani wrote:
> Removed redundant and unused fields from the idpf pmd specific field
> in common Tx queue structure to reduce its memory footprint.
>
> Signed-off-by: Shaiq Wani
> ---
> drivers/net/intel/common/tx.h | 6 +--
> driver
On Tue, May 06, 2025 at 12:50:24PM -0500, Jake Freeland wrote:
> The ice_timestamp_dynfield_offset and ice_timestamp_dynflag global
> variables are used to access the dynamic mbuf timestamp field when
> timestamp offloading is enabled.
>
> Move these into the ice_rx_queue structure, located in sha
Add a script to buildtools to report the minimum meson version given in
our meson.build file. Then use this script in two ways:
1. in the .ci/linux-setup.sh script, use the auto-determined minimum
version to set up the CI, rather than hard-coding it.
2. in meson.build call the script to report
On Thu, May 15, 2025 at 01:15:43PM +0200, Burakov, Anatoly wrote:
> On 5/14/2025 6:32 PM, Bruce Richardson wrote:
> > On Mon, May 12, 2025 at 01:54:27PM +0100, Anatoly Burakov wrote:
> > > The `rdh` (read head) field in the `ixgbe_rx_queue` struct is not used
> > > anywhere in the codebase, and can
On Thu, May 15, 2025 at 01:11:03PM +0200, Burakov, Anatoly wrote:
> On 5/15/2025 12:59 PM, Bruce Richardson wrote:
> > On Mon, May 12, 2025 at 01:54:32PM +0100, Anatoly Burakov wrote:
> > > Make the iavf driver use the new common Rx queue structure.
> > >
> > > Because the iavf driver supports bot
On Thu, May 15, 2025 at 01:09:31PM +0200, Burakov, Anatoly wrote:
> On 5/14/2025 6:52 PM, Bruce Richardson wrote:
> > On Mon, May 12, 2025 at 01:54:30PM +0100, Anatoly Burakov wrote:
> > > Make the i40e driver use the new common Rx queue structure.
> > >
> > > Because the i40e driver supports both
On Thu, May 15, 2025 at 01:39:47PM +0200, David Marchand wrote:
> On Thu, May 15, 2025 at 11:37 AM Bruce Richardson
> wrote:
> >
> > On Thu, May 15, 2025 at 09:09:22AM +0200, David Marchand wrote:
> > > On Tue, May 13, 2025 at 11:00 AM Bruce Richardson
> > > wrote:
> > > >
> > > > Add a script to
Okay, I'll make a ticket for us to look at this again and raise it at the
CI meeting this morning. Thanks for the heads up.
On Thu, May 15, 2025 at 8:59 AM David Marchand
wrote:
> On Tue, Apr 2, 2024 at 2:46 AM Stephen Hemminger
> wrote:
> >
> > On Mon, 1 Apr 2024 18:26:44 -0400
> > Patrick Rob
On Thu, May 15, 2025 at 1:40 PM David Marchand
wrote:
>
>
> Retests show the same error, so I think we are hitting the issue fixed
> with dbcd72f3fba0.
>
> Bumping the minimal version to 0.57.2 seems fine.
> I looked and can't find a distrib that ships meson 0.57.
> So either a user relies on the
On 5/14/2025 3:26 PM, Bruce Richardson wrote:
On Wed, May 14, 2025 at 02:02:11PM +0100, Anatoly Burakov wrote:
Most testpmd help strings are lower case. Amend those that aren't.
Signed-off-by: Anatoly Burakov
---
app/test-pmd/cmdline.c| 4 ++--
app/test-pmd/cmdline_tm.c | 36 ++
On Thu, Mar 13, 2025 at 4:37 PM Stephen Hemminger
wrote:
>
> On Thu, 13 Mar 2025 14:19:03 +0800
> Yang Ming wrote:
>
> > This change ensures that the current process is checked for
> > being run with 'setarch' before verifying the value of
> > '/proc/sys/kernel/randomize_va_space'. The '-R' or
>
On Mon, Apr 14, 2025 at 10:29 PM Andre Muezerie
wrote:
>
> Coverity complained about an overflow in a recently added function:
>
> https://scan4.scan.coverity.com/#/project-view/66295/10075?selectedIssue=461876
>
> CID 461876: (#1 of 1): Overflowed constant (INTEGER_OVERFLOW)
> 21. overflow_const:
On Thu, May 15, 2025 at 11:37 AM Bruce Richardson
wrote:
>
> On Thu, May 15, 2025 at 09:09:22AM +0200, David Marchand wrote:
> > On Tue, May 13, 2025 at 11:00 AM Bruce Richardson
> > wrote:
> > >
> > > Add a script to buildtools to report the minimum meson version given in
> > > our meson.build f
Dispatcher unit test failed which is unrelated to this patch and looks like
infrastructure issue.
Recheck-request: loongarch-unit-testing
--
Gowrishankar
> This change fixes an uninitialized local variable as reported in coverity
> issue
> 445390.
>
> Coverity-issue: 445390
> Fixes: a8ebe94f8
In non-pipeline mode, priority is ignored, a flow rule can be created
as a flow director rule or a switch rule depends on its pattern/action.
Hence removing the priority field check from ice_fdir_parse which is
causing valid flow creation to return failure.
Signed-off-by: Dhanya Pillai
---
drive
On 5/14/2025 6:32 PM, Bruce Richardson wrote:
On Mon, May 12, 2025 at 01:54:27PM +0100, Anatoly Burakov wrote:
The `rdh` (read head) field in the `ixgbe_rx_queue` struct is not used
anywhere in the codebase, and can be removed.
Signed-off-by: Anatoly Burakov
With the one comment below fixed.
On 5/15/2025 12:59 PM, Bruce Richardson wrote:
On Mon, May 12, 2025 at 01:54:32PM +0100, Anatoly Burakov wrote:
Make the iavf driver use the new common Rx queue structure.
Because the iavf driver supports both 16-byte and 32-byte descriptor
formats (controlled by RTE_LIBRTE_IAVF_16BYTE_RX_DESC
On 5/14/2025 6:52 PM, Bruce Richardson wrote:
On Mon, May 12, 2025 at 01:54:30PM +0100, Anatoly Burakov wrote:
Make the i40e driver use the new common Rx queue structure.
Because the i40e driver supports both 16-byte and 32-byte descriptor
formats (controlled by RTE_LIBRTE_I40E_16BYTE_RX_DESC d
On Mon, May 12, 2025 at 01:54:39PM +0100, Anatoly Burakov wrote:
> Currently, there are duplicate implementations of Tx mbuf recycle in some
> drivers, specifically ixgbe and i40e. Move them into a common header.
>
> Signed-off-by: Anatoly Burakov
> ---
> drivers/net/intel/common/recycle_mbufs.h
On Mon, May 12, 2025 at 01:54:32PM +0100, Anatoly Burakov wrote:
> Make the iavf driver use the new common Rx queue structure.
>
> Because the iavf driver supports both 16-byte and 32-byte descriptor
> formats (controlled by RTE_LIBRTE_IAVF_16BYTE_RX_DESC define), the common
> queue structure has
On Mon, May 12, 2025 at 01:54:34PM +0100, Anatoly Burakov wrote:
> The i40e driver has an implementation of vectorized mbuf rearm code that
> is identical to the one in the common code, so just use that.
>
> In addition, the i40e has an implementation of Rx queue rearm for Neon
> instruction set,
On Mon, May 12, 2025 at 01:54:33PM +0100, Anatoly Burakov wrote:
> There is certain amount of duplication between various drivers when it
> comes to Rx ring rearm. This patch takes implementation from ice driver
> as a base because it has support for no IOVA in mbuf as well as all
> vector implemen
Add crypto device operations for zsda devices.
Signed-off-by: Hanxiao Li
---
drivers/crypto/zsda/zsda_sym_pmd.c | 82 --
1 file changed, 77 insertions(+), 5 deletions(-)
diff --git a/drivers/crypto/zsda/zsda_sym_pmd.c
b/drivers/crypto/zsda/zsda_sym_pmd.c
index 9dbb0
> - .stats_get = NULL,
> - .stats_reset = NULL,
> + .stats_get = zsda_crypto_stats_get,
> + .stats_reset = zsda_crypto_stats_reset,
Hi Akhil,
Although I want to name the function "zsda_stats_get" as you suggested,
but the "zsda_crypto_stats_get" will call a function "zsda_stats_
Hi folks,
I recently updated to Fedora 42 that comes with GCC 15.
When building with -fsanitize=address (libasan.so.8), it seems that
the --no-huge mode (along with --no-shconf) fails at initialization for
some obscure reason.
"couldn't allocate memory due to IOVA exceeding limits of current DMA
Add crypto test for zsda device and driver.
Signed-off-by: Hanxiao Li
---
app/test/test_cryptodev.c | 7 +++
app/test/test_cryptodev.h | 1 +
doc/guides/cryptodevs/zsda.rst | 6 ++
3 files changed, 14 insertions(+)
diff --git a/app/test/test_cryptodev.c b/app/test/test_cryptod
Add crypto enqueue datapath configuration for zsda device.
Signed-off-by: Hanxiao Li
---
drivers/common/zsda/meson.build| 2 +-
drivers/crypto/zsda/zsda_sym.c | 256 +
drivers/crypto/zsda/zsda_sym.h | 45 +
drivers/crypto/zsda/zsda_sym_pmd.c | 23 +
Add crypto driver skeleton for zsda devices.
Signed-off-by: Hanxiao Li
---
MAINTAINERS | 6 ++
doc/guides/cryptodevs/index.rst | 1 +
doc/guides/cryptodevs/zsda.rst | 26 ++
drivers/common/zsda/meson.build | 12 ++-
drivers/common/zsda/zsda_devi
Add crypto capabilities scope for zsda device.
Signed-off-by: Hanxiao Li
---
doc/guides/cryptodevs/features/zsda.ini | 51 +
doc/guides/cryptodevs/zsda.rst | 26 +
doc/guides/rel_notes/release_25_07.rst | 8 ++
drivers/crypto/zsda/zsda_sym_capabilities.h | 1
Add crypto session configuration for zsda device.
Signed-off-by: Hanxiao Li
---
drivers/common/zsda/meson.build| 2 +-
drivers/crypto/zsda/zsda_sym_pmd.c | 45 ++-
drivers/crypto/zsda/zsda_sym_session.c | 498 +
drivers/crypto/zsda/zsda_sym_session.h | 81
Add crypto dequeue datapath configuration for zsda device.
Signed-off-by: Hanxiao Li
---
drivers/crypto/zsda/zsda_sym.c | 16
drivers/crypto/zsda/zsda_sym.h | 2 ++
drivers/crypto/zsda/zsda_sym_pmd.c | 12 ++--
3 files changed, 28 insertions(+), 2 deletions(-)
v3:
- Modify the code according to the advise
v2:
- Modify the errors about cryptodevs/zsda.rst.
- Modify the errors about author.
v1:
- Add zsda cryptodev driver.
Hanxiao Li (9):
crypto/zsda: add skeleton
crypto/zsda: add device operations
crypto/zsda: add statistics
crypto/zsda: add q
Add crypto queue pair configuration operations for zsda device.
Signed-off-by: Hanxiao Li
---
drivers/crypto/zsda/zsda_sym_pmd.c | 82 --
1 file changed, 79 insertions(+), 3 deletions(-)
diff --git a/drivers/crypto/zsda/zsda_sym_pmd.c
b/drivers/crypto/zsda/zsda_sym_
Add crypto statistics operations for zsda devices.
Signed-off-by: Hanxiao Li
---
drivers/crypto/zsda/zsda_sym_pmd.c | 23 +--
1 file changed, 21 insertions(+), 2 deletions(-)
diff --git a/drivers/crypto/zsda/zsda_sym_pmd.c
b/drivers/crypto/zsda/zsda_sym_pmd.c
index d19bc3cb
On Wed, May 14, 2025 at 06:15:26PM +, Dhanya Pillai wrote:
> In non-pipeline mode, priority is ignored, a flow rule can be created
> as a flow director rule or a switch rule depends on its pattern/action.
> Hence removing the priority field check from ice_fdir_parse which is
> causing valid flo
On Thu, May 15, 2025 at 09:09:22AM +0200, David Marchand wrote:
> On Tue, May 13, 2025 at 11:00 AM Bruce Richardson
> wrote:
> >
> > Add a script to buildtools to report the minimum meson version given in
> > our meson.build file. Then use this script in two ways:
> >
> > 1. in the .ci/linux-setup
Hi,
On 14/05/2025 10:50 AM, Gregory Etelson wrote:
Patch `2d8763432add` was not applied correctly.
As a result, `mlx5_rxq_data::mark_flag` was assigned wrong value.
Fixes: 2d8763432add ("net/mlx5: fix shared Rx queue data access race")
Signed-off-by: Gregory Etelson
Acked-by: Viacheslav Ovsii
Fix coverity issue on out-of-bounds access.
Coverity-issue: 403166, 403171, 403172
Fixes: 5686b573e4b ("crypto/cnxk: support SM2")
Cc: sta...@dpdk.org
Signed-off-by: Gowrishankar Muthukrishnan
---
drivers/crypto/cnxk/cnxk_ae.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/drivers
Hi,
On 06/05/2025 4:31 PM, Gregory Etelson wrote:
HWS has limited resources to translate flow pattern.
When pattern translation does not fit STE, HWS translation fails with
E2BIG error.
The patch verifies that the E2BIG error value is kept during flow
error notification.
Fixes: e38776c36c8a (
Fix uninitialized variable as reported in coverity scan.
Coverity-issue: 457449
Fixes: d1b484bf187 ("vhost/crypto: support asymmetric RSA")
Signed-off-by: Gowrishankar Muthukrishnan
---
lib/vhost/vhost_crypto.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/vhost/vhost_
This patch fixes incorrect return value when RSA negative
test scenario is validated.
Coverity-issue: 457463
Fixes: 9682e8246ae ("test/crypto: add negative case for RSA verification")
Signed-off-by: Gowrishankar Muthukrishnan
---
app/test/test_cryptodev_asym.c | 1 +
1 file changed, 1 insertion
This change fixes an uninitialized local variable as reported
in coverity issue 445390.
Coverity-issue: 445390
Fixes: a8ebe94f8cc ("crypto/cnxk: support EdDSA")
Cc: sta...@dpdk.org
Signed-off-by: Gowrishankar Muthukrishnan
---
drivers/crypto/cnxk/cnxk_ae.h | 2 +-
1 file changed, 1 insertion(+)
This change fixes coverity issue on overflowed constant.
Bad phys address should be checked before using it in
descriptor.
Coverity issue: 457462, 457465
Fixes: 10702138f1a ("crypto/virtio: support asymmetric RSA")
Signed-off-by: Gowrishankar Muthukrishnan
---
drivers/crypto/virtio/virtio_crypt
On Tue, May 13, 2025 at 11:00 AM Bruce Richardson
wrote:
>
> Add a script to buildtools to report the minimum meson version given in
> our meson.build file. Then use this script in two ways:
>
> 1. in the .ci/linux-setup.sh script, use the auto-determined minimum
>version to set up the CI, rat
68 matches
Mail list logo
