On Tue, Feb 11, 2025 at 02:20:51PM +0100, David Marchand wrote:
> On Thu, Jan 30, 2025 at 6:27 PM Bruce Richardson
> wrote:
> >
> > Add in two extra PCI device id's for future HW versions to be
> > supported by idxd driver. This aligns with the definitions present
> > in kernel idxd driver [1].
>
On Tue, Jan 21, 2025 at 5:36 PM Bruce Richardson
wrote:
>
> The modifying of a global register value, which has an affect on multiple
> ports is an important change to HW. However, the actual details of that
> change is only relevant to developers and not general users. Therefore
> reduce the log
[AMD Official Use Only - AMD Internal Distribution Only]
Acked-by: Selwin Sebastian
-Original Message-
From: K.E., Jesna
Sent: Friday, January 24, 2025 1:00 PM
To: dev@dpdk.org
Cc: Sebastian, Selwin ; K.E., Jesna
Subject: [PATCH v1] net/axgbe: support TSO
Added TSO(Transmit Segmentati
On Thu, Jan 30, 2025 at 6:27 PM Bruce Richardson
wrote:
>
> Add in two extra PCI device id's for future HW versions to be
> supported by idxd driver. This aligns with the definitions present
> in kernel idxd driver [1].
>
> [1]
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/
On Mon, Feb 10, 2025 at 05:19:57PM +, Bruce Richardson wrote:
> The Intel oneAPI DPC++/C++ Compiler (icx), issues warnings on build when
> the "-march=native", or other configured global "-march" flag, is
> overridden to "skylake-avx512", when compiling AVX-512 code.
>
> Allow building with ic
On Tue, 11 Feb 2025 23:18:34 +0800
"11" wrote:
> Hi Stephen，
>
> For rsic-v,windows freebsd plaform, I need to limit compile meson file what
> we not support ?
>
> Regards Wenbo
You can stick to linux for first version.
I assume issue with risc-v is byte order.
All this can be blocked in mes
On 30/01/2025 17:26, Bruce Richardson wrote:
Add in two extra PCI device id's for future HW versions to be
supported by idxd driver. This aligns with the definitions present
in kernel idxd driver [1].
[1]https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/dma/idxd/reg
On Tue, 11 Feb 2025 23:06:10 +0800
Wenbo Cao wrote:
> For This patchset just to support the basic chip init work
> and user can just found the eth_dev, but can't control more.
> For Now just support 2*10g nic,the chip can support
> 2*10g,4*10g,4*1g,8*1g,8*10g.
> The Feature rx side can support rx
On Tue, 11 Feb 2025 16:49:58 +0800
Wenbo Cao wrote:
> +static void rnp_rx_queue_release(void *_rxq)
> +{
> + struct rnp_rx_queue *rxq = (struct rnp_rx_queue *)_rxq;
> +
> + PMD_INIT_FUNC_TRACE();
> +
> + if (rxq) {
> + rnp_rx_queue_release_mbuf(rxq);
> + if (rx
On Tue, 11 Feb 2025 16:50:02 +0800
Wenbo Cao wrote:
> static int
> +rnp_mbx_fw_reply_handler(struct rnp_eth_adapter *adapter,
> + struct rnp_mbx_fw_cmd_reply *reply)
> +{
> + struct rnp_mbx_req_cookie *cookie;
> +
> + RTE_SET_USED(adapter);
> + /* dbg_here; */
> +
On Tue, 11 Feb 2025 16:49:58 +0800
Wenbo Cao wrote:
> diff --git a/drivers/net/rnp/base/rnp_hw.h b/drivers/net/rnp/base/rnp_hw.h
> index da2c1d3b0e..ea4e1b8d83 100644
> --- a/drivers/net/rnp/base/rnp_hw.h
> +++ b/drivers/net/rnp/base/rnp_hw.h
> @@ -120,6 +120,9 @@ struct rnp_hw {
> bool lan
On Tue, 11 Feb 2025 16:49:58 +0800
Wenbo Cao wrote:
> diff --git a/drivers/net/rnp/base/meson.build
> b/drivers/net/rnp/base/meson.build
> index b9db033ff6..c2ef0d05be 100644
> --- a/drivers/net/rnp/base/meson.build
> +++ b/drivers/net/rnp/base/meson.build
> @@ -7,6 +7,7 @@ sources = [
>
On Tue, 11 Feb 2025 16:49:54 +0800
Wenbo Cao wrote:
> +static inline u32
> +rnp_reg_read32(void *base, size_t offset)
> +{
> + unsigned int v = rte_read32(((u8 *)base + offset));
> +
> + RNP_PMD_REG_LOG(DEBUG, "offset=0x%08lx val=0x%04x",
> + (unsigned long)offset, v);
On Tue, 11 Feb 2025 16:49:54 +0800
Wenbo Cao wrote:
> diff --git a/drivers/net/rnp/base/meson.build
> b/drivers/net/rnp/base/meson.build
> new file mode 100644
> index 00..9ea88c3bff
> --- /dev/null
> +++ b/drivers/net/rnp/base/meson.build
> @@ -0,0 +1,22 @@
> +# SPDX-License-Identifier:
On Tue, 11 Feb 2025 23:06:10 +0800
Wenbo Cao wrote:
> For This patchset just to support the basic chip init work
> and user can just found the eth_dev, but can't control more.
> For Now just support 2*10g nic,the chip can support
> 2*10g,4*10g,4*1g,8*1g,8*10g.
> The Feature rx side can support rx
Depending on the scenario, a user may want to run testpmd with
pre-canned startup commands either with or without echoing of those
commands as they are executed. To observe progress of each command, or
to help determine what command an error response is printed for, echoing
is useful. When running
On Fri, Jan 24, 2025 at 7:27 PM Stephen Hemminger
wrote:
>
> With GCC 15, the compiler has changed the default behavior when
> initialization is used for aggregate variables. The new default
> is to follow the standard (C23) and not initialize everything by
> default. This breaks assumptions in so
On Mon, Feb 10, 2025 at 6:19 PM Bruce Richardson
wrote:
>
> The "Intel C++ Compiler" (icc) has been replaced by the newer clang-based
> "Intel oneAPI DPC++/C++ Compiler" (icx) compiler. DPDK compilation has
> also not been tested recently with the icc compiler, so let's remove doc
> and code refe
Recheck-request: loongarch-unit-testing
On Mon, Feb 10, 2025 at 10:32 PM David Marchand
wrote:
>
> As I had reported in 24.11-rc2, the lcore variables allocation have a
> noticeable impact on applications consuming DPDK, even when such
> applications does not use DPDK, or use features associated to
> some lcore variables.
>
> While the
> v26:
> - try to ensure all patches gets compiled incrementally.
Series Acked-by: Akhil Goyal
Updated the patch title while merging and fix release notes.
Applied to dpdk-next-crypto
Thanks.
>
> v25:
> - replace the ``__rte_packed`` macro.
> - fix an extra line at end of file in patch 11.
>
On Tue, 11 Feb 2025 16:49:50 +0800
Wenbo Cao wrote:
> For This patchset just to support the basic chip init work
> and user can just found the eth_dev, but can't control more.
> For Now just support 2*10g nic,the chip can support
> 2*10g,4*10g,4*1g,8*1g,8*10g.
> The Feature rx side can support rx
Will acknowledge this may be more of a meson question and less of a DPDK
issue but things didn't seem to work as expected and had to tweak the
config/meson.build to want to start with the DPDK team.
Our project is cmake based and we use fetch content to pull in our external
dependencies (eg dpdk,
On Mon, Feb 3, 2025 at 10:17 AM Luca Vizzarro wrote:
>
> Make Port models standalone, so that they can be directly manipulated by
> the test suites as needed. Moreover, let them handle themselves and
> retrieve the logical name and mac address in autonomy.
>
> Signed-off-by: Luca Vizzarro
Review
On Mon, Feb 3, 2025 at 10:18 AM Luca Vizzarro wrote:
>
> Enforce separation of concerns by letting test runs being isolated
> through a new TestRun class and respective module. This also means that
> any actions taken on the nodes must be handled exclusively by the test
> run. An example being the
There is a performance failure for Intel E810 for this patch, but it is
coming from dpdk mainline, not this patchseries. I think something was
merged recently which bumped up the variance on forwarding because the
periodic runs on the E810 for DPDK 24.11 are stable, but all the current
runs on new
Compiling with MSVC results in warnings like the one below:
app/test-pmd/util.c(201): warning C4334: '<<': result of 32-bit shift
implicitly converted to 64 bits (was 64-bit shift intended?)
Signed-off-by: Andre Muezerie
---
app/test-pmd/cmdline.c | 4 ++--
app/test-pmd/testpmd.c | 2 +-
ap
Compiling with MSVC results in the warning below:
app/test/test_alarm.c(54): warning C4090: 'function':
different '_Atomic' qualifiers
The fix is to use a macro to explicitly drop the qualifier.
Signed-off-by: Andre Muezerie
---
app/test/test_alarm.c | 12 ++--
1 file changed, 6 in
Compiling with MSVC results in warnings like the one below:
app/test-pmd/csumonly.c(1085): warning C4477: 'printf' : format string
'%d' requires an argument of type 'int',
but variadic argument 1 has type 'uint64_t'
Signed-off-by: Andre Muezerie
---
app/test-pmd/csumonly.c | 12 ++--
Compiling with MSVC results in warnings like below:
app/test-pmd/cmdline.c(9023): warning C5101: use of preprocessor
directive in function-like macro argument list is undefined behavior
Signed-off-by: Andre Muezerie
---
app/test-pmd/cmdline.c | 15 +--
1 file changed, 13 inserti
Compiling with MSVC results in errors like the one below:
app/test-pmd/cmdline_flow.c(8819): error C2099: initializer
is not a constant
Signed-off-by: Andre Muezerie
---
app/test-pmd/cmdline_flow.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/app/test-pmd/c
There's no MSVC equivalent for compiler extension __builtin_constant_p,
so a workaround is needed.
Signed-off-by: Andre Muezerie
---
app/test/test_memcpy_perf.c | 106 ++--
1 file changed, 53 insertions(+), 53 deletions(-)
diff --git a/app/test/test_memcpy_perf.c
Compiling with MSVC results in the warning below:
app/test-pmd/cmdline_flow.c(13964): warning C4098: 'cmd_set_raw_parsed':
'void' function returning a value
Signed-off-by: Andre Muezerie
---
app/test-pmd/cmdline_flow.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff
Compiling with MSVC results in the error below:
app/test/test_ring_perf.c(197): error C7712: address argument to atomic
operation must be a pointer to an atomic integer,
'volatile unsigned int *' is not valid
The fix is to mark lcore_count as atomic when using C11 memory model.
Signed-of
On Tue, 11 Feb 2025 14:02:04 -0800
Andre Muezerie wrote:
> Compiling with MSVC results in the error below:
>
> app/test/test_ring_perf.c(197): error C7712: address argument to atomic
> operation must be a pointer to an atomic integer,
> 'volatile unsigned int *' is not valid
>
> The fix
On Tue, 11 Feb 2025 14:01:57 -0800
Andre Muezerie wrote:
> There's no MSVC equivalent for compiler extension __builtin_constant_p.
> EAL already had __rte_constant which was used as a first attempt to
> workaround __builtin_constant_p when using MSVC. However, there are
> pieces of code that woul
There's no MSVC equivalent for compiler extension __builtin_constant_p.
EAL already had __rte_constant which was used as a first attempt to
workaround __builtin_constant_p when using MSVC. However, there are
pieces of code that would benefit from being able to provide a default
value to be used ins
On Tue, 11 Feb 2025 14:01:59 -0800
Andre Muezerie wrote:
> Compiling with MSVC results in warnings like the one below:
>
> app/test-pmd/csumonly.c(1085): warning C4477: 'printf' : format string
> '%d' requires an argument of type 'int',
> but variadic argument 1 has type 'uint64_t'
>
>
On Tue, 11 Feb 2025 14:02:05 -0800
Andre Muezerie wrote:
> There's no MSVC equivalent for compiler extension __builtin_constant_p,
> so a workaround is needed.
>
> Signed-off-by: Andre Muezerie
Prefer that __rte_constant worked on all platforms,
but template code is hard to maintain.
add support to update vid for vlan filter.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 1 +
doc/guides/nics/rnp.rst | 2 +-
drivers/net/rnp/base/meson.build| 1 +
drivers/net/rnp/base/rnp_bitrev.h | 64 ++
drivers/net/rnp/base/rnp_c
Add support tx tso and tunnel tso.
for tunnel just support vxlan/nvgre
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini | 1 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_bdq_if.h | 1 +
drivers/net/rnp/rnp.h | 2 +-
drivers/net/rnp/rnp_eth
add support mac eth rx tx hw xstats.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 1 +
drivers/net/rnp/base/rnp_eth_regs.h | 3 +
drivers/net/rnp/base/rnp_mac_regs.h | 80 +
drivers/net/rnp/rnp.h | 51 ++
drivers/net/rnp/rnp_ethdev.c|
add support scatter multi segment received.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini | 1 +
doc/guides/nics/rnp.rst | 2 +
drivers/net/rnp/rnp_rxtx.c | 131 ++-
drivers/net/rnp/rnp_rxtx.h | 2 +
4 files changed, 134 inse
add support get queue configure info for user debug
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/rnp_ethdev.c | 2 ++
drivers/net/rnp/rnp_rxtx.c | 42
drivers/net/rnp/rnp_rxtx.h | 4
3 files changed, 48 insertions(+)
diff --git a/drivers/net/rnp/r
add platform method for get rx/tx burst function select
by upload func name.
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/rnp_ethdev.c | 2 ++
drivers/net/rnp/rnp_rxtx.c | 58
drivers/net/rnp/rnp_rxtx.h | 6
3 files changed, 66 insertions(+)
diff
> diff --git a/lib/eal/common/eal_common_trace_ctf.c
> b/lib/eal/common/eal_common_trace_ctf.c
> index 6bc8bb9036..d9b307e076 100644
> --- a/lib/eal/common/eal_common_trace_ctf.c
> +++ b/lib/eal/common/eal_common_trace_ctf.c
> @@ -378,6 +378,9 @@ char *trace_metadata_fixup_field(const char *field)
Hi Stephen,
Thanks for your guideine, I will fixed these soon.
Regards Wenbo
> -Original Message-
> From: Stephen Hemminger
> Sent: 2025年2月11日 3:04
> To: Wenbo Cao
> Cc: tho...@monjalon.net; dev@dpdk.org; ferruh.yi...@amd.com;
> andrew.rybche...@oktetlabs.ru; yao...@mucse.com
> Subject
On Mon, Feb 10, 2025 at 6:19 PM Bruce Richardson
wrote:
> diff --git a/dts/framework/testbed_model/posix_session.py
> b/dts/framework/testbed_model/posix_session.py
> index 981600e24c..eab08a90ce 100644
> --- a/dts/framework/testbed_model/posix_session.py
> +++ b/dts/framework/testbed_model/posix
On Tue, Feb 11, 2025 at 9:44 AM Sunil Kumar Kori wrote:
>
> > diff --git a/lib/eal/common/eal_common_trace_ctf.c
> > b/lib/eal/common/eal_common_trace_ctf.c
> > index 04c4f71462..3e4228ee7f 100644
> > --- a/lib/eal/common/eal_common_trace_ctf.c
> > +++ b/lib/eal/common/eal_common_trace_ctf.c
> > @
On Tue, Feb 11, 2025 at 9:53 AM Sunil Kumar Kori wrote:
>
> > diff --git a/lib/eal/common/eal_common_trace_ctf.c
> > b/lib/eal/common/eal_common_trace_ctf.c
> > index 6bc8bb9036..d9b307e076 100644
> > --- a/lib/eal/common/eal_common_trace_ctf.c
> > +++ b/lib/eal/common/eal_common_trace_ctf.c
> > @
10/02/2025 18:20, Stephen Hemminger:
> +# ignore Linux UAPI kernel headers
> +kernel/linux/uapi/version
> +kernel/linux/uapi/**/*.h
The original .gitignore had some "!" for an opposite logic.
I think we can just completely drop this one, it looks unneeded.
> -Original Message-
> From: Stephen Hemminger
> Sent: Tuesday, February 11, 2025 7:24 AM
> To: Kusztal, ArkadiuszX
> Cc: dev@dpdk.org; ferruh.yi...@amd.com; Ji, Kai ; Dooley,
> Brian
> Subject: Re: [PATCH v6] net: add thread-safe crc api
>
> On Mon, 10 Feb 2025 21:27:10 +
> Arkad
> diff --git a/lib/eal/common/eal_common_trace_ctf.c
> b/lib/eal/common/eal_common_trace_ctf.c
> index 04c4f71462..3e4228ee7f 100644
> --- a/lib/eal/common/eal_common_trace_ctf.c
> +++ b/lib/eal/common/eal_common_trace_ctf.c
> @@ -373,6 +373,11 @@ rte_trace_metadata_dump(FILE *f)
>
> char *trace_
This patch adds support for mailbox of rnp PMD driver,
mailbox is used for communication between pf with fw
and vf driver.
Signed-off-by: Wenbo Cao
Reviewed-by: Stephen Hemminger
---
drivers/net/rnp/base/meson.build | 22 ++
drivers/net/rnp/base/rnp_hw.h| 76 +
drivers/net/rnp/base/rn
add log function for trace or debug
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/rnp_ethdev.c | 2 ++
drivers/net/rnp/rnp_logs.h | 36
2 files changed, 38 insertions(+)
create mode 100644 drivers/net/rnp/rnp_logs.h
diff --git a/drivers/net/rnp/rnp_ethdev
Add basic PMD library and doc build infrastructure
Update maintainers file to claim responsibility.
Signed-off-by: Wenbo Cao
Reviewed-by: Thomas Monjalon
---
.mailmap | 1 +
MAINTAINERS | 6 +++
doc/guides/nics/features/rnp.ini | 8
doc/guide
For This patchset just to support the basic chip init work
and user can just found the eth_dev, but can't control more.
For Now just support 2*10g nic,the chip can support
2*10g,4*10g,4*1g,8*1g,8*10g.
The Feature rx side can support rx-cksum-offload,rss,vlan-filter
flow_clow,uncast_filter,mcast_fil
add firmware communic method and basic device
init, uninit and close resource function.
Signed-off-by: Wenbo Cao
Reviewed-by: Ferruh Yigit
---
drivers/net/rnp/base/meson.build| 4 +
drivers/net/rnp/base/rnp_common.c | 73 ++
drivers/net/rnp/base/rnp_common.h | 12 +
drivers/net/
support tx/rx queue setup and release add hw bd
queue reset,sw queue reset.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/meson.build| 1 +
drivers/net/rnp/base/rnp_bdq_if.c | 398 +++
drivers/net/rnp/base/rnp_bdq_if.h
add support two method of MAC unicast promisc
mulcast promisc broadcast promisc mode
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 2 +
doc/guides/nics/rnp.rst | 5 ++
drivers/net/rnp/base/rnp_common.c | 5 ++
drivers/net/rnp/base/rnp_eth_regs.h | 15 +++
add get device hardware capability function
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini | 1 +
drivers/net/rnp/base/rnp_fw_cmd.c | 20 ++
drivers/net/rnp/base/rnp_fw_cmd.h | 80 +
drivers/net/rnp/base/rnp_mbx_fw.c | 58 +++
drivers/net/
add basic support for device to start/stop function
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/base/rnp_common.c | 22 +++
drivers/net/rnp/base/rnp_common.h | 1 +
drivers/net/rnp/base/rnp_dma_regs.h | 10 +
drivers/net/rnp/base/rnp_eth_regs.h | 5 +
drivers/net/rnp/base/rnp_hw.h
Add basic PCIe ethdev probe and remove.
Signed-off-by: Wenbo Cao
Reviewed-by: Stephen Hemminger
---
drivers/net/rnp/rnp.h| 13 +++
drivers/net/rnp/rnp_ethdev.c | 74
2 files changed, 87 insertions(+)
create mode 100644 drivers/net/rnp/rnp.h
dif
Hi David,
I validated the series and its functional now. Traces are generated properly.
> dmadev trace points have been working around trace point framework
> limitations.
> One of this workaround blocks enabling dmadev on MSVC.
>
> Traces files were not checked in CI, so this series also adds a
add set link_down/link_up implement
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/base/rnp_eth_regs.h | 3 +
drivers/net/rnp/base/rnp_fw_cmd.c | 22 +++
drivers/net/rnp/base/rnp_fw_cmd.h | 6 ++
drivers/net/rnp/base/rnp_mbx_fw.c | 33 ++
drivers/net/rnp/base/rnp_mbx_fw.h |
support rx/tx queue stop/start,for rx queue stop
need to reset a queue,must stop all rx queue
during reset this queue.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini | 1 +
drivers/net/rnp/base/rnp_common.c | 3 +
drivers/net/rnp/rnp_link.c| 340 ++
add support rss reta updata/query rss hash update/get
dev_configure add rss conf check.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 4 +
doc/guides/nics/rnp.rst | 3 +
drivers/net/rnp/base/rnp_eth_regs.h | 16 ++
drivers/net/rnp/meson.build | 1 +
add support multiple segs mbuf send.
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/rnp_rxtx.c | 126 -
drivers/net/rnp/rnp_rxtx.h | 3 +-
2 files changed, 126 insertions(+), 3 deletions(-)
diff --git a/drivers/net/rnp/rnp_rxtx.c b/drivers/net/rnp/rnp_rxtx.c
add mac filter for single/multiple port.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 1 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_eth_regs.h | 4 ++
drivers/net/rnp/base/rnp_hw.h | 3 +
drivers/net/rnp/base/rnp_mac.c | 91
add mtu update limit for multiple port mode.
multiple mode just used the max-mtu of ports
to limit receive.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 1 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_eth_regs.h | 3 +
drivers/net/rnp/rnp.h
add only support simple send pkts.
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/rnp_ethdev.c | 6 +++
drivers/net/rnp/rnp_rxtx.c | 85 +++-
drivers/net/rnp/rnp_rxtx.h | 1 +
3 files changed, 91 insertions(+), 1 deletion(-)
diff --git a/drivers/net/rnp/rnp_e
Add support Rx l3/l4 checum and tunnel
inner l3/l4, out l3 chksum.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 4 ++
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_eth_regs.h | 13
drivers/net/rnp/rnp.h | 7 +++
drivers/net/rnp/rn
On Tue, Feb 11, 2025 at 10:50:23AM +0100, David Marchand wrote:
> On Mon, Feb 10, 2025 at 6:19 PM Bruce Richardson
> wrote:
> > diff --git a/dts/framework/testbed_model/posix_session.py
> > b/dts/framework/testbed_model/posix_session.py
> > index 981600e24c..eab08a90ce 100644
> > --- a/dts/framew
add only support simple recv pkts.
Signed-off-by: Wenbo Cao
---
drivers/net/rnp/rnp_ethdev.c | 7 ++
drivers/net/rnp/rnp_rxtx.c | 129 +++
drivers/net/rnp/rnp_rxtx.h | 5 ++
3 files changed, 141 insertions(+)
diff --git a/drivers/net/rnp/rnp_ethdev.c b/dr
The current net CRC API is not thread-safe, this patch
solves this by adding another, thread-safe API functions.
This API is also safe to use across multiple processes,
yet with limitations on max-simd-bitwidth, which will be checked only by
the process that created the CRC context; all other proce
add support rx vlan strip,filter,tx vlan/qinq insert.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 2 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_bdq_if.h | 2 +-
drivers/net/rnp/base/rnp_eth_regs.h | 5 +
drivers/net/rnp/base/rnp_hw.h
add support hw-missed rx/tx packets bytes.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini| 2 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_eth_regs.h | 3 +
drivers/net/rnp/rnp.h | 10 +-
drivers/net/rnp/rnp_ethdev.c| 147 +
add support parse hw packet types result.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini | 1 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_bdq_if.h | 4 +++
drivers/net/rnp/rnp_rxtx.c| 45 +++
4 files changed, 51 inser
This patch add support poll/irq link get mode.
Signed-off-by: Wenbo Cao
---
doc/guides/nics/features/rnp.ini | 2 +
doc/guides/nics/rnp.rst | 1 +
drivers/net/rnp/base/rnp_fw_cmd.c | 45
drivers/net/rnp/base/rnp_fw_cmd.h | 58 ++-
drivers/net/rnp/base/rn
On Thu, Feb 06, 2025 at 12:54:24PM +0200, Shani Peretz wrote:
> When registering a new PCI device, the device->name field stored
> the user-provided string from devargs (e.g., "08:00.0" or ":08:00.0").
> This approach led to inconsistencies when registering new devices.
>
> This patch fix this
This series handles memset related bugs indentified by PVS Studio.
The root cause is that Gcc and other compilers are free to
optimize away memset called before free.
Most of the places memset was being used like this were bogus;
probably some developer debug habit, and can be safely removed.
v6
When memset() is used before a release function such as free,
the compiler if allowed to optimize the memset away under
the as-if rules. This is normally ok, but in certain cases such
as passwords or security keys it is problematic.
Introduce a DPDK wrapper which is equivalent to the C++ memset_s
Just doing memset() on keys is not enough, compiler can optimize
it away. Need something with a barrier.
Signed-off-by: Stephen Hemminger
Acked-by: Bruce Richardson
---
drivers/crypto/qat/qat_sym_session.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/dri
The memset was always doing 0 bytes since size computed later.
Link: https://pvs-studio.com/en/blog/posts/cpp/1179/
Fixes: 3a80d7fb2ecd ("crypto/qat: support SHA3 plain hash")
Cc: sta...@dpdk.org
Signed-off-by: Stephen Hemminger
Acked-by: Bruce Richardson
---
drivers/crypto/qat/qat_sym_sessio
Although internally rte_free does poison the buffer in most
cases, it is useful to have function that explicitly does
this to avoid any security issues.
Signed-off-by: Stephen Hemminger
---
lib/eal/common/rte_malloc.c | 30 --
lib/eal/include/rte_malloc.h | 18 ++
Doing memset before free maybe removed by compiler, and
is flagged by security scanning tools as potential problem.
In this case the memset is unnecessary.
Signed-off-by: Stephen Hemminger
Acked-by: Chengwen Feng
---
drivers/bus/uacce/uacce.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/d
Calling memset before rte_free not necessary, and could be
removed by the compiler. In this case, the data is not security
sensitive so the memset can be removed. Some security scanning
tools will flag this.
Signed-off-by: Stephen Hemminger
Acked-by: Bruce Richardson
---
drivers/compress/octeon
From: Vamsi Attunuru
Adds code changes to maintain pending count per hw dma channel
instead of per vchan. This enables ringing dbell for exact number
of dma commands present in the DPI queue.
Signed-off-by: Vamsi Attunuru
---
drivers/dma/cnxk/cnxk_dmadev.c| 17 +++
drivers/dma/cnxk/cnx
> DPDK build checks for build support for various instruction sets by
> checking both the target machine and the compiler for support. However,
> any disabling of instruction sets via compiler flags was not taken into
> account in many cases. For AVX512 support, check for a user-specified
> "no-avx
Need to check the result of malloc() before calling memset.
This is only place in this driver that forgot, other code
does check.
Fixes: 0d9bca480e26 ("net/ntnic: add FPGA modules for initialization")
cc: sta...@dpdk.org
Signed-off-by: Stephen Hemminger
---
drivers/net/ntnic/nthw/nthw_rac.c | 4
Calling memset before free() has no effect and will be flagged
by security parsing tools as a potential bug. None of these data
structures have sensitive information.
Signed-off-by: Stephen Hemminger
---
drivers/net/ntnic/nthw/core/nthw_hif.c| 5 +
drivers/net/ntnic/nthw/core
Script that converts memset before free into rte_memset_sensitive
and memset before rte_free into rte_free_sensitive
Signed-off-by: Stephen Hemminger
---
devtools/cocci/memset_free.cocci | 20
1 file changed, 20 insertions(+)
create mode 100644 devtools/cocci/memset_free.co
Regular memset maybe removed by compiler if done before a free
function. Use new rte_free_sensitive instead.
Signed-off-by: Stephen Hemminger
Acked-by: Bruce Richardson
---
drivers/crypto/qat/qat_asym.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/crypto/qat/q
Since tmp is not used later in the function, this memset
is unnecessary. Even though this is harmless,
it causes tools that look for security issues
around memset to flag this a bug.
Signed-off-by: Stephen Hemminger
Acked-by: Bruce Richardson
---
app/test/test_cmdline_cirbuf.c | 2 --
1 file ch
On Tue, 11 Feb 2025 16:44:02 +
Bruce Richardson wrote:
> Depending on the scenario, a user may want to run testpmd with
> pre-canned startup commands either with or without echoing of those
> commands as they are executed. To observe progress of each command, or
> to help determine what comma
On Thu, 6 Feb 2025 02:08:36 +0200
Shani Peretz wrote:
> static int
> -cdx_parse(const char *name, void *addr)
> +cdx_parse(const char *name, void *addr, int *size)
> {
> - const char **out = addr;
> int ret;
>
> ret = strncmp(name, CDX_DEV_PREFIX, strlen(CDX_DEV_PREFIX));
>
On Tue, Feb 11, 2025 at 09:48:32AM -0800, Stephen Hemminger wrote:
> On Thu, 6 Feb 2025 02:08:36 +0200
> Shani Peretz wrote:
>
> > static int
> > -cdx_parse(const char *name, void *addr)
> > +cdx_parse(const char *name, void *addr, int *size)
> > {
> > - const char **out = addr;
> > int r
On Mon, Feb 3, 2025 at 10:17 AM Luca Vizzarro wrote:
>
> The current configuration makes the user re-specify the port links for
> each port in an unintuitive and repetitive way. Moreover, this design
> does not give the user to opportunity to map the port topology as
> desired.
>
> This change add
On Tue, 11 Feb 2025 17:54:26 +
Bruce Richardson wrote:
> On Tue, Feb 11, 2025 at 09:48:32AM -0800, Stephen Hemminger wrote:
> > On Thu, 6 Feb 2025 02:08:36 +0200
> > Shani Peretz wrote:
> >
> > > static int
> > > -cdx_parse(const char *name, void *addr)
> > > +cdx_parse(const char *name,
> The common/iavf driver folder contains the base code for the iavf
> driver, which is also linked against by the ice driver and others.
> However, there is no need for this to be in common, and we can
> move it to the net/intel/iavf as a base code driver. This involves
> updating dependencies that
1 - 100 of 163 matches
Mail list logo
