[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