savevm: fail if migration blockers are present
authorGreg Kurz <gkurz@linux.vnet.ibm.com>
Wed, 4 May 2016 19:44:19 +0000 (21:44 +0200)
committerAmit Shah <amit.shah@redhat.com>
Mon, 23 May 2016 16:14:08 +0000 (21:44 +0530)
commit24f3902b088cd4f2dbebfd90527b5d81d6a050e9
tree86a48cceddb52cbf944f4d05ab3ff30d759cd7d1
parentd85a31d1f4d45462b7ca333190b5d14fad982f3e
savevm: fail if migration blockers are present

QEMU has currently two ways to prevent migration to occur:
- migration blocker when it depends on runtime state
- VMStateDescription.unmigratable when migration is not supported at all

This patch gathers all the logic into a single function to be called from
both the savevm and the migrate paths.

This fixes a bug with 9p, at least, where savevm would succeed and the
following would happen in the guest after loadvm:

$ ls /host
ls: cannot access /host: Protocol error

With this patch:

(qemu) savevm foo
Migration is disabled when VirtFS export path '/' is mounted in the guest
using mount_tag 'host'

Signed-off-by: Greg Kurz <gkurz@linux.vnet.ibm.com>
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Message-Id: <146239057139.11271.9011797645454781543.stgit@bahia.huguette.org>

[Update subject according to Paolo's suggestion - Amit]

Signed-off-by: Amit Shah <amit.shah@redhat.com>
include/migration/migration.h
migration/migration.c
migration/savevm.c