> From: Chengwen Feng [mailto:fengcheng...@huawei.com]
> Sent: Thursday, 11 April 2024 05.08
>
> Fix a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
> which will lead the hns3 NIC can't link up. The root cause is strict
> aliasing violation in rte_eth_linkstatus_set() with hns3
Morten Brørup would like to recall the message, "[PATCH] ethdev: fix strict
aliasing lead to link cannot be up".
> From: Chengwen Feng [mailto:fengcheng...@huawei.com]
> Sent: Thursday, 11 April 2024 05.08
>
> Fix a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
> which will lead the hns3 NIC can't link up. The root cause is strict
> aliasing violation in rte_eth_linkstatus_set() with hns3
From: "hepeng.0320"
mlx5_dev_close currently will set priv->sh->port[priv->dev_port -
1].nl_ih_port_id to RTE_MAX_ETHPORTS to avoid mlx5_dev_interrupt_nl_cb
to use the port's dev_private, because later the rte_eth_dev_close
will free the dev_private and set the pointer to NULL.
However, since ml
Recheck-request: iol-intel-Functional
On Wed, Apr 3, 2024 at 2:23 PM Tyler Retzlaff
wrote:
>
> Some libraries that could be built with MSVC were not being built.
>
> Switch from explicit include to exclude of libs to get immediate CI
> coverage of libraries that already work with MSVC Windows bu
Hi All,
On 2024/4/11 3:58, Tyler Retzlaff wrote:
> On Wed, Apr 10, 2024 at 07:54:27PM +0200, Morten Brørup wrote:
>>> From: Stephen Hemminger [mailto:step...@networkplumber.org]
>>> Sent: Wednesday, 10 April 2024 17.27
>>>
>>> On Wed, 10 Apr 2024 17:33:53 +0800
>>> fengchengwen wrote:
>>>
La
Fix a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
which will lead the hns3 NIC can't link up. The root cause is strict
aliasing violation in rte_eth_linkstatus_set() with hns3 driver, see
[1] for more details.
This commit use union to avoid such aliasing violation.
[1] Stric
Hi Morten,
On 2024/4/10 18:30, Morten Brørup wrote:
>> From: fengchengwen [mailto:fengcheng...@huawei.com]
>> Sent: Wednesday, 10 April 2024 11.34
>>
>> Hi All,
>>
>> We have a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
>> we've done some
>> research but haven't been able to
> >
> > Signed-off-by: Yutang Jiang
> > Signed-off-by: Yutang Jiang
> > ---
Looks like this patch is signed off by Yutang twice with two different emails.
Please remove one and submit again.
Thank you.
> From: Krishna Kanth Reddy
>
> Modified the Configuration file to use the latest ARM Cross-Compiler.
>
> Fixed the linker errors for the undefined references to the APIs
> isal_deflate_init, isal_deflate, isal_inflate_init, isal_inflate,
> isal_inflate_stateless, isal_deflate_stateless, isal_de
> -Original Message-
> From: Yutang Jiang
> Sent: Sunday, April 7, 2024 1:36 AM
> To: dev@dpdk.org
> Cc: patc...@amperecomputing.com; yutang.ji...@amperecomputing.com;
> jiangyut...@os.amperecomputing.com; Ruifeng Wang
> ; nd ; juraj.lin...@pantheon.tech
> Subject: [PATCH] config/arm: a
On Wed, 10 Apr 2024 19:54:27 +0200
Morten Brørup wrote:
> > Please send a patch to replace alias with union.
>
> +1
>
> Fixing this specific bug would be good.
>
> Instinctively, I think we should build with -fno-strict-aliasing, so the
> compiler doesn't make the same mistake with similar
Hello,
The DPDK Community Lab will be offline for a couple hours tomorrow,
April 11, starting at 16:00 UTC. We are doing some Community Lab
dashboard maintenance. We should be able to catch up on all
patchseries submitted during that time as soon as we're back online.
Thanks,
Patrick
On Wed, Apr 10, 2024 at 07:54:27PM +0200, Morten Brørup wrote:
> > From: Stephen Hemminger [mailto:step...@networkplumber.org]
> > Sent: Wednesday, 10 April 2024 17.27
> >
> > On Wed, 10 Apr 2024 17:33:53 +0800
> > fengchengwen wrote:
> >
> > > Last： We think there are two ways to solve this pro
> Automatically generated by devtools/cocci/rte_memcpy.cocci
>
> Signed-off-by: Stephen Hemminger
> ---
> app/test/commands.c | 1 -
> app/test/packet_burst_generator.c | 4 +--
> app/test/test_crc.c | 5 ++--
> app/test/test_cryptodev.c
> From: Stephen Hemminger [mailto:step...@networkplumber.org]
> Sent: Wednesday, 10 April 2024 17.27
>
> On Wed, 10 Apr 2024 17:33:53 +0800
> fengchengwen wrote:
>
> > Last： We think there are two ways to solve this problem.
> > 1. Add the compilation option '-fno-strict-aliasing' for hold DPDK
On Wed, Apr 10, 2024 at 09:27:10AM +0200, Mattias Rönnblom wrote:
> On 2024-04-08 17:27, Tyler Retzlaff wrote:
> >For next technboard meeting.
> >
> >On Sun, Apr 07, 2024 at 10:03:06AM -0700, Stephen Hemminger wrote:
> >>On Sun, 7 Apr 2024 13:07:06 +0200
> >>Morten Brørup wrote:
> >>
> From: M
On Wed, Apr 10, 2024 at 09:32:10AM +0200, Mattias Rönnblom wrote:
> On 2024-04-08 17:53, Morten Brørup wrote:
> >>From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> >>Sent: Monday, 8 April 2024 17.27
> >>
> >>For next technboard meeting.
> >>
> >>On Sun, Apr 07, 2024 at 10:03:06AM -0700,
On Wed, Apr 10, 2024 at 09:58:34AM +, Konstantin Ananyev wrote:
>
>
> > >
> > > > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> > > > > Sent: Monday, 8 April 2024 17.27
> > > > >
> > > > > For next technboard meeting.
> > > > >
> > > > > On Sun, Apr 07, 2024 at 10:03:06AM -07
https://bugs.dpdk.org/show_bug.cgi?id=1413
Bug ID: 1413
Summary: Build initial list of device capabilities required by
testsuites we are trying to write this year
Product: DPDK
Version: 24.07
Hardware: All
O
Honnappa approved this a couple weeks ago, but this was never emailed
out to the DTS mailing list, so doing that now.
1) Write ethdev testsuites:
Jumboframes:
https://git.dpdk.org/tools/dts/tree/test_plans/jumboframes_test_plan.rst
Mac Filter:
https://git.dpdk.org/tools/dts/tree/test_plans/mac_f
#
Attendees
* Patrick Robb
* Jeremy Spewock
* Paul Szczepanek
* Luca Vizzarro
* Juraj Linkeš
#
Minutes
==
On 4/10/2024 4:27 PM, Stephen Hemminger wrote:
> On Wed, 10 Apr 2024 17:33:53 +0800
> fengchengwen wrote:
>
>> Last： We think there are two ways to solve this problem.
>> 1. Add the compilation option '-fno-strict-aliasing' for hold DPDK project.
>> 2. Use union to avoid such aliasing in rte_eth_
On 4/10/2024 11:30 AM, Morten Brørup wrote:
>> From: fengchengwen [mailto:fengcheng...@huawei.com]
>> Sent: Wednesday, 10 April 2024 11.34
>>
>> Hi All,
>>
>> We have a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
>> we've done some
>> research but haven't been able to figure o
> Automatically generated by devtools/cocci/rte_memcpy.cocci
>
> Signed-off-by: Stephen Hemminger
> ---
> lib/cryptodev/rte_cryptodev.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
Acked-by: Akhil Goyal
> Automatically generated by devtools/cocci/rte_memcpy.cocci
>
> Signed-off-by: Stephen Hemminger
> ---
Acked-by: Akhil Goyal
> +To: David Hunt, Distributor maintainer
> +To: Radu Nicolau and Akhil Goyal, IPsec security gateway example maintainers
>
> > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> > Sent: Wednesday, 20 March 2024 16.37
> >
> > Move location of __rte_aligned(a) to new conventional location
On Wed, 10 Apr 2024 17:33:53 +0800
fengchengwen wrote:
> Last： We think there are two ways to solve this problem.
> 1. Add the compilation option '-fno-strict-aliasing' for hold DPDK project.
> 2. Use union to avoid such aliasing in rte_eth_linkstatus_set (please see
> above).
> PS: We prefer fi
This patch resolves a build error with GCC 13 and arm/aarch32 as
targets:
In function ‘mbuf_to_desc’,
inlined from ‘vhost_enqueue_async_packed’ at
../lib/vhost/virtio_net.c:1828:6,
inlined from ‘virtio_dev_rx_async_packed’ at
../lib/vhost/virtio_net.c:1842:6,
inlined from ‘
Hi everyone,
sending in a patch to resolve some build issues encountered with
GCC 13.2 when building against 32-bit targets. These issues were
originally reported by Luca Boccassi in one of his stable builds[1].
Although, I have noticed that this problem presents itself on
more versions (e.g. 21.1
On 10/04/2024 14:35, Juraj Linkeš wrote:
We should also consider that this would introduce a difference in API
between the interactive and non-interactive sessions. Do we want to do
this there as well?
Could definitely add it there as well. You are referring to
RemoteSession I presume, right?
On Wed, Apr 10, 2024 at 3:23 AM Morten Brørup
wrote:
>
> > From: Nicholas Pratte [mailto:npra...@iol.unh.edu]
> > Sent: Tuesday, 9 April 2024 19.28
> >
> > The previous implementation configures and allocates hugepage sizes
> > based on a system default. This can lead to two problems: overallocat
On Wed, Apr 10, 2024 at 1:27 PM Luca Vizzarro wrote:
>
> On 10/04/2024 07:53, Juraj Linkeš wrote:
> > I have a general question. What are these changes for? Do you
> > anticipate us needing this in the future? Wouldn't it be better to add
> > it only when we need it?
>
> It's been sometime since w
Hi Jie,
Consider the following examples:
1) flow create 0 ingress pattern eth / ipv4 proto is 17 / udp / end \
actions queue index 1 / end
2) flow create 0 ingress pattern eth / ipv4 / udp / end \
actions queue index 1 / end
Generally speaking, these two rules might be equivalent, with "pr
On Wed, Apr 10, 2024 at 12:53 PM Luca Vizzarro wrote:
>
> On 09/04/2024 20:12, Juraj Linkeš wrote:
> >> @@ -104,16 +108,15 @@ def pmd_scatter(self, mbsize: int) -> None:
> >> """
> >> testpmd = self.sut_node.create_interactive_shell(
> >> TestPmdShell,
> >> -
On Wed, Apr 10, 2024 at 12:49 PM Luca Vizzarro wrote:
>
> On 09/04/2024 17:37, Juraj Linkeš wrote:
> > As Jeremy pointed out, going forward, this is likely to become bloated
> > and moving it to params.py (for example) may be better.
> >
> > There's a lot of testpmd args here. I commented on the i
On Mon, Apr 08, 2024 at 09:49:54AM -0700, Stephen Hemminger wrote:
> Recent patch set for rte_memcpy, discovered that the
> mail addresses for yuying.zh...@intel.com and zhouguoy...@huawei.com
> could not be found.
>
> Signed-off-by: Stephen Hemminger
> ---
Acked-by: Bruce Richardson
If doing
From: Sunil Kumar Kori
Currently user passed RSS configuration is ignored via
rte_eth_dev_configure() API. Instead default RSS setup is done by
driver.
Adding handling for user passed RSS configuration too via
rte_eth_dev_configure().
Signed-off-by: Sunil Kumar Kori
---
drivers/net/cnxk/cnxk_
Hello,
10/04/2024 13:37, Jie Hai:
> Hi, all,
>
> I have some questions about the sub-options for ``VLAN`` and ``ETH`` item.
If it is not clear in the doxygen documentation,
please do not hesitate to submit a patch to make it more explicit.
> According to the documentation, ``has_vlan`` is sub-o
> From: Konstantin Ananyev [mailto:konstantin.anan...@huawei.com]
> Sent: Wednesday, 10 April 2024 12.35
>
> > > From: Konstantin Ananyev [mailto:konstantin.anan...@huawei.com]
> > > Sent: Tuesday, 9 April 2024 15.39
> > >
> > > > > From: David Marchand [mailto:david.march...@redhat.com]
> > > > >
On 10/04/2024 08:50, Juraj Linkeš wrote:
On Tue, Mar 26, 2024 at 8:04 PM Luca Vizzarro wrote:
This commit provides a state container for TestPmdShell. It currently
only indicates whether the packet forwarding has started
or not, and the number of ports which were given to the shell.
A remin
Hi, all,
I have some questions about the sub-options for ``VLAN`` and ``ETH`` item.
According to the documentation, ``has_vlan`` is sub-option of ``ETH``
item and it means that the pattern contains at least one vlan.
The ``VLAN`` item is used to match tagged packets and have some
sub-options suc
On 10/04/2024 08:41, Juraj Linkeš wrote:
@@ -723,7 +731,13 @@ def _start_application(self, get_privileged_command:
Callable[[str], str] | None
if self._app_args.app_params is None:
self._app_args.app_params = TestPmdParameters()
-self.number_of_ports = len(self
On 10/04/2024 07:53, Juraj Linkeš wrote:
I have a general question. What are these changes for? Do you
anticipate us needing this in the future? Wouldn't it be better to add
it only when we need it?
It's been sometime since we raised this task internally. This patch and
the next one arise from
On 09/04/2024 20:12, Juraj Linkeš wrote:
@@ -104,16 +108,15 @@ def pmd_scatter(self, mbsize: int) -> None:
"""
testpmd = self.sut_node.create_interactive_shell(
TestPmdShell,
-app_parameters=StrParams(
-"--mbcache=200 "
-
On 09/04/2024 17:37, Juraj Linkeš wrote:
As Jeremy pointed out, going forward, this is likely to become bloated
and moving it to params.py (for example) may be better.
There's a lot of testpmd args here. I commented on the implementation
of some of them. I didn't verify that the actual values ma
> diff --git a/dts/framework/testbed_model/linux_session.py
> b/dts/framework/testbed_model/linux_session.py
> index 5d24030c3d..37f5eacb21 100644
> --- a/dts/framework/testbed_model/linux_session.py
> +++ b/dts/framework/testbed_model/linux_session.py
> @@ -15,7 +15,7 @@
>
> from typing_extensio
> > From: Konstantin Ananyev [mailto:konstantin.anan...@huawei.com]
> > Sent: Tuesday, 9 April 2024 15.39
> >
> > > > From: David Marchand [mailto:david.march...@redhat.com]
> > > > Sent: Friday, 5 April 2024 16.46
> > > >
> > > > Mandate use of rte_eth_tx_prepare() in the mbuf Tx checksum offlo
> From: fengchengwen [mailto:fengcheng...@huawei.com]
> Sent: Wednesday, 10 April 2024 11.34
>
> Hi All,
>
> We have a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
> we've done some
> research but haven't been able to figure out why. We'd like the community's
> help.
>
> Env
On Mon, Apr 8, 2024 at 6:20 PM Jeremy Spewock wrote:
>
> On Wed, Apr 3, 2024 at 5:00 AM Juraj Linkeš
> wrote:
> >
> > On Tue, Mar 12, 2024 at 6:26 PM wrote:
> > >
> > > From: Jeremy Spewock
> > >
> > > The current implementation of consuming output from interactive shells
> > > relies on being
On Wed, Apr 10, 2024 at 11:51 AM Luca Vizzarro wrote:
>
> On 10/04/2024 10:15, Juraj Linkeš wrote:
> +
> +def value_only(metadata: dict[str, Any] = {}) -> dict[str, Any]:
> +"""Injects the value of the attribute as-is without flag. Metadata
> modifier for :func:`dataclasse
On Wed, Apr 10, 2024 at 11:34 AM Luca Vizzarro wrote:
>
> On 09/04/2024 15:56, Juraj Linkeš wrote:
> > On Thu, Mar 28, 2024 at 5:48 PM Jeremy Spewock wrote:
> >>
> >> I'm not sure if allowing None should be the solution for these shells
> >> as opposed to just supplying an empty parameter object.
> >
> > > > From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> > > > Sent: Monday, 8 April 2024 17.27
> > > >
> > > > For next technboard meeting.
> > > >
> > > > On Sun, Apr 07, 2024 at 10:03:06AM -0700, Stephen Hemminger wrote:
> > > > > On Sun, 7 Apr 2024 13:07:06 +0200
> > > > > Mo
On 10/04/2024 10:15, Juraj Linkeš wrote:
+
+def value_only(metadata: dict[str, Any] = {}) -> dict[str, Any]:
+"""Injects the value of the attribute as-is without flag. Metadata modifier for
:func:`dataclasses.field`."""
+return {**metadata, META_VALUE_ONLY: True}
These methods, on the
On Tue, Apr 9, 2024 at 5:14 PM Luca Vizzarro wrote:
>
> On 04/04/2024 10:25, Juraj Linkeš wrote:
> > Judging from the code, this patch seems like a convoluted way to implement:
> > 1. An association between an Argument and the corresponding
> > environment variable,
> > 2. A better way to add the
On 09/04/2024 15:56, Juraj Linkeš wrote:
On Thu, Mar 28, 2024 at 5:48 PM Jeremy Spewock wrote:
I'm not sure if allowing None should be the solution for these shells
as opposed to just supplying an empty parameter object. Maybe
something that could be done is the factory method in sut_node allo
Hi All,
We have a problem introduced by a compiler upgrade (from gcc10 to gcc12.3),
we've done some
research but haven't been able to figure out why. We'd like the community's
help.
Environment:
1. Source: DPDK 23.11
2. GCC: 12.3.1 [1]
3. Compiled with target kunpeng SoC (ARM64)
4. Run on kunpe
On Tue, Apr 9, 2024 at 6:28 PM Luca Vizzarro wrote:
>
> Thank you so much for your review Juraj!
>
You're welcome!
> >> +
> >> +def value_only(metadata: dict[str, Any] = {}) -> dict[str, Any]:
> >> +"""Injects the value of the attribute as-is without flag. Metadata
> >> modifier for :func:`
> From: Mattias Rönnblom [mailto:hof...@lysator.liu.se]
> Sent: Wednesday, 10 April 2024 09.32
>
> On 2024-04-08 17:53, Morten Brørup wrote:
> >> From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
> >> Sent: Monday, 8 April 2024 17.27
> >>
[...]
> >> Looks like we need to discuss this at
On Tue, Mar 26, 2024 at 8:04 PM Luca Vizzarro wrote:
>
> This commit provides a state container for TestPmdShell. It currently
> only indicates whether the packet forwarding has started
> or not, and the number of ports which were given to the shell.
>
A reminder, the commit message should explai
On Thu, Mar 28, 2024 at 5:49 PM Jeremy Spewock wrote:
>
> On Tue, Mar 26, 2024 at 3:04 PM Luca Vizzarro wrote:
> >
> > This commit provides a state container for TestPmdShell. It currently
> > only indicates whether the packet forwarding has started
> > or not, and the number of ports which were
On 2024-04-08 17:53, Morten Brørup wrote:
From: Tyler Retzlaff [mailto:roret...@linux.microsoft.com]
Sent: Monday, 8 April 2024 17.27
For next technboard meeting.
On Sun, Apr 07, 2024 at 10:03:06AM -0700, Stephen Hemminger wrote:
On Sun, 7 Apr 2024 13:07:06 +0200
Morten Brørup wrote:
From:
On 2024-04-08 17:27, Tyler Retzlaff wrote:
For next technboard meeting.
On Sun, Apr 07, 2024 at 10:03:06AM -0700, Stephen Hemminger wrote:
On Sun, 7 Apr 2024 13:07:06 +0200
Morten Brørup wrote:
From: Mattias Rönnblom [mailto:hof...@lysator.liu.se]
Sent: Sunday, 7 April 2024 11.32
On 2024-04
> From: Nicholas Pratte [mailto:npra...@iol.unh.edu]
> Sent: Tuesday, 9 April 2024 19.28
>
> The previous implementation configures and allocates hugepage sizes
> based on a system default. This can lead to two problems: overallocation of
> hugepages (which may crash the remote host), and configur
64 matches
Mail list logo
