[PATCH v3 4/4] app/testpmd: support disable DCB command

After the "port config 0 dcb ..." command is invoked, no command is available to disable DCB. This commit introduces disable DCB when num_tcs is 1, so user could disable the DCB by command: port config 0 dcb vt off 1 pfc off Signed-off-by: Chengwen Feng --- app/test-pmd

[PATCH v3 3/4] app/testpmd: support don't adjust queue num in DCB command

ot; command don't adjust Rx/Tx queue number. The new command: port config 0 dcb vt off 4 pfc off keep-qnum If this parameter is not specified, the Rx/Tx queue number was adjusted by default. Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 83 ++

[PATCH v3 1/4] app/testpmd: remove restrict of number of TCs in DCB command

Currently, the "port config 0 dcb ..." command only supports 4 or 8 TCs. Other number of TCs may be used in actual applications. This commit removes this restriction. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 4 ++--

[PATCH v3 2/4] app/testpmd: support config prio-tc map in DCB command

4 pfc off prio-tc 0:1 1:2 2:3 ... If this parameter is not specified, the prio-tc map is configured by default. Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 119 ++-- app/test-pmd/testpmd.c | 21 ++-- ap

[PATCH v3 0/4] enhance testpmd DCB command

This patchset enhance the testpmd DCB command features: 1) remove restrict of number of TCs (currently its 4 or 8) 2) support config prio-tc map 3) support don't adjust queue num 4) support disable DCB Chengwen Feng (4): app/testpmd: remove restrict of number of TCs in DCB command app/te

[PATCH v2 4/4] app/testpmd: support disable DCB command

After the "port config 0 dcb ..." command is invoked, no command is available to disable DCB. This commit introduces disable DCB when num_tcs is 1, so user could disable the DCB by command: port config 0 dcb vt off 1 pfc off Signed-off-by: Chengwen Feng --- app/test-pmd

[PATCH v2 3/4] app/testpmd: support don't adjust queue num in DCB command

ot; command don't adjust Rx/Tx queue number. The new command: port config 0 dcb vt off 4 pfc off keep-qnum If this parameter is not specified, the Rx/Tx queue number was adjusted by default. Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 83 ++

[PATCH v2 0/4] enhance testpmd DCB command

This patchset enhance the testpmd DCB command features: 1) remove restrict of number of TCs (currently its 4 or 8) 2) support config prio-tc map 3) support don't adjust queue num 4) support disable DCB Chengwen Feng (4): app/testpmd: remove restrict of number of TCs in DCB command app/te

[PATCH v2 2/4] app/testpmd: support config prio-tc map in DCB command

4 pfc off prio-tc 0:1 1:2 2:3 ... If this parameter is not specified, the prio-tc map is configured by default. Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 119 ++-- app/test-pmd/testpmd.c | 21 ++-- ap

[PATCH v2 1/4] app/testpmd: remove restrict of number of TCs in DCB command

Currently, the "port config 0 dcb ..." command only supports 4 or 8 TCs. Other number of TCs may be used in actual applications. This commit removes this restriction. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 4 ++--

[PATCH 0/4] enhance testpmd DCB command

This patchset enhance the testpmd DCB command features: 1) remove restrict of number of TCs (currently its 4 or 8) 2) support config prio-tc map 3) support don't adjust queue num 4) support disable DCB Chengwen Feng (4): app/testpmd: remove restrict of number of TCs in DCB command app/te

[PATCH 3/4] app/testpmd: support don't adjust queue num in DCB command

ot; command don't adjust Rx/Tx queue number. The new command: port config 0 dcb vt off 4 pfc off keep-qnum If this parameter is not specified, the Rx/Tx queue number was adjusted by default. Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 83 ++

[PATCH 4/4] app/testpmd: support disable DCB command

After the "port config 0 dcb ..." command is invoked, no command is available to disable DCB. This commit introduces disable DCB when num_tcs is 1, so user could disable the DCB by command: port config 0 dcb vt off 1 pfc off Signed-off-by: Chengwen Feng --- app/test-pmd

[PATCH 2/4] app/testpmd: support config prio-tc map in DCB command

4 pfc off prio-tc 0:1 1:2 2:3 ... If this parameter is not specified, the prio-tc map is configured by default. Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 119 ++-- app/test-pmd/testpmd.c | 21 ++-- ap

