Re: [PATCH v26 00/15] Log subsystem improvements

2024-10-20 Thread Baruch Even
Hi, I'm not sure about the proper way to do a code review here, feel free to educate me if there is a better way to do it. I couldn't find the actual patches text in the mail and patchwork has no interface to write comments on the line itself but I may very well have missed something. https://pat

[PATCH v7 0/5] power: refactor power management library

2024-10-20 Thread Sivaprasad Tummala
This patchset refactors the power management library, addressing both core and uncore power management. The primary changes involve the creation of dedicated directories for each driver within 'drivers/power/core/*' and 'drivers/power/uncore/*'. This refactor significantly improves code organizati

[PATCH v7 1/5] power: refactor core power management library

2024-10-20 Thread Sivaprasad Tummala
This patch introduces a comprehensive refactor to the core power management library. The primary focus is on improving modularity and organization by relocating specific driver implementations from the 'lib/power' directory to dedicated directories within 'drivers/power/core/*'. The adjustment of m

[PATCH v7 5/5] maintainers: update for drivers/power

2024-10-20 Thread Sivaprasad Tummala
Update maintainers for drivers/power/*. Signed-off-by: Sivaprasad Tummala --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6ea7850093..7e29931be9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1744,6 +1744,7 @@ M: Anatoly Burakov M: David Hunt

[PATCH v7 2/5] power: refactor uncore power management library

2024-10-20 Thread Sivaprasad Tummala
iThis patch refactors the power management library, addressing uncore power management. The primary changes involve the creation of dedicated directories for each driver within 'drivers/power/uncore/*'. The adjustment of meson.build files enables the selective activation of individual drivers. Thi

[PATCH v7 4/5] drivers/power: uncore support for AMD EPYC processors

2024-10-20 Thread Sivaprasad Tummala
This patch introduces driver support for power management of uncore components in AMD EPYC processors. v2: - fixed typo in comments section. - added fabric frequency get support for legacy platforms. Signed-off-by: Sivaprasad Tummala --- drivers/power/amd_uncore/amd_uncore.c | 329 +++

[PATCH v7 0/5] power: refactor power management library

2024-10-20 Thread Sivaprasad Tummala
This patchset refactors the power management library, addressing both core and uncore power management. The primary changes involve the creation of dedicated directories for each driver within 'drivers/power/core/*' and 'drivers/power/uncore/*'. This refactor significantly improves code organizati

[PATCH v7 3/5] test/power: removed function pointer validations

2024-10-20 Thread Sivaprasad Tummala
After refactoring the power library, power management operations are now consistently supported regardless of the operating environment, making function pointer checks unnecessary and thus removed from applications. v2: - removed function pointer validation in l3fwd-power app. Signed-off-by: Siv

RE: [PATCH v2] net/bonding: add user callback for bond xmit policy

2024-10-20 Thread Chaoyong He
> From: Vignesh PS > > Add support to bonding PMD to allow user callback function registration for > TX transmit policy. > > Signed-off-by: Vignesh PS > --- > .mailmap| 1 + > drivers/net/bonding/eth_bond_private.h | 6 ++ > drivers/net/bonding/rte_eth_bo

[PATCH 1/9] net/nfp: extract the function to allocate the PHY

2024-10-20 Thread Chaoyong He
From: Peng Zhang Extract the helper function to allocate the PHY representor port. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- .../net/nfp/flower/nfp_flower_representor.c | 86 --- 1 file changed, 53 insertions(+), 33 deletions(-) diff --git

[PATCH 0/9] support multiple PF flower firmware

2024-10-20 Thread Chaoyong He
This patch series add support of the multiple PF flower firmware. Peng Zhang (9): net/nfp: extract the function to allocate the PHY net/nfp: extract the function to initialize the PF net/nfp: extract the function to allocate PF net/nfp: extract the function to allocate the VF net/nfp: re

[PATCH 4/9] net/nfp: extract the function to allocate the VF

2024-10-20 Thread Chaoyong He
From: Peng Zhang Extract the helper function to allocate the VF representor ports. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- .../net/nfp/flower/nfp_flower_representor.c | 85 +++ 1 file changed, 49 insertions(+), 36 deletions(-) diff --git

[PATCH 2/9] net/nfp: extract the function to initialize the PF

2024-10-20 Thread Chaoyong He
From: Peng Zhang Extract the helper function to initialize the basic information of the PF representor port. The logic of check VF is updated because of changing the order of some logics in the initialization function. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu -

[PATCH 7/9] net/nfp: ignore useless message for multiple PF

2024-10-20 Thread Chaoyong He
From: Peng Zhang When using the multiple PF firmware, there is no PF representor port, so we also need not care about the corresponding control messages anymore. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- drivers/net/nfp/flower/nfp_flower_ctrl.c | 10 +++-

[PATCH 3/9] net/nfp: extract the function to allocate PF

2024-10-20 Thread Chaoyong He
From: Peng Zhang Extract the helper function to allocate the PF representor port. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- .../net/nfp/flower/nfp_flower_representor.c | 58 --- 1 file changed, 38 insertions(+), 20 deletions(-) diff --git

[PATCH 5/9] net/nfp: revise the number of PF representor port

2024-10-20 Thread Chaoyong He
From: Peng Zhang When using the multiple PF firmware, there is no need to create the PF representor port anymore. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- drivers/net/nfp/flower/nfp_flower_representor.c | 12 +--- 1 file changed, 9 insertions(+), 3

[PATCH 6/9] net/nfp: initialize the representor port of the multiple PF

2024-10-20 Thread Chaoyong He
From: Peng Zhang When using multiple PF firmware, add the process of initialize the representor port and revise the corresponding logic in the close function. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- .../net/nfp/flower/nfp_flower_representor.c | 80 ++

[PATCH 9/9] net/nfp: update the Tx and Rx function for multiple PF

2024-10-20 Thread Chaoyong He
From: Peng Zhang Update the Tx function and Rx function for multiple PF and simplify the Rx function to enhance the performance. In the RX function, the packets are processed according the firmware type. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- drivers/net/

[PATCH 8/9] net/nfp: add the new operations for multiple PF

2024-10-20 Thread Chaoyong He
From: Peng Zhang When using the multiple PF firmware, add the new operations and update the port start and stop function. Signed-off-by: Peng Zhang Reviewed-by: Chaoyong He Reviewed-by: Long Wu --- drivers/net/nfp/flower/nfp_flower.c | 21 drivers/net/nfp/flower/nfp_fl

[PATCH] net/hns3: fix variable type

2024-10-20 Thread Jie Hai
The patch fixes variable type in case of data trunction. Fixes: dd4b8bba785f ("net/hns3: support reporting names of registers") Cc: sta...@dpdk.org Signed-off-by: Jie Hai --- drivers/net/hns3/hns3_regs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/hns3/hn

RE: DPDK - PCIe Steering Tags Meeting on 10/23/24

2024-10-20 Thread Wathsala Wathawana Vithanage
Here is the updated RFC https://inbox.dpdk.org/dev/20241021015246.304431-1-wathsala.vithan...@arm.com/#t Thanks --wathsal > > Subject: DPDK - PCIe Steering Tags Meeting on 10/23/24 > > Hi all, > > This is an invitation to discuss adding PCIe steering tags support to DPDK. > We have had brief

RE: [PATCH v2] examples/l3fwd: support setting the data size of mbuf

2024-10-20 Thread Chaoyong He
> On Fri, 18 Oct 2024 05:50:20 + > Chaoyong He wrote: > > > > On Fri, 18 Oct 2024 03:21:28 + > > > Chaoyong He wrote: > > > > > > > > RTE_PKTMBUF_HEADROOM); } > > > > > > + > > > > > > > > > > Not sure why this is needed? What is the problem with the original > code? > > > > > Are you tr

Re: [PATCH] net/sxe: add build and doc infrastructure

2024-10-20 Thread Stephen Hemminger
On Sat, 19 Oct 2024 19:42:03 -0700 Jie Liu wrote: > Adding a minimum maintainable directory structure for the > network driver and request maintenance of the sxe driver. > > Signed-off-by: Jie Liu Even the minimum version patch has to compile cleanly, this one doesn't. Please fix and resubmit

Re: [PATCH v6 2/5] power: refactor uncore power management library

2024-10-20 Thread Stephen Hemminger
On Sun, 20 Oct 2024 09:22:29 + Sivaprasad Tummala wrote: > diff --git a/lib/power/rte_power_uncore.c b/lib/power/rte_power_uncore.c > index 48c75a5da0..f11238cc34 100644 > --- a/lib/power/rte_power_uncore.c > +++ b/lib/power/rte_power_uncore.c > @@ -10,100 +10,53 @@ > > #include "power_com

Re: [PATCH v6 2/5] power: refactor uncore power management library

2024-10-20 Thread Stephen Hemminger
On Sun, 20 Oct 2024 09:22:29 + Sivaprasad Tummala wrote: > +uint32_t > +rte_power_get_uncore_freq(unsigned int pkg, unsigned int die) > +{ > + RTE_ASSERT(global_uncore_ops != NULL); All these RTE_ASSERT calls seem like a good idea, they really don't help. If RTE_ASSERT fails it prints a

Re: [PATCH 4/4] net/hns3: support outer VLAN flow match

2024-10-20 Thread fengchengwen
On 2024/10/19 0:41, Stephen Hemminger wrote: > On Fri, 18 Oct 2024 14:19:41 +0800 > Jie Hai wrote: > >> @@ -306,11 +324,16 @@ hns3_parse_devargs(struct rte_eth_dev *dev) >> &hns3_parse_dev_caps_mask, &dev_caps_mask); >> (void)rte_kvargs_process(kvlist, HNS3_DEVARG_MBX

[RFC v3 1/2] pci: introduce the PCIe TLP Processing Hints API

2024-10-20 Thread Wathsala Vithanage
Extend the PCI driver and the library to extract the Steering Tag (ST) for a given Processor/Processor Container and Cache ID pair and validate a Processing Hint from a TPH _DSM associated with a root port device. The rte_pci_device structure passed into the rte_pci_extract_tph_st() function could

[RFC v3 2/2] ethdev: introduce the cache stashing hints API

2024-10-20 Thread Wathsala Vithanage
Extend the ethdev library to enable the stashing of different data objects, such as the ones listed below, into CPU caches directly from the NIC. - Rx/Tx queue descriptors - Rx packets - Packet headers - packet payloads - Data of a packet at an offset from the start of the packet The APIs are des

[RFC v3 0/2] An API for Stashing Packets into CPU caches

2024-10-20 Thread Wathsala Vithanage
DPDK applications benefit from Direct Cache Access (DCA) features like Intel DDIO and Arm's write-allocate-to-SLC. However, those features do not allow fine-grained control of direct cache access, such as stashing packets into upper-level caches (L2 caches) of a processor or the shared cache of a c

[dpdk-dev] [PATCH 1/2] common/cnxk: support NPC flow on cn20k

2024-10-20 Thread psatheesh
From: Satheesh Paul ROC changes to support NPC flow on cn20k. Signed-off-by: Satheesh Paul Reviewed-by: Kiran Kumar K --- drivers/common/cnxk/roc_mbox.h | 144 +- drivers/common/cnxk/roc_nix.h | 18 +- drivers/common/cnxk/roc_nix_mcast.c | 40 +- drivers/common/cn

[dpdk-dev] [PATCH 2/2] net/cnxk: support rte flow on cn20k

2024-10-20 Thread psatheesh
From: Satheesh Paul Support for rte flow in cn20k. Signed-off-by: Satheesh Paul Reviewed-by: Kiran Kumar K --- drivers/net/cnxk/cn10k_ethdev.c| 8 +- drivers/net/cnxk/cn10k_flow.h | 21 -- drivers/net/cnxk/cn20k_ethdev.c| 4 + drivers/net/cnxk/cnxk_ethdev_devargs

[PATCH v6 4/5] drivers/power: uncore support for AMD EPYC processors

2024-10-20 Thread Sivaprasad Tummala
This patch introduces driver support for power management of uncore components in AMD EPYC processors. v2: - fixed typo in comments section. - added fabric frequency get support for legacy platforms. Signed-off-by: Sivaprasad Tummala --- drivers/power/amd_uncore/amd_uncore.c | 329 +++

[PATCH v6 1/5] power: refactor core power management library

2024-10-20 Thread Sivaprasad Tummala
This patch introduces a comprehensive refactor to the core power management library. The primary focus is on improving modularity and organization by relocating specific driver implementations from the 'lib/power' directory to dedicated directories within 'drivers/power/core/*'. The adjustment of m

[PATCH v6 0/5] power: refactor power management library

2024-10-20 Thread Sivaprasad Tummala
This patchset refactors the power management library, addressing both core and uncore power management. The primary changes involve the creation of dedicated directories for each driver within 'drivers/power/core/*' and 'drivers/power/uncore/*'. This refactor significantly improves code organizati

[PATCH v6 2/5] power: refactor uncore power management library

2024-10-20 Thread Sivaprasad Tummala
iThis patch refactors the power management library, addressing uncore power management. The primary changes involve the creation of dedicated directories for each driver within 'drivers/power/uncore/*'. The adjustment of meson.build files enables the selective activation of individual drivers. Thi

[PATCH v6 0/5] power: refactor power management library

2024-10-20 Thread Sivaprasad Tummala
This patchset refactors the power management library, addressing both core and uncore power management. The primary changes involve the creation of dedicated directories for each driver within 'drivers/power/core/*' and 'drivers/power/uncore/*'. This refactor significantly improves code organizati

[PATCH v6 5/5] maintainers: update for drivers/power

2024-10-20 Thread Sivaprasad Tummala
Update maintainers for drivers/power/*. Signed-off-by: Sivaprasad Tummala --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 6ea7850093..7e29931be9 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1744,6 +1744,7 @@ M: Anatoly Burakov M: David Hunt

[PATCH v6 3/5] test/power: removed function pointer validations

2024-10-20 Thread Sivaprasad Tummala
After refactoring the power library, power management operations are now consistently supported regardless of the operating environment, making function pointer checks unnecessary and thus removed from applications. v2: - removed function pointer validation in l3fwd-power app. Signed-off-by: Siv

[PATCH] maintainers: update for hns3

2024-10-20 Thread Jie Hai
This email address is bouncing, remove it from MAINTAINERS. Signed-off-by: Jie Hai --- MAINTAINERS | 1 - 1 file changed, 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index 6ea785009341..cd78bc7db172 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -750,7 +750,6 @@ F: doc/guides/nics/featu