Re: [PATCH 1/4] ethdev: verify queue ID when Tx done cleanup

2024-10-12 Thread fengchengwen
On 2024/10/12 10:27, Ferruh Yigit wrote: > On 9/5/2024 8:33 AM, Andrew Rybchenko wrote: >> On 9/5/24 09:46, Jie Hai wrote: >>> From: Chengwen Feng >>> >>> Verify queue_id for rte_eth_tx_done_cleanup API. >> >> If I'm not mistaken the function is considered data path API (fast). >> If so, it should

Re: [PATCH 1/4] ethdev: verify queue ID when Tx done cleanup

2024-10-11 Thread Ferruh Yigit
On 9/5/2024 8:33 AM, Andrew Rybchenko wrote: > On 9/5/24 09:46, Jie Hai wrote: >> From: Chengwen Feng >> >> Verify queue_id for rte_eth_tx_done_cleanup API. > > If I'm not mistaken the function is considered data path API (fast). > If so, it should not validate it's parameters as in rte_eth_tx_bu

Re: [PATCH 1/4] ethdev: verify queue ID when Tx done cleanup

2024-09-05 Thread Andrew Rybchenko
On 9/5/24 09:46, Jie Hai wrote: From: Chengwen Feng Verify queue_id for rte_eth_tx_done_cleanup API. If I'm not mistaken the function is considered data path API (fast). If so, it should not validate it's parameters as in rte_eth_tx_burst(). It may be done under RTE_ETHDEV_DEBUG_TX only. May

[PATCH 1/4] ethdev: verify queue ID when Tx done cleanup

2024-09-04 Thread Jie Hai
From: Chengwen Feng Verify queue_id for rte_eth_tx_done_cleanup API. Fixes: 44a718c457b5 ("ethdev: add API to free consumed buffers in Tx ring") Cc: sta...@dpdk.org Signed-off-by: Chengwen Feng --- lib/ethdev/rte_ethdev.c | 4 1 file changed, 4 insertions(+) diff --git a/lib/ethdev/rte_