Avoid calling WARN_ON() on allocation failure in cfg80211_switch_netns()

[PATCH v2 0/3] Avoid calling WARN_ON() on allocation failure in cfg80211_switch_netns()

Posted by Ivan Abramov 10 months, 1 week ago

This series is inspired by similar series in ieee802154. As far as I know,
there were no WARN_ON triggers in this code so far, but since it's almost
identical to cfg802154_* counterparts, same changes are proposed.

Link to ieee802154 series: https://lore.kernel.org/netdev/20250403101935.991385-1-i.abramov@mt-integration.ru/T/

v2: Add 2 patches. Also make sure to commit against latest
netdev/net.

Ivan Abramov (3):
  cfg80211: Restore initial state on failed device_rename() in
    cfg80211_switch_netns()
  cfg80211: Avoid calling WARN_ON() on -ENOMEM in
    cfg80211_switch_netns()
  cfg80211: Remove WARN_ON() in cfg80211_pernet_exit()

 net/wireless/core.c | 53 +++++++++++++++++++++++++--------------------
 1 file changed, 30 insertions(+), 23 deletions(-)

-- 
2.39.5

Re: [PATCH v2 0/3] Avoid calling WARN_ON() on allocation failure in cfg80211_switch_netns()

Posted by Kuniyuki Iwashima 10 months ago

From: Ivan Abramov <i.abramov@mt-integration.ru>
Date: Mon, 7 Apr 2025 15:53:41 +0300
> This series is inspired by similar series in ieee802154. As far as I know,
> there were no WARN_ON triggers in this code so far, but since it's almost
> identical to cfg802154_* counterparts, same changes are proposed.
> 
> Link to ieee802154 series: https://lore.kernel.org/netdev/20250403101935.991385-1-i.abramov@mt-integration.ru/T/
> 
> v2: Add 2 patches. Also make sure to commit against latest
> netdev/net.
> 
> Ivan Abramov (3):
>   cfg80211: Restore initial state on failed device_rename() in
>     cfg80211_switch_netns()
>   cfg80211: Avoid calling WARN_ON() on -ENOMEM in
>     cfg80211_switch_netns()
>   cfg80211: Remove WARN_ON() in cfg80211_pernet_exit()

Let's see how this thread goes first.
https://lore.kernel.org/netdev/CANn89i+UQQ6GqhWisHQEL0ECNFoQqVrO+2Ee3oDzysdR7dh=Ag@mail.gmail.com/