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

	F171442834: D54721.id169918.diff
	Fri, Sep 11, 3:59 AM

	F171415387: D54721.id169749.diff
	Fri, Sep 11, 12:27 AM

	Unknown Object (File)
	Thu, Sep 10, 5:45 AM

	Unknown Object (File)
	Tue, Sep 8, 5:33 AM

	Unknown Object (File)
	Mon, Sep 7, 5:10 AM

	Unknown Object (File)
	Sun, Sep 6, 10:52 AM

	Unknown Object (File)
	Sun, Sep 6, 9:27 AM

	Unknown Object (File)
	Sun, Sep 6, 4:31 AM

View All 97 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 Not Applicable

Unit

Tests Not Applicable

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 169918

sys/net/if.c

Loading...