[PATCH 1/4] app/testpmd: remove restrict of number of TCs in DCB command

Currently, the "port config 0 dcb ..." command only supports 4 or 8 TCs. Other number of TCs may be used in actual applications. This commit removes this restriction. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 4 ++--

[PATCH] bus: invoke TAILQ_REMOVE when bus cleanup

0eada9d9 ("bus/vmbus: support cleanup") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- drivers/bus/pci/pci_common.c | 1 + drivers/bus/uacce/uacce.c| 1 + drivers/bus/vdev/vdev.c | 1 + drivers/bus/vmbus/vmbus_common.c | 1 + 4 files changed, 4 insertions(+)

[PATCH 0/2] fix testpmd DCB command

This patchset contains two commit which found when debug DCB module. Chengwen Feng (2): app/testpmd: support display DCB all priority's TC map app/testpmd: fix segment fault when config DCB app/test-pmd/cmdline.c | 6 ++ app/test-pmd/config.c | 4 ++-- 2 files changed, 8 inser

[PATCH 2/2] app/testpmd: fix segment fault when config DCB

The "port config dcb ..." command will segment fault when input with invalid port id, this patch fixes it. Fixes: 9b53e542e9e1 ("app/testpmd: add priority flow control") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- app/test-pmd/cmdline.c | 6 ++ 1 file c

[PATCH 1/2] app/testpmd: support display DCB all priority's TC map

Currently, the "show port dcb_tc" command displays only the mapping in the number of TCs. This patch fixes it by show all priority's TC mapping. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- app/test-pmd/config.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-

[PATCH v4 4/7] app/test: extract with keys testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the with keys tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 60 +- 1 file changed, 36 insertions(+), 24

[PATCH v4 7/7] app/test: add process opt testcase for kvargs

This commit adds rte_kvargs_process_opt() API's testcase. Signed-off-by: Chengwen Feng --- app/test/test_kvargs.c | 106 ++--- 1 file changed, 90 insertions(+), 16 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test/test_kvargs.c index d1e66

[PATCH v4 6/7] app/test: extract parse empty elements testcase for kvargs

Extract parse empty elements test as one stand-alone testcase. And also fix the kvlist was not released when the branch fails. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 68 ++ 1 file changed, 29 insertions

[PATCH v4 2/7] app/test: extract basic token count testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the basic token count tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 120 - 1 file changed, 45 insertions(+), 75

[PATCH v4 3/7] app/test: extract without keys testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the without keys tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 115 ++--- 1 file changed, 63 insertions(+), 52

[PATCH v4 1/7] app/test: introduce UT suite framework for kvargs

Introduce unit test suite framework for test_kvargs.c. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test/test_kvargs.c index 7a60cac4c1

[PATCH v4 0/7] refactor kvargs test

/project/dpdk/cover/20231103073811.13196-1-fengcheng...@huawei.com/ Chengwen Feng (7): app/test: introduce UT suite framework for kvargs app/test: extract basic token count testcase for kvargs app/test: extract without keys testcase for kvargs app/test: extract with keys testcase for kvargs

[PATCH v4 5/7] app/test: extract parse list value testcase for kvargs

Extract parse list value test as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 54 +- 1 file changed, 32 insertions(+), 22 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test

[PATCH] dpdk: remove redundant null check when parse kvargs

normal case, so this commit also contain it. Signed-off-by: Chengwen Feng --- drivers/baseband/la12xx/bbdev_la12xx.c| 2 +- drivers/baseband/null/bbdev_null.c| 2 +- .../baseband/turbo_sw/bbdev_turbo_software.c | 2 +- drivers/bus/ifpga/ifpga_common.h | 4

[PATCH v3 6/8] app/test: extract parse empty elements testcase for kvargs

Extract parse empty elements test as one stand-alone testcase. And also fix the kvlist was not released when the branch fails. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 68 ++ 1 file changed, 29 insertions

[PATCH v3 8/8] maintainers: update for kvargs library

Add myself for voluntary maintenance of the kvargs library. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index cd78bc7db1..f0bcebc651 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1718,6

[PATCH v3 7/8] app/test: add process opt testcase for kvargs

