[libvirt] [PATCH] AppArmorSetSecurityImageLabel: Avoid use of uninitialized variable

2013-09-04 Thread Michal Privoznik
The @profile_name variable can be used uninitialized. Signed-off-by: Michal Privoznik --- Pushing under trivial and build breaker rules. src/security/security_apparmor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/security/security_apparmor.c b/src/security/security

[libvirt] [PATCH] virFileNBDDeviceAssociate: Avoid use of uninitialized variable

2013-09-04 Thread Michal Privoznik
The @qemunbd variable can be used uninitialized. Signed-off-by: Michal Privoznik --- Pushing under trivial and build breaker rules. src/util/virfile.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/virfile.c b/src/util/virfile.c index 2b07ac9..7af0843 100644 --- a

[libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread arun abhinay
Hi, In reference to comment28 in below link https://bugzilla.redhat.com/show_bug.cgi?id=911609#c28 The issue was fixed in libvirt-0.10.2-19.el6 version. I am facing similar crash and want to check with libvirt-0.10.2-19.el6 version. But i couldnt find this version from opensource. We could onl

[libvirt] [PATCH] [RFC] docs: Add OpenStack into references

2013-09-04 Thread Martin Kletzander
diff --git a/docs/apps.html.in b/docs/apps.html.in index 46e5dca..9f22df8 100644 --- a/docs/apps.html.in +++ b/docs/apps.html.in @@ -244,6 +244,15 @@ integrates libvirt for VM monitoring, live migration, and life-cycle management. + + http://www.openstack.org";>OpenSt

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread Michal Privoznik
On 04.09.2013 09:24, arun abhinay wrote: > Hi, > > In reference to comment28 in below link > https://bugzilla.redhat.com/show_bug.cgi?id=911609#c28 > > The issue was fixed in libvirt-0.10.2-19.el6 version. I am facing > similar crash and want to check with libvirt-0.10.2-19.el6 version. But

Re: [libvirt] [PATCH] conf: Remove the actual hostdev when removing a network

2013-09-04 Thread Peter Krempa
On 09/03/13 19:14, Eric Blake wrote: > On 09/03/2013 09:29 AM, Peter Krempa wrote: >> Commit 50348e6edfa reused the code to remove the hostdev portion of a >> network definition on multiple places but forgot to take into account >> that sometimes the "actual" network is passed and in some cases the

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread Jiri Denemark
On Wed, Sep 04, 2013 at 09:24:43 +0200, arun abhinay wrote: > Hi, > > In reference to comment28 in below link > https://bugzilla.redhat.com/show_bug.cgi?id=911609#c28 > > The issue was fixed in libvirt-0.10.2-19.el6 version. I am facing similar > crash and want to check with libvirt-0.10.2-19.el

Re: [libvirt] [PATCH] qemu: Make domain renaming work during migration

2013-09-04 Thread Jiri Denemark
On Tue, Sep 03, 2013 at 09:09:54 -0600, Eric Blake wrote: > On 09/03/2013 07:46 AM, Jiri Denemark wrote: > > https://bugzilla.redhat.com/show_bug.cgi?id=999352 > > > > Since commit v1.0.5-56-g449e6b1 (Pull parsing of migration xml up into > > QEMU driver APIs) any attempt to rename a domain during

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread arun abhinay
Hi Michal, Thanks for your reply. Is it possible to get the patch between the two versions. Thanks in advance. BR Abhinay On Wed, Sep 4, 2013 at 9:34 AM, Michal Privoznik wrote: > On 04.09.2013 09:24, arun abhinay wrote: > > Hi, > > > > In reference to comment28 in below link > > https://bugzil

Re: [libvirt] [PATCH] libxl: Check for regcomp failure

2013-09-04 Thread Michal Privoznik
On 04.09.2013 08:37, Jim Fehlig wrote: > Change libxlGetAutoballoonConf() function to return an int > for success/failure, and fail if regcomp fails. > > Signed-off-by: Jim Fehlig > --- > src/libxl/libxl_conf.c | 30 +++--- > 1 file changed, 19 insertions(+), 11 deletions

Re: [libvirt] [PATCH] libxl: Check for regcomp failure

2013-09-04 Thread Michal Privoznik
On 04.09.2013 10:29, Michal Privoznik wrote: > On 04.09.2013 08:37, Jim Fehlig wrote: >> Change libxlGetAutoballoonConf() function to return an int >> for success/failure, and fail if regcomp fails. >> >> Signed-off-by: Jim Fehlig >> --- >> src/libxl/libxl_conf.c | 30 +++-

Re: [libvirt] [PATCH] build: enforce makefile conditional style

2013-09-04 Thread Michal Privoznik
On 04.09.2013 04:41, Eric Blake wrote: > Automake has builtin support to prevent botched conditional nesting, > but only if you use: > if FOO > else !FOO > endif !FOO > > An example error message when using the wrong name: > > daemon/Makefile.am:378: error: else reminder > (LIBVIRT_INIT_SCRIPT_S

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread arun abhinay
Hi Jirka, Thanks for your detailed information. As part of fixing this issue we have upgraded to libvirt-1.0.5 version and these crashes were resolved. But after this upgrade we observed many other issues like "cpu-stats" command is not working and vcpu pinning was failing with errors. Since we

Re: [libvirt] [PATCH 03/12] libxl: Earlier detection of not running on Xen

2013-09-04 Thread Daniel P. Berrange
On Tue, Sep 03, 2013 at 05:07:25PM -0600, Jim Fehlig wrote: > Daniel P. Berrange wrote: > > On Fri, Aug 30, 2013 at 03:46:49PM -0600, Jim Fehlig wrote: > > > >> Detect early on in libxl driver initialization if the driver > >> should be loaded at all, avoiding needless initialization steps > >>

Re: [libvirt] [PATCH] Fix: Syle issues (daemon/remote.c)

2013-09-04 Thread Osier Yang
On 04/09/13 04:36, Nehal J Wani wrote: s/$subject/Fix coding style issues in daemon/remote.c/, Fix for argument layouts of various functions in daemon/remote.c. Refer: https://www.redhat.com/archives/libvir-list/2013-September/msg00057.html No need for the reference, it's obvious. ACK and pu

Re: [libvirt] [PATCH] virFileNBDDeviceAssociate: Avoid use of uninitialized variable

2013-09-04 Thread Daniel P. Berrange
On Wed, Sep 04, 2013 at 08:59:42AM +0200, Michal Privoznik wrote: > The @qemunbd variable can be used uninitialized. > > Signed-off-by: Michal Privoznik > --- > > Pushing under trivial and build breaker rules. FYI this bug is considered a security issue if the polkit ACL driver is active and th

Re: [libvirt] [PATCH 00/12] Various libxl driver improvements

2013-09-04 Thread John Ferlan
On 09/03/2013 07:57 PM, Jim Fehlig wrote: > Jim Fehlig wrote: <...snip...> > > I addressed the review comments from the various patches and pushed the > series. Thanks for the reviews! > > Regards, > Jim > The changes have resulted in one Coverity found issue that seems to have been there "for

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread Laine Stump
On 09/04/2013 05:06 AM, arun abhinay wrote: > > Thanks for your detailed information. As part of fixing this issue we > have upgraded to libvirt-1.0.5 version and these crashes were > resolved. But after this upgrade we observed many other issues like > "cpu-stats" command is not working and vcp

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread arun abhinay
Hi Laine, Thanks a lot for your inputs. We will try to download the code from libvirt-0.10.2-main t and build libvirt with the patch for crash issue. We are using WR kenerl of below version 2.6.34.13-WR4 Thanks Abhinay On Wed, Sep 4, 2013 at 1:36 PM, Laine Stump wrote: > On 09/04/2013 05:06 A

Re: [libvirt] [PATCH 4/3] qemu: don't leave vm in table on failure to attach

2013-09-04 Thread Michal Privoznik
On 28.08.2013 22:56, Eric Blake wrote: > Failure to attach to a domain during 'virsh qemu-attach' left > the list of domains in an odd state: > > $ virsh qemu-attach 4176 > error: An error occurred, but the cause is unknown > > $ virsh list --all > IdName State > --

Re: [libvirt] [PATCH 3/3] qemu: recognize -machine accel=kvm when parsing native

2013-09-04 Thread Michal Privoznik
On 28.08.2013 06:00, Eric Blake wrote: > In Fedora 19, 'qemu-kvm' is a simple wrapper that calls > 'qemu-system-x86_64 -machine accel=kvm'. Attempting > to use 'virsh qemu-attach $pid' to a machine started as: > > qemu-kvm -cdrom /var/lib/libvirt/images/foo.img \ > -monitor unix:/tmp/demo,server

Re: [libvirt] [PATCH 5/3] qemu: don't leave shutdown inhibited on attach failure

2013-09-04 Thread Michal Privoznik
On 29.08.2013 00:38, Eric Blake wrote: > While debugging a failure of 'virsh qemu-attach', I noticed that > we were leaking the count of active domains on failure. This > means that a libvirtd session that is supposed to quit after > active domains disappear will hang around forever. > > * src/qe

Re: [libvirt] [PATCHv2 2/3] qemu: simplify list cleanup

2013-09-04 Thread Michal Privoznik
On 28.08.2013 23:01, Eric Blake wrote: > No need to open code now that we have a nice function. > > Interestingly, our virStringFreeList function is typed correctly > (a malloc'd list of malloc'd strings is NOT const, whether at the > point where it is created, or at the point where it is cleand u

Re: [libvirt] [PATCH 1/3] qemu: only parse basename when determining emulator properties

2013-09-04 Thread Michal Privoznik
On 28.08.2013 06:00, Eric Blake wrote: > 'virsh domxml-from-native' and 'virsh qemu-attach' could misbehave > for an emulator installed in (a somewhat unlikely) location > such as /usr/local/qemu-1.6/qemu-system-x86_64 or (an even less > likely) /opt/notxen/qemu-system-x86_64. Limit the strstr sea

Re: [libvirt] [PATCH 2/n] build: only run fdstreamtest when libvirtd is built

2013-09-04 Thread Michal Privoznik
On 04.09.2013 00:48, Eric Blake wrote: > An rpm build with client_only set to 1 (for example, RHEL 5 on > s390, or by modifying libvirt.spec.in) failed with > > TEST: fdstreamtest > 1) Stream read blocking ... OK > 2) Stream read non-blocking

Re: [libvirt] [PATCH 1/n] spec: fix rpm build when lxc disabled

2013-09-04 Thread Michal Privoznik
On 03.09.2013 23:27, Eric Blake wrote: > 'make rpm' failed if ~/.rpmmacros contains '%_without_lxc 1', > which simulates the case of not having lxc available. > > RPM build errors: > File not found: > /home/eblake/rpmbuild/BUILDROOT/libvirt-1.1.1-1.fc19.x86_64/etc/libvirt/virt-login-shell.con

Re: [libvirt] link fow downloading libvirt-0.10.2-19.el6

2013-09-04 Thread Eric Blake
On 09/04/2013 05:59 AM, arun abhinay wrote: > Hi Laine, [Please don't top-post on technical lists] > > Thanks a lot for your inputs. We will try to download the code from > libvirt-0.10.2-main t and build libvirt with the patch for crash issue. > > We are using WR kenerl of below version > 2.6.

Re: [libvirt] [PATCH] [RFC] docs: Add OpenStack into references

2013-09-04 Thread Eric Blake
On 09/04/2013 01:26 AM, Martin Kletzander wrote: > diff --git a/docs/apps.html.in b/docs/apps.html.in > index 46e5dca..9f22df8 100644 > --- a/docs/apps.html.in > +++ b/docs/apps.html.in > @@ -244,6 +244,15 @@ > integrates libvirt for VM monitoring, live migration, and life-cycle >

Re: [libvirt] [PATCH 4/n] spec: default vbox according to libvirtd build

2013-09-04 Thread Michal Privoznik
On 04.09.2013 04:56, Eric Blake wrote: > Commit ba5f3c7 moved virtualBox support into libvirtd, but the spec > file was still unconditionally requesting it even when not building > the server side. This leads to the weird result of building > libvirt-client and libvirt-daemon-lxc rpms when client_

Re: [libvirt] [PATCH 5/n RFC] build: avoid stranded file when polkit is not used

2013-09-04 Thread Michal Privoznik
On 04.09.2013 05:07, Eric Blake wrote: > Untested for now (it's late for me), but seems right... > > daemon/Makefile.am installs a .policy file if WITH_POLKIT is set > (which is supposedly the union between WITH_POLKIT0 and WITH_POLKIT1). > src/Makefile.am, on the other hand, installs a .policy fi

Re: [libvirt] [PATCH 3a/n] build: only install nwfilter examples when building nwfilter

2013-09-04 Thread Michal Privoznik
On 04.09.2013 04:52, Eric Blake wrote: > 'make rpm' with client_local set to 1 (by manual modification, > or with RHEL 5 on s390) warns: > > RPM build errors: > Installed (but unpackaged) file(s) found: >/etc/libvirt/nwfilter/allow-arp.xml >/etc/libvirt/nwfilter/allow-dhcp-server.xml >

Re: [libvirt] [PATCHv5 1/5] domifaddr: Implement the public APIs

2013-09-04 Thread Osier Yang
On 04/09/13 03:07, Eric Blake wrote: On 09/01/2013 07:43 AM, Nehal J Wani wrote: Define a new API virDomainInterfaceAddresses, which returns the address information of a running domain's interfaces(s). If no interface name is specified, it returns the information of all interfaces, otherwise it

[libvirt] [PATCH] spec: Only distribute ChangeLog in -devel package

2013-09-04 Thread Cole Robinson
Helps save space on the Fedora livecd, as requested here: https://bugzilla.redhat.com/show_bug.cgi?id=977099 --- libvirt.spec.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libvirt.spec.in b/libvirt.spec.in index e94901a..a327eab 100644 --- a/libvirt.spec.in +++ b/libvir

Re: [libvirt] [PATCH] [RFC] docs: Add OpenStack into references

2013-09-04 Thread Osier Yang
On 04/09/13 20:50, Eric Blake wrote: On 09/04/2013 01:26 AM, Martin Kletzander wrote: diff --git a/docs/apps.html.in b/docs/apps.html.in index 46e5dca..9f22df8 100644 --- a/docs/apps.html.in +++ b/docs/apps.html.in @@ -244,6 +244,15 @@ integrates libvirt for VM monitoring, live migrati

[libvirt] [ott...@fzi.de: [Qemu-devel] [PATCH RFC] Do not set SO_REUSEADDR on Windows]

2013-09-04 Thread Daniel P. Berrange
- Forwarded message from Sebastian Ottlik - > > This patchset disabels all use of SO_REUSEADDR on Windows. On Windows systems > the default behavior is equivalent to SO_REUSEADDR on other operating > systems. SO_REUSEADDR can still be set but results in undesired bahvior > instead. It may

Re: [libvirt] [PATCH] spec: Only distribute ChangeLog in -devel package

2013-09-04 Thread Eric Blake
On 09/04/2013 08:53 AM, Daniel P. Berrange wrote: > On Wed, Sep 04, 2013 at 10:24:06AM -0400, Cole Robinson wrote: >> Helps save space on the Fedora livecd, as requested here: >> >> https://bugzilla.redhat.com/show_bug.cgi?id=977099 >> --- >> libvirt.spec.in | 3 +-- >> 1 file changed, 1 insertion

Re: [libvirt] [PATCH] conf: Remove the actual hostdev when removing a network

2013-09-04 Thread Cole Robinson
On 09/03/2013 11:29 AM, Peter Krempa wrote: > Commit 50348e6edfa reused the code to remove the hostdev portion of a > network definition on multiple places but forgot to take into account > that sometimes the "actual" network is passed and in some cases the > parent of that. > > This patch uses th

Re: [libvirt] [PATCH] spec: Only distribute ChangeLog in -devel package

2013-09-04 Thread Eric Blake
On 09/04/2013 09:11 AM, Daniel P. Berrange wrote: >> client: COPYING COPYING.LESSER >> daemon: none >> other[*]: AUTHORS ChangeLog.gz NEWS README TODO >> >> where I have yet another question: should the other package be >> libvirt-docs or libvirt-devel? Your patch put them in devel, but I >> thin

Re: [libvirt] [PATCHv5 1/5] domifaddr: Implement the public APIs

2013-09-04 Thread Osier Yang
On 04/09/13 03:13, Eric Blake wrote: On 09/03/2013 01:07 PM, Eric Blake wrote: On 09/01/2013 07:43 AM, Nehal J Wani wrote: Define a new API virDomainInterfaceAddresses, which returns the address information of a running domain's interfaces(s). If no interface name is specified, it returns the i

Re: [libvirt] [PATCH] spec: Only distribute ChangeLog in -devel package

2013-09-04 Thread Daniel P. Berrange
On Wed, Sep 04, 2013 at 10:24:06AM -0400, Cole Robinson wrote: > Helps save space on the Fedora livecd, as requested here: > > https://bugzilla.redhat.com/show_bug.cgi?id=977099 > --- > libvirt.spec.in | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/libvirt.spec.in b/l

Re: [libvirt] [PATCH] conf: Remove the actual hostdev when removing a network

2013-09-04 Thread Peter Krempa
On 09/04/13 17:17, Cole Robinson wrote: > On 09/03/2013 11:29 AM, Peter Krempa wrote: >> Commit 50348e6edfa reused the code to remove the hostdev portion of a >> network definition on multiple places but forgot to take into account >> that sometimes the "actual" network is passed and in some cases

Re: [libvirt] [PATCH] spec: Only distribute ChangeLog in -devel package

2013-09-04 Thread Daniel P. Berrange
On Wed, Sep 04, 2013 at 09:02:58AM -0600, Eric Blake wrote: > On 09/04/2013 08:53 AM, Daniel P. Berrange wrote: > > On Wed, Sep 04, 2013 at 10:24:06AM -0400, Cole Robinson wrote: > >> Helps save space on the Fedora livecd, as requested here: > >> > >> https://bugzilla.redhat.com/show_bug.cgi?id=977

[libvirt] [PATCH] conf: Don't deref NULL actual network in virDomainNetGetActualHostdev()

2013-09-04 Thread Peter Krempa
In commit 991270db99690 I've used virDomainNetGetActualHostdev() to get the actual hostdev from a network when removing the network from the list to avoid leaving the hostdev in the list. I didn't notice that this function doesn't check if the actual network is allocated and dereferences it. This c

Re: [libvirt] [PATCHv2 2/3] qemu: simplify list cleanup

2013-09-04 Thread Daniel P. Berrange
On Wed, Aug 28, 2013 at 03:01:23PM -0600, Eric Blake wrote: > No need to open code now that we have a nice function. > > Interestingly, our virStringFreeList function is typed correctly > (a malloc'd list of malloc'd strings is NOT const, whether at the > point where it is created, or at the point

Re: [libvirt] [PATCH] spec: Only distribute ChangeLog in -devel package

2013-09-04 Thread Eric Blake
On 09/04/2013 08:24 AM, Cole Robinson wrote: > Helps save space on the Fedora livecd, as requested here: > > https://bugzilla.redhat.com/show_bug.cgi?id=977099 > --- > libvirt.spec.in | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > ACK - makes more sense to have the files in only on

Re: [libvirt] [PATCH] build: enforce makefile conditional style

2013-09-04 Thread Eric Blake
On 09/04/2013 02:41 AM, Michal Privoznik wrote: > On 04.09.2013 04:41, Eric Blake wrote: >> Automake has builtin support to prevent botched conditional nesting, >> but only if you use: >> if FOO >> else !FOO >> endif !FOO >> >> An example error message when using the wrong name: >> >> daemon/Makefi

Re: [libvirt] [PATCH] conf: Don't deref NULL actual network in virDomainNetGetActualHostdev()

2013-09-04 Thread Eric Blake
On 09/04/2013 09:37 AM, Peter Krempa wrote: > In commit 991270db99690 I've used virDomainNetGetActualHostdev() to get > the actual hostdev from a network when removing the network from the > list to avoid leaving the hostdev in the list. I didn't notice that this > function doesn't check if the act

[libvirt] [PATCH] Ensure 'arch' is always set in cpuArchNodeData

2013-09-04 Thread Daniel P. Berrange
From: "Daniel P. Berrange" The s390, ppc and arm CPU drivers never set the 'arch' field in their impl of cpuArchNodeData. This leads to error messages being reported from cpuDataFree later, due to trying to use VIR_ARCH_NONE. #0 virRaiseErrorFull (filename=filename@entry=0x76f94434 "cpu/cpu.c"

[libvirt] [PATCH] virGet{User, Group}Ent() don't say success on fail

2013-09-04 Thread Doug Goldstein
When virGetUserEnt() and virGetGroupEnt() fail due to the uid or gid not existing on the machine they'll print a message like: $ virsh -c vbox:///session list error: failed to connect to the hypervisor error: Failed to find user record for uid '32655': Success The success at the end is a bit conf

Re: [libvirt] [PATCH] libxl: Check for regcomp failure

2013-09-04 Thread Jim Fehlig
Michal Privoznik wrote: > On 04.09.2013 10:29, Michal Privoznik wrote: > >> On 04.09.2013 08:37, Jim Fehlig wrote: >> >>> Change libxlGetAutoballoonConf() function to return an int >>> for success/failure, and fail if regcomp fails. >>> >>> Signed-off-by: Jim Fehlig >>> --- >>> src/libxl/

Re: [libvirt] [PATCH] virGet{User, Group}Ent() don't say success on fail

2013-09-04 Thread Eric Blake
On 09/04/2013 10:48 AM, Doug Goldstein wrote: > When virGetUserEnt() and virGetGroupEnt() fail due to the uid or gid not > existing on the machine they'll print a message like: > > $ virsh -c vbox:///session list > error: failed to connect to the hypervisor > error: Failed to find user record for

Re: [libvirt] [PATCH] Ensure 'arch' is always set in cpuArchNodeData

2013-09-04 Thread Eric Blake
On 09/04/2013 10:44 AM, Daniel P. Berrange wrote: > From: "Daniel P. Berrange" > > The s390, ppc and arm CPU drivers never set the 'arch' field > in their impl of cpuArchNodeData. This leads to error messages > being reported from cpuDataFree later, due to trying to use > VIR_ARCH_NONE. > > ++

Re: [libvirt] [PATCH 1/n] spec: fix rpm build when lxc disabled

2013-09-04 Thread Eric Blake
On 09/04/2013 06:38 AM, Michal Privoznik wrote: > On 03.09.2013 23:27, Eric Blake wrote: >> 'make rpm' failed if ~/.rpmmacros contains '%_without_lxc 1', >> which simulates the case of not having lxc available. >> >> RPM build errors: >> File not found: >> /home/eblake/rpmbuild/BUILDROOT/libvi

[libvirt] [PATCH] build: shut up automake warnings

2013-09-04 Thread Eric Blake
I'm tired of seeing screenfuls of messages like these when using automake 1.13 (Fedora 19): configure.ac:2121: warning: The 'AM_PROG_MKDIR_P' macro is deprecated, and its use is discouraged. configure.ac:2121: You should use the Autoconf-provided 'AC_PROG_MKDIR_P' macro instead, configure.ac:212

[libvirt] Create ESXi vmknic using libvirt xml

2013-09-04 Thread varun bhatnagar
Hi, I am using libvirt on ESX hypervisor. Iam trying to create "vmknic" using virsh but all my attempts are failing. Can anyone help me in achieving this? Where can I find the xml template for this? or is there any wiki link which I can follow to achieve this? Regards, Varun -- libvir-list mailin

Re: [libvirt] [PATCH 2/n] build: only run fdstreamtest when libvirtd is built

2013-09-04 Thread Eric Blake
On 09/04/2013 06:38 AM, Michal Privoznik wrote: > On 04.09.2013 00:48, Eric Blake wrote: >> An rpm build with client_only set to 1 (for example, RHEL 5 on >> s390, or by modifying libvirt.spec.in) failed with >> >> * tests/Makefile.am (test_programs): Make fdstreamtest conditional. >> >> Signed-of

Re: [libvirt] [PATCH 3a/n] build: only install nwfilter examples when building nwfilter

2013-09-04 Thread Eric Blake
On 09/04/2013 07:22 AM, Michal Privoznik wrote: > On 04.09.2013 04:52, Eric Blake wrote: >> 'make rpm' with client_local set to 1 (by manual modification, >> or with RHEL 5 on s390) warns: >> >> >> The bulk of these are fixed with this patch. >> >> * examples/xml/nwfilter/Makefile.am (install-data

Re: [libvirt] [PATCH] libxl: Check for regcomp failure

2013-09-04 Thread Eric Blake
On 09/04/2013 02:03 PM, Jim Fehlig wrote: >> >> Yeah, good question. I found a few occurrences of regcomp() and friends >> throughout the sources and most seem to do regfree() even when regcomp() >> fails. The man page is not very clear, but the notes on regfree() >> suggest it is not necessary >

Re: [libvirt] [PATCH 00/12] Various libxl driver improvements

2013-09-04 Thread Jim Fehlig
John Ferlan wrote: > On 09/03/2013 07:57 PM, Jim Fehlig wrote: > >> Jim Fehlig wrote: >> > <...snip...> > >> I addressed the review comments from the various patches and pushed the >> series. Thanks for the reviews! >> >> Regards, >> Jim >> >> > > The changes have resulted in one C

[libvirt] [PATCH] build: require libnl-3 if netcf uses it

2013-09-04 Thread Eric Blake
Commits 9298bfb and f6c2951 both tried to make it possible to select the correct libnl (1 vs. 3) according to what netcf used, when both libraries are installed. This works to avoid libnl-3 when netcf used libnl-1. But on the converse side, if only libnl-1 development code is installed, while net

Re: [libvirt] [PATCH 00/12] Various libxl driver improvements

2013-09-04 Thread Eric Blake
On 09/04/2013 03:09 PM, Jim Fehlig wrote: >> > > Thanks for pointing that out. What do you think of the attached patch? > > Regards, > Jim > > > libxl-coverity-fix.patch > > >>From fe87e804478689b7cdf0ab417b459cb0d2f6118e Mon Sep 17 00:00:00 2001 > From: Jim Fehlig > Date: Wed, 4 Sep 201

Re: [libvirt] [PATCH 00/12] Various libxl driver improvements

2013-09-04 Thread Jim Fehlig
Eric Blake wrote: > On 09/04/2013 03:09 PM, Jim Fehlig wrote: > > >> Thanks for pointing that out. What do you think of the attached patch? >> >> Regards, >> Jim >> >> >> libxl-coverity-fix.patch >> >> >> >From fe87e804478689b7cdf0ab417b459cb0d2f6118e Mon Sep 17 00:00:00 2

Re: [libvirt] [PATCH 2/3] virsh: Rename vshMakeStdinRaw to vshTTYMakeRaw and move it to virsh.c

2013-09-04 Thread Eric Blake
On 08/29/2013 10:36 AM, Peter Krempa wrote: > Move the function to virsh.c to the rest of the TTY managing functions > and change the code so that it mirrors the rest. > --- > tools/virsh-console.c | 50 +- > tools/virsh-console.h | 7 +++ > too

Re: [libvirt] [PATCH 5/n RFC] build: avoid stranded file when polkit is not used

2013-09-04 Thread Eric Blake
On 09/04/2013 07:22 AM, Michal Privoznik wrote: > On 04.09.2013 05:07, Eric Blake wrote: >> Untested for now (it's late for me), but seems right... >> >> daemon/Makefile.am installs a .policy file if WITH_POLKIT is set >> (which is supposedly the union between WITH_POLKIT0 and WITH_POLKIT1). >> src

Re: [libvirt] [PATCH] libxl: Check for regcomp failure

2013-09-04 Thread Jim Fehlig
Jim Fehlig wrote: > Michal Privoznik wrote: > >> On 04.09.2013 10:29, Michal Privoznik wrote: >> >> >>> On 04.09.2013 08:37, Jim Fehlig wrote: >>> >>> Change libxlGetAutoballoonConf() function to return an int for success/failure, and fail if regcomp fails. >>>

[libvirt] [PATCH 6/n] spec: make client-only testing easier

2013-09-04 Thread Eric Blake
Several recent patches cleaned up 'make rpm' for the situation when client_only is true; these were done by manual spec file editing (since it's relatively hard to come by a RHEL 5 s390 box). Make it easier to do in the future via a simpler command line override. * libvirt.spec.in (client_only):

Re: [libvirt] [PATCH 4/n] spec: default vbox according to libvirtd build

2013-09-04 Thread Eric Blake
On 09/04/2013 07:22 AM, Michal Privoznik wrote: > On 04.09.2013 04:56, Eric Blake wrote: >> Commit ba5f3c7 moved virtualBox support into libvirtd, but the spec >> file was still unconditionally requesting it even when not building >> the server side. This leads to the weird result of building >> l

[libvirt] [PATCH 0/2] Only free compiled regex when regcomp() succeeds

2013-09-04 Thread Jim Fehlig
During a recent discussion about calling regfree() when regcomp() fails, we decided it was best to not call regfree() when compilation fails. https://www.redhat.com/archives/libvir-list/2013-September/msg00262.html The first patch fixes existing infractions throughout the code. I split the secon

[libvirt] [PATCH 2/2] libxl: Compile regular expression where it is used

2013-09-04 Thread Jim Fehlig
The regular expression used to determine guest capabilities was compiled in libxlCapsInitHost() but used in libxlCapsInitGuests(). Move compilation to libxlCapsInitGuests() where it is used, and free the compiled regex after use. Ensure not to free the regex if compilation fails. Signed-off-by: J

[libvirt] [PATCH 1/2] Don't call regfree() if regcomp() fails

2013-09-04 Thread Jim Fehlig
POSIX states that the preg parameter to regcomp() is undefined on failure, so no need to call regfree() in these cases. http://pubs.opengroup.org/onlinepubs/009695399/functions/regcomp.html Signed-off-by: Jim Fehlig --- src/storage/storage_backend.c | 2 +- src/storage/storage_backend_l

Re: [libvirt] [PATCH 2/2] libxl: Compile regular expression where it is used

2013-09-04 Thread Jim Fehlig
Jim Fehlig wrote: > The regular expression used to determine guest capabilities > was compiled in libxlCapsInitHost() but used in libxlCapsInitGuests(). > Move compilation to libxlCapsInitGuests() where it is used, and free > the compiled regex after use. Ensure not to free the regex if > compilat

Re: [libvirt] [PATCH] libxl: Check for regcomp failure

2013-09-04 Thread Jim Fehlig
Eric Blake wrote: > On 09/04/2013 02:03 PM, Jim Fehlig wrote: > >>> Yeah, good question. I found a few occurrences of regcomp() and friends >>> throughout the sources and most seem to do regfree() even when regcomp() >>> fails. The man page is not very clear, but the notes on regfree() >>> sug

[libvirt] [PATCH] virsh: fix build on mingw, which lacks termios stuff

2013-09-04 Thread Eric Blake
Recent patches to fix handling of Ctrl-C when interacting with ssh are not portable to mingw, which lacks termios handling. The simplest solution is to just compile that code out, and if someone ever appears that has a serious interest in getting virsh fully functional even with ssh connections, th

[libvirt] [PATCH] libxl: Unconditionally call virSysinfoRead() on driver init

2013-09-04 Thread Jim Fehlig
No need to check if priviledged when reading hostsysinfo, since that check was already done in libxlDriverShouldLoad(). The libxl driver fails to load if not priviledged. Signed-off-by: Jim Fehlig --- src/libxl/libxl_driver.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a

Re: [libvirt] [PATCH] libxl: Unconditionally call virSysinfoRead() on driver init

2013-09-04 Thread Eric Blake
On 09/04/2013 05:07 PM, Jim Fehlig wrote: > No need to check if priviledged when reading hostsysinfo, since > that check was already done in libxlDriverShouldLoad(). The > libxl driver fails to load if not priviledged. s/priviledged/privileged/ (twice) > > Signed-off-by: Jim Fehlig > --- > sr

[libvirt] [PATCH] domain_conf: Delete the USB controller check from the USB Device checklist on virDomainDeviceIsUSB

2013-09-04 Thread Liuji (Jeremy)
When using "virsh attach-device" to attach a device, the virDomainDefCompatibleDevice function will check the compatibility. If the device is a USB device, but the VM don't have any USB controller, the execution of "virsh attach-device" will fails. It considers the USB controller as a USB device

Re: [libvirt] [PATCH] build: shut up automake warnings

2013-09-04 Thread Martin Kletzander
On Wed 04 Sep 2013 06:48:46 PM CEST, Eric Blake wrote: > I'm tired of seeing screenfuls of messages like these when using > automake 1.13 (Fedora 19): > > configure.ac:2121: warning: The 'AM_PROG_MKDIR_P' macro is deprecated, and > its use is discouraged. > configure.ac:2121: You should use the Au

Re: [libvirt] [PATCH] [RFC] docs: Add OpenStack into references

2013-09-04 Thread Martin Kletzander
On 09/04/2013 03:56 PM, Osier Yang wrote: > On 04/09/13 20:50, Eric Blake wrote: >> On 09/04/2013 01:26 AM, Martin Kletzander wrote: >>> diff --git a/docs/apps.html.in b/docs/apps.html.in >>> index 46e5dca..9f22df8 100644 >>> --- a/docs/apps.html.in >>> +++ b/docs/apps.html.in >>> @@ -244,6 +244,15

Re: [libvirt] [PATCH] libxl: Unconditionally call virSysinfoRead() on driver init

2013-09-04 Thread Jim Fehlig
Eric Blake wrote: > On 09/04/2013 05:07 PM, Jim Fehlig wrote: > >> No need to check if priviledged when reading hostsysinfo, since >> that check was already done in libxlDriverShouldLoad(). The >> libxl driver fails to load if not priviledged. >> > > s/priviledged/privileged/ (twice) > >