March 2010 - Devel - libvirt List Archives

[libvirt] [PATCH] Avoid libvirtd crash when cgroups is not configured on host
by Jim Fehlig 22 Mar '10

22 Mar '10

Patch for libvirtd segfault discussed on IRC recently ... Regards, Jim

2 2

[libvirt] [PATCH] security: selinux: Fix crash when releasing non-existent label
by Cole Robinson 22 Mar '10

22 Mar '10

This can be triggered by the qemuStartVMDaemon cleanup path if a VM references a non-existent USB device (by product) in the XML. Signed-off-by: Cole Robinson <crobinso(a)redhat.com> --- src/security/security_selinux.c | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/security/security_selinux.c b/src/security/security_selinux.c index 975b315..6680e2d 100644 --- a/src/security/security_selinux.c +++ b/src/security/security_selinux.c @@ -632,7 +632,8 @@ SELinuxReleaseSecurityLabel(virDomainObjPtr vm) { const virSecurityLabelDefPtr secdef = &vm->def->seclabel; - if (secdef->type == VIR_DOMAIN_SECLABEL_STATIC) + if (secdef->type == VIR_DOMAIN_SECLABEL_STATIC || + secdef->label == NULL) return 0; context_t con = context_new(secdef->label); -- 1.6.6.1

2 2

[libvirt] [PATCH] dont't crash in virsh dominfo <domain>
by Guido Günther 22 Mar '10

22 Mar '10

Hi, virsh dominfo domain crashes with: #0 strlen () at ../sysdeps/i386/i486/strlen.S:69 #1 0x080891c9 in qemudNodeGetSecurityModel (conn=0x8133940, secmodel=0xb5676ede) at qemu/qemu_driver.c:4911 #2 0xb7eb5623 in virNodeGetSecurityModel (conn=0x8133940, secmodel=0x0) at libvirt.c:5118 #3 0x0806767a in remoteDispatchNodeGetSecurityModel (server=0x8118888, client=0x8134080, conn=0x8133940, hdr=0x81a8388, rerr=0xb56771d8, args=0xb56771a0, ret=0xb5677144) at remote.c:1306 #4 0x08068acc in remoteDispatchClientCall (server=0x8118888, client=0x8134080, msg=0x8168378) at dispatch.c:506 #5 0x08068ee3 in remoteDispatchClientRequest (server=0x8118888, client=0x8134080, msg=0x8168378) at dispatch.c:388 #6 0x0805baba in qemudWorker (data=0x811de2c) at libvirtd.c:1528 #7 0xb7bb8585 in start_thread (arg=0xb5677b70) at pthread_create.c:300 #8 0xb7b3a29e in clone () at ../sysdeps/unix/sysv/linux/i386/clone.S:130 if there's no primary security driver set since we only intialize the secmodel.model and secmodel.doi if we have one. Attached patch checks for primarySecurityDriver instead of securityDriver since the later is always set in qemudSecurityInit(). Cheers, -- Guido

3 3

[libvirt] [PATCH 1/14] Adding recursive locks
by Stefan Berger 22 Mar '10

22 Mar '10

This patch adds recursive locks necessary due to the processing of network filter XML that can reference other network filters, including references that cause looks. Loops in the XML are prevented but their detection requires recursive locks.

5 9

[libvirt] Bug 526913 - libvirt does not lock disk files
by Ric Foster 22 Mar '10

22 Mar '10

https://bugzilla.redhat.com/show_bug.cgi?id=526913 Will this bug get fixed? I had hoped the using CLVM would keep the guest disk image from getting corrupted if a running guest was inadvertently started on another cluster node.

3 2

[libvirt] [PATCH v2 0/5] Introduce virDomainMigrateSetMaxDowntime API
by Jiri Denemark 22 Mar '10

22 Mar '10

This API call sets maximum tolerable time for which the domain is allowed to be paused at the end of live migration. It's supposed to be called while the domain is being live-migrated as a reaction to migration progress. Changes in version 2: - API renamed to reflect it's maximum downtime - new flags parameter for the future - qemu implementation was fixed so that the call is allowed iff the domain is being migrated - --downtime parameter of virsh migrate command was removed in favor of new virsh migrate-setmaxdowntime which can be run independently - virsh accepts downtime as nanoseconds instead of floating-point seconds (shells don't deal with floating-point numbers well) Jiri Denemark (5): Public virDomainMigrateSetMaxDowntime API Wire protocol and dispatcher for virDomainMigrateSetMaxDowntime Implement virDomainMigrateSetMaxDowntime in remote driver Implement virDomainMigrateSetMaxDowntime in qemu driver Add migrate-setmaxdowntime command to virsh daemon/remote.c | 29 ++++++++++ daemon/remote_dispatch_args.h | 1 + daemon/remote_dispatch_prototypes.h | 8 +++ daemon/remote_dispatch_table.h | 5 ++ include/libvirt/libvirt.h.in | 4 ++ src/libvirt.c | 49 +++++++++++++++++ src/libvirt_public.syms | 5 ++ src/qemu/qemu_driver.c | 70 ++++++++++++++++++++++++- src/qemu/qemu_monitor.c | 15 +++++ src/qemu/qemu_monitor.h | 3 + src/qemu/qemu_monitor_json.c | 29 ++++++++++ src/qemu/qemu_monitor_json.h | 3 + src/qemu/qemu_monitor_text.c | 27 ++++++++++ src/qemu/qemu_monitor_text.h | 3 + src/remote/remote_driver.c | 32 +++++++++++- src/remote/remote_protocol.c | 13 +++++ src/remote/remote_protocol.h | 98 +++++++++++++++++++---------------- src/remote/remote_protocol.x | 10 +++- tools/virsh.c | 66 +++++++++++++++++++++++ tools/virsh.pod | 6 ++ 20 files changed, 429 insertions(+), 47 deletions(-)

4 23

[libvirt] python binding: openAuth() segfaulting
by Dan Kenigsberg 21 Mar '10

21 Mar '10

I was trying to learn how to use openAuth(uri, auth_cb, flags), and got: Python 2.6.2 (r262:71600, Jan 25 2010, 18:46:47) [GCC 4.4.2 20091222 (Red Hat 4.4.2-20)] on linux2 Type "help", "copyright", "credits" or "license" for more information. >>> import libvirt >>> libvirt.openAuth('qemu:///system', lambda x: None, 0) Segmentation fault (core dumped) I guess my lambda expression was not what libvirt expected, but still.. Regards, Dan.

2 2

[libvirt] [PATCH Java] Cleanup close/free handling
by Matthias Bolte 21 Mar '10

21 Mar '10

The Connect class itself can have one ref to the internal virConnectPtr. Therefore, it has to release the VCP in its close method always. Otherwise multiple calls may release more then one ref, resulting in a ref-underflow later on. Make StoragePool's and StorageVol's free return int, like all other free methods, to return the number of remaining refs. Update Javadoc for all free methods to indicate that the return value is the number of remaining refs and that a value >= is returned on success. --- src/main/java/org/libvirt/Connect.java | 12 ++++-------- src/main/java/org/libvirt/Device.java | 2 +- src/main/java/org/libvirt/Domain.java | 2 +- src/main/java/org/libvirt/Interface.java | 2 +- src/main/java/org/libvirt/Network.java | 1 + src/main/java/org/libvirt/StoragePool.java | 14 +++++++++++--- src/main/java/org/libvirt/StorageVol.java | 12 +++++++++--- 7 files changed, 28 insertions(+), 17 deletions(-) diff --git a/src/main/java/org/libvirt/Connect.java b/src/main/java/org/libvirt/Connect.java index 3c6663e..8cec22e 100644 --- a/src/main/java/org/libvirt/Connect.java +++ b/src/main/java/org/libvirt/Connect.java @@ -219,7 +219,7 @@ public class Connect { * the object after close() will result in an exception. * * @throws LibvirtException - * @returns 0 for success, -1 for failure + * @return number of references left (>= 0) for success, -1 for failure. */ public int close() throws LibvirtException { int success = 0; @@ -227,14 +227,10 @@ public class Connect { success = libvirt.virConnectClose(VCP); processError(); // If leave an invalid pointer dangling around JVM crashes and burns - // if - // someone tries to call a method on us + // if someone tries to call a method on us // We rely on the underlying libvirt error handling to detect that - // it's - // called with a null virConnectPointer - if (success == 0) { - VCP = null; - } + // it's called with a null virConnectPointer + VCP = null; } return success; } diff --git a/src/main/java/org/libvirt/Device.java b/src/main/java/org/libvirt/Device.java index 26d8cbf..b4f9a0c 100644 --- a/src/main/java/org/libvirt/Device.java +++ b/src/main/java/org/libvirt/Device.java @@ -137,7 +137,7 @@ public class Device { * structure is freed and should not be used thereafter. * * @throws LibvirtException - * @returns 0 for success, -1 for failure. + * @return number of references left (>= 0) for success, -1 for failure. */ public int free() throws LibvirtException { int success = 0; diff --git a/src/main/java/org/libvirt/Domain.java b/src/main/java/org/libvirt/Domain.java index 8df928f..e7b24ef 100644 --- a/src/main/java/org/libvirt/Domain.java +++ b/src/main/java/org/libvirt/Domain.java @@ -167,7 +167,7 @@ public class Domain { * structure is freed and should not be used thereafter. * * @throws LibvirtException - * @returns 0 for success, -1 for failure. + * @return number of references left (>= 0) for success, -1 for failure. */ public int free() throws LibvirtException { int success = 0; diff --git a/src/main/java/org/libvirt/Interface.java b/src/main/java/org/libvirt/Interface.java index 0955cf9..44bcad5 100644 --- a/src/main/java/org/libvirt/Interface.java +++ b/src/main/java/org/libvirt/Interface.java @@ -117,7 +117,7 @@ public class Interface { * structure is freed and should not be used thereafter. * * @throws LibvirtException - * @returns 0 for success, -1 for failure. + * @return number of references left (>= 0) for success, -1 for failure. */ public int free() throws LibvirtException { int success = 0; diff --git a/src/main/java/org/libvirt/Network.java b/src/main/java/org/libvirt/Network.java index 14193fa..3c6f313 100644 --- a/src/main/java/org/libvirt/Network.java +++ b/src/main/java/org/libvirt/Network.java @@ -74,6 +74,7 @@ public class Network { * return an error. * * @throws LibvirtException + * @return number of references left (>= 0) for success, -1 for failure. */ public int free() throws LibvirtException { int success = 0; diff --git a/src/main/java/org/libvirt/StoragePool.java b/src/main/java/org/libvirt/StoragePool.java index 3256601..7de7d50 100644 --- a/src/main/java/org/libvirt/StoragePool.java +++ b/src/main/java/org/libvirt/StoragePool.java @@ -118,10 +118,18 @@ public class StoragePool { /** * Free a storage pool object, releasing all memory associated with it. Does * not change the state of the pool on the host. + * + * @throws LibvirtException + * @return number of references left (>= 0) for success, -1 for failure. */ - public void free() throws LibvirtException { - libvirt.virStoragePoolFree(VSPP); - processError(); + public int free() throws LibvirtException { + int success = 0; + if (VSPP != null) { + success = libvirt.virStoragePoolFree(VSPP); + processError(); + VSPP = null; + } + return success; } /** diff --git a/src/main/java/org/libvirt/StorageVol.java b/src/main/java/org/libvirt/StorageVol.java index 9213813..5ae5f5c 100644 --- a/src/main/java/org/libvirt/StorageVol.java +++ b/src/main/java/org/libvirt/StorageVol.java @@ -80,10 +80,16 @@ public class StorageVol { * to exist * * @throws LibvirtException + * @return number of references left (>= 0) for success, -1 for failure. */ - public void free() throws LibvirtException { - libvirt.virStorageVolFree(VSVP); - processError(); + public int free() throws LibvirtException { + int success = 0; + if (VSVP != null) { + libvirt.virStorageVolFree(VSVP); + processError(); + VSVP = null; + } + return success; } /** -- 1.6.3.3

1 0

[libvirt] [PATCH 0/9] Rebased volume wiping API patches
by David Allan 20 Mar '10

20 Mar '10

Nobody ack'd (because of general lack of time, I hope) what I think is the final version of the volume wiping API patches, so here is the set again, rebased to the current head. I don't think there is anything controversial in here, as they incorporate all the feedback on previous versions. Dave David Allan (9): Add public API for volume wiping Define the internal driver API for vol wiping Add vol wiping to ESX storage driver struct Implement the public API for vol wiping Define wire protocol format for vol wiping Implement RPC client for vol wiping Implement the remote dispatch bits of vol wiping Simplified version of volume wiping based on feedback from the list. Virsh support for vol wiping daemon/remote.c | 32 +++++ daemon/remote_dispatch_args.h | 1 + daemon/remote_dispatch_prototypes.h | 8 ++ daemon/remote_dispatch_table.h | 5 + include/libvirt/libvirt.h.in | 2 + src/driver.h | 5 + src/esx/esx_storage_driver.c | 1 + src/libvirt.c | 47 ++++++++ src/libvirt_public.syms | 5 + src/remote/remote_driver.c | 27 ++++ src/remote/remote_protocol.c | 11 ++ src/remote/remote_protocol.h | 9 ++ src/remote/remote_protocol.x | 8 +- src/storage/storage_driver.c | 224 +++++++++++++++++++++++++++++++++++ tools/virsh.c | 42 +++++++ 15 files changed, 426 insertions(+), 1 deletions(-)

6 25

[libvirt] libvirt 0.7.7 and kvm 72
by Guido Günther 19 Mar '10

19 Mar '10

Hi, kvm 72 uses "info migration" instead of "info migrate": http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=574272 It's not only that, the output format of these commands differs too so that "virsh save" breaks. Stefan suggested the attached improved error message. Cheers, -- Guido

1 1