Skip to content

Commit

Permalink
selftests: mlxsw: resource_scale: Pass target count to cleanup
Browse files Browse the repository at this point in the history
The scale tests are verifying behavior of mlxsw when number of instances of
some resource reaches the ASIC capacity. The number of instances is
referred to as "target" number.

No scale tests so far needed to know this target number to clean up. E.g.
the tc_flower simply removes the clsact qdisc that all the tested filters
are hooked onto, and that takes care of collecting all the filters.

However, for the RIF counter test, which is being added in a future patch,
VLAN netdevices are created. These are created as part of the test, but of
course the cleanup needs to undo them again. For that it needs to know how
many there were. To support this usage, pass the target number to the
cleanup callback.

Signed-off-by: Petr Machata <[email protected]>
Reviewed-by: Amit Cohen <[email protected]>
Signed-off-by: Ido Schimmel <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
  • Loading branch information
pmachata authored and davem330 committed Jun 17, 2022
1 parent 8cad339 commit 35d5829
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ for current_test in ${TESTS:-$ALL_TESTS}; do
else
log_test "'$current_test' overflow $target"
fi
${current_test}_cleanup
${current_test}_cleanup $target
devlink_reload
RET_FIN=$(( RET_FIN || RET ))
done
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ for current_test in ${TESTS:-$ALL_TESTS}; do
else
log_test "'$current_test' [$profile] overflow $target"
fi
${current_test}_cleanup
${current_test}_cleanup $target
RET_FIN=$(( RET_FIN || RET ))
done
done
Expand Down

0 comments on commit 35d5829

Please sign in to comment.