Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-03 Thread Conor Walsh
For each dmadev instance, perform some basic copy tests to validate that functionality. Signed-off-by: Bruce Richardson --- +static inline void +await_hw(int dev_id, uint16_t vchan) +{ + int idle = rte_dmadev_vchan_idle(dev_id, vchan); + if (idle < 0) { + /* for d

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-03 Thread Kevin Laatz
On 01/09/2021 17:32, Bruce Richardson wrote: For each dmadev instance, perform some basic copy tests to validate that functionality. Signed-off-by: Bruce Richardson --- app/test/test_dmadev.c | 174 + 1 file changed, 174 insertions(+) + +static in

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-02 Thread Bruce Richardson
On Thu, Sep 02, 2021 at 06:35:07PM +0530, Jerin Jacob wrote: > On Thu, Sep 2, 2021 at 5:13 PM Bruce Richardson > wrote: > > > > On Thu, Sep 02, 2021 at 04:24:18PM +0530, Jerin Jacob wrote: > > > > > > I think 25us will not be enough, e.s.p If is PCI-Dev to PCI-Dev kind > > > of test cases. > > > S

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-02 Thread Jerin Jacob
On Thu, Sep 2, 2021 at 5:13 PM Bruce Richardson wrote: > > On Thu, Sep 02, 2021 at 04:24:18PM +0530, Jerin Jacob wrote: > > > > I think 25us will not be enough, e.s.p If is PCI-Dev to PCI-Dev kind > > of test cases. > > Since it is the functional test case, I think, we can keep it a very > > highe

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-02 Thread Bruce Richardson
On Thu, Sep 02, 2021 at 04:24:18PM +0530, Jerin Jacob wrote: > > I think 25us will not be enough, e.s.p If is PCI-Dev to PCI-Dev kind > of test cases. > Since it is the functional test case, I think, we can keep it a very > higher range to > support all cases. Maybe 50ms is a good target. > Sure

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-02 Thread Jerin Jacob
On Thu, Sep 2, 2021 at 1:36 PM Bruce Richardson wrote: > > On Thu, Sep 02, 2021 at 01:14:38PM +0530, Jerin Jacob wrote: > > On Wed, Sep 1, 2021 at 10:02 PM Bruce Richardson > > wrote: > > > > > > For each dmadev instance, perform some basic copy tests to validate that > > > functionality. > > > >

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-02 Thread Bruce Richardson
On Thu, Sep 02, 2021 at 01:14:38PM +0530, Jerin Jacob wrote: > On Wed, Sep 1, 2021 at 10:02 PM Bruce Richardson > wrote: > > > > For each dmadev instance, perform some basic copy tests to validate that > > functionality. > > > > Signed-off-by: Bruce Richardson > > --- > > app/test/test_dmadev.c

Re: [dpdk-dev] [PATCH v2 3/6] app/test: add basic dmadev copy tests

2021-09-02 Thread Jerin Jacob
On Wed, Sep 1, 2021 at 10:02 PM Bruce Richardson wrote: > > For each dmadev instance, perform some basic copy tests to validate that > functionality. > > Signed-off-by: Bruce Richardson > --- > app/test/test_dmadev.c | 174 + > 1 file changed, 174 insertio