selftests: Extend fib tests to run with and without flags notifications
authorAmit Cohen <amcohen@nvidia.com>
Mon, 1 Feb 2021 19:47:56 +0000 (21:47 +0200)
committerJakub Kicinski <kuba@kernel.org>
Wed, 3 Feb 2021 01:45:59 +0000 (17:45 -0800)
Run the test cases with both `fib_notify_on_flag_change` sysctls set to
'1', and then with both sysctls set to '0' to verify there are no
regressions in the test when notifications are added.

Signed-off-by: Amit Cohen <amcohen@nvidia.com>
Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/drivers/net/mlxsw/fib.sh
tools/testing/selftests/drivers/net/netdevsim/fib.sh

index eab79b9..dcbf32b 100755 (executable)
@@ -225,6 +225,16 @@ ipv6_local_replace()
        ip -n $ns link del dev dummy1
 }
 
+fib_notify_on_flag_change_set()
+{
+       local notify=$1; shift
+
+       ip netns exec testns1 sysctl -qw net.ipv4.fib_notify_on_flag_change=$notify
+       ip netns exec testns1 sysctl -qw net.ipv6.fib_notify_on_flag_change=$notify
+
+       log_info "Set fib_notify_on_flag_change to $notify"
+}
+
 setup_prepare()
 {
        ip netns add testns1
@@ -251,6 +261,10 @@ trap cleanup EXIT
 
 setup_prepare
 
+fib_notify_on_flag_change_set 1
+tests_run
+
+fib_notify_on_flag_change_set 0
 tests_run
 
 exit $EXIT_STATUS
index 2f87c3b..251f228 100755 (executable)
@@ -302,6 +302,16 @@ ipv6_error_path()
        ipv6_error_path_replay
 }
 
+fib_notify_on_flag_change_set()
+{
+       local notify=$1; shift
+
+       ip netns exec testns1 sysctl -qw net.ipv4.fib_notify_on_flag_change=$notify
+       ip netns exec testns1 sysctl -qw net.ipv6.fib_notify_on_flag_change=$notify
+
+       log_info "Set fib_notify_on_flag_change to $notify"
+}
+
 setup_prepare()
 {
        local netdev
@@ -336,6 +346,10 @@ trap cleanup EXIT
 
 setup_prepare
 
+fib_notify_on_flag_change_set 1
+tests_run
+
+fib_notify_on_flag_change_set 0
 tests_run
 
 exit $EXIT_STATUS