[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v2 7/8] monitor: Use argument type 'T' for migrate_s
From: |
Markus Armbruster |
Subject: |
[Qemu-devel] [PATCH v2 7/8] monitor: Use argument type 'T' for migrate_set_downtime() |
Date: |
Wed, 20 Jan 2010 17:08:21 +0100 |
Before, it used type 's', which strips quotes and interprets escapes,
and is quite inappropriate for QMP.
Signed-off-by: Markus Armbruster <address@hidden>
---
migration.c | 15 ++-------------
qemu-monitor.hx | 2 +-
2 files changed, 3 insertions(+), 14 deletions(-)
diff --git a/migration.c b/migration.c
index e8802ba..c7763f6 100644
--- a/migration.c
+++ b/migration.c
@@ -134,21 +134,10 @@ uint64_t migrate_max_downtime(void)
void do_migrate_set_downtime(Monitor *mon, const QDict *qdict)
{
- char *ptr;
double d;
- const char *value = qdict_get_str(qdict, "value");
-
- d = strtod(value, &ptr);
- if (!strcmp(ptr,"ms")) {
- d *= 1000000;
- } else if (!strcmp(ptr,"us")) {
- d *= 1000;
- } else if (!strcmp(ptr,"ns")) {
- } else {
- /* all else considered to be seconds */
- d *= 1000000000;
- }
+ d = qdict_get_double(qdict, "value") / 1e9;
+ d = MAX(0, MIN(UINT64_MAX, d));
max_downtime = (uint64_t)d;
}
diff --git a/qemu-monitor.hx b/qemu-monitor.hx
index 20c356a..61b99ba 100644
--- a/qemu-monitor.hx
+++ b/qemu-monitor.hx
@@ -774,7 +774,7 @@ ETEXI
{
.name = "migrate_set_downtime",
- .args_type = "value:s",
+ .args_type = "value:T",
.params = "value",
.help = "set maximum tolerated downtime (in seconds) for
migrations",
.mhandler.cmd = do_migrate_set_downtime,
--
1.6.6
- Re: [Qemu-devel] [PATCH v2 3/8] monitor: New argument type 'b', (continued)
[Qemu-devel] [PATCH v2 2/8] QDict: New qdict_get_double(), Markus Armbruster, 2010/01/20
[Qemu-devel] [PATCH v2 5/8] monitor: convert do_migrate_set_speed() to QObject, Markus Armbruster, 2010/01/20
[Qemu-devel] [PATCH v2 8/8] monitor: convert do_migrate_set_downtime() to QObject, Markus Armbruster, 2010/01/20
[Qemu-devel] [PATCH v2 1/8] monitor: Document argument type 'M', Markus Armbruster, 2010/01/20
[Qemu-devel] [PATCH v2 7/8] monitor: Use argument type 'T' for migrate_set_downtime(),
Markus Armbruster <=
[Qemu-devel] [PATCH v2 6/8] monitor: New argument type 'T', Markus Armbruster, 2010/01/20
[Qemu-devel] Re: [PATCH v2 0/8] Convert migrate_set_speed, migrate_set_downtime to QObject, Markus Armbruster, 2010/01/20
[Qemu-devel] Re: [PATCH v2 0/8] Convert migrate_set_speed, migrate_set_downtime to QObject, Markus Armbruster, 2010/01/22