]> xenbits.xensource.com Git - people/aperard/qemu-dm.git/commitdiff
migration: Disable postcopy + multifd migration
authorLeonardo Bras <leobras@redhat.com>
Tue, 25 Apr 2023 03:33:08 +0000 (00:33 -0300)
committerJuan Quintela <quintela@redhat.com>
Thu, 27 Apr 2023 08:18:25 +0000 (10:18 +0200)
Since the introduction of multifd, it's possible to perform a multifd
migration and finish it using postcopy.

A bug introduced by yank (fixed on cfc3bcf373) was previously preventing
a successful use of this migration scenario, and now thing should be
working on most scenarios.

But since there is not enough testing/support nor any reported users for
this scenario, we should disable this combination before it may cause any
problems for users.

Suggested-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Leonardo Bras <leobras@redhat.com>
Acked-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
migration/options.c

index 8e8753d9be66e0112174936785cda50741d9f15f..dd97c99391aa3abd71f76fdf3bb68df8b91cff84 100644 (file)
@@ -322,6 +322,11 @@ bool migrate_caps_check(bool *old_caps, bool *new_caps, Error **errp)
             error_setg(errp, "Postcopy is not compatible with ignore-shared");
             return false;
         }
+
+        if (new_caps[MIGRATION_CAPABILITY_MULTIFD]) {
+            error_setg(errp, "Postcopy is not yet compatible with multifd");
+            return false;
+        }
     }
 
     if (new_caps[MIGRATION_CAPABILITY_BACKGROUND_SNAPSHOT]) {