[libvirt] [PATCH v2 2/2] qemu: network user/direct/hostdev do not support backend.
Julio Faracco
jcfaracco at gmail.com
Thu Nov 23 20:35:07 UTC 2017
Opssss... Python feelings!
2017-11-23 14:50 GMT-02:00 Ján Tomko <jtomko at redhat.com>:
> On Fri, Nov 17, 2017 at 07:27:41PM -0200, Julio Faracco wrote:
>
>> The tag backend is not supported for user/direct/hostdev network when you
>> try to define them inside the domain XML. So, other ways to include
>> devices
>> cannot be permitted too. But the attach-device command is wrongly adding
>> unsupported features. This commit fixes this bug.
>>
>> After the patch:
>>
>> virsh # attach-device rhel7.4 backend.xml
>> error: Failed to attach device from backend.xml
>> error: unsupported configuration: Custom tap device path is not supported
>> for: user
>>
>> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1480251
>>
>> Signed-off-by: Julio Faracco <jcfaracco at gmail.com>
>> ---
>> src/qemu/qemu_domain.c | 15 +++++++++++++++
>> 1 file changed, 15 insertions(+)
>>
>> diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
>> index cc7596b..0215b2a 100644
>> --- a/src/qemu/qemu_domain.c
>> +++ b/src/qemu/qemu_domain.c
>> @@ -3649,6 +3656,14 @@ qemuDomainDeviceDefValidate(const
>> virDomainDeviceDef *dev,
>> }
>> }
>> }
>> + } else if (net->type == VIR_DOMAIN_NET_TYPE_DIRECT ||
>> + net->type == VIR_DOMAIN_NET_TYPE_HOSTDEV) {
>> + if (net->backend.tap) {
>> + virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
>> + _("Custom tap device path is not
>> supported for: %s"),
>> + virDomainNetTypeToString(net->type));
>> + goto cleanup;
>> + }
>>
>
> Adding this 'else if' clause means that the following check is no longer
> executed for TYPE_DIRECT and TYPE_HOSTDEV:
>
> } else if (net->guestIP.nroutes || net->guestIP.nips) {
>> virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
>> _("Invalid attempt to set network interface "
>>
>
> Jan
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171123/55b10055/attachment-0001.htm>
More information about the libvir-list
mailing list