diff options
| author | Fabiano Rosas <farosas@suse.de> | 2025-02-13 14:59:19 -0300 |
|---|---|---|
| committer | Fabiano Rosas <farosas@suse.de> | 2025-02-14 15:19:05 -0300 |
| commit | a47f0cfba8d33f7001bcc4616f96e42dbd553135 (patch) | |
| tree | a0e19c2be012aeb1ce67e49c5a8de4c19c168a1a /migration/migration.c | |
| parent | e0ad300fe1a0e0b12b90994bab6e4df77dd1ee8a (diff) | |
| download | focaccia-qemu-a47f0cfba8d33f7001bcc4616f96e42dbd553135.tar.gz focaccia-qemu-a47f0cfba8d33f7001bcc4616f96e42dbd553135.zip | |
migration: Set migration error outside of migrate_cancel
There's no point passing the error into migration cancel only for it to call migrate_set_error(). Reviewed-by: Peter Xu <peterx@redhat.com> Message-ID: <20250213175927.19642-2-farosas@suse.de> Signed-off-by: Fabiano Rosas <farosas@suse.de>
Diffstat (limited to 'migration/migration.c')
| -rw-r--r-- | migration/migration.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/migration/migration.c b/migration/migration.c index 396928513a..7728f52aef 100644 --- a/migration/migration.c +++ b/migration/migration.c @@ -342,11 +342,8 @@ void migration_bh_schedule(QEMUBHFunc *cb, void *opaque) qemu_bh_schedule(bh); } -void migration_cancel(const Error *error) +void migration_cancel() { - if (error) { - migrate_set_error(current_migration, error); - } if (migrate_dirty_limit()) { qmp_cancel_vcpu_dirty_limit(false, -1, NULL); } @@ -365,7 +362,7 @@ void migration_shutdown(void) * Cancel the current migration - that will (eventually) * stop the migration using this structure */ - migration_cancel(NULL); + migration_cancel(); object_unref(OBJECT(current_migration)); /* |