Re: [PATCH 2/2] test/dma: support auto adapt ringsize

[Bruce Richardson <bruce.richardson@intel.com>]

On Wed, Aug 27, 2025 at 09:28:29AM +0800, Chengwen Feng wrote:
> The default ringsize is 512, some DMA devices may not support, this
> commit support auto adapt ringsize.
> 
> Signed-off-by: Chengwen Feng <fengchengwen@huawei.com>
> ---

Seems reasonable.

Acked-by: Bruce Richardson <bruce.richardson@intel.com>


>  app/test/test_dmadev.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c
> index 482d564f7a..e1a919cd81 100644
> --- a/app/test/test_dmadev.c
> +++ b/app/test/test_dmadev.c
> @@ -1135,7 +1135,7 @@ test_dmadev_setup(void)
>  	int16_t dev_id = test_dev_id;
>  	struct rte_dma_stats stats;
>  	const struct rte_dma_conf conf = { .nb_vchans = 1};
> -	const struct rte_dma_vchan_conf qconf = {
> +	struct rte_dma_vchan_conf qconf = {
>  			.direction = RTE_DMA_DIR_MEM_TO_MEM,
>  			.nb_desc = TEST_RINGSIZE,
>  	};
> @@ -1151,6 +1151,10 @@ test_dmadev_setup(void)
>  	if (rte_dma_configure(dev_id, &conf) != 0)
>  		ERR_RETURN("Error with rte_dma_configure()\n");
>  
> +	if (qconf.nb_desc < info.min_desc)
> +		qconf.nb_desc = info.min_desc;
> +	if (qconf.nb_desc > info.max_desc)
> +		qconf.nb_desc = info.max_desc;
>  	if (rte_dma_vchan_setup(dev_id, vchan, &qconf) < 0)
>  		ERR_RETURN("Error with queue configuration\n");
>  
> -- 
> 2.17.1
>