On Fri, 20 Sep 2024 12:27:10 +0200
Mattias Rönnblom <mattias.ronnb...@ericsson.com> wrote:
> The DLB2 PMD depended on <rte_vect.h> being included as a side-effect
> of <rte_memcpy.h> being included.
>
> In addition, DLB2 used rte_memcpy() but did not include <rte_memcpy.h>,
> but rather depended on other include files to do so.
>
> This patch addresses both of those issues.
>
> Signed-off-by: Mattias Rönnblom <mattias.ronnb...@ericsson.com>
> Acked-by: Bruce Richardson <bruce.richard...@intel.com>
Have to ask, why is it using rte_memcpy when it could just assignment.
Assignment is better since it keeps type safety.
diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c
index c43ab864ca..51870486ed 100644
--- a/drivers/event/dlb2/dlb2.c
+++ b/drivers/event/dlb2/dlb2.c
@@ -1668,7 +1668,7 @@ dlb2_hw_create_ldb_port(struct dlb2_eventdev *dlb2,
qm_port->issued_releases = 0;
/* Save config message too. */
- rte_memcpy(&qm_port->cfg.ldb, &cfg, sizeof(qm_port->cfg.ldb));
+ qm_port->cfg.ldb = cfg;
/* update state */
qm_port->state = PORT_STARTED; /* enabled at create time */
@@ -1869,7 +1869,7 @@ dlb2_hw_create_dir_port(struct dlb2_eventdev *dlb2,
qm_port->issued_releases = 0;
/* Save config message too. */
- rte_memcpy(&qm_port->cfg.dir, &cfg, sizeof(qm_port->cfg.dir));
+ qm_port->cfg.dir = cfg;
/* update state */
qm_port->state = PORT_STARTED; /* enabled at create time */
