Skip to content

Commit cf3515c

Browse files
daniellertskuba-moo
authored andcommitted
selftests: mlxsw: sharedbuffer: Remove h1 ingress test case
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: a865ad9 ("selftests: mlxsw: Add shared buffer traffic test") Signed-off-by: Danielle Ratson <[email protected]> Reviewed-by: Ido Schimmel <[email protected]> Signed-off-by: Ido Schimmel <[email protected]> Signed-off-by: Petr Machata <[email protected]> Link: https://patch.msgid.link/5b7344608d5e06f38209e48d8af8c92fa11b6742.1733414773.git.petrm@nvidia.com Signed-off-by: Jakub Kicinski <[email protected]>
1 parent 11776cf commit cf3515c

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

tools/testing/selftests/drivers/net/mlxsw/sharedbuffer.sh

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -108,11 +108,6 @@ port_pool_test()
108108

109109
devlink sb occupancy snapshot $DEVLINK_DEV
110110

111-
RET=0
112-
max_occ=$(sb_occ_pool_check $dl_port1 $SB_POOL_ING $exp_max_occ)
113-
check_err $? "Expected iPool($SB_POOL_ING) max occupancy to be $exp_max_occ, but got $max_occ"
114-
log_test "physical port's($h1) ingress pool"
115-
116111
RET=0
117112
max_occ=$(sb_occ_pool_check $dl_port2 $SB_POOL_ING $exp_max_occ)
118113
check_err $? "Expected iPool($SB_POOL_ING) max occupancy to be $exp_max_occ, but got $max_occ"

0 commit comments

Comments
 (0)