- Devel - libvirt List Archives

[libvirt] [PATCH] Cosmetic change to 'virsh nodedev-list --tree' output
by Mark McLoughlin 18 Aug '09

18 Aug '09

Maybe it's just me, but I try to select an item from the tree using double-click and get annoyed when "+-" gets included in the selection. * src/virsh.c: add a space between "+-" and the node device name in 'virsh nodedev-list --tree' --- src/virsh.c | 4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/virsh.c b/src/virsh.c index 94c3c4e..2d0cf81 100644 --- a/src/virsh.c +++ b/src/virsh.c @@ -5370,6 +5370,8 @@ cmdNodeListDevicesPrint(vshControl *ctl, if (depth && depth < MAX_DEPTH) { indentBuf[indentIdx] = '+'; indentBuf[indentIdx+1] = '-'; + indentBuf[indentIdx+2] = ' '; + indentBuf[indentIdx+3] = '\0'; } /* Print this device */ @@ -5398,7 +5400,7 @@ cmdNodeListDevicesPrint(vshControl *ctl, /* If there is a child device, then print another blank line */ if (nextlastdev != -1) { vshPrint(ctl, "%s", indentBuf); - vshPrint(ctl, " |\n"); + vshPrint(ctl, " |\n"); } /* Finally print all children */ -- 1.6.2.5

2 3

[libvirt] XenStore fix
by Jonas Eriksson 17 Aug '09

17 Aug '09

Hi, I have been examining a bug where libvirtd (and virsh) does not show all virtual machines on a xen host. This proved to be because of this program flow: 1. virConnectNumOfDomains -> .. -> xenUnifiedNumOfDomains -> xenHypervisorNumOfDomains => 3 2. virConnectListDomains(max=3) -> .. -> xenUnifiedListDomains(max=3) -> xenStoreNumOfDomains(max=3) => { 0, 2, 7 } The domain with ID 2 is then removed when it is discovered that it is not a running domain, which leads to this: xenhost# xm list Name ID Mem VCPUs State Time(s) Domain-0 0 14970 2 r----- 2544.7 vm1 7 512 1 -b---- 2191.7 vm4 512 1 28.0 vm5 12 512 1 -b---- 467.1 vm6 512 1 0.0 vm7 512 1 482.4 xenhost# virsh list Id Name State ---------------------------------- 0 Domain-0 running 7 vm1 idle xenhost# But where does "2" come from? If we check all "directories" in /local/domain which is queried by the xenstore driver, it is apparent that xenstore is not properly cleaned. We find the sequence {0, 2, 7} as the first entries: xenhost# xenstore ls /local/domain |grep '^[^ ]' 0 = "" 2 = "" 7 = "" 9 = "" 10 = "" 11 = "" 12 = "" xenhost# This patch checks that the path found in /local/domain/<domid>/vm exists in xenstore before adding the domid to the return list. The same thing is done for xenStoreNumOfDomains. I use SLES11 with Xen 3.3.1_18546_12-3.1. /Jonas

4 16

[libvirt] libvirt socket closed unexpectedly (code=39) & libvirt: Broken pipe (code=38)
by dave c 17 Aug '09

17 Aug '09

Hello everyone, I'm trying to get eucalyptus v1.5.2 working on debian lenny 64bit, the relevant info I can give you is as follows: --- libvirtd conf file ---- 01:/etc/libvirt# cat libvirtd.conf|grep -v '#' unix_sock_group = "libvirt" unix_sock_rw_perms = "0770" auth_unix_ro = "none" auth_unix_rw = "none" --------------------- version of libvir installed on my server (cloud controller/cluster controller): 01:/etc/libvirt# dpkg -l|grep libvirt ii libvirt-bin 0.4.6-10 the programs for the libvirt library ii libvirt-dev 0.4.6-10 development files for the libvirt library ii libvirt-doc 0.4.6-10 documentation for the libvirt library ii libvirt0 0.4.6-10 library for interfacing with different virtualization systems version installed on my node: 02:~# dpkg -l |grep libvirt ii libvirt-bin 0.4.6-10 the programs for the libvirt library ii libvirt0 0.4.6-10 library for interfacing with different virtualization systems ================ error in my nc..log (eucalyptus node log): [Sat Aug 15 02:24:58 2009][005115][EUCADEBUG ] system_output(): [//usr/lib/eucalyptus/euca_rootwrap //usr/share/eucalyptus/gen_kvm_libvirt_xml --ramdisk --ephemeral] [Sat Aug 15 02:24:58 2009][005115][EUCAERROR ] libvirt: socket closed unexpectedly (code=39) [Sat Aug 15 02:24:58 2009][005115][EUCAERROR ] libvirt: Broken pipe (code=38) basically when i start up an instance, it starts up and then terminates within 1 min I've already posted on the eucalyptus forums, i thought the libvir-list might be able to assist as well thoughts / ideas ? anything else I can share ?

2 1

[libvirt] remote_protocol.c not compiling
by Kenneth Nagin 17 Aug '09

17 Aug '09

I am experimenting with some new APIs in Libvirt 0.6.5 and added the changes to remote_protocol.x . I'm getting a make error of undefined reference in ./.libs/libvirt.so to the xdr arguments. I think it is related to the fact that a new remote_protocol.c is not being generated. I've tried the obvious "make clean" and "make" and "make -C qemud remote_protocol.c." But it has no affect. [nagin@gorky trunk]$ make -C qemud remote_protocol.c make: Entering directory `/gpfs/reservoir/nagin/workspace/LIBVIRT/LIBVIRT_0_6_5/trunk/qemud' make: Nothing to be done for `remote_protocol.c'. make: Leaving directory `/gpfs/reservoir/nagin/workspace/LIBVIRT/LIBVIRT_0_6_5/trunk/qemud' Any suggestions? Kenneth Nagin

2 2

[libvirt] [PATCH] Compressed save image format for Qemu.
by Chris Lalancette 17 Aug '09

17 Aug '09

Implement a compressed save image format for qemu. While ideally we would have the choice between compressed/non-compressed available to the libvirt API, unfortunately there is no "flags" parameter to the virDomainSave() API. Therefore, implement this as a qemu.conf option. Both gzip and bzip2 are implemented, and it should be very easy to implement additional compression methods. One open question is if/how we should detect the gzip and bzip2 binaries. One way to do it is to do compile-time setting of the paths (via configure.in), but that doesn't seem like a great thing to do. Another solution (my preferred solution) is not to detect at all; when we go to run the commands that need them, if they aren't available, or aren't available in one of the standard paths, then we'll fail. Maybe somebody else has another option or opinion, though. In the future, we'll have a more robust (managed) save/restore API, at which time we can expose this functionality properly in the API. V2: get rid of redundant dd command and just use >> to append data. V3: Add back the missing pieces for the enum and bumping the save version. V4: Make the compressed field in the save_header an int. Implement LZMA compression. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu.conf | 10 ++++++ src/qemu_conf.c | 11 ++++++ src/qemu_conf.h | 2 + src/qemu_driver.c | 93 +++++++++++++++++++++++++++++++++++++++++++++++++---- 4 files changed, 109 insertions(+), 7 deletions(-) diff --git a/src/qemu.conf b/src/qemu.conf index 653f487..1f10b43 100644 --- a/src/qemu.conf +++ b/src/qemu.conf @@ -129,3 +129,13 @@ # "/dev/ptmx", "/dev/kvm", "/dev/kqemu", # "/dev/rtc", "/dev/hpet", "/dev/net/tun", #] + +# The default format for Qemu/KVM guest save images is raw; that is, the +# memory from the domain is dumped out directly to a file. If you have +# guests with a large amount of memory, however, this can take up quite +# a bit of space. If you would like to compress the images while they +# are being saved to disk, you can also set "gzip", "bzip2", or "lzma" +# for save_image_format. Note that this means you slow down the +# process of saving a domain in order to save disk space. +# +# save_image_format = "raw" diff --git a/src/qemu_conf.c b/src/qemu_conf.c index 7ca5a15..ed87e13 100644 --- a/src/qemu_conf.c +++ b/src/qemu_conf.c @@ -280,6 +280,17 @@ int qemudLoadDriverConfig(struct qemud_driver *driver, driver->cgroupDeviceACL[i] = NULL; } + p = virConfGetValue (conf, "save_image_format"); + CHECK_TYPE ("save_image_format", VIR_CONF_STRING); + if (p && p->str) { + VIR_FREE(driver->saveImageFormat); + if (!(driver->saveImageFormat = strdup(p->str))) { + virReportOOMError(NULL); + virConfFree(conf); + return -1; + } + } + virConfFree (conf); return 0; } diff --git a/src/qemu_conf.h b/src/qemu_conf.h index 8f4ef6a..e34baab 100644 --- a/src/qemu_conf.h +++ b/src/qemu_conf.h @@ -111,6 +111,8 @@ struct qemud_driver { char *securityDriverName; virSecurityDriverPtr securityDriver; + + char *saveImageFormat; }; diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 20906ef..3fd153d 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -3411,18 +3411,27 @@ static char *qemudEscapeShellArg(const char *in) } #define QEMUD_SAVE_MAGIC "LibvirtQemudSave" -#define QEMUD_SAVE_VERSION 1 +#define QEMUD_SAVE_VERSION 2 + +enum qemud_save_formats { + QEMUD_SAVE_FORMAT_RAW, + QEMUD_SAVE_FORMAT_GZIP, + QEMUD_SAVE_FORMAT_BZIP2, + QEMUD_SAVE_FORMAT_LZMA, +}; struct qemud_save_header { char magic[sizeof(QEMUD_SAVE_MAGIC)-1]; int version; int xml_len; int was_running; - int unused[16]; + int compressed; + int unused[15]; }; static int qemudDomainSave(virDomainPtr dom, - const char *path) { + const char *path) +{ struct qemud_driver *driver = dom->conn->privateData; virDomainObjPtr vm; char *command = NULL; @@ -3433,11 +3442,28 @@ static int qemudDomainSave(virDomainPtr dom, struct qemud_save_header header; int ret = -1; virDomainEventPtr event = NULL; + int internalret; memset(&header, 0, sizeof(header)); memcpy(header.magic, QEMUD_SAVE_MAGIC, sizeof(header.magic)); header.version = QEMUD_SAVE_VERSION; + if (driver->saveImageFormat == NULL) + header.compressed = QEMUD_SAVE_FORMAT_RAW; + else if (STREQ(driver->saveImageFormat, "raw")) + header.compressed = QEMUD_SAVE_FORMAT_RAW; + else if (STREQ(driver->saveImageFormat, "gzip")) + header.compressed = QEMUD_SAVE_FORMAT_GZIP; + else if (STREQ(driver->saveImageFormat, "bzip2")) + header.compressed = QEMUD_SAVE_FORMAT_BZIP2; + else if (STREQ(driver->saveImageFormat, "lzma")) + header.compressed = QEMUD_SAVE_FORMAT_LZMA; + else { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, + "%s", _("Invalid save image format specified in configuration file")); + return -1; + } + qemuDriverLock(driver); vm = virDomainFindByUUID(&driver->domains, dom->uuid); @@ -3510,11 +3536,28 @@ static int qemudDomainSave(virDomainPtr dom, virReportOOMError(dom->conn); goto cleanup; } - if (virAsprintf(&command, "migrate \"exec:" - "dd of='%s' oflag=append conv=notrunc 2>/dev/null" - "\"", safe_path) == -1) { + + if (header.compressed == QEMUD_SAVE_FORMAT_RAW) + internalret = virAsprintf(&command, "migrate \"exec:" + "dd of='%s' oflag=append conv=notrunc 2>/dev/null" + "\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_GZIP) + internalret = virAsprintf(&command, "migrate \"exec:" + "gzip -c >> '%s' 2>/dev/null\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_BZIP2) + internalret = virAsprintf(&command, "migrate \"exec:" + "bzip2 -c >> '%s' 2>/dev/null\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_LZMA) + internalret = virAsprintf(&command, "migrate \"exec:" + "lzma -c >> '%s' 2>/dev/null\"", safe_path); + else { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_INTERNAL_ERROR, + _("Invalid compress format %d"), + header.compressed); + goto cleanup; + } + if (internalret < 0) { virReportOOMError(dom->conn); - command = NULL; goto cleanup; } @@ -4035,6 +4078,9 @@ static int qemudDomainRestore(virConnectPtr conn, char *xml = NULL; struct qemud_save_header header; virDomainEventPtr event = NULL; + int intermediatefd = -1; + pid_t intermediate_pid = -1; + int childstat; qemuDriverLock(driver); /* Verify the header and read the XML */ @@ -4124,8 +4170,41 @@ static int qemudDomainRestore(virConnectPtr conn, } def = NULL; + if (header.version == 2) { + const char *intermediate_argv[3] = { NULL, "-dc", NULL }; + if (header.compressed == QEMUD_SAVE_FORMAT_GZIP) + intermediate_argv[0] = "gzip"; + else if (header.compressed == QEMUD_SAVE_FORMAT_BZIP2) + intermediate_argv[0] = "bzip2"; + else if (header.compressed == QEMUD_SAVE_FORMAT_LZMA) + intermediate_argv[0] = "lzma"; + else if (header.compressed != QEMUD_SAVE_FORMAT_RAW) { + qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, + _("Unknown compressed save format %d"), + header.compressed); + goto cleanup; + } + if (intermediate_argv[0] != NULL) { + intermediatefd = fd; + fd = -1; + if (virExec(conn, intermediate_argv, NULL, NULL, + &intermediate_pid, intermediatefd, &fd, NULL, 0) < 0) { + qemudReportError(conn, NULL, NULL, VIR_ERR_INTERNAL_ERROR, + _("Failed to start decompression binary %s"), + intermediate_argv[0]); + goto cleanup; + } + } + } /* Set the migration source and start it up. */ ret = qemudStartVMDaemon(conn, driver, vm, "stdio", fd); + if (intermediate_pid != -1) { + /* Wait for intermediate process to exit */ + while (waitpid(intermediate_pid, &childstat, 0) == -1 && + errno == EINTR); + } + if (intermediatefd != -1) + close(intermediatefd); close(fd); fd = -1; if (ret < 0) { -- 1.6.0.6

2 2

[libvirt] [RFC] Interface for disk hotadd/remove
by Wolfgang Mauerer 14 Aug '09

14 Aug '09

Hi, I'm currently interested in implementing hard disk hot-add and -remove support for qemu (as opposed to controller-based hotplugging), and this brings up the question how to best support this feature in libvirt. Many SCSI-Controllers in real machines, for instance, allow to add and remove disks (without adding or removing the controller itself) while the system is up and running, so it would be nice to emulate this in a virtual machine. I'm focusing on qemu on the backend side, but the problem is not related to this particular backend. Rather, the question is how to integrate such a feature best into libvirt. Before implementing the functionality, it would be great to hear the community's opinion which route to take wrt. the interface. Essentially, I can see two options: - Naturally, there are virDomain{At,De}tachDevice, but the pair implements drive-hotadding via adding a new controller with an attached hard disk to the system. By extending the XML description of the drive with a parameter that specifies whether controller- or disk-based hotplugging is to be performed, it would be possible to implement the desired functionality, whilst preserving compatibility with existing semantics. Removing the drive would then require another new parameter in the XML description to identify the drive on the controller, which does not really prettify the thing. - Extend the API with a new method (for instance virDomainDiskAttach) that takes a hard disk description, a controller identifier, and a parameter that identifies the disk on the controller. - (Theoretically, it would also be possible to implement media exchange for hard disks in qemu and re-use the media exchange infrastructure already present in libvirt for CD-ROMs, but since this possibility comes to use on real hardware only very occasionally, guest operating systems are typically not really prepared to handle this well) My preference would be to go for option 2, that is, implement a new API method. Would there be any obstacles against adding such a patch to mainline? Or is anyone already working on similar functionality? Or can this be done in a much simpler way I've missed? If not, then I'd send patches for more detailed review before long. Thanks, Wolfgang

2 3

[libvirt] Share storage using iscsi
by Łukasz Mierzwa 14 Aug '09

14 Aug '09

Hi, I'm trying to setup pool of machines (nodes) for virtual machines hosting and I got few question about shared storage. My main requirements are: 1. central management - I've got simple python app that stores information about all virtual machines and all nodes, this app needs to be able to manage volumes using libvirt API, so I need libvirt volume pools 2. live migration - I got shared storage with HA, I want to use it also for live migration in case one of nodes is dying or if I want to do some load balancing Right now I'm thinking about 2 machines with disks synchronized using drdb, both acting as a identical iscsi targets, iscsi HA will be provided by heartbeat. So I will end up with virtual IP pointing to working iscsi target, drdb should keep storage is sync. But: 1. I can't just use single iscsi LUN and export it as libvirt storage pool to each node, because no pool type would work that way, right? 2. http://libvirt.org/storage.html section "iSCSI volume pools" says: "Volumes must be pre-allocated on the iSCSI server, and cannot be created via the libvirt APIs." So even if I got one LUN per node and set it as iscsi volume pool I would need to create each volume on iscsi target. Libvirt can't manage volumes in such pool, it can only assign already created volumes to virtual machines, right? 3. So maybe my storage could be setup as LVM volume group and this lvm group would be managed as libvirt lvm volume pool on master (from heartbeat POV) iscsi target. I would create one logical volume per virtual machine, export this volume as a separate iscsi LUN, and use this LUN as iscsi volume for virtual machine. To create new virtual machine I would: a) create lvm volume on iscsi target using libvirt (so I don't have to Does it makes any sense? Are there better ways to *manage* volumes for virtual machines using iscsi? Thanks for any tips. Łukasz Mierzwa

1 0

[libvirt] [PATCH 1/2]: VirtualBox: Updated vboxNetworkCreateXML() and vboxNetworkDefineXML()
by Pritesh Kothari 14 Aug '09

14 Aug '09

Hi All, I have made some changes to the functions vboxNetworkCreateXML(), vboxNetworkDefineXML(), vboxNetworkUndefine() and vboxNetworkDestroy() to handle multiple host only interfaces as multiple host only interfaces are supported by VirtualBox 3.0 and greater. The patch's are as below: PATCH 1/2: Merged vboxNetworkCreateXML() and vboxNetworkDefineXML() and added code to handle multiple hostonly interfaces. PATCH 2/2: Merged vboxNetworkUndefine() and vboxNetworkDestroy() and added code to handle multiple hostonly interfaces. Regards, Pritesh

2 1

[libvirt] Connection refcounting fix?
by Chris Lalancette 14 Aug '09

14 Aug '09

DanB, is this the sort of idea you had to fix up the connection refcounting?

2 2

[libvirt] PATCH: Make UML/LXC drivers robust with bad NUMA data
by Daniel P. Berrange 13 Aug '09

13 Aug '09

commit e2052c24f39c71b3b8e92a983287f72176d73c77 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Thu Aug 13 11:56:31 2009 +0100 Make LXC / UML drivers robust against NUMA topology brokenness Some kernel versions expose broken NUMA topology for some machines. This causes the LXC/UML drivers to fail to start. QEMU driver was already fixed for this problem * src/lxc_conf.c: Log and ignore failure to populate NUMA info * src/uml_conf.c: Log and ignore failure to populate NUMA info diff --git a/src/lxc_conf.c b/src/lxc_conf.c index d06a024..fef60ba 100644 --- a/src/lxc_conf.c +++ b/src/lxc_conf.c @@ -30,6 +30,8 @@ #include "lxc_conf.h" #include "nodeinfo.h" #include "virterror_internal.h" +#include "logging.h" + #define VIR_FROM_THIS VIR_FROM_LXC @@ -46,8 +48,14 @@ virCapsPtr lxcCapsInit(void) 0, 0)) == NULL) goto no_memory; - if (nodeCapsInitNUMA(caps) < 0) - goto no_memory; + /* Some machines have problematic NUMA toplogy causing + * unexpected failures. We don't want to break the QEMU + * driver in this scenario, so log errors & carry on + */ + if (nodeCapsInitNUMA(caps) < 0) { + virCapabilitiesFreeNUMAInfo(caps); + VIR_WARN0("Failed to query host NUMA topology, disabling NUMA capabilities"); + } /* XXX shouldn't 'borrow' KVM's prefix */ virCapabilitiesSetMacPrefix(caps, (unsigned char []){ 0x52, 0x54, 0x00 }); diff --git a/src/uml_conf.c b/src/uml_conf.c index 48e05a8..4f756d4 100644 --- a/src/uml_conf.c +++ b/src/uml_conf.c @@ -45,6 +45,7 @@ #include "nodeinfo.h" #include "verify.h" #include "bridge.h" +#include "logging.h" #define VIR_FROM_THIS VIR_FROM_UML @@ -63,8 +64,14 @@ virCapsPtr umlCapsInit(void) { 0, 0)) == NULL) goto no_memory; - if (nodeCapsInitNUMA(caps) < 0) - goto no_memory; + /* Some machines have problematic NUMA toplogy causing + * unexpected failures. We don't want to break the QEMU + * driver in this scenario, so log errors & carry on + */ + if (nodeCapsInitNUMA(caps) < 0) { + virCapabilitiesFreeNUMAInfo(caps); + VIR_WARN0("Failed to query host NUMA topology, disabling NUMA capabilities"); + } if ((guest = virCapabilitiesAddGuest(caps, "uml", -- |: 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 2

[libvirt] [PATCH] unify popen/fopen qemu wrappers
by Paolo Bonzini 13 Aug '09

13 Aug '09

While reading Chris's code for fd migration I noticed the duplication between QEMUFilePopen and QEMUFileStdio. This fixes it, and makes qemu_fopen more similar qemu_fdopen. Signed-off-by: Paolo Bonzini <pbonzini(a)redhat.com> --- hw/hw.h | 2 +- migration-exec.c | 4 +- savevm.c | 99 +++++++++++++++++++++++++++-------------------------- 3 files changed, 53 insertions(+), 52 deletions(-) diff --git a/hw/hw.h b/hw/hw.h index 1e5783d..322f077 100644 --- a/hw/hw.h +++ b/hw/hw.h @@ -52,7 +52,7 @@ QEMUFile *qemu_fopen(const char *filename, const char *mode); QEMUFile *qemu_fopen_socket(int fd); QEMUFile *qemu_popen(FILE *popen_file, const char *mode); QEMUFile *qemu_popen_cmd(const char *command, const char *mode); -int qemu_popen_fd(QEMUFile *f); +int qemu_stdio_fd(QEMUFile *f); void qemu_fflush(QEMUFile *f); int qemu_fclose(QEMUFile *f); void qemu_put_buffer(QEMUFile *f, const uint8_t *buf, int size); diff --git a/migration-exec.c b/migration-exec.c index e3616be..235f3f7 100644 --- a/migration-exec.c +++ b/migration-exec.c @@ -117,7 +117,7 @@ static void exec_accept_incoming_migration(void *opaque) qemu_announce_self(); dprintf("successfully loaded vm state\n"); /* we've successfully migrated, close the fd */ - qemu_set_fd_handler2(qemu_popen_fd(f), NULL, NULL, NULL, NULL); + qemu_set_fd_handler2(qemu_stdio_fd(f), NULL, NULL, NULL, NULL); if (autostart) vm_start(); @@ -136,7 +136,7 @@ int exec_start_incoming_migration(const char *command) return -errno; } - qemu_set_fd_handler2(qemu_popen_fd(f), NULL, + qemu_set_fd_handler2(qemu_stdio_fd(f), NULL, exec_accept_incoming_migration, NULL, (void *)(unsigned long)f); diff --git a/savevm.c b/savevm.c index 570377f..975e7ab 100644 --- a/savevm.c +++ b/savevm.c @@ -172,11 +172,11 @@ struct QEMUFile { int has_error; }; -typedef struct QEMUFilePopen +typedef struct QEMUFileStdio { - FILE *popen_file; + FILE *stdio_file; QEMUFile *file; -} QEMUFilePopen; +} QEMUFileStdio; typedef struct QEMUFileSocket { @@ -206,16 +206,16 @@ static int socket_close(void *opaque) return 0; } -static int popen_put_buffer(void *opaque, const uint8_t *buf, int64_t pos, int size) +static int stdio_put_buffer(void *opaque, const uint8_t *buf, int64_t pos, int size) { - QEMUFilePopen *s = opaque; - return fwrite(buf, 1, size, s->popen_file); + QEMUFileStdio *s = opaque; + return fwrite(buf, 1, size, s->stdio_file); } -static int popen_get_buffer(void *opaque, uint8_t *buf, int64_t pos, int size) +static int stdio_get_buffer(void *opaque, uint8_t *buf, int64_t pos, int size) { - QEMUFilePopen *s = opaque; - FILE *fp = s->popen_file; + QEMUFileStdio *s = opaque; + FILE *fp = s->stdio_file; int bytes; do { @@ -225,31 +225,39 @@ static int popen_get_buffer(void *opaque, uint8_t *buf, int64_t pos, int size) return bytes; } -static int popen_close(void *opaque) +static int stdio_pclose(void *opaque) +{ + QEMUFileStdio *s = opaque; + pclose(s->stdio_file); + qemu_free(s); + return 0; +} + +static int stdio_fclose(void *opaque) { - QEMUFilePopen *s = opaque; - pclose(s->popen_file); + QEMUFileStdio *s = opaque; + fclose(s->stdio_file); qemu_free(s); return 0; } -QEMUFile *qemu_popen(FILE *popen_file, const char *mode) +QEMUFile *qemu_popen(FILE *stdio_file, const char *mode) { - QEMUFilePopen *s; + QEMUFileStdio *s; - if (popen_file == NULL || mode == NULL || (mode[0] != 'r' && mode[0] != 'w') || mode[1] != 0) { + if (stdio_file == NULL || mode == NULL || (mode[0] != 'r' && mode[0] != 'w') || mode[1] != 0) { fprintf(stderr, "qemu_popen: Argument validity check failed\n"); return NULL; } - s = qemu_mallocz(sizeof(QEMUFilePopen)); + s = qemu_mallocz(sizeof(QEMUFileStdio)); - s->popen_file = popen_file; + s->stdio_file = stdio_file; if(mode[0] == 'r') { - s->file = qemu_fopen_ops(s, NULL, popen_get_buffer, popen_close, NULL, NULL); + s->file = qemu_fopen_ops(s, NULL, stdio_get_buffer, stdio_pclose, NULL, NULL); } else { - s->file = qemu_fopen_ops(s, popen_put_buffer, NULL, popen_close, NULL, NULL); + s->file = qemu_fopen_ops(s, stdio_put_buffer, NULL, stdio_pclose, NULL, NULL); } return s->file; } @@ -266,13 +274,13 @@ QEMUFile *qemu_popen_cmd(const char *command, const char *mode) return qemu_popen(popen_file, mode); } -int qemu_popen_fd(QEMUFile *f) +int qemu_stdio_fd(QEMUFile *f) { - QEMUFilePopen *p; + QEMUFileStdio *p; int fd; - p = (QEMUFilePopen *)f->opaque; - fd = fileno(p->popen_file); + p = (QEMUFileStdio *)f->opaque; + fd = fileno(p->stdio_file); return fd; } @@ -286,53 +294,46 @@ QEMUFile *qemu_fopen_socket(int fd) return s->file; } -typedef struct QEMUFileStdio -{ - FILE *outfile; -} QEMUFileStdio; - static int file_put_buffer(void *opaque, const uint8_t *buf, int64_t pos, int size) { QEMUFileStdio *s = opaque; - fseek(s->outfile, pos, SEEK_SET); - fwrite(buf, 1, size, s->outfile); + fseek(s->stdio_file, pos, SEEK_SET); + fwrite(buf, 1, size, s->stdio_file); return size; } static int file_get_buffer(void *opaque, uint8_t *buf, int64_t pos, int size) { QEMUFileStdio *s = opaque; - fseek(s->outfile, pos, SEEK_SET); - return fread(buf, 1, size, s->outfile); -} - -static int file_close(void *opaque) -{ - QEMUFileStdio *s = opaque; - fclose(s->outfile); - qemu_free(s); - return 0; + fseek(s->stdio_file, pos, SEEK_SET); + return fread(buf, 1, size, s->stdio_file); } QEMUFile *qemu_fopen(const char *filename, const char *mode) { QEMUFileStdio *s; + if (mode == NULL || + (mode[0] != 'r' && mode[0] != 'w') || + mode[1] != 'b' || mode[2] != 0) { + fprintf(stderr, "qemu_fdopen: Argument validity check failed\n"); + return NULL; + } + s = qemu_mallocz(sizeof(QEMUFileStdio)); - s->outfile = fopen(filename, mode); - if (!s->outfile) + s->stdio_file = fopen(filename, mode); + if (!s->stdio_file) goto fail; - if (!strcmp(mode, "wb")) - return qemu_fopen_ops(s, file_put_buffer, NULL, file_close, NULL, NULL); - else if (!strcmp(mode, "rb")) - return qemu_fopen_ops(s, NULL, file_get_buffer, file_close, NULL, NULL); - + if(mode[0] == 'w') { + s->file = qemu_fopen_ops(s, file_put_buffer, NULL, stdio_fclose, NULL, NULL); + } else { + s->file = qemu_fopen_ops(s, NULL, file_get_buffer, stdio_fclose, NULL, NULL); + } + return s->file; fail: - if (s->outfile) - fclose(s->outfile); qemu_free(s); return NULL; } -- 1.6.2.5

