summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJiri Pirko <jiri@mellanox.com>2018-03-07 13:57:59 +0100
committerDavid S. Miller <davem@davemloft.net>2018-03-07 15:41:19 -0500
commit0c17db05ec982607121b8f01cf4ce03513964d9c (patch)
tree9a38013adf723611008e4a397ffa960f70853d5d
parentbb5441fc3cc0e11be0ff9bbc6a00a74b3d2b0153 (diff)
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>
-rwxr-xr-xtools/testing/selftests/net/forwarding/tc_actions.sh5
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