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 tha

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 w

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 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 easies

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 w

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,

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 so

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 destinatio

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 dest