[libvirt] [PATCH] esx: Fix freeing of heterogeneous lists

2010-07-29 Thread Matthias Bolte
Always call the free function of the base type. The base type function then dynamically dispatches the call to the free function for the actual type. --- src/esx/esx_vi_generator.py | 14 ++ 1 files changed, 10 insertions(+), 4 deletions(-) diff --git a/src/esx/esx_vi_generator.py

[libvirt] [PATCH] Eliminate memory leak in xenUnifiedDomainInfoListFree

2010-07-29 Thread Laine Stump
From: Laine Stump la...@redhat.com This fixes a leak described in https://bugzilla.redhat.com/show_bug.cgi?id=590073 xenUnifiedDomainInfoList has a pointer to a list of pointers to xenUnifiedDomain. We were freeing up all the domains, but neglecting to free the list. This was found by Paolo

[libvirt] [PATCHv2] Add iptables rule to fixup DHCP response checksum.

2010-07-29 Thread Laine Stump
(The only change to this version of the patch is to make the warning message more verbose when we fail to add the CHECKSUM rule, noting that the failure would likely be solved by updating iptables and the kernel.) This patch attempts to take advantage of a newly added netfilter module to correct

[libvirt] [PATCH] Fix --with-xen-proxy related compile error

2010-07-29 Thread Matthias Bolte
Move virDomainChrTargetTypeToString out of the #ifndef PROXY block, because it's used outside of it. --- src/conf/domain_conf.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index

Re: [libvirt] [PATCH] Eliminate memory leak in xenUnifiedDomainInfoListFree

2010-07-29 Thread Eric Blake
On 07/29/2010 07:47 AM, Laine Stump wrote: From: Laine Stump la...@redhat.com This fixes a leak described in https://bugzilla.redhat.com/show_bug.cgi?id=590073 xenUnifiedDomainInfoList has a pointer to a list of pointers to xenUnifiedDomain. We were freeing up all the domains, but

Re: [libvirt] [PATCH] Fix --with-xen-proxy related compile error

2010-07-29 Thread Eric Blake
On 07/29/2010 08:07 AM, Matthias Bolte wrote: Move virDomainChrTargetTypeToString out of the #ifndef PROXY block, because it's used outside of it. --- src/conf/domain_conf.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) ACK. -- Eric

Re: [libvirt] [PATCH] Eliminate memory leak in xenUnifiedDomainInfoListFree

2010-07-29 Thread Chris Lalancette
On 07/29/10 - 09:47:53AM, Laine Stump wrote: From: Laine Stump la...@redhat.com This fixes a leak described in https://bugzilla.redhat.com/show_bug.cgi?id=590073 xenUnifiedDomainInfoList has a pointer to a list of pointers to xenUnifiedDomain. We were freeing up all the domains, but

Re: [libvirt] [PATCH] Fix --with-xen-proxy related compile error

2010-07-29 Thread Chris Lalancette
On 07/29/10 - 04:07:08PM, Matthias Bolte wrote: Move virDomainChrTargetTypeToString out of the #ifndef PROXY block, because it's used outside of it. --- src/conf/domain_conf.c | 40 1 files changed, 20 insertions(+), 20 deletions(-) diff --git

Re: [libvirt] [PATCH] esx: Fix freeing of heterogeneous lists

2010-07-29 Thread Eric Blake
On 07/29/2010 04:29 AM, Matthias Bolte wrote: Always call the free function of the base type. The base type function then dynamically dispatches the call to the free function for the actual type. --- src/esx/esx_vi_generator.py | 14 ++ 1 files changed, 10 insertions(+), 4

[libvirt] [PATCH] lxc: Fix 'autostart' doesn't take effect actually

2010-07-29 Thread Ryota Ozaki
lxcStartup forgot to call lxcAutostartConfigs. Fix it. This patch should fix https://bugzilla.redhat.com/show_bug.cgi?id=589863 . --- src/lxc/lxc_driver.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/lxc/lxc_driver.c b/src/lxc/lxc_driver.c index 4fc1ecd..b286692

Re: [libvirt] [PATCH] Fix a potential race in pciInitDevice.

