[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 3/6] qemu: Block migration when transient disk option is e
From: |
Peter Krempa |
Subject: |
Re: [PATCH v3 3/6] qemu: Block migration when transient disk option is enabled |
Date: |
Fri, 18 Sep 2020 14:59:21 +0200 |
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Thu, Sep 17, 2020 at 09:30:42 -0400, Masayoshi Mizuma wrote:
> From: Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>
>
> Block migration when transient disk option is enabled because migration
> requires some blockjobs.
>
> Signed-off-by: Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>
> ---
> src/qemu/qemu_migration.c | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/src/qemu/qemu_migration.c b/src/qemu/qemu_migration.c
> index a530c17582..7316d74677 100644
> --- a/src/qemu/qemu_migration.c
> +++ b/src/qemu/qemu_migration.c
> @@ -1397,6 +1397,16 @@ qemuMigrationSrcIsAllowed(virQEMUDriverPtr driver,
> _("cannot migrate this domain without
> dbus-vmstate support"));
> return false;
> }
> +
> + for (i = 0; i < vm->def->ndisks; i++) {
> + virDomainDiskDefPtr disk = vm->def->disks[i];
> +
> + if (disk->transient) {
> + virReportError(VIR_ERR_OPERATION_INVALID, "%s",
> + _("migration with transient disk is not
> supported"));
transient disk '%s' is not supported
and use disk->dst as '%s'
> + return false;
> + }
> + }
Reviewed-by: Peter Krempa <pkrempa@redhat.com>
- [PATCH v3 0/6] qemu: implementation of transient disk option, Masayoshi Mizuma, 2020/09/17
- [PATCH v3 2/6] qemu: Block disk hotplug when transient disk option is enabled, Masayoshi Mizuma, 2020/09/17
- [PATCH v3 3/6] qemu: Block migration when transient disk option is enabled, Masayoshi Mizuma, 2020/09/17
- Re: [PATCH v3 3/6] qemu: Block migration when transient disk option is enabled,
Peter Krempa <=
- [PATCH v3 4/6] qemu: update the validation for transient disk option, Masayoshi Mizuma, 2020/09/17
- [PATCH v3 5/6] qemu: Introduce to handle transient disk, Masayoshi Mizuma, 2020/09/17
- [PATCH v3 6/6] qemu: Add transient disk handler to start and stop the guest, Masayoshi Mizuma, 2020/09/17