Split rtinit() into multiple functions.


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.
  1. 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.
  1. 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.
  1. rtinit() is the last customer passing non-masked prefixes to rib_action(), complicating rib_action() implementation.
  1. 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


melifaroAuthored on Jan 9 2021, 12:19 AM
Differential Revision
D28186: Split rtinit() into multiple functions.
R10:bce5d6aa2485: service.8: Use entire section of literally displayed text