net/ice: fix ice dcf contrl thread crash
Checks
Commit Message
The control thread accesses the hardware resources after the
resources were released, resulting in a segment error.
This commit fixes the bug by exiting thread before resource released.
Signed-off-by: Ke Zhang <ke1x.zhang@intel.com>
---
drivers/net/ice/ice_dcf.c | 4 ++++
1 file changed, 4 insertions(+)
Comments
On Wed, 8 Feb 2023 16:30:05 +0800
Ke Zhang <ke1x.zhang@intel.com> wrote:
> + if (hw->vc_event_msg_cb == NULL)
> + pthread_exit(NULL);
Do we need rte_thread_exit() wrapper to be compatiable with Windows?
@@ -543,6 +543,8 @@ ice_dcf_handle_vsi_update_event(struct ice_dcf_hw *hw)
ice_dcf_disable_irq0(hw);
for (;;) {
+ if (hw->vc_event_msg_cb == NULL)
+ pthread_exit(NULL);
if (ice_dcf_get_vf_resource(hw) == 0 &&
ice_dcf_get_vf_vsi_map(hw) >= 0) {
err = 0;
@@ -760,6 +762,8 @@ ice_dcf_uninit_hw(struct rte_eth_dev *eth_dev, struct ice_dcf_hw *hw)
rte_intr_callback_unregister(intr_handle,
ice_dcf_dev_interrupt_handler, hw);
+ hw->vc_event_msg_cb = NULL;
+
ice_dcf_mode_disable(hw);
iavf_shutdown_adminq(&hw->avf);