This commit adds rte_kvargs_process_opt() API's testcase. Signed-off-by: Chengwen Feng --- app/test/test_kvargs.c | 106 ++--- 1 file changed, 90 insertions(+), 16 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test/test_kvargs.c index d1e66

[PATCH v3 3/8] app/test: extract without keys testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the without keys tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 115 ++--- 1 file changed, 63 insertions(+), 52

[PATCH v3 5/8] app/test: extract parse list value testcase for kvargs

Extract parse list value test as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 54 +- 1 file changed, 32 insertions(+), 22 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test

[PATCH v3 4/8] app/test: extract with keys testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the with keys tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 60 +- 1 file changed, 36 insertions(+), 24

[PATCH v3 2/8] app/test: extract basic token count testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the basic token count tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 120 - 1 file changed, 45 insertions(+), 75

[PATCH v3 0/8] refactor kvargs test

://patchwork.dpdk.org/project/dpdk/cover/20231103073811.13196-1-fengcheng...@huawei.com/ Chengwen Feng (8): app/test: introduce UT suite framework for kvargs app/test: extract basic token count testcase for kvargs app/test: extract without keys testcase for kvargs app/test: extract with keys

[PATCH v3 1/8] app/test: introduce UT suite framework for kvargs

Introduce unit test suite framework for test_kvargs.c. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test/test_kvargs.c index 7a60cac4c1

[PATCH v2 2/7] app/test: extract basic token count testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the basic token count tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 120 - 1 file changed, 45 insertions(+), 75

[PATCH v2 3/7] app/test: extract without keys testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the without keys tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 115 ++--- 1 file changed, 63 insertions(+), 52

[PATCH v2 6/7] app/test: extract parse empty elements testcase for kvargs

Extract parse empty elements test as one stand-alone testcase. And also fix the kvlist was not released when the branch fails. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 68 ++ 1 file changed, 29 insertions

[PATCH v2 7/7] maintainers: update for kvargs library

Add myself for voluntary maintenance of the kvargs library. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 7d171e3d45..7e320335d7 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1713,6

[PATCH v2 4/7] app/test: extract with keys testcase for kvargs

The test_valid_kvargs() function is too long to understand, extract the with keys tests as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 59 +- 1 file changed, 35 insertions(+), 24

[PATCH v2 1/7] app/test: introduce UT suite framework for kvargs

Introduce unit test suite framework for test_kvargs.c. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test/test_kvargs.c index 7a60cac4c1

[PATCH v2 5/7] app/test: extract parse list value testcase for kvargs

Extract parse list value test as one stand-alone testcase. Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- app/test/test_kvargs.c | 54 +- 1 file changed, 32 insertions(+), 22 deletions(-) diff --git a/app/test/test_kvargs.c b/app/test

[PATCH v2 0/7] refactor kvargs test

patch independenct, new API which in patchset [1] are not included in this patchset. [1] https://patchwork.dpdk.org/project/dpdk/cover/20231103073811.13196-1-fengcheng...@huawei.com/ Chengwen Feng (7): app/test: introduce UT suite framework for kvargs app/test: extract basic token count

[PATCH v3] net/mvneta: fix possible out-of-bounds write

The mvneta_ifnames_get() function will save 'iface' value to ifnames, it will out-of-bounds write if passed many iface pairs (e.g. 'iface=xxx,iface=xxx,...'). Fixes: 4ccc8d770d3b ("net/mvneta: add PMD skeleton") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng A

[PATCH v2] net/mvneta: fix possible out-of-bounds write

The mvneta_ifnames_get() function will save 'iface' value to ifnames, it will out-of-bounds write if passed many iface pairs (e.g. 'iface=xxx,iface=xxx,...'). Fixes: 4ccc8d770d3b ("net/mvneta: add PMD skeleton") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng A

[PATCH v6 1/4] kvargs: add one new process API

ned-off-by: Chengwen Feng --- doc/guides/rel_notes/release_24_11.rst | 13 lib/kvargs/rte_kvargs.c| 43 -- lib/kvargs/rte_kvargs.h| 39 +-- lib/kvargs/version.map | 7 + 4 files changed, 90 inserti

[PATCH v6 0/4] fix segment fault when parse args

