Re: [PATCH v2] app/testpmd: add flush multicast MAC address command

2023-10-07 Thread huangdengdui
On 2023/9/29 1:23, Ferruh Yigit wrote: > On 8/2/2023 7:23 AM, Dengdui Huang wrote: >> Add command to flush multicast MAC address >> Usage: >> mcast_addr flush : >> flush multicast MAC address on port_id >> >> Signed-off-by: Dengdui Huang >> --- >> app/test-pmd/cmdline.c

Re: [PATCH v3 1/2] app/testpmd: fix help string

2023-10-07 Thread huangdengdui
On 2023/10/8 10:47, fengchengwen wrote: > Hi Dengdui, > > On 2023/10/8 9:56, Dengdui Huang wrote: >> Command help string is missing 'mcast_addr add|remove'. >> This patch add it. >> >> Fixes: 8fff667578a7 ("app/testpmd: new command to add/remove multicast MAC >> addresses") >> Cc: sta...@dpdk.

Re: [PATCH v6 1/2] bus/pci: fix secondary process PCI uio resource map problem

2024-07-14 Thread huangdengdui
On 2024/7/2 15:40, Chaoyong He wrote: > From: Zerun Fu > > For the primary process, the logic loops all BARs and will skip > the map of BAR with an invalid physical address (0), also will > assign 'uio_res->nb_maps' with the real mapped BARs number. But > for the secondary process, instead of l

Re: [PATCH] app/testpmd: fix segment fault with invalid queue id

2023-05-22 Thread huangdengdui
On 2023/5/16 23:12, Stephen Hemminger wrote: > On Tue, 16 May 2023 19:00:21 +0800 > Dengdui Huang wrote: > >> When input queue id is invalid, it will lead to >> Segmentation fault, like: >> >> dpdk-testpmd -a :01:00.0 -- -i >> testpmd> show port 0 txq/rxq 99 desc 0 status >> Segmentation fa

Re: [PATCH v2 1/2] ethdev: add API to check queue ID validity

2023-05-24 Thread huangdengdui
On 2023/5/22 21:58, Andrew Rybchenko wrote: > On 5/22/23 16:09, Dengdui Huang wrote: >> The API rte_eth_dev_is_valid_rxq/txq checks >> the port ID validity and then the Rx/Tx queue ID is valid. > > What is valid Tx/Rx queue? It depends on on caller > expectations. Some functions are satisfied with

Re: [PATCH v2 1/2] ethdev: add API to check queue ID validity

2023-06-01 Thread huangdengdui
On 2023/6/2 6:13, Ferruh Yigit wrote: > On 5/31/2023 5:31 PM, Ferruh Yigit wrote: >> On 5/22/2023 2:58 PM, Andrew Rybchenko wrote: >>> On 5/22/23 16:09, Dengdui Huang wrote: The API rte_eth_dev_is_valid_rxq/txq checks the port ID validity and then the Rx/Tx queue ID is valid. >>> >>> What

Re: [PATCH] net/hns3: fix Rx packet truncation when KEEP CRC enabled

2024-02-27 Thread huangdengdui
On 2024/2/27 0:43, Ferruh Yigit wrote: > On 2/26/2024 3:16 AM, Jie Hai wrote: >> On 2024/2/23 21:53, Ferruh Yigit wrote: >>> On 2/20/2024 3:58 AM, Jie Hai wrote: Hi, Ferruh, Thanks for your review. On 2024/2/7 22:15, Ferruh Yigit wrote: > On 2/6/2024 1:10 AM, Jie Hai

Re: [PATCH] net/hns3: fix Rx packet truncation when KEEP CRC enabled

2024-02-28 Thread huangdengdui
On 2024/2/28 21:07, Ferruh Yigit wrote: > On 2/28/2024 2:27 AM, huangdengdui wrote: >> >> >> On 2024/2/27 0:43, Ferruh Yigit wrote: >>> On 2/26/2024 3:16 AM, Jie Hai wrote: >>>> On 2024/2/23 21:53, Ferruh Yigit wrote: >>>>&g

Re: [PATCH] net/hns3: fix Rx packet truncation when KEEP CRC enabled

