> -----Original Message----- > From: Intel-wired-lan <intel-wired-lan-bounces@xxxxxxxxxx> On Behalf Of > Alexander Lobakin > Sent: Tuesday, August 26, 2025 9:25 PM > To: intel-wired-lan@xxxxxxxxxxxxxxxx > Cc: Lobakin, Aleksander <aleksander.lobakin@xxxxxxxxx>; Kubiak, Michal > <michal.kubiak@xxxxxxxxx>; Fijalkowski, Maciej > <maciej.fijalkowski@xxxxxxxxx>; Nguyen, Anthony L > <anthony.l.nguyen@xxxxxxxxx>; Kitszel, Przemyslaw > <przemyslaw.kitszel@xxxxxxxxx>; Andrew Lunn <andrew+netdev@xxxxxxx>; > David S. Miller <davem@xxxxxxxxxxxxx>; Eric Dumazet > <edumazet@xxxxxxxxxx>; Jakub Kicinski <kuba@xxxxxxxxxx>; Paolo Abeni > <pabeni@xxxxxxxxxx>; Alexei Starovoitov <ast@xxxxxxxxxx>; Daniel > Borkmann <daniel@xxxxxxxxxxxxx>; Simon Horman <horms@xxxxxxxxxx>; > NXNE CNSE OSDT ITP Upstreaming > <nxne.cnse.osdt.itp.upstreaming@xxxxxxxxx>; bpf@xxxxxxxxxxxxxxx; > netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: [Intel-wired-lan] [PATCH iwl-next v5 04/13] idpf: link NAPIs to queues > > Add the missing linking of NAPIs to netdev queues when enabling interrupt > vectors in order to support NAPI configuration and interfaces requiring > get_rx_queue()->napi to be set (like XSk busy polling). > > As currently, idpf_vport_{start,stop}() is called from several flows with > inconsistent RTNL locking, we need to synchronize them to avoid runtime > assertions. Notably: > > * idpf_{open,stop}() -- regular NDOs, RTNL is always taken; > * idpf_initiate_soft_reset() -- usually called under RTNL; > * idpf_init_task -- called from the init work, needs RTNL; > * idpf_vport_dealloc -- called without RTNL taken, needs it. > > Expand common idpf_vport_{start,stop}() to take an additional bool telling > whether we need to manually take the RTNL lock. > > Suggested-by: Maciej Fijalkowski <maciej.fijalkowski@xxxxxxxxx> # helper > Signed-off-by: Alexander Lobakin <aleksander.lobakin@xxxxxxxxx> > --- > drivers/net/ethernet/intel/idpf/idpf_lib.c | 38 +++++++++++++++------ > drivers/net/ethernet/intel/idpf/idpf_txrx.c | 17 +++++++++ > 2 files changed, 45 insertions(+), 10 deletions(-) > Tested-by: R,Ramu <ramu.r@xxxxxxxxx>