[Qemu-commits] [qemu/qemu] 741d40: migration: Use proper types in json

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 741d4086c856320807a2575389d7c0505578270b
      
https://github.com/qemu/qemu/commit/741d4086c856320807a2575389d7c0505578270b
  Author: Juan Quintela <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M hmp.c
    M migration/migration.c
    M qapi/migration.json

  Log Message:
  -----------
  migration: Use proper types in json

We use int for everything (int64_t), and then we check that value is
between 0 and 255.  Change it to the valid types.

This change only happens for HMP.  QMP always use bytes and similar.

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Eric Blake <address@hidden>


  Commit: 6f0f6428351d09dd904bf81b137bdd4fd25ba029
      
https://github.com/qemu/qemu/commit/6f0f6428351d09dd904bf81b137bdd4fd25ba029
  Author: Juan Quintela <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: print features as on off

Once there, do one thing for line

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Peter Xu <address@hidden>


  Commit: 9102d27e33b2a5b676ce3eb73b714c1f92062058
      
https://github.com/qemu/qemu/commit/9102d27e33b2a5b676ce3eb73b714c1f92062058
  Author: Juan Quintela <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/socket.c

  Log Message:
  -----------
  migration: free addr in the same function that we created it

Otherwise, we can't use it after calling socket_start_incoming_migration

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Peter Xu <address@hidden>


  Commit: 2e3c8f8dbddf2992e29533ded845640d109d77f3
      
https://github.com/qemu/qemu/commit/2e3c8f8dbddf2992e29533ded845640d109d77f3
  Author: Dr. David Alan Gilbert <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    A docs/devel/migration.rst
    R docs/devel/migration.txt

  Log Message:
  -----------
  docs: Convert migration.txt to rst

Mostly just manual conversion with very minor fixes.

Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Daniel P. Berrange <address@hidden>
Reviewed-by: Kashyap Chamarthy <address@hidden>
Reviewed-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 9c43435dc1f872a1452792fd8d9dbcf5c0482e46
      
https://github.com/qemu/qemu/commit/9c43435dc1f872a1452792fd8d9dbcf5c0482e46
  Author: Juan Quintela <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M tests/migration-test.c

  Log Message:
  -----------
  migration: free result string

Signed-off-by: Juan Quintela <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Reported-by: Peter Xu <address@hidden>


  Commit: 029ff892e1049ca4f93f9d8f0059bc02fffd6e32
      
https://github.com/qemu/qemu/commit/029ff892e1049ca4f93f9d8f0059bc02fffd6e32
  Author: Laurent Vivier <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M scripts/analyze-migration.py

  Log Message:
  -----------
  migration: fix analyze-migration.py script with radix table

Since commit 3a38429748 ("Add a "no HPT" encoding to HTAB migration stream")
the HTAB migration stream contains a header set to "-1", meaning there
is no HPT. Teach analyze-migration.py to ignore the section in this case.

Without this fix, the script fails with a dump from a POWER9 guest:

Traceback (most recent call last):
  File "./qemu/scripts/analyze-migration.py", line 602, in <module>
    dump.read(dump_memory = args.memory)
  File "./qemu/scripts/analyze-migration.py", line 539, in read
    section.read()
  File "./qemu/scripts/analyze-migration.py", line 250, in read
    self.file.readvar(n_valid * self.HASH_PTE_SIZE_64)
  File "./qemu/scripts/analyze-migration.py", line 64, in readvar
    raise Exception("Unexpected end of %s at 0x%x" % (self.filename, 
self.file.tell()))
Exception: Unexpected end of migrate.dump at 0x1d4763ba

Fixes: 3a38429748 ("Add a "no HPT" encoding to HTAB migration stream")
Signed-off-by: Laurent Vivier <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Greg Kurz <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 31bf06a9d6844d9977b028bd9b855d53288f979c
      
https://github.com/qemu/qemu/commit/31bf06a9d6844d9977b028bd9b855d53288f979c
  Author: Alexey Perevalov <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c
    M migration/migration.h
    M qapi/migration.json

  Log Message:
  -----------
  migration: introduce postcopy-blocktime capability

Right now it could be used on destination side to
enable vCPU blocktime calculation for postcopy live migration.
vCPU blocktime - it's time since vCPU thread was put into
interruptible sleep, till memory page was copied and thread awake.

Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Alexey Perevalov <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 01a87f0bd38b06ff92e0fc08ecf54eefab6d9ca8
      
https://github.com/qemu/qemu/commit/01a87f0bd38b06ff92e0fc08ecf54eefab6d9ca8
  Author: Alexey Perevalov <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.h
    M migration/postcopy-ram.c

  Log Message:
  -----------
  migration: add postcopy blocktime ctx into MigrationIncomingState

This patch adds request to kernel space for UFFD_FEATURE_THREAD_ID, in
case this feature is provided by kernel.

PostcopyBlocktimeContext is encapsulated inside postcopy-ram.c,
due to it being a postcopy-only feature.
Also it defines PostcopyBlocktimeContext's instance live time.
Information from PostcopyBlocktimeContext instance will be provided
much after postcopy migration end, instance of PostcopyBlocktimeContext
will live till QEMU exit, but part of it (vcpu_addr,
page_fault_vcpu_time) used only during calculation, will be released
when postcopy ended or failed.

To enable postcopy blocktime calculation on destination, need to
request proper compatibility (Patch for documentation will be at the
tail of the patch set).

As an example following command enable that capability, assume QEMU was
started with
-chardev socket,id=charmonitor,path=/var/lib/migrate-vm-monitor.sock
option to control it

address@hidden "{\"execute\" : \"qmp_capabilities\"}\r\n \
{\"execute\": \"migrate-set-capabilities\" , \"arguments\":   {
\"capabilities\": [ { \"capability\": \"postcopy-blocktime\", \"state\":
true } ] } }" | nc -U /var/lib/migrate-vm-monitor.sock

Or just with HMP
(qemu) migrate_set_capability postcopy-blocktime on

Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Alexey Perevalov <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 3be98be4e9f59055afb5f2b27f9296c7093b4e75
      
https://github.com/qemu/qemu/commit/3be98be4e9f59055afb5f2b27f9296c7093b4e75
  Author: Alexey Perevalov <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/postcopy-ram.c
    M migration/trace-events

  Log Message:
  -----------
  migration: calculate vCPU blocktime on dst side

This patch provides blocktime calculation per vCPU,
as a summary and as a overlapped value for all vCPUs.

This approach was suggested by Peter Xu, as an improvements of
previous approch where QEMU kept tree with faulted page address and cpus bitmask
in it. Now QEMU is keeping array with faulted page address as value and vCPU
as index. It helps to find proper vCPU at UFFD_COPY time. Also it keeps
list for blocktime per vCPU (could be traced with page_fault_addr)

Blocktime will not calculated if postcopy_blocktime field of
MigrationIncomingState wasn't initialized.

Signed-off-by: Alexey Perevalov <address@hidden>
Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 2f7dae9dd3b5bb436f6089816cf24f4d0867d32e
      
https://github.com/qemu/qemu/commit/2f7dae9dd3b5bb436f6089816cf24f4d0867d32e
  Author: Alexey Perevalov <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M docs/devel/migration.rst

  Log Message:
  -----------
  migration: postcopy_blocktime documentation

Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Alexey Perevalov <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 5f32dc8ee073d250abd9e8915f6884ba20d5de62
      
https://github.com/qemu/qemu/commit/5f32dc8ee073d250abd9e8915f6884ba20d5de62
  Author: Alexey Perevalov <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M tests/migration-test.c

  Log Message:
  -----------
  migration: add blocktime calculation into migration-test

This patch just requests blocktime calculation,
and check it in case when UFFD_FEATURE_THREAD_ID feature is set
on the host.

Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Alexey Perevalov <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: ca6011c2329121ecc25081f884a5d9016acc6572
      
https://github.com/qemu/qemu/commit/ca6011c2329121ecc25081f884a5d9016acc6572
  Author: Alexey Perevalov <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M hmp.c
    M migration/migration.c
    M migration/migration.h
    M migration/postcopy-ram.c
    M migration/trace-events
    M qapi/migration.json

  Log Message:
  -----------
  migration: add postcopy total blocktime into query-migrate

Postcopy total blocktime is available on destination side only.
But query-migrate was possible only for source. This patch
adds ability to call query-migrate on destination.
To be able to see postcopy blocktime, need to request postcopy-blocktime
capability.

The query-migrate command will show following sample result:
{"return":
    "postcopy-vcpu-blocktime": [115, 100],
    "status": "completed",
    "postcopy-blocktime": 100
}}

postcopy_vcpu_blocktime contains list, where the first item is the first
vCPU in QEMU.

This patch has a drawback, it combines states of incoming and
outgoing migration. Ongoing migration state will overwrite incoming
state. Looks like better to separate query-migrate for incoming and
outgoing migration or add parameter to indicate type of migration.

Reviewed-by: Dr. David Alan Gilbert <address@hidden>
Signed-off-by: Alexey Perevalov <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: bae416e5ba65701d3c5238164517158066d615e5
      
https://github.com/qemu/qemu/commit/bae416e5ba65701d3c5238164517158066d615e5
  Author: Dr. David Alan Gilbert <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/ram.c

  Log Message:
  -----------
  migration: Guard ram_bytes_remaining against early call

Calling ram_bytes_remaining during the early part of setup is unsafe
because the ram_state isn't yet initialised.

This can happen in the sequence:
   migrate
   migrate_cancel
   info migrate

if the migrate sticks trying to connect (e.g. to an unresponsive
destination due to the connect timeout).  Here 'info migrate' sees
a state of CANCELLING and so assumes the migrate has partially happened.

partial fix for:
RH bz: https://bugzilla.redhat.com/show_bug.cgi?id=1525899
Reported-by: Xianxian Wang <address@hidden>
Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 1f8956041ad3851c28f0e3f6ee023d3bcc55ca50
      
https://github.com/qemu/qemu/commit/1f8956041ad3851c28f0e3f6ee023d3bcc55ca50
  Author: Vladimir Sementsov-Ogievskiy <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M include/migration/misc.h
    M migration/migration.c
    M vl.c

  Log Message:
  -----------
  migration: finalize current_migration object

current_migration has .instance_finalize callback, but it is not
called, because nobody unrefs current_migration. Fix that.

Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
Reviewed-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 0d649a0e95b46fab94090f852a43c72d2621ebb9
      
https://github.com/qemu/qemu/commit/0d649a0e95b46fab94090f852a43c72d2621ebb9
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: assert colo instead of check

When reaching here if we are still "active" it means we must be in colo
state.  After a quick discussion offlist, we decided to use the safer
error_report().

Finally I want to use "switch" here rather than lots of complicated if
clauses.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 0ceccd858a8d2d462109c2e0e0ca51861eebdd80
      
https://github.com/qemu/qemu/commit/0ceccd858a8d2d462109c2e0e0ca51861eebdd80
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: qemu_savevm_state_cleanup() in cleanup

Moving existing callers all into migrate_fd_cleanup().  It simplifies
migration_thread() a bit.

Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: deb74fb6701755a4f864977608ea3919e4970fae
      
https://github.com/qemu/qemu/commit/deb74fb6701755a4f864977608ea3919e4970fae
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: remove "enable_colo" var

It's only used once, clean it up a bit.

Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 4af246a34e3155365e6be9dda201f3467d0498d9
      
https://github.com/qemu/qemu/commit/4af246a34e3155365e6be9dda201f3467d0498d9
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c
    M migration/migration.h

  Log Message:
  -----------
  migration: split use of MigrationState.total_time

It was used either to:

1. store initial timestamp of migration start, and
2. store total time used by last migration

Let's provide two parameters for each of them.  Mix use of the two is
slightly misleading.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 7287cbd46e2d6fb582ca78c3cb49b1e53d91a761
      
https://github.com/qemu/qemu/commit/7287cbd46e2d6fb582ca78c3cb49b1e53d91a761
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c
    M migration/migration.h

  Log Message:
  -----------
  migration: move vm_old_running into global state

Firstly, it was passed around.  Let's just move it into MigrationState
just like many other variables as state of migration, renaming it to
vm_was_running.

One thing to mention is that for postcopy, we actually don't need this
knowledge at all since postcopy can't resume a VM even if it fails (we
can see that from the old code too: when we try to resume we also check
against "entered_postcopy" variable).  So further we do this:

- in postcopy_start(), we don't update vm_old_running since useless
- in migration_thread(), we don't need to check entered_postcopy when
  resume, since it's only used for precopy.

Comment this out too for that variable definition.

Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 64909f974012038266a4785a33935742f1e96b84
      
https://github.com/qemu/qemu/commit/64909f974012038266a4785a33935742f1e96b84
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c
    M migration/migration.h

  Log Message:
  -----------
  migration: introduce downtime_start

Introduce MigrationState.downtime_start to replace the local variable
"start_time" in migration_thread to avoid passing things around.

Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: cf011f082d7a2a9abfc0a257cc4de1cfcfbef24d
      
https://github.com/qemu/qemu/commit/cf011f082d7a2a9abfc0a257cc4de1cfcfbef24d
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: introduce migrate_calculate_complete

Generalize the calculation part when migration complete into a
function to simplify migration_thread().

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 39b9e17905c5ff8302c0ff17af546cbf68e0b4f6
      
https://github.com/qemu/qemu/commit/39b9e17905c5ff8302c0ff17af546cbf68e0b4f6
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: use switch at the end of migration

It converts the old if clauses into switch, explicitly mentions the
possible migration states.  The old nested "if"s are not clear on what
we do on different states.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: b15df1ae5063c7c181f8f068f9eba7661b3b5e1c
      
https://github.com/qemu/qemu/commit/b15df1ae5063c7c181f8f068f9eba7661b3b5e1c
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c
    M migration/migration.h

  Log Message:
  -----------
  migration: cleanup stats update into function

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 <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 2ad873057e0d7f227ed8a18f74db6f0433b32e74
      
https://github.com/qemu/qemu/commit/2ad873057e0d7f227ed8a18f74db6f0433b32e74
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: major cleanup for migrate iterations

The major work for migration iterations are to move RAM/block/... data
via qemu_savevm_state_iterate().  Generalize those part into a single
function.

Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 199aa6d4e445b863cc0e373496b9d0900437d8a4
      
https://github.com/qemu/qemu/commit/199aa6d4e445b863cc0e373496b9d0900437d8a4
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: put the finish part into a new function

This patch only moved the last part of migration_thread() into a new
function migration_iteration_finish() to make it much shorter.  With
previous works to remove some local variables, now it's fairly easy to
do that.

Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 26978faf2faa4e7688622694ca787a8abc691147
      
https://github.com/qemu/qemu/commit/26978faf2faa4e7688622694ca787a8abc691147
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: remove some block_cleanup_parameters()

Keep the one in migrate_fd_cleanup() would be enough.  Removing the other
two.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: 816306826a45f4d15352e32d157172af3a35899f
      
https://github.com/qemu/qemu/commit/816306826a45f4d15352e32d157172af3a35899f
  Author: Peter Xu <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    M migration/migration.c

  Log Message:
  -----------
  migration: remove notify in fd_error

It is already called in migrate_fd_cleanup.

Signed-off-by: Peter Xu <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
Signed-off-by: Juan Quintela <address@hidden>


  Commit: f521eeee3bd060b460c99e605472b7e03967db43
      
https://github.com/qemu/qemu/commit/f521eeee3bd060b460c99e605472b7e03967db43
  Author: Peter Maydell <address@hidden>
  Date:   2018-01-15 (Mon, 15 Jan 2018)

  Changed paths:
    A docs/devel/migration.rst
    R docs/devel/migration.txt
    M hmp.c
    M include/migration/misc.h
    M migration/migration.c
    M migration/migration.h
    M migration/postcopy-ram.c
    M migration/ram.c
    M migration/socket.c
    M migration/trace-events
    M qapi/migration.json
    M scripts/analyze-migration.py
    M tests/migration-test.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/juanquintela/tags/migration/20180115' 
into staging

migration/next for 20180115

# gpg: Signature made Mon 15 Jan 2018 11:51:00 GMT
# gpg:                using RSA key 0xF487EF185872D723
# gpg: Good signature from "Juan Quintela <address@hidden>"
# gpg:                 aka "Juan Quintela <address@hidden>"
# Primary key fingerprint: 1899 FF8E DEBF 58CC EE03  4B82 F487 EF18 5872 D723

* remotes/juanquintela/tags/migration/20180115: (27 commits)
  migration: remove notify in fd_error
  migration: remove some block_cleanup_parameters()
  migration: put the finish part into a new function
  migration: major cleanup for migrate iterations
  migration: cleanup stats update into function
  migration: use switch at the end of migration
  migration: introduce migrate_calculate_complete
  migration: introduce downtime_start
  migration: move vm_old_running into global state
  migration: split use of MigrationState.total_time
  migration: remove "enable_colo" var
  migration: qemu_savevm_state_cleanup() in cleanup
  migration: assert colo instead of check
  migration: finalize current_migration object
  migration: Guard ram_bytes_remaining against early call
  migration: add postcopy total blocktime into query-migrate
  migration: add blocktime calculation into migration-test
  migration: postcopy_blocktime documentation
  migration: calculate vCPU blocktime on dst side
  migration: add postcopy blocktime ctx into MigrationIncomingState
  ...

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/fd06527b80c8...f521eeee3bd0