18/07/2025 00:05, Thomas Monjalon:
> The main mlx5 doc (for net driver) is improved
> with better explanations, better organization, better readability,
> and will be a better skeleton for future detailed explanations.
>
> Maayan Kashani (1):
> doc: add testpmd command examples in mlx5 guide
>
18/07/2025 17:36, Stephen Hemminger:
> Looks better, it would be good to look at the table of contents
> (right column in html) to see if could be better in complex document
> like this.
Yes I've paid attention to the table of contents.
Thanks for the suggestions below.
> For example the Usage, d
#
July 17, 2025
Attendees
* Patrick Robb
* Manit Mahajan
* Dean Marx
* Luca Vizzarro
* Paul Szczepanek
#
Minutes
https://bugs.dpdk.org/show_bug.cgi?id=1757
Bug ID: 1757
Summary: Remove optional poetry docs build workflow
Product: DPDK
Version: 25.03
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: minor
https://bugs.dpdk.org/show_bug.cgi?id=1756
Bug ID: 1756
Summary: Document setup process for dual port NIC loopback
topology
Product: DPDK
Version: 25.03
Hardware: All
OS: All
Status: UNCONFIRMED
On 18/07/2025 17:13, Bruce Richardson wrote:
> On Fri, Jul 18, 2025 at 05:05:45PM +0100, Kevin Traynor wrote:
>> On 18/07/2025 16:19, Bruce Richardson wrote:
>>> On Fri, Jul 18, 2025 at 04:03:17PM +0100, Kevin Traynor wrote:
On 03/07/2025 15:31, Ciara Loftus wrote:
> The SSE rx and tx path
On Fri, Jul 18, 2025 at 11:05 AM Thomas Wilks wrote:
>
> Port over the rss_key_update, pmd_rss_reta and pmd_rss_hash
> test suites from old DTS into one file including all of the
> helper functions that are required by all of the test suites
> to work.
>
> The rss_key_update test cases verify that
Previously, upon a device close, the driver would only stop its queues,
whereas a device remove would cause the driver to free its resources.
However, expected behavior in DPDK is to have a device close free all of
its resources because ports are not reused.
This patch adds all device teardown fun
A quick follow-up: I now see that 'self.queue' is in fact a misnomer for an
array of queues and not a single queue index specification. It would be better
to name it 'queues'. But the issue stands: this array differs from GLOBAL table.
As far as I can tell from reading documentation for action 'r
Hi Thomas,
On Fri, 18 Jul 2025, Thomas Wilks wrote:
Hi all,
This is v3 of the RSS test suites which has been rebased onto the latest
patches.
I’m also requesting comments and some help with an issue where the packet RSS
queue and
the predicted RSS queue differ when running on ConnectX-6 NICs
https://bugs.dpdk.org/show_bug.cgi?id=1755
Bug ID: 1755
Summary: Re-evaluate the link requirement decorators on all
testsuites/testcases
Product: DPDK
Version: 25.03
Hardware: All
OS: All
Status:
Use null device to exercise ethdev start/stop in secondary process.
Signed-off-by: Stephen Hemminger
---
app/test/test_mp_secondary.c | 51 +---
1 file changed, 48 insertions(+), 3 deletions(-)
diff --git a/app/test/test_mp_secondary.c b/app/test/test_mp_secondar
When the pdump application is run print a warning.
Add release note about deprecation and mark the API's
as deprecated.
Signed-off-by: Stephen Hemminger
---
app/pdump/main.c | 3 +++
app/pdump/meson.build | 1 +
app/test-pmd/meson.build
Use the new port mirror API instead of pdump.
Signed-off-by: Stephen Hemminger
---
app/dumpcap/main.c | 443 +---
app/dumpcap/meson.build | 2 +-
2 files changed, 373 insertions(+), 72 deletions(-)
diff --git a/app/dumpcap/main.c b/app/dumpcap/main.c
i
Simple API and packet mirroring standalone tests.
Signed-off-by: Stephen Hemminger
---
app/test/meson.build | 1 +
app/test/test_ethdev_mirror.c | 286 ++
2 files changed, 287 insertions(+)
create mode 100644 app/test/test_ethdev_mirror.c
diff --git a
Add new commands to enable testing of port mirror functionality.
Signed-off-by: Stephen Hemminger
---
app/test-pmd/cmdline.c | 1 +
app/test-pmd/cmdline_mirror.c | 129
app/test-pmd/cmdline_mirror.h | 12 ++
app/test-pmd/me
The queue field is optional in pcapng received packet.
Use UINT16_MAX as flag value.
Signed-off-by: Stephen Hemminger
---
lib/pcapng/rte_pcapng.c | 7 ---
lib/pcapng/rte_pcapng.h | 4 ++--
2 files changed, 6 insertions(+), 5 deletions(-)
diff --git a/lib/pcapng/rte_pcapng.c b/lib/pcapng/rte
In new model, the packet was already copied, only need
to wrap it in pcapng format.
Signed-off-by: Stephen Hemminger
---
lib/pcapng/rte_pcapng.c | 178 +---
lib/pcapng/rte_pcapng.h | 27 +-
2 files changed, 120 insertions(+), 85 deletions(-)
diff --git a
Add names for new port mirror fields.
Signed-off-by: Stephen Hemminger
---
lib/mbuf/rte_mbuf_dyn.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/lib/mbuf/rte_mbuf_dyn.h b/lib/mbuf/rte_mbuf_dyn.h
index 865c90f579..72a31b22b3 100644
--- a/lib/mbuf/rte_mbuf_dyn.h
+++ b/lib/mbuf/rte_m
This adds new feature port mirroring to the ethdev layer.
Signed-off-by: Stephen Hemminger
---
config/rte_config.h | 1 +
lib/ethdev/ethdev_driver.h | 6 +
lib/ethdev/ethdev_private.c | 58 -
lib/ethdev/ethdev_private.h | 3 +
lib/ethdev/ethdev_trace.h
Need ability to allow process like dumpcap to make a ring ethdev
with a pre-existing ring. Do this via devargs so it can work
with hotplug.
It looked like the API rte_eth_from_ring() would work for this,
but it doesn't do the right thing. That API makes new vdev but
names the ring based on the par
Need to be able have multiple threads all using same
transmit queue when using SPAN.
Signed-off-by: Stephen Hemminger
---
drivers/net/ring/rte_eth_ring.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/drivers/net/ring/rte_eth_ring.c b/drivers/net/ring/rte_eth_ring.c
index
Best not to depend on other EAL headers to include stuff.
Some of the later changes had build failures on some platforms.
Signed-off-by: Stephen Hemminger
---
lib/ethdev/ethdev_private.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/lib/ethdev/ethdev_private.c b/lib/ethdev/ethdev_
This is a rework of how packet capture is done in DPDK.
The existing mechanism using callbacks has a number of problems:
- can't work when packets are sent and received in secondary process
because callbacks only function in the primary process
- requires "opt-in" from application
The new
Before this patch if secondary process called start/stop
it would only impact the secondary process, the ethdev on the
primary process was not started.
With this patch, when start/stop is called from secondary,
it calls the primary and does the operation there. The design
is generic, and we can la
On Wed, Jul 9, 2025 at 2:47 PM Ronak Doshi wrote:
>
> Index bitmask variable used was uint8_t, too small for bitmask with
> 9 or more queues. This patch changes it to uint16_t for now, to be
> same as in the Vmxnet3_MemoryRegion structure. This way txQueues
> can be lesser than rxQueues and have c
On Fri, Jul 18, 2025 at 05:05:45PM +0100, Kevin Traynor wrote:
> On 18/07/2025 16:19, Bruce Richardson wrote:
> > On Fri, Jul 18, 2025 at 04:03:17PM +0100, Kevin Traynor wrote:
> >> On 03/07/2025 15:31, Ciara Loftus wrote:
> >>> The SSE rx and tx paths will be removed from the i40e, iavf and ice
>
On 18/07/2025 16:19, Bruce Richardson wrote:
> On Fri, Jul 18, 2025 at 04:03:17PM +0100, Kevin Traynor wrote:
>> On 03/07/2025 15:31, Ciara Loftus wrote:
>>> The SSE rx and tx paths will be removed from the i40e, iavf and ice
>>> drivers in the 25.11 release. Each of these drivers have faster vecto
The specific placement of outer/inner VLAN tags in NIC descriptors
is configurable. Therefore, remove the assumption that if the L2Tag2
field is filled in, that the L2Tag1 must also be. Instead, check the
existing mbuf VLAN flags, and move tags and set flags as appropriate.
This fixes an issue wher
On Fri, 18 Jul 2025 00:05:40 +0200
Thomas Monjalon wrote:
> The main mlx5 doc (for net driver) is improved
> with better explanations, better organization, better readability,
> and will be a better skeleton for future detailed explanations.
>
> Maayan Kashani (1):
> doc: add testpmd command e
On Fri, 18 Jul 2025 00:05:43 +0200
Thomas Monjalon wrote:
> The documentation for net mlx5 is reworded and re-organized.
> Design, configuration, requirements and limitations
> are grouped per features for easier reading.
>
> New details are added, and more will come later.
>
> Signed-off-by: T
On Fri, Jul 18, 2025 at 04:03:17PM +0100, Kevin Traynor wrote:
> On 03/07/2025 15:31, Ciara Loftus wrote:
> > The SSE rx and tx paths will be removed from the i40e, iavf and ice
> > drivers in the 25.11 release. Each of these drivers have faster vector
> > paths (AVX2 and AVX-512) which have featur
On Fri, 18 Jul 2025 13:51:56 +0200
Dariusz Sosnowski wrote:
> NVIDIA NICs do not support matching on packet headers appearing
> after IP header if the received packet is an IP fragment.
> This patch updates the relevant documentation.
>
> Bugzilla ID: 1417
>
> Signed-off-by: Dariusz Sosnowski
Port over the rss_key_update, pmd_rss_reta and pmd_rss_hash
test suites from old DTS into one file including all of the
helper functions that are required by all of the test suites
to work.
The rss_key_update test cases verify that setting a new hash
key when Receive Side Scaling (RSS) will result
From: Alex Chapman
This patch adds the required functionality for the RSS key_update,
RETA, and hash test suites. This includes:
The setting of custom RETA values for routing packets to specific
queues.
The setting of the RSS mode on all ports, to specify how to hash
the packets.
The updating of
Hi all,
This is v3 of the RSS test suites which has been rebased onto the latest
patches.
I’m also requesting comments and some help with an issue where the packet RSS
queue and
the predicted RSS queue differ when running on ConnectX-6 NICs while being the
same on E810-C NICs.
I’m currently i
On 03/07/2025 15:31, Ciara Loftus wrote:
> The SSE rx and tx paths will be removed from the i40e, iavf and ice
> drivers in the 25.11 release. Each of these drivers have faster vector
> paths (AVX2 and AVX-512) which have feature parity with the soon to be
> removed SSE paths. In environments where
On Fri, Jul 18, 2025 at 03:33:46PM +0100, Bruce Richardson wrote:
> When processing cmdline arguments in DPDK, we always do so with very
> little context. So, for example, when processing the "-l" flag, we have
> no idea whether there will be later a --proc-type=secondary flag. We
> have all sorts
Use a utility function and macro to simplify the code for checking for
conflicting cmdline options. The checking can also be done at the
initial argument collating stage, shortening the argument
processing function which is very much on the long side.
Signed-off-by: Bruce Richardson
---
lib/eal/
Remove the separate function to check combinations of cmdline
parameters. Instead, just do those checks when parsing the parameters
since we have all info about what parameters are provided at that point.
Signed-off-by: Bruce Richardson
---
lib/eal/common/eal_common_memory.c | 3 +-
lib/eal/c
When rte_eal_init fails part way through, any saved EAL arguments need
to be freed, and the run_once flag needs to be set back to zero again.
The former task was never done on failure, and the latter was only done
on some occasions. Rework the error handling to always go to an err_out
label where c
DPDK traditionally has iterated through all args and processed them as
they appear in the commandline. The arg processing logic can be
simplified if instead we initially gather all arguments into a structure
which is then processed with the arguments dealt with in a fixed/known
order.
Signed-off-b
This table should allow us to parse all the eal args into a single
structure for later parsing in a fixed-order field basis. For those
elements that take multiple values, define a TAILQ and a callback to
process those elements.
Signed-off-by: Bruce Richardson
---
lib/eal/common/eal_common_option
To simplify future rework of the EAL arg handling, add a long-option
equivalent for each short option that doesn't already have one.
When updating the docs with the new long options, standardize the format
of options which have both short and long variants, and drop the
deprecated service-coremask
When the arguments passed to argparse include -h/--help then usage
information is automatically printed. Provide the capability for the
user to supply their own help function for this.
Signed-off-by: Bruce Richardson
---
doc/guides/prog_guide/argparse_lib.rst | 16
lib/argparse/
Make the function to print out the help text for an argparse object a
public function, which takes as a new parameter the file stream on which
to print. This can be used in future to allow application to extend
their own help information.
Signed-off-by: Bruce Richardson
---
lib/argparse/rte_argp
Introduce a string define for the currently running OS, or execution
environment.
Originally, with old make build system, CONFIG_RTE_EXEC_ENV used to hold
this name string, but the variable seems to have been missed in the
meson build system, until commit cadb255e25d6 ("eal: add OS defines for
C c
When processing cmdline arguments in DPDK, we always do so with very
little context. So, for example, when processing the "-l" flag, we have
no idea whether there will be later a --proc-type=secondary flag. We
have all sorts of post-arg-processing checks in place to try and catch
these scenarios.
On Fri, Jul 18, 2025 at 3:06 PM Bruce Richardson
wrote:
>
> On Fri, Jul 18, 2025 at 02:45:48PM +0200, David Marchand wrote:
> > On Fri, Jul 18, 2025 at 2:14 PM Bruce Richardson
> > wrote:
> > > diff --git a/drivers/net/virtio/meson.build
> > > b/drivers/net/virtio/meson.build
> > > index d3caa3a
On Fri, Jul 18, 2025 at 4:23 AM Bruce Richardson
wrote:
>
> On Thu, Jul 17, 2025 at 05:03:13PM -0400, Dean Marx wrote:
> > I've created a v1 of a QinQ test suite around the set of test cases
> > discussed earlier (which is not set in stone, and I expect it to
> > change significantly across many f
On Fri, Jul 18, 2025 at 02:45:48PM +0200, David Marchand wrote:
> On Fri, Jul 18, 2025 at 2:14 PM Bruce Richardson
> wrote:
> > diff --git a/drivers/net/virtio/meson.build b/drivers/net/virtio/meson.build
> > index d3caa3a3b4..a9ff87e863 100644
> > --- a/drivers/net/virtio/meson.build
> > +++ b/dr
On Fri, Jul 18, 2025 at 2:14 PM Bruce Richardson
wrote:
> diff --git a/drivers/net/virtio/meson.build b/drivers/net/virtio/meson.build
> index d3caa3a3b4..a9ff87e863 100644
> --- a/drivers/net/virtio/meson.build
> +++ b/drivers/net/virtio/meson.build
> @@ -31,7 +31,7 @@ if arch_subdir == 'x86'
>
On Fri, Jul 18, 2025 at 02:21:45PM +0200, David Marchand wrote:
> On Fri, Jul 18, 2025 at 2:14 PM Bruce Richardson
> wrote:
> >
> > DPDK now recommends compiler versions GCC 8+, and Clang 7+, so remove
> > checks and workarounds for versions old than these.
> >
> > Signed-off-by: Bruce Richardson
On Fri, Jul 18, 2025 at 2:14 PM Bruce Richardson
wrote:
>
> DPDK now recommends compiler versions GCC 8+, and Clang 7+, so remove
> checks and workarounds for versions old than these.
>
> Signed-off-by: Bruce Richardson
> ---
> config/meson.build | 5
> drivers/event/meson
DPDK now recommends compiler versions GCC 8+, and Clang 7+, so remove
checks and workarounds for versions old than these.
Signed-off-by: Bruce Richardson
---
config/meson.build | 5
drivers/event/meson.build | 5 +---
drivers/meson.build | 2 +-
On Thu, Jul 17, 2025 at 8:55 AM hailinx wrote:
>
> Signed-off-by: hailinx
> ---
> doc/guides/nics/ixgbe.rst | 21 +
> 1 file changed, 21 insertions(+)
>
> diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
> index a03ec7a7e8..6380e18feb 100644
> --- a/doc/guid
Hello,
On Fri, Jul 18, 2025 at 12:52 PM Bruce Richardson
wrote:
>
> On Thu, Jul 17, 2025 at 02:16:04PM +0800, hailinx wrote:
> > Signed-off-by: hailinx
> > ---
> > doc/guides/nics/ixgbe.rst | 21 +
> > 1 file changed, 21 insertions(+)
> Acked-by: Bruce Richardson
>
>
> >
>
Hi,
(please see below)
On Fri, 18 Jul 2025, Gokul K R (MS/ETA7-ETAS) wrote:
Hi Team,
I’m currently working with the dpdk-burst-replay tool and encountered an issue
during execution. Below are the details:
NVIDIA NICs do not support matching on packet headers appearing
after IP header if the received packet is an IP fragment.
This patch updates the relevant documentation.
Bugzilla ID: 1417
Signed-off-by: Dariusz Sosnowski
---
Depends-on: series-35735 ("rework mlx5 guide")
FYI - I added Bugzilla t
On Tue, Jul 15, 2025 at 07:14:49AM -0700, Andre Muezerie wrote:
> The linker parameters to use with MSVC and Clang differ.
> Showing explicitly what to use with each in the documentation.
>
> Signed-off-by: Andre Muezerie
> ---
> doc/guides/platform/mlx5.rst | 27 +++
> 1
On Thu, Jul 17, 2025 at 02:16:04PM +0800, hailinx wrote:
> Signed-off-by: hailinx
> ---
> doc/guides/nics/ixgbe.rst | 21 +
> 1 file changed, 21 insertions(+)
Acked-by: Bruce Richardson
>
> diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
> index a03ec7a7
On Thu, Jul 17, 2025 at 02:16:03PM +0800, hailinx wrote:
> Signed-off-by: hailinx
> ---
Acked-by: Bruce Richardson
The tables in this file really need pruning in next release. They stretch
all the way back almost 10 years!
> doc/guides/nics/i40e.rst | 4
> 1 file changed, 4 insertions(+)
On Fri, Jul 18, 2025 at 10:00:08AM +0800, hailinx wrote:
> Signed-off-by: hailinx
> ---
> doc/guides/nics/ice.rst | 23 +--
> 1 file changed, 21 insertions(+), 2 deletions(-)
>
Acked-by: Bruce Richardson
For next release update, I think we can prune the version tables, remov
Modification of default values should be limited to E610,
as 82599 handles raw IP well.
Fixes: c81daae2383a (net/ixgbe: fix port mask default value in filter)
Cc: sta...@dpdk.org
Signed-off-by: Yuan Wang
---
drivers/net/intel/ixgbe/ixgbe_flow.c | 6 --
1 file changed, 4 insertions(+), 2 del
Hi Team,
I’m currently working with the dpdk-burst-replay tool and encountered an issue
during execution. Below are the details:
Observation:
During replay, we received the following informational message:
port 0 is not on the good numa id (-1)
As per the DPDK mail
E610 supports SCTP port in FDIR filter, so add it to the support list.
Fixes: 86e19565f5e2 (net/ixgbe: fix SCTP port support)
Cc: sta...@dpdk.org
Signed-off-by: Yuan Wang
---
drivers/net/intel/ixgbe/ixgbe_flow.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/net/i
On Thu, Jul 17, 2025 at 05:03:13PM -0400, Dean Marx wrote:
> I've created a v1 of a QinQ test suite around the set of test cases
> discussed earlier (which is not set in stone, and I expect it to
> change significantly across many future versions.) The PASS/FAIL
> values can be mostly disregarded i
67 matches
Mail list logo