[Qemu-devel] [PATCH 00/10] Fix migration with lots of memory

2010-11-23 Thread Juan Quintela
Hi Executive Summary - This series of patches fix migration with lots of memory. With them stalls are removed, and we honored max_dowtime. I also add infrastructure to measure what is happening during migration (#define DEBUG_MIGRATION and DEBUG_SAVEVM). Migration is broken at t

[Qemu-devel] [PATCH 02/10] Add buffered_file_internal constant

2010-11-23 Thread Juan Quintela
From: Juan Quintela This time is each time that buffered_file ticks happen Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- buffered_file.c |6 -- buffered_file.h |2 ++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/buffered_file.c b/buffered_file.c i

[Qemu-devel] [PATCH 04/10] No need to iterate if we already are over the limit

2010-11-23 Thread Juan Quintela
From: Juan Quintela If buffers are full, don't iterate Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- savevm.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/savevm.c b/savevm.c index ceed6de..7c289af 100644 --- a/savevm.c +++ b/savevm.c @@ -1498,6 +

[Qemu-devel] [PATCH 01/10] Add spent time to migration

2010-11-23 Thread Juan Quintela
From: Juan Quintela When printing debug information for migration, print total time spent. Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- migration.c | 13 + 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/migration.c b/migration.c index 9ee8b17..4

[Qemu-devel] [PATCH 07/10] ram_save_remaining() returns an uint64_t

2010-11-23 Thread Juan Quintela
From: Juan Quintela It returns a counter of things, not a ram address. Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- arch_init.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch_init.c b/arch_init.c index df3d91f..9e941a0 100644 --- a/arch_init.

[Qemu-devel] [PATCH 08/10] Count nanoseconds with uint64_t not doubles

2010-11-23 Thread Juan Quintela
From: Juan Quintela Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- arch_init.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/arch_init.c b/arch_init.c index 9e941a0..d32aaae 100644 --- a/arch_init.c +++ b/arch_init.c @@ -216,6 +216,7 @@ int ram_s

[Qemu-devel] [PATCH 06/10] Only calculate expected_time for stage 2

2010-11-23 Thread Juan Quintela
From: Juan Quintela Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- arch_init.c | 10 ++ 1 files changed, 6 insertions(+), 4 deletions(-) diff --git a/arch_init.c b/arch_init.c index 4486925..df3d91f 100644 --- a/arch_init.c +++ b/arch_init.c @@ -217,7 +217,6 @@ int ra

[Qemu-devel] [PATCH 05/10] KVM don't care about TLB handling

2010-11-23 Thread Juan Quintela
From: Juan Quintela TLB handling is only used in TCG mode. It is very costly for guests with lots of memory ad lots of CPU's. Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- exec.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/exec.c b/exec.c index

[Qemu-devel] [PATCH 10/10] Maintaing number of dirty pages

2010-11-23 Thread Juan Quintela
From: Juan Quintela Calculate the number of dirty pages takes a lot on hosts with lots of memory. Just maintain how many pages are dirty. Only sync bitmaps if number is small enough. Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- arch_init.c | 15 +-- cpu-all.h

[Qemu-devel] [PATCH 03/10] Add printf debug to savevm

2010-11-23 Thread Juan Quintela
From: Juan Quintela Once there, print all sections that take more than 100ms to migrate. buffered file runs a timer at that 100ms interval. Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- savevm.c | 48 ++-- 1 files changed, 46 inser

[Qemu-devel] [PATCH 09/10] Exit loop if we have been there too long

2010-11-23 Thread Juan Quintela
From: Juan Quintela cheking each 64 pages is a random magic number as good as any other. We don't want to test too many times, but on the other hand, qemu_get_clock_ns() is not so expensive either. Signed-off-by: Juan Quintela Signed-off-by: Juan Quintela --- arch_init.c | 16 ++

Re: [Qemu-devel] [Patch] Small fix for qemu APIC for Mac OS X support

2010-11-23 Thread Alexander Graf
On 23.11.2010, at 22:25, adq wrote: > This patch ups the APIC version from 0x11 to 0x14. After that Mac OS X > loads successfully (with appropriate kexts, applesmc ain't hooked up > properly yet I see unfortunately). AppleSMC emulation is upstream, but the ACPI entries are missing. Once you add

Re: [Qemu-devel] [PATCH] qemu-kvm: response to SIGUSR1 to start/stop a VCPU (v2)

2010-11-23 Thread Paolo Bonzini
On 11/23/2010 10:46 PM, Anthony Liguori wrote: +static __thread int sigusr1_wfd; While OpenBSD finally updated the default compiler to 4.2.1 from 3.x series, thread local storage is still not supported: Hrm, is there a portable way to do this (distinguish a signal on a particular thread)? Yo

Re: [Qemu-devel] [PATCH] add a command line option to specify the IP address to send multicast packets from

2010-11-23 Thread Mike Ryan
On Tue, Nov 23, 2010 at 03:05:49PM +0200, Michael S. Tsirkin wrote: > On Mon, Nov 22, 2010 at 10:48:45AM -0800, Mike Ryan wrote: > > Michael, this patch implements the feature with a bind address instead > > of a bind interface. It should address the cross-platform issues that > > were raised. > >

Re: [Qemu-devel] [PATCH] qemu-kvm: response to SIGUSR1 to start/stop a VCPU (v2)

2010-11-23 Thread Anthony Liguori
On 11/23/2010 05:43 PM, Paolo Bonzini wrote: On 11/23/2010 10:46 PM, Anthony Liguori wrote: +static __thread int sigusr1_wfd; While OpenBSD finally updated the default compiler to 4.2.1 from 3.x series, thread local storage is still not supported: Hrm, is there a portable way to do this (dist

[Qemu-devel] Re: [PATCHv6 00/16] boot order specification

2010-11-23 Thread Kevin O'Connor
Hi Gleb, On Tue, Nov 23, 2010 at 05:31:41PM +0200, Gleb Natapov wrote: > Anthony, Blue > > No comments on this patch series for almost a week. Can it be applied? My apologies - I haven't had time to review. > On Wed, Nov 17, 2010 at 06:43:47PM +0200, Gleb Natapov wrote: > > I am using open firm

[Qemu-devel] Re: [PATCH] correct migrate_set_speed's args_type

2010-11-23 Thread Wen Congyang
At 2010年11月23日 21:31, Luiz Capitulino Write: > On Tue, 23 Nov 2010 10:43:48 -0200 > Luiz Capitulino wrote: > >> On Tue, 23 Nov 2010 13:41:26 +0800 >> Wen Congyang wrote: >> >>> The args_type of migrate_set_speed in qmp-commands.hx is wrong. >>> When we set migrate speed by json, qemu will be cor

Re: [Qemu-devel] [PATCH] qemu-kvm: response to SIGUSR1 to start/stop a VCPU (v2)

2010-11-23 Thread Paolo Bonzini
On 11/24/2010 02:15 AM, Anthony Liguori wrote: Is it signal safe? Yes, at heart it is just a somewhat more expensive access to pthread_self()->some_array[key]. BTW, this is all only theoretical. This is in the KVM io thread code which is already highly unportable. True, and newer versio

[Qemu-devel] Re: [PATCH 10/12] config: Add header file for device config options

2010-11-23 Thread Juan Quintela
Alexander Graf wrote: > So far we have C preprocessor defines for target and host config > options, but we're lacking any information on which devices are > available. > > We do need that information at times though, for example in the > ahci patch where we need to call a legacy init function depe

[Qemu-devel] Re: [PATCH v2 0/6] qdev reset refactoring and pci bus reset

2010-11-23 Thread Isaku Yamahata
On Tue, Nov 23, 2010 at 08:10:26PM +0200, Michael S. Tsirkin wrote: > On Tue, Nov 23, 2010 at 12:53:12AM +0200, Michael S. Tsirkin wrote: > > On Mon, Nov 22, 2010 at 07:43:37PM +0900, Isaku Yamahata wrote: > > > On Mon, Nov 22, 2010 at 09:54:02AM +0200, Michael S. Tsirkin wrote: > > > > On Fri, Nov

[Qemu-devel] [PATCH v5] virtio-9p: fix build on !CONFIG_UTIMENSAT

2010-11-23 Thread Hidetoshi Seto
This patch introduce a fallback mechanism for old systems that do not support utimensat(). This fix build failure with following warnings: hw/virtio-9p-local.c: In function 'local_utimensat': hw/virtio-9p-local.c:479: warning: implicit declaration of function 'utimensat' hw/virtio-9p-local.c:479:

Re: [Qemu-devel] [Patch] Small fix for qemu APIC for Mac OS X support

2010-11-23 Thread adq
On 23 November 2010 23:41, Alexander Graf wrote: > > On 23.11.2010, at 22:25, adq wrote: > >> This patch ups the APIC version from 0x11 to 0x14. After that Mac OS X >> loads successfully (with appropriate kexts, applesmc ain't hooked up >> properly yet I see unfortunately). >) > AppleSMC emulation

[Qemu-devel] [Bug 680758] [NEW] balloon only resizes by 2M

2010-11-23 Thread Michael Thomas
Public bug reported: when in monitor and running balloon 512 from a 1024M VM, the vm dropped the size to 1020 (this value changes), then every subsequent request to balloon 512 will drop it by another 2M. The system was running at above 60% RAM free when these requests were made. also requesting

[Qemu-devel] Re: [PATCH v2 0/6] qdev reset refactoring and pci bus reset

2010-11-23 Thread Michael S. Tsirkin
On Wed, Nov 24, 2010 at 11:37:25AM +0900, Isaku Yamahata wrote: > On Tue, Nov 23, 2010 at 08:10:26PM +0200, Michael S. Tsirkin wrote: > > On Tue, Nov 23, 2010 at 12:53:12AM +0200, Michael S. Tsirkin wrote: > > > On Mon, Nov 22, 2010 at 07:43:37PM +0900, Isaku Yamahata wrote: > > > > On Mon, Nov 22,

[Qemu-devel] Re: [PATCH] add a command line option to specify the IP address to send multicast packets from

2010-11-23 Thread Michael S. Tsirkin
On Wed, Nov 17, 2010 at 05:16:26PM -0800, Mike Ryan wrote: > Add an option to specify the host IP to send multicast packets from when > using a multicast socket for networking. The option takes an IP address > and sets the IP_MULTICAST_IF socket option, which causes the packets to > use that IP's i

[Qemu-devel] Re: [PATCH v2 0/6] qdev reset refactoring and pci bus reset

2010-11-23 Thread Isaku Yamahata
On Wed, Nov 24, 2010 at 07:27:58AM +0200, Michael S. Tsirkin wrote: > Right. So let's add an inline helper to avoid code duplication here? > > pci: fix bus walk under secondary bus reset > > Take into account secondary bus reset bit for > bus walk: devices behind a reset bus should not > respond

<    1   2