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 <[email protected]>

Reviewed-by: Josh Triplett <[email protected]>

>  .../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
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to