Commit d1c4a346 authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller

selftests: devlink: add a warning for interfaces coming up

NetworkManager (and other daemons) may bring the interface up
and cause failures in quiescence checks. Print a helpful warning,
and take the interface down again.

I seem to forget about this every time I run these tests on a new VM.
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 2fc60e2f
...@@ -47,6 +47,17 @@ if [ -d "${NETDEVSIM_PATH}/devices/netdevsim${DEV_ADDR}" ]; then ...@@ -47,6 +47,17 @@ if [ -d "${NETDEVSIM_PATH}/devices/netdevsim${DEV_ADDR}" ]; then
exit 1 exit 1
fi fi
check_netdev_down()
{
state=$(cat /sys/class/net/${NETDEV}/flags)
if [ $((state & 1)) -ne 0 ]; then
echo "WARNING: unexpected interface UP, disable NetworkManager?"
ip link set dev $NETDEV down
fi
}
init_test() init_test()
{ {
RET=0 RET=0
...@@ -151,6 +162,7 @@ trap_stats_test() ...@@ -151,6 +162,7 @@ trap_stats_test()
RET=0 RET=0
check_netdev_down
for trap_name in $(devlink_traps_get); do for trap_name in $(devlink_traps_get); do
devlink_trap_stats_idle_test $trap_name devlink_trap_stats_idle_test $trap_name
check_err $? "Stats of trap $trap_name not idle when netdev down" check_err $? "Stats of trap $trap_name not idle when netdev down"
...@@ -254,6 +266,7 @@ trap_group_stats_test() ...@@ -254,6 +266,7 @@ trap_group_stats_test()
RET=0 RET=0
check_netdev_down
for group_name in $(devlink_trap_groups_get); do for group_name in $(devlink_trap_groups_get); do
devlink_trap_group_stats_idle_test $group_name devlink_trap_group_stats_idle_test $group_name
check_err $? "Stats of trap group $group_name not idle when netdev down" check_err $? "Stats of trap group $group_name not idle when netdev down"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment