Hi

Changes for v2:
- patches 1 & 2 in previous pull requset
- create function populate_disk_info (dave suggestion)

Please, review.

Thanks, Juan.

[v1]
This series make the output of info migrate be printed in a single
place for the states ACTIVE, CANCELLING and POSTCOPY_ACTIVE.

The code was duplicated, just use the same code.  Once done that, fix
the inconsistences that were not needed.


Juan Quintela (2):
  ram: Print block stats also in the complete case
  ram: Now POSTCOPY_ACTIVE is the same that STATUS_ACTIVE

 migration/migration.c | 43 +++++++++++++------------------------------
 1 file changed, 13 insertions(+), 30 deletions(-)

-- 
2.9.4

On Wed, Jun 07, 2017 at 12:58:08PM +0200, Juan Quintela wrote:
> Hi
> 
> Changes for v2:
> - patches 1 & 2 in previous pull requset
> - create function populate_disk_info (dave suggestion)
> 
> Please, review.
> 
> Thanks, Juan.
> 
> [v1]
> This series make the output of info migrate be printed in a single
> place for the states ACTIVE, CANCELLING and POSTCOPY_ACTIVE.
> 
> The code was duplicated, just use the same code.  Once done that, fix
> the inconsistences that were not needed.

I feel that putting cpu throttling info into populate_ram_info() is
still slightly weird, but hmm it's okay and that's trivial.

Reviewed-by: Peter Xu <peterx@redhat.com>

Thanks,

-- 
Peter Xu