Page MenuHomeFreeBSD

Further refactor IPv4 interface route creation.
ClosedPublic

Authored by melifaro on Jan 19 2021, 11:51 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 5 2024, 1:11 PM
Unknown Object (File)
Dec 20 2023, 4:44 AM
Unknown Object (File)
Dec 10 2023, 9:56 AM
Unknown Object (File)
Nov 10 2023, 2:59 AM
Unknown Object (File)
Nov 9 2023, 4:41 PM
Unknown Object (File)
Nov 8 2023, 12:38 PM
Unknown Object (File)
Nov 8 2023, 2:43 AM
Unknown Object (File)
Nov 5 2023, 5:38 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36377
Build 33266: arc lint + arc unit

Event Timeline

donner added inline comments.
sys/netinet/in.c
774

The comment is misleading. This function checks, if the IP address on the interface is singular enough to add an additional loopback route for the local host in the attached network,

Of course, all excluded cases do need a loopback route, too. But not an extra one,

sys/netinet/in.c
774

Indeed, there is a clash for the "loopback" term. We will always install some loopback route, the question is whether we install it via ifa_maintain_loopback_route() or not.
In the end, we should unify both "loopback" and "prefix" route handling, but it's too late to untangle in 13.

Anyway, I've updated the description, hope it looks more clear now.

Remove in_addprefix() now-unused flags argument.

This revision is now accepted and ready to land.Jan 21 2021, 7:01 AM

this diff fixes https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=252883 thanks can you mention the PR when this patch gets merged? thanks!