- Devel - libvirt List Archives

[libvirt] migration fails
by jens 09 May '09

09 May '09

Hello, I tried to migrate a kvm domain to a second identical machine with the command: virsh migrate --live dom1 qemu+ssh://second.machine.org/system which leads me to an unresponsive virsh on the second machine - 'virsh list' just takes forever and qemu wasn't started (the domain was not migrated), finally I have to kill libvirtd on the second machine. Because I'm new to libvirt I searched the web to find a solution for this behavior but without success. Please can someone on the list point me to the right direction how I can use libvirt to migrate kvm domains? I use kernel 2.6.29.1 and kvm-85 with libvirt 0.6.2 on an debian unstable system. Attached are debug logs generated by the source- and destination-machine. Thank you very much Jens

5 7

[libvirt] [PATCH] change permissions of directories in cgroups
by Ryota Ozaki 08 May '09

08 May '09

Hi, This patch creates a directory in cgroups with an ordinary permission 0755 (rwxr-xr-x) instead of 0655 (rw-r-xr-x). I guess 0655 is not expected and just a mistake, or is there a special reason? Thanks, ozaki-r Signed-off-by: Ryota Ozaki <ozaki.ryota(a)gmail.com> >From 4829855bc0baa3c75806f106bc0e54eb1da75eea Mon Sep 17 00:00:00 2001 From: Ryota Ozaki <ozaki.ryota(a)gmail.com> Date: Fri, 8 May 2009 03:23:50 +0900 Subject: [PATCH] change permissions of directories in cgroups This patch creates a directory in cgroups with an ordinary permission 0755 (rwxr-xr-x) instead of 0655 (rw-r-xr-x). --- src/cgroup.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/cgroup.c b/src/cgroup.c index d1d44a2..50517e2 100644 --- a/src/cgroup.c +++ b/src/cgroup.c @@ -436,7 +436,7 @@ static int virCgroupMakeGroup(const char *name) virCgroupFree(&root); if (access(path, F_OK) != 0) { - if (mkdir(path, 0655) < 0) { + if (mkdir(path, 0755) < 0) { rc = -errno; VIR_FREE(path); break; -- 1.6.0.6

3 3

[libvirt] [PATCH] lxc: fix mknod file type
by Ryota Ozaki 08 May '09

08 May '09

Hi, This patch fixes the 2nd argument of mknod syscall. The argument should include a file type, i.e., S_IFCHR in this case, otherwise created files will be regular files. Thanks, ozaki-r Signed-off-by: Ryota Ozaki <ozaki.ryota(a)gmail.com> >From 60009b725da855f131e8272f0397455ae27bafc3 Mon Sep 17 00:00:00 2001 From: Ryota Ozaki <ozaki.ryota(a)gmail.com> Date: Wed, 6 May 2009 03:47:04 +0900 Subject: [PATCH] lxc: fix mknod file type This patch fixes the 2nd argument of mknod syscall. The argument should include a file type, i.e., S_IFCHR in this case, otherwise created files will be regular files. --- src/lxc_container.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lxc_container.c b/src/lxc_container.c index 3946b84..c30daec 100644 --- a/src/lxc_container.c +++ b/src/lxc_container.c @@ -440,7 +440,7 @@ static int lxcContainerPopulateDevices(void) /* Populate /dev/ with a few important bits */ for (i = 0 ; i < ARRAY_CARDINALITY(devs) ; i++) { dev_t dev = makedev(devs[i].maj, devs[i].min); - if (mknod(devs[i].path, 0, dev) < 0 || + if (mknod(devs[i].path, S_IFCHR, dev) < 0 || chmod(devs[i].path, devs[i].mode)) { virReportSystemError(NULL, errno, _("failed to make device %s"), @@ -457,7 +457,7 @@ static int lxcContainerPopulateDevices(void) } } else { dev_t dev = makedev(LXC_DEV_MAJ_TTY, LXC_DEV_MIN_PTMX); - if (mknod("/dev/ptmx", 0, dev) < 0 || + if (mknod("/dev/ptmx", S_IFCHR, dev) < 0 || chmod("/dev/ptmx", 0666)) { virReportSystemError(NULL, errno, "%s", _("failed to make device /dev/ptmx")); -- 1.6.0.6

2 4

[libvirt] [PATCH] vbox: don't fail on missing vbox
by Guido Günther 08 May '09

08 May '09

Hi, virsh fails with: 10:47:37.104: debug : do_open:922 : no name, allowing driver auto-select 10:47:37.104: debug : do_open:930 : trying driver 0 (Test) ... 10:47:37.104: debug : do_open:936 : driver 0 Test returned DECLINED 10:47:37.104: debug : do_open:930 : trying driver 1 (OPENVZ) ... 10:47:37.105: debug : do_open:936 : driver 1 OPENVZ returned DECLINED 10:47:37.105: debug : do_open:930 : trying driver 2 (VBOX) ... 10:47:37.106: debug : do_open:936 : driver 2 VBOX returned ERROR 10:47:37.106: debug : virUnrefConnect:210 : unref connection 0x10032b10 1 10:47:37.106: debug : virReleaseConnect:171 : release connection 0x10032b10 error: failed to connect to the hypervisor if one has built with vbox support but vbox is not installed. Attached patch calls VBoxCGlueInit() early so we can return 'declined' instead of 'error'. O.k. to apply? -- Guido

3 4

[libvirt] [PATCH] vbox: don't crash if init fails early
by Guido Günther 08 May '09

08 May '09

Hi, not having set up pfuncs already results in a core dump. O.k. to apply? Cheers, -- Guido

3 3

[libvirt] Libvirt-Java Binding
by Marcel Michelmann 07 May '09

07 May '09

Hello, I created a program in Java for communicating with a libvirt/XEN host over SSH. For every update I create a new connection. It just runs about 1 1/2 minutes. I guess it is because I do not close any of these connections. When I tried to close them after having used them (Connect.close()) the program hangs. But after pressing Ctrl+C the program stops and the results are shown. Is that a bug in the libvirt-Java binding? Does anybody else have the same problem with Connect.close()? Thanks, Marcel

3 3

[libvirt] using public vir*Free() (instead of virUnref*()) from vir*FreeName() destructors.
by Laine Stump 07 May '09

07 May '09

I just noticed in datatypes.c that the vir(Network|StoragePool|StorageVol)FreeName destructors call the public API vir*Free() functions rather than the local virUnref*(). vir*Free() all clear any errors, which seems like it might not be the right thing to do (eg, if we're cleaning out the hashtables as part of some error-handling path, and haven't yet logged the error). On the other hand, the public API function does some sanity checking on the object to make sure it really is what we think it is before calling the unref. So it's got that going for it. (5 points to anyone who catches the reference without going to Google) But I don't know enough to know whether or not that's important in this case. So which is more important: the extra sanity check, or not clearing existing error codes?

2 3

[libvirt] PCI Passthrough
by Aaron Clausen 07 May '09

07 May '09

I'm having a heck of a time trying to figure out how to do PCI passthrough for a PCI SCSI controller. I need, if at all possible, to access it so that I can use a SCSI tape drive with a Windows Server 2000 guest. This is xml config I have: <domain type='kvm'> <name>testserver</name> <uuid>f75a2f63-e1bd-72a5-6134-3fcafbcdb970</uuid> <memory>524288</memory> <currentMemory>524288</currentMemory> <vcpu>1</vcpu> <os> <type arch='x86_64' machine='pc'>hvm</type> <boot dev='hd'/> </os> <features> <pae/> </features> <clock offset='localtime'/> <on_poweroff>destroy</on_poweroff> <on_reboot>restart</on_reboot> <on_crash>restart</on_crash> <devices> <emulator>/usr/bin/kvm</emulator> <disk type='file' device='disk'> <source file='/var/lib/libvirt/images/testserver.img'/> <target dev='hda' bus='ide'/> </disk> <disk type='block' device='cdrom'> <source dev='/dev/cdrom'/> <target dev='hdc' bus='ide'/> <readonly/> </disk> <interface type='bridge'> <mac address='54:52:00:50:03:08'/> <source bridge='br0'/> <target dev='vnet2'/> </interface> <serial type='pty'> <source path='/dev/pts/7'/> <target port='0'/> </serial> <console type='pty' tty='/dev/pts/7'> <source path='/dev/pts/7'/> <target port='0'/> </console> <input type='tablet' bus='usb'/> <input type='mouse' bus='ps2'/> <graphics type='vnc' port='-1' autoport='yes' keymap='en-us'/> </devices> </domain> -- Aaron Clausen mightymartianca(a)gmail.com

2 4

[libvirt] Virsh list on xen after stop/restart not working properly
by Thomas Graves 07 May '09

07 May '09

Hello, I am seeing an issue on libvirt 0.6.1 and 0.6.3 with xen3.3.1 where virsh list -all isn't reporting all vms that are active. It happens after you shutdown and then restart a vm. As a side note, it works fine on libvirt 0.3.3. It appears that the domid of the vm is changed when you shutdown/restart a vm on xen. Libvirt is checking xenstore to find domids and it is finding the old ones that are marked as cancelled and not finding the higher number ones that are active/valid. Has anyone seen this issue and have a patch for it? It looks like the old libvirt 0.3.3 went to the hypervisor and xend before the xenstore and that is working properly. Thanks, Tom

2 1

[libvirt] KVM processes -- should we be able to attach them to the libvirtd process?
by Hugh O. Brock 07 May '09

07 May '09

Not too long ago we took a patch that allowed QEMU VMs to keep running even if libvirtd died or was restarted. I was talking to Matt Farellee (cc'd) this afternoon about manageability, and he feels fairly strongly that this behavior should be optional -- in other words, it should be possible to guarantee that if libvirtd dies, it will take all the VMs with the "die-with-libvirtd" flag set down with it. I'm not sure this API is portable to Xen, but it would work on any hypervisor that represents the VM as a normal process. Does this strike anyone else as useful behavior? Thanks, --Hugh

5 6

[libvirt] Java api for libvirt migration
by Zvi Dubitzky 07 May '09

07 May '09

I am trying to do a migrate of VM with libvirt 0.6.1 and kvm-84 . I am successful in doing it many times via the C interface . I can do most of the basic libvirt activity via the libvirt JAVA library (libvirt-java-0.2.1). When it comes to a migrate the JVM fails wit a log file attached below . Can someone help. My memory consumption in my JAVA test code is small. The migrate parameters I am sending with the Java are identical to those I use in the C case and is: destDomain = localDomain.migrate (dcon, // destination host Connect object VIR_MIGRATE_LIVE, null, //const char *dname destination domain name target_uri, //"tcp:9.148.28.13:4444" 0 // unsigned long bandwidth) - do not specify ); thanks Zvi Dubitzky Virtualization and System Architecture Email:dubi@il.ibm.com IBM Haifa Research Laboratory Phone: +972-4-8296182 Haifa, 31905, ISRAEL

1 0

[libvirt] [PATCH] return a sensible error on bogus paths
by Guido Günther 07 May '09

07 May '09

Hi, attached patch makes it easier to detect bogus paths when creating/removing pidfiles. O.k. to apply? Cheers, -- Guido

2 2

[libvirt] [PATCH]: don't crash when calling libvirt_lxc without arguments
by Guido Günther 07 May '09

07 May '09

In this case we reference def with is NULL. O.k. to apply? Cheers, -- Guido

2 2

[libvirt] PATCH: Add LSB init headers
by Frederik Himpe 07 May '09

07 May '09

The following patch is needed to make sure libvirt is not started too early in the boot sequence on Mandriva. --- libvirt-0.6.1/qemud/libvirtd.init.in.orig 2009-03-05 19:34:05.000000000 +0100 +++ libvirt-0.6.1/qemud/libvirtd.init.in 2009-03-05 19:34:54.000000000 +0100 @@ -5,7 +5,11 @@ # ### BEGIN INIT INFO # Provides: libvirtd +# Required-Start: $network messagebus +# Should-Start: $named # Should-Start: xend +# Required-Stop: $network messagebus +# Should-Stop: $named # Default-Start: 3 4 5 # Short-Description: daemon for libvirt virtualization API # Description: This is a daemon for managing guest instances

4 5

[libvirt] [PATCH 2/2] support for multiple graphics devices
by Pritesh Kothari 07 May '09

07 May '09

Hi All, I have added support for multiple graphics devices, the patches are as below. I have checked them against current cvs head and works fine PATCH 1/2: contains changes in libvirt for multiple graphics devices PATCH 2/2: contains corresponding changes in qemu driver. Regards, Pritesh

3 5

[libvirt] Authentication with virConnectAuthPtr
by Eduardo Otubo 07 May '09

07 May '09

Hello all, I'll start using virConnectAuthPtr to handle the authentication proccess at the phyp driver I'm wrinting <https://www.redhat.com/archives/libvir-list/2009-April/msg00493.html> and I have some doubts in my mind: 1) What are the types of authentication that 'int *credtype' can hold? And who fills it with the information I'll need? 2) Once known the credential type, I need to use the function pointer 'virConnectAuthCallbackPtr cb' to get the information whatever it is, right? I mean, it can be a password, a pubkey or anything else, right? Is there a callback able to handle password or pubkeys? 3) And finally I'll be able to use the 'void *cbdata' to manage the authentication in my way. In my case, using libssh. In fact, in the end of the process, I'll just need a password or a key to get things working. There is no driver authenticatin agaist ssh channel, that's why I got so confused with this topic. I don't know if my thoughts are pretty clear here. But I would like to confirm the information 2 and 3, and check how 1 works. Could anyone help me? Thanks in advance, []'s -- Eduardo Otubo Software Engineer Linux Technology Center IBM Systems & Technology Group Mobile: +55 19 8135 0885 otubo(a)linux.vnet.ibm.com

2 2

[libvirt] discrepancies in issuing virsh capabilities with different user-accounts
by Gerrit Slomma 06 May '09

06 May '09

Hello When doing a virsh capabilities with root i get a different output than when issuing the command with my own unprivileged account roadrunner. Furthermore the first issuing of the command as roadrunner throws a error-message. Restarting of the libvirt-daemon gives the correct output. Sample: Last login: Wed May 6 21:00:56 2009 from 192.168.1.120 [root@rr018 ~]# virsh capabilities|grep kvm [root@rr018 ~]# su - roadrunner [roadrunner@rr018 ~]$ virsh capabilities|grep kvm libvir: Remote Fehler : unable to connect to '@/home/roadrunner/.libvirt/libvirt-sock': Verbindungsaufbau abgelehnt Fehler: Verbindung zum Hypervisor scheiterte Failed to bind socket to '@/home/roadrunner/.libvirt/libvirt-sock': Address already in use Failed to bind socket to '@/home/roadrunner/.libvirt/libvirt-sock': Address already in use Failed to bind socket to '@/home/roadrunner/.libvirt/libvirt-sock': Address already in use Failed to bind socket to '@/home/roadrunner/.libvirt/libvirt-sock': Address already in use ^C [roadrunner@rr018 ~]$ virsh capabilities|grep kvm <domain type='kvm'> <emulator>/usr/bin/kvm</emulator> <domain type='kvm'> [roadrunner@rr018 ~]$ exit logout [root@rr018 ~]# virsh capabilities|grep kvm [root@rr018 ~]# /etc/init.d/libvirtd restart libvirtd-Daemon beenden: [ OK ] libvirtd-Daemon starten: [ OK ] [root@rr018 ~]# virsh capabilities|grep kvm <domain type='kvm'> <emulator>/usr/bin/kvm</emulator> <domain type='kvm'> [root@rr018 ~]#

1 1

Re: [libvirt] KVM processes -- should we be able to attach them to the libvirtd process?
by Daniel P. Berrange 06 May '09

06 May '09

On Wed, May 06, 2009 at 08:56:18PM +0200, Gerrit Slomma wrote: > Daniel P. Berrange schrieb: > >On Tue, May 05, 2009 at 11:38:13PM -0500, Matthew Farrellee wrote: > > > >>It doesn't appear to be the case that the libvirtd daemon can trivially > >>restart and continue with no interruptions. Right now it loses track of > >>VMs. > >> > > > >That a is a bug then, if you can reproduce it, please file a BZ ticket > >so we can track it down & fix it. > > > > > >>In a scenario where VMs are not deployed and locked to specific physical > >>nodes, it can be highly valuable to have ways to ensure a VM is no > >>longer running when a layer of its management stops functioning. > >> > > > >IMHO this is a problem to be solved by clustering software. If the > >clustering software detects a failure with the management service, > >then it should power fence the entire node. Relying on management > >service failure to kill the VMs will never be reliable enough. > > > I think he is pointing towards a VM that runs on a host where it isn't > defined at via a corresponding *.xml. > If you restart a libvirt i looses connection to this or these specific > VM(s). That is a bug that needs fixing. Even if there is no persistent config, we should not loose track of the running VM, because we always write out the 'live' XML config to /var/run/libvirt explicitly so that it is available at restart. 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] PATCH: Raise log level for dlopen() problems
by Daniel P. Berrange 06 May '09

06 May '09

Some of the problems when dlopen'ing a module really should be reported to the user/admin more readily. So this raises the logging level for the important failure messages, so they're visible by default Daniel diff -r 15c4668d403b src/driver.c --- a/src/driver.c Wed Apr 29 15:29:17 2009 +0100 +++ b/src/driver.c Thu Apr 30 14:49:27 2009 +0100 @@ -54,13 +54,13 @@ virDriverLoadModule(const char *name) return NULL; if (access(modfile, R_OK) < 0) { - DEBUG("Moodule %s not accessible", modfile); + VIR_WARN("Module %s not accessible", modfile); goto cleanup; } handle = dlopen(modfile, RTLD_NOW | RTLD_LOCAL); if (!handle) { - DEBUG("failed to load module %s %s", modfile, dlerror()); + VIR_ERROR("failed to load module %s %s", modfile, dlerror()); goto cleanup; } @@ -70,12 +70,12 @@ virDriverLoadModule(const char *name) regsym = dlsym(handle, regfunc); if (!regsym) { - DEBUG("Missing module registration symbol %s", regfunc); + VIR_ERROR("Missing module registration symbol %s", regfunc); goto cleanup; } if ((*regsym)() < 0) { - DEBUG("Failed module registration %s", regfunc); + VIR_ERROR("Failed module registration %s", regfunc); 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: More robust name & UUID uniqueness checking for QEMU
by Daniel P. Berrange 06 May '09

06 May '09

When defining a VM config, we need to apply the following logi - If existing VM has same UUID - If name also matches => allow - Else => raise error - Else - If name matches => raise error - Else => allow When creating a live VM, or restoring a VM we need to apply similar, but slightly different logic - If existing VM has same UUID - If name also matches - If existing VM is running => raise error - Else => allow - Else => raise error - Else - If name matches => raise error - Else => allow This patch applies those checks for the QEMU driver Daniel diff -r 4e6a98395da5 src/qemu_driver.c --- a/src/qemu_driver.c Thu Apr 30 14:50:14 2009 +0100 +++ b/src/qemu_driver.c Thu Apr 30 15:03:03 2009 +0100 @@ -2145,22 +2145,37 @@ static virDomainPtr qemudDomainCreate(vi if (virSecurityDriverVerify(conn, def) < 0) goto cleanup; - vm = virDomainFindByName(&driver->domains, def->name); - if (vm) { - qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, - _("domain '%s' is already defined"), - def->name); - goto cleanup; - } + /* See if a VM with matching UUID already exists */ vm = virDomainFindByUUID(&driver->domains, def->uuid); if (vm) { - char uuidstr[VIR_UUID_STRING_BUFLEN]; - - virUUIDFormat(def->uuid, uuidstr); - qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, - _("domain with uuid '%s' is already defined"), - uuidstr); - goto cleanup; + /* 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); + qemudReportError(conn, NULL, 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)) { + qemudReportError(conn, NULL, 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); + qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + def->name, uuidstr); + goto cleanup; + } } if (!(vm = virDomainAssignDef(conn, @@ -2348,6 +2363,11 @@ static int qemudDomainDestroy(virDomainP _("no domain with matching uuid '%s'"), uuidstr); goto cleanup; } + if (!virDomainIsActive(vm)) { + qemudReportError(dom->conn, dom, NULL, VIR_ERR_OPERATION_FAILED, + "%s", _("domain is not running")); + goto cleanup; + } qemudShutdownVMDaemon(dom->conn, driver, vm); event = virDomainEventNewFromObj(vm, @@ -3258,17 +3278,36 @@ static int qemudDomainRestore(virConnect goto cleanup; } - /* Ensure the name and UUID don't already exist in an active VM */ + /* See if a VM with matching UUID already exists */ vm = virDomainFindByUUID(&driver->domains, def->uuid); - if (!vm) - vm = virDomainFindByName(&driver->domains, def->name); 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); + qemudReportError(conn, NULL, 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)) { qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_INVALID, _("domain is already active as '%s'"), vm->def->name); goto cleanup; - } else { - virDomainObjUnlock(vm); + } + 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); + qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + def->name, uuidstr); + goto cleanup; } } @@ -3603,18 +3642,41 @@ static virDomainPtr qemudDomainDefine(vi if (virSecurityDriverVerify(conn, def) < 0) goto cleanup; - vm = virDomainFindByName(&driver->domains, def->name); + /* 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); + qemudReportError(conn, NULL, 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); + qemudReportError(conn, NULL, NULL, VIR_ERR_OPERATION_FAILED, + _("domain '%s' is already defined with uuid %s"), + def->name, uuidstr); + goto cleanup; + } } if (!(vm = virDomainAssignDef(conn, &driver->domains, def))) { - virDomainDefFree(def); - goto cleanup; - } + goto cleanup; + } + def = NULL; vm->persistent = 1; if (virDomainSaveConfig(conn, @@ -3636,6 +3698,7 @@ static virDomainPtr qemudDomainDefine(vi if (dom) dom->id = vm->def->id; cleanup: + virDomainDefFree(def); if (vm) virDomainObjUnlock(vm); if (event) -- |: 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: Don't reset / detach host PCI devices in test scripts !
by Daniel P. Berrange 06 May '09

06 May '09

The PCI passthrough patches made it so that qemudBuildCommandLine() would actually try to detach your host devices & reset them. Most definitely not what you want when running this via a test case! This patch moves the host device management out into a separate method, so that qemudBuildCommandLine() doesn't do anything except safely build the command line. Daniel qemu_conf.c | 46 ----------------------------------- qemu_driver.c | 76 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 76 insertions(+), 46 deletions(-) Index: src/qemu_conf.c =================================================================== RCS file: /data/cvs/libvirt/src/qemu_conf.c,v retrieving revision 1.133 diff -u -p -u -p -r1.133 qemu_conf.c --- src/qemu_conf.c 2 Mar 2009 20:22:35 -0000 1.133 +++ src/qemu_conf.c 2 Mar 2009 20:47:36 -0000 @@ -47,7 +47,6 @@ #include "datatypes.h" #include "xml.h" #include "nodeinfo.h" -#include "pci.h" #define VIR_FROM_THIS VIR_FROM_QEMU @@ -1395,52 +1394,7 @@ int qemudBuildCommandLine(virConnectPtr ADD_ARG_LIT("-pcidevice"); ADD_ARG_LIT(pcidev); VIR_FREE(pcidev); - - if (hostdev->managed) { - pciDevice *dev = pciGetDevice(conn, - hostdev->source.subsys.u.pci.domain, - hostdev->source.subsys.u.pci.bus, - hostdev->source.subsys.u.pci.slot, - hostdev->source.subsys.u.pci.function); - if (!dev) - goto error; - - if (pciDettachDevice(conn, dev) < 0) { - pciFreeDevice(conn, dev); - goto error; - } - - pciFreeDevice(conn, dev); - } /* else { - XXX validate that non-managed device isn't in use, eg - by checking that device is either un-bound, or bound - to pci-stub.ko - } */ } - - } - - /* Now that all the PCI hostdevs have be dettached, we can reset them */ - for (i = 0 ; i < vm->def->nhostdevs ; i++) { - virDomainHostdevDefPtr hostdev = vm->def->hostdevs[i]; - pciDevice *dev; - - if (hostdev->mode != VIR_DOMAIN_HOSTDEV_MODE_SUBSYS || - hostdev->source.subsys.type != VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI) - continue; - - dev = pciGetDevice(conn, - hostdev->source.subsys.u.pci.domain, - hostdev->source.subsys.u.pci.bus, - hostdev->source.subsys.u.pci.slot, - hostdev->source.subsys.u.pci.function); - if (!dev) - goto error; - - if (pciResetDevice(conn, dev) < 0) - goto error; - - pciFreeDevice(conn, dev); } if (migrateFrom) { Index: src/qemu_driver.c =================================================================== RCS file: /data/cvs/libvirt/src/qemu_driver.c,v retrieving revision 1.208 diff -u -p -u -p -r1.208 qemu_driver.c --- src/qemu_driver.c 2 Mar 2009 17:39:43 -0000 1.208 +++ src/qemu_driver.c 2 Mar 2009 20:47:36 -0000 @@ -1133,6 +1133,79 @@ static int qemudNextFreeVNCPort(struct q return -1; } +static int qemuPrepareHostDevices(virConnectPtr conn, + virDomainDefPtr def) { + int i; + + /* We have to use 2 loops here. *All* devices must + * be detached before we reset any of them, because + * in some cases you have to reset the whole PCI bus, + * which impacts all devices on it + */ + + for (i = 0 ; i < def->nhostdevs ; i++) { + virDomainHostdevDefPtr hostdev = def->hostdevs[i]; + + if (hostdev->mode != VIR_DOMAIN_HOSTDEV_MODE_SUBSYS) + continue; + if (hostdev->source.subsys.type != VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI) + continue; + + if (!hostdev->managed) { + pciDevice *dev = pciGetDevice(conn, + hostdev->source.subsys.u.pci.domain, + hostdev->source.subsys.u.pci.bus, + hostdev->source.subsys.u.pci.slot, + hostdev->source.subsys.u.pci.function); + if (!dev) + goto error; + + if (pciDettachDevice(conn, dev) < 0) { + pciFreeDevice(conn, dev); + goto error; + } + + pciFreeDevice(conn, dev); + } /* else { + XXX validate that non-managed device isn't in use, eg + by checking that device is either un-bound, or bound + to pci-stub.ko + } */ + } + + /* Now that all the PCI hostdevs have be dettached, we can safely + * reset them */ + for (i = 0 ; i < def->nhostdevs ; i++) { + virDomainHostdevDefPtr hostdev = def->hostdevs[i]; + pciDevice *dev; + + if (hostdev->mode != VIR_DOMAIN_HOSTDEV_MODE_SUBSYS) + continue; + if (hostdev->source.subsys.type != VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_PCI) + continue; + + dev = pciGetDevice(conn, + hostdev->source.subsys.u.pci.domain, + hostdev->source.subsys.u.pci.bus, + hostdev->source.subsys.u.pci.slot, + hostdev->source.subsys.u.pci.function); + if (!dev) + goto error; + + if (pciResetDevice(conn, dev) < 0) { + pciFreeDevice(conn, dev); + goto error; + } + + pciFreeDevice(conn, dev); + } + + return 0; + +error: + return -1; +} + static virDomainPtr qemudDomainLookupByName(virConnectPtr conn, const char *name); @@ -1210,6 +1283,9 @@ static int qemudStartVMDaemon(virConnect return -1; } + if (qemuPrepareHostDevices(conn, vm->def) < 0) + return -1; + vm->def->id = driver->nextvmid++; if (qemudBuildCommandLine(conn, driver, vm, qemuCmdFlags, &argv, &progenv, -- |: 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 :|

4 5

[libvirt] [PATCH] Refresh QEMU driver caps in getCapabilities
by Cole Robinson 06 May '09

06 May '09

Hi all, The attached patch fixes QEMU getCapabilities to refresh caps before returning them to the user. Currently the capabilities are only refreshed once (at daemon startup), which means libvirtd needs to be restarted to pick up changes if QEMU or KVM are installed while the daemon is running. See: https://bugzilla.redhat.com/show_bug.cgi?id=460649 There are several things 'wrong' with this change: - We reset/rescan fields that won't change (host arch, mac address prefix). This should be fixed at some point, but isn't a big deal since total performance impact is negligible (see below). - We only refresh the capabilities when the user calls getCapabilities, which means we are still carrying around stale caps prior to that, which is what libvirt validates against. In practice, virt-manager and virt-install both call getCapabilities often so this isn't an issue, though the caps internal API should probably address this at some point. To test the performance impact, I used a simple python script: import libvirt conn = libvirt.open("qemu:///system") for i in range(0, 30): conn.getCapabilities() The time difference was on average .02 seconds slower, which I think is negligible. If at somepoint in the future, capabilities generation becomes smarter (searching PATH emulators, scraping device list output, etc.) it might be worth re-checking the time impact. But for now it doesn't seem to be an issue. Thanks, Cole

4 7

[libvirt] [PATCH 1/1] Changes in version detection and moving a global variable in the global struct
by Pritesh Kothari 06 May '09

06 May '09

Hi All, PATCH 1/1: contains changes in version detection and moving a global variable in the global struct, so that no more global vairables exists. Regards, Pritesh

3 4

[libvirt] [PATCH 1/1] Making path detection more robust in VirtualBox
by Pritesh Kothari 06 May '09

06 May '09

Hi All, PATCH 1/1: contains changes in the glue code for making path detection more robust in VirtualBox Regards, Pritesh

3 2

[libvirt] participate in irc chat
by jens 06 May '09

06 May '09

Hello, on http://libvirt.org/contact.html I read about the #virt irc channel. How can I participate in this channel, it seems to be locked because I have to specify a channel key. Greetings Jens

2 2