From 3dc853832db7bf8ac3878e0666c639d3bc02b587 Mon Sep 17 00:00:00 2001 From: Luiz Capitulino Date: Mon, 28 Nov 2011 11:59:37 -0200 Subject: [PATCH] qapi: Convert migrate_set_speed Signed-off-by: Luiz Capitulino --- hmp-commands.hx | 3 +-- hmp.c | 6 ++++++ hmp.h | 1 + migration.c | 12 ++++-------- migration.h | 2 -- qapi-schema.json | 15 +++++++++++++++ qmp-commands.hx | 5 +---- 7 files changed, 28 insertions(+), 16 deletions(-) diff --git a/hmp-commands.hx b/hmp-commands.hx index 6ba694d45a..fdbed15ca9 100644 --- a/hmp-commands.hx +++ b/hmp-commands.hx @@ -786,8 +786,7 @@ ETEXI .params = "value", .help = "set maximum speed (in bytes) for migrations. " "Defaults to MB if no size suffix is specified, ie. B/K/M/G/T", - .user_print = monitor_user_noop, - .mhandler.cmd_new = do_migrate_set_speed, + .mhandler.cmd = hmp_migrate_set_speed, }, STEXI diff --git a/hmp.c b/hmp.c index 04e6b73810..e7659d5ffc 100644 --- a/hmp.c +++ b/hmp.c @@ -673,3 +673,9 @@ void hmp_migrate_set_downtime(Monitor *mon, const QDict *qdict) double value = qdict_get_double(qdict, "value"); qmp_migrate_set_downtime(value, NULL); } + +void hmp_migrate_set_speed(Monitor *mon, const QDict *qdict) +{ + int64_t value = qdict_get_int(qdict, "value"); + qmp_migrate_set_speed(value, NULL); +} diff --git a/hmp.h b/hmp.h index 9182622cc0..093242d626 100644 --- a/hmp.h +++ b/hmp.h @@ -48,5 +48,6 @@ void hmp_block_resize(Monitor *mon, const QDict *qdict); void hmp_snapshot_blkdev(Monitor *mon, const QDict *qdict); void hmp_migrate_cancel(Monitor *mon, const QDict *qdict); void hmp_migrate_set_downtime(Monitor *mon, const QDict *qdict); +void hmp_migrate_set_speed(Monitor *mon, const QDict *qdict); #endif diff --git a/migration.c b/migration.c index a5631622f0..75d4f02373 100644 --- a/migration.c +++ b/migration.c @@ -473,21 +473,17 @@ void qmp_migrate_cancel(Error **errp) migrate_fd_cancel(migrate_get_current()); } -int do_migrate_set_speed(Monitor *mon, const QDict *qdict, QObject **ret_data) +void qmp_migrate_set_speed(int64_t value, Error **errp) { - int64_t d; MigrationState *s; - d = qdict_get_int(qdict, "value"); - if (d < 0) { - d = 0; + if (value < 0) { + value = 0; } s = migrate_get_current(); - s->bandwidth_limit = d; + s->bandwidth_limit = value; qemu_file_set_rate_limit(s->file, s->bandwidth_limit); - - return 0; } void qmp_migrate_set_downtime(double value, Error **errp) diff --git a/migration.h b/migration.h index 539f8f1840..999d60feb5 100644 --- a/migration.h +++ b/migration.h @@ -42,8 +42,6 @@ int qemu_start_incoming_migration(const char *uri); int do_migrate(Monitor *mon, const QDict *qdict, QObject **ret_data); -int do_migrate_set_speed(Monitor *mon, const QDict *qdict, QObject **ret_data); - uint64_t migrate_max_downtime(void); void do_info_migrate_print(Monitor *mon, const QObject *data); diff --git a/qapi-schema.json b/qapi-schema.json index d9b9fa10d9..f358b490b0 100644 --- a/qapi-schema.json +++ b/qapi-schema.json @@ -1153,3 +1153,18 @@ # Since: 0.14.0 ## { 'command': 'migrate_set_downtime', 'data': {'value': 'number'} } + +## +# @migrate_set_speed +# +# Set maximum speed for migration. +# +# @value: maximum speed in bytes. +# +# Returns: nothing on success +# +# Notes: A value lesser than zero will be automatically round up to zero. +# +# Since: 0.14.0 +## +{ 'command': 'migrate_set_speed', 'data': {'value': 'int'} } diff --git a/qmp-commands.hx b/qmp-commands.hx index b4cadebe96..002e7e8bd9 100644 --- a/qmp-commands.hx +++ b/qmp-commands.hx @@ -492,10 +492,7 @@ EQMP { .name = "migrate_set_speed", .args_type = "value:o", - .params = "value", - .help = "set maximum speed (in bytes) for migrations", - .user_print = monitor_user_noop, - .mhandler.cmd_new = do_migrate_set_speed, + .mhandler.cmd_new = qmp_marshal_input_migrate_set_speed, }, SQMP