Re: [PATCH v1 1/2] dmadev: add DMA operation structure

[Bruce Richardson <bruce.richardson@intel.com>]

On Mon, Sep 18, 2023 at 06:02:26PM +0530, Amit Prakash Shukla wrote:
> For the event based DMA transfer, all the required parameters are to
> be sent as part of a structure. This patch adds a structure
> containing the parameters for event based DMA operation.
> 
> Signed-off-by: Amit Prakash Shukla <amitprakashs@marvell.com>
> ---
>  lib/dmadev/rte_dmadev.h | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
> 
> diff --git a/lib/dmadev/rte_dmadev.h b/lib/dmadev/rte_dmadev.h
> index b157ab7600..bf8db2d28d 100644
> --- a/lib/dmadev/rte_dmadev.h
> +++ b/lib/dmadev/rte_dmadev.h
> @@ -794,6 +794,18 @@ struct rte_dma_sge {
>  	uint32_t length; /**< The DMA operation length. */
>  };
>  
> +/**
> + * A structure used to hold event based DMA operation request entry.
> + */
> +struct rte_dma_op {

Would it be better called rte_dma_event_op, if it's just for use with
eventdev?

> +	struct rte_dma_sge *src_seg;      /**< Source segments. */
> +	struct rte_dma_sge *dst_seg;      /**< Destination segments. */
> +	uint16_t nb_src;                  /**< Number of source segments. */
> +	uint16_t nb_dst;                  /**< Number of destination segments. */
> +	uint64_t flags;                   /**< Flags related to the operation. */
> +	struct rte_mempool *op_mp;        /**< Mempool from which op is allocated. */
> +};
> +
>  #include "rte_dmadev_core.h"
>  
>  /**@{@name DMA operation flag
> -- 
> 2.25.1
>