Re: [Qemu-devel] [PATCH] tests: switch pxe and vm gen id tests to use kvm

[Thomas Huth]

On 14.07.2017 16:45, Michael S. Tsirkin wrote:
> On Fri, Jul 14, 2017 at 07:59:54AM +0200, Thomas Huth wrote:
>> On 13.07.2017 19:00, Paolo Bonzini wrote:
>>> On 13/07/2017 18:47, Michael S. Tsirkin wrote:
>>>> Speed up tests on host systems with kvm support.
[...]
>>>> diff --git a/tests/vmgenid-test.c b/tests/vmgenid-test.c
>>>> index 0556ba2..6477dcc 100644
>>>> --- a/tests/vmgenid-test.c
>>>> +++ b/tests/vmgenid-test.c
>>>> @@ -132,7 +132,7 @@ static char disk[] = "tests/vmgenid-test-disk-XXXXXX";
>>>>  
>>>>  static char *guid_cmd_strdup(const char *guid)
>>>>  {
>>>> -    return g_strdup_printf("-machine accel=tcg "
>>>> +    return g_strdup_printf("-machine accel=kvm:tcg "
>>>>                             "-device vmgenid,id=testvgid,guid=%s "
>>>>                             "-drive id=hd0,if=none,file=%s,format=raw "
>>>>                             "-device ide-hd,drive=hd0 ",
>>>
>>> Acked-by: Paolo Bonzini <address@hidden>
>>
>> NACK.
> 
> As a rule, don't send nacks please. Skip directly to the technical
> argument. Nacks add very little to the discussion and tend to scare away
> contributors.

I hope I won't scare *you* away that easily ;-)
But ok, I got the message, and will try to remember it.

>> It's not that simple. If you do it like this, you then suddenly get
>> these ugly '"kvm" accelerator not found' messages during "make check"
>> which we really should avoid.
>>
>> Laurent suggested some better ideas a couple of months ago, but I think
>> the common sense was rather "unit tests should always use TCG" at that
>> point in time:
>>
>>  https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg00400.html
> 
> Looks good. This was never posted as a patch. Would you or him care
> to test it and re-post? I'll include it with my patch.

That URL points to Paolo's "we should always only use TCG mail" ... I
guess you are rather referring to the other URL below?

 Thomas


>> Well, now that we've got the "--disable-tcg" configure option, we should
>> maybe start to reconsider that ;-)
>>
>> So I'm fine with the patch if you also include Laurent's patch to shut
>> up the error message:
>>
>>  https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg00395.html
>>
>>  Thomas