Re: [libvirt] [PATCH] fix check of vcpuid in virDomainVcpuPinDefParseXML

2012-08-23 Thread Daniel Veillard
On Thu, Aug 23, 2012 at 10:31:12AM +0800, Hu Tao wrote: For emulator, the vcpuid field is always set to -1, instead of parsing XML for the value of it. --- src/conf/domain_conf.c | 22 -- 1 file changed, 12 insertions(+), 10 deletions(-) diff --git

Re: [libvirt] [PATCH] fix check of vcpuid in virDomainVcpuPinDefParseXML

2012-08-23 Thread Hu Tao
On Thu, Aug 23, 2012 at 02:12:32PM +0800, Daniel Veillard wrote: On Thu, Aug 23, 2012 at 10:31:12AM +0800, Hu Tao wrote: For emulator, the vcpuid field is always set to -1, instead of parsing XML for the value of it. --- src/conf/domain_conf.c | 22 -- 1 file

Re: [libvirt] [PATCHv4] add-vcpu-usage

2012-08-23 Thread Hu Tao
Hi Rich, The corresponding bugzilla item is 841759. Will you have a look at this patch? On Wed, May 09, 2012 at 04:48:29PM +0800, Hu Tao wrote: show `vcpu usages' by `virt-top -1' Before this patch, `virt-top -1' shows total cpu usages which euqal to `vcpu usages' + `hypervisor usages'.

Re: [libvirt] [test-API][PATCH 1/2] New get_conn function in utils

2012-08-23 Thread Guannan Ren
On 08/16/2012 05:00 PM, Wayne Sun wrote: The get_conn function return connection object from libvirt module. This function could be used by both framework and testcases. The patch includes: * get_conn in utils/utils.py * sync env_inspect.py using the new function Signed-off-by:

Re: [libvirt] [test-API][PATCH 2/2] Update cases with libvirtd restart problem

2012-08-23 Thread Guannan Ren
On 08/16/2012 05:00 PM, Wayne Sun wrote: Restart libvirtd during test will break conn from framework. Update cases with: * Add notes in case description * Using get_conn to reconnect in cases Signed-off-by: Wayne Sun g...@redhat.com --- repos/domain/ownership_test.py |6

Re: [libvirt] [PATCH libvirt-virshcmdref] Add blockpull and blockjob commands

2012-08-23 Thread Stefan Hajnoczi
On Wed, Aug 22, 2012 at 12:07:51PM -0600, Eric Blake wrote: On 08/22/2012 07:40 AM, Stefan Hajnoczi wrote: +++ b/source/blockjob.xml +parameter requirement=required + value type=string requirement=requiredpath/value + description +The fully-qualified path of the

Re: [libvirt] [PATCH] docs: fix blockpull/blockcopy bandwidth Mbps - MiB/s

2012-08-23 Thread Stefan Hajnoczi
On Wed, Aug 22, 2012 at 05:23:43PM +0200, Michal Privoznik wrote: On 22.08.2012 17:12, Eric Blake wrote: On 08/22/2012 07:59 AM, Stefan Hajnoczi wrote: The bandwidth units for blockpull and blockcopy are in Megabytes per Second, not Megabits per Second. Signed-off-by: Stefan Hajnoczi

[libvirt] [PATCH V3] Add proxy FS support to libvirt