2010-07-29 Thread Chris Lalancette
On 07/28/10 - 02:39:38PM, Chris Wright wrote: * Chris Lalancette (clala...@redhat.com) wrote: If detecting the FLR flag of a pci device fails, then we could run into the situation of trying to close a file descriptor twice, once in pciInitDevice() and once in pciFreeDevice(). Fix that by

Re: [libvirt] Schedule for release of libvirt-0.8.3

2010-07-29 Thread Matthias Bolte
2010/7/21 Daniel Veillard veill...@redhat.com:  Small reminder, we will try to get back to a monthly release schedule so I'm targetting an end of month release for 0.8.3 . Assuming Friday 30 this would mean we enter the feature freeze at the end of the week. I'm gonna go over the pending

Re: [libvirt] [PATCH] lxc: Fix 'autostart' doesn't take effect actually

2010-07-29 Thread Eric Blake
On 07/29/2010 08:21 AM, Ryota Ozaki wrote: lxcStartup forgot to call lxcAutostartConfigs. Fix it. This patch should fix https://bugzilla.redhat.com/show_bug.cgi?id=589863 . --- src/lxc/lxc_driver.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git

Re: [libvirt] [PATCH] Fix --with-xen-proxy related compile error

2010-07-29 Thread Matthias Bolte
2010/7/29 Eric Blake ebl...@redhat.com: On 07/29/2010 08:07 AM, Matthias Bolte wrote: Move virDomainChrTargetTypeToString out of the #ifndef PROXY block, because it's used outside of it. ---  src/conf/domain_conf.c |   40  1 files changed, 20

[libvirt] [PATCH] Do not use boot=on on IDE device

2010-07-29 Thread Daniel Veillard
the followup on the boot=on problem, basically it's not needed to specify it when booting out of IDE devices when using KVM (and with newer seabios version it's also not needed for virtio). That patch isn't critical, it's more a cleanup one, I would delay this after the 0.8.3 release in any

[libvirt] IO controller groups?

2010-07-29 Thread Jon Masters
Folks, Anyone got any information about support in libvirt for automatic handling of IO controller throttling/IO limits with recent kernels? I'd love it if libvirt just supported assigning IO bandwidth automatically. :) Jon. -- libvir-list mailing list libvir-list@redhat.com

[libvirt] Virt-install Error on Centos 5.4 64bit and kvm

2010-07-29 Thread Cristian Rojas
Hi everyone: using this command to install a kvm virtual machine: virt-install --connect qemu:///system \ --name p3k0401 \ --ram 2048 \ --file //dev/VolGroup01/p3k0401logvol \ --accelerate \ -s 10 \ --nographics \ --hvm \ --location='http://10.1.4.80' I get: Starting install... Retrieving

[libvirt] [PATCH revision RFC 0/2] lxc: Fix return values of veth.c functions

2010-07-29 Thread Laine Stump
The first of thse two patches is an uncorrupted version of the patch in the parent message. The second is a few suggested changes. Overall, it looks very good. If you're okay with these changes, and at least one other person ACKs the combination, I can squash the two patches together and push

[libvirt] [PATCH revision RFC 2/2] lxc: Fix return values of veth.c functions - suggested changes

2010-07-29 Thread Laine Stump
Some suggested changes to your latest patch (I did the review by applying your patch, then examining the functions that were touched, focusing just on setting of rc) Summary: 1) virAsprintf() will return the number of characters in the new string on success, not 0, so we need to only set rc

[libvirt] [PATCH revision RFC 1/2] lxc: Fix return values of veth.c functions

2010-07-29 Thread Laine Stump
From: Ryota Ozaki ozaki.ry...@gmail.com Previously, the functions in src/lxc/veth.c could sometimes return positive values on failure rather than -1. This made accurate error reporting difficult, and led to one failure to catch an error in a calling function. This patch makes all the functions

Re: [libvirt] [PATCH] Eliminate memory leak in xenUnifiedDomainInfoListFree

2010-07-29 Thread Laine Stump
On 07/29/2010 10:10 AM, Eric Blake wrote: On 07/29/2010 07:47 AM, Laine Stump wrote: From: Laine Stumpla...@redhat.com This fixes a leak described in https://bugzilla.redhat.com/show_bug.cgi?id=590073 xenUnifiedDomainInfoList has a pointer to a list of pointers to xenUnifiedDomain. We

