diff options
| author | Ido Schimmel <idosch@mellanox.com> | 2020-05-17 01:43:10 +0300 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2020-05-17 02:42:31 +0300 |
| commit | 04cc99d9bdb1119172e21c121950a0253f5c659f (patch) | |
| tree | ae1eb050581f285eb24b977b617f397da2992b1b /tools/testing/selftests/net/forwarding/devlink_lib.sh | |
| parent | 84e0d83567df4597b1b624b495d689104227a551 (diff) | |
| download | linux-04cc99d9bdb1119172e21c121950a0253f5c659f.tar.xz | |
selftests: mlxsw: Do not hard code trap group name
It can be derived dynamically from the trap's name, so drop it.
Signed-off-by: Ido Schimmel <idosch@mellanox.com>
Reviewed-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools/testing/selftests/net/forwarding/devlink_lib.sh')
| -rw-r--r-- | tools/testing/selftests/net/forwarding/devlink_lib.sh | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/tools/testing/selftests/net/forwarding/devlink_lib.sh b/tools/testing/selftests/net/forwarding/devlink_lib.sh index 7b6390aea50b..e27236109235 100644 --- a/tools/testing/selftests/net/forwarding/devlink_lib.sh +++ b/tools/testing/selftests/net/forwarding/devlink_lib.sh @@ -365,7 +365,9 @@ devlink_trap_group_stats_idle_test() devlink_trap_exception_test() { local trap_name=$1; shift - local group_name=$1; shift + local group_name + + group_name=$(devlink_trap_group_get $trap_name) devlink_trap_stats_idle_test $trap_name check_fail $? "Trap stats idle when packets should have been trapped" @@ -377,9 +379,11 @@ devlink_trap_exception_test() devlink_trap_drop_test() { local trap_name=$1; shift - local group_name=$1; shift local dev=$1; shift local handle=$1; shift + local group_name + + group_name=$(devlink_trap_group_get $trap_name) # This is the common part of all the tests. It checks that stats are # initially idle, then non-idle after changing the trap action and |