]> git.dujemihanovic.xyz Git - linux.git/commitdiff
selftests: net: local_termination: parameterize test name
authorVladimir Oltean <vladimir.oltean@nxp.com>
Thu, 15 Aug 2024 00:06:56 +0000 (03:06 +0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 16 Aug 2024 08:59:31 +0000 (09:59 +0100)
There are upcoming tests which verify the RX filtering of a bridge
(or bridge port), but under differing vlan_filtering conditions.
Since we currently print $h2 (the DUT) in the log_test() output, it
becomes necessary to make a further distinction between tests, to not
give the user the impression that the exact same thing is run twice.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/forwarding/local_termination.sh

index 92f0e242d119da68a751c87d402533d032ea684f..af284edaf4014f0bd0485619ea920c842e437fc5 100755 (executable)
@@ -68,10 +68,11 @@ send_uc_ipv4()
 
 check_rcv()
 {
-       local if_name=$1
-       local type=$2
-       local pattern=$3
-       local should_receive=$4
+       local if_name=$1; shift
+       local type=$1; shift
+       local pattern=$1; shift
+       local should_receive=$1; shift
+       local test_name="$1"; shift
        local should_fail=
 
        [ $should_receive = true ] && should_fail=0 || should_fail=1
@@ -81,7 +82,7 @@ check_rcv()
 
        check_err_fail "$should_fail" "$?" "reception"
 
-       log_test "$if_name: $type"
+       log_test "$test_name: $type"
 }
 
 mc_route_prepare()
@@ -106,6 +107,7 @@ run_test()
 {
        local send_if_name=$1; shift
        local rcv_if_name=$1; shift
+       local test_name="$1"; shift
        local smac=$(mac_get $send_if_name)
        local rcv_dmac=$(mac_get $rcv_if_name)
 
@@ -150,61 +152,61 @@ run_test()
 
        check_rcv $rcv_if_name "Unicast IPv4 to primary MAC address" \
                "$smac > $rcv_dmac, ethertype IPv4 (0x0800)" \
-               true
+               true "$test_name"
 
        check_rcv $rcv_if_name "Unicast IPv4 to macvlan MAC address" \
                "$smac > $MACVLAN_ADDR, ethertype IPv4 (0x0800)" \
-               true
+               true "$test_name"
 
        xfail_on_veth $h1 \
                check_rcv $rcv_if_name "Unicast IPv4 to unknown MAC address" \
                        "$smac > $UNKNOWN_UC_ADDR1, ethertype IPv4 (0x0800)" \
-                       false
+                       false "$test_name"
 
        check_rcv $rcv_if_name "Unicast IPv4 to unknown MAC address, promisc" \
                "$smac > $UNKNOWN_UC_ADDR2, ethertype IPv4 (0x0800)" \
-               true
+               true "$test_name"
 
        xfail_on_veth $h1 \
                check_rcv $rcv_if_name \
                        "Unicast IPv4 to unknown MAC address, allmulti" \
                        "$smac > $UNKNOWN_UC_ADDR3, ethertype IPv4 (0x0800)" \
-                       false
+                       false "$test_name"
 
        check_rcv $rcv_if_name "Multicast IPv4 to joined group" \
                "$smac > $JOINED_MACV4_MC_ADDR, ethertype IPv4 (0x0800)" \
-               true
+               true "$test_name"
 
        xfail_on_veth $h1 \
                check_rcv $rcv_if_name \
                        "Multicast IPv4 to unknown group" \
                        "$smac > $UNKNOWN_MACV4_MC_ADDR1, ethertype IPv4 (0x0800)" \
-                       false
+                       false "$test_name"
 
        check_rcv $rcv_if_name "Multicast IPv4 to unknown group, promisc" \
                "$smac > $UNKNOWN_MACV4_MC_ADDR2, ethertype IPv4 (0x0800)" \
-               true
+               true "$test_name"
 
        check_rcv $rcv_if_name "Multicast IPv4 to unknown group, allmulti" \
                "$smac > $UNKNOWN_MACV4_MC_ADDR3, ethertype IPv4 (0x0800)" \
-               true
+               true "$test_name"
 
        check_rcv $rcv_if_name "Multicast IPv6 to joined group" \
                "$smac > $JOINED_MACV6_MC_ADDR, ethertype IPv6 (0x86dd)" \
-               true
+               true "$test_name"
 
        xfail_on_veth $h1 \
                check_rcv $rcv_if_name "Multicast IPv6 to unknown group" \
                        "$smac > $UNKNOWN_MACV6_MC_ADDR1, ethertype IPv6 (0x86dd)" \
-                       false
+                       false "$test_name"
 
        check_rcv $rcv_if_name "Multicast IPv6 to unknown group, promisc" \
                "$smac > $UNKNOWN_MACV6_MC_ADDR2, ethertype IPv6 (0x86dd)" \
-               true
+               true "$test_name"
 
        check_rcv $rcv_if_name "Multicast IPv6 to unknown group, allmulti" \
                "$smac > $UNKNOWN_MACV6_MC_ADDR3, ethertype IPv6 (0x86dd)" \
-               true
+               true "$test_name"
 
        tcpdump_cleanup $rcv_if_name
 }