[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] savevm: check vmsd for migratability status (v2
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH] savevm: check vmsd for migratability status (v2) |
Date: |
Wed, 30 Apr 2014 12:47:50 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
(CCing Juan)
On Tue, Apr 29, 2014 at 09:39:03PM -0300, Marcelo Tosatti wrote:
> Check vmsd for unmigratable field, allowing migratibility status
> to be modified after vmstate_register.
>
> Signed-off-by: Marcelo Tosatti <address@hidden>
Reviewed-by: Eduardo Habkost <address@hidden>
>
> diff --git a/savevm.c b/savevm.c
> index da8aa24..c578e42 100644
> --- a/savevm.c
> +++ b/savevm.c
> @@ -232,7 +232,6 @@ typedef struct SaveStateEntry {
> const VMStateDescription *vmsd;
> void *opaque;
> CompatEntry *compat;
> - int no_migrate;
> int is_ram;
> } SaveStateEntry;
>
> @@ -292,7 +291,6 @@ int register_savevm_live(DeviceState *dev,
> se->ops = ops;
> se->opaque = opaque;
> se->vmsd = NULL;
> - se->no_migrate = 0;
> /* if this is a live_savem then set is_ram */
> if (ops->save_live_setup != NULL) {
> se->is_ram = 1;
> @@ -383,7 +381,6 @@ int vmstate_register_with_alias_id(DeviceState *dev, int
> instance_id,
> se->opaque = opaque;
> se->vmsd = vmsd;
> se->alias_id = alias_id;
> - se->no_migrate = vmsd->unmigratable;
>
> if (dev) {
> char *id = qdev_get_dev_path(dev);
> @@ -452,7 +449,7 @@ bool qemu_savevm_state_blocked(Error **errp)
> SaveStateEntry *se;
>
> QTAILQ_FOREACH(se, &savevm_handlers, entry) {
> - if (se->no_migrate) {
> + if (se->vmsd && se->vmsd->unmigratable) {
> error_setg(errp, "State blocked by non-migratable device '%s'",
> se->idstr);
> return true;
>
--
Eduardo
- [Qemu-devel] [patch 2/2] target-i386: block migration and savevm if invariant tsc is exposed, (continued)
- [Qemu-devel] [patch 2/2] target-i386: block migration and savevm if invariant tsc is exposed, Marcelo Tosatti, 2014/04/23
- Re: [Qemu-devel] [patch 2/2] target-i386: block migration and savevm if invariant tsc is exposed, Eduardo Habkost, 2014/04/23
- [Qemu-devel] target-i386: block migration and savevm if invariant tsc is exposed (v3), Marcelo Tosatti, 2014/04/23
- Re: [Qemu-devel] target-i386: block migration and savevm if invariant tsc is exposed (v3), Eduardo Habkost, 2014/04/24
- Re: [Qemu-devel] target-i386: block migration and savevm if invariant tsc is exposed (v3), Marcelo Tosatti, 2014/04/25
- Re: [Qemu-devel] target-i386: block migration and savevm if invariant tsc is exposed (v3), Eduardo Habkost, 2014/04/25
- Re: [Qemu-devel] target-i386: block migration and savevm if invariant tsc is exposed (v3), Marcelo Tosatti, 2014/04/25
- [Qemu-devel] [PATCH] savevm: check vmsd for migratability status, Marcelo Tosatti, 2014/04/25
- Re: [Qemu-devel] [PATCH] savevm: check vmsd for migratability status, Eduardo Habkost, 2014/04/28
- [Qemu-devel] [PATCH] savevm: check vmsd for migratability status (v2), Marcelo Tosatti, 2014/04/29
- Re: [Qemu-devel] [PATCH] savevm: check vmsd for migratability status (v2),
Eduardo Habkost <=