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>
---
 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

Reply via email to