patch/20230320092110.37295-1-fengcheng...@huawei.com/ Chengwen Feng (4): kvargs: add one new process API net/sfc: use new API to parse kvargs net/tap: use new API to parse kvargs common/nfp: use new API to parse kvargs --- v6: rebase to 24.11, refine net/sfc modification, make mvneta

[PATCH v6 2/4] net/sfc: use new API to parse kvargs

Add sfc_kvargs_process_opt() function to handle only-key case, and remove redundancy NULL judgement of value because the rte_kvargs_process (which invoked in sfc_kvargs_process()) will handle it. Signed-off-by: Chengwen Feng --- drivers/common/sfc_efx/sfc_efx.c | 3 --- drivers/net/sfc

[PATCH v6 4/4] common/nfp: use new API to parse kvargs

The nfp_parse_class_options() function could handle both key=value and only-key, so it should use rte_kvargs_process_opt() instead of rte_kvargs_process() to parse. Signed-off-by: Chengwen Feng --- drivers/common/nfp/nfp_common_pci.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions

[PATCH v6 3/4] net/tap: use new API to parse kvargs

Some kvargs could be key=value or only-key, it should use rte_kvargs_process_opt() instead of rte_kvargs_process() to handle these kvargs. Signed-off-by: Chengwen Feng --- drivers/net/tap/rte_eth_tap.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/net

[PATCH] net/mvneta: fix possible out-of-bounds write

The mvneta_ifnames_get() function will save 'iface' value to ifnames, it will out-of-bounds write if passed many iface pairs (e.g. 'iface=xxx,iface=xxx,...'). Fixes: 4ccc8d770d3b ("net/mvneta: add PMD skeleton") Cc: sta...@dpdk.org Signed-off-by: Chengwe

[PATCH v2] examples/eventdev: fix segment fault with generic pipeline

first worker stage. [1] dpdk-eventdev_pipeline -l 0-48 --vdev event_sw0 -- -r1 -t1 -e1 -w ff0 -s5 -n0 -c32 -W1000 -D When launch eventdev_pipeline with command [1], event_sw Fixes: 81fb40f95c82 ("examples/eventdev: add generic worker pipeline") Cc: sta...@dpdk.org Signed-off-by: Che

[PATCH v4 1/7] ethdev: fix race-condition of proactive error handling mode

l.com/ Fixes: eb0d471a8941 ("ethdev: add proactive error handling mode") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Konstantin Ananyev Acked-by: Huisong Li --- doc/guides/prog_guide/ethdev/ethdev.rst | 20 +++- lib/ethdev/ethdev_driver.c |

[PATCH v4 6/7] app/testpmd: extract event handling to event.c

This patch extract event handling (including eth-event and dev-event) to a new file 'event.c'. Signed-off-by: Chengwen Feng Acked-by: Huisong Li --- app/test-pmd/event.c | 390 ++ app/test-pmd/meson.build | 1 + app/test-pmd/paramete

[PATCH v4 5/7] app/testpmd: add error recovery usage demo

when the RTE_ETH_EVENT_RECOVERY_FAILED event is received. In addition, a message is added to the printed information, requiring no command to be executed during the error recovery. Signed-off-by: Chengwen Feng Acked-by: Konstantin Ananyev --- app/test-pmd/testpmd.c | 80

[PATCH v4 3/7] net/bnxt: fix race-condition when report error recovery

error handling mode") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Konstantin Ananyev Acked-by: Ajit Khaparde --- drivers/net/bnxt/bnxt_cpr.c| 13 +++-- drivers/net/bnxt/bnxt_ethdev.c | 4 ++-- 2 files changed, 9 insertions(+), 8 deletions(-) diff --git a/driver

[PATCH v4 7/7] doc: testpmd support event handling section

Add new section of event handling, which documented the ethdev and device events. Signed-off-by: Chengwen Feng --- doc/guides/testpmd_app_ug/event_handling.rst | 81 doc/guides/testpmd_app_ug/index.rst | 1 + 2 files changed, 82 insertions(+) create mode 100644

[PATCH v4 4/7] net/bnxt: use fp ops setup function

Use rte_eth_fp_ops_setup() instead of directly manipulating rte_eth_fp_ops variable. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Konstantin Ananyev Acked-by: Ajit Khaparde Acked-by: Huisong Li --- drivers/net/bnxt/bnxt_cpr.c| 5 + drivers/net/bnxt/bnxt_ethdev.c | 5