Re: [libvirt] [PATCH 1/2] xen: fix logic bug

2010-07-29 Thread Daniel Veillard
On Wed, Jul 28, 2010 at 05:34:14PM -0600, Eric Blake wrote: The recent switch to enable -Wlogical-op paid off again. gcc 4.5.0 (rawhide) is smarter than 4.4.4 (Fedora 13). * src/xen/xend_internal.c (xenDaemonAttachDeviceFlags) (xenDaemonUpdateDeviceFlags, xenDaemonDetachDeviceFlags): Use

Re: [libvirt] [PATCH 2/2] esx: silence spurious compiler warning

2010-07-29 Thread Daniel Veillard
On Wed, Jul 28, 2010 at 05:34:15PM -0600, Eric Blake wrote: * src/esx/esx_vi_types.c (_DESERIALIZE_NUMBER) (ESX_VI__TEMPLATE__DESERIALIZE_NUMBER): Add range check to shut up gcc 4.5.0 regarding long long. --- src/esx/esx_vi_types.c |6 -- 1 files changed, 4 insertions(+), 2

Re: [libvirt] [PATCH revision RFC 1/2] lxc: Fix return values of veth.c functions

2010-07-29 Thread Daniel Veillard
On Thu, Jul 29, 2010 at 12:18:23PM -0400, Laine Stump wrote: From: Ryota Ozaki ozaki.ry...@gmail.com Previously, the functions in src/lxc/veth.c could sometimes return positive values on failure rather than -1. This made accurate error reporting difficult, and led to one failure to catch an

Re: [libvirt] [PATCH revision RFC 2/2] lxc: Fix return values of veth.c functions - suggested changes

2010-07-29 Thread Daniel Veillard
On Thu, Jul 29, 2010 at 12:18:24PM -0400, Laine Stump wrote: Some suggested changes to your latest patch (I did the review by applying your patch, then examining the functions that were touched, focusing just on setting of rc) Summary: 1) virAsprintf() will return the number of characters

[libvirt] [PATCHv3] lxc: Fix return values of veth.c functions - complete

2010-07-29 Thread Laine Stump
From: Ryota Ozaki ozaki.ry...@gmail.com From: Ryota Ozaki ozaki.ry...@gmail.com Previously, the functions in src/lxc/veth.c could sometimes return positive values on failure rather than -1. This made accurate error reporting difficult, and led to one failure to catch an error in a calling

Re: [libvirt] [PATCH] esx: Improve blocked task detection and fix race condition

2010-07-29 Thread Daniel Veillard
On Mon, Jul 26, 2010 at 03:12:25AM +0200, Matthias Bolte wrote: esxVI_WaitForTaskCompletion can take a UUID to lookup the corresponding domain and check if the current task for it is blocked by a question. It calls another function to do this: esxVI_LookupAndHandleVirtualMachineQuestion looks

Re: [libvirt] ruby-libvirt issue

2010-07-29 Thread Jaromír Červenka
Hi, i think that I know where is the problem. My messages log says: Jul 29 19:36:41 divinus libvirtd: 19:36:41.032: error : qemudDispatchServer:1315 : Too many active clients (100), dropping connection It looks like that ruby-libvirt doesn't closing connection, when it's running under

Re: [libvirt] [PATCHv3] lxc: Fix return values of veth.c functions - complete

2010-07-29 Thread Ryota Ozaki
On Fri, Jul 30, 2010 at 2:32 AM, Laine Stump la...@laine.org wrote: From: Ryota Ozaki ozaki.ry...@gmail.com From: Ryota Ozaki ozaki.ry...@gmail.com Previously, the functions in src/lxc/veth.c could sometimes return positive values on failure rather than -1. This made accurate error

Re: [libvirt] [PATCHv3] lxc: Fix return values of veth.c functions - complete

2010-07-29 Thread Laine Stump
On 07/29/2010 01:53 PM, Ryota Ozaki wrote: On Fri, Jul 30, 2010 at 2:32 AM, Laine Stumpla...@laine.org wrote: From: Ryota Ozakiozaki.ry...@gmail.com Previously, the functions in src/lxc/veth.c could sometimes return positive values on failure rather than -1. This made accurate error