Re: [libvirt] [PATCH 1/3 V6] add nodeGetCPUmap() for getting available CPU IDs in a cpumap.

2012-02-21 Thread Lai Jiangshan
Hi, Eric Did you received these new patches? Thanks, Lai On 02/16/2012 03:15 PM, Lai Jiangshan wrote: From: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com Changelog: - fixed typos. - fixed string scan routine. Signed-off-by: KAMEZAWA Hiroyuki kamezawa.hir...@jp.fujitsu.com

Re: [libvirt] [Qemu-devel] [PATCH v5 12/12] suspend: add qmp events

2012-02-21 Thread Gerd Hoffmann
On 02/17/12 18:33, Luiz Capitulino wrote: On Wed, 15 Feb 2012 11:28:21 +0100 Gerd Hoffmann kra...@redhat.com wrote: Send qmp events on suspend and wakeup so libvirt has a chance to track the vm state. [ added libvirt to Cc:, leaving full context. this is about qmp events when the guest

Re: [libvirt] Network not Persistent

2012-02-21 Thread Shradha Shah
On 02/20/2012 06:43 PM, Laine Stump wrote: On 02/20/2012 12:37 PM, Shradha Shah wrote: Hello All, I am currently working on patches for PCI-passthrough of SRIOV VF's and I am facing an issue with the network definition not being persistent. I am using a new forward mode = hostdev and the

[libvirt] [PATCH] virsh: Fix docs for list command

2012-02-21 Thread Peter Krempa
Commit fad5cd210899dfde4afe36712754dc921c3f3051 introduced option to display domain's title in the list command output. There was a mistake in the virsh man page example for this command stating --note instead of --title. --- Pushed under trivial rule. tools/virsh.pod |2 +- 1 files changed,

[libvirt] [PATCH] lib: Fix function documentation for virConnectListDomains

2012-02-21 Thread Peter Krempa
Clarify the documentation of virConnectListDomains. --- src/libvirt.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/libvirt.c b/src/libvirt.c index a55d823..6294196 100644 --- a/src/libvirt.c +++ b/src/libvirt.c @@ -1818,23 +1818,23 @@ error:

Re: [libvirt] [PATCH v3] Support for cpu64-rhel* qemu cpu models

2012-02-21 Thread Daniel P. Berrange
On Mon, Feb 13, 2012 at 03:32:26PM +0100, Jiri Denemark wrote: On Fri, Feb 10, 2012 at 11:22:09 -0700, Eric Blake wrote: On 01/30/2012 09:25 AM, Martin Kletzander wrote: In qemu there are 2 cpu models (cpu64-rhel5 and cpu64-rhel6) not supported by libvirt. This patch adds the support with

Re: [libvirt] [PATCH v4 0/7] Console corruption patchset

2012-02-21 Thread Peter Krempa
I don't like bothering you, but this series somehow keeps getting forgotten. It should still apply cleanly on current HEAD. Peter On 02/06/2012 02:50 PM, Peter Krempa wrote: Rebased and tweaked to current upstream. Some patches still need review: * 1/7 pidfile: Make checking binary path in

Re: [libvirt] [PATCH v3] Support for cpu64-rhel* qemu cpu models

2012-02-21 Thread Jiri Denemark
On Tue, Feb 21, 2012 at 14:54:39 +, Daniel P. Berrange wrote: On Mon, Feb 13, 2012 at 03:32:26PM +0100, Jiri Denemark wrote: It was designed to be okay. Libvirt checks what CPU models are supported by qemu and avoids passing unsupported models to qemu. After all, we support running

[libvirt] non-TLS spice connections

2012-02-21 Thread Christophe Fergeau
Hey, I recently setup ovirt/vdsm on one of my boxes. For some reason spice TLS was disabled through spice_tls = no in /etc/libvirt/qemu.conf. Then I created a VM, started it, and had troubles connecting to it with spicec. After investigating (thanks djasa), I realized that qemu was started with

Re: [libvirt] Network not Persistent

2012-02-21 Thread Laine Stump
On 02/21/2012 07:33 AM, Shradha Shah wrote: On 02/20/2012 06:43 PM, Laine Stump wrote: On 02/20/2012 12:37 PM, Shradha Shah wrote: Hello All, I am currently working on patches for PCI-passthrough of SRIOV VF's and I am facing an issue with the network definition not being persistent. I am

[libvirt] [PATCH] virsh: Enhance list command to ease creation of shell scripts

2012-02-21 Thread Peter Krempa
This patch adds new options to the virsh list command enabling filtering of persistent and transient domains along with the option to print only UUIDs or names of domains instead of printing the table. Option --name prints domain names (one per line) instead of the default table. Similarly --uuid

Re: [libvirt] [vdsm] non-TLS spice connections

2012-02-21 Thread Dan Kenigsberg
On Tue, Feb 21, 2012 at 04:53:24PM +0100, Christophe Fergeau wrote: Hey, I recently setup ovirt/vdsm on one of my boxes. For some reason spice TLS was disabled through spice_tls = no in /etc/libvirt/qemu.conf. Then I created a VM, started it, and had troubles connecting to it with spicec.

[libvirt] [PATCH 1/3] Add support for unsafe migration

2012-02-21 Thread Jiri Denemark
This patch adds VIR_MIGRATE_UNSAFE flag for migration APIs and new VIR_ERR_MIGRATION_UNSAFE error code. The error code should be returned whenever migrating a domain is considered unsafe (e.g., it's configured in a way that does not ensure data integrity once it is migrated). VIR_MIGRATE_UNSAFE

[libvirt] [PATCH 2/3] virsh: Add --unsafe option to migrate command

2012-02-21 Thread Jiri Denemark
--- tools/virsh.c |4 tools/virsh.pod | 10 +- 2 files changed, 13 insertions(+), 1 deletions(-) diff --git a/tools/virsh.c b/tools/virsh.c index 66ba61c..4e30325 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -6733,6 +6733,7 @@ static const vshCmdOptDef opts_migrate[] =

[libvirt] [PATCH 3/3] qemu: Forbid migration with cache != none

2012-02-21 Thread Jiri Denemark
Migrating domains with disks using cache != none is unsafe unless the disk images are stored on coherent clustered filesystem. Thus we forbid migrating such domains unless VIR_MIGRATE_UNSAFE flags is used. --- src/qemu/qemu_driver.c|3 ++- src/qemu/qemu_migration.c | 36

Re: [libvirt] [PATCH 5/6] conf: give each hostdevdef a parent pointer

2012-02-21 Thread Laine Stump
On 02/20/2012 05:02 PM, Eric Blake wrote: diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index 004cba7..12d48fb 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -1318,6 +1318,12 @@ void virDomainHostdevDefFree(virDomainHostdevDefPtr def) if (!def)

Re: [libvirt] [vdsm] non-TLS spice connections

2012-02-21 Thread Christophe Fergeau
Hi, On Tue, Feb 21, 2012 at 06:09:06PM +0200, Dan Kenigsberg wrote: Please note Bug 788092 - VDSM: Disable vdsm's ssl'ability influence spice ssl'ability and prevent VM from starting https://bugzilla.redhat.com/show_bug.cgi?id=788092#c1 Could it be that you have ssl=false in your vdsm.conf?

[libvirt] [PATCH 0/3] Forbid migration with cache != none

2012-02-21 Thread Jiri Denemark
Migrating qemu domains with disks using cache != none is unsafe unless the disk images are stored on coherent clustered filesystem. Thus we forbid migrating such domains unless VIR_MIGRATE_UNSAFE flags is used. This series uses similar aproach to forbidding unsafe PCI passthrough or disk format

Re: [libvirt] [PATCH 0/3] Forbid migration with cache != none

2012-02-21 Thread Daniel P. Berrange
On Tue, Feb 21, 2012 at 05:17:20PM +0100, Jiri Denemark wrote: Migrating qemu domains with disks using cache != none is unsafe unless the disk images are stored on coherent clustered filesystem. Thus we forbid migrating such domains unless VIR_MIGRATE_UNSAFE flags is used. This series uses

Re: [libvirt] [Qemu-devel] [PATCH v5 12/12] suspend: add qmp events

2012-02-21 Thread Eric Blake
On 02/21/2012 03:00 AM, Gerd Hoffmann wrote: [ added libvirt to Cc:, leaving full context. this is about qmp events when the guest enters/leaves s3 ]. @@ -1436,12 +1437,26 @@ void qemu_register_suspend_notifier(Notifier *notifier) void qemu_system_wakeup_request(WakeupReason reason)

Re: [libvirt] Network not Persistent

2012-02-21 Thread Shradha Shah
On 02/21/2012 03:59 PM, Laine Stump wrote: On 02/21/2012 07:33 AM, Shradha Shah wrote: On 02/20/2012 06:43 PM, Laine Stump wrote: On 02/20/2012 12:37 PM, Shradha Shah wrote: Hello All, I am currently working on patches for PCI-passthrough of SRIOV VF's and I am facing an issue with the

Re: [libvirt] [PATCH v3] Support for cpu64-rhel* qemu cpu models

2012-02-21 Thread Jiri Denemark
On Mon, Jan 30, 2012 at 17:25:19 +0100, Martin Kletzander wrote: In qemu there are 2 cpu models (cpu64-rhel5 and cpu64-rhel6) not supported by libvirt. This patch adds the support with the flags specifications from /usr/share/qemu-kvm/cpu-model/cpu-x86_64.conf Hmm, I didn't actually look at

Re: [libvirt] [PATCH] lib: Fix function documentation for virConnectListDomains

2012-02-21 Thread Eric Blake
On 02/21/2012 07:18 AM, Peter Krempa wrote: Clarify the documentation of virConnectListDomains. --- src/libvirt.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) ACK. -- Eric Blake ebl...@redhat.com+1-919-301-3266 Libvirt virtualization library http://libvirt.org

Re: [libvirt] [PATCH] virsh: Enhance list command to ease creation of shell scripts

2012-02-21 Thread Eric Blake
On 02/21/2012 09:05 AM, Peter Krempa wrote: This patch adds new options to the virsh list command enabling filtering of persistent and transient domains along with the option to print only UUIDs or names of domains instead of printing the table. Option --name prints domain names (one per

Re: [libvirt] Network not Persistent

2012-02-21 Thread Eric Blake
On 02/21/2012 09:32 AM, Shradha Shah wrote: Before you restart libvirtd, is the xml file in place in /etc/libvirt/qemu/networks/pci-passthrough-network.xml? And is that file still there after libvirtd restarts? If the file is still there but the definition doesn't show up in net-list --all,

Re: [libvirt] [PATCH 6/6] conf: parse/format type='hostdev' network interfaces

2012-02-21 Thread Laine Stump
On 02/20/2012 05:40 PM, Eric Blake wrote: On 02/20/2012 10:10 AM, Laine Stump wrote: This is the new interface type that sets up a PCI/USB network device to be assigned to the guest with PCI/USB passthrough after initializing some network device-specific things from the config (e.g. MAC

[libvirt] [PATCH 1/6] conf: make hostdev info a separate object

2012-02-21 Thread Laine Stump
I'm also squashing in this (addition of virDomainHostdevDefClear()) into PATCH 1/6, per the discussion in: https://www.redhat.com/archives/libvir-list/2012-February/msg00895.html --- src/conf/domain_conf.c | 17 - src/conf/domain_conf.h |1 + src/libvirt_private.syms |

Re: [libvirt] [PATCH] Use the same MAC address that is defined in domain XML for attached-mac field.

2012-02-21 Thread Ansis Atteka
On Mon, Feb 20, 2012 at 1:46 AM, Daniel P. Berrange berra...@redhat.comwrote: On Sat, Feb 18, 2012 at 10:07:45PM -0500, Laine Stump wrote: On 02/17/2012 02:51 PM, Ansis Atteka wrote: On Fri, Feb 17, 2012 at 10:55 AM, Laine Stump la...@laine.org mailto:la...@laine.org wrote:

Re: [libvirt] virtio-win-0.1-22 balloon install fails on 2008R2

2012-02-21 Thread Eric Blake
On 02/18/2012 12:52 PM, sean darcy wrote: Trying to install the virtio-win-0.1-22.iso balloon driver on Windows 2008R2: E:\Wlh\amd64x:\microsoft\WinDDK\7600.16385.win7_wdk.100208-1538\tools\devcon\amd64\devcon.exe install BALLOON.inf PCI\VEN_1AF4DEV_1002SUBSYS_00051AF4REV_00 Device node

Re: [libvirt] [PATCH] esx: Correctly disable HTTP Expect header usage of libcurl

2012-02-21 Thread Matthias Bolte
2012/2/20 Eric Blake ebl...@redhat.com: On 02/20/2012 02:38 PM, Matthias Bolte wrote: Adding Expect: to the header list stops libcurl from sending a Expect header at all. Before, a dummy Expect header was added that might confuse HTTP proxies and result in HTTP error code 417 being reported.

Re: [libvirt] virtio-win-0.1-22 balloon install fails on 2008R2

2012-02-21 Thread sean darcy
On 02/21/2012 04:29 PM, Eric Blake wrote: On 02/18/2012 12:52 PM, sean darcy wrote: Trying to install the virtio-win-0.1-22.iso balloon driver on Windows 2008R2: E:\Wlh\amd64x:\microsoft\WinDDK\7600.16385.win7_wdk.100208-1538\tools\devcon\amd64\devcon.exe install BALLOON.inf

Re: [libvirt] [PATCH 3/3] qemu: Forbid migration with cache != none

2012-02-21 Thread Shu Ming
On 2012-2-22 0:17, Jiri Denemark wrote: Migrating domains with disks using cache != none is unsafe unless the disk images are stored on coherent clustered filesystem. Thus we forbid migrating such domains unless VIR_MIGRATE_UNSAFE flags is used. --- src/qemu/qemu_driver.c|3 ++-

[libvirt] Effect of cluster size on read write performance

2012-02-21 Thread Pankaj Rawat
Hi all The cluster -size increase write performance when the qcow2 image is not expanded , surely with increase in cluster_size the disk space used increases but that is not my concern. Can anyone tell what is the effect of cluster size on read and write performance on qcow2 image after