Re: [PATCH v7 01/21] net/ntnic: add ethdev and makes PMD available

2024-07-12 Thread zhoumin
Hi Patrick, Thanks for reminding me. The pw_maintainers_cli.py script indeed is outdated in my fork. I have updated this script, but still failed to apply this patch. I saw all the other labs applied this patch based on commit of 'a6c3ec342ee105e322ffdb21e810cdfd38455c62', but I cannot find

[PATCH] doc: update AMD Linux guide for power

2024-07-12 Thread Vipin Varghese
Introduce sections into AMD Linux getting start guide for 1. MAX LCORE 2. CPU power Signed-off-by: Vipin Varghese --- --- doc/guides/linux_gsg/amd_platform.rst | 21 + 1 file changed, 21 insertions(+) diff --git a/doc/guides/linux_gsg/amd_platform.rst b/doc/guides/linux_gs

RE: [PATCH 00/19] Wangxun fixes and supports

2024-07-12 Thread Jiawen Wu
On Fri, Jul 12, 2024 11:31 PM, Ferruh Yigit wrote: > On 7/12/2024 1:47 PM, Ferruh Yigit wrote: > > On 7/12/2024 10:32 AM, David Marchand wrote: > >> On Thu, Jul 11, 2024 at 1:53 PM David Marchand > >> wrote: > >>> On Sat, Jul 6, 2024 at 5:39 AM Ferruh Yigit wrote: > >>> Fix some bugs in txgbe

Re: [PATCH v8 01/21] net/ntnic: add ethdev and makes PMD available

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 4:47 PM, Serhii Iliushyk wrote: > Add initial ntnic ethdev skeleton and register PCI probe functions > Update documentation: Device description and feature list > > Signed-off-by: Serhii Iliushyk > Hi Serhii, I put some comments but main issue is build failure after each patch, can

Re: [PATCH v8 10/21] net/ntnic: add FPGA modules for initialization

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 4:47 PM, Serhii Iliushyk wrote: > New ntnic FPGA modules: > - Host Interface (HIF): Basic FPGA info such as prod ID and build time. > - Inter-Integrated Circuit Controller (IIC): Use the FPGA to access > the other integrated circuits on the ntnic. > - PCI Express Gen3 (PCIE3): Th

Re: [PATCH v8 06/21] net/ntnic: add basic eth dev ops to ntnic

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 4:47 PM, Serhii Iliushyk wrote: > Adds support for eth_dev configure, start, stop, close, and infos_get. > The internal structs of ntnic is also added and initialized. > > Signed-off-by: Serhii Iliushyk > --- > v6 > * Replace if_index with n_intf_no > * Unnecessry resources free was

