diff options
author | Maciej Fijalkowski <[email protected]> | 2024-02-06 13:41:31 +0100 |
---|---|---|
committer | Tony Nguyen <[email protected]> | 2024-02-12 16:58:08 -0800 |
commit | 89a373e9131d4200038a0ef232dad80212209de0 (patch) | |
tree | 940a131ea236420139c1b15ed29bbc407370b55d /lib/test_fortify/write_overflow-strncpy-src.c | |
parent | 343bb39e1f25a73a38a03d98ca383495c0ed6e92 (diff) |
i40e: avoid double calling i40e_pf_rxq_wait()
Currently, when interface is being brought down and
i40e_vsi_stop_rings() is called, i40e_pf_rxq_wait() is called two times,
which is wrong. To showcase this scenario, simplified call stack looks
as follows:
i40e_vsi_stop_rings()
i40e_control wait rx_q()
i40e_control_rx_q()
i40e_pf_rxq_wait()
i40e_vsi_wait_queues_disabled()
i40e_pf_rxq_wait() // redundant call
To fix this, let us s/i40e_control_wait_rx_q/i40e_control_rx_q within
i40e_vsi_stop_rings().
Fixes: 65662a8dcdd0 ("i40e: Fix logic of disabling queues")
Reviewed-by: Simon Horman <[email protected]>
Signed-off-by: Maciej Fijalkowski <[email protected]>
Reviewed-by: Ivan Vecera <[email protected]>
Tested-by: Chandan Kumar Rout <[email protected]> (A Contingent Worker at Intel)
Signed-off-by: Tony Nguyen <[email protected]>
Diffstat (limited to 'lib/test_fortify/write_overflow-strncpy-src.c')
0 files changed, 0 insertions, 0 deletions