2012-08-23 Thread M. Mohan Kumar
From: M. Mohan Kumar mo...@in.ibm.com A new FS driver type 'proxy' is added to QEMU 9p server. This patch adds support for using proxy FS driver from libvirt. QEMU proxy FS driver uses socket for communicating between helper and qemu proxy FS driver. Proxy helper (a stand alone binary part of

[libvirt] [PATCH] Fix a missing build requirement on dbus-devel

2012-08-23 Thread Daniel Veillard
Communication with the firewall daemon uses DBus so if we compile with firewalld support, the dbus-devel is required for building pushed under build breaker rules diff --git a/libvirt.spec.in b/libvirt.spec.in index ea2fd88..18a7fb8 100644 --- a/libvirt.spec.in +++ b/libvirt.spec.in @@ -521,6

[libvirt] [PATCH v2 libvirt-virshcmdref] Add blockpull and blockjob commands

2012-08-23 Thread Stefan Hajnoczi
The blockpull and blockjob commands have been present since 0.9.4. This patch adds basic usage examples and command syntax. Signed-off-by: Stefan Hajnoczi stefa...@linux.vnet.ibm.com --- v2: * Linewrap to 80 chars except pre-formatted terminal tag [Eric] * 'path' argument can be target

[libvirt] [RFC PATCH v1 0/2] Qemu/Gluster support in Libvirt

2012-08-23 Thread Harsh Prateek Bora
This patchset provides support for Gluster protocol based network disks. It is based on the proposed gluster support in Qemu on qemu-devel: http://lists.gnu.org/archive/html/qemu-devel/2012-08/msg01539.html TODO: - Add support for IPv6 format based server addr - Support for transport types other

[libvirt] [RFC PATCH v1 1/2] Qemu/Gluster: Add Gluster protocol as supported network disk formats.

2012-08-23 Thread Harsh Prateek Bora
Qemu accepts gluster protocol as supported storage backend beside others. This patch allows users to specify disks on gluster backends like this: disk type='network' device='disk' driver name='qemu' type='raw'/ source protocol='gluster' name='volume/image' host

[libvirt] [RFC PATCH v1 2/2] tests: Add tests for gluster protocol based network disks support

2012-08-23 Thread Harsh Prateek Bora
Signed-off-by: Harsh Prateek Bora ha...@linux.vnet.ibm.com --- tests/qemuargv2xmltest.c | 1 + .../qemuxml2argv-disk-drive-network-gluster.args | 1 + .../qemuxml2argv-disk-drive-network-gluster.xml| 33 ++ tests/qemuxml2argvtest.c

Re: [libvirt] [PATCH v4 0/7] support guest agent general command

2012-08-23 Thread Daniel Veillard
On Thu, Aug 23, 2012 at 12:19:24PM +0900, MATSUDA, Daiki wrote: Hi. All. I rewrote the patched for git head tree, adding some code. And @timeout must be following VIR_DOMAIN_QEMU_AGENT_COMMAND_BLOCK(-2) : does not block and wait forever. VIR_DOMAIN_QEMU_AGENT_COMMAND_DEFAULT(-1) : use

Re: [libvirt] [PATCH v4 6/7] add python module support Add virDomainQemuAgentCommand() support function to python module.

2012-08-23 Thread Daniel Veillard
On Thu, Aug 23, 2012 at 12:29:26PM +0900, MATSUDA Daiki wrote: Signed-off-by: MATSUDA Daiki matsuda...@intellilink.co.jp --- python/libvirt-qemu-override-api.xml |8 python/libvirt-qemu-override.c | 29 + 2 files changed, 37 insertions(+), 0

Re: [libvirt] [PATCH v4 4/7] add qemu driver support Add qemuDrvDomainAgentCommand() for .qemuDomainArbitraryAgentCommand to qemu driver.

2012-08-23 Thread Daniel Veillard
On Thu, Aug 23, 2012 at 12:29:24PM +0900, MATSUDA Daiki wrote: Signed-off-by: MATSUDA Daiki matsuda...@intellilink.co.jp --- src/qemu/qemu_driver.c | 71 1 files changed, 71 insertions(+), 0 deletions(-) diff --git

[libvirt] [PATCH v4 0/3] Per-guest S3/S4 configuration

2012-08-23 Thread Martin Kletzander
There is a way to tell qemu how (not) to advertise S3/S4 ACPI sleep state capability to the guest OS. This series includes the capability to set this in the XML and also covers all the handling from qemu point of view including checking for the support, parameter parsing, command building,

[libvirt] [PATCH v4 1/3] Add per-guest S3/S4 state configuration

2012-08-23 Thread Martin Kletzander
There is a new pm/ element implemented that can control what ACPI sleeping states will be advertised by BIOS and allowed to be switched to by libvirt. The default keeps defaults on hypervisor, otherwise forces chosen setting. The documentation of the pm element is added as well. ---

[libvirt] [PATCH v4 2/3] qemu: Add support for S3/S4 state configuration

2012-08-23 Thread Martin Kletzander
This patch adds support for running qemu guests with the required parameters to forcefully enable or disable BIOS advertising of S3 and S4 states. The support for this is added to capabilities and there is also a qemu command parameter parsing implemented. --- src/qemu/qemu_capabilities.c | 7

[libvirt] [PATCH v4 3/3] tests: Add tests for qemu S3/S4 state configuration

2012-08-23 Thread Martin Kletzander
Few tests were added which are checking whether the parsing of the xml and command-line arguments is working and compatible with each other. --- tests/qemuargv2xmltest.c | 3 +++ .../qemuxml2argv-misc-disable-s3.args | 4 +++

Re: [libvirt] [Libvirt-announce] Entering freeze for libvirt-0.10.0

2012-08-23 Thread Justin Clift
On 22/08/2012, at 9:51 PM, Daniel Veillard wrote: As planned, I just tagged 0.10.0 release candidate 1 in git and pushed a tarball and rpms to the usual place: ftp://libvirt.org/libvirt/ I gave it a try locally and things seems to work as expected, but that's far from an extensive

Re: [libvirt] [Libvirt-announce] Entering freeze for libvirt-0.10.0

2012-08-23 Thread Daniel P. Berrange
On Thu, Aug 23, 2012 at 11:21:21PM +1000, Justin Clift wrote: On 22/08/2012, at 9:51 PM, Daniel Veillard wrote: As planned, I just tagged 0.10.0 release candidate 1 in git and pushed a tarball and rpms to the usual place: ftp://libvirt.org/libvirt/ I gave it a try locally and things

[libvirt] [PATCH] qemu: forbid suspend if already pmsuspended

2012-08-23 Thread Michal Privoznik
If a domain is pmsuspended then virsh suspend will succeed. Beside obvious flaw, virsh resume will report success and change domain state to running which is another mistake. Therefore we must forbid any attempts for suspend and resume when pmsuspended. --- src/libvirt.c |4

Re: [libvirt] [Libvirt-announce] Entering freeze for libvirt-0.10.0

2012-08-23 Thread Justin Clift
On 23/08/2012, at 11:24 PM, Daniel P. Berrange wrote: On Thu, Aug 23, 2012 at 11:21:21PM +1000, Justin Clift wrote: On 22/08/2012, at 9:51 PM, Daniel Veillard wrote: As planned, I just tagged 0.10.0 release candidate 1 in git and pushed a tarball and rpms to the usual place:

Re: [libvirt] [PATCH v4 2/3] qemu: Add support for S3/S4 state configuration

2012-08-23 Thread Michal Privoznik
On 23.08.2012 13:47, Martin Kletzander wrote: This patch adds support for running qemu guests with the required parameters to forcefully enable or disable BIOS advertising of S3 and S4 states. The support for this is added to capabilities and there is also a qemu command parameter parsing

Re: [libvirt] [PATCH v4 1/3] Add per-guest S3/S4 state configuration

2012-08-23 Thread Michal Privoznik
On 23.08.2012 13:47, Martin Kletzander wrote: There is a new pm/ element implemented that can control what ACPI sleeping states will be advertised by BIOS and allowed to be switched to by libvirt. The default keeps defaults on hypervisor, otherwise forces chosen setting. The documentation of

Re: [libvirt] [PATCH v4 0/3] Per-guest S3/S4 configuration

2012-08-23 Thread Michal Privoznik
On 23.08.2012 13:47, Martin Kletzander wrote: There is a way to tell qemu how (not) to advertise S3/S4 ACPI sleep state capability to the guest OS. This series includes the capability to set this in the XML and also covers all the handling from qemu point of view including checking for the

Re: [libvirt] [PATCH v4 3/3] tests: Add tests for qemu S3/S4 state configuration

2012-08-23 Thread Michal Privoznik
On 23.08.2012 13:47, Martin Kletzander wrote: Few tests were added which are checking whether the parsing of the xml and command-line arguments is working and compatible with each other. --- tests/qemuargv2xmltest.c | 3 +++ .../qemuxml2argv-misc-disable-s3.args

[libvirt] [PATCH] network: fix virtual network bridge delay setting

2012-08-23 Thread Laine Stump
libvirt's network config documents that a bridge's STP forward delay (called delay in the XML) should be specified in seconds, but virNetDevBridgeSetSTPDelay() assumes that it is given a delay in milliseconds (although the comment at the top of the function incorrectly says seconds. This fixes

Re: [libvirt] [PATCH] qemu: forbid suspend if already pmsuspended

2012-08-23 Thread Eric Blake
On 08/23/2012 07:31 AM, Michal Privoznik wrote: If a domain is pmsuspended then virsh suspend will succeed. Beside obvious flaw, virsh resume will report success and change domain state to running which is another mistake. Therefore we must forbid any attempts for suspend and resume when

Re: [libvirt] [PATCH v4 3/7] Add virDrvDomainQemuAgentCommand prototype for drivers. Add virDrvDomainQemuAgentCommand prototype for drivers. Add virDomainQemuAgentCommand() for virDrvDomainQemuAgentCo

2012-08-23 Thread Eric Blake
On 08/22/2012 09:29 PM, MATSUDA Daiki wrote: Signed-off-by: MATSUDA Daiki matsuda...@intellilink.co.jp --- include/libvirt/libvirt-qemu.h |3 ++ python/generator.py|1 + src/driver.h |4 +++ src/libvirt-qemu.c | 54

[libvirt] [PATCH] qemu-agent: available in 0.10.0

2012-08-23 Thread Eric Blake
The recent virDomainQemuAgentCommand addition is part of 0.10.0; also, grouping all libvirt-qemu.so callbacks together makes them easier to identify. * src/libvirt_qemu.syms: Fix release symbol. * src/qemu/qemu_driver.c (qemuDriver): Likewise. * src/remote/remote_driver.c (remote_driver):

Re: [libvirt] [PATCH] qemu: forbid suspend if already pmsuspended

2012-08-23 Thread Michal Privoznik
On 23.08.2012 17:47, Eric Blake wrote: On 08/23/2012 07:31 AM, Michal Privoznik wrote: If a domain is pmsuspended then virsh suspend will succeed. Beside obvious flaw, virsh resume will report success and change domain state to running which is another mistake. Therefore we must forbid any

Re: [libvirt] [PATCH] network: fix virtual network bridge delay setting

2012-08-23 Thread Eric Blake
On 08/23/2012 09:26 AM, Laine Stump wrote: libvirt's network config documents that a bridge's STP forward delay (called delay in the XML) should be specified in seconds, but virNetDevBridgeSetSTPDelay() assumes that it is given a delay in milliseconds (although the comment at the top of the

Re: [libvirt] [PATCH v4 1/3] Add per-guest S3/S4 state configuration

2012-08-23 Thread Eric Blake
On 08/23/2012 05:47 AM, Martin Kletzander wrote: There is a new pm/ element implemented that can control what ACPI sleeping states will be advertised by BIOS and allowed to be switched to by libvirt. The default keeps defaults on hypervisor, otherwise forces chosen setting. The documentation

Re: [libvirt] [PATCH v4 0/3] Per-guest S3/S4 configuration

2012-08-23 Thread Eric Blake
On 08/23/2012 08:04 AM, Michal Privoznik wrote: On 23.08.2012 13:47, Martin Kletzander wrote: There is a way to tell qemu how (not) to advertise S3/S4 ACPI sleep state capability to the guest OS. This series includes the capability to set this in the XML and also covers all the handling from

Re: [libvirt] [PATCH] network: fix virtual network bridge delay setting

2012-08-23 Thread Laine Stump
On 08/23/2012 12:06 PM, Eric Blake wrote: On 08/23/2012 09:26 AM, Laine Stump wrote: libvirt's network config documents that a bridge's STP forward delay (called delay in the XML) should be specified in seconds, but virNetDevBridgeSetSTPDelay() assumes that it is given a delay in milliseconds

[libvirt] Guest log getting written in cache directory rather /var/log/libvirt/qemu

2012-08-23 Thread Richa Marwaha
Hi Everyone I am running libvirt with selinux driver under unprivileged user and the Guest.log file is not written into /var/log/libvirt/qemu/Guest.log but remains in Home Directory/.cache/libvirt/qemu/log/Guest.log. Does anyone know why its not writing to the required log location. Thanks

Re: [libvirt] Guest log getting written in cache directory rather /var/log/libvirt/qemu

2012-08-23 Thread Eric Blake
On 08/23/2012 02:19 PM, Richa Marwaha wrote: Hi Everyone I am running libvirt with selinux driver under unprivileged user and the Guest.log file is not written into /var/log/libvirt/qemu/Guest.log but remains in Home Directory/.cache/libvirt/qemu/log/Guest.log. Does anyone know why its

Re: [libvirt] Guest log getting written in cache directory rather /var/log/libvirt/qemu

2012-08-23 Thread Richa Marwaha
Eric Blake ebl...@redhat.com wrote on 08/23/2012 04:32:02 PM: From: Eric Blake ebl...@redhat.com To: Richa Marwaha/Silicon Valley/IBM@IBMUS Cc: libvir-list@redhat.com Date: 08/23/2012 04:32 PM Subject: Re: [libvirt] Guest log getting written in cache directory rather /

[libvirt] [PATCHv2 0/3] split snapshot conf out of domain conf

2012-08-23 Thread Eric Blake
Rebased version from v1 [1], only changes are catering to other changes in domain_conf in the meantime, and (temporarily) dropping the XML changes of the original patch 4 until I have code that can actually use the XML in 0.10.1. I'm hoping these three patches still qualify for 0.10.0. [1]

[libvirt] [PATCHv2 1/3] snapshot: make virDomainSnapshotObjList opaque

2012-08-23 Thread Eric Blake
We were failing to react to allocation failure when initializing a snapshot object list. Changing things to store a pointer instead of a complete object adds one more possible point of allocation failure, but at the same time, will make it easier to react to failure now, as well as making it

[libvirt] [PATCHv2 3/3] snapshot: rename an enum

2012-08-23 Thread Eric Blake
The name 'virDomainDiskSnapshot' didn't fit in with our normal conventions of using a prefix hinting that it is related to a virDomainSnapshotPtr. Also, a future patch will reuse the enum for declaring where the VM memory is stored. * src/conf/snapshot_conf.h (virDomainDiskSnapshot): Rename...

Re: [libvirt] Guest log getting written in cache directory rather /var/log/libvirt/qemu

2012-08-23 Thread Richa Marwaha
Eric Blake ebl...@redhat.com wrote on 08/23/2012 04:32:02 PM: From: Eric Blake ebl...@redhat.com To: Richa Marwaha/Silicon Valley/IBM@IBMUS Cc: libvir-list@redhat.com Date: 08/23/2012 04:32 PM Subject: Re: [libvirt] Guest log getting written in cache directory rather /

Re: [libvirt] Guest log getting written in cache directory rather /var/log/libvirt/qemu

2012-08-23 Thread Eric Blake
On 08/23/2012 03:09 PM, Richa Marwaha wrote: I have another question shouldn't the label of HomeDirectory/.cache/libvirt/qemu/log/Guest.log be virt_log_t instead of cache_home_t as I am getting the following denial for qemu-kvm when I am running the guest type=AVC

[libvirt] How to solve this problem---libvirtError: this function is not supported by the connection driver: virNWFilterDefineXML

2012-08-23 Thread Zhihong Wang
Dear all: I am trying to deploy openstack on xen hypervisor.Since XenSever or XCP is not available on our customized virtulization platform. We decide to use libvirt on xen as a middleware operating the xen hypervisor. But when I try to boot a instance , xen failed to response and I find the