[PATCH] tests: vm: simplify parameter passing

Johannes Berg johannes at sipsolutions.net
Tue Jul 12 14:44:19 PDT 2022


From: Johannes Berg <johannes.berg at intel.com>

Since the kernel  actually passes the command-line parameters
as environment variables to the init script, there's no need
to parse them out of /proc/cmdline.

Signed-off-by: Johannes Berg <johannes.berg at intel.com>
---
 tests/hwsim/vm/inside.sh | 13 -------------
 tests/hwsim/vm/vm-run.sh |  6 +++---
 2 files changed, 3 insertions(+), 16 deletions(-)

diff --git a/tests/hwsim/vm/inside.sh b/tests/hwsim/vm/inside.sh
index 9c6a9572b8d8..d813cf065f74 100755
--- a/tests/hwsim/vm/inside.sh
+++ b/tests/hwsim/vm/inside.sh
@@ -30,19 +30,6 @@ export HOME=/tmp
 sysctl kernel.panic_on_oops=1
 sysctl kernel.panic=1
 
-# get extra command line variables from /proc/cmdline
-TESTDIR=$(sed 's/.*testdir=\([^ ]*\) .*/\1/' /proc/cmdline)
-TIMEWARP=$(sed 's/.*timewarp=\([^ ]*\) .*/\1/' /proc/cmdline)
-EPATH=$(sed 's/.*EPATH=\([^ ]*\) .*/\1/' /proc/cmdline)
-TELNET=$(sed 's/.*TELNET=\([^ ]*\) .*/\1/' /proc/cmdline)
-ARGS=$(sed 's/.*ARGS=\([^ ]*\)\( \|$\).*/\1/' /proc/cmdline)
-LOGDIR=$(sed 's/.*LOGDIR=\([^ ]*\)\( \|$\).*/\1/' /proc/cmdline)
-if grep -q "commitid=" /proc/cmdline; then
-    COMMITID=$(sed 's/.*commitid=\([^ ]*\)\( \|$\).*/\1/' /proc/cmdline)
-else
-    COMMITID=
-fi
-
 mount --bind "$TESTDIR/vm/regdb/" /lib/firmware
 
 if [ "$MODULEDIR" != "" ] ; then
diff --git a/tests/hwsim/vm/vm-run.sh b/tests/hwsim/vm/vm-run.sh
index b79e1d8f890b..e891676a4dfd 100755
--- a/tests/hwsim/vm/vm-run.sh
+++ b/tests/hwsim/vm/vm-run.sh
@@ -152,8 +152,8 @@ A+="cfg80211.dyndbg=+p "
 A+="mac80211.dyndbg=+p "
 A+="mac80211_hwsim.dyndbg=+p "
 A+="init=$CMD "
-A+="testdir=$TESTDIR "
-A+="timewarp=$TIMEWARP "
+A+="TESTDIR=$TESTDIR "
+A+="TIMEWARP=$TIMEWARP "
 A+="MODULEDIR=$MODULEDIR "
 A+="TELNET=$TELNET_ARG "
 A+="EPATH=$EPATH "
@@ -161,7 +161,7 @@ A+="ARGS=$argsfile "
 A+="console=$KVMOUT "
 COMMITID="$(git rev-parse HEAD)"
 if [ -n "$COMMITID" ]; then
-    A+="commitid=$COMMITID "
+    A+="COMMITID=$COMMITID "
 fi
 A+="ro"
 
-- 
2.36.1




More information about the Hostap mailing list