selftests: mlxsw: Add tc sample tests for new triggers

Test that packets are sampled when tc-sample is used with matchall
egress binding and flower classifier. Verify that when performing
sampling on egress the end-to-end latency is reported as metadata.

Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/tools/testing/selftests/drivers/net/mlxsw/tc_sample.sh b/tools/testing/selftests/drivers/net/mlxsw/tc_sample.sh
index 75d0010..57b05f0 100755 (executable)
--- a/tools/testing/selftests/drivers/net/mlxsw/tc_sample.sh
+++ b/tools/testing/selftests/drivers/net/mlxsw/tc_sample.sh
@@ -41,6 +41,10 @@ ALL_TESTS="
        tc_sample_md_lag_oif_test
        tc_sample_md_out_tc_test
        tc_sample_md_out_tc_occ_test
+       tc_sample_md_latency_test
+       tc_sample_acl_group_conflict_test
+       tc_sample_acl_rate_test
+       tc_sample_acl_max_rate_test
 "
 NUM_NETIFS=8
 CAPTURE_FILE=$(mktemp)
@@ -482,6 +486,137 @@ tc_sample_md_out_tc_occ_test()
        tc filter del dev $rp1 ingress protocol all pref 1 handle 101 matchall
 }
 
+tc_sample_md_latency_test()
+{
+       RET=0
+
+       # Egress sampling not supported on Spectrum-1.
+       [[ "$DEVLINK_VIDDID" == "15b3:cb84" ]] && return
+
+       tc filter add dev $rp2 egress protocol all pref 1 handle 101 matchall \
+               skip_sw action sample rate 5 group 1
+       check_err $? "Failed to configure sampling rule"
+
+       psample_capture_start
+
+       ip vrf exec v$h1 $MZ $h1 -c 3200 -d 1msec -p 64 -A 192.0.2.1 \
+               -B 198.51.100.1 -t udp dp=52768,sp=42768 -q
+
+       psample_capture_stop
+
+       grep -q -e "latency " $CAPTURE_FILE
+       check_err $? "Sampled packets do not have latency attribute"
+
+       log_test "tc sample latency"
+
+       tc filter del dev $rp2 egress protocol all pref 1 handle 101 matchall
+}
+
+tc_sample_acl_group_conflict_test()
+{
+       RET=0
+
+       # Test that two flower sampling rules cannot be configured on the same
+       # port with different groups.
+
+       # Policy-based sampling is not supported on Spectrum-1.
+       [[ "$DEVLINK_VIDDID" == "15b3:cb84" ]] && return
+
+       tc filter add dev $rp1 ingress protocol ip pref 1 handle 101 flower \
+               skip_sw action sample rate 1024 group 1
+       check_err $? "Failed to configure sampling rule"
+
+       tc filter add dev $rp1 ingress protocol ip pref 2 handle 102 flower \
+               skip_sw action sample rate 1024 group 1
+       check_err $? "Failed to configure sampling rule with same group"
+
+       tc filter add dev $rp1 ingress protocol ip pref 3 handle 103 flower \
+               skip_sw action sample rate 1024 group 2 &> /dev/null
+       check_fail $? "Managed to configure sampling rule with conflicting group"
+
+       log_test "tc sample (w/ flower) group conflict test"
+
+       tc filter del dev $rp1 ingress protocol ip pref 2 handle 102 flower
+       tc filter del dev $rp1 ingress protocol ip pref 1 handle 101 flower
+}
+
+__tc_sample_acl_rate_test()
+{
+       local bind=$1; shift
+       local port=$1; shift
+       local pkts pct
+
+       RET=0
+
+       # Policy-based sampling is not supported on Spectrum-1.
+       [[ "$DEVLINK_VIDDID" == "15b3:cb84" ]] && return
+
+       tc filter add dev $port $bind protocol ip pref 1 handle 101 flower \
+               skip_sw dst_ip 198.51.100.1 action sample rate 32 group 1
+       check_err $? "Failed to configure sampling rule"
+
+       psample_capture_start
+
+       ip vrf exec v$h1 $MZ $h1 -c 3200 -d 1msec -p 64 -A 192.0.2.1 \
+               -B 198.51.100.1 -t udp dp=52768,sp=42768 -q
+
+       psample_capture_stop
+
+       pkts=$(grep -e "group 1 " $CAPTURE_FILE | wc -l)
+       pct=$((100 * (pkts - 100) / 100))
+       (( -25 <= pct && pct <= 25))
+       check_err $? "Expected 100 packets, got $pkts packets, which is $pct% off. Required accuracy is +-25%"
+
+       # Setup a filter that should not match any packet and make sure packets
+       # are not sampled.
+       tc filter del dev $port $bind protocol ip pref 1 handle 101 flower
+
+       tc filter add dev $port $bind protocol ip pref 1 handle 101 flower \
+               skip_sw dst_ip 198.51.100.10 action sample rate 32 group 1
+       check_err $? "Failed to configure sampling rule"
+
+       psample_capture_start
+
+       ip vrf exec v$h1 $MZ $h1 -c 3200 -d 1msec -p 64 -A 192.0.2.1 \
+               -B 198.51.100.1 -t udp dp=52768,sp=42768 -q
+
+       psample_capture_stop
+
+       grep -q -e "group 1 " $CAPTURE_FILE
+       check_fail $? "Sampled packets when should not"
+
+       log_test "tc sample (w/ flower) rate ($bind)"
+
+       tc filter del dev $port $bind protocol ip pref 1 handle 101 flower
+}
+
+tc_sample_acl_rate_test()
+{
+       __tc_sample_acl_rate_test ingress $rp1
+       __tc_sample_acl_rate_test egress $rp2
+}
+
+tc_sample_acl_max_rate_test()
+{
+       RET=0
+
+       # Policy-based sampling is not supported on Spectrum-1.
+       [[ "$DEVLINK_VIDDID" == "15b3:cb84" ]] && return
+
+       tc filter add dev $rp1 ingress protocol ip pref 1 handle 101 flower \
+               skip_sw action sample rate $((2 ** 24 - 1)) group 1
+       check_err $? "Failed to configure sampling rule with max rate"
+
+       tc filter del dev $rp1 ingress protocol ip pref 1 handle 101 flower
+
+       tc filter add dev $rp1 ingress protocol ip pref 1 handle 101 flower \
+               skip_sw action sample rate $((2 ** 24)) \
+               group 1 &> /dev/null
+       check_fail $? "Managed to configure sampling rate above maximum"
+
+       log_test "tc sample (w/ flower) maximum rate"
+}
+
 trap cleanup EXIT
 
 setup_prepare