Hi Amit, I think this commit will complicated the test, plus futer we may add more test (e.g. fill)
I agree Bruce's advise in the [1], let also support "lcore_dma0/1/2", User could provide dma info by two way: 1) lcore_dma=, which seperate each dma with ", ", but a maximum of a certain number is allowed. 2) lcore_dma0/1/2/..., each dma device take one line [1] https://patchwork.dpdk.org/project/dpdk/patch/[email protected]/ Thanks On 2024/2/29 22:03, Amit Prakash Shukla wrote: > Hi Chengwen, > > I liked your suggestion and tried making changes, but encountered parsing > issue for CFG files with line greater than CFG_VALUE_LEN=256(current value > set). > > There is a discussion on the similar lines in another patch set: > https://patchwork.dpdk.org/project/dpdk/patch/[email protected]/. > > I believe this patch can be taken as-is and we can come up with the solution > when we can increase the CFG_VALUE_LEN as changing CFG_VALUE_LEN in this > release is causing ABI breakage. > > Thanks, > Amit Shukla > >> -----Original Message----- >> From: Amit Prakash Shukla >> Sent: Wednesday, February 28, 2024 3:08 PM >> To: fengchengwen <[email protected]>; Cheng Jiang >> <[email protected]>; Gowrishankar Muthukrishnan >> <[email protected]> >> Cc: [email protected]; Jerin Jacob <[email protected]>; Anoob Joseph >> <[email protected]>; Kevin Laatz <[email protected]>; Bruce >> Richardson <[email protected]>; Pavan Nikhilesh Bhagavatula >> <[email protected]> >> Subject: RE: [EXT] Re: [PATCH v2] app/dma-perf: support bi-directional >> transfer >> >> Hi Chengwen, >> >> Please see my reply in-line. >> >> Thanks >> Amit Shukla >> >>> -----Original Message----- >>> From: fengchengwen <[email protected]> >>> Sent: Wednesday, February 28, 2024 12:34 PM >>> To: Amit Prakash Shukla <[email protected]>; Cheng Jiang >>> <[email protected]>; Gowrishankar Muthukrishnan >>> <[email protected]> >>> Cc: [email protected]; Jerin Jacob <[email protected]>; Anoob Joseph >>> <[email protected]>; Kevin Laatz <[email protected]>; Bruce >>> Richardson <[email protected]>; Pavan Nikhilesh Bhagavatula >>> <[email protected]> >>> Subject: [EXT] Re: [PATCH v2] app/dma-perf: support bi-directional >>> transfer >>> >>> External Email >>> >>> ---------------------------------------------------------------------- >>> Hi Amit and Gowrishankar, >>> >>> It's nature to support multiple dmadev test in one testcase, and the >>> original framework supports it. >>> But it seem we both complicated it when adding support for non- >> mem2mem >>> dma test. >>> >>> The new added "direction" and "vchan_dev" could treat as the dmadev's >>> private configure, some thing like: >>> >>> >> lcore_dma=lcore10@0000:00:04.2,vchan=0,dir=mem2dev,devtype=pcie,radd >>> r=xxx,coreid=1,pfid=2,vfid=3 >>> >>> then this bi-directional could impl only with config: >>> >>> >> lcore_dma=lcore10@0000:00:04.2,dir=mem2dev,devtype=pcie,raddr=xxx,cor >>> eid=1,pfid=2,vfid=3, >>> >> lcore11@0000:00:04.3,dir=dev2mem,devtype=pcie,raddr=xxx,coreid=1,pfid= >>> 2,vfid=3 >>> so that the lcore10 will do mem2dev with device 0000:00:04.2, while >>> lcore11 will do dev2mem with device 0000:00:04.3. >> >> Thanks for the suggestion. I will make the suggested changes and send the >> next version.
