From c9b26a4cbe1e3ce71521b8a1ff0483f4cc4fa4bf Mon Sep 17 00:00:00 2001 From: Markus Armbruster Date: Fri, 26 Mar 2010 09:07:10 +0100 Subject: [PATCH] monitor: Use argument type 'b' for set_link Second argument is now "on" or "off" instead of "up" or "down". Signed-off-by: Markus Armbruster Signed-off-by: Luiz Capitulino --- net.c | 10 ++-------- qemu-monitor.hx | 8 ++++---- 2 files changed, 6 insertions(+), 12 deletions(-) diff --git a/net.c b/net.c index e00fac90bd..cdba8f3571 100644 --- a/net.c +++ b/net.c @@ -1276,7 +1276,7 @@ void do_set_link(Monitor *mon, const QDict *qdict) VLANState *vlan; VLANClientState *vc = NULL; const char *name = qdict_get_str(qdict, "name"); - const char *up_or_down = qdict_get_str(qdict, "up_or_down"); + int up = qdict_get_bool(qdict, "up"); QTAILQ_FOREACH(vlan, &vlans, next) { QTAILQ_FOREACH(vc, &vlan->clients, next) { @@ -1293,13 +1293,7 @@ done: return; } - if (strcmp(up_or_down, "up") == 0) - vc->link_down = 0; - else if (strcmp(up_or_down, "down") == 0) - vc->link_down = 1; - else - monitor_printf(mon, "invalid link status '%s'; only 'up' or 'down' " - "valid\n", up_or_down); + vc->link_down = !up; if (vc->info->link_status_changed) { vc->info->link_status_changed(vc); diff --git a/qemu-monitor.hx b/qemu-monitor.hx index 8c9a41cede..7b7dcf545a 100644 --- a/qemu-monitor.hx +++ b/qemu-monitor.hx @@ -986,16 +986,16 @@ ETEXI { .name = "set_link", - .args_type = "name:s,up_or_down:s", - .params = "name up|down", + .args_type = "name:s,up:b", + .params = "name on|off", .help = "change the link status of a network adapter", .mhandler.cmd = do_set_link, }, STEXI -@item set_link @var{name} [up|down] +@item set_link @var{name} [on|off] @findex set_link -Set link @var{name} up or down. +Switch link @var{name} on (i.e. up) or off (i.e. down). ETEXI {