[PATCH v4 2/7] net/hns3: replace fp ops config function

This patch replace hns3_eth_dev_fp_ops_config() with rte_eth_fp_ops_setup(). Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Dongdong Liu Acked-by: Konstantin Ananyev Acked-by: Huisong Li --- drivers/net/hns3/hns3_rxtx.c | 21 +++-- 1 file changed, 3 insertions

[PATCH v4 0/7] fix race-condition of proactive error handling mode

This patch fixes race-condition of proactive error handling mode, the discussion thread [1]. [1] http://patchwork.dpdk.org/project/dpdk/patch/20230220060839.1267349-2-ashok.k.kal...@intel.com/ Chengwen Feng (7): ethdev: fix race-condition of proactive error handling mode net/hns3: replace

[PATCH] examples/eventdev: fix segment fault with generic pipeline

.@dpdk.org Signed-off-by: Chengwen Feng Reported-by: Chenxingyu Wang --- examples/eventdev_pipeline/pipeline_worker_generic.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/examples/eventdev_pipeline/pipeline_worker_generic.c b/examples/eventde

[PATCH v2 1/4] cfgfile: remove dead code

This memchr() will never return NULL because rte_cfgfile_load() function will skip lines without useful content. Fixes: 74e0d3a17461 ("cfgfile: fix null pointer dereference in parsing") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- lib/cfgfile/rte

[PATCH v2 0/4] cfgfile: enhance error detecting

When I was trying to debug a problem introduced by config.ini in test-dma-perf, I found the cfgfile library should enhance error detecting, so got this patchset. Chengwen Feng (4): cfgfile: remove dead code cfgfile: support verify name and value cfgfile: verify add section and entry result

[PATCH v2 4/4] cfgfile: add strict parse flag

: Chengwen Feng --- lib/cfgfile/rte_cfgfile.c | 32 +++- lib/cfgfile/rte_cfgfile.h | 7 +++ 2 files changed, 30 insertions(+), 9 deletions(-) diff --git a/lib/cfgfile/rte_cfgfile.c b/lib/cfgfile/rte_cfgfile.c index 3033cff4a4..5d96818728 100644 --- a/lib/cfgfile

[PATCH v2 3/4] cfgfile: verify add section and entry result

The rte_cfgfile_add_section() and _add_entry()'s result were not verified, so that potential errors may not be detected. This commit adds the verification. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- lib/cfgfile/rte_cfgfile.c | 19 ++---

[PATCH v2 2/4] cfgfile: support verify name and value

This patch supports verify section's name, entry's name and entry's value validity. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Acked-by: Stephen Hemminger --- lib/cfgfile/rte_cfgfile.c | 17 + 1 file changed, 17 insertions(+) diff --git a/lib/cfgfile/rte_

[PATCH] dma/hisilicon: remove support for HIP09 platform

The DMA for HIP09 is no longer available, so delete it. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- doc/guides/dmadevs/hisilicon.rst | 1 - doc/guides/rel_notes/release_24_07.rst | 5 drivers/dma/hisilicon/hisi_dmadev.c| 40 +- drivers/dma

[PATCH] dma/hisilicon: remove support for HIP09 platform

The DMA for HIP09 is no longer available, so delete it. Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- doc/guides/dmadevs/hisilicon.rst | 1 - doc/guides/rel_notes/release_24_07.rst | 2 ++ drivers/dma/hisilicon/hisi_dmadev.c| 40 +- drivers/dma

[PATCH v8] ethdev: fix strict aliasing lead to link cannot be up

o the impacted components (cxgbe and qos_sched) have been adapted to the struct change. [1] https://inbox.dpdk.org/dev/8175c905-e661-b910-7f20-59b6ab605...@huawei.com/ Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang Acked-by: Morten Brørup --- v8: address Fer

[PATCH v7] ethdev: fix strict aliasing lead to link cannot be up

.org/archives/test-report/2024-April/637966.html Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang Acked-by: Morten Brørup --- v7: add rte_atomic_store_explicit to get link status which address Morten's comment, and add Morten's acked-by. v6: fix DPDK

[PATCH v6] ethdev: fix strict aliasing lead to link cannot be up

