[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] block migration: Allow compile time disable
From: |
Juan Quintela |
Subject: |
Re: [Qemu-devel] [PATCH] block migration: Allow compile time disable |
Date: |
Wed, 03 May 2017 20:06:28 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
"Dr. David Alan Gilbert (git)" <address@hidden> wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> Many users now prefer to use drive_mirror over NBD as an
> alternative to the older migrate -b option; drive_mirror is
> more complex to setup but gives you more options (e.g. only
> migrating some of the disks if some of them are shared).
>
> Allow the large chunk of block migration code to be compiled
> out for those who don't use it.
>
> Based on a downstream-patch we've had for a while by Jeff Cody.
>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Reviewed-by: Juan Quintela <address@hidden>
> diff --git a/migration/migration.c b/migration/migration.c
> index 353f2728cf..ffce72aabc 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -692,6 +692,7 @@ MigrationInfo *qmp_query_migrate(Error **errp)
>
> populate_ram_info(info, s);
>
> +#ifdef CONFIG_LIVE_BLOCK_MIGRATION
> if (blk_mig_active()) {
> info->has_disk = true;
> info->disk = g_malloc0(sizeof(*info->disk));
> @@ -699,6 +700,7 @@ MigrationInfo *qmp_query_migrate(Error **errp)
> info->disk->remaining = blk_mig_bytes_remaining();
> info->disk->total = blk_mig_bytes_total();
> }
> +#endif
I think it can be cleaner to move this inside block.c and just export a
function that is empty in case it is compiled out, but that it is just
me, so you got the revieweb by anyways.
> }
> }
>
> +#ifdef CONFIG_LIVE_BLOCK_MIGRATION
> blk_mig_init();
> +#endif
> ram_mig_init();
>
> /* If the currently selected machine wishes to override the units-per-bus
Same here, I preffer the ifdef in the header than here.
Later, Juan.