Differential D54721

net: on interface detach purge all its routes before detaching protocols
ClosedPublic
Actions

Authored by glebius on Jan 15 2026, 3:24 AM.

Tags

None

Referenced Files

	F151416256: D54721.id.diff
	Wed, Apr 8, 7:07 AM

	F151328234: D54721.id169749.diff
	Tue, Apr 7, 3:29 PM

	Unknown Object (File)
	Mon, Apr 6, 3:20 PM

	Unknown Object (File)
	Sun, Apr 5, 4:57 PM

	Unknown Object (File)
	Sat, Apr 4, 3:21 PM

	Unknown Object (File)
	Thu, Apr 2, 2:22 AM

	Unknown Object (File)
	Fri, Mar 27, 12:04 AM

	Unknown Object (File)
	Wed, Mar 18, 11:20 PM

View All 35 Files

Subscribers

Details

Reviewers

jhibbits
pouria

Group Reviewers

network
transport

Commits

rGf102a1834933: net: on interface detach purge all its routes before detaching protocols

Summary

Otherwise, a forwarding thread may use the interface being detached. This
is a regression from 0d469d23715d, which manifests itself as a reliably
reproducible panic in in6_selecthlim(). Note that there are old bug
reports about such a panic, and I believe this change will not fix them,
as their nature is not due to a screwed up detach sequence, but due to
lack of proper epoch(9) based synchronization between the detach and
forwarding.

Reported & tested by: jhibbits
PR: 292162
Fixes: 0d469d23715d690b863787ebfa51529e1f6a9092

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Skipped

Unit

Tests Skipped

Build Status

Buildable 69896
Build 66779: arc lint + arc unit

Event Timeline

glebius created this revision.Jan 15 2026, 3:24 AM

Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptJan 15 2026, 3:24 AM

glebius requested review of this revision.Jan 15 2026, 3:24 AM

Harbormaster completed remote builds in B69896: Diff 169749.Jan 15 2026, 3:24 AM

glebius added a child revision: D54722: netinet6: embed struct nd_ifinfo into struct in6_ndireq.Jan 15 2026, 3:24 AM

I will test it with openbgp and bird.
I suspect that removing routes before detaching the actual interface might cause unexpected behavior in them.

LGTM
I also tested interface creation and destruction to advertise/withdraw its routes with bird3 (ospf) and openbgpd8.

This revision is now accepted and ready to land.Jan 16 2026, 3:01 PM

Closed by commit rGf102a1834933: net: on interface detach purge all its routes before detaching protocols (authored by glebius). · Explain WhyJan 17 2026, 12:45 AM

This revision was automatically updated to reflect the committed changes.

glebius added a commit: rGf102a1834933: net: on interface detach purge all its routes before detaching protocols.

Revision Contents
Changeset List

Path

Size

sys/

net/

4 lines

Diff 169749

sys/net/if.c

Loading...