Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-25 Thread Alexey Perevalov
On 04/25/2017 01:25 PM, Peter Xu wrote: On Tue, Apr 25, 2017 at 01:10:30PM +0300, Alexey Perevalov wrote: On 04/25/2017 11:24 AM, Peter Xu wrote: On Fri, Apr 14, 2017 at 04:17:18PM +0300, Alexey Perevalov wrote: [...] +/* + * This function calculates downtime per cpu and trace it + * + *

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-25 Thread Peter Xu
On Tue, Apr 25, 2017 at 01:10:30PM +0300, Alexey Perevalov wrote: > On 04/25/2017 11:24 AM, Peter Xu wrote: > >On Fri, Apr 14, 2017 at 04:17:18PM +0300, Alexey Perevalov wrote: > > > >[...] > > > >>+/* > >>+ * This function calculates downtime per cpu and trace it > >>+ * > >>+ * Also it

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-25 Thread Alexey Perevalov
On 04/25/2017 11:24 AM, Peter Xu wrote: On Fri, Apr 14, 2017 at 04:17:18PM +0300, Alexey Perevalov wrote: [...] +/* + * This function calculates downtime per cpu and trace it + * + * Also it calculates total downtime as an interval's overlap, + * for many vCPU. + * + * The approach is

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-25 Thread Peter Xu
On Fri, Apr 14, 2017 at 04:17:18PM +0300, Alexey Perevalov wrote: [...] > +/* > + * This function calculates downtime per cpu and trace it > + * > + * Also it calculates total downtime as an interval's overlap, > + * for many vCPU. > + * > + * The approach is following: > + * Initially

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side (CPUMASK)

2017-04-24 Thread Dr. David Alan Gilbert
* Alexey (a.pereva...@samsung.com) wrote: > Hello David, > this mail just for CPUMASK discussion. > > On Fri, Apr 21, 2017 at 01:00:32PM +0100, Dr. David Alan Gilbert wrote: > > * Alexey Perevalov (a.pereva...@samsung.com) wrote: > > > This patch provides downtime calculation per vCPU, > > > as a

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-24 Thread Dr. David Alan Gilbert
* Alexey (a.pereva...@samsung.com) wrote: > Hello, David! > > > I apologize, forgot to check patches with checkpatch.pl script, but now I > checked, > and I fixed code styles in patches, however I checked also files, > migration.c has code style errors and glib-compat.h too. > I could send that

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side (CPUMASK)

2017-04-22 Thread Alexey
Hello David, this mail just for CPUMASK discussion. On Fri, Apr 21, 2017 at 01:00:32PM +0100, Dr. David Alan Gilbert wrote: > * Alexey Perevalov (a.pereva...@samsung.com) wrote: > > This patch provides downtime calculation per vCPU, > > as a summary and as a overlapped value for all vCPUs. > > >

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-21 Thread Alexey
Hello, David! I apologize, forgot to check patches with checkpatch.pl script, but now I checked, and I fixed code styles in patches, however I checked also files, migration.c has code style errors and glib-compat.h too. I could send that patches to qemu-trivial, if you not against. On Fri,

Re: [Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-21 Thread Dr. David Alan Gilbert
* Alexey Perevalov (a.pereva...@samsung.com) wrote: > This patch provides downtime calculation per vCPU, > as a summary and as a overlapped value for all vCPUs. > > This approach just keeps tree with page fault addr as a key, > and t1-t2 interval of pagefault time and page copy time, with >

[Qemu-devel] [PATCH 4/6] migration: calculate downtime on dst side

2017-04-14 Thread Alexey Perevalov
This patch provides downtime calculation per vCPU, as a summary and as a overlapped value for all vCPUs. This approach just keeps tree with page fault addr as a key, and t1-t2 interval of pagefault time and page copy time, with affected vCPU bit mask. For more implementation details please see