diff options
| author | Markus Armbruster <armbru@redhat.com> | 2010-03-26 09:07:10 +0100 |
|---|---|---|
| committer | Aurelien Jarno <aurelien@aurel32.net> | 2010-04-18 23:55:19 +0200 |
| commit | c9b26a4cbe1e3ce71521b8a1ff0483f4cc4fa4bf (patch) | |
| tree | e1bcd5f4d38b95f8bb9b7da119d120c1707c7a5f | |
| parent | 942cd1f2889dbc74db850218b270bc5b41656dfd (diff) | |
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 <armbru@redhat.com>
Signed-off-by: Luiz Capitulino <lcapitulino@redhat.com>
| -rw-r--r-- | net.c | 10 | ||||
| -rw-r--r-- | qemu-monitor.hx | 8 |
2 files changed, 6 insertions, 12 deletions
@@ -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 { |