On Thu, Mar 2, 2023 at 4:30 PM Volodymyr Fialko <vfia...@marvell.com> wrote:
>
> dev_fini() can return EAGAIN in case if LF is active(used by other
> devices). Ignoring return code leads to early free of resources that are
> still in use.
>
> Fixes: 8558dcaa05b8 ("event/cnxk: add build infra and device setup")
> Cc: sta...@dpdk.org
>
> Signed-off-by: Volodymyr Fialko <vfia...@marvell.com>Applied to dpdk-next-net-eventdev/for-main. Thanks > --- > drivers/event/cnxk/cnxk_eventdev.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/event/cnxk/cnxk_eventdev.c > b/drivers/event/cnxk/cnxk_eventdev.c > index 9815e84c24..6aecccb45d 100644 > --- a/drivers/event/cnxk/cnxk_eventdev.c > +++ b/drivers/event/cnxk/cnxk_eventdev.c > @@ -670,9 +670,8 @@ cnxk_sso_fini(struct rte_eventdev *event_dev) > > cnxk_tim_fini(); > roc_sso_rsrc_fini(&dev->sso); > - roc_sso_dev_fini(&dev->sso); > > - return 0; > + return roc_sso_dev_fini(&dev->sso); > } > > int > -- > 2.34.1 >
