From patchwork Wed Oct 7 16:32:06 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Capitulino X-Patchwork-Id: 35324 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 ozlabs.org (Postfix) with ESMTPS id 0C077B7B7B for ; Thu, 8 Oct 2009 03:50:13 +1100 (EST) Received: from localhost ([127.0.0.1]:45656 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MvZiM-0005BD-EI for incoming@patchwork.ozlabs.org; Wed, 07 Oct 2009 12:50:10 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MvZRh-00073H-D3 for qemu-devel@nongnu.org; Wed, 07 Oct 2009 12:32:57 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MvZRc-00070s-Bw for qemu-devel@nongnu.org; Wed, 07 Oct 2009 12:32:56 -0400 Received: from [199.232.76.173] (port=48042 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MvZRc-00070b-9W for qemu-devel@nongnu.org; Wed, 07 Oct 2009 12:32:52 -0400 Received: from mx1.redhat.com ([209.132.183.28]:29564) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MvZRb-0003vU-C7 for qemu-devel@nongnu.org; Wed, 07 Oct 2009 12:32:51 -0400 Received: from int-mx04.intmail.prod.int.phx2.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.17]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id n97GWoI3024934; Wed, 7 Oct 2009 12:32:50 -0400 Received: from localhost (vpn-12-190.rdu.redhat.com [10.11.12.190]) by int-mx04.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id n97GWnHs007481; Wed, 7 Oct 2009 12:32:49 -0400 From: Luiz Capitulino To: qemu-devel@nongnu.org Date: Wed, 7 Oct 2009 13:32:06 -0300 Message-Id: <1254933135-21888-10-git-send-email-lcapitulino@redhat.com> In-Reply-To: <1254933135-21888-1-git-send-email-lcapitulino@redhat.com> References: <1254933135-21888-1-git-send-email-lcapitulino@redhat.com> X-Scanned-By: MIMEDefang 2.67 on 10.5.11.17 X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Cc: aliguori@us.ibm.com, avi@redhat.com Subject: [Qemu-devel] [PATCH 09/18] monitor: do_info(): handle new and old info handlers 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 do_info() is special, its job is to call 'info handlers'. This is similar to what monitor_handle_command() does, therefore do_info() also has to distinguish among new and old style info handlers. This commit converts do_info() to the new QObject style and makes the appropriate changes so that it can handle both info handlers styles. In the future, when all handlers are converted to QObject's style, it will be possible to share more code with monitor_handle_command(). This commit also introduces a new function called monitor_user_noop(), it should be used by handlers which do not have data to print. This is the case of do_info(). Signed-off-by: Luiz Capitulino --- monitor.c | 29 ++++++++++++++++++++++------- qemu-monitor.hx | 3 ++- 2 files changed, 24 insertions(+), 8 deletions(-) diff --git a/monitor.c b/monitor.c index d545124..cc1b501 100644 --- a/monitor.c +++ b/monitor.c @@ -76,6 +76,7 @@ typedef struct mon_cmd_t { void (*user_print)(Monitor *mon, const QObject *data); union { void (*info)(Monitor *mon); + void (*info_new)(Monitor *mon, QObject **ret_data); void (*cmd)(Monitor *mon, const QDict *qdict); void (*cmd_new)(Monitor *mon, const QDict *params, QObject **ret_data); } mhandler; @@ -215,6 +216,8 @@ static int monitor_fprintf(FILE *stream, const char *fmt, ...) return 0; } +static void monitor_user_noop(Monitor *mon, const QObject *data) { } + static inline int monitor_handler_ported(const mon_cmd_t *cmd) { return cmd->user_print != NULL; @@ -289,22 +292,34 @@ static void do_commit(Monitor *mon, const QDict *qdict) } } -static void do_info(Monitor *mon, const QDict *qdict) +static void do_info(Monitor *mon, const QDict *qdict, QObject **ret_data) { const mon_cmd_t *cmd; const char *item = qdict_get_try_str(qdict, "item"); if (!item) goto help; - for(cmd = info_cmds; cmd->name != NULL; cmd++) { + + for (cmd = info_cmds; cmd->name != NULL; cmd++) { if (compare_cmd(item, cmd->name)) - goto found; + break; } - help: - help_cmd(mon, "info"); + + if (cmd->name == NULL) + goto help; + + if (monitor_handler_ported(cmd)) { + cmd->mhandler.info_new(mon, ret_data); + if (*ret_data) + cmd->user_print(mon, *ret_data); + } else { + cmd->mhandler.info(mon); + } + return; - found: - cmd->mhandler.info(mon); + +help: + help_cmd(mon, "info"); } static void do_info_version(Monitor *mon) diff --git a/qemu-monitor.hx b/qemu-monitor.hx index c689a07..c37b0cb 100644 --- a/qemu-monitor.hx +++ b/qemu-monitor.hx @@ -40,7 +40,8 @@ ETEXI .args_type = "item:s?", .params = "[subcommand]", .help = "show various information about the system state", - .mhandler.cmd = do_info, + .user_print = monitor_user_noop, + .mhandler.cmd_new = do_info, }, STEXI