diff options
| author |   Jiri Pirko <jiri@mellanox.com> | 2018-03-07 13:57:59 +0100 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-03-07 15:41:19 -0500 |
| commit | 0c17db05ec982607121b8f01cf4ce03513964d9c (patch) | |
| tree | 9a38013adf723611008e4a397ffa960f70853d5d /tools/testing/selftests/net/forwarding/tc_actions.sh | |
| parent | net: cdc_eem: clean up bind error path (diff) | |
| download | linux-dev-0c17db05ec982607121b8f01cf4ce03513964d9c.tar.xz linux-dev-0c17db05ec982607121b8f01cf4ce03513964d9c.zip | |
selftests: forwarding: fix "ok" action test
Fix the "ok" action test so it checks that packet that is okayed does not
continue to be processed by other rules. Fix error message as well.
Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Reviewed-by: Ido Schimmel <idosch@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools/testing/selftests/net/forwarding/tc_actions.sh')
| -rwxr-xr-x | tools/testing/selftests/net/forwarding/tc_actions.sh | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/testing/selftests/net/forwarding/tc_actions.sh b/tools/testing/selftests/net/forwarding/tc_actions.sh index 6b18ba2d3982..ac6b6a1057d8 100755 --- a/tools/testing/selftests/net/forwarding/tc_actions.sh +++ b/tools/testing/selftests/net/forwarding/tc_actions.sh @@ -96,7 +96,10 @@ gact_drop_and_ok_test() -t ip -q tc_check_packets "dev $swp1 ingress" 101 1 - check_err $? "Did not see trapped packet" + check_err $? "Did not see passed packet" + + tc_check_packets "dev $swp1 ingress" 102 2 + check_fail $? "Packet was dropped and it should not reach here" tc filter del dev $swp1 ingress protocol ip pref 2 handle 102 flower tc filter del dev $swp1 ingress protocol ip pref 1 handle 101 flower |