Re: [Qemu-devel] [PATCH 09/11] migration: cleanup stats update into function

2018-01-03 Thread Peter Xu
On Wed, Jan 03, 2018 at 06:55:29PM +0800, Peter Xu wrote: > On Wed, Jan 03, 2018 at 11:08:49AM +0100, Juan Quintela wrote: > > Peter Xu wrote: > > > We have quite a few lines in migration_thread() that calculates some > > > statistics for the migration interations. Isolate it into a single > > >

Re: [Qemu-devel] [PATCH 09/11] migration: cleanup stats update into function

2018-01-03 Thread Peter Xu
On Wed, Jan 03, 2018 at 11:08:49AM +0100, Juan Quintela wrote: > Peter Xu wrote: > > We have quite a few lines in migration_thread() that calculates some > > statistics for the migration interations. Isolate it into a single > > function to improve readability. > > > > Signed-off-by: Peter Xu >

Re: [Qemu-devel] [PATCH 09/11] migration: cleanup stats update into function

2018-01-03 Thread Juan Quintela
Peter Xu wrote: > We have quite a few lines in migration_thread() that calculates some > statistics for the migration interations. Isolate it into a single > function to improve readability. > > Signed-off-by: Peter Xu > +static void migration_update_statistics(MigrationState *s, migration_

[Qemu-devel] [PATCH 09/11] migration: cleanup stats update into function

2018-01-02 Thread Peter Xu
We have quite a few lines in migration_thread() that calculates some statistics for the migration interations. Isolate it into a single function to improve readability. Signed-off-by: Peter Xu --- migration/migration.c | 82 +-- migration/migratio