selftests: devlink: undo changes at the end of resource_test
authorJakub Kicinski <jakub.kicinski@netronome.com>
Tue, 5 Nov 2019 21:28:17 +0000 (13:28 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 6 Nov 2019 19:11:30 +0000 (11:11 -0800)
The netdevsim object is reused by all the tests, but the resource
tests puts it into a broken state (failed reload in a different
namespace). Make sure it's fixed up at the end of that test
otherwise subsequent tests fail.

Fixes: b74c37fd35a2 ("selftests: netdevsim: add tests for devlink reload with resources")
Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Acked-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/drivers/net/netdevsim/devlink.sh

index ee89cd2f5beeec9768246e0d1d6d2ad9765a1b69..753c5b6abe0ae029305218538db0d736636b661c 100755 (executable)
@@ -279,6 +279,12 @@ resource_test()
        devlink -N testns1 dev reload $DL_HANDLE netns testns2
        check_fail $? "Unexpected successful reload from netns \"testns1\" into netns \"testns2\""
 
+       devlink -N testns2 resource set $DL_HANDLE path IPv4/fib size ' -1'
+       check_err $? "Failed to reset IPv4/fib resource size"
+
+       devlink -N testns2 dev reload $DL_HANDLE netns 1
+       check_err $? "Failed to reload devlink back"
+
        ip netns del testns2
        ip netns del testns1