Re: [PATCH 11/13] libxl: set nic type to VIF by default

[Roger Pau Monne <roger.pau@citrix.com>]

Ian Jackson wrote:
> Roger Pau Monne writes ("[PATCH 11/13] libxl: set nic type to VIF by default"):
>> Set the default value for nic interfaces to VIF, since it used to be
>> IOEMU, even for PV guests.
>
> How odd.
>
>> diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c
>> index 2490138..631de15 100644
>> --- a/tools/libxl/libxl.c
>> +++ b/tools/libxl/libxl.c
>> @@ -1637,7 +1637,7 @@ int libxl__device_nic_setdefault(libxl__gc *gc, libxl_device_nic *nic)
>>                                     libxl__xen_script_dir_path())<  0 )
>>           return ERROR_FAIL;
>>       if (!nic->nictype)
>> -        nic->nictype = LIBXL_NIC_TYPE_IOEMU;
>> +        nic->nictype = LIBXL_NIC_TYPE_VIF;
>
> But doesn't this set the default type to VIF even for HVM guests ?

Shouldn't HVM guests use the "ioemu" parameter if they want an emulated 
card? If no parameter is provided then the default type should be VIF, 
because there's no other way to specifically set the type to VIF.