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 <fengcheng...@huawei.com>
> ---

Seems reasonable.

Acked-by: Bruce Richardson <bruce.richard...@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
> 

Reply via email to