Re: [libvirt] Effect of cluster size on read write performance

2012-02-22 Thread Daniel P. Berrange
On Wed, Feb 22, 2012 at 12:36:00PM +0530, Pankaj Rawat wrote: 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

[libvirt] Fwd: a question about sanlock

2012-02-22 Thread Alex Jia
Hi Daniel, Thanks for your quick reply, from safety point of view, yes, libvirt should exit, but, your alternative solution is more friendly for users, however, I haven't any idea about this now, maybe, others want to do this, so forward it to libvirt-list. Thanks Regards, Alex -

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

2012-02-22 Thread Nicolas Sebrecht
The 21/02/12, Eric Blake wrote: On 02/21/2012 09:05 AM, Peter Krempa wrote: @@ -950,77 +986,125 @@ cmdList(vshControl *ctl, const vshCmd *cmd ATTRIBUTE_UNUSED) } } -if (desc) { -vshPrintExtra(ctl, %-5s %-30s %-10s %s\n, _(Id), _(Name), _(State),

[libvirt] [PATCH] util: Fix virFileAccessibleAs return path from parent

2012-02-22 Thread Michal Privoznik
Despite documentation, if we do fork() parent always returns -1 even if file is accessible. Which is wrong obviously. --- src/util/util.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/util/util.c b/src/util/util.c index 3406b7b..04a0e79 100644 ---

[libvirt] [PATCH 1/2] vmware: implement domainXMLFromNative

2012-02-22 Thread Jean-Baptiste Rouault
--- src/vmware/vmware_driver.c | 31 +++ 1 files changed, 31 insertions(+), 0 deletions(-) diff --git a/src/vmware/vmware_driver.c b/src/vmware/vmware_driver.c index 56e9d2d..8f9d922 100644 --- a/src/vmware/vmware_driver.c +++ b/src/vmware/vmware_driver.c @@ -844,6

[libvirt] [PATCH 0/2] VMware Workstation domainXMLFromNative support

2012-02-22 Thread Jean-Baptiste Rouault
These patches add domainXMLFromNative to the vmware driver and add support for more network configurations in vmx files Jean-Baptiste Rouault (2): vmware: implement domainXMLFromNative vmx: Better Workstation vmx handling src/vmware/vmware_driver.c | 31 +++

[libvirt] [PATCH 2/2] vmx: Better Workstation vmx handling

2012-02-22 Thread Jean-Baptiste Rouault
This patch adds support for vmx files with empty networkName values (which is the case for vmx generated by Workstation). It also adds support for vmx containing NATed network interfaces. --- src/vmx/vmx.c | 29 ++--- 1 files changed, 18 insertions(+), 11 deletions(-)

[libvirt] [PATCH] configure: Define program name if not found

2012-02-22 Thread Michal Privoznik
AC_CHECK_PROG checks for program in given path. However, if it doesn't exists, [variable] is set to [value-if-not-found]. We don't want this to be the empty string in case of 'modprobe' and 'scrub' as we want to fallback to runtime detection. --- configure.ac |4 ++-- src/util/pci.c |5

Re: [libvirt] [PATCH 2/2] vmx: Better Workstation vmx handling

2012-02-22 Thread Matthias Bolte
2012/2/22 Jean-Baptiste Rouault jean-baptiste.roua...@diateam.net: This patch adds support for vmx files with empty networkName values (which is the case for vmx generated by Workstation). It also adds support for vmx containing NATed network interfaces. ---  src/vmx/vmx.c |   29

Re: [libvirt] [PATCH 1/2] vmware: implement domainXMLFromNative

2012-02-22 Thread Matthias Bolte
2012/2/22 Jean-Baptiste Rouault jean-baptiste.roua...@diateam.net: ---  src/vmware/vmware_driver.c |   31 +++  1 files changed, 31 insertions(+), 0 deletions(-) ACK. This one is good and independent from the second patch, so I push it now. -- Matthias Bolte

Re: [libvirt] [PATCH 2/2] vmx: Better Workstation vmx handling

2012-02-22 Thread Matthias Bolte
2012/2/22 Matthias Bolte matthias.bo...@googlemail.com: And this needs extra testcases in the vmx2xml and xml2vmx tests to cover this additions to the VMX parser before I can ACK it. The test suite already has some actual in-the-wild ESX and GSX VMX config files. I suggest you add some actual

Re: [libvirt] [PATCH] util: Fix virFileAccessibleAs return path from parent

2012-02-22 Thread Jiri Denemark
On Wed, Feb 22, 2012 at 10:21:04 +0100, Michal Privoznik wrote: Despite documentation, if we do fork() parent always returns -1 even if file is accessible. Which is wrong obviously. --- src/util/util.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git

Re: [libvirt] [PATCH] configure: Define program name if not found

2012-02-22 Thread Jiri Denemark
On Wed, Feb 22, 2012 at 11:28:33 +0100, Michal Privoznik wrote: AC_CHECK_PROG checks for program in given path. However, if it doesn't exists, [variable] is set to [value-if-not-found]. We don't want this to be the empty string in case of 'modprobe' and 'scrub' as we want to fallback to

Re: [libvirt] [PATCH] util: Fix virFileAccessibleAs return path from parent

2012-02-22 Thread Michal Privoznik
On 22.02.2012 12:06, Jiri Denemark wrote: On Wed, Feb 22, 2012 at 10:21:04 +0100, Michal Privoznik wrote: Despite documentation, if we do fork() parent always returns -1 even if file is accessible. Which is wrong obviously. --- src/util/util.c |8 ++-- 1 files changed, 6

Re: [libvirt] [PATCH] configure: Define program name if not found

2012-02-22 Thread Michal Privoznik
On 22.02.2012 12:09, Jiri Denemark wrote: On Wed, Feb 22, 2012 at 11:28:33 +0100, Michal Privoznik wrote: AC_CHECK_PROG checks for program in given path. However, if it doesn't exists, [variable] is set to [value-if-not-found]. We don't want this to be the empty string in case of 'modprobe'

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

2012-02-22 Thread Gerd Hoffmann
Hi, Isn't this something where it is easier to omit first and add later once we have a use case, than to add up front only to find that no one cares? Yes. I'll drop it. cheers, Gerd -- libvir-list mailing list libvir-list@redhat.com https://www.redhat.com/mailman/listinfo/libvir-list

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

2012-02-22 Thread Peter Krempa
On 02/21/2012 06:05 PM, Eric Blake wrote: On 02/21/2012 09:05 AM, Peter Krempa wrote: --- tools/virsh.c | 176 -- tools/virsh.pod | 20 ++- 2 files changed, 148 insertions(+), 48 deletions(-) Thanks for picking up on my

[libvirt] [PATCH v5 2/2] Add de-association handling to macvlan code

2012-02-22 Thread D. Herrendoerfer
From: D. Herrendoerfer d.herrendoer...@herrendoerfer.name Add de-association handling for 802.1qbg (vepa) via lldpad netlink messages. Also adds the possibility to perform an association request without waiting for a confirmation. Signed-off-by: D. Herrendoerfer

[libvirt] [PATCH v5 1/2] util: Add netlink event handling to virnetlink.c

2012-02-22 Thread D. Herrendoerfer
From: D. Herrendoerfer d.herrendoer...@herrendoerfer.name This code adds a netlink event interface to libvirt. It is based upon the event_poll code and makes use of it. An event is generated for each netlink message sent to the libvirt pid. Signed-off-by: D. Herrendoerfer

[libvirt] [PATCH v5 0/2] util: Add netlink event handling code

2012-02-22 Thread D. Herrendoerfer
From: D. Herrendoerfer d.herrendoer...@herrendoerfer.name This is the updated netlink event code based upon the updated virnetlink.[ch] files. Also the related code to virnetdevmacvlan is included as well as the related changes to the setup code. D. Herrendoerfer (2): util: Add netlink event

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

2012-02-22 Thread Luiz Capitulino
On Wed, 22 Feb 2012 13:08:16 +0100 Gerd Hoffmann kra...@redhat.com wrote: Hi, Isn't this something where it is easier to omit first and add later once we have a use case, than to add up front only to find that no one cares? Yes. I'll drop it. Yes, I think it's better to drop it for

[libvirt] Migration failure related to EOF from Monitor

2012-02-22 Thread Shradha Shah
Hello All, I am trying to migrate a KVM guest while implementing the patches for PCI-Passthrough of SRIOV Vf's and I repeatedly see the following error: [root@c6100k cwd]# virsh migrate --live dibenchvm1 qemu+ssh://c6100l.uk.level5networks.com/system error: operation failed: migration job:

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

2012-02-22 Thread Jiri Denemark
On Wed, Feb 22, 2012 at 14:58:31 +0800, Shu Ming wrote: On 2012-2-22 0:17, Jiri Denemark wrote: @@ -817,6 +817,27 @@ qemuMigrationIsAllowed(struct qemud_driver *driver, virDomainObjPtr vm, return true; } +static bool +qemuMigrationIsSafe(virDomainDefPtr def) +{ +int

Re: [libvirt] Migration failure related to EOF from Monitor

2012-02-22 Thread Michal Privoznik
On 22.02.2012 14:46, Shradha Shah wrote: Hello All, I am trying to migrate a KVM guest while implementing the patches for PCI-Passthrough of SRIOV Vf's and I repeatedly see the following error: [root@c6100k cwd]# virsh migrate --live dibenchvm1

[libvirt] [PATCH v2 1/4] Add support for unsafe migration

2012-02-22 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 v2 0/4] Forbid migration with cache != none

2012-02-22 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] caps: Improve error if passed an unknown arch

2012-02-22 Thread Cole Robinson
On 02/20/2012 05:55 PM, Eric Blake wrote: On 02/20/2012 11:54 AM, Cole Robinson wrote: Previously we would have: os type 'hvm' arch 'idontexist' combination is not supported Now we get No guest options available for arch 'idontexist' or if options available but guest OS type not

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

2012-02-22 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. --- Notes: Version 2: - use virStorageFileIsClusterFS src/qemu/qemu_driver.c|

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

2012-02-22 Thread Jiri Denemark
--- Notes: Version 2: - no change 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 e712e53..3be86ed 100644 --- a/tools/virsh.c +++ b/tools/virsh.c @@ -6819,6 +6819,7 @@

Re: [libvirt] Migration failure related to EOF from Monitor

2012-02-22 Thread Shu Ming
On 2012-2-22 21:46, Shradha Shah wrote: Hello All, I am trying to migrate a KVM guest while implementing the patches for PCI-Passthrough of SRIOV Vf's and I repeatedly see the following error: [root@c6100k cwd]# virsh migrate --live dibenchvm1 qemu+ssh://c6100l.uk.level5networks.com/system

[libvirt] [PATCH v2 3/4] Introduce virStorageFileIsClusterFS

2012-02-22 Thread Jiri Denemark
--- Notes: Version 2: - new patch src/libvirt_private.syms |1 + src/util/storage_file.c | 10 ++ src/util/storage_file.h |1 + 3 files changed, 12 insertions(+), 0 deletions(-) diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 9e3573f..18a24e4

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

2012-02-22 Thread Daniel P. Berrange
On Wed, Feb 22, 2012 at 03:51:11PM +0100, 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. --- Notes: Version

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

2012-02-22 Thread Jiri Denemark
On Wed, Feb 22, 2012 at 14:57:00 +, Daniel P. Berrange wrote: On Wed, Feb 22, 2012 at 03:51:11PM +0100, 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

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

2012-02-22 Thread Dan Kenigsberg
On Tue, Feb 21, 2012 at 05:17:50PM +0100, Christophe Fergeau wrote: 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

Re: [libvirt] [PATCH] libvirt-guests: Enable parallel operations and improve error handling

2012-02-22 Thread Eric Blake
On 02/20/2012 06:32 AM, Peter Krempa wrote: This patch modifies the libvirt-guests init script to enable parallel shiutdown on guest machines and gets rid of error messages if the client is unable connect to the URI specified in the config file. Resuming where I left off (and seeing as how you

[libvirt] [perl-Sys-Virt][PATCH] de-mortalize the return value from hv_delete

2012-02-22 Thread Guannan Ren
when calling block_stats() like $dom-block_stats(), it will reprot errors: Attempt to free unreferenced scalar: SV 0x2598498, Perl interpreter: 0x11a0010. Attempt to free unreferenced scalar: SV 0x258c498, Perl interpreter: 0x11a0010. This patch fix it. --- Virt.xs |1 + 1

Re: [libvirt] [perl-Sys-Virt][PATCH] de-mortalize the return value from hv_delete

2012-02-22 Thread Daniel P. Berrange
On Thu, Feb 23, 2012 at 02:14:17AM +0800, Guannan Ren wrote: when calling block_stats() like $dom-block_stats(), it will reprot errors: Attempt to free unreferenced scalar: SV 0x2598498, Perl interpreter: 0x11a0010. Attempt to free unreferenced scalar: SV 0x258c498, Perl

[libvirt] [PATCH] qemu: nicer error message on failed graceful destroy

2012-02-22 Thread Eric Blake
https://bugzilla.redhat.com/show_bug.cgi?id=795656 mentions that a graceful destroy request can time out, meaning that the error message is user-visible and should be more appropriate than just internal error. * src/qemu/qemu_driver.c (qemuDomainDestroyFlags): Swap error type. ---

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

2012-02-22 Thread Eric Blake
On 02/22/2012 07:51 AM, Jiri Denemark wrote: I was also wondering if we should rather use more specific name for both the error code and flag, such as VIR(_ERR)?_MIGRATE_UNSAFE_CACHE (or ...UNSAFE_DISK) in the case we find other unsafe conditions... I think that if we ever have

Re: [libvirt] [PATCH v2 1/4] Add support for unsafe migration

2012-02-22 Thread Eric Blake
On 02/22/2012 07:51 AM, Jiri Denemark wrote: 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

Re: [libvirt] [PATCH v2 2/4] virsh: Add --unsafe option to migrate command

2012-02-22 Thread Eric Blake
On 02/22/2012 07:51 AM, Jiri Denemark wrote: --- Notes: Version 2: - no change tools/virsh.c |4 tools/virsh.pod | 10 +- 2 files changed, 13 insertions(+), 1 deletions(-) @@ -802,6 +802,14 @@ is implicitly enabled when supported by the hypervisor, but

Re: [libvirt] [PATCH v2 3/4] Introduce virStorageFileIsClusterFS

2012-02-22 Thread Eric Blake
On 02/22/2012 07:51 AM, Jiri Denemark wrote: --- Notes: Version 2: - new patch ACK. -- Eric Blake ebl...@redhat.com+1-919-301-3266 Libvirt virtualization library http://libvirt.org signature.asc Description: OpenPGP digital signature -- libvir-list mailing list

[libvirt] [PATCH] conf/default.cfg: Update Fedora download URLs

2012-02-22 Thread Lucas Meneghel Rodrigues
libvirt TCK is currently broke since it can't download from the old alias download.fedora.redhat.com, since that alias was removed and people really should update their download links. Point to the new dl.fedoraproject.org URLs. Signed-off-by: Lucas Meneghel Rodrigues l...@redhat.com ---

Re: [libvirt] [PATCH] conf/default.cfg: Update Fedora download URLs

2012-02-22 Thread Daniel P. Berrange
On Wed, Feb 22, 2012 at 05:12:29PM -0200, Lucas Meneghel Rodrigues wrote: libvirt TCK is currently broke since it can't download from the old alias download.fedora.redhat.com, since that alias was removed and people really should update their download links. Point to the new

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

2012-02-22 Thread Eric Blake
On 02/22/2012 07:51 AM, 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. --- Notes: Version 2: - use

Re: [libvirt] [PATCH v4 3/7] virsh: add support for VIR_DOMAIN_CONSOLE_* flags

2012-02-22 Thread Eric Blake
On 02/06/2012 06:50 AM, Peter Krempa wrote: This patch adds support for the newly introduced VIR_DOMAIN_CONSOLE_FORCE and VIR_DOMAIN_CONSOLE_SAFE flags. The console command now has an optional parameter --force that specifies that the user wants to forcibly interrupt an ongoing console session

Re: [libvirt] [PATCH v4 4/7] fdstream: Emit stream abort callback even if poll() doesnt.

2012-02-22 Thread Eric Blake
On 02/06/2012 06:50 AM, Peter Krempa wrote: This patch causes the fdstream driver to call the stream event callback if virStreamAbort() is issued on a stream using this driver. This prohibited to abort streams from the daemon, as the daemon remote handler installs a callback to watch for

[libvirt] libvirt TCK wrapper for autotest review

2012-02-22 Thread Lucas Meneghel Rodrigues
Hi guys, I was here looking at the autotest wrapper for libvirt TCK and then decided to work on it, as I had the review fresh on my mind. Things that I've worked on: * Fixed some download links, that were already sent to upstream tck and applied (thanks Dan Berrange) * Instead of making all

Re: [libvirt] [PATCH v4 5/7] fdstream: Add internal callback on stream close

2012-02-22 Thread Eric Blake
On 02/06/2012 06:50 AM, Peter Krempa wrote: This patch adds another callback to a FDstream object. The original callback is used by the daemon stream driver to handle events. This callback is called if and only if the stream is about to be closed. This might be used to handle cleanup steps

Re: [libvirt] [PATCH v4 6/7] util: Add helpers for safe domain console operations

2012-02-22 Thread Eric Blake
On 02/06/2012 06:50 AM, Peter Krempa wrote: This patch adds a set of functions used in creating console streams for domains using PTYs and ensures mutually exclusive access to the PTYs. If mutualy exclusive access is not used, two clients may open the same s/mutualy/mutually/ console,

Re: [libvirt] [PATCH v4 7/7] qemu: Add ability to abort existing console while creating new one

2012-02-22 Thread Eric Blake
On 02/06/2012 06:50 AM, Peter Krempa wrote: This patch fixes console corruption, that happens if two concurrent sessions are opened for a single console on a domain. Result of this corruption was that each of the console streams recieved just a part s/recieved/received/ of the data written

[libvirt] RFC: memory units=... [was: libvirt doesn't boot kVM, hangs, qemu on 100% CPU]

2012-02-22 Thread Eric Blake
On 02/21/2012 03:46 PM, Igor Galić wrote: Hi folks, it's been adventurous. Yesterday night I've started debugging this particular issue of why my KVMs don't boot on Ubuntu 11.10. On IRC, we identified the culprit: uuid8cfcb7b0-10ee-7d08-9b64-9f39c154292a/uuid memory2048/memory

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

2012-02-22 Thread Eric Blake
On 02/16/2012 12:15 AM, 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 Signed-off-by: Lai Jiangshan la...@cn.fujitsu.com ---

Re: [libvirt] RFC: memory units=... [was: libvirt doesn't boot kVM, hangs, qemu on 100% CPU]

2012-02-22 Thread Dave Allan
On Wed, Feb 22, 2012 at 04:42:59PM -0700, Eric Blake wrote: On 02/21/2012 03:46 PM, Igor Galić wrote: Hi folks, it's been adventurous. Yesterday night I've started debugging this particular issue of why my KVMs don't boot on Ubuntu 11.10. On IRC, we identified the culprit:

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

2012-02-22 Thread Lai Jiangshan
On 02/23/2012 08:10 AM, Eric Blake wrote: +if (virDomainCpuSetParse(str, 0, map, + VIR_DOMAIN_CPUMASK_LEN) 0) { +goto error; +} + +for (i = 0; i VIR_DOMAIN_CPUMASK_LEN; i++) { +if (map[i]) { +max_id = i; That's off

Re: [libvirt] [PATCH] qemu: nicer error message on failed graceful destroy

2012-02-22 Thread Laine Stump
On 02/22/2012 01:49 PM, Eric Blake wrote: https://bugzilla.redhat.com/show_bug.cgi?id=795656 mentions that a graceful destroy request can time out, meaning that the error message is user-visible and should be more appropriate than just internal error. * src/qemu/qemu_driver.c

[libvirt] [PATCH 0/3] make memory use in XML easier to read

2012-02-22 Thread Eric Blake
As suggested here: https://www.redhat.com/archives/libvir-list/2012-February/msg00954.html Eric Blake (3): xml: output memory unit for clarity xml: drop unenforced minimum memory limit from RNG xml: allow scaled memory on input docs/formatdomain.html.in | 39

[libvirt] [PATCH 2/3] xml: drop unenforced minimum memory limit from RNG

2012-02-22 Thread Eric Blake
The test domain allows memory0/memory, but the RNG was stating that memory had to be at least 4096000 bytes. Hypervisors should enforce their own limits, rather than complicating the RNG. Meanwhile, some copy and paste had introduced some fishy constructs in various unit tests. *

[libvirt] [PATCH 3/3] xml: allow scaled memory on input

2012-02-22 Thread Eric Blake
Output is still in kibibytes, but input can now be in different scales for ease of typing. * src/conf/domain_conf.c (virDomainParseMemory): New helper. (virDomainDefParseXML): Use it when parsing. * docs/schemas/domaincommon.rng: Expand XML. * docs/formatdomain.html.in (elementsMemoryAllocation):

[libvirt] [PATCH 1/3] xml: output memory unit for clarity

2012-02-22 Thread Eric Blake
Make it obvious to 'dumpxml' readers what unit we are using, since our default of KiB (1024) differs from qemu's default of MiB. Tests were updated via: $ find tests/*data tests/*out -name '*.xml' | \ xargs sed -i

[libvirt] Cluster_size parameter issue on qcow2 image format

2012-02-22 Thread Pankaj Rawat
I have been working o the qcow2 image format , I theory regarding cluster size it is written that as the size of cluster increase performance should increase. But something surprising happen The performance is degrading as the size of cluster increased from 64K to 1M ( during expansion

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

2012-02-22 Thread Paolo Bonzini
On 02/22/2012 02:57 PM, Jiri Denemark wrote: how about disk write through flag here? This flag should also imply cache=none. i.e, VIR_DOMAIN_DISK_CACHE_WRITETHRU AFAIK, only VIR_DOMAIN_DISK_CACHE_DISABLE translates into cache=none unless qemu binary is old enough to only support