[PATCH] migration: Update comments mentioning qmp_migrate_finish

Fabiano Rosas posted 1 patch 4 days, 20 hours ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20260202124112.30684-1-farosas@suse.de
Maintainers: Peter Xu <peterx@redhat.com>, Fabiano Rosas <farosas@suse.de>
migration/migration.c | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
[PATCH] migration: Update comments mentioning qmp_migrate_finish
Posted by Fabiano Rosas 4 days, 20 hours ago
The qmp_migrate_finish function was recently removed in commit
2379a04e60 ("migration: Remove qmp_migrate_finish"), update comments
that mention it.

Signed-off-by: Fabiano Rosas <farosas@suse.de>
---
 migration/migration.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/migration/migration.c b/migration/migration.c
index b103a82fc0..1f5c289d68 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -1469,8 +1469,8 @@ void migration_cancel(void)
     }
 
     /*
-     * If qmp_migrate_finish has not been called, then there is no path that
-     * will complete the cancellation.  Do it now.
+     * If migration_connect_outgoing has not been called, then there
+     * is no path that will complete the cancellation. Do it now.
      */
     if (setup && !s->to_dst_file) {
         migrate_set_state(&s->state, MIGRATION_STATUS_CANCELLING,
@@ -2051,11 +2051,12 @@ void qmp_migrate(const char *uri, bool has_channels,
      * For cpr-transfer, the target may not be listening yet on the migration
      * channel, because first it must finish cpr_load_state.  The target tells
      * us it is listening by closing the cpr-state socket.  Wait for that HUP
-     * event before connecting in qmp_migrate_finish.
+     * event before connecting in migration_connect_outgoing.
      *
      * The HUP could occur because the target fails while reading CPR state,
      * in which case the target will not listen for the incoming migration
-     * connection, so qmp_migrate_finish will fail to connect, and then recover.
+     * connection, so migration_connect_outgoing will fail to connect,
+     * and then recover.
      */
     if (migrate_mode() == MIG_MODE_CPR_TRANSFER) {
         cpr_transfer_add_hup_watch(s, migration_connect_outgoing_cb,
-- 
2.51.0
Re: [PATCH] migration: Update comments mentioning qmp_migrate_finish
Posted by Peter Xu 4 days, 16 hours ago
On Mon, Feb 02, 2026 at 09:41:12AM -0300, Fabiano Rosas wrote:
> The qmp_migrate_finish function was recently removed in commit
> 2379a04e60 ("migration: Remove qmp_migrate_finish"), update comments
> that mention it.
> 
> Signed-off-by: Fabiano Rosas <farosas@suse.de>

Reviewed-by: Peter Xu <peterx@redhat.com>

-- 
Peter Xu