diff options
author | Peter Xu <peterx@redhat.com> | 2024-10-24 17:30:53 -0400 |
---|---|---|
committer | Peter Xu <peterx@redhat.com> | 2024-10-31 15:48:18 -0400 |
commit | 34a8892decdcec13d641f57bcb2a9c35fbc4c07b (patch) | |
tree | 5a057942e618bc129cf61cc62d056ea615740289 /hw/virtio | |
parent | f018eb62b2bb7618b0cb8a796ac4b76fd786ecaf (diff) | |
download | qemu-34a8892decdcec13d641f57bcb2a9c35fbc4c07b.zip qemu-34a8892decdcec13d641f57bcb2a9c35fbc4c07b.tar.gz qemu-34a8892decdcec13d641f57bcb2a9c35fbc4c07b.tar.bz2 |
migration: Drop migration_is_idle()
Now with the current migration_is_running(), it will report exactly the
opposite of what will be reported by migration_is_idle().
Drop migration_is_idle(), instead use "!migration_is_running()" which
should be identical on functionality.
In reality, most of the idle check is inverted, so it's even easier to
write with "migrate_is_running()" check.
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Reviewed-by: Fabiano Rosas <farosas@suse.de>
Link: https://lore.kernel.org/r/20241024213056.1395400-6-peterx@redhat.com
Signed-off-by: Peter Xu <peterx@redhat.com>
Diffstat (limited to 'hw/virtio')
-rw-r--r-- | hw/virtio/virtio-mem.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/hw/virtio/virtio-mem.c b/hw/virtio/virtio-mem.c index ae1e81d..80ada89 100644 --- a/hw/virtio/virtio-mem.c +++ b/hw/virtio/virtio-mem.c @@ -188,7 +188,7 @@ static bool virtio_mem_is_busy(void) * after plugging them) until we're running on the destination (as we didn't * migrate these blocks when they were unplugged). */ - return migration_in_incoming_postcopy() || !migration_is_idle(); + return migration_in_incoming_postcopy() || migration_is_running(); } typedef int (*virtio_mem_range_cb)(VirtIOMEM *vmem, void *arg, |