All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH iwl-net v2 0/6] ice: fix synchronization between .ndo_bpf() and reset
@ 2024-07-24 16:48 ` Larysa Zaremba
  0 siblings, 0 replies; 54+ messages in thread
From: Larysa Zaremba @ 2024-07-24 16:48 UTC (permalink / raw)
  To: intel-wired-lan
  Cc: Larysa Zaremba, Tony Nguyen, David S. Miller, Jacob Keller,
	Eric Dumazet, Jakub Kicinski, Paolo Abeni, Alexei Starovoitov,
	Daniel Borkmann, Jesper Dangaard Brouer, John Fastabend,
	Maciej Fijalkowski, netdev, linux-kernel, bpf, magnus.karlsson,
	Michal Kubiak, Wojciech Drewek, Amritha Nambiar

PF reset can be triggered asynchronously, by tx_timeout or by a user. With some
unfortunate timings both ice_vsi_rebuild() and .ndo_bpf will try to access and
modify XDP rings at the same time, causing system crash.

The first patch factors out rtnl-locked code from VSI rebuild code to avoid
deadlock. The following changes lock rebuild and .ndo_bpf() critical sections
with an internal mutex as well and provide complementary fixes.

v1: https://lore.kernel.org/netdev/20240610153716.31493-1-larysa.zaremba@intel.com/
v1->v2:
* use mutex for locking
* redefine critical sections
* account for short time between rebuild and VSI being open
* add netif_queue_set_napi() patch, so ICE_RTNL_WAITS_FOR_RESET strategy can be
  dropped, no more rtnl-locked code in ice_vsi_rebuild()
* change the test case from waiting for tx_timeout to happen to actively firing
  resets through sysfs, this adds more minor fixes on top

Larysa Zaremba (6):
  ice: move netif_queue_set_napi to rtnl-protected sections
  ice: protect XDP configuration with a mutex
  ice: check for XDP rings instead of bpf program when unconfiguring
  ice: check ICE_VSI_DOWN under rtnl_lock when preparing for reset
  ice: remove ICE_CFG_BUSY locking from AF_XDP code
  ice: do not bring the VSI up, if it was down before the XDP setup

 drivers/net/ethernet/intel/ice/ice.h      |   2 +
 drivers/net/ethernet/intel/ice/ice_base.c |  11 +-
 drivers/net/ethernet/intel/ice/ice_lib.c  | 171 +++++++---------------
 drivers/net/ethernet/intel/ice/ice_lib.h  |  10 +-
 drivers/net/ethernet/intel/ice/ice_main.c |  47 ++++--
 drivers/net/ethernet/intel/ice/ice_xsk.c  |  18 +--
 6 files changed, 102 insertions(+), 157 deletions(-)

-- 
2.43.0


^ permalink raw reply	[flat|nested] 54+ messages in thread

end of thread, other threads:[~2024-08-13 13:37 UTC | newest]

Thread overview: 54+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-07-24 16:48 [PATCH iwl-net v2 0/6] ice: fix synchronization between .ndo_bpf() and reset Larysa Zaremba
2024-07-24 16:48 ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 16:48 ` [PATCH iwl-net v2 1/6] ice: move netif_queue_set_napi to rtnl-protected sections Larysa Zaremba
2024-07-24 16:48   ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 18:21   ` Jacob Keller
2024-07-24 18:21     ` [Intel-wired-lan] " Jacob Keller
2024-07-24 20:40   ` Nambiar, Amritha
2024-07-24 20:40     ` [Intel-wired-lan] " Nambiar, Amritha
2024-08-08  2:19   ` Rout, ChandanX
2024-08-08  2:19     ` Rout, ChandanX
2024-07-24 16:48 ` [PATCH iwl-net v2 2/6] ice: protect XDP configuration with a mutex Larysa Zaremba
2024-07-24 16:48   ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 18:24   ` Jacob Keller
2024-07-24 18:24     ` [Intel-wired-lan] " Jacob Keller
2024-08-08  2:16   ` Rout, ChandanX
2024-08-08  2:16     ` Rout, ChandanX
2024-08-13 11:31   ` Maciej Fijalkowski
2024-08-13 11:31     ` [Intel-wired-lan] " Maciej Fijalkowski
2024-08-13 13:36     ` Larysa Zaremba
2024-08-13 13:36       ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 16:48 ` [PATCH iwl-net v2 3/6] ice: check for XDP rings instead of bpf program when unconfiguring Larysa Zaremba
2024-07-24 16:48   ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 18:25   ` Jacob Keller
2024-07-24 18:25     ` [Intel-wired-lan] " Jacob Keller
2024-08-08  2:18   ` Rout, ChandanX
2024-08-08  2:18     ` Rout, ChandanX
2024-08-12 12:58   ` Maciej Fijalkowski
2024-08-12 12:58     ` [Intel-wired-lan] " Maciej Fijalkowski
2024-08-12 15:13     ` Larysa Zaremba
2024-08-12 15:13       ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 16:48 ` [PATCH iwl-net v2 4/6] ice: check ICE_VSI_DOWN under rtnl_lock when preparing for reset Larysa Zaremba
2024-07-24 16:48   ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 18:27   ` Jacob Keller
2024-07-24 18:27     ` [Intel-wired-lan] " Jacob Keller
2024-08-08  2:15   ` Rout, ChandanX
2024-08-08  2:15     ` Rout, ChandanX
2024-07-24 16:48 ` [PATCH iwl-net v2 5/6] ice: remove ICE_CFG_BUSY locking from AF_XDP code Larysa Zaremba
2024-07-24 16:48   ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 18:37   ` Jacob Keller
2024-07-24 18:37     ` [Intel-wired-lan] " Jacob Keller
2024-08-08  2:17   ` Rout, ChandanX
2024-08-08  2:17     ` Rout, ChandanX
2024-08-12 13:03   ` Maciej Fijalkowski
2024-08-12 13:03     ` [Intel-wired-lan] " Maciej Fijalkowski
2024-08-12 15:59     ` Larysa Zaremba
2024-08-12 15:59       ` [Intel-wired-lan] " Larysa Zaremba
2024-08-13 10:28       ` Maciej Fijalkowski
2024-08-13 10:28         ` [Intel-wired-lan] " Maciej Fijalkowski
2024-07-24 16:48 ` [PATCH iwl-net v2 6/6] ice: do not bring the VSI up, if it was down before the XDP setup Larysa Zaremba
2024-07-24 16:48   ` [Intel-wired-lan] " Larysa Zaremba
2024-07-24 18:40   ` Jacob Keller
2024-07-24 18:40     ` [Intel-wired-lan] " Jacob Keller
2024-08-08  2:14   ` Rout, ChandanX
2024-08-08  2:14     ` Rout, ChandanX

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.