Skip to content

Commit

Permalink
Merge branch 'selftests-mlxsw-a-couple-of-fixes'
Browse files Browse the repository at this point in the history
Ido Schimmel says:

====================
selftests: mlxsw: A couple of fixes

Patch #1 fixes a breakage due to a change in iproute2 output. The real
problem is not iproute2, but the fact that the check was not strict
enough. Fixed by using JSON output instead. Targeting at net so that the
test will pass as part of old and new kernels regardless of iproute2
version.

Patch #2 fixes an issue uncovered by the first one.
====================

Link: https://lore.kernel.org/r/20220302161447.217447-1-idosch@nvidia.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
  • Loading branch information
kuba-moo committed Mar 3, 2022
2 parents 10b6bb6 + 196f9bc commit 312f2d5
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,8 @@ for current_test in ${TESTS:-$ALL_TESTS}; do
else
log_test "'$current_test' [$profile] overflow $target"
fi
RET_FIN=$(( RET_FIN || RET ))
done
RET_FIN=$(( RET_FIN || RET ))
done
done
current_test=""
Expand Down
3 changes: 2 additions & 1 deletion tools/testing/selftests/drivers/net/mlxsw/tc_police_scale.sh
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,8 @@ __tc_police_test()

tc_police_rules_create $count $should_fail

offload_count=$(tc filter show dev $swp1 ingress | grep in_hw | wc -l)
offload_count=$(tc -j filter show dev $swp1 ingress |
jq "[.[] | select(.options.in_hw == true)] | length")
((offload_count == count))
check_err_fail $should_fail $? "tc police offload count"
}
Expand Down

0 comments on commit 312f2d5

Please sign in to comment.