On Mon, Jan 09, 2017 at 12:04:53PM +0800, Peter Xu wrote:
> We were using test.log before to keep all the test logs. This patch
> creates one log file per test case under logs/ directory with name
> "TESTNAME.log". Meanwhile, we will keep the last time log into
> logs.old/.
>
> Renaming scripts/functions.bash into scripts/common.bash to store some
> more global variables.
>
> Signed-off-by: Peter Xu <peterx@xxxxxxxxxx>
> ---
> .gitignore | 3 ++-
> Makefile | 4 ++--
> run_tests.sh | 22 +++++++++++++++-------
> scripts/{functions.bash => common.bash} | 13 +++++++++++--
> scripts/mkstandalone.sh | 2 +-
> 5 files changed, 31 insertions(+), 13 deletions(-)
> rename scripts/{functions.bash => common.bash} (75%)
>
> diff --git a/.gitignore b/.gitignore
> index 3155418..0dc9a39 100644
> --- a/.gitignore
> +++ b/.gitignore
> @@ -12,7 +12,8 @@ cscope.*
> /lib/asm
> /config.mak
> /*-run
> -/test.log
> /msr.out
> /tests
> /build-head
> +logs
> +logs.old
Without the leading '/' logs[.old] files or dirs in subdirs will
also be ignored. You can add trailing '/' to all the dirs in
.gitignore too in order to ensure we only ignore dirs.
> diff --git a/Makefile b/Makefile
> index a32333b..bd8843a 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -94,9 +94,9 @@ libfdt_clean:
> $(LIBFDT_objdir)/.*.d
>
> distclean: clean libfdt_clean
> - $(RM) lib/asm config.mak $(TEST_DIR)-run test.log msr.out cscope.* \
> + $(RM) lib/asm config.mak $(TEST_DIR)-run msr.out cscope.* \
> build-head
Can move build-head up now
> - $(RM) -r tests
> + $(RM) -r tests logs logs.old
>
> cscope: cscope_dirs = lib lib/libfdt lib/linux $(TEST_DIR) $(ARCH_LIBDIRS) lib/asm-generic
> cscope:
> diff --git a/run_tests.sh b/run_tests.sh
> index 254129d..1e36d66 100755
> --- a/run_tests.sh
> +++ b/run_tests.sh
> @@ -7,7 +7,7 @@ if [ ! -f config.mak ]; then
> exit 1
> fi
> source config.mak
> -source scripts/functions.bash
> +source scripts/common.bash
>
> function usage()
> {
> @@ -46,17 +46,25 @@ while getopts "g:hv" opt; do
> esac
> done
>
> -RUNTIME_log_stderr () { cat >> test.log; }
> +# RUNTIME_log_file will be configured later
> +RUNTIME_log_stderr () { cat >> $RUNTIME_log_file; }
> RUNTIME_log_stdout () {
> if [ "$PRETTY_PRINT_STACKS" = "yes" ]; then
> - ./scripts/pretty_print_stacks.py $1 >> test.log
> + ./scripts/pretty_print_stacks.py $1 >> $RUNTIME_log_file
> else
> - cat >> test.log
> + cat >> $RUNTIME_log_file
> fi
> }
>
> -
> config=$TEST_DIR/unittests.cfg
> -rm -f test.log
> -printf "BUILD_HEAD=$(cat build-head)\n\n" > test.log
> +
> +rm -rf $unittest_log_dir.old
> +if ! mv $unittest_log_dir $unittest_log_dir.log; then
You remove the destination above, so this should never fail.
> + echo "Failed to backup $unittest_log_dir"
> + exit 2
> +fi
> +
> +mkdir $unittest_log_dir
> +echo "BUILD_HEAD=$(cat build-head)" > $unittest_log_dir/SUMMARY
> +
> for_each_unittest $config run
> diff --git a/scripts/functions.bash b/scripts/common.bash
> similarity index 75%
> rename from scripts/functions.bash
> rename to scripts/common.bash
> index ee9143c..2dd7360 100644
> --- a/scripts/functions.bash
> +++ b/scripts/common.bash
> @@ -1,3 +1,12 @@
> +: ${unittest_log_dir:=logs}
> +
> +function run_task()
> +{
> + local testname="$2"
> +
> + RUNTIME_log_file="${unittest_log_dir}/${testname}.log"
> + "$@"
> +}
>
> function for_each_unittest()
> {
> @@ -17,7 +26,7 @@ function for_each_unittest()
>
> while read -u $fd line; do
> if [[ "$line" =~ ^\[(.*)\]$ ]]; then
> - "$cmd" "$testname" "$groups" "$smp" "$kernel" "$opts" "$arch" "$check" "$accel" "$timeout"
> + run_task "$cmd" "$testname" "$groups" "$smp" "$kernel" "$opts" "$arch" "$check" "$accel" "$timeout"
> testname=${BASH_REMATCH[1]}
> smp=1
> kernel=""
> @@ -45,6 +54,6 @@ function for_each_unittest()
> timeout=${BASH_REMATCH[1]}
> fi
> done
> - "$cmd" "$testname" "$groups" "$smp" "$kernel" "$opts" "$arch" "$check" "$accel" "$timeout"
> + run_task "$cmd" "$testname" "$groups" "$smp" "$kernel" "$opts" "$arch" "$check" "$accel" "$timeout"
> exec {fd}<&-
> }
> diff --git a/scripts/mkstandalone.sh b/scripts/mkstandalone.sh
> index d2bae19..3c1938e 100755
> --- a/scripts/mkstandalone.sh
> +++ b/scripts/mkstandalone.sh
> @@ -5,7 +5,7 @@ if [ ! -f config.mak ]; then
> exit 1
> fi
> source config.mak
> -source scripts/functions.bash
> +source scripts/common.bash
>
> escape ()
> {
> --
> 2.7.4
>
>
thanks,
drew
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
