commit 52d01d4a5d6963e24ff59af0c39b47fd5d229fb7
author Juan Quintela <quintela@redhat.com> Mon May 08 15:08:49 2023 +0200
committer Juan Quintela <quintela@redhat.com> Mon May 15 13:42:07 2023 +0200
tree f4f7a6152b2d2b99c659cb0dfac91ec96f6d64a9
parent dc2836c3806f004f36d423cedba86103c87a21a3

migration: A rate limit value of 0 is valid

And it is the best way to not have rate_limit.

Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Cédric Le Goater <clg@kaod.org>
Message-Id: <20230508130909.65420-2-quintela@redhat.com>

migration/migration.c

diff --git a/migration/migration.c b/migration/migration.c
index 439e865..5636119 100644
--- a/migration/migration.c
+++ b/migration/migration.c
@@ -2140,12 +2140,7 @@
      * will notice we're in POSTCOPY_ACTIVE and not actually
      * wrap their state up here
      */
-    /* 0 max-postcopy-bandwidth means unlimited */
-    if (!bandwidth) {
-        qemu_file_set_rate_limit(ms->to_dst_file, INT64_MAX);
-    } else {
-        qemu_file_set_rate_limit(ms->to_dst_file, bandwidth / XFER_LIMIT_RATIO);
-    }
+    qemu_file_set_rate_limit(ms->to_dst_file, bandwidth / XFER_LIMIT_RATIO);
     if (migrate_postcopy_ram()) {
         /* Ping just for debugging, helps line traces up */
         qemu_savevm_send_ping(ms->to_dst_file, 2);