From mboxrd@z Thu Jan 1 00:00:00 1970 From: Avi Kivity Subject: Re: [PATCH] KVM test: Subtest unittest: append extra_params to qemu cmdline Date: Tue, 03 Aug 2010 14:25:00 +0300 Message-ID: <4C57FC8C.7090702@redhat.com> References: <1280770173-23890-1-git-send-email-lmr@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: autotest@test.kernel.org, kvm@vger.kernel.org To: Lucas Meneghel Rodrigues Return-path: Received: from mx1.redhat.com ([209.132.183.28]:32383 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752109Ab0HCLZC (ORCPT ); Tue, 3 Aug 2010 07:25:02 -0400 In-Reply-To: <1280770173-23890-1-git-send-email-lmr@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: 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 > --- > 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. -- error compiling committee.c: too many arguments to function