2 2

[libvirt] virConnectClose() sometimes seg faults libvirt 0.7.0
by Kaitlin Rupert 13 Aug '09

13 Aug '09

Hi, I'm trying to track down a seg fault I'm seeing with the libvirt-0.7.0-3.fc12.x86_64 rawhide version of libvirt. I'm having trouble finding the exact set of steps, as I see the behavior when interacting with the libvirt-cim providers. Here's the stack trace I was able to get. Looks like the seg fault actually occurs in netcf, which I haven't looked at much. Has anyone seen this? Program received signal SIGSEGV, Segmentation fault. 0x00007fffe4004383 in ?? () (gdb) bt #0 0x00007fffe4004383 in ?? () #1 0x00007ffff6a5cbfc in xmlHashScanFull__internal_alias ( table=0x7fffe4005f50, f=<value optimized out>, data=<value optimized out>) at hash.c:849 #2 0x00007ffff6a5cc4c in xmlHashScan__internal_alias (table=0x7fffe400ef10, f=<value optimized out>, data=<value optimized out>) at hash.c:818 #3 0x00007ffff6ff5104 in xsltShutdownExts () from /usr/lib64/libxslt.so.1 #4 0x00007ffff6fe3e3d in xsltFreeStylesheet () from /usr/lib64/libxslt.so.1 #5 0x00007ffff7452961 in ?? () from /usr/lib64/libnetcf.so.1 #6 0x00007ffff74501d1 in ncf_close () from /usr/lib64/libnetcf.so.1 #7 0x000000000045bb8f in interfaceCloseInterface (conn=0x7fffe4000a90) at interface_driver.c:164 #8 0x00007ffff7af46a6 in virConnectClose (conn=0x7fffe4000a90) at libvirt.c:1244 #9 0x0000000000412828 in qemudFreeClient (client=can't compute CFA for this frame ) at qemud.c:2146 #10 qemudRunLoop (client=can't compute CFA for this frame ) at qemud.c:2219 #11 0x0000000000415b43 in main (argc=<value optimized out>, argv=<value optimized out>) at qemud.c:2994 --------------------------- Relevant part of the log: 17:13:15.930: debug : do_open:1056 : interface driver 2 Interface returned SUCCESS 17:13:15.930: debug : do_open:1076 : storage driver 0 Test returned DECLINED 17:13:15.930: debug : do_open:1076 : storage driver 1 remote returned DECLINED 17:13:15.930: debug : do_open:1076 : storage driver 2 storage returned SUCCESS 17:13:15.930: debug : do_open:1096 : node driver 0 Test returned DECLINED 17:13:15.930: debug : do_open:1096 : node driver 1 remote returned DECLINED 17:13:15.930: debug : do_open:1096 : node driver 2 halDeviceMonitor returned SUCCESS 17:13:15.930: debug : qemudDispatchClientFailure:1394 : Deregistering to relay remote events 17:13:15.930: debug : virConnectDomainEventDeregister:8582 : conn=0x7fffe014dd00, cb=0x41d620 17:13:15.930: debug : virConnectClose:1232 : conn=0x7fffe014dd00 -- Kaitlin Rupert IBM Linux Technology Center kaitlin(a)linux.vnet.ibm.com

2 1

[libvirt] libvirt-java in the maven repo
by Bryan Kearney 13 Aug '09

13 Aug '09

