diff options
| author | Peter Xu | 2021-12-07 12:50:14 +0100 |
|---|---|---|
| committer | Juan Quintela | 2022-01-28 15:38:23 +0100 |
| commit | 739fcc1b0e4b04216f971c9fb87fb0e9ec599c34 (patch) | |
| tree | 157db13e692337c4fd1d6e74d1f0c9847dac292f /migration/migration.c | |
| parent | migration: Do chunk page in postcopy_each_ram_send_discard() (diff) | |
| download | qemu-739fcc1b0e4b04216f971c9fb87fb0e9ec599c34.tar.gz qemu-739fcc1b0e4b04216f971c9fb87fb0e9ec599c34.tar.xz qemu-739fcc1b0e4b04216f971c9fb87fb0e9ec599c34.zip | |
migration: Drop return code for disgard ram process
It will just never fail. Drop those return values where they're constantly
zeros.
A tiny touch-up on the tracepoint so trace_ram_postcopy_send_discard_bitmap()
is called after the logic itself (which sounds more reasonable).
Signed-off-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@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>
Diffstat (limited to 'migration/migration.c')
| -rw-r--r-- | migration/migration.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/migration/migration.c b/migration/migration.c index 3849b33108..771f3e2bec 100644 --- a/migration/migration.c +++ b/migration/migration.c @@ -2991,10 +2991,7 @@ static int postcopy_start(MigrationState *ms) * that are dirty */ if (migrate_postcopy_ram()) { - if (ram_postcopy_send_discard_bitmap(ms)) { - error_report("postcopy send discard bitmap failed"); - goto fail; - } + ram_postcopy_send_discard_bitmap(ms); } /* |