Skip to content

Commit 02cc1bd

Browse files
committed
idpf: disable local BH when scheduling napi for marker packets
JIRA: https://issues.redhat.com/browse/RHEL-29553 commit 3300685 Author: Emil Tantilov <emil.s.tantilov@intel.com> Date: Wed Feb 7 16:42:43 2024 -0800 idpf: disable local BH when scheduling napi for marker packets Fix softirq's not being handled during napi_schedule() call when receiving marker packets for queue disable by disabling local bottom half. The issue can be seen on ifdown: NOHZ tick-stop error: Non-RCU local softirq work is pending, handler #8!!! Using ftrace to catch the failing scenario: ifconfig [003] d.... 22739.830624: softirq_raise: vec=3 [action=NET_RX] <idle>-0 [003] ..s.. 22739.831357: softirq_entry: vec=3 [action=NET_RX] No interrupt and CPU is idle. After the patch when disabling local BH before calling napi_schedule: ifconfig [003] d.... 22993.928336: softirq_raise: vec=3 [action=NET_RX] ifconfig [003] ..s1. 22993.928337: softirq_entry: vec=3 [action=NET_RX] Fixes: c2d548c ("idpf: add TX splitq napi poll support") Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com> Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com> Signed-off-by: Emil Tantilov <emil.s.tantilov@intel.com> Signed-off-by: Alan Brady <alan.brady@intel.com> Reviewed-by: Simon Horman <horms@kernel.org> Tested-by: Krishneil Singh <krishneil.k.singh@intel.com> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com> Signed-off-by: Michal Schmidt <mschmidt@redhat.com>
1 parent 152d84f commit 02cc1bd

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

drivers/net/ethernet/intel/idpf/idpf_virtchnl.c

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1978,8 +1978,10 @@ int idpf_send_disable_queues_msg(struct idpf_vport *vport)
19781978
set_bit(__IDPF_Q_POLL_MODE, vport->txqs[i]->flags);
19791979

19801980
/* schedule the napi to receive all the marker packets */
1981+
local_bh_disable();
19811982
for (i = 0; i < vport->num_q_vectors; i++)
19821983
napi_schedule(&vport->q_vectors[i].napi);
1984+
local_bh_enable();
19831985

19841986
return idpf_wait_for_marker_event(vport);
19851987
}

0 commit comments

Comments
 (0)