From patchwork Sun Nov 1 14:51:16 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luiz Capitulino X-Patchwork-Id: 37385 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 377861007DF for ; Mon, 2 Nov 2009 02:02:17 +1100 (EST) Received: from localhost ([127.0.0.1]:34035 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N4bwb-0001wy-PD for incoming@patchwork.ozlabs.org; Sun, 01 Nov 2009 10:02:13 -0500 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1N4bmV-0006I0-8I for qemu-devel@nongnu.org; Sun, 01 Nov 2009 09:51:47 -0500 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1N4bmQ-0006E1-Ff for qemu-devel@nongnu.org; Sun, 01 Nov 2009 09:51:46 -0500 Received: from [199.232.76.173] (port=35616 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1N4bmP-0006Db-Oh for qemu-devel@nongnu.org; Sun, 01 Nov 2009 09:51:41 -0500 Received: from mx1.redhat.com ([209.132.183.28]:56028) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1N4bmO-0002Mp-Ub for qemu-devel@nongnu.org; Sun, 01 Nov 2009 09:51:41 -0500 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.13.8/8.13.8) with ESMTP id nA1EpeeM006903 for ; Sun, 1 Nov 2009 09:51:40 -0500 Received: from localhost (vpn-10-169.rdu.redhat.com [10.11.10.169]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id nA1EpcoZ000878 for ; Sun, 1 Nov 2009 09:51:39 -0500 From: Luiz Capitulino To: qemu-devel@nongnu.org Date: Sun, 1 Nov 2009 12:51:16 -0200 Message-Id: <1257087078-5757-7-git-send-email-lcapitulino@redhat.com> In-Reply-To: <1257087078-5757-1-git-send-email-lcapitulino@redhat.com> References: <1257087078-5757-1-git-send-email-lcapitulino@redhat.com> X-Scanned-By: MIMEDefang 2.67 on 10.5.11.12 X-detected-operating-system: by monty-python.gnu.org: Genre and OS details not recognized. Subject: [Qemu-devel] [PATCH 6/8] monitor: Convert qemu_chr_info() to QObject 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 Each device is represented by a QDict. The returned QObject is a QList of all devices. The QDict contains the following: - "label": device's label - "filename": device's file This commit should not change user output, the following is an example of the returned QList: [ { "label": "monitor", "filename", "stdio" }, { "label": "serial0", "filename": "vc" } ] Signed-off-by: Luiz Capitulino --- monitor.c | 3 ++- qemu-char.c | 43 +++++++++++++++++++++++++++++++++++++++++-- qemu-char.h | 4 +++- 3 files changed, 46 insertions(+), 4 deletions(-) diff --git a/monitor.c b/monitor.c index b9fed9f..4e19fdf 100644 --- a/monitor.c +++ b/monitor.c @@ -1972,7 +1972,8 @@ static const mon_cmd_t info_cmds[] = { .args_type = "", .params = "", .help = "show the character devices", - .mhandler.info = qemu_chr_info, + .user_print = qemu_chr_print, + .mhandler.info_new = qemu_chr_info, }, { .name = "block", diff --git a/qemu-char.c b/qemu-char.c index 40bd7e8..fa23111 100644 --- a/qemu-char.c +++ b/qemu-char.c @@ -32,6 +32,7 @@ #include "hw/usb.h" #include "hw/baum.h" #include "hw/msmouse.h" +#include "qemu-objects.h" #include #include @@ -2472,13 +2473,51 @@ void qemu_chr_close(CharDriverState *chr) qemu_free(chr); } -void qemu_chr_info(Monitor *mon) +static void qemu_chr_print_qlist(QObject *obj, void *opaque) { + QDict *chr_dict; + Monitor *mon = opaque; + + chr_dict = qobject_to_qdict(obj); + monitor_printf(mon, "%s: filename=%s\n", qdict_get_str(chr_dict, "label"), + qdict_get_str(chr_dict, "filename")); +} + +void qemu_chr_print(Monitor *mon, const QObject *ret_data) +{ + qlist_iter(qobject_to_qlist(ret_data), qemu_chr_print_qlist, mon); +} + +/** + * qemu_chr_info(): Character devices information + * + * Each device is represented by a QDict. The returned QObject is a QList + * of all devices. + * + * The QDict contains the following: + * + * - "label": device's label + * - "filename": device's file + * + * Example: + * + * [ { "label": "monitor", "filename", "stdio" }, + * { "label": "serial0", "filename": "vc" } ] + */ +void qemu_chr_info(Monitor *mon, QObject **ret_data) +{ + QList *chr_list; CharDriverState *chr; + chr_list = qlist_new(); + QTAILQ_FOREACH(chr, &chardevs, next) { - monitor_printf(mon, "%s: filename=%s\n", chr->label, chr->filename); + QObject *obj = qobject_from_jsonf("{ 'label': %s, 'filename': %s }", + NULL, chr->label, chr->filename); + qlist_append_obj(chr_list, obj); } + + *ret_data = QOBJECT(chr_list); } CharDriverState *qemu_chr_find(const char *name) diff --git a/qemu-char.h b/qemu-char.h index 05fe15d..1a7736f 100644 --- a/qemu-char.h +++ b/qemu-char.h @@ -5,6 +5,7 @@ #include "qemu-queue.h" #include "qemu-option.h" #include "qemu-config.h" +#include "qobject.h" /* character device */ @@ -88,7 +89,8 @@ int qemu_chr_can_read(CharDriverState *s); void qemu_chr_read(CharDriverState *s, uint8_t *buf, int len); int qemu_chr_get_msgfd(CharDriverState *s); void qemu_chr_accept_input(CharDriverState *s); -void qemu_chr_info(Monitor *mon); +void qemu_chr_print(Monitor *mon, const QObject *ret_data); +void qemu_chr_info(Monitor *mon, QObject **ret_data); CharDriverState *qemu_chr_find(const char *name); extern int term_escape_char;