.org/archives/test-report/2024-April/637966.html Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang --- v6: fix DPDK CI compiler error with qos_sched. v5: fix DPDK CI compiler error due to GCC extension [-Werror=pedantic] v4: fix DPDK CI compiler error with cxgbe. v3: f

[PATCH v5] ethdev: fix strict aliasing lead to link cannot be up

with rte_eth_linkstatus_set() https://marc.info/?l=dpdk-dev&m=171274148514777&w=3 [2] https://mails.dpdk.org/archives/test-report/2024-April/637966.html Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang --- v5: fix DPDK CI compiler error due to GCC extension [-We

[PATCH v4] ethdev: fix strict aliasing lead to link cannot be up

with rte_eth_linkstatus_set() https://marc.info/?l=dpdk-dev&m=171274148514777&w=3 [2] https://mails.dpdk.org/archives/test-report/2024-April/637966.html Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang --- v4: fix DPDK CI compiler error with cxgbe. v3: fix check

[PATCH v3] ethdev: fix strict aliasing lead to link cannot be up

. [1] Strict aliasing problem with rte_eth_linkstatus_set() https://marc.info/?l=dpdk-dev&m=171274148514777&w=3 Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang --- v3: fix checkpatch warning "missing --in-reply-to". v2: add RTE_ATOMIC(uint6

[PATCH v2] ethdev: fix strict aliasing lead to link cannot be up

. [1] Strict aliasing problem with rte_eth_linkstatus_set() https://marc.info/?l=dpdk-dev&m=171274148514777&w=3 Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang --- v2: add RTE_ATOMIC(uint64_t) wrap which address Morten's comment. --- lib/ethdev/et

[PATCH] ethdev: fix strict aliasing lead to link cannot be up

. [1] Strict aliasing problem with rte_eth_linkstatus_set() https://marc.info/?l=dpdk-dev&m=171274148514777&w=3 Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng Signed-off-by: Dengdui Huang --- lib/ethdev/ethdev_driver.h | 23 +++ lib/ethdev/rte_eth

[PATCH v4 6/6] argparse: fix doc don't display two hyphens

this format for all arguments. Fixes: e3e579f5bab5 ("argparse: introduce argparse library") [1] https://stackoverflow.com/questions/51075907/display-two-dashes-in-rst-file Signed-off-by: Chengwen Feng --- doc/guides/prog_guide/argparse_lib.rst | 47 +

[PATCH v4 3/6] argparse: replace flag enum with marco

The enum rte_argparse_flag's value is u64, but an enum in C is represented as an int. This commit replace these enum values with macro. Fixes: e3e579f5bab5 ("argparse: introduce argparse library") Fixes: 5357c248c960 ("argparse: parse unsigned integers") Signed-off-by

[PATCH v4 4/6] argparse: fix argument flags operate as uint32 type

