Re: [kvm-unit-tests PATCH 7/7] arm/run: add --debug option

["Alex Bennée" <alex.bennee@linaro.org>]

Andrew Jones <drjones@redhat.com> writes:

> On Fri, Jul 03, 2015 at 02:48:47PM +0100, Alex Bennée wrote:
>> This allows you to pass debug options through to the QEMU command line.
>> e.g.:
>> 
>>   ./arm/run --debug "-name debug-threads=on" -- arm/vos-spinlock-test.flat -smp 4
>
> doesn't
> ./arm/run arm/vos-spinlock-test.flat -smp 4 -name debug-threads=on
> do the same thing?

You are right of course, I forgot you needed an explict -append for the
args to the test anyway.

>
>> 
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>>  arm/run | 7 ++++++-
>>  1 file changed, 6 insertions(+), 1 deletion(-)
>> 
>> diff --git a/arm/run b/arm/run
>> index 43d7508..871e35e 100755
>> --- a/arm/run
>> +++ b/arm/run
>> @@ -18,10 +18,15 @@ if [ -c /dev/kvm ]; then
>>  	fi
>>  fi
>>  
>> +debugopts=""
>>  while :; do
>>  	case $1 in
>>  		--force-tcg)
>>  			usingkvm=0
>> +			;;
>> +		--debug)
>> +			shift
>> +			debugopts=$1
>>  			shift
>>  			;;
>>  		--)
>> @@ -78,7 +83,7 @@ if [ $usingkvm = 1 ] && [ "$ARCH" = "arm64" ]; then
>>  fi
>>  
>>  command="$qemu $M -cpu $processor $chr_testdev"
>> -command+=" -display none -serial stdio -kernel"
>> +command+=" -display none -serial stdio $debugopts -kernel"
>>  
>>  echo $command "$@"
>>  $command "$@"
>> -- 
>> 2.4.5
>> 
>> --
>> To unsubscribe from this list: send the line "unsubscribe kvm" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html

-- 
Alex Bennée