From: Danielle Ratson <daniel...@nvidia.com>

On both port_tc_ip_test() and port_tc_arp_test(), the max occupancy is
checked on $h2 twice, when only the error message is different and does not
match the check itself.

Remove the two duplicated test cases from the test.

Fixes: a865ad999603 ("selftests: mlxsw: Add shared buffer traffic test")
Signed-off-by: Danielle Ratson <daniel...@nvidia.com>
Reviewed-by: Ido Schimmel <ido...@nvidia.com>
Signed-off-by: Ido Schimmel <ido...@nvidia.com>
Signed-off-by: Petr Machata <pe...@nvidia.com>
---
 .../selftests/drivers/net/mlxsw/sharedbuffer.sh        | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh 
b/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh
index a7b3d6cf3185..21bebc5726f6 100755
--- a/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh
+++ b/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh
@@ -131,11 +131,6 @@ port_tc_ip_test()
 
        devlink sb occupancy snapshot $DEVLINK_DEV
 
-       RET=0
-       max_occ=$(sb_occ_itc_check $dl_port2 $SB_ITC $exp_max_occ)
-       check_err $? "Expected ingress TC($SB_ITC) max occupancy to be 
$exp_max_occ, but got $max_occ"
-       log_test "physical port's($h1) ingress TC - IP packet"
-
        RET=0
        max_occ=$(sb_occ_itc_check $dl_port2 $SB_ITC $exp_max_occ)
        check_err $? "Expected ingress TC($SB_ITC) max occupancy to be 
$exp_max_occ, but got $max_occ"
@@ -158,11 +153,6 @@ port_tc_arp_test()
 
        devlink sb occupancy snapshot $DEVLINK_DEV
 
-       RET=0
-       max_occ=$(sb_occ_itc_check $dl_port2 $SB_ITC $exp_max_occ)
-       check_err $? "Expected ingress TC($SB_ITC) max occupancy to be 
$exp_max_occ, but got $max_occ"
-       log_test "physical port's($h1) ingress TC - ARP packet"
-
        RET=0
        max_occ=$(sb_occ_itc_check $dl_port2 $SB_ITC $exp_max_occ)
        check_err $? "Expected ingress TC($SB_ITC) max occupancy to be 
$exp_max_occ, but got $max_occ"
-- 
2.47.0


Reply via email to