Commit 7ede22e6 authored by Ido Schimmel's avatar Ido Schimmel Committed by David S. Miller

selftests: mlxsw: Test vetoing of double sampling

Test that two sampling rules cannot be configured on the same port with
the same trigger.
Signed-off-by: default avatarIdo Schimmel <idosch@nvidia.com>
Reviewed-by: default avatarJiri Pirko <jiri@nvidia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 17b96a5c
...@@ -34,6 +34,7 @@ lib_dir=$(dirname $0)/../../../net/forwarding ...@@ -34,6 +34,7 @@ lib_dir=$(dirname $0)/../../../net/forwarding
ALL_TESTS=" ALL_TESTS="
tc_sample_rate_test tc_sample_rate_test
tc_sample_max_rate_test tc_sample_max_rate_test
tc_sample_conflict_test
tc_sample_group_conflict_test tc_sample_group_conflict_test
tc_sample_md_iif_test tc_sample_md_iif_test
tc_sample_md_lag_iif_test tc_sample_md_lag_iif_test
...@@ -272,6 +273,35 @@ tc_sample_max_rate_test() ...@@ -272,6 +273,35 @@ tc_sample_max_rate_test()
log_test "tc sample maximum rate" log_test "tc sample maximum rate"
} }
tc_sample_conflict_test()
{
RET=0
# Test that two sampling rules cannot be configured on the same port,
# even when they share the same parameters.
tc filter add dev $rp1 ingress protocol all pref 1 handle 101 matchall \
skip_sw action sample rate 1024 group 1
check_err $? "Failed to configure sampling rule"
tc filter add dev $rp1 ingress protocol all pref 2 handle 102 matchall \
skip_sw action sample rate 1024 group 1 &> /dev/null
check_fail $? "Managed to configure second sampling rule"
# Delete the first rule and make sure the second rule can now be
# configured.
tc filter del dev $rp1 ingress protocol all pref 1 handle 101 matchall
tc filter add dev $rp1 ingress protocol all pref 2 handle 102 matchall \
skip_sw action sample rate 1024 group 1
check_err $? "Failed to configure sampling rule after deletion"
log_test "tc sample conflict test"
tc filter del dev $rp1 ingress protocol all pref 2 handle 102 matchall
}
tc_sample_group_conflict_test() tc_sample_group_conflict_test()
{ {
RET=0 RET=0
......
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