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

The test is sending only one packet generated with mausezahn from $h1 to
$h2. However, for some reason, it is testing for non-zero maximum occupancy
in both the ingress pool of $h1 and $h2. The former only passes when $h2
happens to send a packet.

Avoid intermittent failures by removing unintentional test case
regarding the ingress pool of $h1.

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>
---
 tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh | 5 -----
 1 file changed, 5 deletions(-)

diff --git a/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh 
b/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh
index 0c47faff9274..a7b3d6cf3185 100755
--- a/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh
+++ b/tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh
@@ -108,11 +108,6 @@ port_pool_test()
 
        devlink sb occupancy snapshot $DEVLINK_DEV
 
-       RET=0
-       max_occ=$(sb_occ_pool_check $dl_port1 $SB_POOL_ING $exp_max_occ)
-       check_err $? "Expected iPool($SB_POOL_ING) max occupancy to be 
$exp_max_occ, but got $max_occ"
-       log_test "physical port's($h1) ingress pool"
-
        RET=0
        max_occ=$(sb_occ_pool_check $dl_port2 $SB_POOL_ING $exp_max_occ)
        check_err $? "Expected iPool($SB_POOL_ING) max occupancy to be 
$exp_max_occ, but got $max_occ"
-- 
2.47.0


Reply via email to