diff options
| author |   Benjamin Poirier <bpoirier@cumulusnetworks.com> | 2020-02-12 10:41:06 +0900 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2020-02-16 18:34:31 -0800 |
| commit | e404b8c7cfb31654c9024d497cec58a501501692 (patch) | |
| tree | 5a24c73695f44b92f343f993d3bee588fe8bf567 /net/ipv6/route.c | |
| parent | selftests: forwarding: use proto icmp for {gretap, ip6gretap}_mac testing (diff) | |
| download | linux-dev-e404b8c7cfb31654c9024d497cec58a501501692.tar.xz linux-dev-e404b8c7cfb31654c9024d497cec58a501501692.zip | |
ipv6: Fix route replacement with dev-only route
After commit 27596472473a ("ipv6: fix ECMP route replacement") it is no
longer possible to replace an ECMP-able route by a non ECMP-able route.
For example,
ip route add 2001:db8::1/128 via fe80::1 dev dummy0
ip route replace 2001:db8::1/128 dev dummy0
does not work as expected.
Tweak the replacement logic so that point 3 in the log of the above commit
becomes:
3. If the new route is not ECMP-able, and no matching non-ECMP-able route
exists, replace matching ECMP-able route (if any) or add the new route.
We can now summarize the entire replace semantics to:
When doing a replace, prefer replacing a matching route of the same
"ECMP-able-ness" as the replace argument. If there is no such candidate,
fallback to the first route found.
Fixes: 27596472473a ("ipv6: fix ECMP route replacement")
Signed-off-by: Benjamin Poirier <bpoirier@cumulusnetworks.com>
Reviewed-by: Michal Kubecek <mkubecek@suse.cz>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/route.c')
0 files changed, 0 insertions, 0 deletions