[EXTERNAL] Re: [PATCH v3 1/2] eventdev/dma: reorganize event DMA ops

[Pavan Nikhilesh Bhagavatula]

> On Thu, May 16, 2024 at 1:09 PM Amit Prakash Shukla
> <amitprakashs at marvell.com> wrote:
> >
> > > -----Original Message-----
> > > From: pbhagavatula at marvell.com <pbhagavatula at marvell.com>
> > > Sent: Wednesday, April 17, 2024 1:57 PM
> > > To: Jerin Jacob <jerinj at marvell.com>; Amit Prakash Shukla
> > > <amitprakashs at marvell.com>; Vamsi Krishna Attunuru
> > > <vattunuru at marvell.com>
> > > Cc: dev at dpdk.org; Pavan Nikhilesh Bhagavatula
> > > <pbhagavatula at marvell.com>
> > > Subject: [PATCH v3 1/2] eventdev/dma: reorganize event DMA ops
> > >
> > > From: Pavan Nikhilesh <pbhagavatula at marvell.com>
> > >
> > > Re-organize event DMA ops structure to allow holding source and
> destination
> > > pointers without the need for additional memory, the mempool allocating
> > > memory for rte_event_dma_adapter_ops can size the structure to
> > > accommodate all the needed source and destination pointers.
> > >
> > > Add multiple words for holding user metadata, adapter implementation
> > > specific metadata and event metadata.
> > >
> > > Signed-off-by: Pavan Nikhilesh <pbhagavatula at marvell.com>
> > > ---
> > >  v3 Changes:
> > >  - Fix stdatomic compilation.
> > >  v2 Changes:
> > >  - Fix 32bit compilation
> > >
> > >  app/test-eventdev/test_perf_common.c        | 26 ++++--------
> > >  app/test/test_event_dma_adapter.c           | 20 +++------
> > >  doc/guides/prog_guide/event_dma_adapter.rst |  2 +-
> > >  drivers/dma/cnxk/cnxk_dmadev_fp.c           | 39 +++++++----------
> 
> Spotted driver change in library patch. Please split.

Since this change modifies the fastpath structure rte_event_dma_adapter_op, 
some driver changes are required, I have moved the non-relevant changes to
2/2.

> 
> > >  lib/eventdev/rte_event_dma_adapter.c        | 27 ++++--------
> > >  lib/eventdev/rte_event_dma_adapter.h        | 46 +++++++++++++++------
> > >  6 files changed, 72 insertions(+), 88 deletions(-)
> > >
> >
> > Acked-by: Amit Prakash Shukla <amitprakashs at marvell.com>