Re: [PATCH v8 03/21] net/ntnic: add minimal initialization for PCI device

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 4:47 PM, Serhii Iliushyk wrote: > add implementation for probe/init and remove/deinit of the PCI device > > Signed-off-by: Serhii Iliushyk <...> > > static int > nthw_pci_dev_deinit(struct rte_eth_dev *eth_dev __rte_unused) > { > - return 0; > + NT_LOG_DBGX(DEB

Re: [PATCH v8 01/21] net/ntnic: add ethdev and makes PMD available

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 4:47 PM, Serhii Iliushyk wrote: > Add initial ntnic ethdev skeleton and register PCI probe functions > Update documentation: Device description and feature list > > Signed-off-by: Serhii Iliushyk > --- > v6 > * Remove unused includes > * Remove EOF markers > * Remove unnecesarry comm

release candidate 24.07-rc2

2024-07-12 Thread David Marchand
A new DPDK release candidate is ready for testing: https://git.dpdk.org/dpdk/tag/?id=v24.07-rc2 There are 461 new patches in this snapshot. Release notes: https://doc.dpdk.org/guides/rel_notes/release_24_07.html Highlights of 24.07-rc2: - SVE support in the hash library

[DPDK/meson Bug 1466] [dpdk-24.07] clang failed to compile DPDK when using parameter "-Db_sanitize=address"

2024-07-12 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1466 David Marchand (david.march...@redhat.com) changed: What|Removed |Added Resolution|--- |FIXED

[DPDK/ethdev Bug 1441] RSS Hash returned by the gvnic pmd dpdk driver not matching Toeplitz

2024-07-12 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1441 David Marchand (david.march...@redhat.com) changed: What|Removed |Added Status|CONFIRMED |RESOLVED Res

Re: [PATCH v3] eal/common: fix inconsistent representation of PCI numbers

2024-07-12 Thread Thomas Monjalon
12/07/2024 15:49, David Marchand: > On Mon, Jul 8, 2024 at 6:52 PM Shani Peretz wrote: > > > > DPDK allows for two ways to specify PCI device numbers: > > a full version (":08:00.0") and a short version ("08:00.0"). > > The problem arises when the application uses one format (e.g., full) > > w

Re: [PATCH v6 2/2] bus/pci: fix secondary process save 'FD' problem

2024-07-12 Thread David Marchand
Hello, On Tue, Jul 2, 2024 at 9:40 AM Chaoyong He wrote: > > From: Zerun Fu > > In the previous logic the 'fd' was only saved in the primary process, > but for some devices this value is also used in the secondary logic. > > For example, the call of 'rte_pci_find_ext_capability()' will fail in >

Re: [PATCH 0/2] eal/riscv: implement prefetch using zicbop

2024-07-12 Thread David Marchand
Hello, On Thu, May 30, 2024 at 7:21 PM Daniel Gregory wrote: > > Instructions from RISC-V's Zicbop extension can be used to implement the > rte_prefetch* family of functions. On modern versions of GCC (13.1.0+) > and Clang (17.0.1+), these are emitted by __builtin_prefetch() when the > extension

Re: [PATCH v2 0/9] riscv: implement accelerated crc using zbc

2024-07-12 Thread David Marchand
On Fri, Jul 12, 2024 at 5:47 PM Daniel Gregory wrote: > > The RISC-V Zbc extension adds instructions for carry-less multiplication > we can use to implement CRC in hardware. This patch set contains two new > implementations: > > - one in lib/hash/rte_crc_riscv64.h that uses a Barrett reduction to

Re: [PATCH 1/1] maintainers: update for vmbus/mana/netvsc drivers

2024-07-12 Thread David Marchand
On Thu, Jun 27, 2024 at 9:24 PM Long Li wrote: > > > Subject: [PATCH 1/1] maintainers: update for vmbus/mana/netvsc drivers > > > > Add myself as maintainer for vmbus, mana and netvsc. > > > > Signed-off-by: Wei Hu > Reviewed-by: Long Li Applied, thanks. -- David Marchand

Re: [PATCH 2/2] maintainers: update maintainers

2024-07-12 Thread David Marchand
On Mon, Jul 1, 2024 at 11:35 AM Ruifeng Wang wrote: > On 2024/6/29 12:11 AM, Jack Bond-Preston wrote: > > Update maintainers for Bitops, Ticketlock, and ARMv8 Crypto. > > > > Signed-off-by: Jack Bond-Preston > > Reviewed-by: Wathsala Vithanage > Acked-by: Ruifeng Wang I want to thank you both

Re: [PATCH 1/1] maintainers: update for Chelsio cxgbe

2024-07-12 Thread David Marchand
On Thu, Jul 4, 2024 at 10:24 AM Potnuri Bharat Teja wrote: > > Update maintainers for chelsio cxgbe drivers. > > Signed-off-by: Potnuri Bharat Teja Applied, thanks Potnuri for taking over. -- David Marchand

Re: [PATCH 00/19] Wangxun fixes and supports

2024-07-12 Thread Thomas Monjalon
12/07/2024 14:47, Ferruh Yigit: > On 7/12/2024 10:32 AM, David Marchand wrote: > > On Thu, Jul 11, 2024 at 1:53 PM David Marchand > > wrote: > >> On Sat, Jul 6, 2024 at 5:39 AM Ferruh Yigit wrote: > >> Fix some bugs in txgbe/ngbe, and support new features in ngbe driver. > >> > >> Jia

[PATCH v8 21/21] net/ntnic: add physical layer control module

2024-07-12 Thread Serhii Iliushyk
Adds functionality to control the physical layer of the OSI model. This takes the form of the module MAC PCS (Media Access Control Physical Coding Sublayer). The functionality is used by the 100G link functionality to establish link. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/n

[PATCH v8 19/21] net/ntnic: add QSFP28 support

2024-07-12 Thread Serhii Iliushyk
Includes support for QSFP28 Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntnic_nim.h| 21 ++ drivers/net/ntnic/link_mgmt/nt4ga_link.c | 25 +++ drivers/net/ntnic/nim/i2c_nim.c | 267 ++- drivers/net/ntnic/nim/nim_defines.h | 1 + 4 files

[PATCH v8 20/21] net/ntnic: add GPIO communication for NIMs

2024-07-12 Thread Serhii Iliushyk
For NIM reset sequence GPIO communication is used. After this commit the NIMs supported by ntnic is able to start up and be controlled fully by the adapter. Signed-off-by: Serhii Iliushyk --- v5 * Fix Typo/Spelling --- drivers/net/ntnic/include/nt4ga_link.h| 1 + .../link_mgmt/link_100

[PATCH v8 18/21] net/ntnic: add QSFP support

2024-07-12 Thread Serhii Iliushyk
Includes support for QSFP and QSFP+. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntnic_nim.h | 10 + .../link_mgmt/link_100g/nt4ga_link_100g.c | 12 +- drivers/net/ntnic/nim/i2c_nim.c | 310 +- drivers/net/ntnic/nim/i2c_nim.h

[PATCH v8 10/21] net/ntnic: add FPGA modules for initialization

2024-07-12 Thread Serhii Iliushyk
New ntnic FPGA modules: - Host Interface (HIF): Basic FPGA info such as prod ID and build time. - Inter-Integrated Circuit Controller (IIC): Use the FPGA to access the other integrated circuits on the ntnic. - PCI Express Gen3 (PCIE3): The FPGA part of PCIe3 initialization, speed te

[PATCH v8 14/21] net/ntnic: add clock profile for the NT200A0X smartNIC

2024-07-12 Thread Serhii Iliushyk
Because the ntnic hardware supports multiple different FPGAs with different pipelines and port speeds, the clock profile is not hardcoded into the product, and need to be initialized from software. The clock profile itself is an array of integers that was generated by Silicon Labs ClockBuilder. S

[PATCH v8 17/21] net/ntnic: add generic NIM and I2C modules

2024-07-12 Thread Serhii Iliushyk
As the ntnic can support different port speeds, it also needs to support different NIMs (Network Interface Module). This commit add the generic NIM support for ntnic, such that the specific modules, such as QSFP28 can be added later. The communication with NIMs is in the form of I2C, so support fo

[PATCH v8 16/21] net/ntnic: add link 100G module ops

2024-07-12 Thread Serhii Iliushyk
As the ntnic can support different speeds, an abstraction layer for 100G speed is needed. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/adapter/nt4ga_adapter.c | 12 - .../link_mgmt/link_100g/nt4ga_link_100g.c | 49 +++ drivers/net/ntnic/meson.build

[PATCH v8 15/21] net/ntnic: add link management skeleton

2024-07-12 Thread Serhii Iliushyk
Add functionality to read and control the link-state of the ntnic. Note that must functions are not implemented yet. Adds the following eth_dev_ops: - dev_set_link_up - dev_set_link_down - link_update - mac_addr_add - mac_addr_set - set_mc_addr_list - promiscuous_enable Signed-off-by: Serhii Iliu

[PATCH v8 13/21] net/ntnic: add startup and reset sequence for NT200A0X

2024-07-12 Thread Serhii Iliushyk
Adds reset (RST) module for FW 9563. Also adds SDRAM Controller (SDC) module, as it is part of the startup and reset sequence. Signed-off-by: Serhii Iliushyk --- v5 * Fix Typo/Spelling --- .../include/ntnic_nthw_fpga_rst_nt200a0x.h| 81 +++ drivers/net/ntnic/meson.build |

[PATCH v8 12/21] net/ntnic: add support of the NT200A0X smartNIC

2024-07-12 Thread Serhii Iliushyk
Add ntnic support for NT200A0X NIC Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/adapter/nt4ga_adapter.c | 6 +++ drivers/net/ntnic/meson.build | 1 + .../net/ntnic/nthw/core/include/nthw_fpga.h | 7 +++ .../nthw/core/nt200a0x/nthw_fpga_nt200a0x.c | 54 +

[PATCH v8 11/21] net/ntnic: add FPGA initialization functionality

2024-07-12 Thread Serhii Iliushyk
Enable FPGA initialization and adds ethdev fw_version_get. Signed-off-by: Serhii Iliushyk --- v5 * Fix Typo/Spelling v7 * Add FW update feature to documentation - INI and RST files --- doc/guides/nics/features/ntnic.ini| 1 + doc/guides/nics/ntnic.rst | 5 + d

[PATCH v8 05/21] net/ntnic: add VFIO module

2024-07-12 Thread Serhii Iliushyk
Adds VFIO functionality and the DMA it requires. The VFIO context is initialized during ntnic ethdev startup. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/meson.build | 1 + drivers/net/ntnic/ntnic_ethdev.c | 23 +++ drivers/net/ntnic/ntnic_vfio.c | 235

[PATCH v8 08/21] net/ntnic: add adapter initialization

2024-07-12 Thread Serhii Iliushyk
Add interfaces for initialize the adapter Signed-off-by: Serhii Iliushyk --- v6 * Function for global var clearing was removed --- drivers/net/ntnic/adapter/nt4ga_adapter.c | 148 ++ drivers/net/ntnic/include/nt4ga_adapter.h | 40 ++ drivers/net/ntnic/include/ntdrv_4ga.h

[PATCH v8 07/21] net/ntnic: add core platform structures

2024-07-12 Thread Serhii Iliushyk
Adds many of the high level structures needed by the ntnic FPGA modules and adapter control. This is considered the first part of the skeleton of ntnic FPGA support Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntos_drv.h | 2 + drivers/net/ntnic/meson.build |

[PATCH v8 06/21] net/ntnic: add basic eth dev ops to ntnic

2024-07-12 Thread Serhii Iliushyk
Adds support for eth_dev configure, start, stop, close, and infos_get. The internal structs of ntnic is also added and initialized. Signed-off-by: Serhii Iliushyk --- v6 * Replace if_index with n_intf_no * Unnecessry resources free was fixed * Fix typo * Useless vars were removed --- drivers/net

[PATCH v8 04/21] net/ntnic: add NT utilities implementation

2024-07-12 Thread Serhii Iliushyk
Add ntnic utilities. Signed-off-by: Serhii Iliushyk --- v6 * NT utils header file was moved --- drivers/net/ntnic/meson.build | 2 ++ drivers/net/ntnic/ntnic_ethdev.c | 2 ++ drivers/net/ntnic/ntutil/nt_util.c | 33 +++ drivers/net/ntnic/ntutil/nt_util.h | 43 +++

[PATCH v8 03/21] net/ntnic: add minimal initialization for PCI device

2024-07-12 Thread Serhii Iliushyk
add implementation for probe/init and remove/deinit of the PCI device Signed-off-by: Serhii Iliushyk --- v6 * Add driver deinitialization * Add API rte_eth_dev_probing_finish * Add correct API for finishing probing the device * Remove duplicated calling 'rte_eth_copy_pci_info()'

[PATCH v8 01/21] net/ntnic: add ethdev and makes PMD available

2024-07-12 Thread Serhii Iliushyk
Add initial ntnic ethdev skeleton and register PCI probe functions Update documentation: Device description and feature list Signed-off-by: Serhii Iliushyk --- v6 * Remove unused includes * Remove EOF markers * Remove unnecesarry commpiller flags * Update documentation INI and RST files *

[PATCH v8 02/21] net/ntnic: add logging implementation

2024-07-12 Thread Serhii Iliushyk
Adds ntnic specific implementation for logging. NT NIC uses this logging abstraction layer to ensure that FPGA module implementations function both within and outside in DPDK environment Signed-off-by: Serhii Iliushyk --- v6 * Logging header file was moved * Default log type was set to NOTICE ---

[PATCH v2 9/9] member: use accelerated crc on riscv

2024-07-12 Thread Daniel Gregory
When the RISC-V Zbc (carryless multiplication) extension is present, an implementation of CRC hashing using hardware instructions is available. Use it rather than jhash. Signed-off-by: Daniel Gregory --- lib/member/rte_member.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

[PATCH v2 8/9] hash/cuckoo: use accelerated crc on riscv

2024-07-12 Thread Daniel Gregory
When the RISC-V Zbc (carryless multiplication) extension is present, an implementation of CRC hashing using hardware instructions is available. Use it rather than jhash. Signed-off-by: Daniel Gregory --- lib/hash/rte_cuckoo_hash.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/lib/hash/r

[PATCH v2 7/9] examples/l3fwd-power: use accelerated crc on riscv

2024-07-12 Thread Daniel Gregory
When the RISC-V Zbc (carryless multiplication) extension is present, an implementation of CRC hashing using hardware instructions is available. Use it rather than jhash. Signed-off-by: Daniel Gregory --- examples/l3fwd-power/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --gi

[PATCH v2 6/9] ipfrag: use accelerated crc on riscv

2024-07-12 Thread Daniel Gregory
When the RISC-V Zbc (carryless multiplication) extension is present, an implementation of CRC hashing using hardware instructions is available. Use it rather than jhash. Signed-off-by: Daniel Gregory --- lib/ip_frag/ip_frag_internal.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

[PATCH v2 5/9] examples/l3fwd: use accelerated crc on riscv

2024-07-12 Thread Daniel Gregory
When the RISC-V Zbc (carryless multiplication) extension is present, an implementation of CRC hashing using hardware instructions is available. Use it rather than jhash. Signed-off-by: Daniel Gregory --- examples/l3fwd/l3fwd_em.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2 4/9] config/riscv: add qemu crossbuild target

2024-07-12 Thread Daniel Gregory
A new cross-compilation target that has extensions that DPDK uses and QEMU supports. Initially, this is just the Zbc extension for hardware crc support. Signed-off-by: Daniel Gregory --- config/riscv/meson.build| 3 ++- config/riscv/riscv64_qemu_linux_gcc | 1

[PATCH v2 3/9] net: implement crc using riscv carryless multiply

2024-07-12 Thread Daniel Gregory
Using carryless multiply instructions (clmul) from RISC-V's Zbc extension, implement CRC-32 and CRC-16 calculations on buffers. Based on the approach described in Intel's whitepaper on "Fast CRC Computation for Generic Polynomails Using PCLMULQDQ Instructions", we perform repeated folds-by-1 whils

[PATCH v2 2/9] hash: implement crc using riscv carryless multiply

2024-07-12 Thread Daniel Gregory
Using carryless multiply instructions from RISC-V's Zbc extension, implement a Barrett reduction that calculates CRC-32C checksums. Based on the approach described by Intel's whitepaper on "Fast CRC Computation for Generic Polynomials Using PCLMULQDQ Instruction", which is also described here (htt

[PATCH v2 0/9] riscv: implement accelerated crc using zbc

2024-07-12 Thread Daniel Gregory
The RISC-V Zbc extension adds instructions for carry-less multiplication we can use to implement CRC in hardware. This patch set contains two new implementations: - one in lib/hash/rte_crc_riscv64.h that uses a Barrett reduction to implement the four rte_hash_crc_* functions - one in lib/net/net

[PATCH v2 1/9] config/riscv: detect presence of Zbc extension

2024-07-12 Thread Daniel Gregory
The RISC-V Zbc extension adds carry-less multiply instructions we can use to implement more efficient CRC hashing algorithms. The RISC-V C api defines architecture extension test macros https://github.com/riscv-non-isa/riscv-c-api-doc/blob/main/riscv-c-api.md#architecture-extension-test-macros The

Re: [PATCH 00/19] Wangxun fixes and supports

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 1:47 PM, Ferruh Yigit wrote: > On 7/12/2024 10:32 AM, David Marchand wrote: >> On Thu, Jul 11, 2024 at 1:53 PM David Marchand >> wrote: >>> On Sat, Jul 6, 2024 at 5:39 AM Ferruh Yigit wrote: >>> Fix some bugs in txgbe/ngbe, and support new features in ngbe driver. >>> >>>

[PATCH v1] crypto/qat: add fix for Gen4 WRITE

2024-07-12 Thread Brian Dooley
All generations of QAT use the same Gen1 raw datapath. Gen4 needs a different WRITE function than other generations. Added separation for configuration of the raw ctx for Gen4 from the Gen1 codepath. Fixes: 85fec6fd9674 ("crypto/qat: unify raw data path functions") Cc: kai...@intel.com Cc: sta...@

RE: [PATCH v1] crypto/aesni_mb: fix typo in error message

2024-07-12 Thread Dooley, Brian
Hi Joel, > -Original Message- > From: Joel Kavanagh > Sent: Thursday, July 11, 2024 2:15 PM > To: Thomas Monjalon ; Ji, Kai ; De > Lara Guarch, Pablo ; Fan Zhang > ; Ciara Power > Cc: dev@dpdk.org; gak...@marvell.com; sta...@dpdk.org; Kavanagh, Joel > > Subject: [PATCH v1] crypto/aesni_

Re: [PATCH v3] net/virtio_user: fix cq descriptor conversion with non vDPA backend

2024-07-12 Thread David Marchand
On Fri, Jul 12, 2024 at 2:36 PM Srujana Challa wrote: > > This patch modifies the code to convert descriptor buffer IOVA > addresses to virtual addresses only when use_va flag is false. > > This patch fixes segmentation fault with vhost-user backend. > > Fixes: 67e9e504dae2 ("net/virtio_user: conv

Re: [PATCH v7 01/21] net/ntnic: add ethdev and makes PMD available

2024-07-12 Thread Patrick Robb
Hello Zhoumin, It looks like Loongarch CI failed to apply this patch, but it worked at the other labs and locally for Serhii when they were checked out to next-net. Maybe your CI did not choose next-net, the right branch? I remember you saying your CI is based on a fork of the dpdk-ci repo. Perha

Re: [PATCH v3] eal/common: fix inconsistent representation of PCI numbers

2024-07-12 Thread David Marchand
On Mon, Jul 8, 2024 at 6:52 PM Shani Peretz wrote: > > DPDK allows for two ways to specify PCI device numbers: > a full version (":08:00.0") and a short version ("08:00.0"). > The problem arises when the application uses one format (e.g., full) > when running testpmd, but then tries to use the

[DPDK/cryptodev Bug 1487] blockcipher_test_case_run() does not log test_msg

2024-07-12 Thread bugzilla
https://bugs.dpdk.org/show_bug.cgi?id=1487 Bug ID: 1487 Summary: blockcipher_test_case_run() does not log test_msg Product: DPDK Version: 24.07 Hardware: All OS: All Status: UNCONFIRMED Severity: normal

Re: [PATCH 00/19] Wangxun fixes and supports

2024-07-12 Thread Ferruh Yigit
On 7/12/2024 10:32 AM, David Marchand wrote: > On Thu, Jul 11, 2024 at 1:53 PM David Marchand > wrote: >> On Sat, Jul 6, 2024 at 5:39 AM Ferruh Yigit wrote: >> Fix some bugs in txgbe/ngbe, and support new features in ngbe driver. >> >> Jiawen Wu (19): >> net/txgbe: fix to parse

[PATCH v3] net/virtio_user: fix cq descriptor conversion with non vDPA backend

2024-07-12 Thread Srujana Challa
This patch modifies the code to convert descriptor buffer IOVA addresses to virtual addresses only when use_va flag is false. This patch fixes segmentation fault with vhost-user backend. Fixes: 67e9e504dae2 ("net/virtio_user: convert cq descriptor IOVA address to Virtual address") Reported-by:

Re: [PATCH v1 1/1] drivers/baseband: fix device queues initialization

2024-07-12 Thread David Marchand
On Mon, Jul 8, 2024 at 9:14 PM Hernan Vargas wrote: > > To prevent incorrect readings, initialize FFT and MLDTS queues to 0. All calls to bb drivers info_get are preceded with a: memset(&dev_info, 0, sizeof(dev_info)); dev->dev_ops->info_get(dev, &dev_info); So I don't really fol

RE: [EXTERNAL] Re: [PATCH v2] net/virtio_user: fix issue with converting cq descriptor IOVA address to VA

2024-07-12 Thread Srujana Challa
> On Thu, Jul 11, 2024 at 7:46 PM Srujana Challa wrote: > > > -virtio_user_iova2virt(rte_iova_t iova) > > > +virtio_user_iova2virt(struct virtio_user_dev *dev, rte_iova_t iova) > > > > > > > { > > > > - if (rte_eal_iova_mode() == RTE_IOVA_VA) > > > > - return (void *)(uintptr_

[PATCH v1 1/1] malloc/mp: fix wait condition handling

2024-07-12 Thread Anatoly Burakov
>From coverity's point of view, it is theoretically possible to have an infinite wait on a wait condition because while we do check for timeout, we do not check for whether the event we are waiting for has already occurred by the time we get to the first cond_wait call (in this case, it's state of

RE: [EXTERNAL] [PATCH v6] graph: expose node context as pointers

2024-07-12 Thread Kiran Kumar Kokkilagadda
> -Original Message- > From: Robin Jarry > Sent: Friday, July 5, 2024 8:23 PM > To: dev@dpdk.org; Jerin Jacob ; Kiran Kumar > Kokkilagadda ; Nithin Kumar Dabilpuram > ; Zhirun Yan > Subject: [EXTERNAL] [PATCH v6] graph: expose node context as pointers > > In some cases, the node conte

[PATCH v1] net/e1000: do not update link status in secondary process

2024-07-12 Thread Jun Wang
The code to update link status is not safe in secondary process. If called from secondary it will crash, example from dumpcap: eth_em_link_update Signed-off-by: Jun Wang --- drivers/net/e1000/em_ethdev.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/net/e1000/em_ethdev.c b/d

Re: [EXTERNAL] Re: [PATCH v2] net/virtio_user: fix issue with converting cq descriptor IOVA address to VA

2024-07-12 Thread David Marchand
On Thu, Jul 11, 2024 at 7:46 PM Srujana Challa wrote: > > -virtio_user_iova2virt(rte_iova_t iova) > > +virtio_user_iova2virt(struct virtio_user_dev *dev, rte_iova_t iova) > > > > > { > > > - if (rte_eal_iova_mode() == RTE_IOVA_VA) > > > - return (void *)(uintptr_t)iova; > > >

Re: [PATCH] buildtools: fix build with clang 17 and ASan

2024-07-12 Thread David Marchand
On Fri, Jul 12, 2024 at 4:29 AM Jiale, SongX wrote: > > ASan included in clang 17 and later suffixes symbols. > > $ nm build/drivers/libtmp_rte_net_null.a | grep this_pmd > > r this_pmd_name3 > > n > > this_pmd_name3.f2cd16678ab09dba8fd23405d8d11fce > > > > This b

Re: [PATCH v3 1/3] net/virtio_user: convert cq descriptor IOVA address to Virtual address

2024-07-12 Thread David Marchand
Hello, On Wed, Jul 3, 2024 at 12:19 PM Jerin Jacob wrote: > > On Wed, Jul 3, 2024 at 3:43 PM Srujana Challa wrote: > > > > This patch modifies the code to convert descriptor buffer IOVA > > addresses to virtual addresses during the processing of shadow > > control queue when IOVA mode is PA. Thi

Minutes of Technical Board Meeting, 2024-June-26

2024-07-12 Thread Jerin Jacob
Minutes of Technical Board Meeting, 2024-June-26 Members Attending - -Aaron -Bruce -Hemant -Honnappa -Jerin -Konstantin -Maxime -Morten -Stephen -Thomas NOTE: The technical board meetings every second Wednesday at https://meet.jit.si/DPDK at 3 pm UTC. Meetings are pub

[PATCH v7 14/21] net/ntnic: add clock profile for the NT200A0X smartNIC

2024-07-12 Thread Serhii Iliushyk
Because the ntnic hardware supports multiple different FPGAs with different pipelines and port speeds, the clock profile is not hardcoded into the product, and need to be initialized from software. The clock profile itself is an array of integers that was generated by Silicon Labs ClockBuilder. S

[PATCH v7 20/21] net/ntnic: add GPIO communication for NIMs

2024-07-12 Thread Serhii Iliushyk
For NIM reset sequence GPIO communication is used. After this commit the NIMs supported by ntnic is able to start up and be controlled fully by the adapter. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/nt4ga_link.h| 1 + .../link_mgmt/link_100g/nt4ga_link_100g.c |

[PATCH v7 21/21] net/ntnic: add physical layer control module

2024-07-12 Thread Serhii Iliushyk
Adds functionality to control the physical layer of the OSI model. This takes the form of the module MAC PCS (Media Access Control Physical Coding Sublayer). The functionality is used by the 100G link functionality to establish link. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/n

[PATCH v7 12/21] net/ntnic: add support of the NT200A0X smartNIC

2024-07-12 Thread Serhii Iliushyk
Add ntnic support for NT200A0X NIC Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/adapter/nt4ga_adapter.c | 6 +++ drivers/net/ntnic/meson.build | 1 + .../net/ntnic/nthw/core/include/nthw_fpga.h | 7 +++ .../nthw/core/nt200a0x/nthw_fpga_nt200a0x.c | 54 +

[PATCH v7 19/21] net/ntnic: add QSFP28 support

2024-07-12 Thread Serhii Iliushyk
Includes support for QSFP28 Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntnic_nim.h| 21 ++ drivers/net/ntnic/link_mgmt/nt4ga_link.c | 25 +++ drivers/net/ntnic/nim/i2c_nim.c | 267 ++- drivers/net/ntnic/nim/nim_defines.h | 1 + 4 files

[PATCH v7 18/21] net/ntnic: add QSFP support

2024-07-12 Thread Serhii Iliushyk
Includes support for QSFP and QSFP+. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntnic_nim.h | 10 + .../link_mgmt/link_100g/nt4ga_link_100g.c | 12 +- drivers/net/ntnic/nim/i2c_nim.c | 310 +- drivers/net/ntnic/nim/i2c_nim.h

[PATCH v7 17/21] net/ntnic: add generic NIM and I2C modules

2024-07-12 Thread Serhii Iliushyk
As the ntnic can support different port speeds, it also needs to support different NIMs (Network Interface Module). This commit add the generic NIM support for ntnic, such that the specific modules, such as QSFP28 can be added later. The communication with NIMs is in the form of I2C, so support fo

[PATCH v7 10/21] net/ntnic: add FPGA modules for initialization

2024-07-12 Thread Serhii Iliushyk
New ntnic FPGA modules: - Host Interface (HIF): Basic FPGA info such as prod ID and build time. - Inter-Integrated Circuit Controller (IIC): Use the FPGA to access the other integrated circuits on the ntnic. - PCI Express Gen3 (PCIE3): The FPGA part of PCIe3 initialization, speed te

[PATCH v7 13/21] net/ntnic: add startup and reset sequence for NT200A0X

2024-07-12 Thread Serhii Iliushyk
Adds reset (RST) module for FW 9563. Also adds SDRAM Controller (SDC) module, as it is part of the startup and reset sequence. Signed-off-by: Serhii Iliushyk --- .../include/ntnic_nthw_fpga_rst_nt200a0x.h| 81 +++ drivers/net/ntnic/meson.build | 3 + .../net/ntnic/nthw/cor

[PATCH v7 15/21] net/ntnic: add link management skeleton

2024-07-12 Thread Serhii Iliushyk
Add functionality to read and control the link-state of the ntnic. Note that must functions are not implemented yet. Adds the following eth_dev_ops: - dev_set_link_up - dev_set_link_down - link_update - mac_addr_add - mac_addr_set - set_mc_addr_list - promiscuous_enable Signed-off-by: Serhii Iliu

[PATCH v7 16/21] net/ntnic: add link 100G module ops

2024-07-12 Thread Serhii Iliushyk
As the ntnic can support different speeds, an abstraction layer for 100G speed is needed. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/adapter/nt4ga_adapter.c | 12 - .../link_mgmt/link_100g/nt4ga_link_100g.c | 49 +++ drivers/net/ntnic/meson.build

[PATCH v7 11/21] net/ntnic: add FPGA initialization functionality

2024-07-12 Thread Serhii Iliushyk
Enable FPGA initialization and adds ethdev fw_version_get. Signed-off-by: Serhii Iliushyk --- doc/guides/nics/features/ntnic.ini| 1 + doc/guides/nics/ntnic.rst | 5 + drivers/net/ntnic/adapter/nt4ga_adapter.c | 52 +++- drivers/net/ntnic/meson.build

[PATCH v7 01/21] net/ntnic: add ethdev and makes PMD available

2024-07-12 Thread Serhii Iliushyk
Add initial ntnic ethdev skeleton and register PCI probe functions Update documentation: Device description and feature list Signed-off-by: Serhii Iliushyk --- .mailmap | 1 + MAINTAINERS| 7 doc/guides/nics/features/ntnic.ini

[PATCH v7 08/21] net/ntnic: add adapter initialization

2024-07-12 Thread Serhii Iliushyk
Add interfaces for initialize the adapter Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/adapter/nt4ga_adapter.c | 148 ++ drivers/net/ntnic/include/nt4ga_adapter.h | 40 ++ drivers/net/ntnic/include/ntdrv_4ga.h | 2 + drivers/net/ntnic/meson.build

[PATCH v7 06/21] net/ntnic: add basic eth dev ops to ntnic

2024-07-12 Thread Serhii Iliushyk
Adds support for eth_dev configure, start, stop, close, and infos_get. The internal structs of ntnic is also added and initialized. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntdrv_4ga.h | 17 ++ drivers/net/ntnic/include/ntos_drv.h| 32 drivers/net/ntnic/include/n

[PATCH v7 07/21] net/ntnic: add core platform structures

2024-07-12 Thread Serhii Iliushyk
Adds many of the high level structures needed by the ntnic FPGA modules and adapter control. This is considered the first part of the skeleton of ntnic FPGA support Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/include/ntos_drv.h | 2 + drivers/net/ntnic/meson.build |

[PATCH v7 02/21] net/ntnic: add logging implementation

2024-07-12 Thread Serhii Iliushyk
Adds ntnic specific implementation for logging. NT NIC uses this logging abstraction layer to ensure that FPGA module implementations function both within and outside in DPDK environment Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/meson.build| 2 ++ drivers/net/ntnic/ntlog/ntlog.c

[PATCH v7 04/21] net/ntnic: add NT utilities implementation

2024-07-12 Thread Serhii Iliushyk
Add ntnic utilities. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/meson.build | 2 ++ drivers/net/ntnic/ntnic_ethdev.c | 2 ++ drivers/net/ntnic/ntutil/nt_util.c | 33 +++ drivers/net/ntnic/ntutil/nt_util.h | 43 ++ 4 files changed

[PATCH v7 05/21] net/ntnic: add VFIO module

2024-07-12 Thread Serhii Iliushyk
Adds VFIO functionality and the DMA it requires. The VFIO context is initialized during ntnic ethdev startup. Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/meson.build | 1 + drivers/net/ntnic/ntnic_ethdev.c | 23 +++ drivers/net/ntnic/ntnic_vfio.c | 235

[PATCH v7 03/21] net/ntnic: add minimal initialization for PCI device

2024-07-12 Thread Serhii Iliushyk
add implementation for probe/init and remove/deinit of the PCI device Signed-off-by: Serhii Iliushyk --- drivers/net/ntnic/ntnic_ethdev.c | 109 +-- 1 file changed, 104 insertions(+), 5 deletions(-) diff --git a/drivers/net/ntnic/ntnic_ethdev.c b/drivers/net/ntnic/nt

Re: [PATCH] net/ice: fix sizing of filter hash table

2024-07-12 Thread Bruce Richardson
On Fri, Jul 12, 2024 at 10:40:02AM +0200, David Marchand wrote: > On Thu, Jul 11, 2024 at 6:57 PM Bruce Richardson > wrote: > > > > The hash table used for managing the filter rules in the ice driver was > > dimensioned to a hard-coded 16k, which is insufficient for holding all > > the filters sup

Re: [PATCH 00/19] Wangxun fixes and supports

2024-07-12 Thread David Marchand
On Thu, Jul 11, 2024 at 1:53 PM David Marchand wrote: > On Sat, Jul 6, 2024 at 5:39 AM Ferruh Yigit wrote: > > >>> Fix some bugs in txgbe/ngbe, and support new features in ngbe driver. > > >>> > > >>> Jiawen Wu (19): > > >>> net/txgbe: fix to parse tunnel packets > > >>> net/txgbe: fix flow f

Re: [PATCH v7] eal: verify mmu type for DPDK support (ppc64le)

2024-07-12 Thread David Marchand
On Thu, Jul 11, 2024 at 1:38 AM David Christensen wrote: > > IBM POWER systems support more than one type of memory management unit > (MMU). The Power ISA 3.0 specification, which applies to P9 and later > CPUs, defined a new Radix MMU which, among other things, allows an > anonymous memory page

[PATCH v8 5/5] dts: add API doc generation

2024-07-12 Thread Juraj Linkeš
The tool used to generate DTS API docs is Sphinx, which is already in use in DPDK. The same configuration is used to preserve style with one DTS-specific configuration (so that the DPDK docs are unchanged) that modifies how the sidebar displays the content. Sphinx generates the documentation from

[PATCH v8 4/5] doc: guides and API meson improvements

2024-07-12 Thread Juraj Linkeš
The Sphinx script argument parsing improvement gives us more flexibility going forward, such as the ability to add non-positional arguments. Signed-off-by: Juraj Linkeš Reviewed-by: Luca Vizzarro Reviewed-by: Jeremy Spewock Acked-by: Bruce Richardson Tested-by: Luca Vizzarro Tested-by: Nichol

[PATCH v8 3/5] dts: add API doc sources

2024-07-12 Thread Juraj Linkeš
These sources could be generated with the sphinx-apidoc utility, but that doesn't give us enough flexibility, such as sorting the order of modules or changing the headers of the modules. The sources included in this patch were in fact generated by said utility, but modified to improve the look of

[PATCH v8 2/5] dts: add doc generation dependencies

2024-07-12 Thread Juraj Linkeš
Sphinx imports every Python module when generating documentation from docstrings, meaning all DTS dependencies, including Python version, must be satisfied. By adding Sphinx to DTS dependencies we provide a convenient way to generate the DTS API docs which satisfies all dependencies. Signed-off-by

[PATCH v8 1/5] dts: update params and parser docstrings

2024-07-12 Thread Juraj Linkeš
Address a few errors reported by Sphinx when generating documentation: framework/params/__init__.py:docstring of framework.params.modify_str:3: WARNING: Inline interpreted text or phrase reference start-string without end-string. framework/params/eal.py:docstring of framework.params.eal.Eal

[PATCH v8 0/5] dts: API docs generation

2024-07-12 Thread Juraj Linkeš
The generation is done with Sphinx, which DPDK already uses, with slightly modified configuration of the sidebar present in an if block. Dependencies are installed using Poetry from the dts directory: poetry install --with docs After installing, enter the Poetry shell: poetry shell And then ru

Re: [PATCH v1 1/1] fbarray: fix find_next_n for unaligned length

2024-07-12 Thread David Marchand
Hello, On Wed, Jul 10, 2024 at 1:49 PM Anatoly Burakov wrote: > > When array length is not aligned on a power of 2, we need to mask out the > unaligned bits from the mask whenever we reach the last mask. However, when > both > ignore mask (e.g. due to starting at unaligned bit) and last index ig

Re: [PATCH] net/ice: fix sizing of filter hash table

2024-07-12 Thread David Marchand
On Thu, Jul 11, 2024 at 6:57 PM Bruce Richardson wrote: > > The hash table used for managing the filter rules in the ice driver was > dimensioned to a hard-coded 16k, which is insufficient for holding all > the filters supported by E830 HW. > > Rather than using a hard-coded value which may need u

Re: [PATCH 3/3] net/vdev: fix insert vdev core dump

2024-07-12 Thread Burakov, Anatoly
On 7/12/2024 4:18 AM, Ye, MingjinX wrote: -Original Message- From: Burakov, Anatoly Sent: Friday, July 12, 2024 12:10 AM To: Ye, MingjinX ; dev@dpdk.org Cc: sta...@dpdk.org Subject: Re: [PATCH 3/3] net/vdev: fix insert vdev core dump On 3/14/2024 10:36 AM, Mingjin Ye wrote: Insertin

Re: [EXTERNAL] Re: [PATCH v2] net/virtio_user: fix issue with converting cq descriptor IOVA address to VA

2024-07-12 Thread David Marchand
On Thu, Jul 11, 2024 at 7:46 PM Srujana Challa wrote: > > > +963,8 @@ virtio_user_handle_ctrl_msg_split(struct virtio_user_dev *dev, > > struct vring *vri > > > (struct virtio_pmd_ctrl *)hdr, dlen, > > > nb_dlen); > > > > > > /* Update status */ > > > -

  1   2   >