]> git.baikalelectronics.ru Git - kernel.git/commit
nexthop: Do not flush blackhole nexthops when loopback goes down
authorIdo Schimmel <idosch@nvidia.com>
Thu, 4 Mar 2021 08:57:53 +0000 (10:57 +0200)
committerDavid S. Miller <davem@davemloft.net>
Thu, 4 Mar 2021 22:04:49 +0000 (14:04 -0800)
commit8e9dfe5fcedc3cc60ed5dcffba5f1342947c7b64
tree7675d7f8f0e4980d3052ad49a8311133dd39e45b
parent17a4ba1b9e9fe096d64316b5e210ee4249fdffe3
nexthop: Do not flush blackhole nexthops when loopback goes down

As far as user space is concerned, blackhole nexthops do not have a
nexthop device and therefore should not be affected by the
administrative or carrier state of any netdev.

However, when the loopback netdev goes down all the blackhole nexthops
are flushed. This happens because internally the kernel associates
blackhole nexthops with the loopback netdev.

This behavior is both confusing to those not familiar with kernel
internals and also diverges from the legacy API where blackhole IPv4
routes are not flushed when the loopback netdev goes down:

 # ip route add blackhole 198.51.100.0/24
 # ip link set dev lo down
 # ip route show 198.51.100.0/24
 blackhole 198.51.100.0/24

Blackhole IPv6 routes are flushed, but at least user space knows that
they are associated with the loopback netdev:

 # ip -6 route show 2001:db8:1::/64
 blackhole 2001:db8:1::/64 dev lo metric 1024 pref medium

Fix this by only flushing blackhole nexthops when the loopback netdev is
unregistered.

Fixes: 4d9784be391e ("net: Initial nexthop code")
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Reported-by: Donald Sharp <sharpd@nvidia.com>
Reviewed-by: David Ahern <dsahern@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/nexthop.c