Re: [PATCH] KVM test: Subtest unittest: append extra_params to qemu cmdline

[Avi Kivity <avi@redhat.com>]

  On 08/03/2010 02:25 PM, Avi Kivity wrote:
>  On 08/02/2010 08:29 PM, Lucas Meneghel Rodrigues wrote:
>> The extra_param config option on qemu-kvm's unittest config
>> file wasn't being honored due to a silly mistake on the latest
>> version of the unittest patchset (forgot to add the extra_params
>> to the params dictionary). This patch fixes the problem.
>>
>> Signed-off-by: Lucas Meneghel Rodrigues<lmr@redhat.com>
>> ---
>>   client/tests/kvm/tests/unittest.py |    1 +
>>   1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/client/tests/kvm/tests/unittest.py 
>> b/client/tests/kvm/tests/unittest.py
>> index 8be1f27..ad95720 100644
>> --- a/client/tests/kvm/tests/unittest.py
>> +++ b/client/tests/kvm/tests/unittest.py
>> @@ -75,6 +75,7 @@ def run_unittest(test, params, env):
>>           extra_params = None
>>           if parser.has_option(t, 'extra_params'):
>>               extra_params = parser.get(t, 'extra_params')
>> +            params['extra_params'] += ' %s' % extra_params
>
> Not quite:
>
>
> 08/03 13:57:04 DEBUG|    kvm_vm:0637| Running qemu command:
> /root/autotest/client/tests/kvm/qemu -name 'vm1' -monitor 
> unix:'/tmp/monitor-humanmonitor1-20100803-135522-SqL2',server,nowait 
> -serial unix:'/tmp/serial-20100803-135522-SqL2',server,nowait -m 512 
> -kernel '/root/autotest/client/tests/kvm/unittests/svm.flat' -vnc :0 
> -chardev file,id=testlog,path=/tmp/testlog-20100803-135522-SqL2 
> -device testdev,chardev=testlog  -S -cpu qemu64,-svm -cpu 
> qemu64,+x2apic -enable-nesting -cpu qemu64,+svm
>
>
> Looks the += is a little excessive.
>

It also leaks to other tests, screwing them up.  So I think you might 
need to keep the += (so you inherit global settings) but undo it after 
the unit test completes.

-- 
error compiling committee.c: too many arguments to function