]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Split rtinit() into multiple functions.
authorAlexander V. Chernikov <melifaro@FreeBSD.org>
Sat, 9 Jan 2021 00:19:25 +0000 (00:19 +0000)
committerAlexander V. Chernikov <melifaro@FreeBSD.org>
Sat, 16 Jan 2021 22:42:41 +0000 (22:42 +0000)
commit81728a538d24f483d0986850fa3f51d5d84d8f26
treed9912fcca45eb86fc9d1347191ca485fb03ad049
parentbce5d6aa2485925c37240289a5479812a5bb46ff
Split rtinit() into multiple functions.

rtinit[1]() is a function used to add or remove interface address prefix routes,
  similar to ifa_maintain_loopback_route().
It was intended to be family-agnostic. There is a problem with this approach
 in reality.

1) IPv6 code does not use it for the ifa routes. There is a separate layer,
  nd6_prelist_(), providing interface for maintaining interface routes. Its part,
  responsible for the actual route table interaction, mimics rtenty() code.

2) rtinit tries to combine multiple actions in the same function: constructing
  proper route attributes and handling iterations over multiple fibs, for the
  non-zero net.add_addr_allfibs use case. It notably increases the code complexity.

3) dstaddr handling. flags parameter re-uses RTF_ flags. As there is no special flag
 for p2p connections, host routes and p2p routes are handled in the same way.
 Additionally, mapping IFA flags to RTF flags makes the interface pretty messy.
 It make rtinit() to clash with ifa_mainain_loopback_route() for IPV4 interface
 aliases.

4) rtinit() is the last customer passing non-masked prefixes to rib_action(),
 complicating rib_action() implementation.

5) rtinit() coupled ifa announce/withdrawal notifications, producing "false positive"
 ifa messages in certain corner cases.

To address all these points, the following has been done:

* rtinit() has been split into multiple functions:
- Route attribute construction were moved to the per-address-family functions,
 dealing with (2), (3) and (4).
- funnction providing net.add_addr_allfibs handling and route rtsock notificaions
 is the new routing table inteface.
- rtsock ifa notificaion has been moved out as well. resulting set of funcion are only
 responsible for the actual route notifications.

Side effects:
* /32 alias does not result in interface routes (/32 route and "host" route)
* RTF_PINNED is now set for IPv6 prefixes corresponding to the interface addresses

Differential revision: https://reviews.freebsd.org/D28186
sys/net/if_spppsubr.c
sys/net/route.c
sys/net/route.h
sys/net/route/route_ctl.h
sys/net/route/route_ifaddrs.c
sys/netinet/in.c
sys/netinet/in_var.h
sys/netinet/raw_ip.c
sys/netinet6/in6.c
sys/netinet6/nd6_rtr.c
tests/sys/net/routing/test_rtsock_l3.c