Coverity issue: 385414, 374016
Fixes: c836a7ba33e ("net/octeon_ep: support mailbox between VF and PF")
Fixes: bb5b5bf1e5c ("net/enetfec: support queue configuration")
Signed-off-by: Ariel Otilibili
--
Cc: sta...@dpdk.org
Cc: Vamsi Attunuru
Cc: Apeksha Gupta
Cc: Sachin Saxena
---
drivers/net/en
Hello,
This patch clears out the Coverity issues 385414 & 374016.
Thank you,
Ariel Otilibili (1):
net/{octeon_ep,enetfec}: remove unused value
drivers/net/enetfec/enet_ethdev.c | 1 -
drivers/net/octeon_ep/otx_ep_mbox.c | 3 ---
2 files changed, 4 deletions(-)
--
2.47.1
> On Wed, 18 Dec 2024 14:30:36 +0800
> Chaoyong He wrote:
>
> > +int
> > +nfp_net_rx_burst_mode_get(struct rte_eth_dev *eth_dev,
> > + uint16_t queue_id __rte_unused,
> > + struct rte_eth_burst_mode *mode)
> > +{
> > + eth_rx_burst_t pkt_burst;
> > +
> > + pkt_burst = eth_
On 2024-12-18 09:59, Maxime Coquelin wrote:
Hi,
On 12/18/24 08:34, Wangyunjian(wangyunjian,TongTu) wrote:
-Original Message-
From: Maxime Coquelin [mailto:maxime.coque...@redhat.com]
Sent: Tuesday, December 17, 2024 11:33 PM
To: dev@dpdk.org
Cc: Olivier Matz ; Maxime Gouin
; Maxime Coqu
The cleanup loop would derefence the dpio_dev after freeing.
Use TAILQ_FOREACH_SAFE to fix that.
Found by building with sanitizer undefined flag.
Fixes: e55d0494ab98 ("bus/fslmc: support secondary process")
Cc: shreyansh.j...@nxp.com
Cc: sta...@dpdk.org
Signed-off-by: Stephen Hemminger
---
drive
10/12/2024 21:41, Stephen Hemminger:
> Change of maintainers for next-net tree.
>
> Signed-off-by: Stephen Hemminger
> Acked-by: Ferruh Yigit
> ---
> Next-net Tree
> -M: Ferruh Yigit
> -M: Andrew Rybchenko
> +M: Stephen Hemminger
> T: git://dpdk.org/next/dpdk-next-net
Applied, thanks all f
On 2024-12-16 10:49, David Marchand wrote:
On Mon, Dec 16, 2024 at 10:42 AM Burakov, Anatoly
wrote:
On 12/5/2024 6:57 PM, David Marchand wrote:
As I had reported in rc2, the lcore variables allocation have a
noticeable impact on applications consuming DPDK, even when such
applications does no
This issue was flagged by MSVC warning below:
drivers\common\idpf\base/virtchnl2.h(269): warning C4309:
'initializing': truncation of constant value
The problem is that 64-bit numbers are initialized in an enum.
The C11 standard states: The expression that defines the value of an
enumeration
On Tue, 17 Dec 2024 09:59:49 +0100
David Marchand wrote:
> The RNG is documented as being seeded as part of EAL init.
>
> Move the initialisation (seeding) helper out of a constructor and
> call it explicitly from rte_eal_init() as it was done before commit
> 3f002f069612 ("eal: replace libc-bas
On Wed, 18 Dec 2024 10:20:47 +0100
Olivier Matz wrote:
> Hi,
>
> On Wed, Dec 18, 2024 at 09:59:05AM +0100, Maxime Coquelin wrote:
> > Hi,
> >
> > On 12/18/24 08:34, Wangyunjian(wangyunjian,TongTu) wrote:
> > > > -Original Message-
> > > > From: Maxime Coquelin [mailto:maxime.coque...@
Hi,
On Wed, Dec 18, 2024 at 09:59:05AM +0100, Maxime Coquelin wrote:
> Hi,
>
> On 12/18/24 08:34, Wangyunjian(wangyunjian,TongTu) wrote:
> > > -Original Message-
> > > From: Maxime Coquelin [mailto:maxime.coque...@redhat.com]
> > > Sent: Tuesday, December 17, 2024 11:33 PM
> > > To: dev@d
On 12/17/2024 9:59 AM, David Marchand wrote:
Now that the per-lcore state was moved into a lcore variable,
there is no reason to align a per-lcore state on a cache line to avoid
false sharing.
Remove this alignment and save a few bytes.
Fixes: 130643319579 ("power: keep per-lcore state in lcore
On 12/17/2024 9:59 AM, David Marchand wrote:
The lcore variable in this code unit is only used through
rte_power_ethdev_pmgmt_queue_*() public symbols.
Defer the unconditional lcore variable allocation in those symbols.
Fixes: 130643319579 ("power: keep per-lcore state in lcore variable")
Cc: s
On 12/17/2024 9:59 AM, David Marchand wrote:
The lcore variable in this code unit is only used through
rte_power_monitor*() public symbols.
Defer the unconditional lcore variable allocation in those symbols.
Fixes: 18b5049ab4fe ("eal/x86: keep power intrinsics state in lcore variable")
Cc: sta.
On Fri, Dec 13, 2024 at 09:15:40AM -0800, Stephen Hemminger wrote:
> On Fri, 13 Dec 2024 09:08:22 -0800
> Andre Muezerie wrote:
>
> > + struct rte_ipv6_addr *ip_batch =
> > + (struct rte_ipv6_addr *)rte_malloc("ip_batch",
> > + sizeof(struct rte_ipv6_addr) *
Test lpm6_perf_autotest was hitting a stack overflow on Windows
with both MSVC and Clang.
The fix is to move some of the data from the stack to the heap.
Signed-off-by: Andre Muezerie
---
app/test/test_lpm6_perf.c | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git
On Tue, 17 Dec 2024 20:39:12 +
bugzi...@dpdk.org wrote:
> https://bugs.dpdk.org/show_bug.cgi?id=1605
>
> Bug ID: 1605
>Summary: lib/eal build failure with address and undefined
> behavior sanitizers
>Product: DPDK
>Version: 2
On 2024-12-18 17:35, Stephen Hemminger wrote:
On Tue, 17 Dec 2024 09:59:49 +0100
David Marchand wrote:
The RNG is documented as being seeded as part of EAL init.
Move the initialisation (seeding) helper out of a constructor and
call it explicitly from rte_eal_init() as it was done before comm
Hi,
On 12/18/24 08:34, Wangyunjian(wangyunjian,TongTu) wrote:
-Original Message-
From: Maxime Coquelin [mailto:maxime.coque...@redhat.com]
Sent: Tuesday, December 17, 2024 11:33 PM
To: dev@dpdk.org
Cc: Olivier Matz ; Maxime Gouin
; Maxime Coquelin
Subject: [PATCH] net/virtio: fix Rx che
On 12/17/2024 9:59 AM, David Marchand wrote:
Add an assert to double check the passed handle is not NULL, as it
points at an initialisation/allocation issue prior to accessing this
lcore variable.
Signed-off-by: David Marchand
---
lib/eal/include/rte_lcore_var.h | 2 ++
1 file changed, 2 ins
https://bugs.dpdk.org/show_bug.cgi?id=1606
Bug ID: 1606
Summary: flow_filtering/flow_filtering_mismatch_rule: Some
mismatch rule packets match the rule
Product: DPDK
Version: 24.11
Hardware: x86
OS: All
delete port tables in host.
Signed-off-by: Junlong Wang
---
drivers/net/zxdh/zxdh_ethdev.c | 18 ++
drivers/net/zxdh/zxdh_msg.h| 1 +
drivers/net/zxdh/zxdh_np.c | 103 +
drivers/net/zxdh/zxdh_np.h | 9 +++
drivers/net/zxdh/zxdh_tables.c | 33
provided dev simple rx implementations.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 1 +
doc/guides/nics/zxdh.rst | 1 +
drivers/net/zxdh/zxdh_ethdev.c| 1 +
drivers/net/zxdh/zxdh_rxtx.c | 313 ++
drivers/net/zxdh/zxdh_r
provided link info update, set link up /down,
and link intr.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 2 +
doc/guides/nics/zxdh.rst | 3 +
drivers/net/zxdh/meson.build | 1 +
drivers/net/zxdh/zxdh_ethdev.c | 14 ++-
drivers/net/zxdh/zxdh_eth
(np)network processor release resources in host.
Signed-off-by: Junlong Wang
---
drivers/net/zxdh/zxdh_ethdev.c | 48
drivers/net/zxdh/zxdh_np.c | 470 +
drivers/net/zxdh/zxdh_np.h | 107
3 files changed, 625 insertions(+)
diff --git a/driv
dev start/stop implementations, start/stop the rx/tx queues.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 2 +
doc/guides/nics/zxdh.rst | 2 +
drivers/net/zxdh/zxdh_ethdev.c| 61 +
drivers/net/zxdh/zxdh_pci.c | 24
drivers/
rx/tx queue setup and intr enable implementations.
Signed-off-by: Junlong Wang
---
drivers/net/zxdh/zxdh_ethdev.c | 4 +
drivers/net/zxdh/zxdh_queue.c | 149 +
drivers/net/zxdh/zxdh_queue.h | 33
3 files changed, 186 insertions(+)
diff --git a/drive
V4:
- resolved ci compile issues.
V3:
- use rte_zmalloc and rte_calloc to avoid memset.
- remove unnecessary initialization, which first usage will set.
- adjust some function which is always return 0, changed to void
and skip the ASSERTION later.
- resolved some WARNING:MACRO_ARG_U
provided promiscuous/allmulticast ops.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 2 +
doc/guides/nics/zxdh.rst | 2 +
drivers/net/zxdh/zxdh_ethdev.c | 21 ++-
drivers/net/zxdh/zxdh_ethdev.h | 2 +
drivers/net/zxdh/zxdh_ethdev_ops.c | 132 ++
provided rss hash config/update, reta update/get ops.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 3 +
doc/guides/nics/zxdh.rst | 1 +
drivers/net/zxdh/zxdh_ethdev.c | 52
drivers/net/zxdh/zxdh_ethdev.h | 3 +
drivers/net/zxdh/zxdh_ethdev_op
provided mac set/add/remove ops.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 2 +
doc/guides/nics/zxdh.rst | 2 +
drivers/net/zxdh/zxdh_common.c | 24 +++
drivers/net/zxdh/zxdh_common.h | 1 +
drivers/net/zxdh/zxdh_ethdev.c | 32 +++-
driver
provided dev simple tx implementations.
Signed-off-by: Junlong Wang
---
drivers/net/zxdh/meson.build | 1 +
drivers/net/zxdh/zxdh_ethdev.c | 21 ++
drivers/net/zxdh/zxdh_queue.h | 26 ++-
drivers/net/zxdh/zxdh_rxtx.c | 396 +
drivers/net/zxdh/zxdh_rxtx.h
provided vlan filter, vlan offload ops.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 3 +
doc/guides/nics/zxdh.rst | 3 +
drivers/net/zxdh/zxdh_ethdev.c | 40 +-
drivers/net/zxdh/zxdh_ethdev_ops.c | 223 +
drivers/net/zxd
mtu update ops implementations.
Signed-off-by: Junlong Wang
---
doc/guides/nics/features/zxdh.ini | 1 +
doc/guides/nics/zxdh.rst | 2 +
drivers/net/zxdh/zxdh_ethdev.c | 5 ++
drivers/net/zxdh/zxdh_ethdev_ops.c | 78 ++
drivers/net/zxdh/zxdh_ethdev_o
Implement the burst mode get operation functions for both Rx and Tx.
Signed-off-by: Chaoyong He
Reviewed-by: Peng Zhang
---
v2:
* Replace 'snprintf()' with 'strlcpy()'.
---
drivers/net/nfp/nfp_ethdev.c| 2 ++
drivers/net/nfp/nfp_ethdev_vf.c | 2 ++
drivers/net/nfp/nfp_rxtx.c | 46 +++
On Wed, 18 Dec 2024 15:46:35 -0800
Joshua Washington wrote:
> From: Praveen Kaligineedi
>
> Allocating QPL for an RX queue might fail if enough contiguous IOVA
> memory cannot be allocated. However, the only requirement for QPL
> for RX is that each 4K buffer be IOVA contiguous, not the entire
> > On Wed, 4 Dec 2024 06:21:00 +
> > Chaoyong He wrote:
> >
> > > > The definition of what a "dedicated queue" is a bit confusing.
> > > > If it is only for LACP packets, it should never need to be very big.
> > > > Only under a mis-configuration and DoS kind of flood should there
> > > > eve
The hdr->csum_start does two successive reads from user space to read a
variable length data structure. The result overflow if the data structure
changes between the two reads.
To fix this, we can prevent double fetch issue by copying virtio_hdr to
the temporary variable.
Fixes: 4dc4e33ffa10 ("ne
This RFC introduces a new API, rte_event_eth_rx_adapter_queues_add(),
designed to enhance the flexibility of configuring multiple Rx queues
in eventdev Rx adapter.
The existing rte_event_eth_rx_adapter_queue_add() API supports adding
multiple queues by specifying rx_queue_id = -1, but it lacks the
This RFC introduces a new API, rte_event_eth_rx_adapter_queues_add(),
designed to enhance the flexibility of configuring multiple Rx queues
in eventdev Rx adapter.
The existing rte_event_eth_rx_adapter_queue_add() API supports adding
multiple queues by specifying rx_queue_id = -1, but it lacks the
>This RFC introduces a new API, rte_event_eth_rx_adapter_queues_add(),
>designed to enhance the flexibility of configuring multiple Rx queues
>in eventdev Rx adapter.
>
>The existing rte_event_eth_rx_adapter_queue_add() API supports adding
>multiple queues by specifying rx_queue_id = -1, but it lac
From: Praveen Kaligineedi
Allocating QPL for an RX queue might fail if enough contiguous IOVA
memory cannot be allocated. However, the only requirement for QPL
for RX is that each 4K buffer be IOVA contiguous, not the entire
QPL. Therefore, use malloc to allocate 4K buffers if the allocation
usin
42 matches
Mail list logo