ask marcros and removes an internal duplicate macro. Fixes: 6c5c6571601c ("argparse: verify argument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c| 18 ++ lib/argparse/rte_argparse.c

[PATCH v4 5/6] test/argparse: refine testcases

: verify argument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c | 40 +--- 1 file changed, 25 insertions(+), 15 deletions(-) diff --git a/app/test/test_argparse.c b/app/tes

[PATCH v4 1/6] argparse: refine error message

This patch refines the error message. Signed-off-by: Chengwen Feng --- lib/argparse/rte_argparse.c | 31 +-- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/lib/argparse/rte_argparse.c b/lib/argparse/rte_argparse.c index 2d953f1694..48738cd07b 100644

[PATCH v4 2/6] argparse: remove dead code

The judgement "obj->callback == NULL" is dead code which can't be reached, because verify_arg_saver() already make sure obj->callback must not be NULL when arg->val_saver is NULL. Signed-off-by: Chengwen Feng --- app/test/test_argparse.c| 7 --- lib/argparse/rte_

[PATCH v4 0/6] refine argparse library

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 flag enum with marco argparse: fix argument flags operate as uint32 type test/argparse: refine

[PATCH v3 1/6] argparse: refine error message

This patch refines the error message. Signed-off-by: Chengwen Feng --- lib/argparse/rte_argparse.c | 31 +-- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/lib/argparse/rte_argparse.c b/lib/argparse/rte_argparse.c index 2d953f1694..48738cd07b 100644

[PATCH v3 3/6] argparse: replace flag enum with marco

The enum rte_argparse_flag's value is u64, but an enum in C is represented as an int. This commit replace these enum values with macro. Fixes: e3e579f5bab5 ("argparse: introduce argparse library") Fixes: 5357c248c960 ("argparse: parse unsigned integers") Signed-off-by

[PATCH v3 5/6] test/argparse: refine testcases

: verify argument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c | 40 +--- 1 file changed, 25 insertions(+), 15 deletions(-) diff --git a/app/test/test_argparse.c b/app/tes

[PATCH v3 4/6] argparse: fix argument flags operate as uint32 type

ask marcros and removes an internal duplicate macro. Fixes: 6c5c6571601c ("argparse: verify argument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c| 21 + lib/argparse/rte_argparse.c

[PATCH v3 6/6] argparse: fix doc don't display two hyphens

this format for all arguments. Fixes: e3e579f5bab5 ("argparse: introduce argparse library") [1] https://stackoverflow.com/questions/51075907/display-two-dashes-in-rst-file Signed-off-by: Chengwen Feng --- doc/guides/prog_guide/argparse_lib.rst | 47 +

[PATCH v3 0/6] refine argparse library

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 flag enum with marco argparse: fix argument flags operate as uint32 type test/argparse: refine

[PATCH v3 2/6] argparse: remove dead code

The judgement "obj->callback == NULL" is dead code which can't be reached, because verify_arg_saver() already make sure obj->callback must not be NULL when arg->val_saver is NULL. Signed-off-by: Chengwen Feng --- app/test/test_argparse.c| 7 --- lib/argparse/rte_

[PATCH v2 1/5] argparse: refine error message

This patch refines the error message. Signed-off-by: Chengwen Feng --- lib/argparse/rte_argparse.c | 31 +-- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/lib/argparse/rte_argparse.c b/lib/argparse/rte_argparse.c index 2d953f1694..48738cd07b 100644

[PATCH v2 3/5] argparse: replace flag enum with marco

The enum rte_argparse_flag's value is u64, but an enum in C is represented as an int. This commit replace these enum values with macro. Fixes: e3e579f5bab5 ("argparse: introduce argparse library") Fixes: 5357c248c960 ("argparse: parse unsigned integers") Signed-off-by

[PATCH v2 5/5] test/argparse: refine testcases

: verify argument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c | 40 +--- 1 file changed, 25 insertions(+), 15 deletions(-) diff --git a/app/test/test_argparse.c b/app/tes

[PATCH v2 4/5] argparse: fix argument flags operate as uint32 type

ument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c| 21 + lib/argparse/rte_argparse.c | 4 ++-- 2 files changed, 15 insertions(+), 10 deletions(-) diff --git a/app/test/test_argparse

[PATCH v2 0/5] refine argparse library

I found a couple of issues when I revisited the argparse_autotest output, so got this patchset. Chengwen Feng (5): argparse: refine error message argparse: remove dead code argparse: replace flag enum with marco argparse: fix argument flags operate as uint32 type test/argparse: refine

[PATCH v2 2/5] argparse: remove dead code

The judgement "obj->callback == NULL" is dead code which can't be reached, because verify_arg_saver() already make sure obj->callback must not be NULL when arg->val_saver is NULL. Signed-off-by: Chengwen Feng --- app/test/test_argparse.c| 7 --- lib/argparse/rte_

[PATCH 4/4] test/argparse: refine testcases

: verify argument config") Fixes: 31ed9f9f43bb ("argparse: parse parameters") Signed-off-by: Chengwen Feng --- app/test/test_argparse.c | 40 +--- 1 file changed, 25 insertions(+), 15 deletions(-) diff --git a/app/test/test_argparse.c b/app/tes

[PATCH 1/4] argparse: refine error message

This patch refines the error message. Signed-off-by: Chengwen Feng --- lib/argparse/rte_argparse.c | 31 +-- 1 file changed, 17 insertions(+), 14 deletions(-) diff --git a/lib/argparse/rte_argparse.c b/lib/argparse/rte_argparse.c index 2d953f1694..48738cd07b 100644

  1   2   3   4   5   6   7   8   >