diff options
author | zhanghailiang <zhang.zhanghailiang@huawei.com> | 2016-11-02 15:42:09 +0800 |
---|---|---|
committer | Juan Quintela <quintela@redhat.com> | 2016-11-14 14:50:56 +0100 |
commit | fe39a4d44004408b8c847151191f1948a4aa6570 (patch) | |
tree | b357078cea314e2676e7605f650536bb4f0a2b9a /hmp.c | |
parent | 83c83f9a5266ff113060f887f106a47920fa6974 (diff) |
migration: fix missing assignment for has_x_checkpoint_delay
We forgot to assign true to params->has_x_checkpoint_delay parameter
in qmp_query_migrate_parameters.
Without this, qmp command 'query-migrate-parameters' doesn't show the
default value for x-checkpoint-delay option.
This also fixes the fact that HMP was relying on unspecified behavior by
reading x_checkpoint_delay without checking has_x_checkpoint_delay.
Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'hmp.c')
-rw-r--r-- | hmp.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -318,6 +318,7 @@ void hmp_info_migrate_parameters(Monitor *mon, const QDict *qdict) monitor_printf(mon, " %s: %" PRId64 " milliseconds", MigrationParameter_lookup[MIGRATION_PARAMETER_DOWNTIME_LIMIT], params->downtime_limit); + assert(params->has_x_checkpoint_delay); monitor_printf(mon, " %s: %" PRId64, MigrationParameter_lookup[MIGRATION_PARAMETER_X_CHECKPOINT_DELAY], params->x_checkpoint_delay); |