2024-02-29 Thread huangdengdui
On 2024/2/29 17:25, Ferruh Yigit wrote: > On 2/29/2024 3:58 AM, huangdengdui wrote: >> >> >> On 2024/2/28 21:07, Ferruh Yigit wrote: >>> On 2/28/2024 2:27 AM, huangdengdui wrote: >>>> >>>> >>>> On 2024/2/27 0:43, Ferruh Yigit wro

Re: [PATCH] app/testpmd: fix crash in multi-process packet forwarding

2024-01-29 Thread huangdengdui
On 2024/1/26 14:23, fengchengwen wrote: > Hi Dengdui, > > On 2024/1/26 10:41, Dengdui Huang wrote: >> On multi-process scenario, each process creates flows based on the >> number of queues. When nbcore is greater than 1, multiple cores may >> use the same queue to forward packet, like: >> dpdk-

Re: [PATCH v2 0/6] support setting lanes

2024-05-16 Thread huangdengdui
Hi, Ferruh Sorry for replying your email very late. The answers to your questions are as follows. Please correct me if I am wrong. On 2024/4/4 21:58, Ferruh Yigit wrote: > > Hi Dengdui, Damodharam, > > As details of the implementation under discussion, I have a high level > question. > > Why/wh

Re: [PATCH v3] ethdev: Add link_speed lanes support

2024-06-19 Thread huangdengdui
Hi Damodharam Here are some suggestions. See below. On 2024/6/18 4:34, Damodharam Ammepalli wrote: > Update the eth_dev_ops structure with new function vectors > to get, get capabilities and set ethernet link speed lanes. > Update the testpmd to provide required config and information > display in

Re: [PATCH v3] ethdev: Add link_speed lanes support

2024-06-25 Thread huangdengdui
On 2024/6/26 5:07, Damodharam Ammepalli wrote: > On Wed, Jun 19, 2024 at 8:23 PM huangdengdui wrote: >> >> Hi Damodharam >> Here are some suggestions. See below. >> > Thank you for the review. > >> On 2024/6/18 4:34, Damodharam Ammepalli wrote: >&g

Re: [PATCH] devtools: fix version variable not initialized

2024-06-27 Thread huangdengdui
On 2024/6/27 22:28, David Marchand wrote: > On Wed, Apr 17, 2024 at 11:32 AM Dengdui Huang > wrote: >> >> The version variable is not initialized. Therefore, if the -V option >> is not specified, the value of $version is obtained from the context, >> which may cause the version map parsing fail

Re: [PATCH v4 0/6] refine argparse library

2024-07-09 Thread huangdengdui
For the patchset, Reviewed-by: Dengdui Huang On 2024/3/18 19:18, Chengwen Feng wrote: > I found a couple of issues when I revisited the argparse_autotest > output, so got this patchset. > > Chengwen Feng (6): > argparse: refine error message > argparse: remove dead code > argparse: replace

Re: [PATCH] app/testpmd: add flush multicast MAC address command

2023-08-01 Thread huangdengdui
在 2023/8/1 23:18, Stephen Hemminger 写道: > On Tue, 1 Aug 2023 10:43:04 +0800 > Dengdui Huang wrote: > >> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c >> index 0d0723f659..2d9d925776 100644 >> --- a/app/test-pmd/cmdline.c >> +++ b/app/test-pmd/cmdline.c >> @@ -494,6 +494,9 @@ sta

Re: [PATCH] app/testpmd: add flush multicast MAC address command

2023-08-01 Thread huangdengdui
在 2023/8/1 23:18, Stephen Hemminger 写道: > On Tue, 1 Aug 2023 10:43:04 +0800 > Dengdui Huang wrote: > >> diff --git a/app/test-pmd/cmdline.c b/app/test-pmd/cmdline.c >> index 0d0723f659..2d9d925776 100644 >> --- a/app/test-pmd/cmdline.c >> +++ b/app/test-pmd/cmdline.c >> @@ -494,6 +494,9 @@ sta

Re: [PATCH] app/testpmd: add flush multicast MAC address command

2023-08-01 Thread huangdengdui
在 2023/8/2 12:16, Stephen Hemminger 写道: > On Wed, 2 Aug 2023 10:41:46 +0800 > huangdengdui wrote: > >> 在 2023/8/1 23:18, Stephen Hemminger 写道: >>> On Tue, 1 Aug 2023 10:43:04 +0800 >>> Dengdui Huang wrote: >>> >>>> diff --git a/app/

Re: [PATCH 3/3] app/testpmd: support setting lanes

