Now that current_migration is static, there is no reason for max_throotle variable.
Signed-off-by: Juan Quintela <quint...@redhat.com> --- migration.c | 16 +++++++--------- 1 files changed, 7 insertions(+), 9 deletions(-) diff --git a/migration.c b/migration.c index 7b1e679..312a029 100644 --- a/migration.c +++ b/migration.c @@ -31,11 +31,10 @@ do { } while (0) #endif -/* Migration speed throttling */ -static int64_t max_throttle = (32 << 20); - static MigrationState current_migration = { .state = MIG_STATE_NONE, + /* Migration speed throttling */ + .bandwidth_limit = (32 << 20), }; static NotifierList migration_state_notifiers = @@ -364,13 +363,11 @@ void migrate_fd_connect(MigrationState *s) migrate_fd_put_ready(s); } -static void migrate_init_state(Monitor *mon, int64_t bandwidth_limit, - int detach, int blk, int inc) +static void migrate_init_state(Monitor *mon, int detach, int blk, int inc) { current_migration.blk = blk; current_migration.shared = inc; current_migration.mon = NULL; - current_migration.bandwidth_limit = bandwidth_limit; current_migration.state = MIG_STATE_NONE; if (!detach) { @@ -396,7 +393,7 @@ int do_migrate(Monitor *mon, const QDict *qdict, QObject **ret_data) return -1; } - migrate_init_state(mon, max_throttle, detach, blk, inc); + migrate_init_state(mon, detach, blk, inc); if (strstart(uri, "tcp:", &p)) { ret = tcp_start_outgoing_migration(¤t_migration, p); @@ -437,9 +434,10 @@ int do_migrate_set_speed(Monitor *mon, const QDict *qdict, QObject **ret_data) if (d < 0) { d = 0; } - max_throttle = d; + current_migration.bandwidth_limit = d; - qemu_file_set_rate_limit(current_migration.file, max_throttle); + qemu_file_set_rate_limit(current_migration.file, + current_migration.bandwidth_limit); return 0; } -- 1.7.4