- Devel - libvirt List Archives

[libvirt] PATCH: Always start dnsmasq to provide DNS
by Daniel P. Berrange 30 Jul '09

30 Jul '09

Currently we only start dnsmasq if DHCP ranges are configured. https://bugzilla.redhat.com/show_bug.cgi?id=508965 points out that this prevents use of DNS in guests using static IP config. Thus this patch makes us always start dnsmasq, regardless of DHCP config if we have an IP addr configured. There is no need for the --no-dhcp-interface= that BZ mentions, since that's just to allow you to restrict which interfaces dnsmasq does DHCP on - we've already restricted it to just out interface, and leaving out the DHCP ranges disables its DHCP server for every interface Regards, Daniel commit deb71927a3551b3a1554351ef098644aec978a09 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Thu Jul 30 14:52:31 2009 +0100 Allow dnsmasq to provide DNS without DHCP * src/network_driver.c: Always start dnsmasq to allow it to provide DNS, even if no DHCP ranges are enabled diff --git a/src/network_driver.c b/src/network_driver.c index 9621782..1683631 100644 --- a/src/network_driver.c +++ b/src/network_driver.c @@ -136,8 +136,9 @@ networkFindActiveConfigs(struct network_driver *driver) { brHasBridge(driver->brctl, obj->def->bridge) == 0) { obj->active = 1; - /* Finally try and read dnsmasq pid if any DHCP ranges are set */ - if (obj->def->nranges && + /* Finally try and read dnsmasq pid if any */ + if ((obj->def->ipAddress || + obj->def->nranges) && virFileReadPid(NETWORK_PID_DIR, obj->def->name, &obj->dnsmasqPid) == 0) { @@ -844,7 +845,8 @@ static int networkStartNetworkDaemon(virConnectPtr conn, goto err_delbr2; } - if (network->def->nranges && + if ((network->def->ipAddress || + network->def->nranges) && dhcpStartDhcpDaemon(conn, network) < 0) goto err_delbr2; -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

2 1

[libvirt] [PATCH] Remove MAX_TAP_ID, take 3
by Aron Griffis 30 Jul '09

30 Jul '09

This is a resend of take 2 to fix formatting problems in the patch. No other changes. As far as I can tell, there's no reason to format the device string in brAddTap(). Delegate the job to TUNSETIFF, thereby removing the loop and the MAX_TAP_ID artificial limit. This patch allows me to get 421 guests running before hitting other limits. Signed-off-by: Aron Griffis <aron.griffis(a)hp.com> diff --git a/src/bridge.c b/src/bridge.c index 0509afd..ec37192 100644 --- a/src/bridge.c +++ b/src/bridge.c @@ -49,8 +49,6 @@ #include "util.h" #include "logging.h" -#define MAX_TAP_ID 256 - #define JIFFIES_TO_MS(j) (((j)*1000)/HZ) #define MS_TO_JIFFIES(ms) (((ms)*HZ)/1000) @@ -466,76 +464,52 @@ brAddTap(brControl *ctl, int vnet_hdr, int *tapfd) { - int id, subst, fd; + int fd, len; + struct ifreq ifr = {0}; if (!ctl || !ctl->fd || !bridge || !ifname) return EINVAL; - subst = id = 0; - - if (strstr(*ifname, "%d")) - subst = 1; - if ((fd = open("/dev/net/tun", O_RDWR)) < 0) return errno; - if (vnet_hdr) - vnet_hdr = brProbeVnetHdr(fd); + ifr.ifr_flags = IFF_TAP|IFF_NO_PI; - do { - struct ifreq try; - int len; +#ifdef IFF_VNET_HDR + if (vnet_hdr && brProbeVnetHdr(fd)) + ifr.ifr_flags |= IFF_VNET_HDR; +#endif - memset(&try, 0, sizeof(struct ifreq)); + strncpy(ifr.ifr_name, *ifname, IFNAMSIZ-1); - try.ifr_flags = IFF_TAP|IFF_NO_PI; + if (ioctl(fd, TUNSETIFF, &ifr) < 0) + goto error; -#ifdef IFF_VNET_HDR - if (vnet_hdr) - try.ifr_flags |= IFF_VNET_HDR; -#endif + len = strlen(ifr.ifr_name); + if (len >= BR_IFNAME_MAXLEN - 1) { + errno = EINVAL; + goto error; + } - if (subst) { - len = snprintf(try.ifr_name, BR_IFNAME_MAXLEN, *ifname, id); - if (len >= BR_IFNAME_MAXLEN) { - errno = EADDRINUSE; - goto error; - } - } else { - len = strlen(*ifname); - if (len >= BR_IFNAME_MAXLEN - 1) { - errno = EINVAL; - goto error; - } - - strncpy(try.ifr_name, *ifname, len); - try.ifr_name[len] = '\0'; - } - - if (ioctl(fd, TUNSETIFF, &try) == 0) { - /* We need to set the interface MTU before adding it - * to the bridge, because the bridge will have its - * MTU adjusted automatically when we add the new interface. - */ - if ((errno = brSetInterfaceMtu(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brAddInterface(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brSetInterfaceUp(ctl, try.ifr_name, 1))) - goto error; - if (!tapfd && - (errno = ioctl(fd, TUNSETPERSIST, 1))) - goto error; - VIR_FREE(*ifname); - if (!(*ifname = strdup(try.ifr_name))) - goto error; - if (tapfd) - *tapfd = fd; - return 0; - } - - id++; - } while (subst && id <= MAX_TAP_ID); + /* We need to set the interface MTU before adding it + * to the bridge, because the bridge will have its + * MTU adjusted automatically when we add the new interface. + */ + if ((errno = brSetInterfaceMtu(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brAddInterface(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brSetInterfaceUp(ctl, ifr.ifr_name, 1))) + goto error; + if (!tapfd && + (errno = ioctl(fd, TUNSETPERSIST, 1))) + goto error; + VIR_FREE(*ifname); + if (!(*ifname = strdup(ifr.ifr_name))) + goto error; + if (tapfd) + *tapfd = fd; + return 0; error: close(fd);

4 9

