RE: [EXT] Re: [PATCH] test/dma: fix for buffer auto free

2023-11-02 Thread Amit Prakash Shukla
k.org; Jerin Jacob Kollanukkaran ; > conor.wa...@intel.com; Vamsi Krishna Attunuru ; > g.si...@nxp.com; sachin.sax...@oss.nxp.com; hemant.agra...@nxp.com; > cheng1.ji...@intel.com; Nithin Kumar Dabilpuram > ; Anoob Joseph ; > m...@smartsharesystems.com > Subject: [EXT] Re: [PATCH

Re: [PATCH] test/dma: fix for buffer auto free

2023-11-01 Thread fengchengwen
Hi Amit, I prefer not use static variable to control it because it introduce many coupling. Suggest add one function which prepare the test_m2d_auto_free, like prepare_m2d_auto_free if ((info.dev_capa & RTE_DMA_CAPA_M2D_AUTO_FREE) && dma_add_test[TEST_M2D_AUTO_FREE].ena

RE: [PATCH] test/dma: fix for buffer auto free

2023-11-01 Thread Hemant Agrawal
> diff --git a/app/test/test_dmadev.c b/app/test/test_dmadev.c index > 216f84b6bb..3d4cb37ee6 100644 > --- a/app/test/test_dmadev.c > +++ b/app/test/test_dmadev.c > @@ -49,6 +49,8 @@ struct dma_add_test dma_add_test[] = { > [TEST_M2D_AUTO_FREE] = {.name = "m2d_auto_free", .enabled = > false},

[PATCH] test/dma: fix for buffer auto free

2023-11-01 Thread Amit Prakash Shukla
Buffer auto free test failed for more than 1 dma device as the device initialization for the test was been done only for the first dma device. This changeset fixes the same. Fixes: 877cb3e37426 ("dmadev: add buffer auto free offload") Signed-off-by: Amit Prakash Shukla --- app/test/test_dmadev.