iavf_dev_init() takes a reference on the shared event handler, but
does not release it if initialisation subsequently fails. The event
thread and its pipe are then never released.
Release the reference on these error paths. Skip it during reset
recovery, where no reference is taken.
Fixes: cb5c1b91f76f ("net/iavf: add thread for event callbacks")
Cc: <redacted>
Signed-off-by: Ciara Loftus <redacted>
---
drivers/net/intel/iavf/iavf_ethdev.c | 5 +++++
1 file changed, 5 insertions(+)
diff --git a/drivers/net/intel/iavf/iavf_ethdev.c b/drivers/net/intel/iavf/iavf_ethdev.c
index fd1067c252..38bf2e502a 100644
--- a/drivers/net/intel/iavf/iavf_ethdev.c
+++ b/drivers/net/intel/iavf/iavf_ethdev.c
@@ -3089,6 +3089,8 @@ iavf_dev_init(struct rte_eth_dev *eth_dev)
if (iavf_init_vf(eth_dev) != 0) {
PMD_INIT_LOG(ERR, "Init vf failed");
+ if (!rte_atomic_load_explicit(&vf->in_reset_recovery, rte_memory_order_relaxed))
+ iavf_dev_event_handler_fini();
return -1;
}
@@ -3185,6 +3187,9 @@ iavf_dev_init(struct rte_eth_dev *eth_dev)
eth_dev->data->mac_addrs = NULL;
+ if (!rte_atomic_load_explicit(&vf->in_reset_recovery, rte_memory_order_relaxed))
+ iavf_dev_event_handler_fini();
+
init_vf_err:
iavf_uninit_vf(eth_dev);
--
2.43.0