> -----Original Message-----
> From: McDaniel, Timothy <[email protected]>
> Sent: Saturday, October 17, 2020 1:21 PM
> Cc: [email protected]; Carrillo, Erik G <[email protected]>; Eads, Gage
> <[email protected]>; Van Haaren, Harry <[email protected]>;
> [email protected]
> Subject: [PATCH v2 20/22] event/dlb2: add queue and port release
> 
> DLB does not support reconfiguring individual queues
> or ports on the fly. The entire device must be reconfigured.
> Previously allocated port QE ond memzone memory
> is freed in this patch.
> 
> Signed-off-by: Timothy McDaniel <[email protected]>
> ---
>  drivers/event/dlb2/dlb2.c | 28 ++++++++++++++++++++++++++--
>  1 file changed, 26 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/event/dlb2/dlb2.c b/drivers/event/dlb2/dlb2.c
> index 06a59a5..968923e 100644
> --- a/drivers/event/dlb2/dlb2.c
> +++ b/drivers/event/dlb2/dlb2.c
> @@ -89,8 +89,8 @@ dlb2_free_qe_mem(struct dlb2_port *qm_port)
>       rte_free(qm_port->consume_qe);
>       qm_port->consume_qe = NULL;
> 
> -     rte_free(dlb2_port[qm_port->id][PORT_TYPE(qm_port)].cq_base);
> -     rte_free(dlb2_port[qm_port->id][PORT_TYPE(qm_port)].pp_addr);
> +     rte_memzone_free(dlb2_port[qm_port-
> >id][PORT_TYPE(qm_port)].mz);
> +     dlb2_port[qm_port->id][PORT_TYPE(qm_port)].mz = NULL;
>  }

This code should be a part of patch 8, when free_qe_mem is introduced. With 
that change:
Reviewed-by: Gage Eads <[email protected]>

Thanks,
Gage

Reply via email to