Commit 8eadf9bb authored by David S. Miller's avatar David S. Miller

Merge branch 'selftests-forwarding-GRE-multipath-fixes'

Ido Schimmel says:

====================
selftests: forwarding: GRE multipath fixes

Patch #1 ensures IPv4 forwarding is enabled during the test.

Patch #2 fixes the flower filters used to measure the distribution of
the traffic between the two nexthops, so that the test will pass
regardless if traffic is offloaded or not.
====================
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents b3e487c0 1be79d89
...@@ -93,18 +93,10 @@ sw1_create() ...@@ -93,18 +93,10 @@ sw1_create()
ip route add vrf v$ol1 192.0.2.16/28 \ ip route add vrf v$ol1 192.0.2.16/28 \
nexthop dev g1a \ nexthop dev g1a \
nexthop dev g1b nexthop dev g1b
tc qdisc add dev $ul1 clsact
tc filter add dev $ul1 egress pref 111 prot ipv4 \
flower dst_ip 192.0.2.66 action pass
tc filter add dev $ul1 egress pref 222 prot ipv4 \
flower dst_ip 192.0.2.82 action pass
} }
sw1_destroy() sw1_destroy()
{ {
tc qdisc del dev $ul1 clsact
ip route del vrf v$ol1 192.0.2.16/28 ip route del vrf v$ol1 192.0.2.16/28
ip route del vrf v$ol1 192.0.2.82/32 via 192.0.2.146 ip route del vrf v$ol1 192.0.2.82/32 via 192.0.2.146
...@@ -139,10 +131,18 @@ sw2_create() ...@@ -139,10 +131,18 @@ sw2_create()
ip route add vrf v$ol2 192.0.2.0/28 \ ip route add vrf v$ol2 192.0.2.0/28 \
nexthop dev g2a \ nexthop dev g2a \
nexthop dev g2b nexthop dev g2b
tc qdisc add dev $ul2 clsact
tc filter add dev $ul2 ingress pref 111 prot 802.1Q \
flower vlan_id 111 action pass
tc filter add dev $ul2 ingress pref 222 prot 802.1Q \
flower vlan_id 222 action pass
} }
sw2_destroy() sw2_destroy()
{ {
tc qdisc del dev $ul2 clsact
ip route del vrf v$ol2 192.0.2.0/28 ip route del vrf v$ol2 192.0.2.0/28
ip route del vrf v$ol2 192.0.2.81/32 via 192.0.2.145 ip route del vrf v$ol2 192.0.2.81/32 via 192.0.2.145
...@@ -187,12 +187,16 @@ setup_prepare() ...@@ -187,12 +187,16 @@ setup_prepare()
sw1_create sw1_create
sw2_create sw2_create
h2_create h2_create
forwarding_enable
} }
cleanup() cleanup()
{ {
pre_cleanup pre_cleanup
forwarding_restore
h2_destroy h2_destroy
sw2_destroy sw2_destroy
sw1_destroy sw1_destroy
...@@ -211,15 +215,15 @@ multipath4_test() ...@@ -211,15 +215,15 @@ multipath4_test()
nexthop dev g1a weight $weight1 \ nexthop dev g1a weight $weight1 \
nexthop dev g1b weight $weight2 nexthop dev g1b weight $weight2
local t0_111=$(tc_rule_stats_get $ul1 111 egress) local t0_111=$(tc_rule_stats_get $ul2 111 ingress)
local t0_222=$(tc_rule_stats_get $ul1 222 egress) local t0_222=$(tc_rule_stats_get $ul2 222 ingress)
ip vrf exec v$h1 \ ip vrf exec v$h1 \
$MZ $h1 -q -p 64 -A 192.0.2.1 -B 192.0.2.18 \ $MZ $h1 -q -p 64 -A 192.0.2.1 -B 192.0.2.18 \
-d 1msec -t udp "sp=1024,dp=0-32768" -d 1msec -t udp "sp=1024,dp=0-32768"
local t1_111=$(tc_rule_stats_get $ul1 111 egress) local t1_111=$(tc_rule_stats_get $ul2 111 ingress)
local t1_222=$(tc_rule_stats_get $ul1 222 egress) local t1_222=$(tc_rule_stats_get $ul2 222 ingress)
local d111=$((t1_111 - t0_111)) local d111=$((t1_111 - t0_111))
local d222=$((t1_222 - t0_222)) local d222=$((t1_222 - t0_222))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment