diff mbox series

[net-next] test: hsr: Extend the hsr_redbox.sh to have more SAN devices connected

Message ID 20240510143710.3916631-1-lukma@denx.de (mailing list archive)
State Accepted
Commit eafbf0574e05160c9a256666261d04a1bc59fa71
Delegated to: Netdev Maintainers
Headers show
Series [net-next] test: hsr: Extend the hsr_redbox.sh to have more SAN devices connected | expand

Checks

Context Check Description
netdev/series_format success Single patches do not need cover letters
netdev/tree_selection success Clearly marked for net-next
netdev/ynl success Generated files up to date; no warnings/errors; no diff in generated;
netdev/fixes_present success Fixes tag not required for -next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 8 this patch: 8
netdev/build_tools success Errors and warnings before: 0 this patch: 0
netdev/cc_maintainers success CCed 7 of 7 maintainers
netdev/build_clang success Errors and warnings before: 8 this patch: 8
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success net selftest script(s) already in Makefile
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 8 this patch: 8
netdev/checkpatch warning WARNING: line length of 85 exceeds 80 columns
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0
netdev/contest success net-next-2024-05-13--18-00 (tests: 1019)

Commit Message

Lukasz Majewski May 10, 2024, 2:37 p.m. UTC
After this change the single SAN device (ns3eth1) is now replaced with
two SAN devices - respectively ns4eth1 and ns5eth1.

It is possible to extend this script to have more SAN devices connected
by adding them to ns3br1 bridge.

Signed-off-by: Lukasz Majewski <lukma@denx.de>
---
 tools/testing/selftests/net/hsr/hsr_redbox.sh | 71 +++++++++++++------
 1 file changed, 49 insertions(+), 22 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org May 13, 2024, 11:10 p.m. UTC | #1
Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Fri, 10 May 2024 16:37:10 +0200 you wrote:
> After this change the single SAN device (ns3eth1) is now replaced with
> two SAN devices - respectively ns4eth1 and ns5eth1.
> 
> It is possible to extend this script to have more SAN devices connected
> by adding them to ns3br1 bridge.
> 
> Signed-off-by: Lukasz Majewski <lukma@denx.de>
> 
> [...]

Here is the summary with links:
  - [net-next] test: hsr: Extend the hsr_redbox.sh to have more SAN devices connected
    https://git.kernel.org/netdev/net-next/c/eafbf0574e05

You are awesome, thank you!
diff mbox series

Patch

diff --git a/tools/testing/selftests/net/hsr/hsr_redbox.sh b/tools/testing/selftests/net/hsr/hsr_redbox.sh
index db69be95ecb3..1f36785347c0 100755
--- a/tools/testing/selftests/net/hsr/hsr_redbox.sh
+++ b/tools/testing/selftests/net/hsr/hsr_redbox.sh
@@ -8,12 +8,19 @@  source ./hsr_common.sh
 do_complete_ping_test()
 {
 	echo "INFO: Initial validation ping (HSR-SAN/RedBox)."
-	# Each node has to be able each one.
+	# Each node has to be able to reach each one.
 	do_ping "${ns1}" 100.64.0.2
 	do_ping "${ns2}" 100.64.0.1
-	# Ping from SAN to hsr1 (via hsr2)
+	# Ping between SANs (test bridge)
+	do_ping "${ns4}" 100.64.0.51
+	do_ping "${ns5}" 100.64.0.41
+	# Ping from SANs to hsr1 (via hsr2) (and opposite)
 	do_ping "${ns3}" 100.64.0.1
 	do_ping "${ns1}" 100.64.0.3
+	do_ping "${ns1}" 100.64.0.41
+	do_ping "${ns4}" 100.64.0.1
+	do_ping "${ns1}" 100.64.0.51
+	do_ping "${ns5}" 100.64.0.1
 	stop_if_error "Initial validation failed."
 
 	# Wait for MGNT HSR frames being received and nodes being
@@ -23,8 +30,12 @@  do_complete_ping_test()
 	echo "INFO: Longer ping test (HSR-SAN/RedBox)."
 	# Ping from SAN to hsr1 (via hsr2)
 	do_ping_long "${ns3}" 100.64.0.1
-	# Ping from hsr1 (via hsr2) to SAN
+	# Ping from hsr1 (via hsr2) to SANs (and opposite)
 	do_ping_long "${ns1}" 100.64.0.3
+	do_ping_long "${ns1}" 100.64.0.41
+	do_ping_long "${ns4}" 100.64.0.1
+	do_ping_long "${ns1}" 100.64.0.51
+	do_ping_long "${ns5}" 100.64.0.1
 	stop_if_error "Longer ping test failed."
 
 	echo "INFO: All good."
@@ -35,22 +46,26 @@  setup_hsr_interfaces()
 	local HSRv="$1"
 
 	echo "INFO: preparing interfaces for HSRv${HSRv} (HSR-SAN/RedBox)."
-
-#       |NS1                     |
-#       |                        |
-#       |    /-- hsr1 --\        |
-#       | ns1eth1     ns1eth2    |
-#       |------------------------|
-#            |            |
-#            |            |
-#            |            |
-#       |------------------------|        |-----------|
-#       | ns2eth1     ns2eth2    |        |           |
-#       |    \-- hsr2 --/        |        |           |
-#       |            \           |        |           |
-#       |             ns2eth3    |--------| ns3eth1   |
-#       |             (interlink)|        |           |
-#       |NS2 (RedBOX)            |        |NS3 (SAN)  |
+#
+# IPv4 addresses (100.64.X.Y/24), and [X.Y] is presented on below diagram:
+#
+#
+# |NS1                     |               |NS4                |
+# |       [0.1]            |               |                   |
+# |    /-- hsr1 --\        |               |    [0.41]         |
+# | ns1eth1     ns1eth2    |               |    ns4eth1 (SAN)  |
+# |------------------------|               |-------------------|
+#      |            |                                |
+#      |            |                                |
+#      |            |                                |
+# |------------------------|   |-------------------------------|
+# | ns2eth1     ns2eth2    |   |                  ns3eth2      |
+# |    \-- hsr2 --/        |   |                 /             |
+# |      [0.2] \           |   |                /              |  |------------|
+# |             ns2eth3    |---| ns3eth1 -- ns3br1 -- ns3eth3--|--| ns5eth1    |
+# |             (interlink)|   | [0.3]      [0.11]             |  | [0.51]     |
+# |NS2 (RedBOX)            |   |NS3 (BR)                       |  | NS5 (SAN)  |
+#
 #
 	# Check if iproute2 supports adding interlink port to hsrX device
 	ip link help hsr | grep -q INTERLINK
@@ -59,7 +74,9 @@  setup_hsr_interfaces()
 	# Create interfaces for name spaces
 	ip link add ns1eth1 netns "${ns1}" type veth peer name ns2eth1 netns "${ns2}"
 	ip link add ns1eth2 netns "${ns1}" type veth peer name ns2eth2 netns "${ns2}"
-	ip link add ns3eth1 netns "${ns3}" type veth peer name ns2eth3 netns "${ns2}"
+	ip link add ns2eth3 netns "${ns2}" type veth peer name ns3eth1 netns "${ns3}"
+	ip link add ns3eth2 netns "${ns3}" type veth peer name ns4eth1 netns "${ns4}"
+	ip link add ns3eth3 netns "${ns3}" type veth peer name ns5eth1 netns "${ns5}"
 
 	sleep 1
 
@@ -70,21 +87,31 @@  setup_hsr_interfaces()
 	ip -n "${ns2}" link set ns2eth2 up
 	ip -n "${ns2}" link set ns2eth3 up
 
-	ip -n "${ns3}" link set ns3eth1 up
+	ip -n "${ns3}" link add name ns3br1 type bridge
+	ip -n "${ns3}" link set ns3br1 up
+	ip -n "${ns3}" link set ns3eth1 master ns3br1 up
+	ip -n "${ns3}" link set ns3eth2 master ns3br1 up
+	ip -n "${ns3}" link set ns3eth3 master ns3br1 up
+
+	ip -n "${ns4}" link set ns4eth1 up
+	ip -n "${ns5}" link set ns5eth1 up
 
 	ip -net "${ns1}" link add name hsr1 type hsr slave1 ns1eth1 slave2 ns1eth2 supervision 45 version ${HSRv} proto 0
 	ip -net "${ns2}" link add name hsr2 type hsr slave1 ns2eth1 slave2 ns2eth2 interlink ns2eth3 supervision 45 version ${HSRv} proto 0
 
 	ip -n "${ns1}" addr add 100.64.0.1/24 dev hsr1
 	ip -n "${ns2}" addr add 100.64.0.2/24 dev hsr2
+	ip -n "${ns3}" addr add 100.64.0.11/24 dev ns3br1
 	ip -n "${ns3}" addr add 100.64.0.3/24 dev ns3eth1
+	ip -n "${ns4}" addr add 100.64.0.41/24 dev ns4eth1
+	ip -n "${ns5}" addr add 100.64.0.51/24 dev ns5eth1
 
 	ip -n "${ns1}" link set hsr1 up
 	ip -n "${ns2}" link set hsr2 up
 }
 
 check_prerequisites
-setup_ns ns1 ns2 ns3
+setup_ns ns1 ns2 ns3 ns4 ns5
 
 trap cleanup_all_ns EXIT