selftests: xsk: cleanup bash scripts
authorMagnus Karlsson <magnus.karlsson@intel.com>
Tue, 10 May 2022 11:55:56 +0000 (13:55 +0200)
committerAlexei Starovoitov <ast@kernel.org>
Wed, 11 May 2022 15:03:15 +0000 (08:03 -0700)
Remove the spec-file that is not used any longer from the shell
scripts. Also remove an unused option.

Signed-off-by: Magnus Karlsson <magnus.karlsson@intel.com>
Link: https://lore.kernel.org/r/20220510115604.8717-2-magnus.karlsson@gmail.com
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
tools/testing/selftests/bpf/test_xsk.sh
tools/testing/selftests/bpf/xsk_prereqs.sh

index cd7bf32..2bd12c1 100755 (executable)
@@ -43,7 +43,6 @@
 #   ** veth<xxxx> in root namespace
 #   ** veth<yyyy> in af_xdp<xxxx> namespace
 #   ** namespace af_xdp<xxxx>
-#   * create a spec file veth.spec that includes this run-time configuration
 #   *** xxxx and yyyy are randomly generated 4 digit numbers used to avoid
 #       conflict with any existing interface
 #   * tests the veth and xsk layers of the topology
@@ -77,7 +76,7 @@
 
 . xsk_prereqs.sh
 
-while getopts "cvD" flag
+while getopts "vD" flag
 do
        case "${flag}" in
                v) verbose=1;;
@@ -130,12 +129,7 @@ if [ $retval -ne 0 ]; then
        exit $retval
 fi
 
-echo "${VETH0}:${VETH1},${NS1}" > ${SPECFILE}
-
-validate_veth_spec_file
-
 if [[ $verbose -eq 1 ]]; then
-        echo "Spec file created: ${SPECFILE}"
        VERBOSE_ARG="-v"
 fi
 
index bf29d25..7606d59 100755 (executable)
@@ -8,7 +8,6 @@ ksft_xfail=2
 ksft_xpass=3
 ksft_skip=4
 
-SPECFILE=veth.spec
 XSKOBJ=xdpxceiver
 
 validate_root_exec()
@@ -34,13 +33,6 @@ validate_veth_support()
        fi
 }
 
-validate_veth_spec_file()
-{
-       if [ ! -f ${SPECFILE} ]; then
-               test_exit $ksft_skip 1
-       fi
-}
-
 test_status()
 {
        statusval=$1
@@ -74,9 +66,6 @@ clear_configs()
        #veth node inside NS won't get removed so we explicitly remove it
        [ $(ip link show $1 &>/dev/null; echo $?;) == 0 ] &&
                { ip link del $1; }
-       if [ -f ${SPECFILE} ]; then
-               rm -f ${SPECFILE}
-       fi
 }
 
 cleanup_exit()