On Wed, May 11, 2016 at 06:30:40PM +0200, Radim Krčmář wrote: > Summary should provide all important information. > > Signed-off-by: Radim Krčmář <rkrcmar@xxxxxxxxxx> > --- > v4: new > > run_tests.sh | 1 + > scripts/mkstandalone.sh | 1 + > scripts/runtime.bash | 3 ++- > 3 files changed, 4 insertions(+), 1 deletion(-) Reviewed-by: Andrew Jones <drjones@xxxxxxxxxx> > > diff --git a/run_tests.sh b/run_tests.sh > index b634e472a834..254129d8723c 100755 > --- a/run_tests.sh > +++ b/run_tests.sh > @@ -46,6 +46,7 @@ while getopts "g:hv" opt; do > esac > done > > +RUNTIME_log_stderr () { cat >> test.log; } > RUNTIME_log_stdout () { > if [ "$PRETTY_PRINT_STACKS" = "yes" ]; then > ./scripts/pretty_print_stacks.py $1 >> test.log > diff --git a/scripts/mkstandalone.sh b/scripts/mkstandalone.sh > index 77793fd64c76..d2bae1922f4b 100755 > --- a/scripts/mkstandalone.sh > +++ b/scripts/mkstandalone.sh > @@ -70,6 +70,7 @@ generate_test () > > echo "exec {stdout}>&1" > echo "RUNTIME_log_stdout () { cat >&\$stdout; }" > + echo "RUNTIME_log_stderr () { cat >&2; }" > > cat scripts/runtime.bash > > diff --git a/scripts/runtime.bash b/scripts/runtime.bash > index 6d9d8bffb82f..deae077e50a1 100644 > --- a/scripts/runtime.bash > +++ b/scripts/runtime.bash > @@ -55,7 +55,8 @@ function run() > > # extra_params in the config file may contain backticks that need to be > # expanded, so use eval to start qemu. Same for $RUNTIME_log_stdout. > - summary=$(eval $cmdline > >(tee >(RUNTIME_log_stdout $kernel) | extract_summary)) > + summary=$(eval $cmdline 2> >(RUNTIME_log_stderr) \ > + > >(tee >(RUNTIME_log_stdout $kernel) | extract_summary)) > ret=$? > > if [ $ret -eq 0 ]; then > -- > 2.8.2 > > -- > 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 -- 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