[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 4/6] migration: Remove block migration
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 4/6] migration: Remove block migration |
Date: |
Fri, 26 Apr 2024 16:15:31 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) |
Fabiano Rosas <farosas@suse.de> writes:
> The block migration has been considered obsolete since QEMU 8.2 in
> favor of the more flexible storage migration provided by the
> blockdev-mirror driver. Two releases have passed so now it's time to
> remove it.
>
> Deprecation commit 66db46ca83 ("migration: Deprecate block
> migration").
>
> Signed-off-by: Fabiano Rosas <farosas@suse.de>
[...]
> diff --git a/migration/migration.c b/migration/migration.c
> index a3dc8a7974..0f4df893e5 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
[...]
> @@ -1997,8 +1983,6 @@ static bool migrate_prepare(MigrationState *s, bool
> resume, Error **errp)
> }
> }
>
> - s->must_remove_block_options = true;
> -
> if (migrate_init(s, errp)) {
> return false;
> }
> @@ -2080,7 +2064,6 @@ void qmp_migrate(const char *uri, bool has_channels,
> "a valid migration protocol");
> migrate_set_state(&s->state, MIGRATION_STATUS_SETUP,
> MIGRATION_STATUS_FAILED);
> - block_cleanup_parameters();
> }
>
> if (local_err) {
> diff --git a/migration/options.c b/migration/options.c
> index 638eeeb9a0..5049bfb78e 100644
> --- a/migration/options.c
> +++ b/migration/options.c
[...]
> @@ -942,17 +917,6 @@ ZeroPageDetection migrate_zero_page_detection(void)
>
> /* parameters helpers */
>
> -void block_cleanup_parameters(void)
> -{
> - MigrationState *s = migrate_get_current();
> -
> - if (s->must_remove_block_options) {
> - /* setting to false can never fail */
> - migrate_cap_set(MIGRATION_CAPABILITY_BLOCK, false, &error_abort);
> - s->must_remove_block_options = false;
> - }
> -}
> -
MigrationState member @must_remove_block_options is now unused. Please
delete it.
> AnnounceParameters *migrate_announce_params(void)
> {
> static AnnounceParameters ap;
[...]
With that:
Reviewed-by: Markus Armbruster <armbru@redhat.com>
- [PATCH 1/6] migration: Remove 'skipped' field from MigrationStats, (continued)
- [PATCH 4/6] migration: Remove block migration, Fabiano Rosas, 2024/04/25
- Re: [PATCH 4/6] migration: Remove block migration,
Markus Armbruster <=
- [PATCH 6/6] migration: Deprecate fd: for file migration, Fabiano Rosas, 2024/04/25
- [PATCH 5/6] migration: Remove non-multifd compression, Fabiano Rosas, 2024/04/25
- Re: [PATCH 0/6] migration removals & deprecations, Markus Armbruster, 2024/04/26