linux-kernel - Re: [PATCH tip/core/rcu 32/45] torture: Better summary diagnostics for build failures

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140507221139.GP27924@cloud>
Date:	Wed, 7 May 2014 15:11:39 -0700
From:	josh@...htriplett.org
To:	"Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
Cc:	linux-kernel@...r.kernel.org, mingo@...nel.org,
	laijs@...fujitsu.com, dipankar@...ibm.com,
	akpm@...ux-foundation.org, mathieu.desnoyers@...icios.com,
	niv@...ibm.com, tglx@...utronix.de, peterz@...radead.org,
	rostedt@...dmis.org, dhowells@...hat.com, edumazet@...gle.com,
	darren@...art.com, fweisbec@...il.com, oleg@...hat.com, sbw@....edu
Subject: Re: [PATCH tip/core/rcu 32/45] torture: Better summary diagnostics
 for build failures

On Mon, Apr 28, 2014 at 05:25:20PM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney" <paulmck@...ux.vnet.ibm.com>
> 
> The reaction of kvm-recheck.sh is obscure at best, and easy to miss
> completely.  This commit therefore prints "BUG: Build failed" in the
> summary at the end of a run.

This commit also changes a dozen other things about the output that this
commit message does not document. :)

> Signed-off-by: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>
> ---
>  .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
>  .../selftests/rcutorture/bin/kvm-recheck-rcu.sh    |  2 +-
>  .../selftests/rcutorture/bin/kvm-recheck.sh        | 24 ++++++++++++++++------
>  .../selftests/rcutorture/bin/kvm-test-1-run.sh     |  1 +
>  4 files changed, 21 insertions(+), 8 deletions(-)
> 
> diff --git a/tools/testing/selftests/rcutorture/bin/kvm-recheck-lock.sh b/tools/testing/selftests/rcutorture/bin/kvm-recheck-lock.sh
> index 829186e19eb1..7f1ff1a8fc4b 100755
> --- a/tools/testing/selftests/rcutorture/bin/kvm-recheck-lock.sh
> +++ b/tools/testing/selftests/rcutorture/bin/kvm-recheck-lock.sh
> @@ -35,7 +35,7 @@ configfile=`echo $i | sed -e 's/^.*\///'`
>  ncs=`grep "Writes:  Total:" $i/console.log 2> /dev/null | tail -1 | sed -e 's/^.* Total: //' -e 's/ .*$//'`
>  if test -z "$ncs"
>  then
> -	echo $configfile
> +	echo "$configfile -------"
>  else
>  	title="$configfile ------- $ncs acquisitions/releases"
>  	dur=`sed -e 's/^.* locktorture.shutdown_secs=//' -e 's/ .*$//' < $i/qemu-cmd 2> /dev/null`
> diff --git a/tools/testing/selftests/rcutorture/bin/kvm-recheck-rcu.sh b/tools/testing/selftests/rcutorture/bin/kvm-recheck-rcu.sh
> index d75b1dc5ae53..307c4b95f325 100755
> --- a/tools/testing/selftests/rcutorture/bin/kvm-recheck-rcu.sh
> +++ b/tools/testing/selftests/rcutorture/bin/kvm-recheck-rcu.sh
> @@ -35,7 +35,7 @@ configfile=`echo $i | sed -e 's/^.*\///'`
>  ngps=`grep ver: $i/console.log 2> /dev/null | tail -1 | sed -e 's/^.* ver: //' -e 's/ .*$//'`
>  if test -z "$ngps"
>  then
> -	echo $configfile
> +	echo "$configfile -------"
>  else
>  	title="$configfile ------- $ngps grace periods"
>  	dur=`sed -e 's/^.* rcutorture.shutdown_secs=//' -e 's/ .*$//' < $i/qemu-cmd 2> /dev/null`
> diff --git a/tools/testing/selftests/rcutorture/bin/kvm-recheck.sh b/tools/testing/selftests/rcutorture/bin/kvm-recheck.sh
> index 26d78b7eaccf..ee1f6cae3d70 100755
> --- a/tools/testing/selftests/rcutorture/bin/kvm-recheck.sh
> +++ b/tools/testing/selftests/rcutorture/bin/kvm-recheck.sh
> @@ -25,6 +25,7 @@
>  # Authors: Paul E. McKenney <paulmck@...ux.vnet.ibm.com>
>  
>  PATH=`pwd`/tools/testing/selftests/rcutorture/bin:$PATH; export PATH
> +. tools/testing/selftests/rcutorture/bin/functions.sh
>  for rd in "$@"
>  do
>  	firsttime=1
> @@ -39,13 +40,24 @@ do
>  		fi
>  		TORTURE_SUITE="`cat $i/../TORTURE_SUITE`"
>  		kvm-recheck-${TORTURE_SUITE}.sh $i
> -		configcheck.sh $i/.config $i/ConfigFragment
> -		parse-build.sh $i/Make.out $configfile
> -		parse-torture.sh $i/console.log $configfile
> -		parse-console.sh $i/console.log $configfile
> -		if test -r $i/Warnings
> +		if test -f "$i/console.log"
>  		then
> -			cat $i/Warnings
> +			configcheck.sh $i/.config $i/ConfigFragment
> +			parse-build.sh $i/Make.out $configfile
> +			parse-torture.sh $i/console.log $configfile
> +			parse-console.sh $i/console.log $configfile
> +			if test -r $i/Warnings
> +			then
> +				cat $i/Warnings
> +			fi
> +		else
> +			if test -f "$i/qemu-cmd"
> +			then
> +				print_bug qemu failed
> +			else
> +				print_bug Build failed
> +			fi
> +			echo "   $i"
>  		fi
>  	done
>  done
> 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 7a95f86cc85a..51c34a91a375 100755
> --- a/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> +++ b/tools/testing/selftests/rcutorture/bin/kvm-test-1-run.sh
> @@ -106,6 +106,7 @@ then
>  	fi
>  else
>  	cp $builddir/Make*.out $resdir
> +	cp $builddir/.config $resdir || :
>  	echo Build failed, not running KVM, see $resdir.
>  	if test -f $builddir.wait
>  	then
> -- 
> 1.8.1.5
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/