>-----Original Message-----
>From: Intel-wired-lan <[email protected]> On Behalf Of
>Fijalkowski, Maciej
>Sent: Wednesday, February 21, 2024 3:16 AM
>To: [email protected]
>Cc: [email protected]; Fijalkowski, Maciej
><[email protected]>; Nguyen, Anthony L
><[email protected]>; Pavel Vazharov <[email protected]>; Karlsson,
>Magnus <[email protected]>
>Subject: [Intel-wired-lan] [PATCH iwl-net 1/3] ixgbe: {dis, en}able irqs in
>ixgbe_txrx_ring_{dis, en}able
>
>Currently routines that are supposed to toggle state of ring pair do not take
>care of associated interrupt with queue vector that these rings belong to. This
>causes funky issues such as dead interface due to irq misconfiguration, as per
>Pavel's report from Closes: tag.
>
>Add a function responsible for disabling single IRQ in EIMC register and call
>this
>as a very first thing when disabling ring pair during xsk_pool setup. For
>enable
>let's reuse ixgbe_irq_enable_queues(). Besides this, disable/enable NAPI as
>first/last thing when dealing with closing or opening ring pair that xsk_pool
>is
>being configured on.
>
>Reported-by: Pavel Vazharov <[email protected]>
>Closes:
>https://lore.kernel.org/netdev/CAJEV1ijxNyPTwASJER1bcZzS9nMoZJqfR86nu
>[email protected]/
>Fixes: 024aa5800f32 ("ixgbe: added Rx/Tx ring disable/enable functions")
>Signed-off-by: Maciej Fijalkowski <[email protected]>
>---
> drivers/net/ethernet/intel/ixgbe/ixgbe_main.c | 56 ++++++++++++++++---
> 1 file changed, 49 insertions(+), 7 deletions(-)
>
Tested-by: Chandan Kumar Rout <[email protected]> (A Contingent Worker at
Intel)