Re: [PATCH 07/32] event/dpaa2: replace memcpy with structure assignment

[Hemant Agrawal <hemant.agrawal@oss.nxp.com>]

Acked-by: Hemant Agrawal <hemant.agrawal@nxp.com>

On 09-02-2025 01:51, Stephen Hemminger wrote:
> Prefer structure assignment over memcpy.
> Found by struct-assign.cocci.
>
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>   drivers/event/dpaa2/dpaa2_eventdev.c | 7 +++----
>   1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/event/dpaa2/dpaa2_eventdev.c b/drivers/event/dpaa2/dpaa2_eventdev.c
> index 6c8ed3ff6b..95d9154b11 100644
> --- a/drivers/event/dpaa2/dpaa2_eventdev.c
> +++ b/drivers/event/dpaa2/dpaa2_eventdev.c
> @@ -224,7 +224,7 @@ static void dpaa2_eventdev_process_parallel(struct qbman_swp *swp,
>   
>   	RTE_SET_USED(rxq);
>   
> -	rte_memcpy(ev, ev_temp, sizeof(struct rte_event));
> +	*ev = *ev_temp;
>   	rte_free(ev_temp);
>   
>   	qbman_swp_dqrr_consume(swp, dq);
> @@ -243,7 +243,7 @@ static void dpaa2_eventdev_process_atomic(struct qbman_swp *swp,
>   	RTE_SET_USED(swp);
>   	RTE_SET_USED(rxq);
>   
> -	rte_memcpy(ev, ev_temp, sizeof(struct rte_event));
> +	*ev = *ev_temp;
>   	rte_free(ev_temp);
>   	*dpaa2_seqn(ev->mbuf) = dqrr_index + 1;
>   	DPAA2_PER_LCORE_DQRR_SIZE++;
> @@ -588,8 +588,7 @@ dpaa2_eventdev_port_link(struct rte_eventdev *dev, void *port,
>   
>   	for (i = 0; i < nb_links; i++) {
>   		evq_info = &priv->evq_info[queues[i]];
> -		memcpy(&dpaa2_portal->evq_info[queues[i]], evq_info,
> -			   sizeof(struct dpaa2_eventq));
> +		dpaa2_portal->evq_info[queues[i]] = *evq_info;
>   		dpaa2_portal->evq_info[queues[i]].event_port = port;
>   		dpaa2_portal->num_linked_evq++;
>   	}