The java bindings made it into the central repo (http://repo1.maven.org/maven2/org/libvirt/) So.. they should be replicated to your favorite mirror "real soon". -- bk

2 1

[libvirt] [PATCH] Compressed save image format for Qemu.
by Chris Lalancette 13 Aug '09

13 Aug '09

Implement a compressed save image format for qemu. While ideally we would have the choice between compressed/non-compressed available to the libvirt API, unfortunately there is no "flags" parameter to the virDomainSave() API. Therefore, implement this as a qemu.conf option. Both gzip and bzip2 are implemented, and it should be very easy to implement additional compression methods. One open question is if/how we should detect the gzip and bzip2 binaries. One way to do it is to do compile-time setting of the paths (via configure.in), but that doesn't seem like a great thing to do. Another solution (my preferred solution) is not to detect at all; when we go to run the commands that need them, if they aren't available, or aren't available in one of the standard paths, then we'll fail. Maybe somebody else has another option or opinion, though. In the future, we'll have a more robust (managed) save/restore API, at which time we can expose this functionality properly in the API. V2: get rid of redundant dd command and just use >> to append data. V3: Add back the missing pieces for the enum and bumping the save version. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu.conf | 10 ++++++ src/qemu_conf.c | 11 +++++++ src/qemu_conf.h | 2 + src/qemu_driver.c | 85 ++++++++++++++++++++++++++++++++++++++++++++++++---- 4 files changed, 101 insertions(+), 7 deletions(-) diff --git a/src/qemu.conf b/src/qemu.conf index 653f487..86dcc9d 100644 --- a/src/qemu.conf +++ b/src/qemu.conf @@ -129,3 +129,13 @@ # "/dev/ptmx", "/dev/kvm", "/dev/kqemu", # "/dev/rtc", "/dev/hpet", "/dev/net/tun", #] + +# The default format for Qemu/KVM guest save images is raw; that is, the +# memory from the domain is dumped out directly to a file. If you have +# guests with a large amount of memory, however, this can take up quite +# a bit of space. If you would like to compress the images while they +# are being saved to disk, you can also set "gzip" or "bzip2" for the +# save_image_format. Note that this means you slow down the process +# of saving a domain in order to save disk space. +# +# save_image_format = "raw" diff --git a/src/qemu_conf.c b/src/qemu_conf.c index 7ca5a15..ed87e13 100644 --- a/src/qemu_conf.c +++ b/src/qemu_conf.c @@ -280,6 +280,17 @@ int qemudLoadDriverConfig(struct qemud_driver *driver, driver->cgroupDeviceACL[i] = NULL; } + p = virConfGetValue (conf, "save_image_format"); + CHECK_TYPE ("save_image_format", VIR_CONF_STRING); + if (p && p->str) { + VIR_FREE(driver->saveImageFormat); + if (!(driver->saveImageFormat = strdup(p->str))) { + virReportOOMError(NULL); + virConfFree(conf); + return -1; + } + } + virConfFree (conf); return 0; } diff --git a/src/qemu_conf.h b/src/qemu_conf.h index 8f4ef6a..e34baab 100644 --- a/src/qemu_conf.h +++ b/src/qemu_conf.h @@ -111,6 +111,8 @@ struct qemud_driver { char *securityDriverName; virSecurityDriverPtr securityDriver; + + char *saveImageFormat; }; diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 20906ef..8d700bf 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -3411,18 +3411,26 @@ static char *qemudEscapeShellArg(const char *in) } #define QEMUD_SAVE_MAGIC "LibvirtQemudSave" -#define QEMUD_SAVE_VERSION 1 +#define QEMUD_SAVE_VERSION 2 + +enum qemud_save_formats { + QEMUD_SAVE_FORMAT_RAW, + QEMUD_SAVE_FORMAT_GZIP, + QEMUD_SAVE_FORMAT_BZIP2, +}; struct qemud_save_header { char magic[sizeof(QEMUD_SAVE_MAGIC)-1]; int version; int xml_len; int was_running; - int unused[16]; + char compressed; + int unused[15]; }; static int qemudDomainSave(virDomainPtr dom, - const char *path) { + const char *path) +{ struct qemud_driver *driver = dom->conn->privateData; virDomainObjPtr vm; char *command = NULL; @@ -3433,11 +3441,26 @@ static int qemudDomainSave(virDomainPtr dom, struct qemud_save_header header; int ret = -1; virDomainEventPtr event = NULL; + int internalret; memset(&header, 0, sizeof(header)); memcpy(header.magic, QEMUD_SAVE_MAGIC, sizeof(header.magic)); header.version = QEMUD_SAVE_VERSION; + if (driver->saveImageFormat == NULL) + header.compressed = QEMUD_SAVE_FORMAT_RAW; + else if (STREQ(driver->saveImageFormat, "raw")) + header.compressed = QEMUD_SAVE_FORMAT_RAW; + else if (STREQ(driver->saveImageFormat, "gzip")) + header.compressed = QEMUD_SAVE_FORMAT_GZIP; + else if (STREQ(driver->saveImageFormat, "bzip2")) + header.compressed = QEMUD_SAVE_FORMAT_BZIP2; + else { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, + "%s", _("Invalid save image format specified in configuration file")); + return -1; + } + qemuDriverLock(driver); vm = virDomainFindByUUID(&driver->domains, dom->uuid); @@ -3510,11 +3533,25 @@ static int qemudDomainSave(virDomainPtr dom, virReportOOMError(dom->conn); goto cleanup; } - if (virAsprintf(&command, "migrate \"exec:" - "dd of='%s' oflag=append conv=notrunc 2>/dev/null" - "\"", safe_path) == -1) { + + if (header.compressed == QEMUD_SAVE_FORMAT_RAW) + internalret = virAsprintf(&command, "migrate \"exec:" + "dd of='%s' oflag=append conv=notrunc 2>/dev/null" + "\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_GZIP) + internalret = virAsprintf(&command, "migrate \"exec:" + "gzip -c >> '%s' 2>/dev/null\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_BZIP2) + internalret = virAsprintf(&command, "migrate \"exec:" + "bzip2 -c >> '%s' 2>/dev/null\"", safe_path); + else { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_INTERNAL_ERROR, + _("Invalid compress format %d"), + header.compressed); + goto cleanup; + } + if (internalret < 0) { virReportOOMError(dom->conn); - command = NULL; goto cleanup; } @@ -4035,6 +4072,9 @@ static int qemudDomainRestore(virConnectPtr conn, char *xml = NULL; struct qemud_save_header header; virDomainEventPtr event = NULL; + int intermediatefd = -1; + pid_t intermediate_pid = -1; + int childstat; qemuDriverLock(driver); /* Verify the header and read the XML */ @@ -4124,8 +4164,39 @@ static int qemudDomainRestore(virConnectPtr conn, } def = NULL; + if (header.version == 2) { + const char *intermediate_argv[3] = { NULL, "-dc", NULL }; + if (header.compressed == QEMUD_SAVE_FORMAT_GZIP) + intermediate_argv[0] = "gzip"; + else if (header.compressed == QEMUD_SAVE_FORMAT_BZIP2) + intermediate_argv[0] = "bzip2"; + else if (header.compressed != QEMUD_SAVE_FORMAT_RAW) { + qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, + _("Unknown compressed save format %d"), + header.compressed); + goto cleanup; + } + if (intermediate_argv[0] != NULL) { + intermediatefd = fd; + fd = -1; + if (virExec(conn, intermediate_argv, NULL, NULL, + &intermediate_pid, intermediatefd, &fd, NULL, 0) < 0) { + qemudReportError(conn, NULL, NULL, VIR_ERR_INTERNAL_ERROR, + _("Failed to start decompression binary %s"), + intermediate_argv[0]); + goto cleanup; + } + } + } /* Set the migration source and start it up. */ ret = qemudStartVMDaemon(conn, driver, vm, "stdio", fd); + if (intermediate_pid != -1) { + /* Wait for intermediate process to exit */ + while (waitpid(intermediate_pid, &childstat, 0) == -1 && + errno == EINTR); + } + if (intermediatefd != -1) + close(intermediatefd); close(fd); fd = -1; if (ret < 0) { -- 1.6.0.6

3 4

[libvirt] [PATCH] Fix some URLs in virsh manpage
by Mark McLoughlin 13 Aug '09

13 Aug '09

* docs/virsh.pod: don't reference format.html anymore, reference the formatdomain.html etc. pages * virsh.1: re-generate --- docs/virsh.pod | 6 +++--- virsh.1 | 8 ++++---- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/docs/virsh.pod b/docs/virsh.pod index 9530ba6..10bb991 100644 --- a/docs/virsh.pod +++ b/docs/virsh.pod @@ -140,7 +140,7 @@ we are currently connected to. This includes a section on the host capabilities in terms of CPU and features, and a set of description for each kind of guest which can be virtualized. For a more complete description see: - L<http://libvirt.org/format.html#Capa1> + L<http://libvirt.org/formatcaps.html> The XML also show the NUMA topology information if available. =item B<list> @@ -424,7 +424,7 @@ is not available the processes will provide an exit code of 1. The following commands manipulate devices associated to domains. The domain-id can be specified as an short integer, a name or a full UUID. To better understand the values allowed as options for the command -reading the documentation at L<http://libvirt.org/format.html> on the +reading the documentation at L<http://libvirt.org/formatdomain.html> on the format of the device sections to get the most accurate set of accepted values. =over 4 @@ -476,7 +476,7 @@ if more than one are present on the domain. The following commands manipulate networks. Libvirt has the capability to define virtual networks which can then be used by domains and linked to actual network devices. For more detailed information about this feature -see the documentation at L<http://libvirt.org/format.html#Net1> . A lot +see the documentation at L<http://libvirt.org/formatnetwork.html> . A lot of the command for virtual networks are similar to the one used for domains, but the way to name a virtual network is either by its name or UUID. diff --git a/virsh.1 b/virsh.1 index ca03cc1..0a5b1c1 100644 --- a/virsh.1 +++ b/virsh.1 @@ -132,7 +132,7 @@ .\" ======================================================================== .\" .IX Title "VIRSH 1" -.TH VIRSH 1 "2009-07-02" "libvirt-0.6.4" "Virtualization Support" +.TH VIRSH 1 "2009-08-11" "libvirt-0.7.0" "Virtualization Support" .\" For nroff, turn off justification. Always turn off hyphenation; it makes .\" way too many mistakes in technical documents. .if n .ad l @@ -263,7 +263,7 @@ we are currently connected to. This includes a section on the host capabilities in terms of \s-1CPU\s0 and features, and a set of description for each kind of guest which can be virtualized. For a more complete description see: - <http://libvirt.org/format.html#Capa1> + <http://libvirt.org/formatcaps.html> The \s-1XML\s0 also show the \s-1NUMA\s0 topology information if available. .IP "\fBlist\fR" 4 .IX Item "list" @@ -501,7 +501,7 @@ is not available the processes will provide an exit code of 1. The following commands manipulate devices associated to domains. The domain-id can be specified as an short integer, a name or a full \s-1UUID\s0. To better understand the values allowed as options for the command -reading the documentation at <http://libvirt.org/format.html> on the +reading the documentation at <http://libvirt.org/formatdomain.html> on the format of the device sections to get the most accurate set of accepted values. .IP "\fBattach-device\fR \fIdomain-id\fR \fI\s-1FILE\s0\fR" 4 .IX Item "attach-device domain-id FILE" @@ -542,7 +542,7 @@ if more than one are present on the domain. The following commands manipulate networks. Libvirt has the capability to define virtual networks which can then be used by domains and linked to actual network devices. For more detailed information about this feature -see the documentation at <http://libvirt.org/format.html#Net1> . A lot +see the documentation at <http://libvirt.org/formatnetwork.html> . A lot of the command for virtual networks are similar to the one used for domains, but the way to name a virtual network is either by its name or \s-1UUID\s0. .IP "\fBnet-autostart\fR \fInetwork\fR optional \fI\-\-disable\fR" 4 -- 1.6.2.5

2 1

Re: [libvirt] Unable to create cgroup for driver
by Daniel P. Berrange 12 Aug '09

12 Aug '09

On Tue, Aug 11, 2009 at 02:45:06AM -0700, Shahar Klein wrote: > Jun > To avoid just configure --without-lxc > > Daniel here is the back-trace > (Didn't have time to explore) > > [root@rain8 libvirt-0.7.0]# ./qemud/libvirtd > 16:11:02.700: warning : qemudStartup:521 : Unable to create cgroup for driver: No such device or address > Segmentation fault (core dumped) > > gdb ./qemud/.libs/libvirtd ../core.19690 > > (gdb) where > #0 virDomainEventCallbackListFree (list=0x0) at domain_event.c:41 > #1 0x000000000043ecee in lxcShutdown () at lxc_driver.c:1582 > #2 0x000000000043f122 in lxcStartup (privileged=1) at lxc_driver.c:1523 > #3 0x00007f05103e03c2 in virStateInitialize (privileged=1) at libvirt.c:785 > #4 0x00000000004134cf in main (argc=6776272, argv=<value optimized out>) at qemud.c:2970 Try out the following patch which ought to fix this problem Regards, Daniel diff --git a/src/domain_conf.c b/src/domain_conf.c index 2301a96..dd2b361 100644 --- a/src/domain_conf.c +++ b/src/domain_conf.c @@ -532,6 +532,9 @@ void virDomainObjListFree(virDomainObjListPtr vms) { unsigned int i; + if (!vms) + return; + for (i = 0 ; i < vms->count ; i++) virDomainObjFree(vms->objs[i]); diff --git a/src/domain_event.c b/src/domain_event.c index daa6fd5..0fa2822 100644 --- a/src/domain_event.c +++ b/src/domain_event.c @@ -38,6 +38,9 @@ void virDomainEventCallbackListFree(virDomainEventCallbackListPtr list) { int i; + if (!list) + return; + for (i=0; i<list->count; i++) { virFreeCallback freecb = list->callbacks[i]->freecb; if (freecb) diff --git a/src/lxc_driver.c b/src/lxc_driver.c index a9c4f79..bd0cf0e 100644 --- a/src/lxc_driver.c +++ b/src/lxc_driver.c @@ -1439,10 +1439,12 @@ static int lxcStartup(int privileged) lxcDriverLock(lxc_driver); /* Check that this is a container enabled kernel */ - if(lxcContainerAvailable(0) < 0) + if (lxcContainerAvailable(0) < 0) { + VIR_INFO0("LXC support not available in this kernel, disabling driver"); goto cleanup; + } - if(VIR_ALLOC(lxc_driver->domainEventCallbacks) < 0) + if (VIR_ALLOC(lxc_driver->domainEventCallbacks) < 0) goto cleanup; if (!(lxc_driver->domainEventQueue = virDomainEventQueueNew())) goto cleanup; -- |: 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 :|

3 2

Re: [libvirt] Unable to create cgroup for driver
by Daniel P. Berrange 12 Aug '09

12 Aug '09

On Tue, Aug 11, 2009 at 04:42:32AM -0700, Shahar Klein wrote: > I looked into the code > seems like a successfull clone() with the CLONE_NEWUSER (0x10000000)means lxc is supported > however man 2 clone does not state this flag nor a flag with this value The man pages are useless. This stuff was added to kernels around 2.6.25, so either you have too old a kernel, or your kernel config has it disabled Regards, Daniel -- |: 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 :|

1 0

[libvirt] Unable to create cgroup for driver
by Jun Koi 12 Aug '09

12 Aug '09

Hi, The libvirt 0.7 has a following error when i start libvirtd: warning : qemudStartup:521 : Unable to create cgroup for driver: No such device or address Then it fails to start. How can I fix this? Thanks, J

3 6

[libvirt] [PATCH 00/20] Add support for (qcow*) volume encryption.
by Miloslav Trmač 12 Aug '09

12 Aug '09

Hello, the following patches add full support for qcow/qcow2 volume encryption, assuming a client that supports it. (Main changes since the previous version: * Use a separate API for "secret" management * Auto-generate the encryption format and passphrase if not supplied See the specific patch change logs for more details; patches without change logs are entirely new.) New XML tags are defined to represent encryption parameters (currently format and passphrase, more can be added in the future), e.g. <encryption format='qcow'> <secret type='passphrase' secret_id='724d95f2-0ed2-6ff9-84d0-0f3d1618428d'/> </encryption> The <encryption> tag can be added to a <volume> node passed to virStorageVolCreateXML() to create an encrypted volume, or to a <disk> node inside a <domain> to specify what encryption parameters to use for a domain. secret_id above refers to a separately-managed secret, which was set using virSecretSetValue(). Other properties of the secret can be managed using an XML representation. Detailed documentation of the formats and features is inside the patches.

3 25

[libvirt] [PATCH] add file descriptor migration
by Paolo Bonzini 12 Aug '09

12 Aug '09

This is the patch from Chris, with the accept removed and rebased above my previous cleanup. Signed-off-by: Paolo Bonzini <pbonzini(a)redhat.com> Cc: Chris Lalancette <clalance(a)redhat.com> Cc: Avi Kivity <avi(a)redhat.com> --- Makefile | 2 +- hw/hw.h | 1 + migration-fd.c | 137 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++ migration.c | 4 ++ migration.h | 7 +++ savevm.c | 28 +++++++++++ 6 files changed, 178 insertions(+), 1 deletions(-) create mode 100644 migration-fd.c diff --git a/Makefile b/Makefile index 5279504..c7ff0be 100644 --- a/Makefile +++ b/Makefile @@ -93,7 +93,7 @@ obj-y += qdev.o qdev-properties.o ssi.o obj-$(CONFIG_BRLAPI) += baum.o obj-$(CONFIG_WIN32) += tap-win32.o -obj-$(CONFIG_POSIX) += migration-exec.o +obj-$(CONFIG_POSIX) += migration-exec.o migration-fd.o audio/audio.o audio/fmodaudio.o: QEMU_CFLAGS += $(FMOD_CFLAGS) diff --git a/hw/hw.h b/hw/hw.h index 322f077..91bf800 100644 --- a/hw/hw.h +++ b/hw/hw.h @@ -49,6 +49,7 @@ QEMUFile *qemu_fopen_ops(void *opaque, QEMUFilePutBufferFunc *put_buffer, QEMUFileRateLimit *rate_limit, QEMUFileSetRateLimit *set_rate_limit); QEMUFile *qemu_fopen(const char *filename, const char *mode); +QEMUFile *qemu_fdopen(int fd, const char *mode); QEMUFile *qemu_fopen_socket(int fd); QEMUFile *qemu_popen(FILE *popen_file, const char *mode); QEMUFile *qemu_popen_cmd(const char *command, const char *mode); diff --git a/migration-fd.c b/migration-fd.c new file mode 100644 index 0000000..794c2ac --- /dev/null +++ b/migration-fd.c @@ -0,0 +1,137 @@ +/* + * QEMU live migration via generic fd + * + * Copyright Red Hat, Inc. 2009 + * + * Authors: + * Chris Lalancette <clalance(a)redhat.com> + * + * This work is licensed under the terms of the GNU GPL, version 2. See + * the COPYING file in the top-level directory. + * + */ + +#include "qemu-common.h" +#include "qemu_socket.h" +#include "migration.h" +#include "monitor.h" +#include "qemu-char.h" +#include "sysemu.h" +#include "buffered_file.h" +#include "block.h" +#include "qemu_socket.h" + +//#define DEBUG_MIGRATION_FD + +#ifdef DEBUG_MIGRATION_FD +#define dprintf(fmt, ...) \ + do { printf("migration-fd: " fmt, ## __VA_ARGS__); } while (0) +#else +#define dprintf(fmt, ...) \ + do { } while (0) +#endif + +static int fd_errno(FdMigrationState *s) +{ + return errno; +} + +static int fd_write(FdMigrationState *s, const void * buf, size_t size) +{ + return write(s->fd, buf, size); +} + +static int fd_close(FdMigrationState *s) +{ + dprintf("fd_close\n"); + if (s->fd != -1) { + close(s->fd); + s->fd = -1; + } + return 0; +} + +MigrationState *fd_start_outgoing_migration(Monitor *mon, + const char *fdname, + int64_t bandwidth_limit, + int detach) +{ + FdMigrationState *s; + + s = qemu_mallocz(sizeof(*s)); + + s->fd = monitor_get_fd(mon, fdname); + if (s->fd == -1) { + dprintf("fd_migration: invalid file descriptor identifier\n"); + goto err_after_alloc; + } + + if (fcntl(s->fd, F_SETFD, O_NONBLOCK) == -1) { + dprintf("Unable to set nonblocking mode on file descriptor\n"); + goto err_after_open; + } + + s->get_error = fd_errno; + s->write = fd_write; + s->close = fd_close; + s->mig_state.cancel = migrate_fd_cancel; + s->mig_state.get_status = migrate_fd_get_status; + s->mig_state.release = migrate_fd_release; + + s->state = MIG_STATE_ACTIVE; + s->mon_resume = NULL; + s->bandwidth_limit = bandwidth_limit; + + if (!detach) + migrate_fd_monitor_suspend(s); + + migrate_fd_connect(s); + return &s->mig_state; + +err_after_open: + close(s->fd); +err_after_alloc: + qemu_free(s); + return NULL; +} + +static void fd_accept_incoming_migration(void *opaque) +{ + QEMUFile *f = opaque; + int ret; + + ret = qemu_loadvm_state(f); + if (ret < 0) { + fprintf(stderr, "load of migration failed\n"); + goto err; + } + qemu_announce_self(); + dprintf("successfully loaded vm state\n"); + /* we've successfully migrated, close the fd */ + qemu_set_fd_handler2(qemu_stdio_fd(f), NULL, NULL, NULL, NULL); + if (autostart) + vm_start(); + +err: + qemu_fclose(f); +} + +int fd_start_incoming_migration(const char *infd) +{ + int fd; + QEMUFile *f; + + dprintf("Attempting to start an incoming migration via fd\n"); + + fd = strtol(infd, NULL, 0); + f = qemu_fdopen(fd, "rb"); + if(f == NULL) { + dprintf("Unable to apply qemu wrapper to file descriptor\n"); + return -errno; + } + + qemu_set_fd_handler2(fd, NULL, fd_accept_incoming_migration, NULL, + (void *)(unsigned long)f); + + return 0; +} diff --git a/migration.c b/migration.c index ee64d41..0ca4399 100644 --- a/migration.c +++ b/migration.c @@ -43,6 +43,8 @@ void qemu_start_incoming_migration(const char *uri) #if !defined(WIN32) else if (strstart(uri, "exec:", &p)) exec_start_incoming_migration(p); + else if (strstart(uri, "fd:", &p)) + fd_start_incoming_migration(p); #endif else fprintf(stderr, "unknown migration protocol: %s\n", uri); @@ -58,6 +60,8 @@ void do_migrate(Monitor *mon, int detach, const char *uri) #if !defined(WIN32) else if (strstart(uri, "exec:", &p)) s = exec_start_outgoing_migration(p, max_throttle, detach); + else if (strstart(uri, "fd:", &p)) + s = fd_start_outgoing_migration(mon, p, max_throttle, detach); #endif else monitor_printf(mon, "unknown migration protocol: %s\n", uri); diff --git a/migration.h b/migration.h index 37c7f8e..5033d02 100644 --- a/migration.h +++ b/migration.h @@ -73,6 +73,13 @@ MigrationState *tcp_start_outgoing_migration(const char *host_port, int64_t bandwidth_limit, int detach); +int fd_start_incoming_migration(const char *path); + +MigrationState *fd_start_outgoing_migration(Monitor *mon, + const char *fdname, + int64_t bandwidth_limit, + int detach); + void migrate_fd_monitor_suspend(FdMigrationState *s); void migrate_fd_error(FdMigrationState *s); diff --git a/savevm.c b/savevm.c index 975e7ab..4575653 100644 --- a/savevm.c +++ b/savevm.c @@ -285,6 +285,34 @@ int qemu_stdio_fd(QEMUFile *f) return fd; } +QEMUFile *qemu_fdopen(int fd, const char *mode) +{ + QEMUFileStdio *s; + + if (mode == NULL || + (mode[0] != 'r' && mode[0] != 'w') || + mode[1] != 'b' || mode[2] != 0) { + fprintf(stderr, "qemu_fdopen: Argument validity check failed\n"); + return NULL; + } + + s = qemu_mallocz(sizeof(QEMUFileStdio)); + s->stdio_file = fdopen(fd, mode); + if (!s->stdio_file) + goto fail; + + if(mode[0] == 'r') { + s->file = qemu_fopen_ops(s, NULL, stdio_get_buffer, stdio_fclose, NULL, NULL); + } else { + s->file = qemu_fopen_ops(s, stdio_put_buffer, NULL, stdio_fclose, NULL, NULL); + } + return s->file; + +fail: + qemu_free(s); + return NULL; +} + QEMUFile *qemu_fopen_socket(int fd) { QEMUFileSocket *s = qemu_mallocz(sizeof(QEMUFileSocket)); -- 1.6.2.5

1 0

[libvirt] [PATCH] Split virDomainMigrate into functions.
by Chris Lalancette 11 Aug '09

11 Aug '09

Re-factor virDomainMigrate to split out the version 1 and version 2 protocols into their own functions. In reality, the two versions share very little in common, so forcing them together in the same function was just confusing. This will also make adding tunnelled migration easier. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/libvirt.c | 258 ++++++++++++++++++++++++++++++++++----------------------- 1 files changed, 155 insertions(+), 103 deletions(-) diff --git a/src/libvirt.c b/src/libvirt.c index 9fd864d..6c1cc3d 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -2874,6 +2874,146 @@ error: } +static virDomainPtr +virDomainMigrateVersion1 (virDomainPtr domain, + virConnectPtr dconn, + unsigned long flags, + const char *dname, + const char *uri, + unsigned long bandwidth) +{ + virDomainPtr ddomain = NULL; + char *uri_out = NULL; + char *cookie = NULL; + int cookielen = 0; + + /* Prepare the migration. + * + * The destination host may return a cookie, or leave cookie as + * NULL. + * + * The destination host MUST set uri_out if uri_in is NULL. + * + * If uri_in is non-NULL, then the destination host may modify + * the URI by setting uri_out. If it does not wish to modify + * the URI, it should leave uri_out as NULL. + */ + if (dconn->driver->domainMigratePrepare + (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, + bandwidth) == -1) + goto done; + + if (uri == NULL && uri_out == NULL) { + virLibConnError (domain->conn, VIR_ERR_INTERNAL_ERROR, + _("domainMigratePrepare did not set uri")); + goto done; + } + if (uri_out) + uri = uri_out; /* Did domainMigratePrepare change URI? */ + assert (uri != NULL); + + /* Perform the migration. The driver isn't supposed to return + * until the migration is complete. + */ + if (domain->conn->driver->domainMigratePerform + (domain, cookie, cookielen, uri, flags, dname, bandwidth) == -1) + goto done; + + /* Get the destination domain and return it or error. + * 'domain' no longer actually exists at this point + * (or so we hope), but we still use the object in memory + * in order to get the name. + */ + dname = dname ? dname : domain->name; + if (dconn->driver->domainMigrateFinish) + ddomain = dconn->driver->domainMigrateFinish + (dconn, dname, cookie, cookielen, uri, flags); + else + ddomain = virDomainLookupByName (dconn, dname); + + done: + VIR_FREE (uri_out); + VIR_FREE (cookie); + return ddomain; +} + +static virDomainPtr +virDomainMigrateVersion2 (virDomainPtr domain, + virConnectPtr dconn, + unsigned long flags, + const char *dname, + const char *uri, + unsigned long bandwidth) +{ + virDomainPtr ddomain = NULL; + char *uri_out = NULL; + char *cookie = NULL; + char *dom_xml = NULL; + int cookielen = 0, ret; + + /* Prepare the migration. + * + * The destination host may return a cookie, or leave cookie as + * NULL. + * + * The destination host MUST set uri_out if uri_in is NULL. + * + * If uri_in is non-NULL, then the destination host may modify + * the URI by setting uri_out. If it does not wish to modify + * the URI, it should leave uri_out as NULL. + */ + + /* In version 2 of the protocol, the prepare step is slightly + * different. We fetch the domain XML of the source domain + * and pass it to Prepare2. + */ + if (!domain->conn->driver->domainDumpXML) { + virLibConnError (domain->conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__); + return NULL; + } + dom_xml = domain->conn->driver->domainDumpXML (domain, + VIR_DOMAIN_XML_SECURE); + if (!dom_xml) + return NULL; + + ret = dconn->driver->domainMigratePrepare2 + (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, + bandwidth, dom_xml); + VIR_FREE (dom_xml); + if (ret == -1) + goto done; + + if (uri == NULL && uri_out == NULL) { + virLibConnError (domain->conn, VIR_ERR_INTERNAL_ERROR, + _("domainMigratePrepare2 did not set uri")); + goto done; + } + if (uri_out) + uri = uri_out; /* Did domainMigratePrepare2 change URI? */ + assert (uri != NULL); + + /* Perform the migration. The driver isn't supposed to return + * until the migration is complete. + */ + ret = domain->conn->driver->domainMigratePerform + (domain, cookie, cookielen, uri, flags, dname, bandwidth); + if (ret == -1) + goto done; + + /* In version 2 of the migration protocol, we pass the + * status code from the sender to the destination host, + * so it can do any cleanup if the migration failed. + */ + dname = dname ? dname : domain->name; + ddomain = dconn->driver->domainMigrateFinish2 + (dconn, dname, cookie, cookielen, uri, flags, ret); + + done: + VIR_FREE (uri_out); + VIR_FREE (cookie); + return ddomain; +} + /** * virDomainMigrate: * @domain: a domain object @@ -2930,140 +3070,52 @@ virDomainMigrate (virDomainPtr domain, const char *uri, unsigned long bandwidth) { - virConnectPtr conn; virDomainPtr ddomain = NULL; - char *uri_out = NULL; - char *cookie = NULL; - char *dom_xml = NULL; - int cookielen = 0, ret, version = 0; DEBUG("domain=%p, dconn=%p, flags=%lu, dname=%s, uri=%s, bandwidth=%lu", domain, dconn, flags, NULLSTR(dname), NULLSTR(uri), bandwidth); virResetLastError(); + /* First checkout the source */ if (!VIR_IS_CONNECTED_DOMAIN (domain)) { virLibDomainError(NULL, VIR_ERR_INVALID_DOMAIN, __FUNCTION__); return NULL; } - conn = domain->conn; /* Source connection. */ - if (!VIR_IS_CONNECT (dconn)) { - virLibConnError (conn, VIR_ERR_INVALID_CONN, __FUNCTION__); + if (domain->conn->flags & VIR_CONNECT_RO) { + virLibDomainError(domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); goto error; } - if (domain->conn->flags & VIR_CONNECT_RO) { - virLibDomainError(domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); + /* Now checkout the destination */ + if (!VIR_IS_CONNECT (dconn)) { + virLibConnError (domain->conn, VIR_ERR_INVALID_CONN, __FUNCTION__); goto error; } if (dconn->flags & VIR_CONNECT_RO) { - /* NB, delibrately report error against source object, not dest here */ - virLibDomainError(domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); + /* NB, deliberately report error against source object, not dest */ + virLibDomainError (domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); goto error; } /* Check that migration is supported by both drivers. */ - if (VIR_DRV_SUPPORTS_FEATURE (conn->driver, conn, + if (VIR_DRV_SUPPORTS_FEATURE (domain->conn->driver, domain->conn, VIR_DRV_FEATURE_MIGRATION_V1) && VIR_DRV_SUPPORTS_FEATURE (dconn->driver, dconn, VIR_DRV_FEATURE_MIGRATION_V1)) - version = 1; - else if (VIR_DRV_SUPPORTS_FEATURE (conn->driver, conn, + ddomain = virDomainMigrateVersion1 (domain, dconn, flags, dname, uri, bandwidth); + else if (VIR_DRV_SUPPORTS_FEATURE (domain->conn->driver, domain->conn, VIR_DRV_FEATURE_MIGRATION_V2) && VIR_DRV_SUPPORTS_FEATURE (dconn->driver, dconn, VIR_DRV_FEATURE_MIGRATION_V2)) - version = 2; + ddomain = virDomainMigrateVersion2 (domain, dconn, flags, dname, uri, bandwidth); else { - virLibConnError (conn, VIR_ERR_NO_SUPPORT, __FUNCTION__); + virLibConnError (domain->conn, VIR_ERR_NO_SUPPORT, __FUNCTION__); goto error; } - /* Prepare the migration. - * - * The destination host may return a cookie, or leave cookie as - * NULL. - * - * The destination host MUST set uri_out if uri_in is NULL. - * - * If uri_in is non-NULL, then the destination host may modify - * the URI by setting uri_out. If it does not wish to modify - * the URI, it should leave uri_out as NULL. - */ - if (version == 1) { - ret = dconn->driver->domainMigratePrepare - (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, - bandwidth); - if (ret == -1) goto done; - if (uri == NULL && uri_out == NULL) { - virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, - _("domainMigratePrepare did not set uri")); - goto done; - } - if (uri_out) uri = uri_out; /* Did domainMigratePrepare change URI? */ - - assert (uri != NULL); - } - else /* if (version == 2) */ { - /* In version 2 of the protocol, the prepare step is slightly - * different. We fetch the domain XML of the source domain - * and pass it to Prepare2. - */ - if (!conn->driver->domainDumpXML) { - virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__); - goto error; - } - dom_xml = conn->driver->domainDumpXML (domain, - VIR_DOMAIN_XML_SECURE); - - if (!dom_xml) - goto error; - - ret = dconn->driver->domainMigratePrepare2 - (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, - bandwidth, dom_xml); - VIR_FREE (dom_xml); - if (ret == -1) goto done; - if (uri == NULL && uri_out == NULL) { - virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, - _("domainMigratePrepare2 did not set uri")); - goto done; - } - if (uri_out) uri = uri_out; /* Did domainMigratePrepare2 change URI? */ + if (ddomain == NULL) + goto error; - assert (uri != NULL); - } - - /* Perform the migration. The driver isn't supposed to return - * until the migration is complete. - */ - ret = conn->driver->domainMigratePerform - (domain, cookie, cookielen, uri, flags, dname, bandwidth); - - if (version == 1) { - if (ret == -1) goto done; - /* Get the destination domain and return it or error. - * 'domain' no longer actually exists at this point - * (or so we hope), but we still use the object in memory - * in order to get the name. - */ - dname = dname ? dname : domain->name; - if (dconn->driver->domainMigrateFinish) - ddomain = dconn->driver->domainMigrateFinish - (dconn, dname, cookie, cookielen, uri, flags); - else - ddomain = virDomainLookupByName (dconn, dname); - } else /* if (version == 2) */ { - /* In version 2 of the migration protocol, we pass the - * status code from the sender to the destination host, - * so it can do any cleanup if the migration failed. - */ - dname = dname ? dname : domain->name; - ddomain = dconn->driver->domainMigrateFinish2 - (dconn, dname, cookie, cookielen, uri, flags, ret); - } - - done: - VIR_FREE (uri_out); - VIR_FREE (cookie); return ddomain; error: -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix QEMU domain status after restore.
by Chris Lalancette 11 Aug '09

11 Aug '09

When doing a restore, we were forgetting to update the state file for the VM. That means that if you do a save/restore, then shut down libvirtd, then start it back up, you'll see the state of the guest as "paused", even though it is really running. We were just forgetting a "virDomainSaveStatus" call in the restor path. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 1fb2417..d789c4d 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -4099,6 +4099,7 @@ static int qemudDomainRestore(virConnectPtr conn, } VIR_FREE(info); vm->state = VIR_DOMAIN_RUNNING; + virDomainSaveStatus(conn, driver->stateDir, vm); } ret = 0; -- 1.6.0.6

2 2

[libvirt] [PATCH] Run 'cont' on successful migration finish.
by Chris Lalancette 11 Aug '09

11 Aug '09

As of qemu 0.10.6, qemu now honors the -S flag on incoming migration. That means that when the migration completes, we have to issue a 'cont' command to get the VM running again. We do it unconditionally since it won't hurt on older qemu. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 11 +++++++++++ 1 files changed, 11 insertions(+), 0 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index bcacd41..1fb2417 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -6600,7 +6600,18 @@ qemudDomainMigrateFinish2 (virConnectPtr dconn, */ if (retcode == 0) { dom = virGetDomain (dconn, vm->def->name, vm->def->uuid); + + /* run 'cont' on the destination, which allows migration on qemu + * >= 0.10.6 to work properly. This isn't strictly necessary on + * older qemu's, but it also doesn't hurt anything there + */ + if (qemudMonitorCommand(vm, "cont", &info) < 0) { + qemudReportError(dconn, NULL, NULL, VIR_ERR_INTERNAL_ERROR, + "%s", _("resume operation failed")); + goto cleanup; + } VIR_FREE(info); + vm->state = VIR_DOMAIN_RUNNING; event = virDomainEventNewFromObj(vm, VIR_DOMAIN_EVENT_RESUMED, -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix up a stray whitespace in virHashGrow.
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/hash.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/hash.c b/src/hash.c index bde3a0b..9308c0c 100644 --- a/src/hash.c +++ b/src/hash.c @@ -141,7 +141,7 @@ virHashGrow(virHashTablePtr table, int size) } table->size = size; - /* If the two loops are merged, there would be situations where + /* If the two loops are merged, there would be situations where * a new entry needs to allocated and data copied into it from * the main table. So instead, we run through the array twice, first * copying all the elements in the main array (where we can't get -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix up a whitespace in comments in src/console.c
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/console.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/console.c b/src/console.c index badb62c..4a1a80b 100644 --- a/src/console.c +++ b/src/console.c @@ -78,7 +78,7 @@ int vshRunConsole(const char *tty) { return -1; } - /* Put STDIN into raw mode so that stuff typed + /* Put STDIN into raw mode so that stuff typed does not echo to the screen (the TTY reads will result in it being echoed back already), and also ensure Ctrl-C, etc is blocked, and misc -- 1.6.0.6

2 2

[libvirt] [PATCH] Minor cleanup of error path for c_oneVmInfo.
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/opennebula/one_client.c | 16 ++++++---------- 1 files changed, 6 insertions(+), 10 deletions(-) diff --git a/src/opennebula/one_client.c b/src/opennebula/one_client.c index d1641bf..21f303a 100644 --- a/src/opennebula/one_client.c +++ b/src/opennebula/one_client.c @@ -176,6 +176,7 @@ int c_oneVmInfo(int vmid, char* ret_info,int length) xmlrpc_value *resultP; int return_code; char *return_string; + int retval = -1; resultP = xmlrpc_client_call(&one_client.env, one_client.url, "one.vmget_info", "(si)", one_client.session, vmid); @@ -188,18 +189,13 @@ int c_oneVmInfo(int vmid, char* ret_info,int length) strncpy(ret_info, return_string, length-1); ret_info[length-1] = '\0'; - xmlrpc_DECREF(resultP); - free(return_string); - - return 0; + retval = 0; } - else - { - xmlrpc_DECREF(resultP); - free(return_string); - return -1; - } + xmlrpc_DECREF(resultP); + free(return_string); + + return retval; } void c_oneFree() -- 1.6.0.6

2 2

[libvirt] [PATCH] Make openvzGetVPSUUID take a len.
by Chris Lalancette 11 Aug '09

11 Aug '09

Minor fix to openvzGetVPSUUID to make it take a length parameter. This ensures that it doesn't make assumptions about the length of the UUID buffer, and paves the way for removal of strncpy in the future. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/openvz_conf.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/openvz_conf.c b/src/openvz_conf.c index 6e9af67..a172fe3 100644 --- a/src/openvz_conf.c +++ b/src/openvz_conf.c @@ -54,7 +54,7 @@ #define VIR_FROM_THIS VIR_FROM_OPENVZ static char *openvzLocateConfDir(void); -static int openvzGetVPSUUID(int vpsid, char *uuidstr); +static int openvzGetVPSUUID(int vpsid, char *uuidstr, size_t len); static int openvzLocateConfFile(int vpsid, char *conffile, int maxlen, const char *ext); static int openvzAssignUUIDs(void); @@ -469,7 +469,7 @@ int openvzLoadDomains(struct openvz_driver *driver) { if (virAsprintf(&dom->def->name, "%i", veid) < 0) goto no_memory; - openvzGetVPSUUID(veid, uuidstr); + openvzGetVPSUUID(veid, uuidstr, sizeof(uuidstr)); ret = virUUIDParse(uuidstr, dom->def->uuid); if (ret == -1) { @@ -805,7 +805,7 @@ openvz_readline(int fd, char *ptr, int maxlen) } static int -openvzGetVPSUUID(int vpsid, char *uuidstr) +openvzGetVPSUUID(int vpsid, char *uuidstr, size_t len) { char conf_file[PATH_MAX]; char line[1024]; @@ -832,7 +832,7 @@ openvzGetVPSUUID(int vpsid, char *uuidstr) sscanf(line, "%s %s\n", iden, uuidbuf); if(STREQ(iden, "#UUID:")) { - strncpy(uuidstr, uuidbuf, VIR_UUID_STRING_BUFLEN); + strncpy(uuidstr, uuidbuf, len); break; } } @@ -856,7 +856,7 @@ openvzSetDefinedUUID(int vpsid, unsigned char *uuid) if (openvzLocateConfFile(vpsid, conf_file, PATH_MAX, "conf")<0) return -1; - if (openvzGetVPSUUID(vpsid, uuidstr)) + if (openvzGetVPSUUID(vpsid, uuidstr, sizeof(uuidstr))) return -1; if (uuidstr[0] == 0) { -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix phyp escape_specialcharacters.
by Chris Lalancette 11 Aug '09

11 Aug '09

A couple of minor fixes to phyp escape_specialcharacters. Make it a static function (since it's only used in phyp/phyp_driver.c), and make it take a dstlen parameter. This paves the way for removing strncpy in the future. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/phyp/phyp_driver.c | 10 ++++++---- src/phyp/phyp_driver.h | 2 -- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/phyp/phyp_driver.c b/src/phyp/phyp_driver.c index cbfd31b..f457cf4 100644 --- a/src/phyp/phyp_driver.c +++ b/src/phyp/phyp_driver.c @@ -53,6 +53,8 @@ #define VIR_FROM_THIS VIR_FROM_PHYP +static int escape_specialcharacters(char *src, char *dst, size_t dstlen); + /* * URI: phyp://user@[hmc|ivm]/managed_system * */ @@ -94,7 +96,7 @@ phypOpen(virConnectPtr conn, return VIR_DRV_OPEN_ERROR; } - if (escape_specialcharacters(conn->uri->path, string) == -1) { + if (escape_specialcharacters(conn->uri->path, string, sizeof(string)) == -1) { virRaiseError(conn, NULL, NULL, 0, VIR_FROM_PHYP, VIR_ERR_ERROR, NULL, NULL, NULL, 0, 0, "%s", _("Error parsing 'path'. Invalid characters.")); @@ -1341,8 +1343,8 @@ init_uuid_db(virConnectPtr conn) return; } -int -escape_specialcharacters(char *src, char *dst) +static int +escape_specialcharacters(char *src, char *dst, size_t dstlen) { size_t len = strlen(src); char temp_buffer[len]; @@ -1367,7 +1369,7 @@ escape_specialcharacters(char *src, char *dst) } temp_buffer[j] = '\0'; - if (strncpy(dst, temp_buffer, j) == NULL) + if (strncpy(dst, temp_buffer, dstlen) == NULL) return -1; return 0; diff --git a/src/phyp/phyp_driver.h b/src/phyp/phyp_driver.h index fd824b3..f16b6fe 100644 --- a/src/phyp/phyp_driver.h +++ b/src/phyp/phyp_driver.h @@ -62,5 +62,3 @@ char *phypGetBackingDevice(virConnectPtr conn, const char *managed_system, int phypDiskType(virConnectPtr conn, char *backing_device); SSH_SESSION *openSSHSession(virConnectPtr conn, virConnectAuthPtr auth); - -int escape_specialcharacters(char *src, char *dst); -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix up a few minor indentation issues.
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/xen_internal.c | 8 ++++---- src/xend_internal.c | 2 +- src/xm_internal.c | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/xen_internal.c b/src/xen_internal.c index b6bf10d..ae78f84 100644 --- a/src/xen_internal.c +++ b/src/xen_internal.c @@ -1147,7 +1147,7 @@ static const char *str_cap = "cap"; */ int xenHypervisorGetSchedulerParameters(virDomainPtr domain, - virSchedParameterPtr params, int *nparams) + virSchedParameterPtr params, int *nparams) { xenUnifiedPrivatePtr priv; @@ -1209,19 +1209,19 @@ xenHypervisorGetSchedulerParameters(virDomainPtr domain, return(-1); strncpy (params[0].field, str_weight, VIR_DOMAIN_SCHED_FIELD_LENGTH); - params[0].field[VIR_DOMAIN_SCHED_FIELD_LENGTH-1] = '\0'; + params[0].field[VIR_DOMAIN_SCHED_FIELD_LENGTH-1] = '\0'; params[0].type = VIR_DOMAIN_SCHED_FIELD_UINT; params[0].value.ui = op_dom.u.getschedinfo.u.credit.weight; strncpy (params[1].field, str_cap, VIR_DOMAIN_SCHED_FIELD_LENGTH); - params[1].field[VIR_DOMAIN_SCHED_FIELD_LENGTH-1] = '\0'; + params[1].field[VIR_DOMAIN_SCHED_FIELD_LENGTH-1] = '\0'; params[1].type = VIR_DOMAIN_SCHED_FIELD_UINT; params[1].value.ui = op_dom.u.getschedinfo.u.credit.cap; *nparams = 2; break; default: - virXenErrorFunc(domain->conn, VIR_ERR_INVALID_ARG, __FUNCTION__, + virXenErrorFunc(domain->conn, VIR_ERR_INVALID_ARG, __FUNCTION__, "Unknown scheduler", op_sys.u.getschedulerid.sched_id); return -1; } diff --git a/src/xend_internal.c b/src/xend_internal.c index 2585703..7bcee7d 100644 --- a/src/xend_internal.c +++ b/src/xend_internal.c @@ -4732,7 +4732,7 @@ static const char *str_cap = "cap"; */ static int xenDaemonGetSchedulerParameters(virDomainPtr domain, - virSchedParameterPtr params, int *nparams) + virSchedParameterPtr params, int *nparams) { xenUnifiedPrivatePtr priv; struct sexpr *root; diff --git a/src/xm_internal.c b/src/xm_internal.c index c869120..b7f9e67 100644 --- a/src/xm_internal.c +++ b/src/xm_internal.c @@ -1418,7 +1418,7 @@ xenXMDomainConfigParse(virConnectPtr conn, virConfPtr conf) { no_memory: virReportOOMError(conn); /* fallthrough */ - cleanup: +cleanup: virDomainGraphicsDefFree(graphics); virDomainNetDefFree(net); virDomainDiskDefFree(disk); -- 1.6.0.6

2 2

[libvirt] [PATCH] Remove a duplicated assignment in Xen PCI parsing.
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/xm_internal.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/src/xm_internal.c b/src/xm_internal.c index b7f9e67..71b852e 100644 --- a/src/xm_internal.c +++ b/src/xm_internal.c @@ -1160,7 +1160,6 @@ xenXMDomainConfigParse(virConnectPtr conn, virConfPtr conf) { goto skippci; /* pci=['0000:00:1b.0','0000:00:13.0'] */ - key = list->str; if (!(key = list->str)) goto skippci; if (!(nextkey = strchr(key, ':'))) -- 1.6.0.6

2 2

[libvirt] vbox save/restore
by Lucian Adrian Grijincu 11 Aug '09

11 Aug '09

Hello, I see that the vbox driver in virDomainSave doesn't use the destination file path and calls vbox's IConsole::SaveState. As I see it IConsole::SaveState is more like a "pause" command: you can save the state you're in and wait for the machine to stop running and you can resume it later. This limits the uses of the vbox support of libvirt. There is another way you can do virDomainSave/virDomainRestore: use snapshots. This way, you can specify the file in which you want to save the snapshot, you can continue to use the machine and do some crazy experiments and then restore the machine to the known good state. As of the current implementation, I don't think there's a way to do this in libvirt. I'd like to know if there's a real reason why virDomainSave is implemented this way (and virDomainRestore is unimplemented), or it just got implemented like this and no one cared much about it. -- . ..: Lucian

3 4

[libvirt] [PATCH] Compressed save image format for Qemu.
by Chris Lalancette 11 Aug '09

11 Aug '09

Implement a compressed save image format for qemu. While ideally we would have the choice between compressed/non-compressed available to the libvirt API, unfortunately there is no "flags" parameter to the virDomainSave() API. Therefore, implement this as a qemu.conf option. Both gzip and bzip2 are implemented, and it should be very easy to implement additional compression methods. One open question is if/how we should detect the gzip and bzip2 binaries. One way to do it is to do compile-time setting of the paths (via configure.in), but that doesn't seem like a great thing to do. Another solution (my preferred solution) is not to detect at all; when we go to run the commands that need them, if they aren't available, or aren't available in one of the standard paths, then we'll fail. Maybe somebody else has another option or opinion, though. In the future, we'll have a more robust (managed) save/restore API, at which time we can expose this functionality properly in the API. V2: get rid of redundant dd command and just use >> to append data. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu.conf | 10 +++++++ src/qemu_conf.c | 11 ++++++++ src/qemu_conf.h | 2 + src/qemu_driver.c | 71 ++++++++++++++++++++++++++++++++++++++++++++++++++--- 4 files changed, 90 insertions(+), 4 deletions(-) diff --git a/src/qemu.conf b/src/qemu.conf index 653f487..86dcc9d 100644 --- a/src/qemu.conf +++ b/src/qemu.conf @@ -129,3 +129,13 @@ # "/dev/ptmx", "/dev/kvm", "/dev/kqemu", # "/dev/rtc", "/dev/hpet", "/dev/net/tun", #] + +# The default format for Qemu/KVM guest save images is raw; that is, the +# memory from the domain is dumped out directly to a file. If you have +# guests with a large amount of memory, however, this can take up quite +# a bit of space. If you would like to compress the images while they +# are being saved to disk, you can also set "gzip" or "bzip2" for the +# save_image_format. Note that this means you slow down the process +# of saving a domain in order to save disk space. +# +# save_image_format = "raw" diff --git a/src/qemu_conf.c b/src/qemu_conf.c index 7ca5a15..ed87e13 100644 --- a/src/qemu_conf.c +++ b/src/qemu_conf.c @@ -280,6 +280,17 @@ int qemudLoadDriverConfig(struct qemud_driver *driver, driver->cgroupDeviceACL[i] = NULL; } + p = virConfGetValue (conf, "save_image_format"); + CHECK_TYPE ("save_image_format", VIR_CONF_STRING); + if (p && p->str) { + VIR_FREE(driver->saveImageFormat); + if (!(driver->saveImageFormat = strdup(p->str))) { + virReportOOMError(NULL); + virConfFree(conf); + return -1; + } + } + virConfFree (conf); return 0; } diff --git a/src/qemu_conf.h b/src/qemu_conf.h index 8f4ef6a..e34baab 100644 --- a/src/qemu_conf.h +++ b/src/qemu_conf.h @@ -111,6 +111,8 @@ struct qemud_driver { char *securityDriverName; virSecurityDriverPtr securityDriver; + + char *saveImageFormat; }; diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 3c92635..b146330 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -3437,11 +3437,26 @@ static int qemudDomainSave(virDomainPtr dom, struct qemud_save_header header; int ret = -1; virDomainEventPtr event = NULL; + int internalret; memset(&header, 0, sizeof(header)); memcpy(header.magic, QEMUD_SAVE_MAGIC, sizeof(header.magic)); header.version = QEMUD_SAVE_VERSION; + if (driver->saveImageFormat == NULL) + header.compressed = QEMUD_SAVE_FORMAT_RAW; + else if (STREQ(driver->saveImageFormat, "raw")) + header.compressed = QEMUD_SAVE_FORMAT_RAW; + else if (STREQ(driver->saveImageFormat, "gzip")) + header.compressed = QEMUD_SAVE_FORMAT_GZIP; + else if (STREQ(driver->saveImageFormat, "bzip2")) + header.compressed = QEMUD_SAVE_FORMAT_BZIP2; + else { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, + "%s", _("Invalid save image format specified in configuration file")); + return -1; + } + qemuDriverLock(driver); vm = virDomainFindByUUID(&driver->domains, dom->uuid); @@ -3514,11 +3529,25 @@ static int qemudDomainSave(virDomainPtr dom, virReportOOMError(dom->conn); goto cleanup; } - if (virAsprintf(&command, "migrate \"exec:" - "dd of='%s' oflag=append conv=notrunc 2>/dev/null" - "\"", safe_path) == -1) { + + if (header.compressed == QEMUD_SAVE_FORMAT_RAW) + internalret = virAsprintf(&command, "migrate \"exec:" + "dd of='%s' oflag=append conv=notrunc 2>/dev/null" + "\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_GZIP) + internalret = virAsprintf(&command, "migrate \"exec:" + "gzip -c >> '%s' 2>/dev/null\"", safe_path); + else if (header.compressed == QEMUD_SAVE_FORMAT_BZIP2) + internalret = virAsprintf(&command, "migrate \"exec:" + "bzip2 -c >> '%s' 2>/dev/null\"", safe_path); + else { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_INTERNAL_ERROR, + _("Invalid compress format %d"), + header.compressed); + goto cleanup; + } + if (internalret < 0) { virReportOOMError(dom->conn); - command = NULL; goto cleanup; } @@ -4039,6 +4068,9 @@ static int qemudDomainRestore(virConnectPtr conn, char *xml = NULL; struct qemud_save_header header; virDomainEventPtr event = NULL; + int intermediatefd = -1; + pid_t intermediate_pid = -1; + int childstat; qemuDriverLock(driver); /* Verify the header and read the XML */ @@ -4128,8 +4160,39 @@ static int qemudDomainRestore(virConnectPtr conn, } def = NULL; + if (header.version == 2) { + const char *intermediate_argv[3] = { NULL, "-dc", NULL }; + if (header.compressed == QEMUD_SAVE_FORMAT_GZIP) + intermediate_argv[0] = "gzip"; + else if (header.compressed == QEMUD_SAVE_FORMAT_BZIP2) + intermediate_argv[0] = "bzip2"; + else if (header.compressed != QEMUD_SAVE_FORMAT_RAW) { + qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, + _("Unknown compressed save format %d"), + header.compressed); + goto cleanup; + } + if (intermediate_argv[0] != NULL) { + intermediatefd = fd; + fd = -1; + if (virExec(conn, intermediate_argv, NULL, NULL, + &intermediate_pid, intermediatefd, &fd, NULL, 0) < 0) { + qemudReportError(conn, NULL, NULL, VIR_ERR_INTERNAL_ERROR, + _("Failed to start decompression binary %s"), + intermediate_argv[0]); + goto cleanup; + } + } + } /* Set the migration source and start it up. */ ret = qemudStartVMDaemon(conn, driver, vm, "stdio", fd); + if (intermediate_pid != -1) { + /* Wait for intermediate process to exit */ + while (waitpid(intermediate_pid, &childstat, 0) == -1 && + errno == EINTR); + } + if (intermediatefd != -1) + close(intermediatefd); close(fd); fd = -1; if (ret < 0) { -- 1.6.0.6

2 2

[libvirt] [PATCH] Remove use of strncpy in qemudExtractMonitorPath.
by Chris Lalancette 11 Aug '09

11 Aug '09

qemudExtractMonitorPath() was doing a VIR_ALLOC_N followed by a strncpy. However, this isn't necessary; we can do the same thing using virAsprintf(), which is much safer. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 55a09f5..37fdec2 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -1017,12 +1017,13 @@ qemudExtractMonitorPath(virConnectPtr conn, */ while (*tmp) { if (c_isspace(*tmp)) { - if (VIR_ALLOC_N(*path, (tmp-dev)+1) < 0) { + if (virAsprintf(path, "%s", dev) < 0) { virReportOOMError(conn); return -1; } - strncpy(*path, dev, (tmp-dev)); - (*path)[(tmp-dev)] = '\0'; + /* the last character is a \n, so back up one to overwrite */ + (*path)[tmp-dev] = '\0'; + /* ... now further update offset till we get EOL */ *offset = tmp - haystack; return 0; -- 1.6.0.6

2 1

[libvirt] [PATCH] Convert one use of strncpy + strcat to snprintf in test driver.
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/test.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/src/test.c b/src/test.c index 470816b..ab6cfdf 100644 --- a/src/test.c +++ b/src/test.c @@ -417,9 +417,8 @@ static char *testBuildFilename(const char *relativeTo, char *absFile; if (VIR_ALLOC_N(absFile, baseLen + strlen(filename) + 1) < 0) return NULL; - strncpy(absFile, relativeTo, baseLen); - absFile[baseLen] = '\0'; - strcat(absFile, filename); + snprintf(absFile, baseLen + strlen(filename) + 1, "%s%s", + relativeTo, filename); return absFile; } else { return strdup(filename); -- 1.6.0.6

2 1

[libvirt] [PATCH] Error path mutex unlock in virGetStorageVol.
by Chris Lalancette 11 Aug '09

11 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/datatypes.c | 5 +---- 1 files changed, 1 insertions(+), 4 deletions(-) diff --git a/src/datatypes.c b/src/datatypes.c index 1889dc3..ac61682 100644 --- a/src/datatypes.c +++ b/src/datatypes.c @@ -863,19 +863,16 @@ virGetStorageVol(virConnectPtr conn, const char *pool, const char *name, const c ret = (virStorageVolPtr) virHashLookup(conn->storageVols, key); if (ret == NULL) { if (VIR_ALLOC(ret) < 0) { - virMutexUnlock(&conn->lock); virReportOOMError(conn); goto error; } ret->pool = strdup(pool); if (ret->pool == NULL) { - virMutexUnlock(&conn->lock); virReportOOMError(conn); goto error; } ret->name = strdup(name); if (ret->name == NULL) { - virMutexUnlock(&conn->lock); virReportOOMError(conn); goto error; } @@ -885,7 +882,6 @@ virGetStorageVol(virConnectPtr conn, const char *pool, const char *name, const c ret->conn = conn; if (virHashAddEntry(conn->storageVols, key, ret) < 0) { - virMutexUnlock(&conn->lock); virLibConnError(conn, VIR_ERR_INTERNAL_ERROR, "%s", _("failed to add storage vol to connection hash table")); goto error; @@ -897,6 +893,7 @@ virGetStorageVol(virConnectPtr conn, const char *pool, const char *name, const c return(ret); error: + virMutexUnlock(&conn->lock); if (ret != NULL) { VIR_FREE(ret->name); VIR_FREE(ret->pool); -- 1.6.0.6

2 1

[libvirt] Crash while using virConnectRef
by Chris Lalancette 11 Aug '09

11 Aug '09

All, I've been doing some testing of the virConnectRef() function, and as far as I can tell, it's busted. Here's the simple example program I'm using: #include <stdio.h> #include <stdlib.h> #include <libvirt/libvirt.h> #include <unistd.h> int main(int argc, char *argv[]) { virConnectPtr conn; conn = virConnectOpen(NULL); printf("type is %s\n",virConnectGetType(conn)); virConnectRef(conn); virConnectClose(conn); virConnectClose(conn); return 0; } When compiling and running this program, I get a segfault during the *second* virConnectClose(). The crash comes in the remote_internal driver, because the first virConnectClose() actually freed up the remote_internal private memory, and now the second one comes along and accesses a NULL pointer. I think this actually points to a bug in virConnectClose(); we shouldn't be running the ->close() callbacks on the drivers until we are going to be really closing up the driver, i.e. we should move all of the ->close() callbacks into virUnrefConnect(). Is that the right way to fix this? -- Chris Lalancette

2 1

[libvirt] [PATCH] Handle kernels with no ipv6 support
by Mark McLoughlin 11 Aug '09

11 Aug '09

If the ipv6 kernel module is not loaded, then we get this when starting a virtual network: libvir: Network Config error : cannot enable /proc/sys/net/ipv6/conf/virbr0/disable_ipv6: No such file or directory If disable_ipv6 is not present, we should just merrily continue on our way. * src/network_driver.c: make networkDisableIPV6() not fail if the kernel has no ipv6 support --- src/network_driver.c | 6 ++++++ 1 files changed, 6 insertions(+), 0 deletions(-) diff --git a/src/network_driver.c b/src/network_driver.c index eaea454..84910ab 100644 --- a/src/network_driver.c +++ b/src/network_driver.c @@ -801,6 +801,12 @@ static int networkDisableIPV6(virConnectPtr conn, goto cleanup; } + if (access(field, W_OK) < 0 && errno == ENOENT) { + VIR_DEBUG("ipv6 appears to already be disabled on %s", network->def->bridge); + ret = 0; + goto cleanup; + } + if (virFileWriteStr(field, "1") < 0) { virReportSystemError(conn, errno, _("cannot enable %s"), field); -- 1.6.2.5

3 2

[libvirt] does libvirt support qemu-kvm vmchannel option?
by cyliu7 11 Aug '09

11 Aug '09

Hi all, I noticed that qemu-kvm support an option vmchannel (-net channel ...) to create a new device to communicate between host and guest. does libvirt support this option when creating a KVM based VM. if so, how to config the VM? is there any sample to show this feature? any suggestions are greatly appreciated. Regards, ------------------------ Chunyang Liu

2 1

[libvirt] PATCH: Support PolicyKit 1.0
by Daniel P. Berrange 10 Aug '09

10 Aug '09

In the seriously annoying way of things, the newest PolicyKit in Fedora 12 has been completely re-written from scratch with a totally incompatible application facing API. Conceptually it is still pretty similar though, with the exception that client applications no longer need to explicitly launch an auth dialog - that's done out-of-band by policykit itself. This patch adjusts libvirtd to the new API, and removes the libvirt client side code that spawned the auth helper. On the libvirtd side avoid their APIs and instead spawn an external auth checking program 'pkcheck', which returns 0 on success, non-0 for denial. In the final annoying bit, the XML format for the policy has remained with exactly the same DTD version, except that it has quietly changed the allowed values for some attributes in an incompatible manner. So I have to add a new policy file too. NB, it may not look like we've changed the client side, but we have, since the #ifdef for the external auth agent is no longer set. Tested with old policy kit, and new policykit, and with none at all. b/configure.in | 73 +++++++++++++++++++++++++++-------------- b/qemud/Makefile.am | 11 +++++- b/qemud/libvirtd.policy-0 | 42 +++++++++++++++++++++++ b/qemud/libvirtd.policy-1 | 42 +++++++++++++++++++++++ b/qemud/qemud.c | 4 +- b/qemud/qemud.h | 4 +- b/qemud/remote.c | 81 +++++++++++++++++++++++++++++++++++++++++++--- qemud/libvirtd.policy | 42 ----------------------- 8 files changed, 223 insertions(+), 76 deletions(-) Regards, Daniel diff --git a/configure.in b/configure.in index b905b23..77d6c9e 100644 --- a/configure.in +++ b/configure.in @@ -607,40 +607,61 @@ AC_SUBST([SASL_LIBS]) dnl PolicyKit library POLKIT_CFLAGS= POLKIT_LIBS= +PKCHECK_PATH= AC_ARG_WITH([polkit], [ --with-polkit use PolicyKit for UNIX socket access checks], [], [with_polkit=check]) +with_polkit0=no +with_polkit1=no if test "x$with_polkit" = "xyes" -o "x$with_polkit" = "xcheck"; then - PKG_CHECK_MODULES(POLKIT, polkit-dbus >= $POLKIT_REQUIRED, - [with_polkit=yes], [ - if test "x$with_polkit" = "xcheck" ; then - with_polkit=no - else - AC_MSG_ERROR( - [You must install PolicyKit >= $POLKIT_REQUIRED to compile libvirt]) - fi - ]) - if test "x$with_polkit" = "xyes" ; then + dnl Check for new polkit first - just a binary + AC_PATH_PROG([PKCHECK_PATH],[pkcheck], [], [/usr/sbin:$PATH]) + if test "x$PKCHECK_PATH" != "x" ; then + AC_DEFINE_UNQUOTED([PKCHECK_PATH],["$PKCHECK_PATH"],[Location of pkcheck program]) AC_DEFINE_UNQUOTED([HAVE_POLKIT], 1, - [use PolicyKit for UNIX socket access checks]) - - old_CFLAGS=$CFLAGS - old_LDFLAGS=$LDFLAGS - CFLAGS="$CFLAGS $POLKIT_CFLAGS" - LDFLAGS="$LDFLAGS $POLKIT_LIBS" - AC_CHECK_FUNCS([polkit_context_is_caller_authorized]) - CFLAGS="$old_CFLAGS" - LDFLAGS="$old_LDFLAGS" - - AC_PATH_PROG([POLKIT_AUTH], [polkit-auth]) - if test "x$POLKIT_AUTH" != "x"; then - AC_DEFINE_UNQUOTED([POLKIT_AUTH],["$POLKIT_AUTH"],[Location of polkit-auth program]) + [use PolicyKit for UNIX socket access checks]) + AC_DEFINE_UNQUOTED([HAVE_POLKIT1], 1, + [use PolicyKit for UNIX socket access checks]) + with_polkit="yes" + with_polkit1="yes" + else + dnl Check for old polkit second - library + binary + PKG_CHECK_MODULES(POLKIT, polkit-dbus >= $POLKIT_REQUIRED, + [with_polkit=yes], [ + if test "x$with_polkit" = "xcheck" ; then + with_polkit=no + else + AC_MSG_ERROR( + [You must install PolicyKit >= $POLKIT_REQUIRED to compile libvirt]) + fi + ]) + if test "x$with_polkit" = "xyes" ; then + AC_DEFINE_UNQUOTED([HAVE_POLKIT], 1, + [use PolicyKit for UNIX socket access checks]) + AC_DEFINE_UNQUOTED([HAVE_POLKIT0], 1, + [use PolicyKit for UNIX socket access checks]) + + old_CFLAGS=$CFLAGS + old_LDFLAGS=$LDFLAGS + CFLAGS="$CFLAGS $POLKIT_CFLAGS" + LDFLAGS="$LDFLAGS $POLKIT_LIBS" + AC_CHECK_FUNCS([polkit_context_is_caller_authorized]) + CFLAGS="$old_CFLAGS" + LDFLAGS="$old_LDFLAGS" + + AC_PATH_PROG([POLKIT_AUTH], [polkit-auth]) + if test "x$POLKIT_AUTH" != "x"; then + AC_DEFINE_UNQUOTED([POLKIT_AUTH],["$POLKIT_AUTH"],[Location of polkit-auth program]) + fi + with_polkit0="yes" fi fi fi AM_CONDITIONAL([HAVE_POLKIT], [test "x$with_polkit" = "xyes"]) +AM_CONDITIONAL([HAVE_POLKIT0], [test "x$with_polkit0" = "xyes"]) +AM_CONDITIONAL([HAVE_POLKIT1], [test "x$with_polkit1" = "xyes"]) AC_SUBST([POLKIT_CFLAGS]) AC_SUBST([POLKIT_LIBS]) @@ -1621,7 +1642,11 @@ else AC_MSG_NOTICE([ avahi: no]) fi if test "$with_polkit" = "yes" ; then -AC_MSG_NOTICE([ polkit: $POLKIT_CFLAGS $POLKIT_LIBS]) +if test "$with_polkit0" = "yes" ; then +AC_MSG_NOTICE([ polkit: $POLKIT_CFLAGS $POLKIT_LIBS (version 0)]) +else +AC_MSG_NOTICE([ polkit: $PKCHECK_PATH (version 1)]) +fi else AC_MSG_NOTICE([ polkit: no]) fi diff --git a/qemud/Makefile.am b/qemud/Makefile.am index 959ff88..3d143da 100644 --- a/qemud/Makefile.am +++ b/qemud/Makefile.am @@ -21,7 +21,8 @@ EXTRA_DIST = \ remote_protocol.x \ libvirtd.conf \ libvirtd.init.in \ - libvirtd.policy \ + libvirtd.policy-0 \ + libvirtd.policy-1 \ libvirtd.sasl \ libvirtd.sysconf \ libvirtd.aug \ @@ -147,7 +148,13 @@ endif libvirtd_LDADD += ../src/libvirt.la if HAVE_POLKIT +if HAVE_POLKIT0 policydir = $(datadir)/PolicyKit/policy +policyfile = libvirtd.policy-0 +else +policydir = $(datadir)/polkit-1/actions +policyfile = libvirtd.policy-1 +endif endif if HAVE_AVAHI @@ -197,7 +204,7 @@ endif if HAVE_POLKIT install-data-polkit:: install-init mkdir -p $(DESTDIR)$(policydir) - $(INSTALL_DATA) $(srcdir)/libvirtd.policy $(DESTDIR)$(policydir)/org.libvirt.unix.policy + $(INSTALL_DATA) $(srcdir)/$(policyfile) $(DESTDIR)$(policydir)/org.libvirt.unix.policy uninstall-data-polkit:: install-init rm -f $(DESTDIR)$(policydir)/org.libvirt.unix.policy else diff --git a/qemud/libvirtd.policy b/qemud/libvirtd.policy deleted file mode 100644 index b6da946..0000000 --- a/qemud/libvirtd.policy +++ /dev/null @@ -1,42 +0,0 @@ -<!DOCTYPE policyconfig PUBLIC - "-//freedesktop//DTD PolicyKit Policy Configuration 1.0//EN" - "http://www.freedesktop.org/standards/PolicyKit/1.0/policyconfig.dtd"> - - - -<policyconfig> - <action id="org.libvirt.unix.monitor"> - <description>Monitor local virtualized systems</description> - <message>System policy prevents monitoring of local virtualized systems</message> - <defaults> -  - <allow_any>yes</allow_any> - <allow_inactive>yes</allow_inactive> - <allow_active>yes</allow_active> - </defaults> - </action> - - <action id="org.libvirt.unix.manage"> - <description>Manage local virtualized systems</description> - <message>System policy prevents management of local virtualized systems</message> - <defaults> -  - <allow_any>no</allow_any> - <allow_inactive>no</allow_inactive> - <allow_active>auth_admin_keep_session</allow_active> - </defaults> - </action> -</policyconfig> diff --git a/qemud/libvirtd.policy-0 b/qemud/libvirtd.policy-0 new file mode 100644 index 0000000..b6da946 --- /dev/null +++ b/qemud/libvirtd.policy-0 @@ -0,0 +1,42 @@ +<!DOCTYPE policyconfig PUBLIC + "-//freedesktop//DTD PolicyKit Policy Configuration 1.0//EN" + "http://www.freedesktop.org/standards/PolicyKit/1.0/policyconfig.dtd"> + + + +<policyconfig> + <action id="org.libvirt.unix.monitor"> + <description>Monitor local virtualized systems</description> + <message>System policy prevents monitoring of local virtualized systems</message> + <defaults> +  + <allow_any>yes</allow_any> + <allow_inactive>yes</allow_inactive> + <allow_active>yes</allow_active> + </defaults> + </action> + + <action id="org.libvirt.unix.manage"> + <description>Manage local virtualized systems</description> + <message>System policy prevents management of local virtualized systems</message> + <defaults> +  + <allow_any>no</allow_any> + <allow_inactive>no</allow_inactive> + <allow_active>auth_admin_keep_session</allow_active> + </defaults> + </action> +</policyconfig> diff --git a/qemud/libvirtd.policy-1 b/qemud/libvirtd.policy-1 new file mode 100644 index 0000000..6fa3a5e --- /dev/null +++ b/qemud/libvirtd.policy-1 @@ -0,0 +1,42 @@ +<!DOCTYPE policyconfig PUBLIC + "-//freedesktop//DTD PolicyKit Policy Configuration 1.0//EN" + "http://www.freedesktop.org/standards/PolicyKit/1.0/policyconfig.dtd"> + + + +<policyconfig> + <action id="org.libvirt.unix.monitor"> + <description>Monitor local virtualized systems</description> + <message>System policy prevents monitoring of local virtualized systems</message> + <defaults> +  + <allow_any>yes</allow_any> + <allow_inactive>yes</allow_inactive> + <allow_active>yes</allow_active> + </defaults> + </action> + + <action id="org.libvirt.unix.manage"> + <description>Manage local virtualized systems</description> + <message>System policy prevents management of local virtualized systems</message> + <defaults> +  + <allow_any>no</allow_any> + <allow_inactive>no</allow_inactive> + <allow_active>auth_admin_keep</allow_active> + </defaults> + </action> +</policyconfig> diff --git a/qemud/qemud.c b/qemud/qemud.c index 3e551ca..50b0cdd 100644 --- a/qemud/qemud.c +++ b/qemud/qemud.c @@ -895,7 +895,7 @@ static struct qemud_server *qemudNetworkInit(struct qemud_server *server) { } #endif -#ifdef HAVE_POLKIT +#if HAVE_POLKIT0 if (auth_unix_rw == REMOTE_AUTH_POLKIT || auth_unix_ro == REMOTE_AUTH_POLKIT) { DBusError derr; @@ -982,7 +982,7 @@ static struct qemud_server *qemudNetworkInit(struct qemud_server *server) { sock = sock->next; } -#ifdef HAVE_POLKIT +#if HAVE_POLKIT0 if (server->sysbus) dbus_connection_unref(server->sysbus); #endif diff --git a/qemud/qemud.h b/qemud/qemud.h index 254db44..e8ce209 100644 --- a/qemud/qemud.h +++ b/qemud/qemud.h @@ -34,7 +34,7 @@ #include <sasl/sasl.h> #endif -#ifdef HAVE_POLKIT +#if HAVE_POLKIT0 #include <dbus/dbus.h> #endif @@ -253,7 +253,7 @@ struct qemud_server { #if HAVE_SASL char **saslUsernameWhitelist; #endif -#if HAVE_POLKIT +#if HAVE_POLKIT0 DBusConnection *sysbus; #endif }; diff --git a/qemud/remote.c b/qemud/remote.c index d32d513..490a807 100644 --- a/qemud/remote.c +++ b/qemud/remote.c @@ -43,7 +43,7 @@ #include <fnmatch.h> #include "virterror_internal.h" -#ifdef HAVE_POLKIT +#if HAVE_POLKIT0 #include <polkit/polkit.h> #include <polkit-dbus/polkit-dbus.h> #endif @@ -3106,7 +3106,80 @@ remoteDispatchAuthSaslStep (struct qemud_server *server ATTRIBUTE_UNUSED, #endif /* HAVE_SASL */ -#if HAVE_POLKIT +#if HAVE_POLKIT1 +static int +remoteDispatchAuthPolkit (struct qemud_server *server, + struct qemud_client *client, + virConnectPtr conn ATTRIBUTE_UNUSED, + remote_error *rerr, + void *args ATTRIBUTE_UNUSED, + remote_auth_polkit_ret *ret) +{ + pid_t callerPid; + uid_t callerUid; + const char *action; + int status = -1; + char pidbuf[50]; + int rv; + + virMutexLock(&server->lock); + virMutexLock(&client->lock); + virMutexUnlock(&server->lock); + + action = client->readonly ? + "org.libvirt.unix.monitor" : + "org.libvirt.unix.manage"; + + const char * const pkcheck [] = { + PKCHECK_PATH, + "--action-id", action, + "--process", pidbuf, + "--allow-user-interaction", + NULL + }; + + REMOTE_DEBUG("Start PolicyKit auth %d", client->fd); + if (client->auth != REMOTE_AUTH_POLKIT) { + VIR_ERROR0(_("client tried invalid PolicyKit init request")); + goto authfail; + } + + if (qemudGetSocketIdentity(client->fd, &callerUid, &callerPid) < 0) { + VIR_ERROR0(_("cannot get peer socket identity")); + goto authfail; + } + + VIR_INFO(_("Checking PID %d running as %d"), callerPid, callerUid); + + rv = snprintf(pidbuf, sizeof pidbuf, "%d", callerPid); + if (rv < 0 || rv >= sizeof pidbuf) { + VIR_ERROR(_("Caller PID was too large %d"), callerPid); + goto authfail; + } + + if (virRun(NULL, pkcheck, &status) < 0) { + VIR_ERROR(_("Cannot invoke %s"), PKCHECK_PATH); + goto authfail; + } + if (status != 0) { + VIR_ERROR(_("Policy kit denied action %s from pid %d, uid %d, result: %d\n"), + action, callerPid, callerUid, status); + goto authfail; + } + VIR_INFO(_("Policy allowed action %s from pid %d, uid %d"), + action, callerPid, callerUid); + ret->complete = 1; + client->auth = REMOTE_AUTH_NONE; + + virMutexUnlock(&client->lock); + return 0; + +authfail: + remoteDispatchAuthError(rerr); + virMutexUnlock(&client->lock); + return -1; +} +#elif HAVE_POLKIT0 static int remoteDispatchAuthPolkit (struct qemud_server *server, struct qemud_client *client, @@ -3217,7 +3290,7 @@ authfail: return -1; } -#else /* HAVE_POLKIT */ +#else /* !HAVE_POLKIT0 & !HAVE_POLKIT1*/ static int remoteDispatchAuthPolkit (struct qemud_server *server ATTRIBUTE_UNUSED, @@ -3231,7 +3304,7 @@ remoteDispatchAuthPolkit (struct qemud_server *server ATTRIBUTE_UNUSED, remoteDispatchAuthError(rerr); return -1; } -#endif /* HAVE_POLKIT */ +#endif /* HAVE_POLKIT1 */ /*************************************************************** -- |: 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] Project announcement: abiCloud
by Diego Mariño 10 Aug '09

10 Aug '09

Hi all, This is a one-time post to announce a project which supports libvirt for managing virtual machines: abiCloud. AbiCloud is an open source cloud platform manager which allows to easily deploy a private cloud in your datacenter. One of the key differences of AbiCloud is the the web rich interface for managing the infrastructure. You can deploy a new service just dragging and dropping a VM. Currently, we use libvirt for: - Managing (deployment, configuration, administration...) of Xen & KVM virtual machines trought the java binding - Events monitoring trough a C API for openwsman We truly believe that V2V will boost the adoption of new hypervisors from users, and libvirt will have a prominent role in this space. So, we will continue adding support for more hypervisors trough libvirt. homepage: http://community.abiquo.com/display/AbiCloud Features: - Rich interface - User management through ACL - Infrastructure management (from datacenters to virtual machines) - Network management - Appliance repository - Design virtual datacenters Requirements: - Linux/Windows/MacOSX - Any hypervisor supported installed in a node - Sun Java Runtime Environment (JRE) 1.6 Licenses - CPAL 1.0 for the UI - MPL 1.1 for the services Thanks, PS: We will really appreciate if you could add us as in your website a "Third Party Libvirt Application" - Diego Mariño Co-founder & Community Manager +34 636-016-062 +1 408-386-8559 www.abiquo.com

3 4

[libvirt][PATCH] running libvirt on the latest kvm snapshot
by Anton Protopopov 10 Aug '09

10 Aug '09

Hi. Libvirt tryes to extract kvm version from the string "QEMU PC emulator version <major>.<minor>.<micro> (kvm-<version>)..." For examle, # kvm -help | head -1 QEMU PC emulator version 0.10.0 (kvm-85), Copyright (c) 2003-2008 Fabrice Bellard The problem occures if you are trying to run it on the devel version of kvm: # kvm -help | head -1 QEMU PC emulator version 0.10.0 (kvm-devel), Copyright (c) 2003-2008 Fabrice Bellard The virParseNumber() obvioulsy fails on the "devel" string here. I proposeto setup `kvm_version' to 1000 (that's enough, I think) in that case, so all checks, such as if (kvm_version > 74) ... will pass.

5 7

[libvirt] Is it possible to use more then 2 CPUs in the guest?
by Mirko Raasch 09 Aug '09

09 Aug '09

Hi, in my server i use an Intel Quad Core CPU and i want to use in one WindowsXP guest all 4 cores. The device manager shows four CPU (Qemu Virtual CPU version 0.10.5). When i start prime95 or superpi, only 2 of them are in use. CPU-Z shows also 2 Cores, instead of 4 like the device manager. How can i use all four cores in the guest? Greetings

4 10

[libvirt] ABI compliance checker
by Андрей Пономаренко 09 Aug '09

09 Aug '09

Colleagues, I'm from Institute for System Programing of Russian Academy of Sciences and we are developing a free lightweight tool for checking backward/forward binary compatibility of shared C/C++ libraries in OS Linux. It checks interface signatures and data type definitions in two library versions (headers and shared objects) and searches ABI changes that may lead to incompatibility. We have released 1.2 version of this tool and we'd like you to consider its usefulness for your project. The wiki-page with the latest release of binary compatibility checker is http://ispras.linux-foundation.org/index.php/ABI_compliance_checker Andrey Ponomarenko

1 0

[libvirt] Corrected patch
by Jonas Eriksson 08 Aug '09

08 Aug '09

Sorry about the fauly patch, this is the correct one. /Jonas

4 4

[libvirt] [PATCH] Remove unsafe strncpy from esx_vmx.c
by Chris Lalancette 07 Aug '09

07 Aug '09

While trying to remove uses of unsafe strncpy in the tree, I came across a couple of usage in the ESX driver. To my eyes, the snprintf replacements do the same thing in less code, and are also safer. This has been compile tested only. Mattias, is there any reason we can't use these simpler (and safer) string routines? Signed-off-by: Chris Lalancette <clalance(a)redhat.com> Cc: Mattias Bolte <matthias.bolte(a)googlemail.com> --- src/esx/esx_vmx.c | 22 +++++++--------------- 1 files changed, 7 insertions(+), 15 deletions(-) diff --git a/src/esx/esx_vmx.c b/src/esx/esx_vmx.c index af5234e..91a86e2 100644 --- a/src/esx/esx_vmx.c +++ b/src/esx/esx_vmx.c @@ -397,10 +397,7 @@ def->parallels[0]... #define ESX_BUILD_VMX_NAME(_suffix) \ do { \ - strncpy(_suffix##_name, prefix, sizeof (_suffix##_name) - 1); \ - _suffix##_name[sizeof (_suffix##_name) - 1] = '\0'; \ - strncat(_suffix##_name, "."#_suffix, \ - sizeof (_suffix##_name) - 1 - strlen(_suffix##_name)); \ + snprintf(_suffix##_name, sizeof(_suffix##_name), "%s."#_suffix, prefix); \ } while (0) @@ -839,11 +836,9 @@ esxVMX_ParseSCSIController(virConnectPtr conn, virConfPtr conf, int controller, goto failure; } - strncpy(present_name, "scsiX.present", sizeof (virtualDev_name)); - strncpy(virtualDev_name, "scsiX.virtualDev", sizeof (virtualDev_name)); - - present_name[4] = '0' + controller; - virtualDev_name[4] = '0' + controller; + snprintf(present_name, sizeof(present_name), "scsi%d.present", controller); + snprintf(virtualDev_name, sizeof(virtualDev_name), "scsi%d.virtualDev", + controller); if (esxUtil_GetConfigBoolean(conn, conf, present_name, present, 0, 1) < 0) { goto failure; @@ -1333,8 +1328,7 @@ esxVMX_ParseEthernet(virConnectPtr conn, virConfPtr conf, int controller, goto failure; } - strncpy(prefix, "ethernetX", sizeof (prefix)); - prefix[8] = '0' + controller; + snprintf(prefix, sizeof(prefix), "ethernet%d", controller); ESX_BUILD_VMX_NAME(present); ESX_BUILD_VMX_NAME(startConnected); @@ -1514,8 +1508,7 @@ esxVMX_ParseSerial(virConnectPtr conn, virConfPtr conf, int port, goto failure; } - strncpy(prefix, "serialX", sizeof (prefix)); - prefix[6] = '0' + port; + snprintf(prefix, sizeof(prefix), "serial%d", port); ESX_BUILD_VMX_NAME(present); ESX_BUILD_VMX_NAME(startConnected); @@ -1627,8 +1620,7 @@ esxVMX_ParseParallel(virConnectPtr conn, virConfPtr conf, int port, goto failure; } - strncpy(prefix, "parallelX", sizeof (prefix)); - prefix[8] = '0' + port; + snprintf(prefix, sizeof(prefix), "parallel%d", port); ESX_BUILD_VMX_NAME(present); ESX_BUILD_VMX_NAME(startConnected); -- 1.6.0.6

2 2

[libvirt] compiling one driver
by Henrique Teófilo 07 Aug '09

07 Aug '09

Hello, I'm having problems to compile libvirt with opennebula driver. I executed: ./autogen.sh --with-one and got the following message: configure: error: You must install XMLRPC-C >= 1.14.0 to compile libvirt ONE driver but I already have installed libxmlrpc version 1.16.07-1 Where to configure this? I've tried to set the environment variables XMLRPC_CFLAGS and XMLRPC_LIBS, but nothing has changed. So I've tried to change these variables directly on configure script, that was empty, then errors started to appear on make execution: ../src/.libs/libvirt_driver_one.a(libvirt_driver_one_la-one_client.o): In function `c_oneFree': /home/objectweb/git/libvirt/src/./opennebula/one_client.c:207: undefined reference to `xmlrpc_env_clean' /home/objectweb/git/libvirt/src/./opennebula/one_client.c:208: undefined reference to `xmlrpc_client_cleanup' ../src/.libs/libvirt_driver_one.a(libvirt_driver_one_la-one_client.o): In function `c_oneReturnCode': /home/objectweb/git/libvirt/src/./opennebula/one_client.c:49: undefined reference to `xmlrpc_decompose_value' .....etc... What am I doing wrong?? Thanks, Henrique Teófilo

4 4

[libvirt] [PATCH 0/5] Various logging cleanups
by Amy Griffis 07 Aug '09

07 Aug '09

The following patches fixup libvirt's logging code, to hopefully make it a little more consistent and predictable. --- Amy Griffis (5): Tighten libvirt's parsing of logging environment variables Several fixes to libvirtd's log setup Cleanup VIR_LOG_DEBUG parsing in eventtest Consolidate code for parsing the logging environment variables Update logging documentation docs/logging.html.in | 22 +++++++- qemud/qemud.c | 90 +++++++++++++++++--------------- src/libvirt.c | 19 ------- src/libvirt_private.syms | 5 ++ src/logging.c | 129 +++++++++++++++++++++++++++++++++++++++------- src/logging.h | 5 ++ tests/eventtest.c | 12 +--- 7 files changed, 190 insertions(+), 92 deletions(-)

4 19

[libvirt] [PATCH] chown kernel/initrd before spawning qemu
by Mark McLoughlin 07 Aug '09

07 Aug '09

If we're running qemu unprivileged, we need to chown any supplied kernel or initrd before spawning it. * src/qemu_driver.c: rename qemuDomainSetDiskOwnership() to qemuDomainSetFileOwnership(), pass it a path string instead of a disk definition and use it for chowning the kernel/initrd in qemuDomainSetAllDeviceOwnership() --- src/qemu_driver.c | 20 ++++++++++++-------- 1 files changed, 12 insertions(+), 8 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 412b68d..bd58435 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -1684,18 +1684,18 @@ static int qemuDomainSetHostdevOwnership(virConnectPtr conn, } -static int qemuDomainSetDiskOwnership(virConnectPtr conn, - virDomainDiskDefPtr def, +static int qemuDomainSetFileOwnership(virConnectPtr conn, + const char *path, uid_t uid, gid_t gid) { - if (!def->src) + if (!path) return 0; - VIR_DEBUG("Setting ownership on %s to %d:%d", def->src, uid, gid); - if (chown(def->src, uid, gid) < 0) { + VIR_DEBUG("Setting ownership on %s to %d:%d", path, uid, gid); + if (chown(path, uid, gid) < 0) { virReportSystemError(conn, errno, _("cannot set ownership on %s"), - def->src); + path); return -1; } return 0; @@ -1725,7 +1725,7 @@ static int qemuDomainSetDeviceOwnership(virConnectPtr conn, (def->data.disk->readonly || def->data.disk->shared)) return 0; - return qemuDomainSetDiskOwnership(conn, def->data.disk, uid, gid); + return qemuDomainSetFileOwnership(conn, def->data.disk->src, uid, gid); case VIR_DOMAIN_DEVICE_HOSTDEV: return qemuDomainSetHostdevOwnership(conn, def->data.hostdev, uid, gid); @@ -1753,12 +1753,16 @@ static int qemuDomainSetAllDeviceOwnership(virConnectPtr conn, uid = restore ? 0 : driver->user; gid = restore ? 0 : driver->group; + if (qemuDomainSetFileOwnership(conn, def->os.kernel, uid, gid) < 0 || + qemuDomainSetFileOwnership(conn, def->os.initrd, uid, gid) < 0) + return -1; + for (i = 0 ; i < def->ndisks ; i++) { if (restore && (def->disks[i]->readonly || def->disks[i]->shared)) continue; - if (qemuDomainSetDiskOwnership(conn, def->disks[i], uid, gid) < 0) + if (qemuDomainSetFileOwnership(conn, def->disks[i]->src, uid, gid) < 0) return -1; } -- 1.6.2.5

2 3

[libvirt] VMware ESX 3/3: Cleanup varaible naming
by Matthias Bolte 06 Aug '09

06 Aug '09

Subject says it all. No functional change included. Matthias

3 2

[libvirt] virt-manager
by Michael Zeller 06 Aug '09

06 Aug '09

okay so using virt-manager it only lists my root partition as storage, or atleast thats what it appears So it says I can only make a vm the size of 9G when my home partition has 800+G is there a way to change that?

2 1

[libvirt] libvirt-0.3.3-14.el5 (redhat rpm) vs libvirt-0.6.5 (source)
by Jim Bergensen 06 Aug '09

06 Aug '09

Hello everyone, I am looking to write a simple php interface to create, stop, start, restart, configure, backup, transfer VM's across a network of servers. Storage is done locally on the particular node (lvm), but I am considering to do this through iscsi in the future. Xen, KVM, IP management, firewalling is an important requirement. The redhat backport version looks outdated, but I know the version number isn't the best way to compare features. My questions are: - Featurewise, is the difference between the redhat version and the libvirt.org source version small or as big as it looks (branch versions?) - Considering the above functions I am going to need, will the redhat version suffice? Any help would be truely appreciated, Thank you, Jim

4 4

[Fwd: Re: [libvirt] Release of libvirt-0.7.0]
by Robert L Cochran 06 Aug '09

06 Aug '09

Ooops! I sent this to Dan when I meant to send it to the list. To be a bit repetitive...how do I get this for Fedora 11 -- and all of its dependencies? Thanks Bob -------- Original Message -------- Subject: Re: [libvirt] Release of libvirt-0.7.0 Date: Wed, 05 Aug 2009 16:37:09 -0400 From: Robert L Cochran <cochranb(a)speakeasy.net> To: veillard(a)redhat.com How do I get this for Fedora 11 -- and all of its dependencies? Thanks a lot! Bob On 08/05/2009 11:15 AM, Daniel Veillard wrote: > A couple days later than expected, but considering the current > flow of fixes, that's not a bad thing. This is a huge release, > this includes more than 250 commits in a month and many new > functionalities or drivers, and a lot of improvements and bug fixes: > > New features: > - Interface implementation based on netcf (Laine Stump Daniel Veillard) > - Add new net filesystem glusterfs (Harshavardhana) > - Initial VMWare ESX driver (Matthias Bolte) > - Add support for VBox 3 and event callbacks on vbox (Pritesh Kothari) > - First version of the Power Hypervisor driver (Eduardo Otubo) > - Run QEMU guests as an unprivileged user (Daniel P. Berrange) > - Support cgroups in QEMU driver (Daniel P. Berrange) > - QEmu hotplug NIC support (Mark McLoughlin) > - Storage cloning for LVM and Disk backends(Cole Robinson) > - Switching to GIT (Jim Meyering) > > Documentation: > - Typo and comment fixes (Aron Griffis) > - Fix virCapabilitiesDefaultGuestMachine documentation. (Chris Lalancette) > - ESX Scheduler documentation and cleanup (Matthias Bolte) > - Update the java bindings page (Bryan Kearney) > - Added Matthias Bolte to AUTHORS list (Daniel Veillard) > - doc: clone+build instructions (Jim Meyering) > - docs: say that the old repository is deprecated... (Jim Meyering) > - document tcp listen and raw wire option (Guido Günther) > - Fix docs and code disagreements for character devices. (Cole Robinson) > - Fix documentation of virStoragePoolUndefine return (Thomas Treutner) > - Fix gitweb link on download page. (Cole Robinson) > - update download informations after switch to git (Daniel Veillard) > - Update links to bugzilla (Garry Dolley) > - Update the links for RHEL libvirt bugzillas (Garry Dolley) > > Portability: > - Xen Inotify support needs sys/inotify.h (Maximilian Wilhelm) > - Workaround for broken GCC in Debian Etch (Maximilian Wilhelm) > - LXC driver requires sched.h and unshare() (Maximilian Wilhelm) > - Configure UML support only if sys/inotify.h present (Maximilian Wilhelm) > - Fix libcurl automatic check and ESX status (Maximilian Wilhelm) > - Enable ESX driver build on Mingw32 (Daniel P. Berrange) > - Fix build on mingw32 by disabling netcf (Daniel P. Berrange) > - Reduce glusterfs dependency to 2.0.1 (Mark McLoughlin) > - Desactivate phyp build and indicate libssh builreq (Daniel Veillard) > - Fix misc Win32 compile warnings (Daniel P. Berrange) > - Rename variable for compilation in Mingw32 (end) (Laine Stump) > - Rename variable for compilation in Mingw32 (Laine Stump) > - rpm spec cleanup and split off client only package (Daniel Veillard) > > Bug fixes: > - Add uniqueness checking for LXC define/create methods (Daniel P. Berrange) > - Fix removal of transient VMs when LXC aborts (Daniel P. Berrange) > - Don't try to activate cgroups if not present for LXC (Daniel P. Berrange) > - Refresh /etc/xen if inotify wasn't (Cole Robinson) > - Don't loose id on xen domain redefine (Cole Robinson) > - Fix memory leak in openvz driver (Daniel P. Berrange) > - Protected against potential crash scenarios (Daniel P. Berrange) > - Fix crash when attempting to shutdown inactive QEMU vm (Daniel P. Berrange) > - Fix PCIe FLR detection (Mark McLoughlin) > - Set perms on /var/lib/libvirt/images to 0711 (Mark McLoughlin) > - Fix problem writing QEMU pidfile (Daniel P. Berrange) > - Fix vcpupin on Xen problem (Henrik Persson) > - Fix RPM upgrades from F11 to F12 (Daniel P. Berrange) > - Fix deadlock in remote driver domain events (Daniel P. Berrange) > - qemu: fix monitor socket reconnection (Ryota Ozaki) > - Fix polkit/netcf disabling on older fedoras (Mark McLoughlin) > - Fix crashes in Xen capabilities code (Daniel P. Berrange) > - Always add -no-kvm and -no-kqemu, for qemu domains (Jim Paris) > - Avoid raising an internal error (Paolo Bonzini) > - Don't allow NULL paths for BlockStats and InterfaceStats (Cole Robinson) > - Don't leak vm->monitorpath on re-connect (Mark McLoughlin) > - Don't restore labels on shared/readonly disks (Daniel P. Berrange) > - Ensure spawned children have a stderr/out set to /dev/null if requested (Daniel P. Berrange) > - Ensure test:/// URIs get routed to the non-privileged libvirtd (Daniel P. Berrange) > - fix another failing "make distcheck" (qemuhelptest) (Jim Meyering) > - Fix an uninitialized variable in Unix socket open (Jun Koi) > - Fix configure flags in spec file (Daniel Veillard) > - Fix error reporting for security driver over remote protocol (Daniel P. Berrange) > - fix failing "make distcheck" (Jim Meyering) > - Fix free of unitialized data upon PCI open fail (Daniel P. Berrange) > - Fix informations about previous git server (Daniel Veillard) > - Fix memory leaks in esxDomainDumpXML (Matthias Bolte) > - Fix multiple memory leaks in virsh (Laine Stump) > - Fix PCI device hotplug/unplug with newer QEMU (Daniel P. Berrange) > - Fix problem with QEMU monitor welcome prompt confusing libvirt after a libvirtd daemon restart with active guests (Daniel P. Berrange) > - Fix python examples to use read-write conn (Dan Kenigsberg) > - Fix reconnect bug for VBox (Pritesh Kothari) > - Fix SELinux denial during hotplug (Daniel P. Berrange) > - Fix typo in check for glusterfs format pools (Daniel P. Berrange) > - Fix typo in storage cloning (Daniel P. Berrange) > - qemu: Check driver is initialized up front, to avoid segfault. (Cole Robinson) > - qemu: Try multiple times to open unix monitor socket (Cole Robinson) > - Release conn lock before reporting errors (end) (Laine Stump) > - Release conn lock before reporting interface errors (Laine Stump) > - Remove the network backend if NIC hotplug fails (Mark McLoughlin) > - Set specific flags for glusterfs fs mounts (Harshavardhana) > - storage: disk: Default to 'ext2' for new volumes. (Cole Robinson) > - storage: disk: Fix parthelper '-g' option handling. (Cole Robinson) > - storage: disk: Fix segfault creating volume without target path (Cole Robinson) > - storage: Fix deadlock when cloning across pools. (Cole Robinson) > - Update modified mac address in place in virGetInterface (Laine Stump) > > Improvements: > - Add an allocation unit when calling qemu-img (Ryota Ozaki) > - Improve diagnostics when pidfile writing fails (Daniel P. Berrange) > - Disable IPv6 on virtual networks (Daniel P. Berrange) > - Allow dnsmasq to provide DNS without DHCP (Daniel P. Berrange) > - Fix an initialization problem in previous patch (Aron Griffis) > - Remove MAX_TAP_ID and let kernel do numbering (Aron Griffis) > - Kernel command line support for UML (Ron Yorston) > - Activate the interface drivers, and cleanups (Daniel Veillard) > - Add an error code for conflicting mac addresses (Laine Stump) > - Add a test interface driver (Laine Stump) > - Add canonical machine name to capabilities output (Mark McLoughlin) > - add cd and pwd commands to virsh (Paolo Bonzini) > - Add checks for some NIC hotplug related features added in qemu-0.10.0 (Mark McLoughlin) > - Add domain autostart for LXC driver (Daniel P. Berrange) > - Add domain events support to LXC driver (Daniel P. Berrange) > - Add interface object list manipulation functions (Laine Stump) > - Add internal XML parsing/formatting flag (Mark McLoughlin) > - Add netcf XML schemas and test data (Daniel Veillard) > - Add new net filesystem glusterfs (Harshavardhana) > - Add NIC and hostnet names to domain state XML (Mark McLoughlin) > - Add no_verify query parameter to ESX URIs (Matthias Bolte) > - Add SCM_RIGHTS support to QEMU monitor code (Mark McLoughlin) > - Add support for attaching network/bridge NICs in QEMU driver (Mark McLoughlin) > - add support for netcf XML import and export (Daniel Veillard) > - Add support for network device detach (Mark McLoughlin) > - Add support for physical memory access for QEmu (Nguyen Anh Quynh) > - Add support for VBox 3 and event callbacks on vbox (Pritesh Kothari) > - Add the monitor type to the domain state XML (Mark McLoughlin) > - Add virCapsGuestMachine structure (Mark McLoughlin) > - Add virsh commands for network interface management (Laine Stump) > - Allow autostart of libvirtd to be disabled with LIBVIRT_AUTOSTART=0 (Daniel P. Berrange) > - Allow leading dots in VMX config entry names (Matthias Bolte) > - Assign names to qemu NICs and network backends (Mark McLoughlin) > - Basic qemu NIC hotplug support (Mark McLoughlin) > - build: adjust aclocal's search patch to prefer gnulib's m4 files. (Jim Meyering) > - build: automatically rerun ./bootstrap when needed (Jim Meyering) > - build: make autogen.sh use autoreconf -if (Jim Meyering) > - build: submodule machinery now works also when no tag is reachable (Jim Meyering) > - Canonicalize qemu machine types (Mark McLoughlin) > - Change code generator to give async event messages their own postfix (Daniel P. Berrange) > - Change the way client event loop watches are managed (Daniel P. Berrange) > - Decode incoming request header before invoking dispatch code (Daniel P. Berrange) > - Define an API for registering incoming message dispatch filters (Daniel P. Berrange) > - ESX driver accept VI API version 4.0 (Matthias Bolte) > - Fill in vCPU<-> pCPU current mapping, and vCPU cpuTime for QEMU (Daniel P. Berrange) > - generate ChangeLog from git logs into distribution tarball (Jim Meyering) > - Implement qemu dump capabilities (Paolo Bonzini) > - Implement schedular tunables API using cgroups (Daniel P. Berrange) > - Implement the new virinterface functions (Laine Stump) > - Make cgroups a little more efficient (Daniel P. Berrange) > - Make it easier to debug tests running programs (Daniel P. Berrange) > - Make qemuBuildHostNetStr() take tapfd as a string (Mark McLoughlin) > - Make QEMU cgroups use configurable (Daniel P. Berrange) > - Make qemuCmdFlags available in qemudDomainAttachDevice() (Mark McLoughlin) > - Move queuing of RPC replies into dispatch code (Daniel P. Berrange) > - Move vnet_hdr logic into qemudNetworkIfaceConnect() and export it (Mark McLoughlin) > - Netcf based interface driver implementation (Laine Stump) > - netcf XML validation and input and output tests (Daniel Veillard) > - Only probe qemu for machine types when binary changes (Mark McLoughlin) > - Place every QEMU guest in a private cgroup (Daniel P. Berrange) > - Probe for QEMU machine types (Mark McLoughlin) > - Probe QEMU directly for machine aliases if not found in capabilties (Mark McLoughlin) > - Public API for new virInterface functions (Laine Stump) > - python: Raise exceptions if virDomain*Stats fail. (Cole Robinson) > - Refactor cgroups to allow a group per driver to be managed directly (Daniel P. Berrange) > - Re-factor pci_add reply parsing and parse domain/bus numbers (Mark McLoughlin) > - Retain disk PCI address across libvirtd restarts (Mark McLoughlin) > - Retain PCI address from NIC attach (Mark McLoughlin) > - Run QEMU guests as an unprivileged user (Daniel P. Berrange) > - Separate code for encoding outgoing remote message headers (Daniel P. Berrange) > - Split generic RPC message dispatch code out from remote protocol API handlers (Daniel P. Berrange) > - Split out code for handling incoming method call messages (Daniel P. Berrange) > - storage: Break out actual raw cloning to separate function. (Cole Robinson) > - storage: cleanup: do away with 'createFile' (Cole Robinson) > - storage: disk: Use capacity, not allocation, when creating volume. (Cole Robinson) > - storage: Don't try sparse detection if writing to block device. (Cole Robinson) > - storage: Implement 'CreateBlockFrom' helper. (Cole Robinson) > - storage: Implement CreateVolFrom for logical and disk backend. (Cole Robinson) > - storage: Move most of the FS creation functions to common backend. (Cole Robinson) > - storage: Refactor FS backend 'create' function choosing. (Cole Robinson) > - Store the interface vlan number in the domain state (Mark McLoughlin) > - Support<video> element for QEMU guests (Daniel P. Berrange) > - Support<video> tag for defining VGA card properties (Daniel P. Berrange) > - Switch to using a unix socket for the qemu monitor (Mark McLoughlin) > - test: Generate net interface names when assigning XML. (Cole Robinson) > - test: Implement BlockStats and InterfaceStats (Cole Robinson) > - Use cgroups for block device whitelisting in QEMU guests (Daniel P. Berrange) > - Use enums for cgroup controller types / labels (Daniel P. Berrange) > - Use sendmsg() on QEMU monitor socket (Mark McLoughlin) > - Use virDomainChrTypeFromString() instead of open coding (Mark McLoughlin) > - Use virFileReadAll/virFileWriteStr for key cgroup read/write helpers (Daniel P. Berrange) > - virGetinterface matching of MAC and interface name (Laine Stump) > > Cleanups: > - Fix configure checks from previous commits (Daniel P. Berrange) > - Avoid a warning if compiling without inotify (Daniel P. Berrange) > - Remove a stray semicolon (Daniel Veillard) > - Extend the ESX URL to habdle ports and GSX (Matthias Bolte) > - Fix escaping of 8-bit high characters (Daniel P. Berrange) > - Remove ATTRIBUTE_UNUSED from flags to qemudDomainMigratePerform. (Chris Lalancette) > - Add a comment about setting errors after qemudStartVMDaemon(). (Chris Lalancette) > - Fix an erroneous debug error to KVM; it should read QEMU/KVM. (Chris Lalancette) > - Remove a stray semicolon in qemudDomainMigratePrepare2. (Chris Lalancette) > - Convert a few stray users of free() in libvirt.c to VIR_FREE(). (Chris Lalancette) > - Use virGetHostname instead of gethostname. (Chris Lalancette) > - Fix up a minor indentation issue with virDomainMigratePrepare. (Chris Lalancette) > - Fix up a silly typo in apibuild.py. (Chris Lalancette) > - Avoid warning when compiling without IFF_VNET_HDR (Maximilian Wilhelm) > - Capilize libvirt-client summary (Mark McLoughlin) > - Move ldconfig calls to libvirt-client %post/%postun (Mark McLoughlin) > - Convert NEWS to UTF-8 (Mark McLoughlin) > - Fix trailing whitespace in NEWS (Mark McLoughlin) > - No need to build require both python-devel and python (Mark McLoughlin) > - Remove executable perms from /etc/sysconfig/libvirtd (Mark McLoughlin) > - Use a %postun -p for one line scriptlet (Mark McLoughlin) > - Don't explicitly require libxml2 (Mark McLoughlin) > - Fix some unowned directories (Mark McLoughlin) > - Kill qemu BuildRequires (Mark McLoughlin) > - Enable netcf by default (Mark McLoughlin) > - Default to with_polkit (Mark McLoughlin) > - Make vbox support configurable (Mark McLoughlin) > - Build with --without-capng if capng is disabled (Mark McLoughlin) > - BuildRequires libcap-ng-devel not capng-devel (Mark McLoughlin) > - Drop curl host check when using ESX without check (Shahar Klein) > - Fix typo in xen capabilities code (Daniel P. Berrange) > - Add bare format string to printf-derivatives troubles (Laine Stump) > - Add phyp files to POTFILES, to make syntax-check happy. (Cole Robinson) > - avoid a make distcheck failure: distribute docs/schemas/interface.rng (Jim Meyering) > - avoid a make distcheck failure: distribute tests/interfaceschemadata/ (Jim Meyering) > - avoid a "make syntax-check" failure (Jim Meyering) > - build: do not emit a trailing blank line into VC'd file, NEWS (Jim Meyering) > - build: update from gnulib, for latest maint.mk (Jim Meyering) > - Clean up error handling in qemudDomainAttachNetDevice() (Mark McLoughlin) > - Cleanup qemu binary detection logic in qemudCapsInitGuest() (Mark McLoughlin) > - Commit newly generated docs, after changes from commit 2348cf. (Cole Robinson) > - Factor qemuBuildHostNetStr() out from qemuBuildCommandLine() (Mark McLoughlin) > - Factor qemuBuildNicStr() out from qemuBuildCommandLine() (Mark McLoughlin) > - Factor qemudMonitorSend() out of qemudMonitorCommandExtra() (Mark McLoughlin) > - Fix cgroup compile warnings (Daniel P. Berrange) > - Fix misc build problems due to new drivers (Daniel P. Berrange) > - Fix wierd build problems due to autopoint overwriting gnulib m4 (Daniel P. Berrange) > - Makefile.cfg: Rename to... (Jim Meyering) > - make .gnulib a submodule (Jim Meyering) > - make "make syntax-check" consistent with "git diff --check" (Jim Meyering) > - Minor qemu monitor coding style fixes (Mark McLoughlin) > - Prepare to use maint.mk from gnulib (Jim Meyering) > - Refactor incoming message handling to prepare for data stream support (Daniel P. Berrange) > - Refactor message sending to allow code reuse for data streams (Daniel P. Berrange) > - remove all .cvsignore files (Jim Meyering) > - remove all trailing blank lines (Jim Meyering) > - Remove some unused variables and cut long lines (Daniel Veillard) > - Remove trailing blank lines (Daniel Veillard) > - Rename a bunch of internal methods to clarify their meaning (Daniel P. Berrange) > - Rename 'direction' to 'type' in remote_message_header (Daniel P. Berrange) > - Report the object name on lookup error (Daniel Veillard) > - Simplify remote driver error reporting (Daniel P. Berrange) > - skip some of gnulib's new rules (Jim Meyering) > - use automake-1.11's silent-rules option, when possible (Jim Meyering) > - use gnumakefile and maintainer-makefile modules from gnulib (Jim Meyering) > > > It's also good to see a lot of new contributors in this release, > so thanks to everybody ! > > Daniel > >

2 1

[libvirt] VMware ESX 2/3: Fix memleak if esxOpen fails
by Matthias Bolte 06 Aug '09

06 Aug '09

Subject says it all. Matthias

2 1

[libvirt] VMware ESX 1/3: Add proper OOM reporting for esxDomainGetOSType
by Matthias Bolte 06 Aug '09

06 Aug '09

Subject says it all. Matthias

2 1

[libvirt] running with UID != 0: internal error about bridged network
by Harald Dunkel 06 Aug '09

06 Aug '09

Hi folks, I would like to avoid running the virtual hosts with UID=0. But if I try to create a new virtual machine without being root, then it fails with % name=Lenny_amd64 % kvmdir=/local/kvm % mkdir -p $kvmdir/$name % qemu-img create -f qcow2 $kvmdir/$name/hda.qcow2 32G Formatting '/local/kvm/Lenny_amd64/hda.qcow2', fmt=qcow2, size=33554432 kB % virt-install --connect=qemu:///session -n $name -r 1024 -vcpus=1 -f $kvmdir/$name/hda.qcow2 --network=bridge:br0 --vnc --accelerate -v -c /usr/local/source/hdunkel/debian-502-i386-netinst.iso --os-type=linux --os-variant=debianLenny 14:18:35.586: warning : Cannot set group when not running as root 14:18:35.757: error : Domain not found: no domain with matching name 'Lenny_amd64' libvir: QEMU error : Domain not found: no domain with matching name 'Lenny_amd64' The log file shows [Wed, 05 Aug 2009 14:18:35 virt-install 14229] ERROR (virt-install:786) internal error Failed to add tap interface 'vnet%d' to bridge 'br0' : Operation not permitted Traceback (most recent call last): File "/usr/bin/virt-install", line 780, in <module> main() File "/usr/bin/virt-install", line 678, in main start_time, guest.start_install) File "/usr/bin/virt-install", line 733, in do_install dom = install_func(conscb, progresscb, wait=(not wait)) File "/var/lib/python-support/python2.5/virtinst/Guest.py", line 541, in start_install return self._do_install(consolecb, meter, removeOld, wait) File "/var/lib/python-support/python2.5/virtinst/Guest.py", line 633, in _do_install self.domain = self.conn.createLinux(install_xml, 0) File "/usr/lib/python2.5/site-packages/libvirt.py", line 892, in createLinux if ret is None:raise libvirtError('virDomainCreateLinux() failed', conn=self) libvirtError: internal error Failed to add tap interface 'vnet%d' to bridge 'br0' : Operation not permitted Of course an unprivileged user is not allowed to mess around with the network interfaces, but since it says "internal error" I wonder how this is supposed to work? Is there something misconfigured? Any helpful comments would be highly appreciated Harri

2 1

[libvirt] [PATCH 0/7] QEMU network/bridge backed NIC hotplug
by Mark McLoughlin 06 Aug '09

06 Aug '09

Hey, So, here are the patches needed to complete the QEMU NIC hotplug work by supporting network and bridge backed interfaces. We support this by passing a tap file descriptor to qemu over the monitor socket using SCM_RIGHTS. The monitor commands required for this (getfd and closefd) are queued up in Anthony's queue and should be in qemu-0.11-rc1 which is due to be released very soon: http://repo.or.cz/w/qemu/aliguori-queue.git?a=commitdiff;h=4edd6c2645 http://repo.or.cz/w/qemu/aliguori-queue.git?a=commitdiff;h=f4069061e3 Note: I just remembered I'm not handling missing support for these commands very well. I'll make it handle an 'unknown command' reply gracefully by failing the attach tommorrow. Cheers, Mark.

6 41

[libvirt] Release of libvirt-0.7.0
by Daniel Veillard 06 Aug '09

06 Aug '09

A couple days later than expected, but considering the current flow of fixes, that's not a bad thing. This is a huge release, this includes more than 250 commits in a month and many new functionalities or drivers, and a lot of improvements and bug fixes: New features: - Interface implementation based on netcf (Laine Stump Daniel Veillard) - Add new net filesystem glusterfs (Harshavardhana) - Initial VMWare ESX driver (Matthias Bolte) - Add support for VBox 3 and event callbacks on vbox (Pritesh Kothari) - First version of the Power Hypervisor driver (Eduardo Otubo) - Run QEMU guests as an unprivileged user (Daniel P. Berrange) - Support cgroups in QEMU driver (Daniel P. Berrange) - QEmu hotplug NIC support (Mark McLoughlin) - Storage cloning for LVM and Disk backends(Cole Robinson) - Switching to GIT (Jim Meyering) Documentation: - Typo and comment fixes (Aron Griffis) - Fix virCapabilitiesDefaultGuestMachine documentation. (Chris Lalancette) - ESX Scheduler documentation and cleanup (Matthias Bolte) - Update the java bindings page (Bryan Kearney) - Added Matthias Bolte to AUTHORS list (Daniel Veillard) - doc: clone+build instructions (Jim Meyering) - docs: say that the old repository is deprecated... (Jim Meyering) - document tcp listen and raw wire option (Guido Günther) - Fix docs and code disagreements for character devices. (Cole Robinson) - Fix documentation of virStoragePoolUndefine return (Thomas Treutner) - Fix gitweb link on download page. (Cole Robinson) - update download informations after switch to git (Daniel Veillard) - Update links to bugzilla (Garry Dolley) - Update the links for RHEL libvirt bugzillas (Garry Dolley) Portability: - Xen Inotify support needs sys/inotify.h (Maximilian Wilhelm) - Workaround for broken GCC in Debian Etch (Maximilian Wilhelm) - LXC driver requires sched.h and unshare() (Maximilian Wilhelm) - Configure UML support only if sys/inotify.h present (Maximilian Wilhelm) - Fix libcurl automatic check and ESX status (Maximilian Wilhelm) - Enable ESX driver build on Mingw32 (Daniel P. Berrange) - Fix build on mingw32 by disabling netcf (Daniel P. Berrange) - Reduce glusterfs dependency to 2.0.1 (Mark McLoughlin) - Desactivate phyp build and indicate libssh builreq (Daniel Veillard) - Fix misc Win32 compile warnings (Daniel P. Berrange) - Rename variable for compilation in Mingw32 (end) (Laine Stump) - Rename variable for compilation in Mingw32 (Laine Stump) - rpm spec cleanup and split off client only package (Daniel Veillard) Bug fixes: - Add uniqueness checking for LXC define/create methods (Daniel P. Berrange) - Fix removal of transient VMs when LXC aborts (Daniel P. Berrange) - Don't try to activate cgroups if not present for LXC (Daniel P. Berrange) - Refresh /etc/xen if inotify wasn't (Cole Robinson) - Don't loose id on xen domain redefine (Cole Robinson) - Fix memory leak in openvz driver (Daniel P. Berrange) - Protected against potential crash scenarios (Daniel P. Berrange) - Fix crash when attempting to shutdown inactive QEMU vm (Daniel P. Berrange) - Fix PCIe FLR detection (Mark McLoughlin) - Set perms on /var/lib/libvirt/images to 0711 (Mark McLoughlin) - Fix problem writing QEMU pidfile (Daniel P. Berrange) - Fix vcpupin on Xen problem (Henrik Persson) - Fix RPM upgrades from F11 to F12 (Daniel P. Berrange) - Fix deadlock in remote driver domain events (Daniel P. Berrange) - qemu: fix monitor socket reconnection (Ryota Ozaki) - Fix polkit/netcf disabling on older fedoras (Mark McLoughlin) - Fix crashes in Xen capabilities code (Daniel P. Berrange) - Always add -no-kvm and -no-kqemu, for qemu domains (Jim Paris) - Avoid raising an internal error (Paolo Bonzini) - Don't allow NULL paths for BlockStats and InterfaceStats (Cole Robinson) - Don't leak vm->monitorpath on re-connect (Mark McLoughlin) - Don't restore labels on shared/readonly disks (Daniel P. Berrange) - Ensure spawned children have a stderr/out set to /dev/null if requested (Daniel P. Berrange) - Ensure test:/// URIs get routed to the non-privileged libvirtd (Daniel P. Berrange) - fix another failing "make distcheck" (qemuhelptest) (Jim Meyering) - Fix an uninitialized variable in Unix socket open (Jun Koi) - Fix configure flags in spec file (Daniel Veillard) - Fix error reporting for security driver over remote protocol (Daniel P. Berrange) - fix failing "make distcheck" (Jim Meyering) - Fix free of unitialized data upon PCI open fail (Daniel P. Berrange) - Fix informations about previous git server (Daniel Veillard) - Fix memory leaks in esxDomainDumpXML (Matthias Bolte) - Fix multiple memory leaks in virsh (Laine Stump) - Fix PCI device hotplug/unplug with newer QEMU (Daniel P. Berrange) - Fix problem with QEMU monitor welcome prompt confusing libvirt after a libvirtd daemon restart with active guests (Daniel P. Berrange) - Fix python examples to use read-write conn (Dan Kenigsberg) - Fix reconnect bug for VBox (Pritesh Kothari) - Fix SELinux denial during hotplug (Daniel P. Berrange) - Fix typo in check for glusterfs format pools (Daniel P. Berrange) - Fix typo in storage cloning (Daniel P. Berrange) - qemu: Check driver is initialized up front, to avoid segfault. (Cole Robinson) - qemu: Try multiple times to open unix monitor socket (Cole Robinson) - Release conn lock before reporting errors (end) (Laine Stump) - Release conn lock before reporting interface errors (Laine Stump) - Remove the network backend if NIC hotplug fails (Mark McLoughlin) - Set specific flags for glusterfs fs mounts (Harshavardhana) - storage: disk: Default to 'ext2' for new volumes. (Cole Robinson) - storage: disk: Fix parthelper '-g' option handling. (Cole Robinson) - storage: disk: Fix segfault creating volume without target path (Cole Robinson) - storage: Fix deadlock when cloning across pools. (Cole Robinson) - Update modified mac address in place in virGetInterface (Laine Stump) Improvements: - Add an allocation unit when calling qemu-img (Ryota Ozaki) - Improve diagnostics when pidfile writing fails (Daniel P. Berrange) - Disable IPv6 on virtual networks (Daniel P. Berrange) - Allow dnsmasq to provide DNS without DHCP (Daniel P. Berrange) - Fix an initialization problem in previous patch (Aron Griffis) - Remove MAX_TAP_ID and let kernel do numbering (Aron Griffis) - Kernel command line support for UML (Ron Yorston) - Activate the interface drivers, and cleanups (Daniel Veillard) - Add an error code for conflicting mac addresses (Laine Stump) - Add a test interface driver (Laine Stump) - Add canonical machine name to capabilities output (Mark McLoughlin) - add cd and pwd commands to virsh (Paolo Bonzini) - Add checks for some NIC hotplug related features added in qemu-0.10.0 (Mark McLoughlin) - Add domain autostart for LXC driver (Daniel P. Berrange) - Add domain events support to LXC driver (Daniel P. Berrange) - Add interface object list manipulation functions (Laine Stump) - Add internal XML parsing/formatting flag (Mark McLoughlin) - Add netcf XML schemas and test data (Daniel Veillard) - Add new net filesystem glusterfs (Harshavardhana) - Add NIC and hostnet names to domain state XML (Mark McLoughlin) - Add no_verify query parameter to ESX URIs (Matthias Bolte) - Add SCM_RIGHTS support to QEMU monitor code (Mark McLoughlin) - Add support for attaching network/bridge NICs in QEMU driver (Mark McLoughlin) - add support for netcf XML import and export (Daniel Veillard) - Add support for network device detach (Mark McLoughlin) - Add support for physical memory access for QEmu (Nguyen Anh Quynh) - Add support for VBox 3 and event callbacks on vbox (Pritesh Kothari) - Add the monitor type to the domain state XML (Mark McLoughlin) - Add virCapsGuestMachine structure (Mark McLoughlin) - Add virsh commands for network interface management (Laine Stump) - Allow autostart of libvirtd to be disabled with LIBVIRT_AUTOSTART=0 (Daniel P. Berrange) - Allow leading dots in VMX config entry names (Matthias Bolte) - Assign names to qemu NICs and network backends (Mark McLoughlin) - Basic qemu NIC hotplug support (Mark McLoughlin) - build: adjust aclocal's search patch to prefer gnulib's m4 files. (Jim Meyering) - build: automatically rerun ./bootstrap when needed (Jim Meyering) - build: make autogen.sh use autoreconf -if (Jim Meyering) - build: submodule machinery now works also when no tag is reachable (Jim Meyering) - Canonicalize qemu machine types (Mark McLoughlin) - Change code generator to give async event messages their own postfix (Daniel P. Berrange) - Change the way client event loop watches are managed (Daniel P. Berrange) - Decode incoming request header before invoking dispatch code (Daniel P. Berrange) - Define an API for registering incoming message dispatch filters (Daniel P. Berrange) - ESX driver accept VI API version 4.0 (Matthias Bolte) - Fill in vCPU <-> pCPU current mapping, and vCPU cpuTime for QEMU (Daniel P. Berrange) - generate ChangeLog from git logs into distribution tarball (Jim Meyering) - Implement qemu dump capabilities (Paolo Bonzini) - Implement schedular tunables API using cgroups (Daniel P. Berrange) - Implement the new virinterface functions (Laine Stump) - Make cgroups a little more efficient (Daniel P. Berrange) - Make it easier to debug tests running programs (Daniel P. Berrange) - Make qemuBuildHostNetStr() take tapfd as a string (Mark McLoughlin) - Make QEMU cgroups use configurable (Daniel P. Berrange) - Make qemuCmdFlags available in qemudDomainAttachDevice() (Mark McLoughlin) - Move queuing of RPC replies into dispatch code (Daniel P. Berrange) - Move vnet_hdr logic into qemudNetworkIfaceConnect() and export it (Mark McLoughlin) - Netcf based interface driver implementation (Laine Stump) - netcf XML validation and input and output tests (Daniel Veillard) - Only probe qemu for machine types when binary changes (Mark McLoughlin) - Place every QEMU guest in a private cgroup (Daniel P. Berrange) - Probe for QEMU machine types (Mark McLoughlin) - Probe QEMU directly for machine aliases if not found in capabilties (Mark McLoughlin) - Public API for new virInterface functions (Laine Stump) - python: Raise exceptions if virDomain*Stats fail. (Cole Robinson) - Refactor cgroups to allow a group per driver to be managed directly (Daniel P. Berrange) - Re-factor pci_add reply parsing and parse domain/bus numbers (Mark McLoughlin) - Retain disk PCI address across libvirtd restarts (Mark McLoughlin) - Retain PCI address from NIC attach (Mark McLoughlin) - Run QEMU guests as an unprivileged user (Daniel P. Berrange) - Separate code for encoding outgoing remote message headers (Daniel P. Berrange) - Split generic RPC message dispatch code out from remote protocol API handlers (Daniel P. Berrange) - Split out code for handling incoming method call messages (Daniel P. Berrange) - storage: Break out actual raw cloning to separate function. (Cole Robinson) - storage: cleanup: do away with 'createFile' (Cole Robinson) - storage: disk: Use capacity, not allocation, when creating volume. (Cole Robinson) - storage: Don't try sparse detection if writing to block device. (Cole Robinson) - storage: Implement 'CreateBlockFrom' helper. (Cole Robinson) - storage: Implement CreateVolFrom for logical and disk backend. (Cole Robinson) - storage: Move most of the FS creation functions to common backend. (Cole Robinson) - storage: Refactor FS backend 'create' function choosing. (Cole Robinson) - Store the interface vlan number in the domain state (Mark McLoughlin) - Support <video> element for QEMU guests (Daniel P. Berrange) - Support <video> tag for defining VGA card properties (Daniel P. Berrange) - Switch to using a unix socket for the qemu monitor (Mark McLoughlin) - test: Generate net interface names when assigning XML. (Cole Robinson) - test: Implement BlockStats and InterfaceStats (Cole Robinson) - Use cgroups for block device whitelisting in QEMU guests (Daniel P. Berrange) - Use enums for cgroup controller types / labels (Daniel P. Berrange) - Use sendmsg() on QEMU monitor socket (Mark McLoughlin) - Use virDomainChrTypeFromString() instead of open coding (Mark McLoughlin) - Use virFileReadAll/virFileWriteStr for key cgroup read/write helpers (Daniel P. Berrange) - virGetinterface matching of MAC and interface name (Laine Stump) Cleanups: - Fix configure checks from previous commits (Daniel P. Berrange) - Avoid a warning if compiling without inotify (Daniel P. Berrange) - Remove a stray semicolon (Daniel Veillard) - Extend the ESX URL to habdle ports and GSX (Matthias Bolte) - Fix escaping of 8-bit high characters (Daniel P. Berrange) - Remove ATTRIBUTE_UNUSED from flags to qemudDomainMigratePerform. (Chris Lalancette) - Add a comment about setting errors after qemudStartVMDaemon(). (Chris Lalancette) - Fix an erroneous debug error to KVM; it should read QEMU/KVM. (Chris Lalancette) - Remove a stray semicolon in qemudDomainMigratePrepare2. (Chris Lalancette) - Convert a few stray users of free() in libvirt.c to VIR_FREE(). (Chris Lalancette) - Use virGetHostname instead of gethostname. (Chris Lalancette) - Fix up a minor indentation issue with virDomainMigratePrepare. (Chris Lalancette) - Fix up a silly typo in apibuild.py. (Chris Lalancette) - Avoid warning when compiling without IFF_VNET_HDR (Maximilian Wilhelm) - Capilize libvirt-client summary (Mark McLoughlin) - Move ldconfig calls to libvirt-client %post/%postun (Mark McLoughlin) - Convert NEWS to UTF-8 (Mark McLoughlin) - Fix trailing whitespace in NEWS (Mark McLoughlin) - No need to build require both python-devel and python (Mark McLoughlin) - Remove executable perms from /etc/sysconfig/libvirtd (Mark McLoughlin) - Use a %postun -p for one line scriptlet (Mark McLoughlin) - Don't explicitly require libxml2 (Mark McLoughlin) - Fix some unowned directories (Mark McLoughlin) - Kill qemu BuildRequires (Mark McLoughlin) - Enable netcf by default (Mark McLoughlin) - Default to with_polkit (Mark McLoughlin) - Make vbox support configurable (Mark McLoughlin) - Build with --without-capng if capng is disabled (Mark McLoughlin) - BuildRequires libcap-ng-devel not capng-devel (Mark McLoughlin) - Drop curl host check when using ESX without check (Shahar Klein) - Fix typo in xen capabilities code (Daniel P. Berrange) - Add bare format string to printf-derivatives troubles (Laine Stump) - Add phyp files to POTFILES, to make syntax-check happy. (Cole Robinson) - avoid a make distcheck failure: distribute docs/schemas/interface.rng (Jim Meyering) - avoid a make distcheck failure: distribute tests/interfaceschemadata/ (Jim Meyering) - avoid a "make syntax-check" failure (Jim Meyering) - build: do not emit a trailing blank line into VC'd file, NEWS (Jim Meyering) - build: update from gnulib, for latest maint.mk (Jim Meyering) - Clean up error handling in qemudDomainAttachNetDevice() (Mark McLoughlin) - Cleanup qemu binary detection logic in qemudCapsInitGuest() (Mark McLoughlin) - Commit newly generated docs, after changes from commit 2348cf. (Cole Robinson) - Factor qemuBuildHostNetStr() out from qemuBuildCommandLine() (Mark McLoughlin) - Factor qemuBuildNicStr() out from qemuBuildCommandLine() (Mark McLoughlin) - Factor qemudMonitorSend() out of qemudMonitorCommandExtra() (Mark McLoughlin) - Fix cgroup compile warnings (Daniel P. Berrange) - Fix misc build problems due to new drivers (Daniel P. Berrange) - Fix wierd build problems due to autopoint overwriting gnulib m4 (Daniel P. Berrange) - Makefile.cfg: Rename to... (Jim Meyering) - make .gnulib a submodule (Jim Meyering) - make "make syntax-check" consistent with "git diff --check" (Jim Meyering) - Minor qemu monitor coding style fixes (Mark McLoughlin) - Prepare to use maint.mk from gnulib (Jim Meyering) - Refactor incoming message handling to prepare for data stream support (Daniel P. Berrange) - Refactor message sending to allow code reuse for data streams (Daniel P. Berrange) - remove all .cvsignore files (Jim Meyering) - remove all trailing blank lines (Jim Meyering) - Remove some unused variables and cut long lines (Daniel Veillard) - Remove trailing blank lines (Daniel Veillard) - Rename a bunch of internal methods to clarify their meaning (Daniel P. Berrange) - Rename 'direction' to 'type' in remote_message_header (Daniel P. Berrange) - Report the object name on lookup error (Daniel Veillard) - Simplify remote driver error reporting (Daniel P. Berrange) - skip some of gnulib's new rules (Jim Meyering) - use automake-1.11's silent-rules option, when possible (Jim Meyering) - use gnumakefile and maintainer-makefile modules from gnulib (Jim Meyering) It's also good to see a lot of new contributors in this release, so thanks to everybody ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

1 0

[libvirt] [PATCH] address incompatible change of qemu-img
by Ryota Ozaki 05 Aug '09

05 Aug '09

Hi, Latest qemu-img (maybe from 0.11.0-rc0 or later?) changes default unit of size as bytes, it was kilo-bytes so far. (see http://www.mail-archive.com/kvm@vger.kernel.org/msg19183.html) This patch addresses this incompatibility by adding postfix 'K'. This fix is able to keep backward compatibility for old qemu as well, and also even if qemu-img falls back old style or any, this explicit specifying should work well. Thanks, ozaki-r >From 7d3dd9e592deab5a1d793532cf230d8367372280 Mon Sep 17 00:00:00 2001 From: Ryota Ozaki <ozaki.ryota(a)gmail.com> Date: Tue, 4 Aug 2009 11:11:31 +0900 Subject: [PATCH] address incompatible change of qemu-img --- src/storage_backend.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/storage_backend.c b/src/storage_backend.c index 07a2e48..8824218 100644 --- a/src/storage_backend.c +++ b/src/storage_backend.c @@ -446,7 +446,7 @@ virStorageBackendCreateQemuImg(virConnectPtr conn, /* Size in KB */ - snprintf(size, sizeof(size), "%llu", vol->capacity/1024); + snprintf(size, sizeof(size), "%lluK", vol->capacity/1024); if (virRun(conn, imgargv, NULL) < 0) { VIR_FREE(imgargv[0]); -- 1.6.0.6

2 1

[libvirt] [PATCH] Support the usage of socat instead of netcat
by Jonas Eriksson 05 Aug '09

05 Aug '09

Some netcat implementations does not support unix domain sockets (-U). Distros that ship these implementations often also ships socat, that do support this. This is a reworked patch based on the openSUSE libvirt package. The original patch may be found in this source rpm: http://download.opensuse.org/source/distribution/11.1/repo/oss/suse/src/lib… * configure.in: Check for --with-socat and set USE_SOCAT in config.h * src/remote_internal.c: Honour USE_SOCAT by selecting between netcat/nc and socat at compile time. --- configure.in | 7 +++++++ src/remote_internal.c | 13 +++++++++++++ 2 files changed, 20 insertions(+), 0 deletions(-) diff --git a/configure.in b/configure.in index b905b23..732ef68 100644 --- a/configure.in +++ b/configure.in @@ -153,6 +153,13 @@ if test -n "$MODPROBE"; then [Location or name of the modprobe program]) fi +dnl Check if socat should be used instead of netcat +AC_ARG_WITH([socat], +[ --with-socat use socat instead of netcat (off)]) +if test "$with_socat" = "yes" ; then + AC_DEFINE_UNQUOTED([USE_SOCAT], 1, [whether socat should be used instead of netcat]) +fi + dnl Specific dir for HTML output ? AC_ARG_WITH([html-dir], [AC_HELP_STRING([--with-html-dir=path], [path to base html directory, default $datadir/doc/html])], diff --git a/src/remote_internal.c b/src/remote_internal.c index a58b768..7678317 100644 --- a/src/remote_internal.c +++ b/src/remote_internal.c @@ -725,6 +725,18 @@ doRemoteOpen (virConnectPtr conn, cmd_argv[j++] = strdup ("-e"); cmd_argv[j++] = strdup ("none"); } +#ifdef USE_SOCAT + cmd_argv[j++] = strdup ("socat"); + cmd_argv[j++] = strdup ("-"); + + char *socat_addr = 0; + if ((asprintf (&socat_addr, "GOPEN:%s", + sockname ? sockname : LIBVIRTD_PRIV_UNIX_SOCKET)) < 0) { + error (conn, VIR_ERR_SYSTEM_ERROR, strerror (ENOMEM)); + goto failed; + } + cmd_argv[j++] = socat_addr; +#else cmd_argv[j++] = strdup (priv->hostname); cmd_argv[j++] = strdup (netcat ? netcat : "nc"); cmd_argv[j++] = strdup ("-U"); @@ -732,6 +744,7 @@ doRemoteOpen (virConnectPtr conn, (flags & VIR_CONNECT_RO ? LIBVIRTD_PRIV_UNIX_SOCKET_RO : LIBVIRTD_PRIV_UNIX_SOCKET)); +#endif cmd_argv[j++] = 0; assert (j == nr_args); for (j = 0; j < (nr_args-1); j++) -- 1.6.2

1 1

[libvirt] PATCH: Add name uniqueness checking to LXC driver
by Daniel P. Berrange 05 Aug '09

05 Aug '09

The LXC driver define/create methods didn't yet have name uniqueness checking enabled Daniel commit 4b23a44f34392b5c6d1c5aa483cbe81b010e47c4 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Fri Jul 31 15:25:03 2009 +0100 Add uniqness checking for LXC define/create methods * src/lxc_driver.c: Check for name & UUID uniqueness when defining or creating domains diff --git a/src/lxc_driver.c b/src/lxc_driver.c index d62c2d7..a9c4f79 100644 --- a/src/lxc_driver.c +++ b/src/lxc_driver.c @@ -311,6 +311,35 @@ static virDomainPtr lxcDomainDefine(virConnectPtr conn, const char *xml) VIR_DOMAIN_XML_INACTIVE))) goto cleanup; + /* See if a VM with matching UUID already exists */ + vm = virDomainFindByUUID(&driver->domains, def->uuid); + if (vm) { + /* UUID matches, but if names don't match, refuse it */ + if (STRNEQ(vm->def->name, def->name)) { + char uuidstr[VIR_UUID_STRING_BUFLEN]; + virUUIDFormat(vm->def->uuid, uuidstr); + lxcError(conn, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + vm->def->name, uuidstr); + goto cleanup; + } + + /* UUID & name match */ + virDomainObjUnlock(vm); + newVM = 0; + } else { + /* UUID does not match, but if a name matches, refuse it */ + vm = virDomainFindByName(&driver->domains, def->name); + if (vm) { + char uuidstr[VIR_UUID_STRING_BUFLEN]; + virUUIDFormat(vm->def->uuid, uuidstr); + lxcError(conn, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + def->name, uuidstr); + goto cleanup; + } + } + if ((def->nets != NULL) && !(driver->have_netns)) { lxcError(conn, NULL, VIR_ERR_NO_SUPPORT, "%s", _("System lacks NETNS support")); @@ -1082,6 +1111,39 @@ lxcDomainCreateAndStart(virConnectPtr conn, VIR_DOMAIN_XML_INACTIVE))) goto cleanup; + /* See if a VM with matching UUID already exists */ + vm = virDomainFindByUUID(&driver->domains, def->uuid); + if (vm) { + /* UUID matches, but if names don't match, refuse it */ + if (STRNEQ(vm->def->name, def->name)) { + char uuidstr[VIR_UUID_STRING_BUFLEN]; + virUUIDFormat(vm->def->uuid, uuidstr); + lxcError(conn, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + vm->def->name, uuidstr); + goto cleanup; + } + + /* UUID & name match, but if VM is already active, refuse it */ + if (virDomainIsActive(vm)) { + lxcError(conn, NULL, VIR_ERR_OPERATION_FAILED, + _("domain is already active as '%s'"), vm->def->name); + goto cleanup; + } + virDomainObjUnlock(vm); + } else { + /* UUID does not match, but if a name matches, refuse it */ + vm = virDomainFindByName(&driver->domains, def->name); + if (vm) { + char uuidstr[VIR_UUID_STRING_BUFLEN]; + virUUIDFormat(vm->def->uuid, uuidstr); + lxcError(conn, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + def->name, uuidstr); + goto cleanup; + } + } + if ((def->nets != NULL) && !(driver->have_netns)) { lxcError(conn, NULL, VIR_ERR_NO_SUPPORT, "%s", _("System lacks NETNS support")); -- |: 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 cleanup of transient VMs
by Daniel P. Berrange 05 Aug '09

05 Aug '09

If a transient VM quit unexpectedly it would never be cleaned up, resulting in a guest that's impossible to remove Daniel commit 7781032c3d870f5c9dacfc02baecc151689d9c57 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Fri Jul 31 14:38:46 2009 +0100 Fix removal of transient VMs when LXC aborts * src/lxc_driver.c: Remove transient VM after monitor triggered shutdown diff --git a/src/lxc_driver.c b/src/lxc_driver.c index f37fc5d..d62c2d7 100644 --- a/src/lxc_driver.c +++ b/src/lxc_driver.c @@ -774,6 +774,10 @@ static void lxcMonitorEvent(int watch, VIR_DOMAIN_EVENT_STOPPED, VIR_DOMAIN_EVENT_STOPPED_SHUTDOWN); } + if (!vm->persistent) { + virDomainRemoveInactive(&driver->domains, vm); + vm = NULL; + } cleanup: if (vm) -- |: 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 startup of LXC vms without cgroups
by Daniel P. Berrange 05 Aug '09

05 Aug '09

If cgroups was not mounted at all, LXC would fail to start any VM, since we missed an error code check Daniel commit 8713c7e8111e73879f8380b920995ff3d1f3f9e9 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Fri Jul 31 14:37:25 2009 +0100 Don't try to activate cgroups if not present for LXC * src/lxc_controller.c: Don't throw error in LXC startup if the cgroups driver mount isn't available. Improve error logging for resource setup diff --git a/src/lxc_controller.c b/src/lxc_controller.c index 9ad48a7..8d11238 100644 --- a/src/lxc_controller.c +++ b/src/lxc_controller.c @@ -84,25 +84,38 @@ static int lxcSetContainerResources(virDomainDefPtr def) rc = virCgroupForDriver("lxc", &driver, 1, 0); if (rc != 0) { - lxcError(NULL, NULL, VIR_ERR_INTERNAL_ERROR, "%s", - _("Unable to get cgroup for driver")); + /* Skip all if no driver cgroup is configured */ + if (rc == -ENXIO || rc == -ENOENT) + return 0; + + virReportSystemError(NULL, -rc, "%s", + _("Unable to get cgroup for driver")); return rc; } rc = virCgroupForDomain(driver, def->name, &cgroup, 1); if (rc != 0) { - lxcError(NULL, NULL, VIR_ERR_INTERNAL_ERROR, - _("Unable to create cgroup for domain %s"), def->name); + virReportSystemError(NULL, -rc, + _("Unable to create cgroup for domain %s"), + def->name); goto cleanup; } rc = virCgroupSetMemory(cgroup, def->maxmem); - if (rc != 0) - goto out; + if (rc != 0) { + virReportSystemError(NULL, -rc, + _("Unable to set memory limit for domain %s"), + def->name); + goto cleanup; + } rc = virCgroupDenyAllDevices(cgroup); - if (rc != 0) - goto out; + if (rc != 0) { + virReportSystemError(NULL, -rc, + _("Unable to deny devices for domain %s"), + def->name); + goto cleanup; + } for (i = 0; devices[i].type != 0; i++) { struct cgroup_device_policy *dev = &devices[i]; @@ -110,19 +123,27 @@ static int lxcSetContainerResources(virDomainDefPtr def) dev->type, dev->major, dev->minor); - if (rc != 0) - goto out; + if (rc != 0) { + virReportSystemError(NULL, -rc, + _("Unable to allow device %c:%d:%d for domain %s"), + dev->type, dev->major, dev->minor, def->name); + goto cleanup; + } } rc = virCgroupAllowDeviceMajor(cgroup, 'c', LXC_DEV_MAJ_PTY); - if (rc != 0) - goto out; + if (rc != 0) { + virReportSystemError(NULL, -rc, + _("Unable to allow PYT devices for domain %s"), + def->name); + goto cleanup; + } rc = virCgroupAddTask(cgroup, getpid()); -out: if (rc != 0) { - virReportSystemError(NULL, -rc, "%s", - _("Failed to set lxc resources")); + virReportSystemError(NULL, -rc, + _("Unable to add task %d to cgroup for domain %s"), + getpid(), def->name); } cleanup: -- |: 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] Refresh /etc/xen if inotify wasn't initialized
by Cole Robinson 05 Aug '09

05 Aug '09

If libvirt was built against inotify, but an API user isn't using domain events (virsh, virt-manager), the xen xm driver doesn't poll /etc/xen for new config files. This means that domains created or deleted on other libvirt connections aren't reflected in the original connection (but they will show up for all newly opened connections). This causes newly created VMs to disappear from virt-manager < 0.8.0 once they are shutdown, since VMs were installed on a separate connection. The attached patch refreshes /etc/xen in all cases, unless inotify was successfully initialized (user is using domain events). Thanks, Cole

3 3

[libvirt] [PATCH] Allow libvirtd to RPC to external libvirtd.
by Chris Lalancette 05 Aug '09

05 Aug '09

Allow the daemon itself to make RPCs to an external libvirtd, but only if the URI is fully specified. While this isn't used at the moment, it will be for the tunnelled migration support in the future. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/remote_internal.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/remote_internal.c b/src/remote_internal.c index ad0cbf0..ff4fb6c 100644 --- a/src/remote_internal.c +++ b/src/remote_internal.c @@ -997,7 +997,7 @@ remoteOpen (virConnectPtr conn, int ret, rflags = 0; const char *autostart = getenv("LIBVIRT_AUTOSTART"); - if (inside_daemon) + if (inside_daemon && (!conn->uri || (conn->uri && !conn->uri->server))) return VIR_DRV_OPEN_DECLINED; if (!(priv = remoteAllocPrivateData(conn))) -- 1.6.0.6

3 3

[libvirt] [PATCH] rng comment fixes
by Aron Griffis 05 Aug '09

05 Aug '09

--- docs/schemas/capability.rng | 3 +-- docs/schemas/nodedev.rng | 4 ---- docs/schemas/storagepool.rng | 3 +-- docs/schemas/storagevol.rng | 3 +-- 4 files changed, 3 insertions(+), 10 deletions(-) diff --git a/docs/schemas/capability.rng b/docs/schemas/capability.rng index 775bbdb..1e3c5f3 100644 --- a/docs/schemas/capability.rng +++ b/docs/schemas/capability.rng @@ -1,7 +1,6 @@ - + <grammar xmlns="http://relaxng.org/ns/structure/1.0" datatypeLibrary="http://www.w3.org/2001/XMLSchema-datatypes"> -  <start> <ref name='capabilities'/> </start> diff --git a/docs/schemas/nodedev.rng b/docs/schemas/nodedev.rng index 78bd6a8..7060274 100644 --- a/docs/schemas/nodedev.rng +++ b/docs/schemas/nodedev.rng @@ -1,14 +1,10 @@  <grammar xmlns="http://relaxng.org/ns/structure/1.0" datatypeLibrary="http://www.w3.org/2001/XMLSchema-datatypes"> -  <start> <ref name='device'/> </start> -  <define name='device'> <element name="device">  + <grammar xmlns="http://relaxng.org/ns/structure/1.0" datatypeLibrary="http://www.w3.org/2001/XMLSchema-datatypes"> -  <start> <ref name='pool'/> </start> diff --git a/docs/schemas/storagevol.rng b/docs/schemas/storagevol.rng index 7dc7876..5b0b038 100644 --- a/docs/schemas/storagevol.rng +++ b/docs/schemas/storagevol.rng @@ -1,7 +1,6 @@ - + <grammar xmlns="http://relaxng.org/ns/structure/1.0" datatypeLibrary="http://www.w3.org/2001/XMLSchema-datatypes"> -  <start> <ref name='vol'/> </start>

2 1

[libvirt] openvzGetType() leaks memory
by Matthias Bolte 05 Aug '09

05 Aug '09

All other drivers return a static string from their getType() function (conforming with the API documentation), but the OpenVZ driver does strdup() a string, generating a leak: static const char *openvzGetType(virConnectPtr conn ATTRIBUTE_UNUSED) { return strdup("OpenVZ"); } Matthias

2 1

[libvirt] [PATCH] Don't clobber ID when redefining dom in xm driver
by Cole Robinson 05 Aug '09

05 Aug '09

If redefining a domain in the xen xm driver, we unconditionally set the domain ID to -1. This can make a running domain appear inactive, and really confuse things. Just drop the offending code, since virGetDomain already sets the ID to -1 if required. Fix pointed out by Dan Berrange, more info can be found at https://bugzilla.redhat.com/show_bug.cgi?id=508278 Thanks, Cole

3 2

[libvirt] [PATCH] Fix latent buffer overflow in qemudOpenMonitorUnix.
by Chris Lalancette 05 Aug '09

05 Aug '09

Fix a possible latent bug in qemudOpenMonitorUnix(). If the pathname to the monitor is very long (i.e. >= UNIX_MAX_PATH), then strncpy will *not* place a final \0 on the string (see "man strncpy"). NULL terminate the buffer to ensure we don't run off the end. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 9fcc07a..4f173b7 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -910,6 +910,7 @@ qemudOpenMonitorUnix(virConnectPtr conn, memset(&addr, 0, sizeof(addr)); addr.sun_family = AF_UNIX; strncpy(addr.sun_path, monitor, sizeof(addr.sun_path)); + NUL_TERMINATE(addr.sun_path); do { ret = connect(monfd, (struct sockaddr *) &addr, sizeof(addr)); -- 1.6.0.6

4 5

[libvirt] [PATCH] Remove a stray semicolon in qemudDomainMigratePrepare2.
by Chris Lalancette 05 Aug '09

05 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 83cbcf3..5f4b28b 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -6274,7 +6274,7 @@ qemudDomainMigratePrepare2 (virConnectPtr dconn, char migrateFrom [64]; const char *p; virDomainEventPtr event = NULL; - int ret = -1;; + int ret = -1; int internalret; *uri_out = NULL; -- 1.6.0.6

3 3

[libvirt] [PATCH] Workaround for broken GCC in Debian Etch
by Maximilian Wilhelm 05 Aug '09

05 Aug '09

Hi again! I missed one patch for working around a GCC bug in Debian Etch regarding limit definitions. Sorry for the unstripped git-format-patch outputs, I just realised this after pressing send :( (This seems to be not my day :)) Ciao Max -- Gib Dein Bestes. Dann übertriff Dich selbst!

3 4

[libvirt] [PATCH] Add some automatic dependency checks for configure to enable/disable {ESX, UML, LXC, Xen-Inotify}
by Maximilian Wilhelm 05 Aug '09

05 Aug '09

Hi! While building libVirt on an older Debian Etch machine, I stumbled across some automatic enabled drivers which stopped configure and make very badly due to missing header files, old libraries and stuff. So I decided to improve the automagic tests so others won't face the same problem again :) Find four patches attached. Ciao Max -- Fortschritt bedeutet, einen Schritt so zu machen, daß man den nächsten auch noch machen kann.

3 2

Re: [libvirt] VMware support and Server 2.0.1
by Matthias Bolte 05 Aug '09

05 Aug '09

2009/7/30 Timo Makinen <tmakinen(a)ee.oulu.fi>: > On Thu, Jul 30, 2009 at 03:22:59AM +0200, Matthias Bolte wrote: >> 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>: >> >> 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 > > Works like a charm now. :) > > - Timo > Fine! The attached second GSX patch adds a gsx:// scheme to the ESX driver, so you can connect to a GSX host using virsh -c gsx://host and the driver will select the port automatically dependent on the scheme and transport. Here gsx scheme and default transport https result in port 8333. Matthias

3 2

[libvirt] [PATCH] spelling fix
by Aron Griffis 05 Aug '09

05 Aug '09

--- src/qemu_conf.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qemu_conf.c b/src/qemu_conf.c index 4fe4e39..6b0b404 100644 --- a/src/qemu_conf.c +++ b/src/qemu_conf.c @@ -1970,7 +1970,7 @@ int qemudBuildCommandLine(virConnectPtr conn, if (def->nvideos) { if (def->nvideos > 1) { qemudReportError(conn, NULL, NULL, VIR_ERR_INTERNAL_ERROR, - "%s", _("only one video card is currentely supported")); + "%s", _("only one video card is currently supported")); goto error; }

1 0

[libvirt] PATCH: Fix escaping for 8-bit high characters
by Daniel P. Berrange 05 Aug '09

05 Aug '09

The virBufferEscapeString method has a broken test *cur > 0x20 which causes all 8-bit high characters to be lost as 'cur' is a signed char. This obviously dooms anyone using UTF-8 characters outside the boring old ASCII-7 range, like this guy https://bugzilla.redhat.com/show_bug.cgi?id=479517 Daniel diff --git a/src/buf.c b/src/buf.c index 259175d..c802aa2 100644 --- a/src/buf.c +++ b/src/buf.c @@ -304,7 +304,7 @@ virBufferEscapeString(const virBufferPtr buf, const char *format, const char *st *out++ = 'o'; *out++ = 's'; *out++ = ';'; - } else if ((*cur >= 0x20) || (*cur == '\n') || (*cur == '\t') || + } else if (((unsigned char)*cur >= 0x20) || (*cur == '\n') || (*cur == '\t') || (*cur == '\r')) { /* * default case, just copy ! -- |: 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 ATTRIBUTE_UNUSED from flags to qemudDomainMigratePerform.
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index eadb53f..9fcc07a 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -6424,7 +6424,7 @@ qemudDomainMigratePerform (virDomainPtr dom, const char *cookie ATTRIBUTE_UNUSED, int cookielen ATTRIBUTE_UNUSED, const char *uri, - unsigned long flags ATTRIBUTE_UNUSED, + unsigned long flags, const char *dname ATTRIBUTE_UNUSED, unsigned long resource) { -- 1.6.0.6

2 2

[libvirt] [PATCH] Add a comment about setting errors after qemudStartVMDaemon().
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index fcfbfbc..eadb53f 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -6390,6 +6390,9 @@ qemudDomainMigratePrepare2 (virConnectPtr dconn, */ snprintf (migrateFrom, sizeof (migrateFrom), "tcp:0.0.0.0:%d", this_port); if (qemudStartVMDaemon (dconn, driver, vm, migrateFrom, -1) < 0) { + /* Note that we don't set an error here because qemudStartVMDaemon + * should have already done that. + */ if (!vm->persistent) { virDomainRemoveInactive(&driver->domains, vm); vm = NULL; -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix an erroneous debug error to KVM; it should read QEMU/KVM.
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 5f4b28b..fcfbfbc 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -6321,7 +6321,7 @@ qemudDomainMigratePrepare2 (virConnectPtr dconn, */ if (!STRPREFIX (uri_in, "tcp:")) { qemudReportError (dconn, NULL, NULL, VIR_ERR_INVALID_ARG, - "%s", _("only tcp URIs are supported for KVM migrations")); + "%s", _("only tcp URIs are supported for KVM/QEMU migrations")); goto cleanup; } -- 1.6.0.6

2 2

[libvirt] [PATCH] Convert a few stray users of free() in libvirt.c to VIR_FREE().
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/libvirt.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/libvirt.c b/src/libvirt.c index 167359f..9fd864d 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -3020,7 +3020,7 @@ virDomainMigrate (virDomainPtr domain, ret = dconn->driver->domainMigratePrepare2 (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, bandwidth, dom_xml); - free (dom_xml); + VIR_FREE (dom_xml); if (ret == -1) goto done; if (uri == NULL && uri_out == NULL) { virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, @@ -3062,8 +3062,8 @@ virDomainMigrate (virDomainPtr domain, } done: - free (uri_out); - free (cookie); + VIR_FREE (uri_out); + VIR_FREE (cookie); return ddomain; error: -- 1.6.0.6

2 2

[libvirt] [PATCH] Use virGetHostname instead of gethostname.
by Chris Lalancette 02 Aug '09

02 Aug '09

Fix up qemudDomainMigratePrepare2 to use virGetHostname instead of gethostname. Besides the fact that virGetHostname is far more clever, there was a latent bug in the handling that could cause a buffer overflow on a very long hostname. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/qemu_driver.c | 9 ++++++--- 1 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/qemu_driver.c b/src/qemu_driver.c index cee2164..83cbcf3 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -6270,11 +6270,12 @@ qemudDomainMigratePrepare2 (virConnectPtr dconn, virDomainDefPtr def = NULL; virDomainObjPtr vm = NULL; int this_port; - char hostname [HOST_NAME_MAX+1]; + char *hostname; char migrateFrom [64]; const char *p; virDomainEventPtr event = NULL; int ret = -1;; + int internalret; *uri_out = NULL; @@ -6300,14 +6301,16 @@ qemudDomainMigratePrepare2 (virConnectPtr dconn, if (port == QEMUD_MIGRATION_NUM_PORTS) port = 0; /* Get hostname */ - if (gethostname (hostname, HOST_NAME_MAX+1) == -1) { + if ((hostname = virGetHostname()) == NULL) { virReportSystemError (dconn, errno, "%s", _("failed to determine host name")); goto cleanup; } /* Caller frees */ - if (virAsprintf(uri_out, "tcp:%s:%d", hostname, this_port) < 0) { + internalret = virAsprintf(uri_out, "tcp:%s:%d", hostname, this_port); + VIR_FREE(hostname); + if (internalret < 0) { virReportOOMError (dconn); goto cleanup; } -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix up a minor indentation issue with virDomainMigratePrepare.
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/libvirt.c | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/libvirt.c b/src/libvirt.c index 7ecf1ec..167359f 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -3078,13 +3078,13 @@ error: */ int virDomainMigratePrepare (virConnectPtr dconn, - char **cookie, - int *cookielen, - const char *uri_in, - char **uri_out, - unsigned long flags, - const char *dname, - unsigned long bandwidth) + char **cookie, + int *cookielen, + const char *uri_in, + char **uri_out, + unsigned long flags, + const char *dname, + unsigned long bandwidth) { VIR_DEBUG("dconn=%p, cookie=%p, cookielen=%p, uri_in=%s, uri_out=%p, " "flags=%lu, dname=%s, bandwidth=%lu", dconn, cookie, cookielen, -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix virCapabilitiesDefaultGuestMachine documentation.
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/capabilities.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/capabilities.c b/src/capabilities.c index d186961..c6766b6 100644 --- a/src/capabilities.c +++ b/src/capabilities.c @@ -567,7 +567,7 @@ virCapabilitiesDefaultGuestMachine(virCapsPtr caps, } /** - * virCapabilitiesDefaultGuestMachine: + * virCapabilitiesDefaultGuestEmulator: * @caps: capabilities to query * @ostype: OS type to search for ('xen', 'hvm') * @arch: architecture to search for -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix up a silly typo in apibuild.py.
by Chris Lalancette 02 Aug '09

02 Aug '09

Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- docs/apibuild.py | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/docs/apibuild.py b/docs/apibuild.py index 6fec049..84bc1ac 100755 --- a/docs/apibuild.py +++ b/docs/apibuild.py @@ -39,7 +39,7 @@ ignored_functions = { "virDomainMigratePrepare": "private function for migration", "virDomainMigratePrepare2": "private function for migration", "virDrvSupportsFeature": "private function for remote access", - "DllMain": "specific function fo Win32", + "DllMain": "specific function for Win32", } def escape(raw): -- 1.6.0.6

2 2

[libvirt] PATCH: Fix crash attempting to shutdown inactive QEMU vm
by Daniel P. Berrange 01 Aug '09

01 Aug '09

If the virDomainShutdown() op was run on an active QEMu vm, it would crash, since the def->monitor_chr was NULL. Daniel commit 49ec121fd806aaa07bbcba9668a8f9dd43dda9c4 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Fri Jul 31 15:50:46 2009 +0100 Fix crash when attempting to shutdown inactive QEMU vm * src/qemu_driver.c: Add check that QEMU is active before attemting to shutdown. Fix error code for check in destroy method diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 26897d3..cebbafb 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -2940,6 +2940,12 @@ static int qemudDomainShutdown(virDomainPtr dom) { goto cleanup; } + if (!virDomainIsActive(vm)) { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_INVALID, + "%s", _("domain is not running")); + goto cleanup; + } + if (qemudMonitorCommand(vm, "system_powerdown", &info) < 0) { qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, "%s", _("shutdown operation failed")); @@ -2971,7 +2977,7 @@ static int qemudDomainDestroy(virDomainPtr dom) { goto cleanup; } if (!virDomainIsActive(vm)) { - qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_INVALID, "%s", _("domain is not running")); goto cleanup; } -- |: 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] Split virDomainMigrate into functions.
by Chris Lalancette 01 Aug '09

01 Aug '09

Re-factor virDomainMigrate to split out the version 1 and version 2 protocols into their own functions. In reality, the two versions share very little in common, so forcing them together in the same function was just confusing. This will also make adding tunnelled migration easier. Signed-off-by: Chris Lalancette <clalance(a)redhat.com> --- src/libvirt.c | 258 ++++++++++++++++++++++++++++++++++----------------------- 1 files changed, 155 insertions(+), 103 deletions(-) diff --git a/src/libvirt.c b/src/libvirt.c index 9fd864d..dede2b9 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -2874,6 +2874,146 @@ error: } +static virDomainPtr +migrate_version_1 (virDomainPtr domain, + virConnectPtr dconn, + unsigned long flags, + const char *dname, + const char *uri, + unsigned long bandwidth) +{ + virDomainPtr ddomain = NULL; + char *uri_out = NULL; + char *cookie = NULL; + int cookielen = 0; + + /* Prepare the migration. + * + * The destination host may return a cookie, or leave cookie as + * NULL. + * + * The destination host MUST set uri_out if uri_in is NULL. + * + * If uri_in is non-NULL, then the destination host may modify + * the URI by setting uri_out. If it does not wish to modify + * the URI, it should leave uri_out as NULL. + */ + if (dconn->driver->domainMigratePrepare + (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, + bandwidth) == -1) + goto done; + + if (uri == NULL && uri_out == NULL) { + virLibConnError (domain->conn, VIR_ERR_INTERNAL_ERROR, + _("domainMigratePrepare did not set uri")); + goto done; + } + if (uri_out) + uri = uri_out; /* Did domainMigratePrepare change URI? */ + assert (uri != NULL); + + /* Perform the migration. The driver isn't supposed to return + * until the migration is complete. + */ + if (domain->conn->driver->domainMigratePerform + (domain, cookie, cookielen, uri, flags, dname, bandwidth) == -1) + goto done; + + /* Get the destination domain and return it or error. + * 'domain' no longer actually exists at this point + * (or so we hope), but we still use the object in memory + * in order to get the name. + */ + dname = dname ? dname : domain->name; + if (dconn->driver->domainMigrateFinish) + ddomain = dconn->driver->domainMigrateFinish + (dconn, dname, cookie, cookielen, uri, flags); + else + ddomain = virDomainLookupByName (dconn, dname); + + done: + VIR_FREE (uri_out); + VIR_FREE (cookie); + return ddomain; +} + +static virDomainPtr +migrate_version_2 (virDomainPtr domain, + virConnectPtr dconn, + unsigned long flags, + const char *dname, + const char *uri, + unsigned long bandwidth) +{ + virDomainPtr ddomain = NULL; + char *uri_out = NULL; + char *cookie = NULL; + char *dom_xml = NULL; + int cookielen = 0, ret; + + /* Prepare the migration. + * + * The destination host may return a cookie, or leave cookie as + * NULL. + * + * The destination host MUST set uri_out if uri_in is NULL. + * + * If uri_in is non-NULL, then the destination host may modify + * the URI by setting uri_out. If it does not wish to modify + * the URI, it should leave uri_out as NULL. + */ + + /* In version 2 of the protocol, the prepare step is slightly + * different. We fetch the domain XML of the source domain + * and pass it to Prepare2. + */ + if (!domain->conn->driver->domainDumpXML) { + virLibConnError (domain->conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__); + return NULL; + } + dom_xml = domain->conn->driver->domainDumpXML (domain, + VIR_DOMAIN_XML_SECURE); + if (!dom_xml) + return NULL; + + ret = dconn->driver->domainMigratePrepare2 + (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, + bandwidth, dom_xml); + VIR_FREE (dom_xml); + if (ret == -1) + goto done; + + if (uri == NULL && uri_out == NULL) { + virLibConnError (domain->conn, VIR_ERR_INTERNAL_ERROR, + _("domainMigratePrepare2 did not set uri")); + goto done; + } + if (uri_out) + uri = uri_out; /* Did domainMigratePrepare2 change URI? */ + assert (uri != NULL); + + /* Perform the migration. The driver isn't supposed to return + * until the migration is complete. + */ + ret = domain->conn->driver->domainMigratePerform + (domain, cookie, cookielen, uri, flags, dname, bandwidth); + if (ret == -1) + goto done; + + /* In version 2 of the migration protocol, we pass the + * status code from the sender to the destination host, + * so it can do any cleanup if the migration failed. + */ + dname = dname ? dname : domain->name; + ddomain = dconn->driver->domainMigrateFinish2 + (dconn, dname, cookie, cookielen, uri, flags, ret); + + done: + VIR_FREE (uri_out); + VIR_FREE (cookie); + return ddomain; +} + /** * virDomainMigrate: * @domain: a domain object @@ -2930,140 +3070,52 @@ virDomainMigrate (virDomainPtr domain, const char *uri, unsigned long bandwidth) { - virConnectPtr conn; virDomainPtr ddomain = NULL; - char *uri_out = NULL; - char *cookie = NULL; - char *dom_xml = NULL; - int cookielen = 0, ret, version = 0; DEBUG("domain=%p, dconn=%p, flags=%lu, dname=%s, uri=%s, bandwidth=%lu", domain, dconn, flags, NULLSTR(dname), NULLSTR(uri), bandwidth); virResetLastError(); + /* First checkout the source */ if (!VIR_IS_CONNECTED_DOMAIN (domain)) { virLibDomainError(NULL, VIR_ERR_INVALID_DOMAIN, __FUNCTION__); return NULL; } - conn = domain->conn; /* Source connection. */ - if (!VIR_IS_CONNECT (dconn)) { - virLibConnError (conn, VIR_ERR_INVALID_CONN, __FUNCTION__); + if (domain->conn->flags & VIR_CONNECT_RO) { + virLibDomainError(domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); goto error; } - if (domain->conn->flags & VIR_CONNECT_RO) { - virLibDomainError(domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); + /* Now checkout the destination */ + if (!VIR_IS_CONNECT (dconn)) { + virLibConnError (domain->conn, VIR_ERR_INVALID_CONN, __FUNCTION__); goto error; } if (dconn->flags & VIR_CONNECT_RO) { - /* NB, delibrately report error against source object, not dest here */ - virLibDomainError(domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); + /* NB, deliberately report error against source object, not dest */ + virLibDomainError (domain, VIR_ERR_OPERATION_DENIED, __FUNCTION__); goto error; } /* Check that migration is supported by both drivers. */ - if (VIR_DRV_SUPPORTS_FEATURE (conn->driver, conn, + if (VIR_DRV_SUPPORTS_FEATURE (domain->conn->driver, domain->conn, VIR_DRV_FEATURE_MIGRATION_V1) && VIR_DRV_SUPPORTS_FEATURE (dconn->driver, dconn, VIR_DRV_FEATURE_MIGRATION_V1)) - version = 1; - else if (VIR_DRV_SUPPORTS_FEATURE (conn->driver, conn, + ddomain = migrate_version_1 (domain, dconn, flags, dname, uri, bandwidth); + else if (VIR_DRV_SUPPORTS_FEATURE (domain->conn->driver, domain->conn, VIR_DRV_FEATURE_MIGRATION_V2) && VIR_DRV_SUPPORTS_FEATURE (dconn->driver, dconn, VIR_DRV_FEATURE_MIGRATION_V2)) - version = 2; + ddomain = migrate_version_2 (domain, dconn, flags, dname, uri, bandwidth); else { - virLibConnError (conn, VIR_ERR_NO_SUPPORT, __FUNCTION__); + virLibConnError (domain->conn, VIR_ERR_NO_SUPPORT, __FUNCTION__); goto error; } - /* Prepare the migration. - * - * The destination host may return a cookie, or leave cookie as - * NULL. - * - * The destination host MUST set uri_out if uri_in is NULL. - * - * If uri_in is non-NULL, then the destination host may modify - * the URI by setting uri_out. If it does not wish to modify - * the URI, it should leave uri_out as NULL. - */ - if (version == 1) { - ret = dconn->driver->domainMigratePrepare - (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, - bandwidth); - if (ret == -1) goto done; - if (uri == NULL && uri_out == NULL) { - virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, - _("domainMigratePrepare did not set uri")); - goto done; - } - if (uri_out) uri = uri_out; /* Did domainMigratePrepare change URI? */ - - assert (uri != NULL); - } - else /* if (version == 2) */ { - /* In version 2 of the protocol, the prepare step is slightly - * different. We fetch the domain XML of the source domain - * and pass it to Prepare2. - */ - if (!conn->driver->domainDumpXML) { - virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, __FUNCTION__); - goto error; - } - dom_xml = conn->driver->domainDumpXML (domain, - VIR_DOMAIN_XML_SECURE); - - if (!dom_xml) - goto error; + if (ddomain == NULL) + goto error; - ret = dconn->driver->domainMigratePrepare2 - (dconn, &cookie, &cookielen, uri, &uri_out, flags, dname, - bandwidth, dom_xml); - VIR_FREE (dom_xml); - if (ret == -1) goto done; - if (uri == NULL && uri_out == NULL) { - virLibConnError (conn, VIR_ERR_INTERNAL_ERROR, - _("domainMigratePrepare2 did not set uri")); - goto done; - } - if (uri_out) uri = uri_out; /* Did domainMigratePrepare2 change URI? */ - - assert (uri != NULL); - } - - /* Perform the migration. The driver isn't supposed to return - * until the migration is complete. - */ - ret = conn->driver->domainMigratePerform - (domain, cookie, cookielen, uri, flags, dname, bandwidth); - - if (version == 1) { - if (ret == -1) goto done; - /* Get the destination domain and return it or error. - * 'domain' no longer actually exists at this point - * (or so we hope), but we still use the object in memory - * in order to get the name. - */ - dname = dname ? dname : domain->name; - if (dconn->driver->domainMigrateFinish) - ddomain = dconn->driver->domainMigrateFinish - (dconn, dname, cookie, cookielen, uri, flags); - else - ddomain = virDomainLookupByName (dconn, dname); - } else /* if (version == 2) */ { - /* In version 2 of the migration protocol, we pass the - * status code from the sender to the destination host, - * so it can do any cleanup if the migration failed. - */ - dname = dname ? dname : domain->name; - ddomain = dconn->driver->domainMigrateFinish2 - (dconn, dname, cookie, cookielen, uri, flags, ret); - } - - done: - VIR_FREE (uri_out); - VIR_FREE (cookie); return ddomain; error: -- 1.6.0.6

2 2

[libvirt] [PATCH] Fix PCIe FLR detection
by Mark McLoughlin 01 Aug '09

01 Aug '09

PCIe DevCap register is actually 32 bits, not 16 bits. Since FLR is bit 28, we clearly are failing to detect FLR support. Known to fix device reset with some SR-IOV devices. * src/pci.c: fix pciDetectFunctionLevelReset() --- src/pci.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/pci.c b/src/pci.c index 4030a14..2dc2e1c 100644 --- a/src/pci.c +++ b/src/pci.c @@ -321,7 +321,7 @@ pciFindCapabilityOffset(pciDevice *dev, unsigned capability) static unsigned pciDetectFunctionLevelReset(pciDevice *dev) { - uint16_t caps; + uint32_t caps; uint8_t pos; /* The PCIe Function Level Reset capability allows @@ -331,7 +331,7 @@ pciDetectFunctionLevelReset(pciDevice *dev) * on SR-IOV NICs at the moment. */ if (dev->pcie_cap_pos) { - caps = pciRead16(dev, dev->pcie_cap_pos + PCI_EXP_DEVCAP); + caps = pciRead32(dev, dev->pcie_cap_pos + PCI_EXP_DEVCAP); if (caps & PCI_EXP_DEVCAP_FLR) { VIR_DEBUG("%s %s: detected PCIe FLR capability", dev->id, dev->name); return 1; -- 1.6.2.5

2 1

[libvirt] PATCH: Re-enable ESX driver on Mingw32
by Daniel P. Berrange 31 Jul '09

31 Jul '09

After finding an RPM of mingw32-curl for Fedora 11, I was able to successfully build the ESX driver on Ming32 with only a couple of tweaks. Daniel commit 8b135962ef026ef3ef1b5c5163b1213eeba9ec36 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Thu Jul 30 16:40:29 2009 +0100 Enable ESX driver build on Mingw32 * autobuild.sh, mingw32-libvirt.spec.in: Enable esx on mingw32 * src/esx/esx_driver.c: Define AI_ADDRCONFIG if not set * src/esx/esx_util.c, src/esx/esx_vi_types.c: Always use %lld & friends, since gnulib guarentees we have these and not the target's own variants diff --git a/autobuild.sh b/autobuild.sh index 239372c..d3934ea 100755 --- a/autobuild.sh +++ b/autobuild.sh @@ -78,7 +78,6 @@ if [ -x /usr/bin/i686-pc-mingw32-gcc ]; then --without-openvz \ --without-one \ --without-phyp \ - --without-esx \ --without-netcf \ --without-libvirtd diff --git a/mingw32-libvirt.spec.in b/mingw32-libvirt.spec.in index 8fd08a6..40a8124 100644 --- a/mingw32-libvirt.spec.in +++ b/mingw32-libvirt.spec.in @@ -54,7 +54,6 @@ MinGW Windows libvirt virtualization library. --without-openvz \ --without-one \ --without-phyp \ - --without-esx \ --without-netcf \ --without-libvirtd make diff --git a/src/esx/esx_driver.c b/src/esx/esx_driver.c index e3d60a5..08d82f9 100644 --- a/src/esx/esx_driver.c +++ b/src/esx/esx_driver.c @@ -1769,7 +1769,7 @@ esxDomainGetInfo(virDomainPtr domain, virDomainInfoPtr info) for (value = perfMetricIntSeries->value; value != NULL; value = value->_next) { - VIR_DEBUG("value %"PRIi64, value->value); + VIR_DEBUG("value %lld", (long long int)value->value); } } } @@ -2399,8 +2399,8 @@ esxDomainGetSchedulerParameters(virDomainPtr domain, default: ESX_ERROR(domain->conn, VIR_ERR_INTERNAL_ERROR, - "Shares level has unknown value %"PRIi32, - sharesInfo->level); + "Shares level has unknown value %d", + (int)sharesInfo->level); goto failure; } diff --git a/src/esx/esx_util.c b/src/esx/esx_util.c index 5c87ea3..a113fde 100644 --- a/src/esx/esx_util.c +++ b/src/esx/esx_util.c @@ -41,6 +41,10 @@ virReportErrorHelper (conn, VIR_FROM_ESX, code, __FILE__, __FUNCTION__, \ __LINE__, fmt) +/* AI_ADDRCONFIG is missing on some systems. */ +#ifndef AI_ADDRCONFIG +# define AI_ADDRCONFIG 0 +#endif char * @@ -435,12 +439,12 @@ esxUtil_GetConfigLong(virConnectPtr conn, virConfPtr conf, const char *name, int esxUtil_GetConfigBoolean(virConnectPtr conn, virConfPtr conf, - const char *name, int *boolean, int default_, + const char *name, int *boolval, int default_, int optional) { virConfValuePtr value; - *boolean = default_; + *boolval = default_; value = virConfGetValue(conf, name); if (value == NULL) { @@ -465,9 +469,9 @@ esxUtil_GetConfigBoolean(virConnectPtr conn, virConfPtr conf, } if (STRCASEEQ(value->str, "true")) { - *boolean = 1; + *boolval = 1; } else if (STRCASEEQ(value->str, "false")) { - *boolean = 0; + *boolval = 0; } else { ESX_ERROR(conn, VIR_ERR_INTERNAL_ERROR, "Config entry '%s' must represent a boolean value " diff --git a/src/esx/esx_vi_types.c b/src/esx/esx_vi_types.c index 684b8c7..ab17460 100644 --- a/src/esx/esx_vi_types.c +++ b/src/esx/esx_vi_types.c @@ -962,7 +962,7 @@ esxVI_Int_DeepCopy(virConnectPtr conn, esxVI_Int **dest, esxVI_Int *src) /* esxVI_Int_Serialize */ ESX_VI__TEMPLATE__SERIALIZE_EXTRA(Int, "xsd:int", { - virBufferVSprintf(output, "%"PRIi32, item->value); + virBufferVSprintf(output, "%d", (int)item->value); }); /* esxVI_Int_SerializeList */ @@ -992,7 +992,7 @@ ESX_VI__TEMPLATE__LIST__APPEND(Long); /* esxVI_Long_Serialize */ ESX_VI__TEMPLATE__SERIALIZE_EXTRA(Long, "xsd:long", { - virBufferVSprintf(output, "%"PRIi64, item->value); + virBufferVSprintf(output, "%lld", (long long int)item->value); }); /* esxVI_Long_SerializeList */ -- |: 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 permissions problem starting QEMU
by Daniel P. Berrange 31 Jul '09

31 Jul '09

There is a minor bug when running QEMU non-root, and having capng enabled. libvirt is unable to write the PID file in /var/run/libvirt/qemu, since its now owned by 'qemu', but libvirtd has dropped all capabilties at this point. The fix is to delay dropping capabilities until after the PID file has been created. We should also be sure to kill the child if writing the PID file fails Daniel commit ce246587178bc6539a3ea6181cdf06ea45878fd3 Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Thu Jul 30 14:58:16 2009 +0100 Fix problem writing QEMU pidfile * src/util.c: Don't drop capabilities until after the PID file has been written. Kill off child if writing the PID file fails * src/qemu_driver.c: Remove bogus trailing '/' in state dir diff --git a/src/qemu_driver.c b/src/qemu_driver.c index 9fb8506..26897d3 100644 --- a/src/qemu_driver.c +++ b/src/qemu_driver.c @@ -468,7 +468,7 @@ qemudStartup(int privileged) { goto out_of_memory; if (virAsprintf(&qemu_driver->stateDir, - "%s/run/libvirt/qemu/", LOCAL_STATE_DIR) == -1) + "%s/run/libvirt/qemu", LOCAL_STATE_DIR) == -1) goto out_of_memory; } else { uid_t uid = geteuid(); diff --git a/src/util.c b/src/util.c index ee64b28..39aae24 100644 --- a/src/util.c +++ b/src/util.c @@ -513,12 +513,6 @@ __virExec(virConnectPtr conn, if ((hook)(data) != 0) _exit(1); - /* The hook above may need todo something privileged, so - * we delay clearing capabilities until now */ - if ((flags & VIR_EXEC_CLEAR_CAPS) && - virClearCapabilities() < 0) - _exit(1); - /* Daemonize as late as possible, so the parent process can detect * the above errors with wait* */ if (flags & VIR_EXEC_DAEMON) { @@ -543,6 +537,9 @@ __virExec(virConnectPtr conn, if (pid > 0) { if (pidfile && virFileWritePidPath(pidfile,pid)) { + kill(pid, SIGTERM); + usleep(500*1000); + kill(pid, SIGTERM); virReportSystemError(conn, errno, "%s", _("could not write pidfile")); _exit(1); @@ -551,6 +548,12 @@ __virExec(virConnectPtr conn, } } + /* The steps above may need todo something privileged, so + * we delay clearing capabilities until the last minute */ + if ((flags & VIR_EXEC_CLEAR_CAPS) && + virClearCapabilities() < 0) + _exit(1); + if (envp) execve(argv[0], (char **) argv, (char**)envp); else -- |: 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 :|

3 4

[libvirt] PATCH: Disable IPv6 on virtual network bridges
by Daniel P. Berrange 31 Jul '09

31 Jul '09

This is to address: https://bugzilla.redhat.com/show_bug.cgi?id=501934 which allows the guest to DOS the host IPv6 connectivity Daniel commit 763cf06ff76b4ded03a9b577cd8c541729190edc Author: Daniel P. Berrange <berrange(a)redhat.com> Date: Thu Jul 30 16:34:56 2009 +0100 Disable IPv6 on virtual networks If the bridge device is configured to have IPv6 address and accept router advertisments, then a malicious guest can send out bogus advertisments and hijack/DOS host IPv6 connectivity * src/network_driver.c: Set accept_ra=0, disable_ipv6=1, autoconf=0 for IPv6 sysctl on virual network bridge devices diff --git a/src/network_driver.c b/src/network_driver.c index 1683631..eaea454 100644 --- a/src/network_driver.c +++ b/src/network_driver.c @@ -788,6 +788,55 @@ networkEnableIpForwarding(void) return virFileWriteStr("/proc/sys/net/ipv4/ip_forward", "1\n"); } +#define SYSCTL_PATH "/proc/sys" + +static int networkDisableIPV6(virConnectPtr conn, + virNetworkObjPtr network) +{ + char *field = NULL; + int ret = -1; + + if (virAsprintf(&field, SYSCTL_PATH "/net/ipv6/conf/%s/disable_ipv6", network->def->bridge) < 0) { + virReportOOMError(conn); + goto cleanup; + } + + if (virFileWriteStr(field, "1") < 0) { + virReportSystemError(conn, errno, + _("cannot enable %s"), field); + goto cleanup; + } + VIR_FREE(field); + + if (virAsprintf(&field, SYSCTL_PATH "/net/ipv6/conf/%s/accept_ra", network->def->bridge) < 0) { + virReportOOMError(conn); + goto cleanup; + } + + if (virFileWriteStr(field, "0") < 0) { + virReportSystemError(conn, errno, + _("cannot disable %s"), field); + goto cleanup; + } + VIR_FREE(field); + + if (virAsprintf(&field, SYSCTL_PATH "/net/ipv6/conf/%s/autoconf", network->def->bridge) < 0) { + virReportOOMError(conn); + goto cleanup; + } + + if (virFileWriteStr(field, "1") < 0) { + virReportSystemError(conn, errno, + _("cannot enable %s"), field); + goto cleanup; + } + + ret = 0; +cleanup: + VIR_FREE(field); + return ret; +} + static int networkStartNetworkDaemon(virConnectPtr conn, struct network_driver *driver, virNetworkObjPtr network) { @@ -806,6 +855,9 @@ static int networkStartNetworkDaemon(virConnectPtr conn, return -1; } + if (networkDisableIPV6(conn, network) < 0) + goto err_delbr; + if (brSetForwardDelay(driver->brctl, network->def->bridge, network->def->delay) < 0) goto err_delbr; -- |: 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 4

[libvirt] [PATCH] BZ481602: Attempted to reproduce the bug led to issues around type mapping. They are resolved, and the code from the bug runs aginst hte test driver. Once published I will have the submitter reproduce it.
by Bryan Kearney 31 Jul '09

31 Jul '09

--- src/main/java/org/libvirt/Domain.java | 4 +--- src/main/java/org/libvirt/SchedLongParameter.java | 2 +- src/main/java/org/libvirt/SchedParameter.java | 2 ++ src/main/java/org/libvirt/SchedUintParameter.java | 2 +- src/main/java/org/libvirt/jna/Libvirt.java | 2 +- src/test/java/test.java | 8 ++++++++ 6 files changed, 14 insertions(+), 6 deletions(-) diff --git a/src/main/java/org/libvirt/Domain.java b/src/main/java/org/libvirt/Domain.java index a9a6b25..a7b49ee 100644 --- a/src/main/java/org/libvirt/Domain.java +++ b/src/main/java/org/libvirt/Domain.java @@ -607,13 +607,11 @@ public class Domain { * @throws LibvirtException */ public void setSchedulerParameters(SchedParameter[] params) throws LibvirtException { - IntByReference nParams = new IntByReference(); - nParams.setValue(params.length); virSchedParameter[] input = new virSchedParameter[params.length]; for (int x = 0; x < params.length; x++) { input[x] = SchedParameter.toNative(params[x]); } - libvirt.virDomainSetSchedulerParameters(VDP, input, nParams); + libvirt.virDomainSetSchedulerParameters(VDP, input, params.length); processError(); } diff --git a/src/main/java/org/libvirt/SchedLongParameter.java b/src/main/java/org/libvirt/SchedLongParameter.java index 50628b0..7999777 100644 --- a/src/main/java/org/libvirt/SchedLongParameter.java +++ b/src/main/java/org/libvirt/SchedLongParameter.java @@ -21,7 +21,7 @@ public final class SchedLongParameter extends SchedParameter { } public int getType() { - return 2; + return 3; } public String getTypeAsString() { diff --git a/src/main/java/org/libvirt/SchedParameter.java b/src/main/java/org/libvirt/SchedParameter.java index 92b3547..370b0da 100644 --- a/src/main/java/org/libvirt/SchedParameter.java +++ b/src/main/java/org/libvirt/SchedParameter.java @@ -4,6 +4,7 @@ import java.util.Arrays; import org.libvirt.jna.Libvirt; import org.libvirt.jna.virSchedParameter; +import org.libvirt.jna.virSchedParameterValue; import com.sun.jna.Native; @@ -43,6 +44,7 @@ public abstract class SchedParameter { public static virSchedParameter toNative(SchedParameter param) { virSchedParameter returnValue = new virSchedParameter(); + returnValue.value = new virSchedParameterValue() ; returnValue.field = Arrays.copyOf(param.field.getBytes(), Libvirt.VIR_DOMAIN_SCHED_FIELD_LENGTH); returnValue.type = param.getType(); switch (param.getType()) { diff --git a/src/main/java/org/libvirt/SchedUintParameter.java b/src/main/java/org/libvirt/SchedUintParameter.java index 1b96758..9305579 100644 --- a/src/main/java/org/libvirt/SchedUintParameter.java +++ b/src/main/java/org/libvirt/SchedUintParameter.java @@ -22,7 +22,7 @@ public final class SchedUintParameter extends SchedParameter { } public int getType() { - return 3; + return 2; } public String getTypeAsString() { diff --git a/src/main/java/org/libvirt/jna/Libvirt.java b/src/main/java/org/libvirt/jna/Libvirt.java index 62f7ba3..0b30656 100644 --- a/src/main/java/org/libvirt/jna/Libvirt.java +++ b/src/main/java/org/libvirt/jna/Libvirt.java @@ -107,7 +107,7 @@ public interface Libvirt extends Library { public int virDomainSetAutostart(DomainPointer virDomainPtr, int autoStart); public int virDomainSetMaxMemory(DomainPointer virDomainPtr, NativeLong maxMemory); public int virDomainSetMemory(DomainPointer virDomainPtr, NativeLong maxMemory); - public int virDomainSetSchedulerParameters(DomainPointer virDomainPtr, virSchedParameter[] params, IntByReference nparams); + public int virDomainSetSchedulerParameters(DomainPointer virDomainPtr, virSchedParameter[] params, int nparams); public int virDomainSetVcpus(DomainPointer virDomainPtr, int nvcpus); public int virDomainShutdown(DomainPointer virDomainPtr); public int virDomainSuspend(DomainPointer virDomainPtr); diff --git a/src/test/java/test.java b/src/test/java/test.java index 9427cc5..be12d5e 100644 --- a/src/test/java/test.java +++ b/src/test/java/test.java @@ -245,6 +245,14 @@ public class test { for(SchedParameter c: testDomain.getSchedulerParameters()){ System.out.println(c.getTypeAsString() +":"+ c.field +":"+ c.getValueAsString()); } + + // test setting a scheduled parameter + SchedUintParameter[] pars = new SchedUintParameter[1]; + pars[0] = new SchedUintParameter(); + pars[0].field = "weight"; + pars[0].value = 100; + testDomain.setSchedulerParameters(pars); + System.out.println("virDomainGetUUID:" + testDomain.getUUID()); for(int c: testDomain.getUUID()) System.out.print(String.format("%02x", c)); -- 1.6.0.6

1 0

[libvirt] Pre-0.7.0 release for testing
by Daniel Veillard 31 Jul '09

31 Jul '09

Considering the large amount of new code and features expected for 0.7.0 at the end of the week, I made a pre-release at ftp://libvirt.org/libvirt/libvirt-0.7.0-0.1.gitf055724.tar.gz if people have time to test and report, that would be great ! I expect to make the actual release this thursday, unless there is too many issues in which case I would postpone to beginning of next week (or whenever main problems get fixed...) thanks ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel(a)veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/

2 4

[libvirt] Fix -Werror build error regarding to '#ifdef IFF_VNET_HDR'
by Maximilian Wilhelm 31 Jul '09

31 Jul '09

Hi! Our build bot found a build error when building libVirt with -Werror introduced in b4f62abbf1191c8fbab3306b4bf2f2567e18067f kvm/virtio: Set IFF_VNET_HDR when setting up tap fds The attached patch kind of fixes the build error but I'm not sure, if this is the final solution for this problem, though... Ciao Max -- The real problem with C++ for kernel modules is: the language just sucks. -- Linus Torvalds

2 3

[libvirt] [PATCH] Drop the leading lib from libvirt. Based on testing by thomas@scripty.at this is a better way to load the libarary
by Bryan Kearney 31 Jul '09

31 Jul '09

--- src/main/java/org/libvirt/jna/Libvirt.java | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/main/java/org/libvirt/jna/Libvirt.java b/src/main/java/org/libvirt/jna/Libvirt.java index 33914ac..62f7ba3 100644 --- a/src/main/java/org/libvirt/jna/Libvirt.java +++ b/src/main/java/org/libvirt/jna/Libvirt.java @@ -20,7 +20,7 @@ public interface Libvirt extends Library { public int authCallback(virConnectCredential cred, int ncred, Pointer cbdata); } - Libvirt INSTANCE = (Libvirt) Native.loadLibrary("libvirt", Libvirt.class); + Libvirt INSTANCE = (Libvirt) Native.loadLibrary("virt", Libvirt.class); // Constants we need public static int VIR_UUID_BUFLEN = 16; -- 1.6.0.6

3 2

[libvirt] RFC: libvirt java bindings based on JNA
by Bryan Kearney 31 Jul '09

31 Jul '09

I would like to get some comments on an initial cut of the java bindings on top of JNA[1]. They are not 100% complete (see below), but they are good enough to be criticized. In addition, if there are features additions to the API which people are interested in, I would appreciate the feedback on that. JNA is a java implementation of the Foreign Function Interface [2]. The main benefit is that it removes the need for all the nasty C glue code. What replaces it is pure java mapping files. If you look in the org.libvirt.jna package in the code, you will see that the 2928 lines of jni code have been replaced by 427 lines of java code. To be fair, some of the business logic from the C code is now in the org.libvirt package... but you get the point. The original java API has remained the same. The open issues with the code are: 1) Authorization callback has not been tested. 2) GetUUID is not returning the correct int array. 3) Find by UUID is not working These all are mapping issues to be worked out. The rest of the code should be working, but I would appreciate it being exercised. The main drawback to this approach is that it requires the use of java 1.6. I looked on EPEL, F10, and F11.. and java 1.6 and the JNA jar file are available so I did not believe this was an issue. I will send a patch stream to the list, but you can also get the code via an SRPM [3] or via github [4]. -- bk [1] https://jna.dev.java.net/ [2] http://en.wikipedia.org/wiki/Foreign_function_interface [3] http://bkearney.fedorapeople.org/libvirt-java-0.3.0pre-1.fc10.src.rpm [4] http://github.com/bkearney/libvirt-java/tree/jna

3 11

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

31 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