2024-03-18 Thread huangdengdui
On 2024/3/16 5:47, Damodharam Ammepalli wrote: > On Tue, Mar 12, 2024 at 12:52 AM Dengdui Huang > wrote: >> >> Extended speed command for setting lane number and >> Show info print lane number. >> >> Signed-off-by: Dengdui Huang >> --- >> app/test-pmd/cmdline.c | 110 +++

Re: [PATCH 1/3] ethdev: support setting lanes

2024-03-19 Thread huangdengdui
On 2024/3/19 11:02, Stephen Hemminger wrote: > On Tue, 12 Mar 2024 15:52:36 +0800 > Dengdui Huang wrote: > >> -ret = snprintf(str, len, "Link up at %s %s %s", >> +ret = snprintf(str, len, "Link up at %s %ulanes %s %s", > > Don't you want a space after %u? > > Could yo

Re: [PATCH 0/3] support setting lanes

2024-03-20 Thread huangdengdui
On 2024/3/20 20:31, Ferruh Yigit wrote: > On 3/18/2024 9:26 PM, Damodharam Ammepalli wrote: >> On Mon, Mar 18, 2024 at 7:56 AM Thomas Monjalon wrote: >>> >>> 12/03/2024 08:52, Dengdui Huang: Some speeds can be achieved with different number of lanes. For example, 100Gbps can be achiev

Re: [PATCH 0/3] support setting lanes

2024-03-21 Thread huangdengdui
On 2024/3/21 16:28, Thomas Monjalon wrote: > 21/03/2024 03:02, huangdengdui: >> >> On 2024/3/20 20:31, Ferruh Yigit wrote: >>> On 3/18/2024 9:26 PM, Damodharam Ammepalli wrote: >>>> On Mon, Mar 18, 2024 at 7:56 AM Thomas Monjalon >>>> wr

Re: [PATCH v2 1/6] ethdev: support setting lanes

2024-03-24 Thread huangdengdui
On 2024/3/22 21:58, Thomas Monjalon wrote: > 22/03/2024 08:09, Dengdui Huang: >> -#define RTE_ETH_LINK_SPEED_10G RTE_BIT32(8) /**< 10 Gbps */ >> -#define RTE_ETH_LINK_SPEED_20G RTE_BIT32(9) /**< 20 Gbps */ >> -#define RTE_ETH_LINK_SPEED_25G RTE_BIT32(10) /**< 25 Gbps */ >> -#define

Re: [PATCH v2 1/6] ethdev: support setting lanes

2024-03-30 Thread huangdengdui
;>>> >>>> 在 2024/3/26 18:30, Thomas Monjalon 写道: >>>>> 26/03/2024 02:42, lihuisong (C): >>>>>> 在 2024/3/25 17:30, Thomas Monjalon 写道: >>>>>>> 25/03/2024 07:24, huangdengdui: >>>>>>>> On

Re: [PATCH v2 1/6] ethdev: support setting lanes

2024-04-02 Thread huangdengdui
On 2024/4/2 4:07, Thomas Monjalon wrote: > 30/03/2024 12:38, huangdengdui: >> But, there are different solutions for the device to report the setting >> lane capability, as following: >> 1. Like the current patch, reporting device capabilities in speed and >>lane

Re: [PATCH] app/testpmd: handle IEEE1588 init fail

2024-04-07 Thread huangdengdui
On 2024/4/6 0:44, Stephen Hemminger wrote: > On Sat, 30 Mar 2024 15:44:09 +0800 > Dengdui Huang wrote: > >> When the port's timestamping function failed to initialize >> (for example, the device does not support PTP), the packets >> received by the hardware do not contain the timestamp. >> In

Re: [PATCH] app/testpmd: handle IEEE1588 init fail

2024-04-08 Thread huangdengdui
On 2024/4/8 16:45, Ferruh Yigit wrote: > On 4/8/2024 6:52 AM, huangdengdui wrote: >> >> >> On 2024/4/6 0:44, Stephen Hemminger wrote: >>> On Sat, 30 Mar 2024 15:44:09 +0800 >>> Dengdui Huang wrote: >>> >>>> When the port's

Re: [PATCH] app/testpmd: handle IEEE1588 init fail

2024-04-17 Thread huangdengdui
On 2024/4/9 10:50, Stephen Hemminger wrote: > On Tue, 9 Apr 2024 10:06:01 +0800 > huangdengdui wrote: > >> On 2024/4/8 16:45, Ferruh Yigit wrote: >>> On 4/8/2024 6:52 AM, huangdengdui wrote: >>>> >>>> >>>> On 2024/4/6 0:44, Stephen

Re: [PATCH v4 1/2] ethdev: add API to check if queue is valid

2023-06-04 Thread huangdengdui
On 2023/6/2 20:47, Ferruh Yigit wrote: > On 6/2/2023 8:52 AM, Dengdui Huang wrote: >> The API rte_eth_dev_is_valid_rxq/txq which >> is used to check if Rx/Tx queue is valid. >> If the queue has been setup, it is considered valid. >> >> Signed-off-by: Dengdui Huang >> --- >> doc/guides/rel_notes/r

Re: [PATCH v6] app/dma-perf: introduce dma-perf application

2023-06-13 Thread huangdengdui
Hi Cheng, Few comments inline. Please check. Thanks, Dengdui On 2023/6/13 12:31, Cheng Jiang wrote: > There are many high-performance DMA devices supported in DPDK now, and > these DMA devices can also be integrated into other modules of DPDK as > accelerators, such as Vhost. Before integrating

Re: [PATCH v4 00/42] replace strerror

2024-11-01 Thread huangdengdui
On 2024/10/26 5:56, Thomas Monjalon wrote: > 24/10/2024 08:47, huangdengdui: >> On 2024/10/23 23:42, Stephen Hemminger wrote: >>> On Wed, 23 Oct 2024 16:28:10 +0800 >>> Dengdui Huang wrote: >>> >>>> The function strerror() is insecure in a mu

Re: [PATCH v4 00/42] replace strerror

2024-10-23 Thread huangdengdui
Hi, Stephen Hemminger On 2024/10/23 23:42, Stephen Hemminger wrote: > On Wed, 23 Oct 2024 16:28:10 +0800 > Dengdui Huang wrote: > >> The function strerror() is insecure in a multi-thread environment. >> It is better to use rte_strerror() instead of strerror(). >> In this patchset, only the libs

Re: [PATCH 00/43] replace strerror

2024-11-06 Thread huangdengdui
On 2024/11/7 2:57, Stephen Hemminger wrote: > On Tue, 14 Nov 2023 16:24:56 +0800 > Dengdui Huang wrote: > >> The function strerror() is insecure in a multi-thread environment. >> This series of patches fix it. In this patchset, only the libs and >> drivers are modified. > > strerror is ok in m

Re: [PATCH v4] net/hns3: fix Rx packet without CRC data

2024-11-29 Thread huangdengdui
On 2024/11/30 1:12, Stephen Hemminger wrote: > On Fri, 29 Nov 2024 09:36:43 +0800 > Jie Hai wrote: > >>> + >>> +static inline void >>> +hns3_recalculate_crc(struct rte_mbuf *m) >>> +{ >>> + char *append_data; >>> + uint32_t crc; >>> + >>> + crc = rte_net_crc_calc(rte_pktmbuf_mtod(m, void

Re: [PATCH v3 3/3] net/hns3: fix Rx packet without CRC data

2024-11-25 Thread huangdengdui
On 2024/11/26 1:45, Stephen Hemminger wrote: > On Fri, 19 Jul 2024 17:04:15 +0800 > Jie Hai wrote: > >> +static inline void >> +hns3_recalculate_crc(struct rte_mbuf *m) >> +{ >> +char *append_data; >> +uint32_t crc; >> + >> +crc = rte_net_crc_calc(rte_pktmbuf_mtod(m, void *), >> +

Re: [PATCH] mbuf: add packet offload Rx flag for keep CRC

2025-02-06 Thread huangdengdui
On 2025/1/24 22:34, Morten Brørup wrote: >> From: Dengdui Huang [mailto:huangdeng...@huawei.com] >> Sent: Friday, 24 January 2025 11.00 >> >> After discussion[1], the drivers do not include the CRC in the packet >> length calculation. This will cause users to be confused about whether >> the mbuf

Re: [PATCH] mbuf: add packet offload Rx flag for keep CRC

2025-02-06 Thread huangdengdui
On 2025/1/25 1:34, Stephen Hemminger wrote: > On Fri, 24 Jan 2025 17:59:57 +0800 > Dengdui Huang wrote: > >> diff --git a/lib/mbuf/rte_mbuf.c b/lib/mbuf/rte_mbuf.c >> index 559d5ad8a7..c828200ea1 100644 >> --- a/lib/mbuf/rte_mbuf.c >> +++ b/lib/mbuf/rte_mbuf.c >> @@ -771,7 +771,7 @@ const char

Re: [PATCH 1/3] net/hns3: fix simple Tx path incorrect free the mbuf

2025-01-08 Thread huangdengdui
On 2025/1/9 0:57, Stephen Hemminger wrote: > On Wed, 8 Jan 2025 10:40:43 +0800 > Jie Hai wrote: > >> On 2024/12/31 1:55, Stephen Hemminger wrote: >>> On Mon, 30 Dec 2024 14:54:03 +0800 >>> Jie Hai wrote: >>> From: Jie Hai To: , , , , , Chengwen Feng , "Wei Hu (Xavie

Re: [PATCH] examples/l3fwd: add option to set refetch offset

2025-01-14 Thread huangdengdui
On 2025/1/11 1:20, Stephen Hemminger wrote: > This will make it slower for many platforms. > GCC will unroll a loop of fixed small size, which is what we want. Do you mean to replace option with a macro? But most of prefetch_offset are used with the nb_rx, So using macros is the same as using o

Re: [PATCH] examples/l3fwd: add option to set refetch offset

2025-01-14 Thread huangdengdui
On 2025/1/11 1:19, Stephen Hemminger wrote: > On Fri, 10 Jan 2025 17:37:15 +0800 > Dengdui Huang wrote: > >> +#define DEFAULT_PREFECH_OFFSET 4 > > Spelling I made a mistake. I'll fix it for the next version.

Re: [PATCH v2] mbuf: add fast free bulk function

2025-01-15 Thread huangdengdui
On 2025/1/15 17:38, Morten Brørup wrote: >> From: huangdengdui [mailto:huangdeng...@huawei.com] >> Sent: Wednesday, 15 January 2025 07.52 >> >> On 2025/1/15 0:39, Morten Brørup wrote: >>> mbuf: add fast free bulk function >>> >>> When putti

Re: [PATCH 1/3] net/hns3: fix simple Tx path incorrect free the mbuf

2025-01-14 Thread huangdengdui
On 2025/1/10 1:03, Stephen Hemminger wrote: > On Thu, 9 Jan 2025 15:43:12 +0800 > huangdengdui wrote: > >> On 2025/1/9 0:57, Stephen Hemminger wrote: >>> On Wed, 8 Jan 2025 10:40:43 +0800 >>> Jie Hai wrote: >>> >>>> On 2024/12/31 1:55

Re: [PATCH v2] mbuf: add fast free bulk function

2025-01-14 Thread huangdengdui
On 2025/1/15 0:39, Morten Brørup wrote: > mbuf: add fast free bulk function > > When putting an mbuf back into its mempool, there are certain requirements > to the mbuf. Specifically, some of its fields must be initialized. > > These requirements are in fact invariants about free mbufs, held in

Re: [PATCH] examples/l3fwd: optimize packet prefetch

2025-01-09 Thread huangdengdui
On 2025/1/8 21:42, Konstantin Ananyev wrote: > > >> >> The prefetch window depending on the hardware platform. The current prefetch >> policy may not be applicable to all platforms. In most cases, the number of >> packets received by Rx burst is small (64 is used in most performance >> reports

Re: [PATCH 0/2] fix the problem of dma-perf infinite loop

2025-03-23 Thread huangdengdui
On 2025/3/21 15:49, David Marchand wrote: > On Fri, Mar 21, 2025 at 5:03 AM Dengdui Huang wrote: >> >> After CPU isolation is configured, an infinite loop occurs when >> dma-perf is executed using the default config file. >> >> This patchset fix it. >> >> Dengdui Huang (2): >> eal: fix wake a

Re: [PATCH 2/2] app/dma-perf: fix infinite loop

2025-03-23 Thread huangdengdui
On 2025/3/21 23:58, Stephen Hemminger wrote: > On Fri, 21 Mar 2025 12:03:16 +0800 > Dengdui Huang wrote: > >> When a core that is not used by the rte is specified in the config >> for testing, the problem of infinite loop occurs. The root cause >> is that the program waits for the completion of