* [PATCH] rcutorture: Remove obsolete kversion param in kvm.sh
@ 2014-10-04 17:04 Pranith Kumar
2014-10-04 19:02 ` Josh Triplett
0 siblings, 1 reply; 3+ messages in thread
From: Pranith Kumar @ 2014-10-04 17:04 UTC (permalink / raw)
To: Paul E. McKenney, Josh Triplett, Steven Rostedt,
Mathieu Desnoyers, Lai Jiangshan, open list
Now that we have removed configs based on kernel version, we can also remove the
kversion parameter in kvm.sh.
Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
---
.../selftests/rcutorture/bin/kvm-test-1-run.sh | 2 +-
tools/testing/selftests/rcutorture/bin/kvm.sh | 19 +++++--------------
2 files changed, 6 insertions(+), 15 deletions(-)
diff --git a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
index e0f8969..8ca9f21 100755
--- a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
+++ b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
@@ -45,7 +45,7 @@ trap 'rm -rf $T' 0
touch $T
. $KVM/bin/functions.sh
-. $KVPATH/ver_functions.sh
+. $CONFIGFRAG/ver_functions.sh
config_template=${1}
config_dir=`echo $config_template | sed -e 's,/[^/]*$,,'`
diff --git a/tools/testing/selftests/rcutorture/bin/kvm.sh b/tools/testing/selftests/rcutorture/bin/kvm.sh
index e527dc9..368d64a 100755
--- a/tools/testing/selftests/rcutorture/bin/kvm.sh
+++ b/tools/testing/selftests/rcutorture/bin/kvm.sh
@@ -47,7 +47,6 @@ resdir=""
configs=""
cpus=0
ds=`date +%Y.%m.%d-%H:%M:%S`
-kversion=""
. functions.sh
@@ -64,7 +63,6 @@ usage () {
echo " --duration minutes"
echo " --interactive"
echo " --kmake-arg kernel-make-arguments"
- echo " --kversion vN.NN"
echo " --mac nn:nn:nn:nn:nn:nn"
echo " --no-initrd"
echo " --qemu-args qemu-system-..."
@@ -128,11 +126,6 @@ do
TORTURE_KMAKE_ARG="$2"
shift
;;
- --kversion)
- checkarg --kversion "(kernel version)" $# "$2" '^v[0-9.]*$' '^error'
- kversion=$2
- shift
- ;;
--mac)
checkarg --mac "(MAC address)" $# "$2" '^\([0-9a-fA-F]\{2\}:\)\{5\}[0-9a-fA-F]\{2\}$' error
TORTURE_QEMU_MAC=$2
@@ -170,11 +163,10 @@ do
done
CONFIGFRAG=${KVM}/configs/${TORTURE_SUITE}; export CONFIGFRAG
-KVPATH=${CONFIGFRAG}/$kversion; export KVPATH
if test -z "$configs"
then
- configs="`cat $CONFIGFRAG/$kversion/CFLIST`"
+ configs="`cat $CONFIGFRAG/CFLIST`"
fi
if test -z "$resdir"
@@ -186,10 +178,10 @@ fi
touch $T/cfgcpu
for CF in $configs
do
- if test -f "$CONFIGFRAG/$kversion/$CF"
+ if test -f "$CONFIGFRAG/$CF"
then
- cpu_count=`configNR_CPUS.sh $CONFIGFRAG/$kversion/$CF`
- cpu_count=`configfrag_boot_cpus "$TORTURE_BOOTARGS" "$CONFIGFRAG/$kversion/$CF" "$cpu_count"`
+ cpu_count=`configNR_CPUS.sh $CONFIGFRAG/$CF`
+ cpu_count=`configfrag_boot_cpus "$TORTURE_BOOTARGS" "$CONFIGFRAG/$CF" "$cpu_count"`
echo $CF $cpu_count >> $T/cfgcpu
else
echo "The --configs file $CF does not exist, terminating."
@@ -252,7 +244,6 @@ END {
cat << ___EOF___ > $T/script
CONFIGFRAG="$CONFIGFRAG"; export CONFIGFRAG
KVM="$KVM"; export KVM
-KVPATH="$KVPATH"; export KVPATH
PATH="$PATH"; export PATH
TORTURE_BOOT_IMAGE="$TORTURE_BOOT_IMAGE"; export TORTURE_BOOT_IMAGE
TORTURE_BUILDONLY="$TORTURE_BUILDONLY"; export TORTURE_BUILDONLY
@@ -285,7 +276,7 @@ then
fi
___EOF___
awk < $T/cfgcpu.pack \
- -v CONFIGDIR="$CONFIGFRAG/$kversion/" \
+ -v CONFIGDIR="$CONFIGFRAG/" \
-v KVM="$KVM" \
-v ncpus=$cpus \
-v rd=$resdir/$ds/ \
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread* Re: [PATCH] rcutorture: Remove obsolete kversion param in kvm.sh
2014-10-04 17:04 [PATCH] rcutorture: Remove obsolete kversion param in kvm.sh Pranith Kumar
@ 2014-10-04 19:02 ` Josh Triplett
2014-10-05 8:51 ` Paul E. McKenney
0 siblings, 1 reply; 3+ messages in thread
From: Josh Triplett @ 2014-10-04 19:02 UTC (permalink / raw)
To: Pranith Kumar
Cc: Paul E. McKenney, Steven Rostedt, Mathieu Desnoyers,
Lai Jiangshan, open list
On Sat, Oct 04, 2014 at 01:04:15PM -0400, Pranith Kumar wrote:
> Now that we have removed configs based on kernel version, we can also remove the
> kversion parameter in kvm.sh.
>
> Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
> .../selftests/rcutorture/bin/kvm-test-1-run.sh | 2 +-
> tools/testing/selftests/rcutorture/bin/kvm.sh | 19 +++++--------------
> 2 files changed, 6 insertions(+), 15 deletions(-)
>
> diff --git a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> index e0f8969..8ca9f21 100755
> --- a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> +++ b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> @@ -45,7 +45,7 @@ trap 'rm -rf $T' 0
> touch $T
>
> . $KVM/bin/functions.sh
> -. $KVPATH/ver_functions.sh
> +. $CONFIGFRAG/ver_functions.sh
>
> config_template=${1}
> config_dir=`echo $config_template | sed -e 's,/[^/]*$,,'`
> diff --git a/tools/testing/selftests/rcutorture/bin/kvm.sh b/tools/testing/selftests/rcutorture/bin/kvm.sh
> index e527dc9..368d64a 100755
> --- a/tools/testing/selftests/rcutorture/bin/kvm.sh
> +++ b/tools/testing/selftests/rcutorture/bin/kvm.sh
> @@ -47,7 +47,6 @@ resdir=""
> configs=""
> cpus=0
> ds=`date +%Y.%m.%d-%H:%M:%S`
> -kversion=""
>
> . functions.sh
>
> @@ -64,7 +63,6 @@ usage () {
> echo " --duration minutes"
> echo " --interactive"
> echo " --kmake-arg kernel-make-arguments"
> - echo " --kversion vN.NN"
> echo " --mac nn:nn:nn:nn:nn:nn"
> echo " --no-initrd"
> echo " --qemu-args qemu-system-..."
> @@ -128,11 +126,6 @@ do
> TORTURE_KMAKE_ARG="$2"
> shift
> ;;
> - --kversion)
> - checkarg --kversion "(kernel version)" $# "$2" '^v[0-9.]*$' '^error'
> - kversion=$2
> - shift
> - ;;
> --mac)
> checkarg --mac "(MAC address)" $# "$2" '^\([0-9a-fA-F]\{2\}:\)\{5\}[0-9a-fA-F]\{2\}$' error
> TORTURE_QEMU_MAC=$2
> @@ -170,11 +163,10 @@ do
> done
>
> CONFIGFRAG=${KVM}/configs/${TORTURE_SUITE}; export CONFIGFRAG
> -KVPATH=${CONFIGFRAG}/$kversion; export KVPATH
>
> if test -z "$configs"
> then
> - configs="`cat $CONFIGFRAG/$kversion/CFLIST`"
> + configs="`cat $CONFIGFRAG/CFLIST`"
> fi
>
> if test -z "$resdir"
> @@ -186,10 +178,10 @@ fi
> touch $T/cfgcpu
> for CF in $configs
> do
> - if test -f "$CONFIGFRAG/$kversion/$CF"
> + if test -f "$CONFIGFRAG/$CF"
> then
> - cpu_count=`configNR_CPUS.sh $CONFIGFRAG/$kversion/$CF`
> - cpu_count=`configfrag_boot_cpus "$TORTURE_BOOTARGS" "$CONFIGFRAG/$kversion/$CF" "$cpu_count"`
> + cpu_count=`configNR_CPUS.sh $CONFIGFRAG/$CF`
> + cpu_count=`configfrag_boot_cpus "$TORTURE_BOOTARGS" "$CONFIGFRAG/$CF" "$cpu_count"`
> echo $CF $cpu_count >> $T/cfgcpu
> else
> echo "The --configs file $CF does not exist, terminating."
> @@ -252,7 +244,6 @@ END {
> cat << ___EOF___ > $T/script
> CONFIGFRAG="$CONFIGFRAG"; export CONFIGFRAG
> KVM="$KVM"; export KVM
> -KVPATH="$KVPATH"; export KVPATH
> PATH="$PATH"; export PATH
> TORTURE_BOOT_IMAGE="$TORTURE_BOOT_IMAGE"; export TORTURE_BOOT_IMAGE
> TORTURE_BUILDONLY="$TORTURE_BUILDONLY"; export TORTURE_BUILDONLY
> @@ -285,7 +276,7 @@ then
> fi
> ___EOF___
> awk < $T/cfgcpu.pack \
> - -v CONFIGDIR="$CONFIGFRAG/$kversion/" \
> + -v CONFIGDIR="$CONFIGFRAG/" \
> -v KVM="$KVM" \
> -v ncpus=$cpus \
> -v rd=$resdir/$ds/ \
> --
> 1.9.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread* Re: [PATCH] rcutorture: Remove obsolete kversion param in kvm.sh
2014-10-04 19:02 ` Josh Triplett
@ 2014-10-05 8:51 ` Paul E. McKenney
0 siblings, 0 replies; 3+ messages in thread
From: Paul E. McKenney @ 2014-10-05 8:51 UTC (permalink / raw)
To: Josh Triplett
Cc: Pranith Kumar, Steven Rostedt, Mathieu Desnoyers, Lai Jiangshan,
open list
On Sat, Oct 04, 2014 at 12:02:37PM -0700, Josh Triplett wrote:
> On Sat, Oct 04, 2014 at 01:04:15PM -0400, Pranith Kumar wrote:
> > Now that we have removed configs based on kernel version, we can also remove the
> > kversion parameter in kvm.sh.
> >
> > Signed-off-by: Pranith Kumar <bobby.prani@gmail.com>
>
> Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Queued, thank you both!
Thanx, Paul
> > .../selftests/rcutorture/bin/kvm-test-1-run.sh | 2 +-
> > tools/testing/selftests/rcutorture/bin/kvm.sh | 19 +++++--------------
> > 2 files changed, 6 insertions(+), 15 deletions(-)
> >
> > diff --git a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> > index e0f8969..8ca9f21 100755
> > --- a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> > +++ b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> > @@ -45,7 +45,7 @@ trap 'rm -rf $T' 0
> > touch $T
> >
> > . $KVM/bin/functions.sh
> > -. $KVPATH/ver_functions.sh
> > +. $CONFIGFRAG/ver_functions.sh
> >
> > config_template=${1}
> > config_dir=`echo $config_template | sed -e 's,/[^/]*$,,'`
> > diff --git a/tools/testing/selftests/rcutorture/bin/kvm.sh b/tools/testing/selftests/rcutorture/bin/kvm.sh
> > index e527dc9..368d64a 100755
> > --- a/tools/testing/selftests/rcutorture/bin/kvm.sh
> > +++ b/tools/testing/selftests/rcutorture/bin/kvm.sh
> > @@ -47,7 +47,6 @@ resdir=""
> > configs=""
> > cpus=0
> > ds=`date +%Y.%m.%d-%H:%M:%S`
> > -kversion=""
> >
> > . functions.sh
> >
> > @@ -64,7 +63,6 @@ usage () {
> > echo " --duration minutes"
> > echo " --interactive"
> > echo " --kmake-arg kernel-make-arguments"
> > - echo " --kversion vN.NN"
> > echo " --mac nn:nn:nn:nn:nn:nn"
> > echo " --no-initrd"
> > echo " --qemu-args qemu-system-..."
> > @@ -128,11 +126,6 @@ do
> > TORTURE_KMAKE_ARG="$2"
> > shift
> > ;;
> > - --kversion)
> > - checkarg --kversion "(kernel version)" $# "$2" '^v[0-9.]*$' '^error'
> > - kversion=$2
> > - shift
> > - ;;
> > --mac)
> > checkarg --mac "(MAC address)" $# "$2" '^\([0-9a-fA-F]\{2\}:\)\{5\}[0-9a-fA-F]\{2\}$' error
> > TORTURE_QEMU_MAC=$2
> > @@ -170,11 +163,10 @@ do
> > done
> >
> > CONFIGFRAG=${KVM}/configs/${TORTURE_SUITE}; export CONFIGFRAG
> > -KVPATH=${CONFIGFRAG}/$kversion; export KVPATH
> >
> > if test -z "$configs"
> > then
> > - configs="`cat $CONFIGFRAG/$kversion/CFLIST`"
> > + configs="`cat $CONFIGFRAG/CFLIST`"
> > fi
> >
> > if test -z "$resdir"
> > @@ -186,10 +178,10 @@ fi
> > touch $T/cfgcpu
> > for CF in $configs
> > do
> > - if test -f "$CONFIGFRAG/$kversion/$CF"
> > + if test -f "$CONFIGFRAG/$CF"
> > then
> > - cpu_count=`configNR_CPUS.sh $CONFIGFRAG/$kversion/$CF`
> > - cpu_count=`configfrag_boot_cpus "$TORTURE_BOOTARGS" "$CONFIGFRAG/$kversion/$CF" "$cpu_count"`
> > + cpu_count=`configNR_CPUS.sh $CONFIGFRAG/$CF`
> > + cpu_count=`configfrag_boot_cpus "$TORTURE_BOOTARGS" "$CONFIGFRAG/$CF" "$cpu_count"`
> > echo $CF $cpu_count >> $T/cfgcpu
> > else
> > echo "The --configs file $CF does not exist, terminating."
> > @@ -252,7 +244,6 @@ END {
> > cat << ___EOF___ > $T/script
> > CONFIGFRAG="$CONFIGFRAG"; export CONFIGFRAG
> > KVM="$KVM"; export KVM
> > -KVPATH="$KVPATH"; export KVPATH
> > PATH="$PATH"; export PATH
> > TORTURE_BOOT_IMAGE="$TORTURE_BOOT_IMAGE"; export TORTURE_BOOT_IMAGE
> > TORTURE_BUILDONLY="$TORTURE_BUILDONLY"; export TORTURE_BUILDONLY
> > @@ -285,7 +276,7 @@ then
> > fi
> > ___EOF___
> > awk < $T/cfgcpu.pack \
> > - -v CONFIGDIR="$CONFIGFRAG/$kversion/" \
> > + -v CONFIGDIR="$CONFIGFRAG/" \
> > -v KVM="$KVM" \
> > -v ncpus=$cpus \
> > -v rd=$resdir/$ds/ \
> > --
> > 1.9.1
> >
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-10-05 8:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-10-04 17:04 [PATCH] rcutorture: Remove obsolete kversion param in kvm.sh Pranith Kumar
2014-10-04 19:02 ` Josh Triplett
2014-10-05 8:51 ` Paul E. McKenney
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.