[libvirt] VM creation failed : Permission denied : bind(unix:/var/run/libvirt/qemu//VM.monitor)
by Pierre-Gilles Mialon 30 Jul '09

30 Jul '09

I use Ovirt under Fedora 11 with the latest developpement package and the VM creation failed with : bind(unix:/var/run/libvirt/qemu//VM-4.monitor): Permission denied See the logs : On the Ovirt Node ==> /var/log/messages <== Jul 29 20:43:37 node62 kernel: device vnet0 entered promiscuous mode Jul 29 20:43:37 node62 kernel: breth0: port 2(vnet0) entering learning state Jul 29 20:43:37 node62 libvirtd: 20:43:37.873: info : qemudDispatchSignalEvent:370 : Received unexpected signal 17 Jul 29 20:43:37 node62 libvirtd: 20:43:37.877: info : qemudDispatchSignalEvent:370 : Received unexpected signal 17 Jul 29 20:43:37 node62 kernel: breth0: port 2(vnet0) entering disabled state Jul 29 20:43:37 node62 kernel: device vnet0 left promiscuous mode Jul 29 20:43:37 node62 kernel: breth0: port 2(vnet0) entering disabled state Jul 29 20:43:37 node62 libvirtd: 20:43:37.977: error : qemudReadLogOutput:816 : internal error Process exited while reading console log output Jul 29 20:43:37 node62 libvirtd: 20:43:37.977: error : qemudWaitForMonitor:1103 : internal error unable to start guest: bind(unix:/var/run/libvirt/qemu//VM-4.monitor): Permission denied#012qemu: could not open monitor device 'unix:/var/run/libvirt/qemu//VM-4.monitor,server,nowait'#012 Jul 29 20:43:38 node62 libvirtd: 20:43:38.073: error : qemudDomainLookupByUUID:2644 : Domain not found: no domain with matching uuid '492f836f-5123-e185-39c2-09c5dd43a7f6' Jul 29 20:43:38 node62 libvirt-qpid: Error: virDomainLookupByUUIDString Subsystem qemu: Domain not found: no domain with matching uuid '492f836f-5123- e185-39c2-09c5dd43a7f6' in NodeWrap.cpp:syncDomains:241 code: 42 ==> /var/log/libvirt/qemu/VM-4.log <== LC_ALL=C PATH=/sbin:/usr/sbin:/bin:/usr/bin HOME=/root USER=root LOGNAME=root /usr/bin/qemu-kvm -S -M pc -m 256 -smp 1 -name VM-4 -uuid 492f836f-5123- e185-39c2-09c5dd43a7f6 -monitor unix:/var/run/libvirt/qemu//VM-4.monitor,server,nowait -boot n -net nic,macaddr=00:16:3e:10:de:fe,vlan=0,name=nic.0 -net tap,fd=18,vlan=0,name=tap.0 -serial pty -parallel none -usb -vnc 0.0.0.0:0 - vga cirrus bind(unix:/var/run/libvirt/qemu//VM-4.monitor): Permission denied qemu: could not open monitor device 'unix:/var/run/libvirt/qemu//VM-4.monitor,server,nowait' ==> # rpm -qa | grep libvirt <== libvirt-0.7.0-0.2.gitf055724.fc11.x86_64 libvirt-python-0.7.0-0.2.gitf055724.fc11.x86_64 libvirt-client-0.7.0-0.2.gitf055724.fc11.x86_64 libvirt-qpid-0.2.17-0.fc11.x86_64 Regards -- Pierre-Gilles Mialon Linagora :: http://www.linagora.com Responsable hébergement :: Head of Hosting services pmialon(a)linagora.com :: +33.1 58 18 65 46

5 11

[libvirt] VMware ESX: Scheduler documentation and cleanup
by Matthias Bolte 30 Jul '09

30 Jul '09

Add some documentation about the CPU scheduler parameters and remove some old, unnecessary compensation code, since virsh uses the proposed parameter types now. Documentation in HTML form will be added soon. Matthias

2 1

[libvirt] vcpupin broken towards Xen using xend
by Henrik Persson 30 Jul '09

30 Jul '09

Hi, During the update of the cpu affinity map format, the old format was broken ( '[' was switched with ']'). Also, the mapstr might contain garbage since the string is not initiate (however buf incorrectly is). This small patch fixes both the problems. Regards, Henrik

2 1

[libvirt] PATCH: Fix RPM upgrades to create qemu user
by Daniel P. Berrange 30 Jul '09

30 Jul '09

If upgrading from F11 to F12 it is possible the qemu user/group does not yet exist. Since the libvirt RPM needs to deploy files owned by this user/group, regardless of whether QEMU is yet installed, we need to make sure to create this user/group in %pre, otherwise fiels will end up with wrong ownership. Daniel diff -u -p -r1.159 libvirt.spec --- libvirt.spec 29 Jul 2009 14:58:18 -0000 1.159 +++ libvirt.spec 30 Jul 2009 11:06:53 -0000 @@ -419,6 +419,18 @@ chmod 0644 $RPM_BUILD_ROOT%{_sysconfdir} %clean rm -fr %{buildroot} +%pre +%if 0%{?fedora} >= 12 +# Normally 'setup' adds this in /etc/passwd, but this is +# here for case of upgrades from earlier Fedora. This +# UID/GID pair is reserved for qemu:qemu +getent group kvm >/dev/null || groupadd -g 36 -r kvm +getent group qemu >/dev/null || groupadd -g 107 -r qemu +getent passwd qemu >/dev/null || \ + useradd -r -u 107 -g qemu -G kvm -d / -s /sbin/nologin \ + -c "qemu user" qemu +%endif + %post %if %{with_libvirtd} -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

2 1

[libvirt] Feature request? Console tunnelling..
by r-c＠rocketmail.com 30 Jul '09

30 Jul '09

Hello libvirt-list.. Firstly I hope this is the appropriate forum for this question, I tried the IRC channel but didn't get any response... Anyway, currently libvirt has console functionality built into it that can be used on a Dom0 e.g. 'virsh console xenU' to connect and open guest xenU's console. I think it would be excellent if there could be a feature added to the libvirt daemon to operate this console externally! E.g. from an external machine I can do (in python) something like: domain.openConsole() .readConsole() .writeConsole() flushConsole()... hopefully that could be very nice to implement a remote Ajaxterm with more security and no reliance to run unnecessary services on the Dom0 since libvirtd is already running :-) I hope this makes sense, but if you perhaps thinks I going about this the wrong way please let me know! Cheers

2 1

Re: [libvirt] VMware support and Server 2.0.1
by Matthias Bolte 30 Jul '09

30 Jul '09

2009/7/29 Timo Makinen <tmakinen(a)ee.oulu.fi>: > On Wed, Jul 29, 2009 at 06:12:59PM +0200, Matthias Bolte wrote: >> 2009/7/27 Timo Makinen <tmakinen(a)ee.oulu.fi>: [...] > The dumpxml seems to be failing with this config option: > > ethernet0.connectionType = "custom" > > without this it works correctly. Ah, I found the cause. ethernet0.connectionType = "custom" was a good hint. I missed to initialize one variable correctly that is only used in the ethernet0.connectionType = "custom" case. >> Could you do some basic testing beside listing and dumpxml, as I have >> currently no VMware GSX installation at hand? Just test if the >> following commands work as expected in virsh: >> >> - start >> - shutdown >> - reboot >> - suspend >> - resume >> - nodeinfo >> - dominfo >> >> Regards, >> Matthias > > All these worked out of the box. Nice! Could you test the attached patch? It adds handling for the port in the URI, extends the version checking for GSX 2.0 and fixes the bug in VMX parsing. Matthias

2 1

[libvirt] [PATCH] Modify the java bindings html page to reflect the new repositories, JNA, and maven access
by Bryan Kearney 29 Jul '09

29 Jul '09

--- docs/java.html.in | 109 +++++++++++++++++++++++++++-------------------------- 1 files changed, 55 insertions(+), 54 deletions(-) diff --git a/docs/java.html.in b/docs/java.html.in index 2d9a24f..b0db2e2 100644 --- a/docs/java.html.in +++ b/docs/java.html.in @@ -4,10 +4,9 @@ <h1>Java API bindings</h1> <h2>Presentation</h2> - <p>The Java bindings are currently a work in progress based mostly -on the work of Toth Istvan. The first usable release is 0.2.0, where -most of the naming conventions were defined. Further release will try -as much as possible to stay compatible</p> + <p>The Java bindings make use of <a href="https://jna.dev.java.net/">JNA</a> + to expose the C API in a Java friendly way. The bindings are are based on + work initiated by Toth Istvan.</p> <h2>Getting it</h2> <p> @@ -19,82 +18,84 @@ as much as possible to stay compatible</p> <li><a href="http://libvirt.org/sources/java/">libvirt.org HTTP server</a></li> </ul> -<h2>Hourly development snapshots</h2> -<p> Once an hour, an automated snapshot is made from the latest CVS server - source tree. These snapshots should be usable, but we make no guarantees - about their stability: </p> +<h3>Maven</h3> +<p>A maven repository is located at <a href="http://www.libvirt.org/maven2/">http://www.libvirt.org/maven2/</a> +which you can use to include this in your maven projects.</p> -<ul> - <li><a href="ftp://libvirt.org/libvirt/java/libvirt-java-snapshot.tar.gz">libvirt.org FTP server</a></li> - <li><a href="http://libvirt.org/sources/java/libvirt-java-snapshot.tar.gz">libvirt.org HTTP server</a></li> -</ul> +<h2>GIT source repository</h2> +<p> The Java bindings code source is now maintained in a <a +href="http://git-scm.com/">git</a> repository available on +<a href="http://libvirt.org/git/">libvirt.org</a>: +</p> +<pre> +git clone git://libvirt.org/libvirt-java.git +</pre> +<p> +It can also be browsed at +</p> +<pre> -<h2>CVS repository access</h2> +<a href="http://libvirt.org/git/?p=libvirt-java.git;a=summary">http://libvirt.org/git/?p=libvirt-java.git;a=summary</a> +</pre> -<p> The master source repository uses <a -href="http://ximbiot.com/cvs/cvshome/docs/">CVS</a> and anonymous access -is provided. Prior to accessing the server is it necessary to authenticate -using the password <code>anoncvs</code>. This can be accomplished with -the <code>cvs login</code> command: +<p></p> +<h2>GIT repository mirror</h2> + +<p> + The source code is also mirrored on <a href="http://www.gitorious.org">gitorious</a> + to aid in public development. You clone the repo with </p> -<pre> - # cvs -d :pserver:anoncvs@libvirt.org:2401/data/cvs login +<pre> + git clone git://gitorious.org/libvirt/libvirt-java.git </pre> <p> - Once authenticated, a checkout can be obtained using + It can also be browsed at </p> <pre> - # cvs -d :pserver:anoncvs@libvirt.org:2401/data/cvs co libvirt-java + <a href="http://gitorious.org/libvirt/libvirt-java">http://gitorious.org/libvirt/libvirt-java</a> </pre> -<p> -The libvirt-java build process uses GNU autotools, so after obtaining a checkout -it is necessary to generate the configure script and Makefile.in templates -using the <code>autogen.sh</code> command. As an example, to do a complete -build and install it into your home directory run: +<p></p> +<h2>CVS repository access (Deprecated)</h2> + +<p> The old source repository uses <a +href="http://ximbiot.com/cvs/cvshome/docs/">CVS</a> and anonymous access +is provided. Prior to accessing the server is it necessary to authenticate +using the password <code>anoncvs</code>. This can be accomplished with +the <code>cvs login</code> command: </p> + <pre> - ./autogen.sh --prefix=$HOME/usr - make - make install + # cvs -d :pserver:anoncvs@libvirt.org:2401/data/cvs login </pre> -<p>Other build system are not currently available but as usual we take patches -(Eclipse/ant/...) would be welcome !</p> -<p> currently libvirt-java requires a version of libvirt >= 0.4.0 to be -installed as well as the associated libvirt development files/package. It -should compile with any java implementation >= 1.5, the bindings uses -the enum construct which appeared only in that version.</p> -<p>Libvirt-java bindings development occurs on the same mailing-list as -the normal libvirt work see <a href="contact.html">the associated contact -page</a></p> - -<h2>GIT repository mirror</h2> <p> - The CVS source repository is also mirrored using GIT, and is available - for anonymous access via: + Once authenticated, a checkout can be obtained using </p> -<pre> - git clone git://git.et.redhat.com/libvirt-java +<pre> + # cvs -d :pserver:anoncvs@libvirt.org:2401/data/cvs co libvirt-java </pre> -<p> - It can also be browsed at -</p> +<p></p> +<h2>Building</h2> +<p>The code is built using ant, and assumes that you have the jna jar installed. Once you have downloaded +the code you can build the code with</p> <pre> - <a href="http://git.et.redhat.com/?p=libvirt-java.git;a=summary">http://git.et.redhat.com/?p=libvirt-java.git;a=summary</a> +% cd libvirt-java +% ant build </pre> + <h2>Content</h2> <p>The bindings are articulated around a few -classes in the <code>org/libvirt</code> namespace, notably the +classes in the <code>org/libvirt</code> package, notably the <code>Connect</code>, <code>Domain</code> and <code>Network</code> ones. Functions in the <a href="html/libvirt-libvirt.html">C API</a> taking <code>virConnectPtr</code>, <code>virDomainPtr</code> or @@ -112,13 +113,13 @@ lower case, for example the C functions:</p> </p> <p>become</p> <p> - <code>virConn::numOfDomains()</code> + <code>virConn.numOfDomains()</code> </p> <p> - <code>virDomain::setMaxMemory(long memory)</code> + <code>virDomain.setMaxMemory(long memory)</code> </p> <p> There is of course some functions where the mapping is less direct -and using extra classes to map complex arguments. The <a href="http://libvirt.org/org/libvirt/package-summary.html">Javadoc</a> is available online or as +and using extra classes to map complex arguments. The <a href="http://libvirt.org/sources/java/javadoc">Javadoc</a> is available online or as part of a separate libvirt-java-javadoc package.</p> <p>So let's look at a simple example inspired from the <code>test.java</code> test found in <code>src</code> in the source tree:</p> @@ -147,7 +148,7 @@ public class minitest { <p>There is not much to comment about it, it really is a straight mapping from the C API, the only points to notice are:</p> <ul> - <li>the import of the modules in the <code><span style="color: #0071FF; background-color: #FFFFFF">org.libvirt</span></code> namespace</li> + <li>the import of the modules in the <code><span style="color: #0071FF; background-color: #FFFFFF">org.libvirt</span></code> package</li> <li>getting a connection to the hypervisor, in that case using the readonly access to the default test hypervisor.</li> <li>getting an object representing the test domain using <span style="color: #007F00; background-color: #FFFFFF">lookupByName</span></li> -- 1.6.0.6

2 1

[libvirt] ANNOUNCE: Java Libvirt bindings version 0.3.0
by Bryan Kearney 29 Jul '09

29 Jul '09

I am happy to announce the availability of version 0.3.0 of the Libvirt Java bindings. The main change in this release is the use of JNA [1] to access the Libvirt C code instead of JNI. This results in all the glue code being pure Java which should make it easier to keep these bindings in sync with the C code. The RPMS are in fedora-testing for 10, 11 and rawhide. They will move to updates per the normal process. The code can be directly downloaded from [2]. The updated repository has been moved to git, which can be found at [3]. I am able to generate a maven repository for the bits, and was planning on putting a maven repo on libvirt.org unless folks would prefer the bits to be mirrored in a different location. Please let me know. Thanks! -- bk [1] https://jna.dev.java.net/ [2] http://www.libvirt.org/sources/java/libvirt-java-0.3.0.tar.gz [3] http://www.libvirt.org/git/?p=libvirt-java.git;a=summary

1 0

[libvirt] [PATCH] Remove MAX_TAP_ID
by Aron Griffis 29 Jul '09

29 Jul '09

As far as I can tell, there's no reason to format the device string in brAddTap(). Delegate the job to TUNSETIFF, thereby removing the loop and the MAX_TAP_ID artificial limit. This patch allows me to get 421 guests running before hitting other limits. Signed-off-by: Aron Griffis <aron.griffis(a)hp.com> --- src/bridge.c | 94 +++++++++++++++++++++------------------------------------- 1 files changed, 34 insertions(+), 60 deletions(-) diff --git a/src/bridge.c b/src/bridge.c index 0509afd..ec37192 100644 --- a/src/bridge.c +++ b/src/bridge.c @@ -49,8 +49,6 @@ #include "util.h" #include "logging.h" -#define MAX_TAP_ID 256 - #define JIFFIES_TO_MS(j) (((j)*1000)/HZ) #define MS_TO_JIFFIES(ms) (((ms)*HZ)/1000) @@ -466,76 +464,52 @@ brAddTap(brControl *ctl, int vnet_hdr, int *tapfd) { - int id, subst, fd; + int fd, len; + struct ifreq ifr = {0}; if (!ctl || !ctl->fd || !bridge || !ifname) return EINVAL; - subst = id = 0; - - if (strstr(*ifname, "%d")) - subst = 1; - if ((fd = open("/dev/net/tun", O_RDWR)) < 0) return errno; - if (vnet_hdr) - vnet_hdr = brProbeVnetHdr(fd); + ifr.ifr_flags = IFF_TAP|IFF_NO_PI; - do { - struct ifreq try; - int len; +#ifdef IFF_VNET_HDR + if (vnet_hdr && brProbeVnetHdr(fd)) + ifr.ifr_flags |= IFF_VNET_HDR; +#endif - memset(&try, 0, sizeof(struct ifreq)); + strncpy(ifr.ifr_name, *ifname, IFNAMSIZ-1); - try.ifr_flags = IFF_TAP|IFF_NO_PI; + if (ioctl(fd, TUNSETIFF, &ifr) < 0) + goto error; -#ifdef IFF_VNET_HDR - if (vnet_hdr) - try.ifr_flags |= IFF_VNET_HDR; -#endif + len = strlen(ifr.ifr_name); + if (len >= BR_IFNAME_MAXLEN - 1) { + errno = EINVAL; + goto error; + } - if (subst) { - len = snprintf(try.ifr_name, BR_IFNAME_MAXLEN, *ifname, id); - if (len >= BR_IFNAME_MAXLEN) { - errno = EADDRINUSE; - goto error; - } - } else { - len = strlen(*ifname); - if (len >= BR_IFNAME_MAXLEN - 1) { - errno = EINVAL; - goto error; - } - - strncpy(try.ifr_name, *ifname, len); - try.ifr_name[len] = '\0'; - } - - if (ioctl(fd, TUNSETIFF, &try) == 0) { - /* We need to set the interface MTU before adding it - * to the bridge, because the bridge will have its - * MTU adjusted automatically when we add the new interface. - */ - if ((errno = brSetInterfaceMtu(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brAddInterface(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brSetInterfaceUp(ctl, try.ifr_name, 1))) - goto error; - if (!tapfd && - (errno = ioctl(fd, TUNSETPERSIST, 1))) - goto error; - VIR_FREE(*ifname); - if (!(*ifname = strdup(try.ifr_name))) - goto error; - if (tapfd) - *tapfd = fd; - return 0; - } - - id++; - } while (subst && id <= MAX_TAP_ID); + /* We need to set the interface MTU before adding it + * to the bridge, because the bridge will have its + * MTU adjusted automatically when we add the new interface. + */ + if ((errno = brSetInterfaceMtu(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brAddInterface(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brSetInterfaceUp(ctl, ifr.ifr_name, 1))) + goto error; + if (!tapfd && + (errno = ioctl(fd, TUNSETPERSIST, 1))) + goto error; + VIR_FREE(*ifname); + if (!(*ifname = strdup(ifr.ifr_name))) + goto error; + if (tapfd) + *tapfd = fd; + return 0; error: close(fd);

2 2

[libvirt] [PATCH] Remove MAX_TAP_ID, take 2
by Aron Griffis 29 Jul '09

29 Jul '09

(Sorry, my first posting included some gnulib droppings in the patch. This removes that, everything else is the same.) As far as I can tell, there's no reason to format the device string in brAddTap(). Delegate the job to TUNSETIFF, thereby removing the loop and the MAX_TAP_ID artificial limit. This patch allows me to get 421 guests running before hitting other limits. Signed-off-by: Aron Griffis <aron.griffis(a)hp.com> --- src/bridge.c | 94 +++++++++++++++++++++------------------------------------- 1 files changed, 34 insertions(+), 60 deletions(-) diff --git a/src/bridge.c b/src/bridge.c index 0509afd..ec37192 100644 --- a/src/bridge.c +++ b/src/bridge.c @@ -49,8 +49,6 @@ #include "util.h" #include "logging.h" -#define MAX_TAP_ID 256 - #define JIFFIES_TO_MS(j) (((j)*1000)/HZ) #define MS_TO_JIFFIES(ms) (((ms)*HZ)/1000) @@ -466,76 +464,52 @@ brAddTap(brControl *ctl, int vnet_hdr, int *tapfd) { - int id, subst, fd; + int fd, len; + struct ifreq ifr = {0}; if (!ctl || !ctl->fd || !bridge || !ifname) return EINVAL; - subst = id = 0; - - if (strstr(*ifname, "%d")) - subst = 1; - if ((fd = open("/dev/net/tun", O_RDWR)) < 0) return errno; - if (vnet_hdr) - vnet_hdr = brProbeVnetHdr(fd); + ifr.ifr_flags = IFF_TAP|IFF_NO_PI; - do { - struct ifreq try; - int len; +#ifdef IFF_VNET_HDR + if (vnet_hdr && brProbeVnetHdr(fd)) + ifr.ifr_flags |= IFF_VNET_HDR; +#endif - memset(&try, 0, sizeof(struct ifreq)); + strncpy(ifr.ifr_name, *ifname, IFNAMSIZ-1); - try.ifr_flags = IFF_TAP|IFF_NO_PI; + if (ioctl(fd, TUNSETIFF, &ifr) < 0) + goto error; -#ifdef IFF_VNET_HDR - if (vnet_hdr) - try.ifr_flags |= IFF_VNET_HDR; -#endif + len = strlen(ifr.ifr_name); + if (len >= BR_IFNAME_MAXLEN - 1) { + errno = EINVAL; + goto error; + } - if (subst) { - len = snprintf(try.ifr_name, BR_IFNAME_MAXLEN, *ifname, id); - if (len >= BR_IFNAME_MAXLEN) { - errno = EADDRINUSE; - goto error; - } - } else { - len = strlen(*ifname); - if (len >= BR_IFNAME_MAXLEN - 1) { - errno = EINVAL; - goto error; - } - - strncpy(try.ifr_name, *ifname, len); - try.ifr_name[len] = '\0'; - } - - if (ioctl(fd, TUNSETIFF, &try) == 0) { - /* We need to set the interface MTU before adding it - * to the bridge, because the bridge will have its - * MTU adjusted automatically when we add the new interface. - */ - if ((errno = brSetInterfaceMtu(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brAddInterface(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brSetInterfaceUp(ctl, try.ifr_name, 1))) - goto error; - if (!tapfd && - (errno = ioctl(fd, TUNSETPERSIST, 1))) - goto error; - VIR_FREE(*ifname); - if (!(*ifname = strdup(try.ifr_name))) - goto error; - if (tapfd) - *tapfd = fd; - return 0; - } - - id++; - } while (subst && id <= MAX_TAP_ID); + /* We need to set the interface MTU before adding it + * to the bridge, because the bridge will have its + * MTU adjusted automatically when we add the new interface. + */ + if ((errno = brSetInterfaceMtu(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brAddInterface(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brSetInterfaceUp(ctl, ifr.ifr_name, 1))) + goto error; + if (!tapfd && + (errno = ioctl(fd, TUNSETPERSIST, 1))) + goto error; + VIR_FREE(*ifname); + if (!(*ifname = strdup(ifr.ifr_name))) + goto error; + if (tapfd) + *tapfd = fd; + return 0; error: close(fd);

3 3

[libvirt] VMware support and Server 2.0.1
by Timo Makinen 29 Jul '09

29 Jul '09

I tested libvirt with VMware Server 2.0.1 (not ESX). Server starts http on port 8333 as default and libvirt api did not have any option to give port number when connecting. Maybe connecting should be changed to support uris like: esx://host:123/ Next i tried to hardcode port into libvirt and got following error: error: internal error Expecting server major/minor version '3.5' or '4.0' but found '2.0.1' I patched libvirt again and finally got connected. Well at least list command gave out list of vms. With dumpxml i got following error: virsh # dumpxml vm02 error: internal error Missing essential config entry '' - Timo

2 1

[libvirt] [PATCH] qemu: fix monitor socket reconnection
by Ryota Ozaki 29 Jul '09

29 Jul '09

Hi, This patch fixes a handling of a qemu monitor unix socket. Current code addresses 'Socket may not have shown up yet' case with catching EACCES of connect(2), however, in that case, an errno which should be catched is ENOENT, not EACCES. Thanks, ozaki-r >From 5a4b7ee2296ca6756127c26c6bfbbe1f49ab8722 Mon Sep 17 00:00:00 2001 From: Ryota Ozaki <ozaki.ryota(a)gmail.com> Date: Tue, 28 Jul 2009 23:48:22 +0900 Subject: [PATCH] qemu: fix monitor socket reconnection --- src/qemu_driver.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 52a77be..9fb8506 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -917,8 +917,8 @@ qemudOpenMonitorUnix(virConnectPtr conn, if (ret == 0) break; - if (errno == EACCES || errno == ECONNREFUSED) { - /* EACCES : Socket may not have shown up yet + if (errno == ENOENT || errno == ECONNREFUSED) { + /* ENOENT : Socket may not have shown up yet * ECONNREFUSED : Leftover socket hasn't been removed yet */ continue; } -- 1.6.0.6

4 4

[libvirt] [PATCH] Kernel command line support for UML
by Ron Yorston 29 Jul '09

29 Jul '09

When experimenting with libvirt and UML I found it necessary to add support for additional kernel command line arguments. Ron diff -up libvirt-0.6.5/src/uml_conf.c.uml-cmdline libvirt-0.6.5/src/uml_conf.c --- libvirt-0.6.5/src/uml_conf.c.uml-cmdline 2009-07-10 16:28:51.000000000 +0100 +++ libvirt-0.6.5/src/uml_conf.c 2009-07-27 19:39:59.000000000 +0100 @@ -326,6 +326,31 @@ umlBuildCommandLineChr(virConnectPtr con } /* + * Null-terminate the current argument and return a pointer to the next. + * This should follow the same rules as the Linux kernel: arguments are + * separated by spaces; arguments can be quoted with double quotes; double + * quotes can't be escaped. + */ +static char *umlNextArg(char *args) +{ + int in_quote = 0; + + for (; *args; args++) { + if (*args == ' ' && !in_quote) { + *args++ = '\0'; + break; + } + if (*args == '"') + in_quote = !in_quote; + } + + while (*args == ' ') + args++; + + return args; +} + +/* * Constructs a argv suitable for launching uml with config defined * for a given virtual machine. */ @@ -342,6 +367,7 @@ int umlBuildCommandLine(virConnectPtr co const char **qargv = NULL; int qenvc = 0, qenva = 0; const char **qenv = NULL; + char *cmdline = NULL; uname(&ut); @@ -474,6 +500,22 @@ int umlBuildCommandLine(virConnectPtr co ADD_ARG(ret); } + if (vm->def->os.cmdline) { + char *args, *next_arg; + if ((cmdline = strdup(vm->def->os.cmdline)) == NULL) + goto no_memory; + + args = cmdline; + while (*args == ' ') + args++; + + while (*args) { + next_arg = umlNextArg(args); + ADD_ARG_LIT(args); + args = next_arg; + } + } + ADD_ARG(NULL); ADD_ENV(NULL); @@ -495,6 +537,7 @@ int umlBuildCommandLine(virConnectPtr co VIR_FREE((qenv)[i]); VIR_FREE(qenv); } + VIR_FREE(cmdline); return -1; #undef ADD_ARG

3 2

[libvirt] [PATCH 1/3] Move ldconfig calls to libvirt-client %post/%postun
by Mark McLoughlin 29 Jul '09

29 Jul '09

Now that the library has moved to libvirt-client --- libvirt.spec.in | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index 6228d0b..b79e766 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -411,7 +411,6 @@ chmod 0644 $RPM_BUILD_ROOT%{_sysconfdir}/sysconfig/libvirtd rm -fr %{buildroot} %post -/sbin/ldconfig %if %{with_libvirtd} %if %{with_qemu} @@ -440,7 +439,9 @@ if [ $1 = 0 ]; then fi %endif -%postun -p /sbin/ldconfig +%post client -p /sbin/ldconfig + +%postun client -p /sbin/ldconfig %files %defattr(-, root, root) -- 1.6.2.5

2 5

[libvirt] [PATCH] libvirt-glib: add missing details argument to callbacks
by Guido Günther 29 Jul '09

29 Jul '09

Hi, the python example is lacking the details argument in the callback. This makes the example fail due to the wrong number of arguments. Cheers, -- Guido

2 5

[libvirt] Debug the username/passwords
by Bryan Kearney 29 Jul '09

29 Jul '09

Hopefully this is the last question I have on the Java drivers. I am trying to debug the callback from openAuth(). I am passing back what I think is valid information, but I see in /var/log/messages the following: Jul 28 16:21:24 localhost libvirtd: sasl step failed -20 (SASL(-13): user not found: no secret in database) Is there a way to output what I have passed back in the credential responses? Setting the following in the /etc/libvirt/libvirtd.conf file does not seem to increase the logging: log_level=1 log_outputs="1:stderr 1:syslog:bk 1:file:virsh.log" and setting LIBVIRT_DEBUG=1,and launching libvirtd from the command line does not spit out the info. -- bk

2 2

[libvirt] ANNOUNCE: New release virt-manager 0.8.0
by Cole Robinson 29 Jul '09

29 Jul '09

I'm happy to announce a new virt-manager release, version 0.8.0. The release can be downloaded from: http://virt-manager.org/download.html The direct download link is: http://virt-manager.org/download/sources/virt-manager/virt-manager-0.8.0.ta… This release includes: - New 'Clone VM' Wizard - Improved UI, including an overhaul of the main 'manager' view - System tray icon for easy VM access (start, stop, view console/details) - Wizard for adding serial, parallel, and video devices to existing VMs. - CPU pinning support (Michal Novotny) - Ability to view and change VM security (sVirt) settings (Dan Walsh) - Many bug fixes and improvements Thanks to everyone who has contributed to this release through testing, bug reporting, submitting patches, and otherwise sending in feedback! Thanks, Cole

4 3

[libvirt] PATCH: Fix remote domain events deadlock
by Daniel P. Berrange 29 Jul '09

29 Jul '09

There is a deadlock in the remote driver's dispatch code for domain events. If your callback triggers a API call that does an RPC call, then the driver will deadlock. The fix is the same as we did in the QEMU driver. We releae the lock & re-aquire it, with some extreme cleverness to protect against people deleting callbacks during the course of dispatch Daniel commit e2e6f795dd1832227c7d5c7832596be3642fe6e4 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Tue Jul 28 17:45:19 2009 +0100 Fix deadlock in remote driver domain events * src/remote_internal.c: Release driver lock when dispatching events to callbacks diff --git a/src/remote_internal.c b/src/remote_internal.c index f20ed6e..a58b768 100644 --- a/src/remote_internal.c +++ b/src/remote_internal.c @@ -167,6 +167,8 @@ struct private_data { virDomainEventQueuePtr domainEvents; /* Timer for flushing domainEvents queue */ int eventFlushTimer; + /* Flag if we're in process of dispatching */ + int domainEventDispatching; /* Self-pipe to wakeup threads waiting in poll() */ int wakeupSendFD; @@ -6288,18 +6290,26 @@ static int remoteDomainEventDeregister (virConnectPtr conn, remoteDriverLock(priv); - if (virDomainEventCallbackListRemove(conn, priv->callbackList, - callback) < 0) { - error (conn, VIR_ERR_RPC, _("removing cb fron list")); - goto done; - } - - if ( priv->callbackList->count == 0 ) { - /* Tell the server when we are the last callback deregistering */ - if (call (conn, priv, 0, REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER, - (xdrproc_t) xdr_void, (char *) NULL, - (xdrproc_t) xdr_void, (char *) NULL) == -1) + if (priv->domainEventDispatching) { + if (virDomainEventCallbackListMarkDelete(conn, priv->callbackList, + callback) < 0) { + error (conn, VIR_ERR_RPC, _("marking cb for deletion")); goto done; + } + } else { + if (virDomainEventCallbackListRemove(conn, priv->callbackList, + callback) < 0) { + error (conn, VIR_ERR_RPC, _("removing cb from list")); + goto done; + } + + if ( priv->callbackList->count == 0 ) { + /* Tell the server when we are the last callback deregistering */ + if (call (conn, priv, 0, REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER, + (xdrproc_t) xdr_void, (char *) NULL, + (xdrproc_t) xdr_void, (char *) NULL) == -1) + goto done; + } } rv = 0; @@ -7309,18 +7319,54 @@ done: remoteDriverUnlock(priv); } +static void remoteDomainEventDispatchFunc(virConnectPtr conn, + virDomainEventPtr event, + virConnectDomainEventCallback cb, + void *cbopaque, + void *opaque) +{ + struct private_data *priv = opaque; + + /* Drop the lock whle dispatching, for sake of re-entrancy */ + remoteDriverUnlock(priv); + virDomainEventDispatchDefaultFunc(conn, event, cb, cbopaque, NULL); + remoteDriverLock(priv); +} + void remoteDomainEventQueueFlush(int timer ATTRIBUTE_UNUSED, void *opaque) { virConnectPtr conn = opaque; struct private_data *priv = conn->privateData; + virDomainEventQueue tempQueue; remoteDriverLock(priv); - virDomainEventQueueDispatch(priv->domainEvents, priv->callbackList, - virDomainEventDispatchDefaultFunc, NULL); + priv->domainEventDispatching = 1; + + /* Copy the queue, so we're reentrant safe */ + tempQueue.count = priv->domainEvents->count; + tempQueue.events = priv->domainEvents->events; + priv->domainEvents->count = 0; + priv->domainEvents->events = NULL; + + virDomainEventQueueDispatch(&tempQueue, priv->callbackList, + remoteDomainEventDispatchFunc, priv); virEventUpdateTimeout(priv->eventFlushTimer, -1); + /* Purge any deleted callbacks */ + virDomainEventCallbackListPurgeMarked(priv->callbackList); + + if ( priv->callbackList->count == 0 ) { + /* Tell the server when we are the last callback deregistering */ + if (call (conn, priv, 0, REMOTE_PROC_DOMAIN_EVENTS_DEREGISTER, + (xdrproc_t) xdr_void, (char *) NULL, + (xdrproc_t) xdr_void, (char *) NULL) == -1) + VIR_WARN0("Failed to de-register events"); + } + + priv->domainEventDispatching = 0; + remoteDriverUnlock(priv); } -- |: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :| |: http://libvirt.org -o- http://virt-manager.org -o- http://ovirt.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|

2 1

[libvirt] [PATCH] Fix trailing whitespace in NEWS
by Mark McLoughlin 29 Jul '09

29 Jul '09

Hey, I had to commit this before I could push Cheers, Mark.

1 0

[libvirt] [PATCH 00/14] Clean up spec files mess
by Mark McLoughlin 29 Jul '09

29 Jul '09

Hey, I get treated like a war criminal when I make a change in the Fedora spec file and don't send it upstream, yet there are a whole heap of changes which haven't been pushed upstream ... only one of which is mine ... two changes less than DV himself :-) The following series of patches pulls in the changes from Fedora which make sense. Some of them I'm not 100% sure of: - defaulting to policy kit - is there some reason we don't want that upstream? - The python cruft thing, not sure its needed - Converting NEWS to UTF-8 could probably be done upstream - Requiring libselinux is dubious, doesn't autorequires handle that? My final contention is that maintaining these spec files in the upstream repo actually makes it harder to maintain the Fedora spec file and helps nobody. Because of half-assed attempts to keep the two in sync, I ended up having to fix a bunch of stuff and re-instate another bunch of stuff in Fedora: - Enable netcf support - Pass --with-qemu-user=qemu etc. to configure - Move various requires to the libvirt-client sub-package - Sync some trivial cleanups from upstream spec file - Remove explicit libxml2 requires, again - Build with --without-capng if capng support is disabled - Remove explicit dir creating in makeinstall, replaced by attr in files - Set perms on /var/{run,lib,cache}/libvirt/qemu I really think we should kill these off and just point people at the Fedora spec file if they're looking for a reference spec file. I need some very strong rationale before I ever go wasting my time (2 hours) on this again. Cheers, Mark.

4 23

[libvirt] [PATCH] Remove MAX_TAP_ID
by Aron Griffis 29 Jul '09

29 Jul '09

As far as I can tell, there's no reason to format the device string in brAddTap(). Delegate the job to TUNSETIFF, thereby removing the loop and the MAX_TAP_ID artificial limit. This patch allows me to get 421 guests running before hitting other limits. Signed-off-by: Aron Griffis <aron.griffis(a)hp.com> --- src/bridge.c | 94 +++++++++++++++++++++------------------------------------- 1 files changed, 34 insertions(+), 60 deletions(-) diff --git a/.gnulib b/.gnulib deleted file mode 160000 index b653eda..0000000 --- a/.gnulib +++ /dev/null @@ -1 +0,0 @@ -Subproject commit b653eda3ac4864de205419d9f41eec267cb89eeb diff --git a/src/bridge.c b/src/bridge.c index 0509afd..ec37192 100644 --- a/src/bridge.c +++ b/src/bridge.c @@ -49,8 +49,6 @@ #include "util.h" #include "logging.h" -#define MAX_TAP_ID 256 - #define JIFFIES_TO_MS(j) (((j)*1000)/HZ) #define MS_TO_JIFFIES(ms) (((ms)*HZ)/1000) @@ -466,76 +464,52 @@ brAddTap(brControl *ctl, int vnet_hdr, int *tapfd) { - int id, subst, fd; + int fd, len; + struct ifreq ifr = {0}; if (!ctl || !ctl->fd || !bridge || !ifname) return EINVAL; - subst = id = 0; - - if (strstr(*ifname, "%d")) - subst = 1; - if ((fd = open("/dev/net/tun", O_RDWR)) < 0) return errno; - if (vnet_hdr) - vnet_hdr = brProbeVnetHdr(fd); + ifr.ifr_flags = IFF_TAP|IFF_NO_PI; - do { - struct ifreq try; - int len; +#ifdef IFF_VNET_HDR + if (vnet_hdr && brProbeVnetHdr(fd)) + ifr.ifr_flags |= IFF_VNET_HDR; +#endif - memset(&try, 0, sizeof(struct ifreq)); + strncpy(ifr.ifr_name, *ifname, IFNAMSIZ-1); - try.ifr_flags = IFF_TAP|IFF_NO_PI; + if (ioctl(fd, TUNSETIFF, &ifr) < 0) + goto error; -#ifdef IFF_VNET_HDR - if (vnet_hdr) - try.ifr_flags |= IFF_VNET_HDR; -#endif + len = strlen(ifr.ifr_name); + if (len >= BR_IFNAME_MAXLEN - 1) { + errno = EINVAL; + goto error; + } - if (subst) { - len = snprintf(try.ifr_name, BR_IFNAME_MAXLEN, *ifname, id); - if (len >= BR_IFNAME_MAXLEN) { - errno = EADDRINUSE; - goto error; - } - } else { - len = strlen(*ifname); - if (len >= BR_IFNAME_MAXLEN - 1) { - errno = EINVAL; - goto error; - } - - strncpy(try.ifr_name, *ifname, len); - try.ifr_name[len] = '\0'; - } - - if (ioctl(fd, TUNSETIFF, &try) == 0) { - /* We need to set the interface MTU before adding it - * to the bridge, because the bridge will have its - * MTU adjusted automatically when we add the new interface. - */ - if ((errno = brSetInterfaceMtu(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brAddInterface(ctl, bridge, try.ifr_name))) - goto error; - if ((errno = brSetInterfaceUp(ctl, try.ifr_name, 1))) - goto error; - if (!tapfd && - (errno = ioctl(fd, TUNSETPERSIST, 1))) - goto error; - VIR_FREE(*ifname); - if (!(*ifname = strdup(try.ifr_name))) - goto error; - if (tapfd) - *tapfd = fd; - return 0; - } - - id++; - } while (subst && id <= MAX_TAP_ID); + /* We need to set the interface MTU before adding it + * to the bridge, because the bridge will have its + * MTU adjusted automatically when we add the new interface. + */ + if ((errno = brSetInterfaceMtu(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brAddInterface(ctl, bridge, ifr.ifr_name))) + goto error; + if ((errno = brSetInterfaceUp(ctl, ifr.ifr_name, 1))) + goto error; + if (!tapfd && + (errno = ioctl(fd, TUNSETPERSIST, 1))) + goto error; + VIR_FREE(*ifname); + if (!(*ifname = strdup(ifr.ifr_name))) + goto error; + if (tapfd) + *tapfd = fd; + return 0; error: close(fd);

1 0

[libvirt] ANNOUNCE: New release virtinst 0.500.0
by Cole Robinson 29 Jul '09

29 Jul '09

I'm happy to announce a new virtinst release, version 0.500.0. The release can be downloaded from: http://virt-manager.org/download.html The direct download link is: http://virt-manager.org/download/sources/virtinst/virtinst-0.500.0.tar.gz This release includes: - New virt-install device options --serial, --parallel, and --video - Allow various auth types for libvirt connections (PolicyKit, SASL, ...) - New virt-clone option --auto-clone: generates all needed input. - Option to specify network device model via virt-install --network (Guido Gunther) - New virt-install option --virt-type for specifying hypervisor type (kvm, qemu). --accelerate is now the default behavior: To provision a plain qemu VM on a KVM enabled host, use '--virt-type qemu' - OVF input support for virt-convert - Many bug fixes and improvements Thanks to everyone who has contributed to this release through testing, bug reporting, submitting patches, and otherwise sending in feedback! Thanks, Cole

1 0

[libvirt] [PATCH] qemu: fix missing ifname in qemu argument
by Ryota Ozaki 28 Jul '09

28 Jul '09

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(a)gmail.com> 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) { virReportOOMError(conn); return -1; } -- 1.6.0.6

2 2