From patchwork Mon Aug 3 16:57:07 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Capitulino X-Patchwork-Id: 30663 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by bilbo.ozlabs.org (Postfix) with ESMTPS id BFA85B6F31 for ; Tue, 4 Aug 2009 05:38:44 +1000 (EST) Received: from localhost ([127.0.0.1]:47953 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MY3Ml-0005KM-7J for incoming@patchwork.ozlabs.org; Mon, 03 Aug 2009 15:38:39 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MY0rx-00068k-VQ for qemu-devel@nongnu.org; Mon, 03 Aug 2009 12:58:42 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MY0rt-00065w-E8 for qemu-devel@nongnu.org; Mon, 03 Aug 2009 12:58:41 -0400 Received: from [199.232.76.173] (port=55281 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MY0rs-00065s-Ua for qemu-devel@nongnu.org; Mon, 03 Aug 2009 12:58:37 -0400 Received: from mx2.redhat.com ([66.187.237.31]:56336) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MY0rs-0003Ag-6V for qemu-devel@nongnu.org; Mon, 03 Aug 2009 12:58:36 -0400 Received: from int-mx2.corp.redhat.com (int-mx2.corp.redhat.com [172.16.27.26]) by mx2.redhat.com (8.13.8/8.13.8) with ESMTP id n73GwZQT026313; Mon, 3 Aug 2009 12:58:35 -0400 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n73GwY9c015407; Mon, 3 Aug 2009 12:58:34 -0400 Received: from localhost (vpn-10-1.bos.redhat.com [10.16.10.1]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n73GwWwi030116; Mon, 3 Aug 2009 12:58:33 -0400 From: Luiz Capitulino To: qemu-devel@nongnu.org Date: Mon, 3 Aug 2009 13:57:07 -0300 Message-Id: <1249318642-19324-11-git-send-email-lcapitulino@redhat.com> In-Reply-To: <1249318642-19324-1-git-send-email-lcapitulino@redhat.com> References: <1249318642-19324-1-git-send-email-lcapitulino@redhat.com> X-Scanned-By: MIMEDefang 2.58 on 172.16.27.26 X-detected-operating-system: by monty-python.gnu.org: GNU/Linux 2.6 (newer, 3) Cc: jan.kiszka@siemens.com, aliguori@us.ibm.com, avi@redhat.com Subject: [Qemu-devel] [PATCH 10/25] monitor: Port handler_1 to use the dictionary X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org This commit ports command handlers that receive one argument to use the new monitor's dictionary. Signed-off-by: Luiz Capitulino --- console.h | 3 +- hw/pci-hotplug.c | 4 +- migration.c | 6 +++- migration.h | 4 +- monitor.c | 66 +++++++++++++++++++++++++++++++---------------------- savevm.c | 10 +++++--- sysemu.h | 13 +++++----- vl.c | 11 +++++---- 8 files changed, 68 insertions(+), 49 deletions(-) diff --git a/console.h b/console.h index 3518339..c8922e4 100644 --- a/console.h +++ b/console.h @@ -2,6 +2,7 @@ #define CONSOLE_H #include "qemu-char.h" +#include "qdict.h" /* keyboard/mouse support */ @@ -44,7 +45,7 @@ struct MouseTransformInfo { }; void do_info_mice(Monitor *mon); -void do_mouse_set(Monitor *mon, int index); +void do_mouse_set(Monitor *mon, const QDict *qdict); /* keysym is a unicode code except for special keys (see QEMU_KEY_xxx constants) */ diff --git a/hw/pci-hotplug.c b/hw/pci-hotplug.c index 335681d..892be36 100644 --- a/hw/pci-hotplug.c +++ b/hw/pci-hotplug.c @@ -199,9 +199,9 @@ void pci_device_hot_remove(Monitor *mon, const char *pci_addr) qemu_system_device_hot_add(bus, slot, 0); } -void do_pci_device_hot_remove(Monitor *mon, const char *pci_addr) +void do_pci_device_hot_remove(Monitor *mon, const QDict *qdict) { - pci_device_hot_remove(mon, pci_addr); + pci_device_hot_remove(mon, qdict_get(qdict, "pci_addr")); } static int pci_match_fn(void *dev_private, void *arg) diff --git a/migration.c b/migration.c index bec8052..cc5caf2 100644 --- a/migration.c +++ b/migration.c @@ -80,11 +80,12 @@ void do_migrate_cancel(Monitor *mon, const QDict *qdict) s->cancel(s); } -void do_migrate_set_speed(Monitor *mon, const char *value) +void do_migrate_set_speed(Monitor *mon, const QDict *qdict) { double d; char *ptr; FdMigrationState *s; + const char *value = qdict_get(qdict, "value"); d = strtod(value, &ptr); switch (*ptr) { @@ -118,10 +119,11 @@ uint64_t migrate_max_downtime(void) return max_downtime; } -void do_migrate_set_downtime(Monitor *mon, const char *value) +void do_migrate_set_downtime(Monitor *mon, const QDict *qdict) { char *ptr; double d; + const char *value = qdict_get(qdict, "value"); d = strtod(value, &ptr); if (!strcmp(ptr,"ms")) { diff --git a/migration.h b/migration.h index 320512b..842ff50 100644 --- a/migration.h +++ b/migration.h @@ -54,11 +54,11 @@ void do_migrate(Monitor *mon, int detach, const char *uri); void do_migrate_cancel(Monitor *mon, const QDict *qdict); -void do_migrate_set_speed(Monitor *mon, const char *value); +void do_migrate_set_speed(Monitor *mon, const QDict *qdict); uint64_t migrate_max_downtime(void); -void do_migrate_set_downtime(Monitor *mon, const char *value); +void do_migrate_set_downtime(Monitor *mon, const QDict *qdict); void do_info_migrate(Monitor *mon); diff --git a/monitor.c b/monitor.c index 4cd9d6a..fa5f1ea 100644 --- a/monitor.c +++ b/monitor.c @@ -254,15 +254,16 @@ static void help_cmd(Monitor *mon, const char *name) } } -static void do_help_cmd(Monitor *mon, const char *name) +static void do_help_cmd(Monitor *mon, const QDict *qdict) { - help_cmd(mon, name); + help_cmd(mon, qdict_get(qdict, "name")); } -static void do_commit(Monitor *mon, const char *device) +static void do_commit(Monitor *mon, const QDict *qdict) { int all_devices; DriveInfo *dinfo; + const char *device = qdict_get(qdict, "device"); all_devices = !strcmp(device, "all"); TAILQ_FOREACH(dinfo, &drives, next) { @@ -273,9 +274,10 @@ static void do_commit(Monitor *mon, const char *device) } } -static void do_info(Monitor *mon, const char *item) +static void do_info(Monitor *mon, const QDict *qdict) { const mon_cmd_t *cmd; + const char *item = qdict_get(qdict, "item"); void (*handler)(Monitor *); if (!item) @@ -387,8 +389,9 @@ static void do_info_cpus(Monitor *mon) } } -static void do_cpu_set(Monitor *mon, int index) +static void do_cpu_set(Monitor *mon, const QDict *qdict) { + int index = (long) qdict_get(qdict, "index"); if (mon_set_cpu(index) < 0) monitor_printf(mon, "Invalid CPU index\n"); } @@ -522,19 +525,20 @@ static void do_change(Monitor *mon, const char *device, const char *target, } } -static void do_screen_dump(Monitor *mon, const char *filename) +static void do_screen_dump(Monitor *mon, const QDict *qdict) { - vga_hw_screen_dump(filename); + vga_hw_screen_dump(qdict_get(qdict, "filename")); } -static void do_logfile(Monitor *mon, const char *filename) +static void do_logfile(Monitor *mon, const QDict *qdict) { - cpu_set_log_filename(filename); + cpu_set_log_filename(qdict_get(qdict, "filename")); } -static void do_log(Monitor *mon, const char *items) +static void do_log(Monitor *mon, const QDict *qdict) { int mask; + const char *items = qdict_get(qdict, "items"); if (!strcmp(items, "none")) { mask = 0; @@ -548,8 +552,9 @@ static void do_log(Monitor *mon, const char *items) cpu_set_log(mask); } -static void do_singlestep(Monitor *mon, const char *option) +static void do_singlestep(Monitor *mon, const QDict *qdict) { + const char *option = qdict_get(qdict, "option"); if (!option || !strcmp(option, "on")) { singlestep = 1; } else if (!strcmp(option, "off")) { @@ -601,8 +606,9 @@ static void encrypted_bdrv_it(void *opaque, BlockDriverState *bs) } } -static void do_gdbserver(Monitor *mon, const char *device) +static void do_gdbserver(Monitor *mon, const QDict *qdict) { + const char *device = qdict_get(qdict, "device"); if (!device) device = "tcp::" DEFAULT_GDBSTUB_PORT; if (gdbserver_start(device) < 0) { @@ -616,8 +622,9 @@ static void do_gdbserver(Monitor *mon, const char *device) } } -static void do_watchdog_action(Monitor *mon, const char *action) +static void do_watchdog_action(Monitor *mon, const QDict *qdict) { + const char *action = qdict_get(qdict, "action"); if (select_watchdog_action(action) == -1) { monitor_printf(mon, "Unknown watchdog action '%s'\n", action); } @@ -1175,8 +1182,9 @@ static void do_mouse_move(Monitor *mon, const char *dx_str, const char *dy_str, kbd_mouse_event(dx, dy, dz, mouse_button_state); } -static void do_mouse_button(Monitor *mon, int button_state) +static void do_mouse_button(Monitor *mon, const QDict *qdict) { + int button_state = (long) qdict_get(qdict, "button_state"); mouse_button_state = button_state; kbd_mouse_event(0, 0, 0, mouse_button_state); } @@ -1231,9 +1239,10 @@ static void do_ioport_write(Monitor *mon, int count, int format, int size, } } -static void do_boot_set(Monitor *mon, const char *bootdevice) +static void do_boot_set(Monitor *mon, const QDict *qdict) { int res; + const char *bootdevice = qdict_get(qdict, "bootdevice"); res = qemu_boot_set(bootdevice); if (res == 0) { @@ -1530,9 +1539,10 @@ static void do_info_capture(Monitor *mon) } #ifdef HAS_AUDIO -static void do_stop_capture(Monitor *mon, int n) +static void do_stop_capture(Monitor *mon, const QDict *qdict) { int i; + int n = (long) qdict_get(qdict, "n"); CaptureState *s; for (s = capture_head.lh_first, i = 0; s; s = s->entries.le_next, ++i) { @@ -1567,9 +1577,10 @@ static void do_wav_capture(Monitor *mon, const char *path, #endif #if defined(TARGET_I386) -static void do_inject_nmi(Monitor *mon, int cpu_index) +static void do_inject_nmi(Monitor *mon, const QDict *qdict) { CPUState *env; + int cpu_index = (long) qdict_get(qdict, "cpu_index"); for (env = first_cpu; env != NULL; env = env->next_cpu) if (env->cpu_index == cpu_index) { @@ -1592,8 +1603,9 @@ static void do_info_status(Monitor *mon) } -static void do_balloon(Monitor *mon, int value) +static void do_balloon(Monitor *mon, const QDict *qdict) { + int value = (long) qdict_get(qdict, "value"); ram_addr_t target = value; qemu_balloon(target << 20); } @@ -1622,8 +1634,9 @@ static qemu_acl *find_acl(Monitor *mon, const char *name) return acl; } -static void do_acl_show(Monitor *mon, const char *aclname) +static void do_acl_show(Monitor *mon, const QDict *qdict) { + const char *aclname = qdict_get(qdict, "aclname"); qemu_acl *acl = find_acl(mon, aclname); qemu_acl_entry *entry; int i = 0; @@ -1639,8 +1652,9 @@ static void do_acl_show(Monitor *mon, const char *aclname) } } -static void do_acl_reset(Monitor *mon, const char *aclname) +static void do_acl_reset(Monitor *mon, const QDict *qdict) { + const char *aclname = qdict_get(qdict, "aclname"); qemu_acl *acl = find_acl(mon, aclname); if (acl) { @@ -1732,8 +1746,9 @@ static void do_inject_mce(Monitor *mon, } #endif -static void do_getfd(Monitor *mon, const char *fdname) +static void do_getfd(Monitor *mon, const QDict *qdict) { + const char *fdname = qdict_get(qdict, "fdname"); mon_fd_t *monfd; int fd; @@ -1772,8 +1787,9 @@ static void do_getfd(Monitor *mon, const char *fdname) LIST_INSERT_HEAD(&mon->fds, monfd, next); } -static void do_closefd(Monitor *mon, const char *fdname) +static void do_closefd(Monitor *mon, const QDict *qdict) { + const char *fdname = qdict_get(qdict, "fdname"); mon_fd_t *monfd; LIST_FOREACH(monfd, &mon->fds, next) { @@ -2641,7 +2657,6 @@ static void monitor_handle_command(Monitor *mon, const char *cmdline) void *args[MAX_ARGS]; QDict *qdict; void (*handler_d)(Monitor *mon, const QDict *qdict); - void (*handler_1)(Monitor *mon, void *arg0); void (*handler_2)(Monitor *mon, void *arg0, void *arg1); void (*handler_3)(Monitor *mon, void *arg0, void *arg1, void *arg2); void (*handler_4)(Monitor *mon, void *arg0, void *arg1, void *arg2, @@ -2942,13 +2957,10 @@ static void monitor_handle_command(Monitor *mon, const char *cmdline) switch(nb_args) { case 0: + case 1: handler_d = cmd->handler; handler_d(mon, qdict); break; - case 1: - handler_1 = cmd->handler; - handler_1(mon, args[0]); - break; case 2: handler_2 = cmd->handler; handler_2(mon, args[0], args[1]); diff --git a/savevm.c b/savevm.c index 9f988e6..c5f4c92 100644 --- a/savevm.c +++ b/savevm.c @@ -1048,7 +1048,7 @@ static int bdrv_snapshot_find(BlockDriverState *bs, QEMUSnapshotInfo *sn_info, return ret; } -void do_savevm(Monitor *mon, const char *name) +void do_savevm(Monitor *mon, const QDict *qdict) { DriveInfo *dinfo; BlockDriverState *bs, *bs1; @@ -1062,6 +1062,7 @@ void do_savevm(Monitor *mon, const char *name) #else struct timeval tv; #endif + const char *name = qdict_get(qdict, "name"); bs = get_bs_snapshots(); if (!bs) { @@ -1217,16 +1218,17 @@ void qemu_loadvm(Monitor *mon, const char *name) vm_start(); } -void do_loadvm(Monitor *mon, const char *name) +void do_loadvm(Monitor *mon, const QDict *qdict) { - qemu_loadvm(mon, name); + qemu_loadvm(mon, qdict_get(qdict, "name")); } -void do_delvm(Monitor *mon, const char *name) +void do_delvm(Monitor *mon, const QDict *qdict) { DriveInfo *dinfo; BlockDriverState *bs, *bs1; int ret; + const char *name = qdict_get(qdict, "name"); bs = get_bs_snapshots(); if (!bs) { diff --git a/sysemu.h b/sysemu.h index b839134..b006813 100644 --- a/sysemu.h +++ b/sysemu.h @@ -5,6 +5,7 @@ #include "qemu-common.h" #include "qemu-option.h" #include "sys-queue.h" +#include "qdict.h" #ifdef _WIN32 #include @@ -57,10 +58,10 @@ void qemu_system_powerdown(void); #endif void qemu_system_reset(void); -void do_savevm(Monitor *mon, const char *name); -void do_loadvm(Monitor *mon, const char *name); +void do_savevm(Monitor *mon, const QDict *qdict); +void do_loadvm(Monitor *mon, const QDict *qdict); void qemu_loadvm(Monitor *mon, const char *name); -void do_delvm(Monitor *mon, const char *name); +void do_delvm(Monitor *mon, const QDict *qdict); void do_info_snapshots(Monitor *mon); void qemu_announce_self(void); @@ -212,8 +213,8 @@ void destroy_bdrvs(dev_match_fn *match_fn, void *arg); void pci_device_hot_add(Monitor *mon, const char *pci_addr, const char *type, const char *opts); void drive_hot_add(Monitor *mon, const char *pci_addr, const char *opts); -void do_pci_device_hot_remove(Monitor *mon, const char *pci_addr); void pci_device_hot_remove(Monitor *mon, const char *pci_addr); +void do_pci_device_hot_remove(Monitor *mon, const QDict *qdict); void pci_device_hot_remove_success(int pcibus, int slot); /* serial ports */ @@ -269,8 +270,8 @@ struct soundhw { extern struct soundhw soundhw[]; #endif -void do_usb_add(Monitor *mon, const char *devname); -void do_usb_del(Monitor *mon, const char *devname); +void do_usb_add(Monitor *mon, const QDict *qdict); +void do_usb_del(Monitor *mon, const QDict *qdict); void usb_info(Monitor *mon); void register_devices(void); diff --git a/vl.c b/vl.c index 8325530..c7fb36f 100644 --- a/vl.c +++ b/vl.c @@ -478,10 +478,11 @@ void do_info_mice(Monitor *mon) } } -void do_mouse_set(Monitor *mon, int index) +void do_mouse_set(Monitor *mon, const QDict *qdict) { QEMUPutMouseEntry *cursor; int i = 0; + int index = (long) qdict_get(qdict, "index"); if (!qemu_put_mouse_event_head) { monitor_printf(mon, "No mouse devices connected\n"); @@ -2587,14 +2588,14 @@ static int usb_parse(const char *cmdline) return usb_device_add(cmdline, 0); } -void do_usb_add(Monitor *mon, const char *devname) +void do_usb_add(Monitor *mon, const QDict *qdict) { - usb_device_add(devname, 1); + usb_device_add(qdict_get(qdict, "devname"), 1); } -void do_usb_del(Monitor *mon, const char *devname) +void do_usb_del(Monitor *mon, const QDict *qdict) { - usb_device_del(devname); + usb_device_del(qdict_get(qdict, "devname")); } void usb_info(Monitor *mon)