| Message ID | 20210707081530.1107289-1-liuhangbin@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Fix selftests icmp_redirect.sh failures | expand |
Hello: This series was applied to netdev/net.git (refs/heads/master): On Wed, 7 Jul 2021 16:15:28 +0800 you wrote: > This patchset fixes 2 failures for selftests icmp_redirect.sh. > > The first patch is for IPv6 redirecting tests. When disable option > CONFIG_IPV6_SUBTREES, there is not "from ::" info in ip route output. > So we should not check this key word. > > The second patch is for testing "mtu exception plus redirect", which do > a PMTU update first and do redirect second. For topology like > > [...] Here is the summary with links: - [net,1/2] selftests: icmp_redirect: remove from checking for IPv6 route get https://git.kernel.org/netdev/net/c/24b671aad4ea - [net,2/2] selftests: icmp_redirect: IPv6 PMTU info should be cleared after redirect https://git.kernel.org/netdev/net/c/0e02bf5de46a You are awesome, thank you! -- Deet-doot-dot, I am a bot. https://korg.docs.kernel.org/patchwork/pwbot.html
This patchset fixes 2 failures for selftests icmp_redirect.sh. The first patch is for IPv6 redirecting tests. When disable option CONFIG_IPV6_SUBTREES, there is not "from ::" info in ip route output. So we should not check this key word. The second patch is for testing "mtu exception plus redirect", which do a PMTU update first and do redirect second. For topology like (MTU 1300) |----- R1_1 -------- R1_2 --------| H1 ---| | ---- H2 (MTU 1500) |----- R2_1 -------- R2_2 --------| After redirecting to R2 patch, the PMTU info in R1 path should be cleard. So we should not check PMTU info for test "mtu exception plus redirect". After the fixes, all the test passed now ]# ./icmp_redirect.sh ########################################################################### Legacy routing ########################################################################### TEST: IPv4: redirect exception [ OK ] TEST: IPv6: redirect exception [ OK ] TEST: IPv4: redirect exception plus mtu [ OK ] TEST: IPv6: redirect exception plus mtu [ OK ] TEST: IPv4: routing reset [ OK ] TEST: IPv6: routing reset [ OK ] TEST: IPv4: mtu exception [ OK ] TEST: IPv6: mtu exception [ OK ] TEST: IPv4: mtu exception plus redirect [ OK ] TEST: IPv6: mtu exception plus redirect [ OK ] [...] ########################################################################### Routing with nexthop objects and VRF ########################################################################### TEST: IPv4: redirect exception [ OK ] TEST: IPv6: redirect exception [ OK ] TEST: IPv4: redirect exception plus mtu [ OK ] TEST: IPv6: redirect exception plus mtu [ OK ] TEST: IPv4: routing reset [ OK ] TEST: IPv6: routing reset [ OK ] TEST: IPv4: mtu exception [ OK ] TEST: IPv6: mtu exception [ OK ] TEST: IPv4: mtu exception plus redirect [ OK ] TEST: IPv6: mtu exception plus redirect [ OK ] Tests passed: 40 Tests failed: 0 Tests xfailed: 0 Hangbin Liu (2): selftests: icmp_redirect: remove from checking for IPv6 route get selftests: icmp_redirect: IPv6 PMTU info should be cleared after redirect tools/testing/selftests/net/icmp_redirect.sh | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)