Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-12 Thread Paolo Bonzini
Il 08/06/2013 22:53, Benjamin Herrenschmidt ha scritto: On Sun, 2013-06-09 at 10:16 +0800, Wenchao Xia wrote: If an page was not received and destination knows that page should exist according to total size, fill it with zero at destination, would it solve the problem? The easiest way to

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-12 Thread Benjamin Herrenschmidt
On Wed, 2013-06-12 at 10:00 -0400, Paolo Bonzini wrote: The only bug we have is not a performance bug related to compression; it's that writing zero pages breaks overcommit. Let's fix that, and only that. Right, do we have a way to madvise throw away these instead ? Or do we have a way to

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-12 Thread Paolo Bonzini
Il 12/06/2013 10:11, Benjamin Herrenschmidt ha scritto: On Wed, 2013-06-12 at 10:00 -0400, Paolo Bonzini wrote: The only bug we have is not a performance bug related to compression; it's that writing zero pages breaks overcommit. Let's fix that, and only that. Right, do we have a way to

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-12 Thread Wenchao Xia
于 2013-6-13 4:10, Paolo Bonzini 写道: Il 12/06/2013 10:11, Benjamin Herrenschmidt ha scritto: On Wed, 2013-06-12 at 10:00 -0400, Paolo Bonzini wrote: The only bug we have is not a performance bug related to compression; it's that writing zero pages breaks overcommit. Let's fix that, and only

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-10 Thread Benjamin Herrenschmidt
On Mon, 2013-06-10 at 19:10 +1000, Alexey Kardashevskiy wrote: I would prefer not to completely drop the patch since it saves bandwidth and resources. I would like migration to do what it should do - send pages no matter what, this is exactly what migration is for. If there any many, many

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-10 Thread Peter Lieven
On 10.06.2013 11:33, Benjamin Herrenschmidt wrote: On Mon, 2013-06-10 at 19:10 +1000, Alexey Kardashevskiy wrote: I would prefer not to completely drop the patch since it saves bandwidth and resources. I would like migration to do what it should do - send pages no matter what, this is exactly

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-08 Thread Benjamin Herrenschmidt
On Sun, 2013-06-09 at 12:34 +1000, Alexey Kardashevskiy wrote: It is _live_ migration, the source sends changes, same pages can change and be sent several times. So we would need to turn tracking on on the destination to know if some page was received from the source or changed by the

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-08 Thread Benjamin Herrenschmidt
On Sun, 2013-06-09 at 10:16 +0800, Wenchao Xia wrote: If an page was not received and destination knows that page should exist according to total size, fill it with zero at destination, would it solve the problem? The easiest way to do that is to not write to those pages at the destination

Re: [Qemu-devel] [Qemu-ppc] broken incoming migration

2013-06-08 Thread Alexey Kardashevskiy
On 06/09/2013 12:52 PM, Benjamin Herrenschmidt wrote: On Sun, 2013-06-09 at 12:34 +1000, Alexey Kardashevskiy wrote: It is _live_ migration, the source sends changes, same pages can change and be sent several times. So we would need to turn tracking on on the destination to know if some page