On Wed, Jul 29, 2009 at 12:21 AM, Mark McLoughlin<markmc@xxxxxxxxxx> wrote:
> On Wed, 2009-07-29 at 00:10 +0900, Ryota Ozaki wrote:
>> Hi,
>>
>> This patch adds missing ifname, which is a tap name for network/bridge
>> configuration, in qemu argument of -net. Without this patch, a specified
>> tap name, e.g., <target dev='tap01'/>, in a domain XML will be ignored.
>>
>> Thanks,
>> ozaki-r
>>
>> >From feeca4f2cecf5c6836f7aec6e004585f3b855bed Mon Sep 17 00:00:00 2001
>> From: Ryota Ozaki <ozaki.ryota@xxxxxxxxx>
>> Date: Tue, 28 Jul 2009 23:46:12 +0900
>> Subject: [PATCH] qemu: fix missing ifname in qemu argument
>>
>> ---
>> src/qemu_conf.c | 6 ++++--
>> 1 files changed, 4 insertions(+), 2 deletions(-)
>>
>> diff --git a/src/qemu_conf.c b/src/qemu_conf.c
>> index 4fe4e39..48bf928 100644
>> --- a/src/qemu_conf.c
>> +++ b/src/qemu_conf.c
>> @@ -1187,11 +1187,13 @@ qemuBuildHostNetStr(virConnectPtr conn,
>> switch (net->type) {
>> case VIR_DOMAIN_NET_TYPE_NETWORK:
>> case VIR_DOMAIN_NET_TYPE_BRIDGE:
>> - if (virAsprintf(str, "%stap%cfd=%s,vlan=%d%s%s",
>> + if (virAsprintf(str, "%stap%cfd=%s,vlan=%d%s%s%s%s",
>> prefix ? prefix : "",
>> type_sep, tapfd, vlan,
>> (net->hostnet_name ? ",name=" : ""),
>> - (net->hostnet_name ? net->hostnet_name : "")) < 0) {
>> + (net->hostnet_name ? net->hostnet_name : ""),
>> + (net->ifname ? ",ifname=" : ""),
>> + (net->ifname ? net->ifname : "")) < 0) {
>
> I think latest qemu.git fails if you pass ifname= with fd=
>
> Older versions will just ignore it
>
> <target dev='tap01'/> just means that libvirt will create the tap
> interface as tap01
Oh, I really didn't know the change.
>
> What exactly is the problem you're seeing?
I've retried without ifname and confirmed there is nothing problem.
I seem making a quick judgment the change as a regression.
Apologies for the fuss and please drop my patch.
Thanks,
ozaki-r
>
> Thanks,
> Mark.
>
>
--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list