]> git.ipfire.org Git - thirdparty/qemu.git/commitdiff
migration: Make RAM_SAVE_FLAG_HOOK a normal case entry
authorJuan Quintela <quintela@redhat.com>
Thu, 4 May 2023 11:44:39 +0000 (13:44 +0200)
committerJuan Quintela <quintela@redhat.com>
Thu, 4 May 2023 23:04:32 +0000 (01:04 +0200)
Fixes this commit, clearly a bad merge after a rebase or similar, it
should have been its own case since that point.

commit 5b0e9dd46fbda5152566a4a26fd96bc0d0452bf7
Author: Peter Lieven <pl@kamp.de>
Date:   Tue Jun 24 11:32:36 2014 +0200

    migration: catch unknown flag combinations in ram_load

Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
Message-Id: <20230504114443.23891-2-quintela@redhat.com>

migration/ram.c

index 23ba1cefff59a69884387a4b2b1ab55e11679e0c..5e7bf20ca53eb22d05cbbfc8f83a396f4ffd5bd8 100644 (file)
@@ -4445,14 +4445,12 @@ static int ram_load_precopy(QEMUFile *f)
                 multifd_recv_sync_main();
             }
             break;
+        case RAM_SAVE_FLAG_HOOK:
+            ram_control_load_hook(f, RAM_CONTROL_HOOK, NULL);
+            break;
         default:
-            if (flags & RAM_SAVE_FLAG_HOOK) {
-                ram_control_load_hook(f, RAM_CONTROL_HOOK, NULL);
-            } else {
-                error_report("Unknown combination of migration flags: 0x%x",
-                             flags);
-                ret = -EINVAL;
-            }
+            error_report("Unknown combination of migration flags: 0x%x", flags);
+            ret = -EINVAL;
         }
         if (!ret) {
             ret = qemu_file_get_error(f);