Re: [PROPOSED PATCH] Change the creation of the TAP interface for generic ethernet interfaces

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/07/2011 06:49 PM, Tyler Coumbes wrote:
> The code changes it so that the TAP interface is created by libvirt
> and then the file descriptor is passed to qemu. Much like what is done
> when using bridged networking. This is done using fd= instead of
> ifname= just like with the bridged networking.

And in general this agrees with our overall desire to use fd passing
instead of filename passing, so I'm all for seeing this patch polished
against the latest libvirt.git.

> Option 1:
> 
> Proposed change would be that if there is no script parameter it is
> assumed that fd= is used instead of ifname= and if script= is present
> then use ifname=
> 
> Would use fd= after the patch.
> <interface type='ethernet'>
>     <target dev='tap0'/>
> </interface>
> 
> Would use ifname= after patch since it has a script= parameter. If a
> script parameter is needed the work around would still need to be in
> place.
> <interface type='ethernet'>
>     <target dev='tap0'/>
>     <script path='/etc/qemu-ifup'/>
> </interface>

I prefer this.  Libvirt is allowed to use the optimal interface under
the hood - the only reason to expose to the user which interface libvirt
is using is if there is ever a reason to suspect that using an fd is not
always optimal, and that a user would want to choose an alternate method
to experiment if the alternate method really is better in their case.

> 
> I like option 1 unless anyone sees an issue with that method.

Good to here, since I liked it too!

Looking forward to patches.

-- 
Eric Blake   eblake@xxxxxxxxxx    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]