Re: [syzbot ci] Re: ipmr: No RTNL for RTNL_FAMILY_IPMR rtnetlink.
From: Kuniyuki Iwashima <kuniyu@google.com>
Date: 2026-02-26 19:15:18
Also in:
syzbot
Subsystem:
networking [general], networking [ipv4/ipv6], the rest · Maintainers:
"David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, David Ahern, Ido Schimmel, Linus Torvalds
From: syzbot ci <redacted> Date: Thu, 26 Feb 2026 09:00:11 -0800
syzbot ci has tested the following series [v1] ipmr: No RTNL for RTNL_FAMILY_IPMR rtnetlink. https://lore.kernel.org/all/20260226023637.1335854-1-kuniyu@google.com (local) * [PATCH v1 net-next 01/15] selftest: net: Add basic functionality tests for ipmr. * [PATCH v1 net-next 02/15] ipmr: Annotate access to mrt->mroute_do_{pim,assert,wrvifwhole}. * [PATCH v1 net-next 03/15] ipmr: Convert ipmr_rtm_dumplink() to RCU. * [PATCH v1 net-next 04/15] ipmr: Use MAXVIFS in mroute_msgsize(). * [PATCH v1 net-next 05/15] ipmr: Convert ipmr_rtm_getroute() to RCU. * [PATCH v1 net-next 06/15] ipmr: Convert ipmr_rtm_dumproute() to RCU. * [PATCH v1 net-next 07/15] ipmr: Move unregister_netdevice_many() out of mroute_clean_tables(). * [PATCH v1 net-next 08/15] ipmr: Move unregister_netdevice_many() out of ipmr_free_table(). * [PATCH v1 net-next 09/15] ipmr: Convert ipmr_net_exit_batch() to ->exit_rtnl(). * [PATCH v1 net-next 10/15] ipmr: Remove RTNL in ipmr_rules_init() and ipmr_net_init(). * [PATCH v1 net-next 11/15] ipmr: Call fib_rules_unregister() without RTNL. * [PATCH v1 net-next 12/15] ipmr: Define net->ipv4.{ipmr_notifier_ops,ipmr_seq} under CONFIG_IP_MROUTE. * [PATCH v1 net-next 13/15] ipmr/ip6mr: Convert net->ipv[46].ipmr_seq to atomic_t. * [PATCH v1 net-next 14/15] ipmr: Add dedicated mutex for mrt->{mfc_hash,mfc_cache_list}. * [PATCH v1 net-next 15/15] ipmr: Don't hold RTNL for ipmr_rtm_route(). and found the following issue: WARNING in ipmr_free_table Full report is available here: https://ci.syzbot.org/series/d7068eba-72b1-4110-aeb0-d4528e03fd24 *** WARNING in ipmr_free_table tree: net-next URL: https://kernel.googlesource.com/pub/scm/linux/kernel/git/netdev/net-next.git base: c7f5c6fb0f2b1a44490a36582a251f0a304d6b0c arch: amd64 compiler: Debian clang version 21.1.8 (++20251221033036+2078da43e25a-1~exp1~20251221153213.50), Debian LLD 21.1.8 config: https://ci.syzbot.org/builds/0afa280b-14ff-4aec-8a20-7f8507025dd8/config C repro: https://ci.syzbot.org/findings/e38687c7-5a8b-4c68-909c-a0ffdd17a674/c_repro syz repro: https://ci.syzbot.org/findings/e38687c7-5a8b-4c68-909c-a0ffdd17a674/syz_repro ------------[ cut here ]------------ !net_initialized(net) && !list_empty(dev_kill_list) WARNING: net/ipv4/ipmr.c:450 at ipmr_free_table+0x150/0x180 net/ipv4/ipmr.c:450, CPU#1: syz.1.18/6004
I forgot dev_kill_list could have some devices collected from _other_ ->rtnl_exit() _after_ successful ipmr init. My assumption in patch 10 is still correct, and I'll squash this in v2.
diff --git a/net/ipv4/ipmr.c b/net/ipv4/ipmr.c
index 4e645a87dc6c..89e151121cea 100644
--- a/net/ipv4/ipmr.c
+++ b/net/ipv4/ipmr.c@@ -437,17 +437,19 @@ static struct mr_table *ipmr_new_table(struct net *net, u32 id) static void ipmr_free_table(struct mr_table *mrt, struct list_head *dev_kill_list) { struct net *net = read_pnet(&mrt->net); + LIST_HEAD(ipmr_dev_kill_list); WARN_ON_ONCE(!mr_can_free_table(net)); timer_shutdown_sync(&mrt->ipmr_expire_timer); mroute_clean_tables(mrt, MRT_FLUSH_VIFS | MRT_FLUSH_VIFS_STATIC | MRT_FLUSH_MFC | MRT_FLUSH_MFC_STATIC, - dev_kill_list); + &ipmr_dev_kill_list); rhltable_destroy(&mrt->mfc_hash); kfree(mrt); - WARN_ON_ONCE(!net_initialized(net) && !list_empty(dev_kill_list)); + WARN_ON_ONCE(!net_initialized(net) && !list_empty(&ipmr_dev_kill_list)); + list_splice(&ipmr_dev_kill_list, dev_kill_list); } /* Service routines creating virtual interfaces: DVMRP tunnels and PIMREG */