[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/5] migration: Disable hotplug/unplug during mi
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 4/5] migration: Disable hotplug/unplug during migration |
Date: |
Thu, 6 Apr 2017 09:09:10 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 04/06/2017 08:13 AM, Juan Quintela wrote:
> Until we have reviewed what can/can't be hotplug during migration,
s/hotplug/hotplugged/
> disable it. We can enable it later for the things that we know that
> work. For instance, memory hotplug during postcopy don't work
s/don't/doesn't/
> currently.
>
> Signed-off-by: Juan Quintela <address@hidden>
>
> --
>
> @@ -603,6 +604,11 @@ DeviceState *qdev_device_add(QemuOpts *opts, Error
> **errp)
> return NULL;
> }
>
> + if (!migration_is_idle()) {
> + error_setg(errp, "device_add not allowed while migrating");
> + return NULL;
> + }
> +
> /* create device */
> dev = DEVICE(object_new(driver));
>
> @@ -853,6 +859,11 @@ void qdev_unplug(DeviceState *dev, Error **errp)
> return;
> }
>
> + if (!migration_is_idle()) {
> + error_setg(errp, "device_add not allowed while migrating");
s/device_add/device_del/ ?
> + return;
> + }
> +
> qdev_hot_removed = true;
>
> hotplug_ctrl = qdev_get_hotplug_handler(dev);
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH 1/5] qdev: qdev_hotplug is really a bool, (continued)
- [Qemu-devel] [PATCH 1/5] qdev: qdev_hotplug is really a bool, Juan Quintela, 2017/04/06
- [Qemu-devel] [PATCH 3/5] qdev: Move qdev_unplug() to qdev-monitor.c, Juan Quintela, 2017/04/06
- [Qemu-devel] [PATCH 2/5] qdev: Export qdev_hot_removed, Juan Quintela, 2017/04/06
- [Qemu-devel] [PATCH 4/5] migration: Disable hotplug/unplug during migration, Juan Quintela, 2017/04/06
- Re: [Qemu-devel] [PATCH 4/5] migration: Disable hotplug/unplug during migration,
Eric Blake <=
- [Qemu-devel] [PATCH 5/5] ram: Remove migration_bitmap_extend(), Juan Quintela, 2017/04/06
- Re: [Qemu-devel] [PATCH v2 0/5] Disable hotplug during migration, Hailiang Zhang, 2017/04/10