From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559915909; cv=none; d=zoho.com; s=zohoarc; b=iiiIqcKhC6s3cUxChWAEoQJYQBHgibUDK6Cgbfm5SE/ksUYWrn2D+th4aC27Kk6KZSFca8hGQRqPFl6BPMyCmdBI5H35uKOOdwwgN6EXVEUn3kxLHeEPuQKWMJE0bRJKXiKuxE10CSaTJP6RVXORidV9TE+jWjC/AAvHs5b8D2s= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559915909; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=+CIFYtiT5jeAz3wD4+ALwlCSSmQHbbklhZnBy+tLorQ=; b=ho2SM1t9LZv+8QSakuCqnnxlZmP8k3n2FRlWjVPIdSakuXNhGjp762mGdWWyZIbm1oRDMwZ0XhV8CNgxCeNnF5BFV1dRhxRLmI2iFD+uwlDY8LIJb+LuB0fLNNzmeD6P4FZBB7Cz5G81+xEE2jkCmL0zjqsWoO3QWkXksPE4w3E= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559915909537450.52948950151836; Fri, 7 Jun 2019 06:58:29 -0700 (PDT) Received: from localhost ([::1]:51422 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFNy-0005bl-Jk for importer@patchew.org; Fri, 07 Jun 2019 09:58:26 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:44958) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKa-0001Qd-18 for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:54:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKY-00032C-VZ for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:54:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57484) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKO-0002Yb-4N; Fri, 07 Jun 2019 09:54:44 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 09F453001A63; Fri, 7 Jun 2019 13:54:38 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id C5A631001B08; Fri, 7 Jun 2019 13:54:36 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:21 +0200 Message-Id: <20190607135430.22149-2-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.45]); Fri, 07 Jun 2019 13:54:38 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 01/10] monitor: Remove unused password prompting fields X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Commit 788cf9f8c removed the code for password prompting from the monitor. Since then, the Monitor fields password_completion_cb and password_opaque have been unused. Remove them. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- monitor.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/monitor.c b/monitor.c index 6428eb3b7e..70ce9e8a77 100644 --- a/monitor.c +++ b/monitor.c @@ -220,8 +220,6 @@ struct Monitor { =20 MonitorQMP qmp; gchar *mon_cpu_path; - BlockCompletionFunc *password_completion_cb; - void *password_opaque; mon_cmd_t *cmd_table; QTAILQ_ENTRY(Monitor) entry; =20 --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559915816; cv=none; d=zoho.com; s=zohoarc; b=AgYgrjWun18IeBfRSz7hk87GWxy/wgvtTN3b+JI8vgELwjQSYfjvCHwONOJ/JlJh5KErwNWw2ObcwCgxsAB99gwp/eCWQjwmBL2KtwrBCvdDOK+BL+7F6SdCKQYt1HYw5l5n1zKxi7o98LVmyk3DR4MNtA075omOro98NOnXS/M= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559915816; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=eqO8GbpBydg89wmWqupPUdssksC7jUAwCE+Zd68oe9s=; b=UvC7svo4GTsMBQ9l8zFJaSZqYVM5pTOpCgaZvitg10nCz7Nwov9JNN7Y+khhhqAF9xFHOvWBOlpnygs6r+37uZ1YbShm3s9a+6OoR2QazGKg9xiScK5oXTsRhfL+DaHfhBMoHU+33dhZ45YxVDxB06Z/xRo78jmx0GbzomH9VxM= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559915816194196.7872529069714; Fri, 7 Jun 2019 06:56:56 -0700 (PDT) Received: from localhost ([::1]:51397 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFMV-0003aS-2o for importer@patchew.org; Fri, 07 Jun 2019 09:56:55 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45141) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKi-0001gL-6h for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:06 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKg-0003HX-7A for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:04 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57715) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKa-0002hG-7i; Fri, 07 Jun 2019 09:54:56 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 7D2FC2E95A2; Fri, 7 Jun 2019 13:54:39 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id 4796A1001B08; Fri, 7 Jun 2019 13:54:38 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:22 +0200 Message-Id: <20190607135430.22149-3-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.29]); Fri, 07 Jun 2019 13:54:39 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 02/10] monitor: Split monitor_init in HMP and QMP function X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Instead of mixing HMP and QMP monitors in the same function, separate the monitor creation function for both. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- monitor.c | 86 +++++++++++++++++++++++++++++++------------------------ 1 file changed, 49 insertions(+), 37 deletions(-) diff --git a/monitor.c b/monitor.c index 70ce9e8a77..bb23cc0450 100644 --- a/monitor.c +++ b/monitor.c @@ -702,7 +702,7 @@ static void handle_hmp_command(Monitor *mon, const char= *cmdline); =20 static void monitor_iothread_init(void); =20 -static void monitor_data_init(Monitor *mon, bool skip_flush, +static void monitor_data_init(Monitor *mon, int flags, bool skip_flush, bool use_io_thread) { if (use_io_thread && !mon_iothread) { @@ -717,6 +717,7 @@ static void monitor_data_init(Monitor *mon, bool skip_f= lush, mon->skip_flush =3D skip_flush; mon->use_io_thread =3D use_io_thread; mon->qmp.qmp_requests =3D g_queue_new(); + mon->flags =3D flags; } =20 static void monitor_data_destroy(Monitor *mon) @@ -740,7 +741,7 @@ char *qmp_human_monitor_command(const char *command_lin= e, bool has_cpu_index, char *output =3D NULL; Monitor *old_mon, hmp; =20 - monitor_data_init(&hmp, true, false); + monitor_data_init(&hmp, 0, true, false); =20 old_mon =3D cur_mon; cur_mon =3D &hmp; @@ -4603,19 +4604,48 @@ static void monitor_qmp_setup_handlers_bh(void *opa= que) monitor_list_append(mon); } =20 -void monitor_init(Chardev *chr, int flags) +static void monitor_init_qmp(Chardev *chr, int flags) { Monitor *mon =3D g_malloc(sizeof(*mon)); - bool use_readline =3D flags & MONITOR_USE_READLINE; =20 /* Note: we run QMP monitor in I/O thread when @chr supports that */ - monitor_data_init(mon, false, - (flags & MONITOR_USE_CONTROL) - && qemu_chr_has_feature(chr, - QEMU_CHAR_FEATURE_GCONTEXT)); + monitor_data_init(mon, flags, false, + qemu_chr_has_feature(chr, QEMU_CHAR_FEATURE_GCONTEXT= )); =20 qemu_chr_fe_init(&mon->chr, chr, &error_abort); - mon->flags =3D flags; + qemu_chr_fe_set_echo(&mon->chr, true); + + json_message_parser_init(&mon->qmp.parser, handle_qmp_command, mon, NU= LL); + if (mon->use_io_thread) { + /* + * Make sure the old iowatch is gone. It's possible when + * e.g. the chardev is in client mode, with wait=3Don. + */ + remove_fd_in_watch(chr); + /* + * We can't call qemu_chr_fe_set_handlers() directly here + * since chardev might be running in the monitor I/O + * thread. Schedule a bottom half. + */ + aio_bh_schedule_oneshot(iothread_get_aio_context(mon_iothread), + monitor_qmp_setup_handlers_bh, mon); + /* The bottom half will add @mon to @mon_list */ + } else { + qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, + monitor_qmp_read, monitor_qmp_event, + NULL, mon, NULL, true); + monitor_list_append(mon); + } +} + +static void monitor_init_hmp(Chardev *chr, int flags) +{ + Monitor *mon =3D g_malloc(sizeof(*mon)); + bool use_readline =3D flags & MONITOR_USE_READLINE; + + monitor_data_init(mon, flags, false, false); + qemu_chr_fe_init(&mon->chr, chr, &error_abort); + if (use_readline) { mon->rs =3D readline_init(monitor_readline_printf, monitor_readline_flush, @@ -4624,36 +4654,18 @@ void monitor_init(Chardev *chr, int flags) monitor_read_command(mon, 0); } =20 - if (monitor_is_qmp(mon)) { - qemu_chr_fe_set_echo(&mon->chr, true); - json_message_parser_init(&mon->qmp.parser, handle_qmp_command, - mon, NULL); - if (mon->use_io_thread) { - /* - * Make sure the old iowatch is gone. It's possible when - * e.g. the chardev is in client mode, with wait=3Don. - */ - remove_fd_in_watch(chr); - /* - * We can't call qemu_chr_fe_set_handlers() directly here - * since chardev might be running in the monitor I/O - * thread. Schedule a bottom half. - */ - aio_bh_schedule_oneshot(iothread_get_aio_context(mon_iothread), - monitor_qmp_setup_handlers_bh, mon); - /* The bottom half will add @mon to @mon_list */ - return; - } else { - qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, - monitor_qmp_read, monitor_qmp_event, - NULL, mon, NULL, true); - } + qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, monitor_read, + monitor_event, NULL, mon, NULL, true); + monitor_list_append(mon); +} + +void monitor_init(Chardev *chr, int flags) +{ + if (flags & MONITOR_USE_CONTROL) { + monitor_init_qmp(chr, flags); } else { - qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, monitor_read, - monitor_event, NULL, mon, NULL, true); + monitor_init_hmp(chr, flags); } - - monitor_list_append(mon); } =20 void monitor_cleanup(void) --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559916710; cv=none; d=zoho.com; s=zohoarc; b=TL2z42g043fCAE1Sjc8JUWXmfc+KcnHzq1zQd+Tmm8I9h9JkcR4ka4YM+l0RZiYOxgIrMIjWc6r2s9dlgXIsonOEhuvyg5iWAE8SCwA/3JxNuGXBTgEm2+ZSSexg/1mI7VvJMz97BmwR8Zsj1CDP0vRNm7cV9JQnQihe44ifS10= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559916710; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=61W/3kmWOm4k0cAdH3Xkbbn9zYfBH2elJ3CyFKYMDQI=; b=iU+ZL/ZoDdXcj9dRguLIjUSCkEjAdBg252C/E4lfE+jw6monf9nAMfvLVqoimz73/rN5HZyh8VMkWeTbVwdc2olq5uFvXIfAxJYBhn5wakaCEkJaQmOsPsmliutWBL4Dprrb+V+nZC8e019VQvY9CiOpca5qmeQQgmIC4VV+9bk= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (209.51.188.47 [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559916710102166.48664228748214; Fri, 7 Jun 2019 07:11:50 -0700 (PDT) Received: from localhost ([::1]:51532 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFak-0005cu-H2 for importer@patchew.org; Fri, 07 Jun 2019 10:11:38 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45262) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKt-0001oR-VW for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKp-0003XC-4P for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:15 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51012) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKf-0003Ef-I6; Fri, 07 Jun 2019 09:55:01 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 231686697A; Fri, 7 Jun 2019 13:54:41 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id BF3F51001B08; Fri, 7 Jun 2019 13:54:39 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:23 +0200 Message-Id: <20190607135430.22149-4-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.38]); Fri, 07 Jun 2019 13:54:50 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 03/10] monitor: Make MonitorQMP a child class of Monitor X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Currently, struct Monitor mixes state that is only relevant for HMP, state that is only relevant for QMP, and some actually shared state. In particular, a MonitorQMP field is present in the state of any monitor, even if it's not a QMP monitor and therefore doesn't use the state. As a first step towards a clean separation between QMP and HMP, let MonitorQMP extend Monitor and create a MonitorQMP object only when the monitor is actually a QMP monitor. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- monitor.c | 214 ++++++++++++++++++++++++++++++------------------------ 1 file changed, 118 insertions(+), 96 deletions(-) diff --git a/monitor.c b/monitor.c index bb23cc0450..d18cf18393 100644 --- a/monitor.c +++ b/monitor.c @@ -166,26 +166,6 @@ struct MonFdset { QLIST_ENTRY(MonFdset) next; }; =20 -typedef struct { - JSONMessageParser parser; - /* - * When a client connects, we're in capabilities negotiation mode. - * @commands is &qmp_cap_negotiation_commands then. When command - * qmp_capabilities succeeds, we go into command mode, and - * @command becomes &qmp_commands. - */ - QmpCommandList *commands; - bool capab_offered[QMP_CAPABILITY__MAX]; /* capabilities offered */ - bool capab[QMP_CAPABILITY__MAX]; /* offered and accepted */ - /* - * Protects qmp request/response queue. - * Take monitor_lock first when you need both. - */ - QemuMutex qmp_queue_lock; - /* Input queue that holds all the parsed QMP requests */ - GQueue *qmp_requests; -} MonitorQMP; - /* * To prevent flooding clients, events can be throttled. The * throttling is calculated globally, rather than per-Monitor @@ -218,7 +198,6 @@ struct Monitor { */ ReadLineState *rs; =20 - MonitorQMP qmp; gchar *mon_cpu_path; mon_cmd_t *cmd_table; QTAILQ_ENTRY(Monitor) entry; @@ -239,6 +218,27 @@ struct Monitor { int mux_out; }; =20 +typedef struct { + Monitor common; + JSONMessageParser parser; + /* + * When a client connects, we're in capabilities negotiation mode. + * @commands is &qmp_cap_negotiation_commands then. When command + * qmp_capabilities succeeds, we go into command mode, and + * @command becomes &qmp_commands. + */ + QmpCommandList *commands; + bool capab_offered[QMP_CAPABILITY__MAX]; /* capabilities offered */ + bool capab[QMP_CAPABILITY__MAX]; /* offered and accepted */ + /* + * Protects qmp request/response queue. + * Take monitor_lock first when you need both. + */ + QemuMutex qmp_queue_lock; + /* Input queue that holds all the parsed QMP requests */ + GQueue *qmp_requests; +} MonitorQMP; + /* Shared monitor I/O thread */ IOThread *mon_iothread; =20 @@ -247,7 +247,7 @@ QEMUBH *qmp_dispatcher_bh; =20 struct QMPRequest { /* Owner of the request */ - Monitor *mon; + MonitorQMP *mon; /* * Request object to be handled or Error to be reported * (exactly one of them is non-null) @@ -355,18 +355,18 @@ static void qmp_request_free(QMPRequest *req) } =20 /* Caller must hold mon->qmp.qmp_queue_lock */ -static void monitor_qmp_cleanup_req_queue_locked(Monitor *mon) +static void monitor_qmp_cleanup_req_queue_locked(MonitorQMP *mon) { - while (!g_queue_is_empty(mon->qmp.qmp_requests)) { - qmp_request_free(g_queue_pop_head(mon->qmp.qmp_requests)); + while (!g_queue_is_empty(mon->qmp_requests)) { + qmp_request_free(g_queue_pop_head(mon->qmp_requests)); } } =20 -static void monitor_qmp_cleanup_queues(Monitor *mon) +static void monitor_qmp_cleanup_queues(MonitorQMP *mon) { - qemu_mutex_lock(&mon->qmp.qmp_queue_lock); + qemu_mutex_lock(&mon->qmp_queue_lock); monitor_qmp_cleanup_req_queue_locked(mon); - qemu_mutex_unlock(&mon->qmp.qmp_queue_lock); + qemu_mutex_unlock(&mon->qmp_queue_lock); } =20 =20 @@ -478,17 +478,17 @@ int monitor_printf(Monitor *mon, const char *fmt, ...) return ret; } =20 -static void qmp_send_response(Monitor *mon, const QDict *rsp) +static void qmp_send_response(MonitorQMP *mon, const QDict *rsp) { const QObject *data =3D QOBJECT(rsp); QString *json; =20 - json =3D mon->flags & MONITOR_USE_PRETTY ? qobject_to_json_pretty(data= ) : - qobject_to_json(data); + json =3D mon->common.flags & MONITOR_USE_PRETTY ? + qobject_to_json_pretty(data) : qobject_to_json(data); assert(json !=3D NULL); =20 qstring_append_chr(json, '\n'); - monitor_puts(mon, qstring_get_str(json)); + monitor_puts(&mon->common, qstring_get_str(json)); =20 qobject_unref(json); } @@ -511,12 +511,17 @@ static MonitorQAPIEventConf monitor_qapi_event_conf[Q= API_EVENT__MAX] =3D { static void monitor_qapi_event_emit(QAPIEvent event, QDict *qdict) { Monitor *mon; + MonitorQMP *qmp_mon; =20 trace_monitor_protocol_event_emit(event, qdict); QTAILQ_FOREACH(mon, &mon_list, entry) { - if (monitor_is_qmp(mon) - && mon->qmp.commands !=3D &qmp_cap_negotiation_commands) { - qmp_send_response(mon, qdict); + if (!monitor_is_qmp(mon)) { + continue; + } + + qmp_mon =3D container_of(mon, MonitorQMP, common); + if (qmp_mon->commands !=3D &qmp_cap_negotiation_commands) { + qmp_send_response(qmp_mon, qdict); } } } @@ -710,29 +715,33 @@ static void monitor_data_init(Monitor *mon, int flags= , bool skip_flush, } memset(mon, 0, sizeof(Monitor)); qemu_mutex_init(&mon->mon_lock); - qemu_mutex_init(&mon->qmp.qmp_queue_lock); mon->outbuf =3D qstring_new(); /* Use *mon_cmds by default. */ mon->cmd_table =3D mon_cmds; mon->skip_flush =3D skip_flush; mon->use_io_thread =3D use_io_thread; - mon->qmp.qmp_requests =3D g_queue_new(); mon->flags =3D flags; } =20 +static void monitor_data_destroy_qmp(MonitorQMP *mon) +{ + json_message_parser_destroy(&mon->parser); + qemu_mutex_destroy(&mon->qmp_queue_lock); + monitor_qmp_cleanup_req_queue_locked(mon); + g_queue_free(mon->qmp_requests); +} + static void monitor_data_destroy(Monitor *mon) { g_free(mon->mon_cpu_path); qemu_chr_fe_deinit(&mon->chr, false); if (monitor_is_qmp(mon)) { - json_message_parser_destroy(&mon->qmp.parser); + MonitorQMP *qmp_mon =3D container_of(mon, MonitorQMP, common); + monitor_data_destroy_qmp(qmp_mon); } readline_free(mon->rs); qobject_unref(mon->outbuf); qemu_mutex_destroy(&mon->mon_lock); - qemu_mutex_destroy(&mon->qmp.qmp_queue_lock); - monitor_qmp_cleanup_req_queue_locked(mon); - g_queue_free(mon->qmp.qmp_requests); } =20 char *qmp_human_monitor_command(const char *command_line, bool has_cpu_ind= ex, @@ -1085,8 +1094,9 @@ static void query_commands_cb(QmpCommand *cmd, void *= opaque) CommandInfoList *qmp_query_commands(Error **errp) { CommandInfoList *list =3D NULL; + MonitorQMP *mon =3D container_of(cur_mon, MonitorQMP, common); =20 - qmp_for_each_command(cur_mon->qmp.commands, query_commands_cb, &list); + qmp_for_each_command(mon->commands, query_commands_cb, &list); =20 return list; } @@ -1153,16 +1163,16 @@ static void monitor_init_qmp_commands(void) qmp_marshal_qmp_capabilities, QCO_ALLOW_PRECONFIG= ); } =20 -static bool qmp_oob_enabled(Monitor *mon) +static bool qmp_oob_enabled(MonitorQMP *mon) { - return mon->qmp.capab[QMP_CAPABILITY_OOB]; + return mon->capab[QMP_CAPABILITY_OOB]; } =20 -static void monitor_qmp_caps_reset(Monitor *mon) +static void monitor_qmp_caps_reset(MonitorQMP *mon) { - memset(mon->qmp.capab_offered, 0, sizeof(mon->qmp.capab_offered)); - memset(mon->qmp.capab, 0, sizeof(mon->qmp.capab)); - mon->qmp.capab_offered[QMP_CAPABILITY_OOB] =3D mon->use_io_thread; + memset(mon->capab_offered, 0, sizeof(mon->capab_offered)); + memset(mon->capab, 0, sizeof(mon->capab)); + mon->capab_offered[QMP_CAPABILITY_OOB] =3D mon->common.use_io_thread; } =20 /* @@ -1170,7 +1180,7 @@ static void monitor_qmp_caps_reset(Monitor *mon) * On success, set mon->qmp.capab[], and return true. * On error, set @errp, and return false. */ -static bool qmp_caps_accept(Monitor *mon, QMPCapabilityList *list, +static bool qmp_caps_accept(MonitorQMP *mon, QMPCapabilityList *list, Error **errp) { GString *unavailable =3D NULL; @@ -1179,7 +1189,7 @@ static bool qmp_caps_accept(Monitor *mon, QMPCapabili= tyList *list, memset(capab, 0, sizeof(capab)); =20 for (; list; list =3D list->next) { - if (!mon->qmp.capab_offered[list->value]) { + if (!mon->capab_offered[list->value]) { if (!unavailable) { unavailable =3D g_string_new(QMPCapability_str(list->value= )); } else { @@ -1196,25 +1206,27 @@ static bool qmp_caps_accept(Monitor *mon, QMPCapabi= lityList *list, return false; } =20 - memcpy(mon->qmp.capab, capab, sizeof(capab)); + memcpy(mon->capab, capab, sizeof(capab)); return true; } =20 void qmp_qmp_capabilities(bool has_enable, QMPCapabilityList *enable, Error **errp) { - if (cur_mon->qmp.commands =3D=3D &qmp_commands) { + MonitorQMP *mon =3D container_of(cur_mon, MonitorQMP, common); + + if (mon->commands =3D=3D &qmp_commands) { error_set(errp, ERROR_CLASS_COMMAND_NOT_FOUND, "Capabilities negotiation is already complete, command " "ignored"); return; } =20 - if (!qmp_caps_accept(cur_mon, enable, errp)) { + if (!qmp_caps_accept(mon, enable, errp)) { return; } =20 - cur_mon->qmp.commands =3D &qmp_commands; + mon->commands =3D &qmp_commands; } =20 /* Set the current CPU defined by the user. Callers must hold BQL. */ @@ -4121,27 +4133,27 @@ static int monitor_can_read(void *opaque) * Null @rsp can only happen for commands with QCO_NO_SUCCESS_RESP. * Nothing is emitted then. */ -static void monitor_qmp_respond(Monitor *mon, QDict *rsp) +static void monitor_qmp_respond(MonitorQMP *mon, QDict *rsp) { if (rsp) { qmp_send_response(mon, rsp); } } =20 -static void monitor_qmp_dispatch(Monitor *mon, QObject *req) +static void monitor_qmp_dispatch(MonitorQMP *mon, QObject *req) { Monitor *old_mon; QDict *rsp; QDict *error; =20 old_mon =3D cur_mon; - cur_mon =3D mon; + cur_mon =3D &mon->common; =20 - rsp =3D qmp_dispatch(mon->qmp.commands, req, qmp_oob_enabled(mon)); + rsp =3D qmp_dispatch(mon->commands, req, qmp_oob_enabled(mon)); =20 cur_mon =3D old_mon; =20 - if (mon->qmp.commands =3D=3D &qmp_cap_negotiation_commands) { + if (mon->commands =3D=3D &qmp_cap_negotiation_commands) { error =3D qdict_get_qdict(rsp, "error"); if (error && !g_strcmp0(qdict_get_try_str(error, "class"), @@ -4166,24 +4178,30 @@ static void monitor_qmp_dispatch(Monitor *mon, QObj= ect *req) * monitor to the end of mon_list queue. * * Note: if the function returned with non-NULL, then the caller will - * be with mon->qmp.qmp_queue_lock held, and the caller is responsible + * be with qmp_mon->qmp_queue_lock held, and the caller is responsible * to release it. */ static QMPRequest *monitor_qmp_requests_pop_any_with_lock(void) { QMPRequest *req_obj =3D NULL; Monitor *mon; + MonitorQMP *qmp_mon; =20 qemu_mutex_lock(&monitor_lock); =20 QTAILQ_FOREACH(mon, &mon_list, entry) { - qemu_mutex_lock(&mon->qmp.qmp_queue_lock); - req_obj =3D g_queue_pop_head(mon->qmp.qmp_requests); + if (!monitor_is_qmp(mon)) { + continue; + } + + qmp_mon =3D container_of(mon, MonitorQMP, common); + qemu_mutex_lock(&qmp_mon->qmp_queue_lock); + req_obj =3D g_queue_pop_head(qmp_mon->qmp_requests); if (req_obj) { /* With the lock of corresponding queue held */ break; } - qemu_mutex_unlock(&mon->qmp.qmp_queue_lock); + qemu_mutex_unlock(&qmp_mon->qmp_queue_lock); } =20 if (req_obj) { @@ -4205,7 +4223,7 @@ static void monitor_qmp_bh_dispatcher(void *data) QMPRequest *req_obj =3D monitor_qmp_requests_pop_any_with_lock(); QDict *rsp; bool need_resume; - Monitor *mon; + MonitorQMP *mon; =20 if (!req_obj) { return; @@ -4214,8 +4232,8 @@ static void monitor_qmp_bh_dispatcher(void *data) mon =3D req_obj->mon; /* qmp_oob_enabled() might change after "qmp_capabilities" */ need_resume =3D !qmp_oob_enabled(mon) || - mon->qmp.qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1; - qemu_mutex_unlock(&mon->qmp.qmp_queue_lock); + mon->qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1; + qemu_mutex_unlock(&mon->qmp_queue_lock); if (req_obj->req) { QDict *qdict =3D qobject_to(QDict, req_obj->req); QObject *id =3D qdict ? qdict_get(qdict, "id") : NULL; @@ -4231,7 +4249,7 @@ static void monitor_qmp_bh_dispatcher(void *data) =20 if (need_resume) { /* Pairs with the monitor_suspend() in handle_qmp_command() */ - monitor_resume(mon); + monitor_resume(&mon->common); } qmp_request_free(req_obj); =20 @@ -4241,7 +4259,7 @@ static void monitor_qmp_bh_dispatcher(void *data) =20 static void handle_qmp_command(void *opaque, QObject *req, Error *err) { - Monitor *mon =3D opaque; + MonitorQMP *mon =3D opaque; QObject *id =3D NULL; QDict *qdict; QMPRequest *req_obj; @@ -4273,7 +4291,7 @@ static void handle_qmp_command(void *opaque, QObject = *req, Error *err) req_obj->err =3D err; =20 /* Protect qmp_requests and fetching its length. */ - qemu_mutex_lock(&mon->qmp.qmp_queue_lock); + qemu_mutex_lock(&mon->qmp_queue_lock); =20 /* * Suspend the monitor when we can't queue more requests after @@ -4282,8 +4300,8 @@ static void handle_qmp_command(void *opaque, QObject = *req, Error *err) * command, for backward compatibility. */ if (!qmp_oob_enabled(mon) || - mon->qmp.qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1) { - monitor_suspend(mon); + mon->qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1) { + monitor_suspend(&mon->common); } =20 /* @@ -4291,9 +4309,9 @@ static void handle_qmp_command(void *opaque, QObject = *req, Error *err) * handled in time order. Ownership for req_obj, req, * etc. will be delivered to the handler side. */ - assert(mon->qmp.qmp_requests->length < QMP_REQ_QUEUE_LEN_MAX); - g_queue_push_tail(mon->qmp.qmp_requests, req_obj); - qemu_mutex_unlock(&mon->qmp.qmp_queue_lock); + assert(mon->qmp_requests->length < QMP_REQ_QUEUE_LEN_MAX); + g_queue_push_tail(mon->qmp_requests, req_obj); + qemu_mutex_unlock(&mon->qmp_queue_lock); =20 /* Kick the dispatcher routine */ qemu_bh_schedule(qmp_dispatcher_bh); @@ -4301,9 +4319,9 @@ static void handle_qmp_command(void *opaque, QObject = *req, Error *err) =20 static void monitor_qmp_read(void *opaque, const uint8_t *buf, int size) { - Monitor *mon =3D opaque; + MonitorQMP *mon =3D opaque; =20 - json_message_parser_feed(&mon->qmp.parser, (const char *) buf, size); + json_message_parser_feed(&mon->parser, (const char *) buf, size); } =20 static void monitor_read(void *opaque, const uint8_t *buf, int size) @@ -4389,7 +4407,7 @@ void monitor_resume(Monitor *mon) trace_monitor_suspend(mon, -1); } =20 -static QDict *qmp_greeting(Monitor *mon) +static QDict *qmp_greeting(MonitorQMP *mon) { QList *cap_list =3D qlist_new(); QObject *ver =3D NULL; @@ -4398,7 +4416,7 @@ static QDict *qmp_greeting(Monitor *mon) qmp_marshal_query_version(NULL, &ver, NULL); =20 for (cap =3D 0; cap < QMP_CAPABILITY__MAX; cap++) { - if (mon->qmp.capab_offered[cap]) { + if (mon->capab_offered[cap]) { qlist_append_str(cap_list, QMPCapability_str(cap)); } } @@ -4411,11 +4429,11 @@ static QDict *qmp_greeting(Monitor *mon) static void monitor_qmp_event(void *opaque, int event) { QDict *data; - Monitor *mon =3D opaque; + MonitorQMP *mon =3D opaque; =20 switch (event) { case CHR_EVENT_OPENED: - mon->qmp.commands =3D &qmp_cap_negotiation_commands; + mon->commands =3D &qmp_cap_negotiation_commands; monitor_qmp_caps_reset(mon); data =3D qmp_greeting(mon); qmp_send_response(mon, data); @@ -4430,8 +4448,8 @@ static void monitor_qmp_event(void *opaque, int event) * is closed. */ monitor_qmp_cleanup_queues(mon); - json_message_parser_destroy(&mon->qmp.parser); - json_message_parser_init(&mon->qmp.parser, handle_qmp_command, + json_message_parser_destroy(&mon->parser); + json_message_parser_init(&mon->parser, handle_qmp_command, mon, NULL); mon_refcount--; monitor_fdsets_cleanup(); @@ -4593,30 +4611,34 @@ static void monitor_list_append(Monitor *mon) =20 static void monitor_qmp_setup_handlers_bh(void *opaque) { - Monitor *mon =3D opaque; + MonitorQMP *mon =3D opaque; GMainContext *context; =20 - assert(mon->use_io_thread); + assert(mon->common.use_io_thread); context =3D iothread_get_g_main_context(mon_iothread); assert(context); - qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, monitor_qmp_read, - monitor_qmp_event, NULL, mon, context, true); - monitor_list_append(mon); + qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, + monitor_qmp_read, monitor_qmp_event, + NULL, &mon->common, context, true); + monitor_list_append(&mon->common); } =20 static void monitor_init_qmp(Chardev *chr, int flags) { - Monitor *mon =3D g_malloc(sizeof(*mon)); + MonitorQMP *mon =3D g_malloc0(sizeof(*mon)); =20 /* Note: we run QMP monitor in I/O thread when @chr supports that */ - monitor_data_init(mon, flags, false, + monitor_data_init(&mon->common, flags, false, qemu_chr_has_feature(chr, QEMU_CHAR_FEATURE_GCONTEXT= )); =20 - qemu_chr_fe_init(&mon->chr, chr, &error_abort); - qemu_chr_fe_set_echo(&mon->chr, true); + qemu_mutex_init(&mon->qmp_queue_lock); + mon->qmp_requests =3D g_queue_new(); =20 - json_message_parser_init(&mon->qmp.parser, handle_qmp_command, mon, NU= LL); - if (mon->use_io_thread) { + qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); + qemu_chr_fe_set_echo(&mon->common.chr, true); + + json_message_parser_init(&mon->parser, handle_qmp_command, mon, NULL); + if (mon->common.use_io_thread) { /* * Make sure the old iowatch is gone. It's possible when * e.g. the chardev is in client mode, with wait=3Don. @@ -4631,10 +4653,10 @@ static void monitor_init_qmp(Chardev *chr, int flag= s) monitor_qmp_setup_handlers_bh, mon); /* The bottom half will add @mon to @mon_list */ } else { - qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, + qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, monitor_qmp_read, monitor_qmp_event, - NULL, mon, NULL, true); - monitor_list_append(mon); + NULL, &mon->common, NULL, true); + monitor_list_append(&mon->common); } } =20 --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559916370; cv=none; d=zoho.com; s=zohoarc; b=S3l8yO2SDtTiF08zYxJL6w7lZGkmztk3V4NrE47GR7uETFU5Kc7McIPDTfq/UpViWOHeyp7pTlshjarO1gE6JpTzknDiHz/C0UZXfVHSVibu3fNy7DVjzo7Gl1hOJQpDaKfFEztvV2qmpum15Ax36gdG20MJ3D8n2GbEktX85B0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559916370; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=o0wqCVzozPrNAItLIVB0DgObT/lbVQKnvEkGB/wq9NI=; b=d7UUuK3u4eCicwOCq0LKXD9meDsLtGDDPUhsWnhJbuWkHpjuPueL2vroKA3/4wgWk8gdsL7Y8Y/3EOqluCEwleIln34KVgy6oB7dNr7lu9LPiYJwEtgCyfM8X1E8xK/yxWUVsg3WRUvHV15IfQdkLYuWhteomeV6ZYBhFYVA2MM= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559916370392488.61237564520627; Fri, 7 Jun 2019 07:06:10 -0700 (PDT) Received: from localhost ([::1]:51480 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFVL-0001jE-V2 for importer@patchew.org; Fri, 07 Jun 2019 10:06:03 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45201) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKl-0001lJ-L0 for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKj-0003Mo-4O for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:07 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40666) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKa-0002nD-8V; Fri, 07 Jun 2019 09:54:58 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 9ABE581124; Fri, 7 Jun 2019 13:54:43 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id 654151001B15; Fri, 7 Jun 2019 13:54:41 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:24 +0200 Message-Id: <20190607135430.22149-5-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Fri, 07 Jun 2019 13:54:43 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 04/10] monitor: Create MonitorHMP with readline state X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" The ReadLineState in Monitor is only used for HMP monitors. Create MonitorHMP and move it there. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- include/monitor/monitor.h | 5 +- hmp.c | 4 +- monitor.c | 123 +++++++++++++++++++++----------------- 3 files changed, 75 insertions(+), 57 deletions(-) diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h index 86656297f1..1ba354f811 100644 --- a/include/monitor/monitor.h +++ b/include/monitor/monitor.h @@ -7,6 +7,7 @@ #include "qemu/readline.h" =20 extern __thread Monitor *cur_mon; +typedef struct MonitorHMP MonitorHMP; =20 /* flags for monitor_init */ /* 0x01 unused */ @@ -35,8 +36,8 @@ void monitor_flush(Monitor *mon); int monitor_set_cpu(int cpu_index); int monitor_get_cpu_index(void); =20 -void monitor_read_command(Monitor *mon, int show_prompt); -int monitor_read_password(Monitor *mon, ReadLineFunc *readline_func, +void monitor_read_command(MonitorHMP *mon, int show_prompt); +int monitor_read_password(MonitorHMP *mon, ReadLineFunc *readline_func, void *opaque); =20 AddfdInfo *monitor_fdset_add_fd(int fd, bool has_fdset_id, int64_t fdset_i= d, diff --git a/hmp.c b/hmp.c index be5e345c6f..99414cd39c 100644 --- a/hmp.c +++ b/hmp.c @@ -1943,6 +1943,8 @@ static void hmp_change_read_arg(void *opaque, const c= har *password, =20 void hmp_change(Monitor *mon, const QDict *qdict) { + /* FIXME Make MonitorHMP public and use container_of */ + MonitorHMP *hmp_mon =3D (MonitorHMP *) mon; const char *device =3D qdict_get_str(qdict, "device"); const char *target =3D qdict_get_str(qdict, "target"); const char *arg =3D qdict_get_try_str(qdict, "arg"); @@ -1960,7 +1962,7 @@ void hmp_change(Monitor *mon, const QDict *qdict) if (strcmp(target, "passwd") =3D=3D 0 || strcmp(target, "password") =3D=3D 0) { if (!arg) { - monitor_read_password(mon, hmp_change_read_arg, NULL); + monitor_read_password(hmp_mon, hmp_change_read_arg, NULL); return; } } diff --git a/monitor.c b/monitor.c index d18cf18393..810f3dcf9c 100644 --- a/monitor.c +++ b/monitor.c @@ -190,14 +190,6 @@ struct Monitor { bool skip_flush; bool use_io_thread; =20 - /* - * State used only in the thread "owning" the monitor. - * If @use_io_thread, this is @mon_iothread. - * Else, it's the main thread. - * These members can be safely accessed without locks. - */ - ReadLineState *rs; - gchar *mon_cpu_path; mon_cmd_t *cmd_table; QTAILQ_ENTRY(Monitor) entry; @@ -218,6 +210,17 @@ struct Monitor { int mux_out; }; =20 +struct MonitorHMP { + Monitor common; + /* + * State used only in the thread "owning" the monitor. + * If @use_io_thread, this is @mon_iothread. + * Else, it's the main thread. + * These members can be safely accessed without locks. + */ + ReadLineState *rs; +}; + typedef struct { Monitor common; JSONMessageParser parser; @@ -324,7 +327,7 @@ bool monitor_cur_is_qmp(void) return cur_mon && monitor_is_qmp(cur_mon); } =20 -void monitor_read_command(Monitor *mon, int show_prompt) +void monitor_read_command(MonitorHMP *mon, int show_prompt) { if (!mon->rs) return; @@ -334,7 +337,7 @@ void monitor_read_command(Monitor *mon, int show_prompt) readline_show_prompt(mon->rs); } =20 -int monitor_read_password(Monitor *mon, ReadLineFunc *readline_func, +int monitor_read_password(MonitorHMP *mon, ReadLineFunc *readline_func, void *opaque) { if (mon->rs) { @@ -342,7 +345,8 @@ int monitor_read_password(Monitor *mon, ReadLineFunc *r= eadline_func, /* prompt is printed on return from the command handler */ return 0; } else { - monitor_printf(mon, "terminal does not support password prompting\= n"); + monitor_printf(&mon->common, + "terminal does not support password prompting\n"); return -ENOTTY; } } @@ -703,7 +707,7 @@ static void monitor_qapi_event_init(void) qapi_event_throttle_equal); } =20 -static void handle_hmp_command(Monitor *mon, const char *cmdline); +static void handle_hmp_command(MonitorHMP *mon, const char *cmdline); =20 static void monitor_iothread_init(void); =20 @@ -738,8 +742,10 @@ static void monitor_data_destroy(Monitor *mon) if (monitor_is_qmp(mon)) { MonitorQMP *qmp_mon =3D container_of(mon, MonitorQMP, common); monitor_data_destroy_qmp(qmp_mon); + } else { + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); + readline_free(hmp_mon->rs); } - readline_free(mon->rs); qobject_unref(mon->outbuf); qemu_mutex_destroy(&mon->mon_lock); } @@ -748,12 +754,13 @@ char *qmp_human_monitor_command(const char *command_l= ine, bool has_cpu_index, int64_t cpu_index, Error **errp) { char *output =3D NULL; - Monitor *old_mon, hmp; + Monitor *old_mon; + MonitorHMP hmp =3D {}; =20 - monitor_data_init(&hmp, 0, true, false); + monitor_data_init(&hmp.common, 0, true, false); =20 old_mon =3D cur_mon; - cur_mon =3D &hmp; + cur_mon =3D &hmp.common; =20 if (has_cpu_index) { int ret =3D monitor_set_cpu(cpu_index); @@ -768,16 +775,16 @@ char *qmp_human_monitor_command(const char *command_l= ine, bool has_cpu_index, handle_hmp_command(&hmp, command_line); cur_mon =3D old_mon; =20 - qemu_mutex_lock(&hmp.mon_lock); - if (qstring_get_length(hmp.outbuf) > 0) { - output =3D g_strdup(qstring_get_str(hmp.outbuf)); + qemu_mutex_lock(&hmp.common.mon_lock); + if (qstring_get_length(hmp.common.outbuf) > 0) { + output =3D g_strdup(qstring_get_str(hmp.common.outbuf)); } else { output =3D g_strdup(""); } - qemu_mutex_unlock(&hmp.mon_lock); + qemu_mutex_unlock(&hmp.common.mon_lock); =20 out: - monitor_data_destroy(&hmp); + monitor_data_destroy(&hmp.common); return output; } =20 @@ -1341,14 +1348,15 @@ static void hmp_info_sync_profile(Monitor *mon, con= st QDict *qdict) =20 static void hmp_info_history(Monitor *mon, const QDict *qdict) { + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); int i; const char *str; =20 - if (!mon->rs) + if (!hmp_mon->rs) return; i =3D 0; for(;;) { - str =3D readline_get_history(mon->rs, i); + str =3D readline_get_history(hmp_mon->rs, i); if (!str) break; monitor_printf(mon, "%d: '%s'\n", i, str); @@ -3048,11 +3056,12 @@ static const mon_cmd_t *search_dispatch_table(const= mon_cmd_t *disp_table, * Do not assume the return value points into @table! It doesn't when * the command is found in a sub-command table. */ -static const mon_cmd_t *monitor_parse_command(Monitor *mon, +static const mon_cmd_t *monitor_parse_command(MonitorHMP *hmp_mon, const char *cmdp_start, const char **cmdp, mon_cmd_t *table) { + Monitor *mon =3D &hmp_mon->common; const char *p; const mon_cmd_t *cmd; char cmdname[256]; @@ -3083,7 +3092,7 @@ static const mon_cmd_t *monitor_parse_command(Monitor= *mon, *cmdp =3D p; /* search sub command */ if (cmd->sub_table !=3D NULL && *p !=3D '\0') { - return monitor_parse_command(mon, cmdp_start, cmdp, cmd->sub_table= ); + return monitor_parse_command(hmp_mon, cmdp_start, cmdp, cmd->sub_t= able); } =20 return cmd; @@ -3460,7 +3469,7 @@ fail: return NULL; } =20 -static void handle_hmp_command(Monitor *mon, const char *cmdline) +static void handle_hmp_command(MonitorHMP *mon, const char *cmdline) { QDict *qdict; const mon_cmd_t *cmd; @@ -3468,26 +3477,26 @@ static void handle_hmp_command(Monitor *mon, const = char *cmdline) =20 trace_handle_hmp_command(mon, cmdline); =20 - cmd =3D monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table); + cmd =3D monitor_parse_command(mon, cmdline, &cmdline, mon->common.cmd_= table); if (!cmd) { return; } =20 - qdict =3D monitor_parse_arguments(mon, &cmdline, cmd); + qdict =3D monitor_parse_arguments(&mon->common, &cmdline, cmd); if (!qdict) { while (cmdline > cmd_start && qemu_isspace(cmdline[-1])) { cmdline--; } - monitor_printf(mon, "Try \"help %.*s\" for more information\n", + monitor_printf(&mon->common, "Try \"help %.*s\" for more informati= on\n", (int)(cmdline - cmd_start), cmd_start); return; } =20 - cmd->cmd(mon, qdict); + cmd->cmd(&mon->common, qdict); qobject_unref(qdict); } =20 -static void cmd_completion(Monitor *mon, const char *name, const char *lis= t) +static void cmd_completion(MonitorHMP *mon, const char *name, const char *= list) { const char *p, *pstart; char cmd[128]; @@ -3511,7 +3520,7 @@ static void cmd_completion(Monitor *mon, const char *= name, const char *list) } } =20 -static void file_completion(Monitor *mon, const char *input) +static void file_completion(MonitorHMP *mon, const char *input) { DIR *ffs; struct dirent *d; @@ -4000,7 +4009,7 @@ void loadvm_completion(ReadLineState *rs, int nb_args= , const char *str) } } =20 -static void monitor_find_completion_by_table(Monitor *mon, +static void monitor_find_completion_by_table(MonitorHMP *mon, const mon_cmd_t *cmd_table, char **args, int nb_args) @@ -4095,7 +4104,7 @@ static void monitor_find_completion_by_table(Monitor = *mon, static void monitor_find_completion(void *opaque, const char *cmdline) { - Monitor *mon =3D opaque; + MonitorHMP *mon =3D opaque; char *args[MAX_ARGS]; int nb_args, len; =20 @@ -4115,7 +4124,7 @@ static void monitor_find_completion(void *opaque, } =20 /* 2. auto complete according to args */ - monitor_find_completion_by_table(mon, mon->cmd_table, args, nb_args); + monitor_find_completion_by_table(mon, mon->common.cmd_table, args, nb_= args); =20 cleanup: free_cmdline_args(args, nb_args); @@ -4326,19 +4335,21 @@ static void monitor_qmp_read(void *opaque, const ui= nt8_t *buf, int size) =20 static void monitor_read(void *opaque, const uint8_t *buf, int size) { + MonitorHMP *mon; Monitor *old_mon =3D cur_mon; int i; =20 cur_mon =3D opaque; + mon =3D container_of(cur_mon, MonitorHMP, common); =20 - if (cur_mon->rs) { + if (mon->rs) { for (i =3D 0; i < size; i++) - readline_handle_byte(cur_mon->rs, buf[i]); + readline_handle_byte(mon->rs, buf[i]); } else { if (size =3D=3D 0 || buf[size - 1] !=3D 0) monitor_printf(cur_mon, "corrupted command\n"); else - handle_hmp_command(cur_mon, (char *)buf); + handle_hmp_command(mon, (char *)buf); } =20 cur_mon =3D old_mon; @@ -4347,11 +4358,11 @@ static void monitor_read(void *opaque, const uint8_= t *buf, int size) static void monitor_command_cb(void *opaque, const char *cmdline, void *readline_opaque) { - Monitor *mon =3D opaque; + MonitorHMP *mon =3D opaque; =20 - monitor_suspend(mon); + monitor_suspend(&mon->common); handle_hmp_command(mon, cmdline); - monitor_resume(mon); + monitor_resume(&mon->common); } =20 int monitor_suspend(Monitor *mon) @@ -4397,8 +4408,9 @@ void monitor_resume(Monitor *mon) } =20 if (!monitor_is_qmp(mon)) { - assert(mon->rs); - readline_show_prompt(mon->rs); + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); + assert(hmp_mon->rs); + readline_show_prompt(hmp_mon->rs); } =20 aio_bh_schedule_oneshot(ctx, monitor_accept_input, mon); @@ -4460,6 +4472,7 @@ static void monitor_qmp_event(void *opaque, int event) static void monitor_event(void *opaque, int event) { Monitor *mon =3D opaque; + MonitorHMP *hmp_mon =3D container_of(cur_mon, MonitorHMP, common); =20 switch (event) { case CHR_EVENT_MUX_IN: @@ -4467,7 +4480,7 @@ static void monitor_event(void *opaque, int event) mon->mux_out =3D 0; qemu_mutex_unlock(&mon->mon_lock); if (mon->reset_seen) { - readline_restart(mon->rs); + readline_restart(hmp_mon->rs); monitor_resume(mon); monitor_flush(mon); } else { @@ -4494,8 +4507,8 @@ static void monitor_event(void *opaque, int event) monitor_printf(mon, "QEMU %s monitor - type 'help' for more " "information\n", QEMU_VERSION); if (!mon->mux_out) { - readline_restart(mon->rs); - readline_show_prompt(mon->rs); + readline_restart(hmp_mon->rs); + readline_show_prompt(hmp_mon->rs); } mon->reset_seen =3D 1; mon_refcount++; @@ -4556,15 +4569,17 @@ void monitor_init_globals(void) static void GCC_FMT_ATTR(2, 3) monitor_readline_printf(void *opaque, const char *fmt, ..= .) { + MonitorHMP *mon =3D opaque; va_list ap; va_start(ap, fmt); - monitor_vprintf(opaque, fmt, ap); + monitor_vprintf(&mon->common, fmt, ap); va_end(ap); } =20 static void monitor_readline_flush(void *opaque) { - monitor_flush(opaque); + MonitorHMP *mon =3D opaque; + monitor_flush(&mon->common); } =20 /* @@ -4662,11 +4677,11 @@ static void monitor_init_qmp(Chardev *chr, int flag= s) =20 static void monitor_init_hmp(Chardev *chr, int flags) { - Monitor *mon =3D g_malloc(sizeof(*mon)); + MonitorHMP *mon =3D g_malloc0(sizeof(*mon)); bool use_readline =3D flags & MONITOR_USE_READLINE; =20 - monitor_data_init(mon, flags, false, false); - qemu_chr_fe_init(&mon->chr, chr, &error_abort); + monitor_data_init(&mon->common, flags, false, false); + qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); =20 if (use_readline) { mon->rs =3D readline_init(monitor_readline_printf, @@ -4676,9 +4691,9 @@ static void monitor_init_hmp(Chardev *chr, int flags) monitor_read_command(mon, 0); } =20 - qemu_chr_fe_set_handlers(&mon->chr, monitor_can_read, monitor_read, - monitor_event, NULL, mon, NULL, true); - monitor_list_append(mon); + qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, monitor_r= ead, + monitor_event, NULL, &mon->common, NULL, true= ); + monitor_list_append(&mon->common); } =20 void monitor_init(Chardev *chr, int flags) --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559915993; cv=none; d=zoho.com; s=zohoarc; b=RRNWKIWtFD54BJ5gWa79xY3Bys2Fk/VpQc8LuivZyBIMxvsDppXKSGd1X8XzWl19QzGb0u7IrpKGFYdVfulnEB436imflymB6ai2oSGGbFe30PvbGRQO27/EEdJI+DxxZ91cwCN3aJ3YiJTfYgKdpaT2IwWvMALtMRgs5+Xx1FY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559915993; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=gyUfsOS3qZQKuJKF4cFKsg+0lNJQSet/KKcKRSFR/nA=; b=hOfXaN9qkANv4Wo2Lxigfoz/l2akuSQ9YAJ1+MQrPMjXwaSSqt87C0OsPpWXNkdUYji0DpIW85041xWTL+zJ1F4DLm1f/8irH/t7cMHkEHzO+iiBQXhURKrjS1f/bxCybgXHmzM491gNWpMdjttl+7F02dRtsx7X3AI9rNAlyYM= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (209.51.188.47 [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559915993321328.7525398169995; Fri, 7 Jun 2019 06:59:53 -0700 (PDT) Received: from localhost ([::1]:51438 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFPE-0006ol-1v for importer@patchew.org; Fri, 07 Jun 2019 09:59:44 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45104) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKf-0001ae-QW for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKd-0003Ca-SQ for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:01 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40050) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKZ-0002jl-Vs; Fri, 07 Jun 2019 09:54:56 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 1CCA630BC578; Fri, 7 Jun 2019 13:54:46 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id DD2531001B0C; Fri, 7 Jun 2019 13:54:43 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:25 +0200 Message-Id: <20190607135430.22149-6-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.49]); Fri, 07 Jun 2019 13:54:46 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 05/10] monitor: Move cmd_table to MonitorHMP X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Monitor.cmd_table contains the handlers for HMP commands, so there is no reason to keep it in the state shared with QMP. Move it to MonitorHMP. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- monitor.c | 23 +++++++++++++++-------- 1 file changed, 15 insertions(+), 8 deletions(-) diff --git a/monitor.c b/monitor.c index 810f3dcf9c..d964dd1969 100644 --- a/monitor.c +++ b/monitor.c @@ -191,7 +191,6 @@ struct Monitor { bool use_io_thread; =20 gchar *mon_cpu_path; - mon_cmd_t *cmd_table; QTAILQ_ENTRY(Monitor) entry; =20 /* @@ -219,6 +218,7 @@ struct MonitorHMP { * These members can be safely accessed without locks. */ ReadLineState *rs; + mon_cmd_t *cmd_table; }; =20 typedef struct { @@ -720,13 +720,19 @@ static void monitor_data_init(Monitor *mon, int flags= , bool skip_flush, memset(mon, 0, sizeof(Monitor)); qemu_mutex_init(&mon->mon_lock); mon->outbuf =3D qstring_new(); - /* Use *mon_cmds by default. */ - mon->cmd_table =3D mon_cmds; mon->skip_flush =3D skip_flush; mon->use_io_thread =3D use_io_thread; mon->flags =3D flags; } =20 +static void monitor_data_init_hmp(MonitorHMP *mon, int flags, bool skip_fl= ush) +{ + monitor_data_init(&mon->common, flags, skip_flush, false); + + /* Use *mon_cmds by default. */ + mon->cmd_table =3D mon_cmds; +} + static void monitor_data_destroy_qmp(MonitorQMP *mon) { json_message_parser_destroy(&mon->parser); @@ -757,7 +763,7 @@ char *qmp_human_monitor_command(const char *command_lin= e, bool has_cpu_index, Monitor *old_mon; MonitorHMP hmp =3D {}; =20 - monitor_data_init(&hmp.common, 0, true, false); + monitor_data_init_hmp(&hmp, 0, true); =20 old_mon =3D cur_mon; cur_mon =3D &hmp.common; @@ -1002,6 +1008,7 @@ static void help_cmd_dump(Monitor *mon, const mon_cmd= _t *cmds, =20 static void help_cmd(Monitor *mon, const char *name) { + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); char *args[MAX_ARGS]; int nb_args =3D 0; =20 @@ -1024,7 +1031,7 @@ static void help_cmd(Monitor *mon, const char *name) } =20 /* 2. dump the contents according to parsed args */ - help_cmd_dump(mon, mon->cmd_table, args, nb_args, 0); + help_cmd_dump(mon, hmp_mon->cmd_table, args, nb_args, 0); =20 free_cmdline_args(args, nb_args); } @@ -3477,7 +3484,7 @@ static void handle_hmp_command(MonitorHMP *mon, const= char *cmdline) =20 trace_handle_hmp_command(mon, cmdline); =20 - cmd =3D monitor_parse_command(mon, cmdline, &cmdline, mon->common.cmd_= table); + cmd =3D monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table); if (!cmd) { return; } @@ -4124,7 +4131,7 @@ static void monitor_find_completion(void *opaque, } =20 /* 2. auto complete according to args */ - monitor_find_completion_by_table(mon, mon->common.cmd_table, args, nb_= args); + monitor_find_completion_by_table(mon, mon->cmd_table, args, nb_args); =20 cleanup: free_cmdline_args(args, nb_args); @@ -4680,7 +4687,7 @@ static void monitor_init_hmp(Chardev *chr, int flags) MonitorHMP *mon =3D g_malloc0(sizeof(*mon)); bool use_readline =3D flags & MONITOR_USE_READLINE; =20 - monitor_data_init(&mon->common, flags, false, false); + monitor_data_init_hmp(mon, flags, false); qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); =20 if (use_readline) { --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559915832; cv=none; d=zoho.com; s=zohoarc; b=WRJX8Yp4sul7lYIRxwPEX6r/efGOILRXTwp1gs2/oqg6uyN9zhh3HP9RZF3LwYDK1EAOpzEHuMOOuzJ2UHtCeY6s2ovjwcDmSiU2UudWTrXhkbAGiM86zOhNFnsP9onHY0SG1F/Raed3MERmhaA40zmIjqau/9AbCekXg9odtmM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559915832; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=KYJMCmwR5b+bNi1IjfpSDVtYHvOfDs7S4K/C0pUMMS0=; b=IRu5cfH2BmY1eEWBj0Xug3qAW2sWl0t8ib5eT1bJ5Iyn2Ma6JNnSVX01IZidfN3/Gwi4icJsNifQZ9hRV6yWudst8yx0J4vcwHBEoEjglJSoO7xQcY5r0MZrWMBYJoFmCi6h/El8KtODbWM//htBhCPoAZxHx2IPiivENJjM4Ac= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559915832953815.3002571174845; Fri, 7 Jun 2019 06:57:12 -0700 (PDT) Received: from localhost ([::1]:51398 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFMk-0003i4-C4 for importer@patchew.org; Fri, 07 Jun 2019 09:57:11 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45176) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKk-0001jl-7m for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:07 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKi-0003M2-Ty for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:06 -0400 Received: from mx1.redhat.com ([209.132.183.28]:39932) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKf-0003Ed-Ex; Fri, 07 Jun 2019 09:55:01 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 9197F3092647; Fri, 7 Jun 2019 13:54:47 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id 5D6FB1001B0C; Fri, 7 Jun 2019 13:54:46 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:26 +0200 Message-Id: <20190607135430.22149-7-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.43]); Fri, 07 Jun 2019 13:54:55 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 06/10] Move monitor.c to monitor/misc.c X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Create a new monitor/ subdirectory and move monitor.c there. As the plan is to move the monitor core into separate files, use the chance to rename it to misc.c. Signed-off-by: Kevin Wolf --- monitor.c =3D> monitor/misc.c | 0 Makefile.target | 3 ++- monitor/Makefile.objs | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) rename monitor.c =3D> monitor/misc.c (100%) create mode 100644 monitor/Makefile.objs diff --git a/monitor.c b/monitor/misc.c similarity index 100% rename from monitor.c rename to monitor/misc.c diff --git a/Makefile.target b/Makefile.target index ecd856e3a3..72c267f7dc 100644 --- a/Makefile.target +++ b/Makefile.target @@ -148,9 +148,10 @@ endif #CONFIG_BSD_USER ######################################################### # System emulator target ifdef CONFIG_SOFTMMU -obj-y +=3D arch_init.o cpus.o monitor.o gdbstub.o balloon.o ioport.o numa.o +obj-y +=3D arch_init.o cpus.o gdbstub.o balloon.o ioport.o numa.o obj-y +=3D qtest.o obj-y +=3D hw/ +obj-y +=3D monitor/ obj-y +=3D qapi/ obj-y +=3D memory.o obj-y +=3D memory_mapping.o diff --git a/monitor/Makefile.objs b/monitor/Makefile.objs new file mode 100644 index 0000000000..e783b0616b --- /dev/null +++ b/monitor/Makefile.objs @@ -0,0 +1 @@ +obj-y +=3D misc.o --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559916194; cv=none; d=zoho.com; s=zohoarc; b=L3Np0qQ04Hj6uJjIL357yoBb9y2B7eLgFXvW21jFFcAyf7g/uMOFJnMfcks7fmAWBbM/ewxZ/wltWKF8oDf58sXDGz1RXvlMDF5bFB8Ol4+Pp/Wr2Usu6ftqnKSlc1etwpgj57HxrC3EsuDBg+iHQ3XwUf32dZ48QydPRoAvJj0= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559916194; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=OXxKZ2jfsMPfNcSL5Fm2ACrbZR7CGWKJTb+fcoM+t+4=; b=FiaLbeuJGmDpqsBwRYgteWcGlI4+72BXVeVPKPcOTFe5bvydIhYjh0LenlK7fBFpllgwTLAR+XZLwEJUGsykRQCphsmaTRGx9hlLhGMH/RzDHVbFrWTAi5dsdA2QAbkA5E+x6ksW0fEAX8siBLydblfTamBnzUrr5augj8VYulg= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559916194556478.89997382468937; Fri, 7 Jun 2019 07:03:14 -0700 (PDT) Received: from localhost ([::1]:51474 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFSZ-0000lY-Gk for importer@patchew.org; Fri, 07 Jun 2019 10:03:11 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45199) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKl-0001lI-KR for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:11 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKi-0003M8-T3 for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:07 -0400 Received: from mx1.redhat.com ([209.132.183.28]:49094) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKc-00034B-7N; Fri, 07 Jun 2019 09:54:58 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 14C0C3082E42; Fri, 7 Jun 2019 13:54:50 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id D4D511001B0E; Fri, 7 Jun 2019 13:54:47 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:27 +0200 Message-Id: <20190607135430.22149-8-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.46]); Fri, 07 Jun 2019 13:54:55 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 07/10] monitor: Create monitor_int.h with common definitions X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Before we can split monitor.c, we need to create a header file that contains the common definitions that will be used by multiple source files. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- monitor/monitor_int.h | 147 ++++++++++++++++++++++++++++++++++++++++++ monitor/misc.c | 110 +------------------------------ 2 files changed, 148 insertions(+), 109 deletions(-) create mode 100644 monitor/monitor_int.h diff --git a/monitor/monitor_int.h b/monitor/monitor_int.h new file mode 100644 index 0000000000..ab87013b6f --- /dev/null +++ b/monitor/monitor_int.h @@ -0,0 +1,147 @@ +/* + * QEMU monitor + * + * Copyright (c) 2003-2004 Fabrice Bellard + * + * Permission is hereby granted, free of charge, to any person obtaining a= copy + * of this software and associated documentation files (the "Software"), t= o deal + * in the Software without restriction, including without limitation the r= ights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or se= ll + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included= in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS= OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OT= HER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING= FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS = IN + * THE SOFTWARE. + */ + +#ifndef MONITOR_INT_H +#define MONITOR_INT_H + +#include "qemu-common.h" +#include "monitor/monitor.h" + +#include "qapi/qmp/qdict.h" +#include "qapi/qmp/json-parser.h" +#include "qapi/qapi-commands.h" + +#include "qemu/readline.h" +#include "chardev/char-fe.h" + +/* + * Supported types: + * + * 'F' filename + * 'B' block device name + * 's' string (accept optional quote) + * 'S' it just appends the rest of the string (accept optional qu= ote) + * 'O' option string of the form NAME=3DVALUE,... + * parsed according to QemuOptsList given by its name + * Example: 'device:O' uses qemu_device_opts. + * Restriction: only lists with empty desc are supported + * TODO lift the restriction + * 'i' 32 bit integer + * 'l' target long (32 or 64 bit) + * 'M' Non-negative target long (32 or 64 bit), in user mode the + * value is multiplied by 2^20 (think Mebibyte) + * 'o' octets (aka bytes) + * user mode accepts an optional E, e, P, p, T, t, G, g, M, m, + * K, k suffix, which multiplies the value by 2^60 for suffix= es E + * and e, 2^50 for suffixes P and p, 2^40 for suffixes T and = t, + * 2^30 for suffixes G and g, 2^20 for M and m, 2^10 for K an= d k + * 'T' double + * user mode accepts an optional ms, us, ns suffix, + * which divides the value by 1e3, 1e6, 1e9, respectively + * '/' optional gdb-like print format (like "/10x") + * + * '?' optional type (for all types, except '/') + * '.' other form of optional type (for 'i' and 'l') + * 'b' boolean + * user mode accepts "on" or "off" + * '-' optional parameter (eg. '-f') + * + */ + +typedef struct mon_cmd_t { + const char *name; + const char *args_type; + const char *params; + const char *help; + const char *flags; /* p=3Dpreconfig */ + void (*cmd)(Monitor *mon, const QDict *qdict); + /* @sub_table is a list of 2nd level of commands. If it does not exist, + * cmd should be used. If it exists, sub_table[?].cmd should be + * used, and cmd of 1st level plays the role of help function. + */ + struct mon_cmd_t *sub_table; + void (*command_completion)(ReadLineState *rs, int nb_args, const char = *str); +} mon_cmd_t; + +struct Monitor { + CharBackend chr; + int reset_seen; + int flags; + int suspend_cnt; /* Needs to be accessed atomically */ + bool skip_flush; + bool use_io_thread; + + gchar *mon_cpu_path; + QTAILQ_ENTRY(Monitor) entry; + + /* + * The per-monitor lock. We can't access guest memory when holding + * the lock. + */ + QemuMutex mon_lock; + + /* + * Members that are protected by the per-monitor lock + */ + QLIST_HEAD(, mon_fd_t) fds; + QString *outbuf; + guint out_watch; + /* Read under either BQL or mon_lock, written with BQL+mon_lock. */ + int mux_out; +}; + +struct MonitorHMP { + Monitor common; + /* + * State used only in the thread "owning" the monitor. + * If @use_io_thread, this is @mon_iothread. + * Else, it's the main thread. + * These members can be safely accessed without locks. + */ + ReadLineState *rs; + mon_cmd_t *cmd_table; +}; + +typedef struct { + Monitor common; + JSONMessageParser parser; + /* + * When a client connects, we're in capabilities negotiation mode. + * @commands is &qmp_cap_negotiation_commands then. When command + * qmp_capabilities succeeds, we go into command mode, and + * @command becomes &qmp_commands. + */ + QmpCommandList *commands; + bool capab_offered[QMP_CAPABILITY__MAX]; /* capabilities offered */ + bool capab[QMP_CAPABILITY__MAX]; /* offered and accepted */ + /* + * Protects qmp request/response queue. + * Take monitor_lock first when you need both. + */ + QemuMutex qmp_queue_lock; + /* Input queue that holds all the parsed QMP requests */ + GQueue *qmp_requests; +} MonitorQMP; + +#endif diff --git a/monitor/misc.c b/monitor/misc.c index d964dd1969..6ae7561105 100644 --- a/monitor/misc.c +++ b/monitor/misc.c @@ -23,6 +23,7 @@ */ =20 #include "qemu/osdep.h" +#include "monitor_int.h" #include "qemu/units.h" #include #include "cpu.h" @@ -91,55 +92,6 @@ #include "hw/s390x/storage-attributes.h" #endif =20 -/* - * Supported types: - * - * 'F' filename - * 'B' block device name - * 's' string (accept optional quote) - * 'S' it just appends the rest of the string (accept optional qu= ote) - * 'O' option string of the form NAME=3DVALUE,... - * parsed according to QemuOptsList given by its name - * Example: 'device:O' uses qemu_device_opts. - * Restriction: only lists with empty desc are supported - * TODO lift the restriction - * 'i' 32 bit integer - * 'l' target long (32 or 64 bit) - * 'M' Non-negative target long (32 or 64 bit), in user mode the - * value is multiplied by 2^20 (think Mebibyte) - * 'o' octets (aka bytes) - * user mode accepts an optional E, e, P, p, T, t, G, g, M, m, - * K, k suffix, which multiplies the value by 2^60 for suffix= es E - * and e, 2^50 for suffixes P and p, 2^40 for suffixes T and = t, - * 2^30 for suffixes G and g, 2^20 for M and m, 2^10 for K an= d k - * 'T' double - * user mode accepts an optional ms, us, ns suffix, - * which divides the value by 1e3, 1e6, 1e9, respectively - * '/' optional gdb-like print format (like "/10x") - * - * '?' optional type (for all types, except '/') - * '.' other form of optional type (for 'i' and 'l') - * 'b' boolean - * user mode accepts "on" or "off" - * '-' optional parameter (eg. '-f') - * - */ - -typedef struct mon_cmd_t { - const char *name; - const char *args_type; - const char *params; - const char *help; - const char *flags; /* p=3Dpreconfig */ - void (*cmd)(Monitor *mon, const QDict *qdict); - /* @sub_table is a list of 2nd level of commands. If it does not exist, - * cmd should be used. If it exists, sub_table[?].cmd should be - * used, and cmd of 1st level plays the role of help function. - */ - struct mon_cmd_t *sub_table; - void (*command_completion)(ReadLineState *rs, int nb_args, const char = *str); -} mon_cmd_t; - /* file descriptors passed via SCM_RIGHTS */ typedef struct mon_fd_t mon_fd_t; struct mon_fd_t { @@ -182,66 +134,6 @@ typedef struct { int64_t rate; /* Minimum time (in ns) between two events */ } MonitorQAPIEventConf; =20 -struct Monitor { - CharBackend chr; - int reset_seen; - int flags; - int suspend_cnt; /* Needs to be accessed atomically */ - bool skip_flush; - bool use_io_thread; - - gchar *mon_cpu_path; - QTAILQ_ENTRY(Monitor) entry; - - /* - * The per-monitor lock. We can't access guest memory when holding - * the lock. - */ - QemuMutex mon_lock; - - /* - * Members that are protected by the per-monitor lock - */ - QLIST_HEAD(, mon_fd_t) fds; - QString *outbuf; - guint out_watch; - /* Read under either BQL or mon_lock, written with BQL+mon_lock. */ - int mux_out; -}; - -struct MonitorHMP { - Monitor common; - /* - * State used only in the thread "owning" the monitor. - * If @use_io_thread, this is @mon_iothread. - * Else, it's the main thread. - * These members can be safely accessed without locks. - */ - ReadLineState *rs; - mon_cmd_t *cmd_table; -}; - -typedef struct { - Monitor common; - JSONMessageParser parser; - /* - * When a client connects, we're in capabilities negotiation mode. - * @commands is &qmp_cap_negotiation_commands then. When command - * qmp_capabilities succeeds, we go into command mode, and - * @command becomes &qmp_commands. - */ - QmpCommandList *commands; - bool capab_offered[QMP_CAPABILITY__MAX]; /* capabilities offered */ - bool capab[QMP_CAPABILITY__MAX]; /* offered and accepted */ - /* - * Protects qmp request/response queue. - * Take monitor_lock first when you need both. - */ - QemuMutex qmp_queue_lock; - /* Input queue that holds all the parsed QMP requests */ - GQueue *qmp_requests; -} MonitorQMP; - /* Shared monitor I/O thread */ IOThread *mon_iothread; =20 --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559917027; cv=none; d=zoho.com; s=zohoarc; b=MxnVGywmDTeT/92ROkt8qNMECHDBzvC+umV+PQZ+WvpM9qqKn7qDVu3EyqocUo5fgvNVI88zgr9rKrKK2y8zFQvm0eYZKX6qcgNgCgPXrH/PjdN8NwKDVN/Ld2v/I824L25QxuRp+T8EVdf32IExuHsNuOkqB+Fzsdgopzmt4mM= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559917027; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=1Y3C3gejwkfUtifT5snZj0guG2DxvndbW3MyVDIa7qc=; b=PJcFRrS8kmhqDceQuaivwle/8m/BzPC3yPPE1Kif/yCdOXwYuRr6/SqXi25SgsFtsVvDxAhfCVbw31RKrYEE40tb6e7JV+pR5pJHkaHC7hua2jRoGtzH1RqGQh+wYPHPYIIJyd1g+qNJxTgAAVyWsrtb2iyRHom5LOydLxFg9Zw= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559917027674180.0381623095741; Fri, 7 Jun 2019 07:17:07 -0700 (PDT) Received: from localhost ([::1]:51598 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFfa-0001Ev-5g for importer@patchew.org; Fri, 07 Jun 2019 10:16:38 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45333) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKy-0001xR-3U for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKt-0003et-Tr for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:20 -0400 Received: from mx1.redhat.com ([209.132.183.28]:57662) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKk-0003OF-HU; Fri, 07 Jun 2019 09:55:06 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 20E013082A24; Fri, 7 Jun 2019 13:54:52 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id B7FFF1001B17; Fri, 7 Jun 2019 13:54:50 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:28 +0200 Message-Id: <20190607135430.22149-9-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.45]); Fri, 07 Jun 2019 13:55:00 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 08/10] monitor: Split out monitor/qmp.c X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Move QMP infrastructure from monitor/misc.c to monitor/qmp.c. This is code that can be shared for all targets, so compile it only once. The amount of function and particularly extern variables in monitor_int.h is probably a bit larger than it needs to be, but this way no non-trivial code modifications are needed. The interfaces between QMP and the monitor core can be cleaned up later. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- include/monitor/monitor.h | 1 + monitor/monitor_int.h | 30 ++- monitor/misc.c | 394 +------------------------------------ monitor/qmp.c | 404 ++++++++++++++++++++++++++++++++++++++ Makefile.objs | 1 + monitor/Makefile.objs | 1 + 6 files changed, 445 insertions(+), 386 deletions(-) create mode 100644 monitor/qmp.c diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h index 1ba354f811..7bbab05320 100644 --- a/include/monitor/monitor.h +++ b/include/monitor/monitor.h @@ -21,6 +21,7 @@ bool monitor_cur_is_qmp(void); =20 void monitor_init_globals(void); void monitor_init(Chardev *chr, int flags); +void monitor_init_qmp(Chardev *chr, int flags); void monitor_cleanup(void); =20 int monitor_suspend(Monitor *mon); diff --git a/monitor/monitor_int.h b/monitor/monitor_int.h index ab87013b6f..487618392f 100644 --- a/monitor/monitor_int.h +++ b/monitor/monitor_int.h @@ -30,10 +30,11 @@ =20 #include "qapi/qmp/qdict.h" #include "qapi/qmp/json-parser.h" -#include "qapi/qapi-commands.h" +#include "qapi/qmp/dispatch.h" =20 #include "qemu/readline.h" #include "chardev/char-fe.h" +#include "sysemu/iothread.h" =20 /* * Supported types: @@ -144,4 +145,31 @@ typedef struct { GQueue *qmp_requests; } MonitorQMP; =20 +/** + * Is @mon a QMP monitor? + */ +static inline bool monitor_is_qmp(const Monitor *mon) +{ + return (mon->flags & MONITOR_USE_CONTROL); +} + +typedef QTAILQ_HEAD(MonitorList, Monitor) MonitorList; +extern IOThread *mon_iothread; +extern QEMUBH *qmp_dispatcher_bh; +extern QmpCommandList qmp_commands, qmp_cap_negotiation_commands; +extern QemuMutex monitor_lock; +extern MonitorList mon_list; +extern int mon_refcount; + +int monitor_puts(Monitor *mon, const char *str); +void monitor_data_init(Monitor *mon, int flags, bool skip_flush, + bool use_io_thread); +int monitor_can_read(void *opaque); +void monitor_list_append(Monitor *mon); +void monitor_fdsets_cleanup(void); + +void qmp_send_response(MonitorQMP *mon, const QDict *rsp); +void monitor_data_destroy_qmp(MonitorQMP *mon); +void monitor_qmp_bh_dispatcher(void *data); + #endif diff --git a/monitor/misc.c b/monitor/misc.c index 6ae7561105..6c67f0978c 100644 --- a/monitor/misc.c +++ b/monitor/misc.c @@ -140,51 +140,29 @@ IOThread *mon_iothread; /* Bottom half to dispatch the requests received from I/O thread */ QEMUBH *qmp_dispatcher_bh; =20 -struct QMPRequest { - /* Owner of the request */ - MonitorQMP *mon; - /* - * Request object to be handled or Error to be reported - * (exactly one of them is non-null) - */ - QObject *req; - Error *err; -}; -typedef struct QMPRequest QMPRequest; - /* QMP checker flags */ #define QMP_ACCEPT_UNKNOWNS 1 =20 /* Protects mon_list, monitor_qapi_event_state, monitor_destroyed. */ -static QemuMutex monitor_lock; +QemuMutex monitor_lock; static GHashTable *monitor_qapi_event_state; -static QTAILQ_HEAD(, Monitor) mon_list; +MonitorList mon_list; static bool monitor_destroyed; =20 /* Protects mon_fdsets */ static QemuMutex mon_fdsets_lock; static QLIST_HEAD(, MonFdset) mon_fdsets; =20 -static int mon_refcount; +int mon_refcount; =20 static mon_cmd_t mon_cmds[]; static mon_cmd_t info_cmds[]; =20 -QmpCommandList qmp_commands, qmp_cap_negotiation_commands; - __thread Monitor *cur_mon; =20 static void monitor_command_cb(void *opaque, const char *cmdline, void *readline_opaque); =20 -/** - * Is @mon a QMP monitor? - */ -static inline bool monitor_is_qmp(const Monitor *mon) -{ - return (mon->flags & MONITOR_USE_CONTROL); -} - /** * Is @mon is using readline? * Note: not all HMP monitors use readline, e.g., gdbserver has a @@ -243,28 +221,6 @@ int monitor_read_password(MonitorHMP *mon, ReadLineFun= c *readline_func, } } =20 -static void qmp_request_free(QMPRequest *req) -{ - qobject_unref(req->req); - error_free(req->err); - g_free(req); -} - -/* Caller must hold mon->qmp.qmp_queue_lock */ -static void monitor_qmp_cleanup_req_queue_locked(MonitorQMP *mon) -{ - while (!g_queue_is_empty(mon->qmp_requests)) { - qmp_request_free(g_queue_pop_head(mon->qmp_requests)); - } -} - -static void monitor_qmp_cleanup_queues(MonitorQMP *mon) -{ - qemu_mutex_lock(&mon->qmp_queue_lock); - monitor_qmp_cleanup_req_queue_locked(mon); - qemu_mutex_unlock(&mon->qmp_queue_lock); -} - =20 static void monitor_flush_locked(Monitor *mon); =20 @@ -324,7 +280,7 @@ void monitor_flush(Monitor *mon) } =20 /* flush at every end of line */ -static int monitor_puts(Monitor *mon, const char *str) +int monitor_puts(Monitor *mon, const char *str) { int i; char c; @@ -374,21 +330,6 @@ int monitor_printf(Monitor *mon, const char *fmt, ...) return ret; } =20 -static void qmp_send_response(MonitorQMP *mon, const QDict *rsp) -{ - const QObject *data =3D QOBJECT(rsp); - QString *json; - - json =3D mon->common.flags & MONITOR_USE_PRETTY ? - qobject_to_json_pretty(data) : qobject_to_json(data); - assert(json !=3D NULL); - - qstring_append_chr(json, '\n'); - monitor_puts(&mon->common, qstring_get_str(json)); - - qobject_unref(json); -} - static MonitorQAPIEventConf monitor_qapi_event_conf[QAPI_EVENT__MAX] =3D { /* Limit guest-triggerable events to 1 per second */ [QAPI_EVENT_RTC_CHANGE] =3D { 1000 * SCALE_MS }, @@ -603,8 +544,8 @@ static void handle_hmp_command(MonitorHMP *mon, const c= har *cmdline); =20 static void monitor_iothread_init(void); =20 -static void monitor_data_init(Monitor *mon, int flags, bool skip_flush, - bool use_io_thread) +void monitor_data_init(Monitor *mon, int flags, bool skip_flush, + bool use_io_thread) { if (use_io_thread && !mon_iothread) { monitor_iothread_init(); @@ -625,14 +566,6 @@ static void monitor_data_init_hmp(MonitorHMP *mon, int= flags, bool skip_flush) mon->cmd_table =3D mon_cmds; } =20 -static void monitor_data_destroy_qmp(MonitorQMP *mon) -{ - json_message_parser_destroy(&mon->parser); - qemu_mutex_destroy(&mon->qmp_queue_lock); - monitor_qmp_cleanup_req_queue_locked(mon); - g_queue_free(mon->qmp_requests); -} - static void monitor_data_destroy(Monitor *mon) { g_free(mon->mon_cpu_path); @@ -1069,18 +1002,6 @@ static void monitor_init_qmp_commands(void) qmp_marshal_qmp_capabilities, QCO_ALLOW_PRECONFIG= ); } =20 -static bool qmp_oob_enabled(MonitorQMP *mon) -{ - return mon->capab[QMP_CAPABILITY_OOB]; -} - -static void monitor_qmp_caps_reset(MonitorQMP *mon) -{ - memset(mon->capab_offered, 0, sizeof(mon->capab_offered)); - memset(mon->capab, 0, sizeof(mon->capab)); - mon->capab_offered[QMP_CAPABILITY_OOB] =3D mon->common.use_io_thread; -} - /* * Accept QMP capabilities in @list for @mon. * On success, set mon->qmp.capab[], and return true. @@ -2251,7 +2172,7 @@ static void monitor_fdset_cleanup(MonFdset *mon_fdset) } } =20 -static void monitor_fdsets_cleanup(void) +void monitor_fdsets_cleanup(void) { MonFdset *mon_fdset; MonFdset *mon_fdset_next; @@ -4029,209 +3950,13 @@ cleanup: free_cmdline_args(args, nb_args); } =20 -static int monitor_can_read(void *opaque) +int monitor_can_read(void *opaque) { Monitor *mon =3D opaque; =20 return !atomic_mb_read(&mon->suspend_cnt); } =20 -/* - * Emit QMP response @rsp with ID @id to @mon. - * Null @rsp can only happen for commands with QCO_NO_SUCCESS_RESP. - * Nothing is emitted then. - */ -static void monitor_qmp_respond(MonitorQMP *mon, QDict *rsp) -{ - if (rsp) { - qmp_send_response(mon, rsp); - } -} - -static void monitor_qmp_dispatch(MonitorQMP *mon, QObject *req) -{ - Monitor *old_mon; - QDict *rsp; - QDict *error; - - old_mon =3D cur_mon; - cur_mon =3D &mon->common; - - rsp =3D qmp_dispatch(mon->commands, req, qmp_oob_enabled(mon)); - - cur_mon =3D old_mon; - - if (mon->commands =3D=3D &qmp_cap_negotiation_commands) { - error =3D qdict_get_qdict(rsp, "error"); - if (error - && !g_strcmp0(qdict_get_try_str(error, "class"), - QapiErrorClass_str(ERROR_CLASS_COMMAND_NOT_FOUND))) { - /* Provide a more useful error message */ - qdict_del(error, "desc"); - qdict_put_str(error, "desc", "Expecting capabilities negotiati= on" - " with 'qmp_capabilities'"); - } - } - - monitor_qmp_respond(mon, rsp); - qobject_unref(rsp); -} - -/* - * Pop a QMP request from a monitor request queue. - * Return the request, or NULL all request queues are empty. - * We are using round-robin fashion to pop the request, to avoid - * processing commands only on a very busy monitor. To achieve that, - * when we process one request on a specific monitor, we put that - * monitor to the end of mon_list queue. - * - * Note: if the function returned with non-NULL, then the caller will - * be with qmp_mon->qmp_queue_lock held, and the caller is responsible - * to release it. - */ -static QMPRequest *monitor_qmp_requests_pop_any_with_lock(void) -{ - QMPRequest *req_obj =3D NULL; - Monitor *mon; - MonitorQMP *qmp_mon; - - qemu_mutex_lock(&monitor_lock); - - QTAILQ_FOREACH(mon, &mon_list, entry) { - if (!monitor_is_qmp(mon)) { - continue; - } - - qmp_mon =3D container_of(mon, MonitorQMP, common); - qemu_mutex_lock(&qmp_mon->qmp_queue_lock); - req_obj =3D g_queue_pop_head(qmp_mon->qmp_requests); - if (req_obj) { - /* With the lock of corresponding queue held */ - break; - } - qemu_mutex_unlock(&qmp_mon->qmp_queue_lock); - } - - if (req_obj) { - /* - * We found one request on the monitor. Degrade this monitor's - * priority to lowest by re-inserting it to end of queue. - */ - QTAILQ_REMOVE(&mon_list, mon, entry); - QTAILQ_INSERT_TAIL(&mon_list, mon, entry); - } - - qemu_mutex_unlock(&monitor_lock); - - return req_obj; -} - -static void monitor_qmp_bh_dispatcher(void *data) -{ - QMPRequest *req_obj =3D monitor_qmp_requests_pop_any_with_lock(); - QDict *rsp; - bool need_resume; - MonitorQMP *mon; - - if (!req_obj) { - return; - } - - mon =3D req_obj->mon; - /* qmp_oob_enabled() might change after "qmp_capabilities" */ - need_resume =3D !qmp_oob_enabled(mon) || - mon->qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1; - qemu_mutex_unlock(&mon->qmp_queue_lock); - if (req_obj->req) { - QDict *qdict =3D qobject_to(QDict, req_obj->req); - QObject *id =3D qdict ? qdict_get(qdict, "id") : NULL; - trace_monitor_qmp_cmd_in_band(qobject_get_try_str(id) ?: ""); - monitor_qmp_dispatch(mon, req_obj->req); - } else { - assert(req_obj->err); - rsp =3D qmp_error_response(req_obj->err); - req_obj->err =3D NULL; - monitor_qmp_respond(mon, rsp); - qobject_unref(rsp); - } - - if (need_resume) { - /* Pairs with the monitor_suspend() in handle_qmp_command() */ - monitor_resume(&mon->common); - } - qmp_request_free(req_obj); - - /* Reschedule instead of looping so the main loop stays responsive */ - qemu_bh_schedule(qmp_dispatcher_bh); -} - -static void handle_qmp_command(void *opaque, QObject *req, Error *err) -{ - MonitorQMP *mon =3D opaque; - QObject *id =3D NULL; - QDict *qdict; - QMPRequest *req_obj; - - assert(!req !=3D !err); - - qdict =3D qobject_to(QDict, req); - if (qdict) { - id =3D qdict_get(qdict, "id"); - } /* else will fail qmp_dispatch() */ - - if (req && trace_event_get_state_backends(TRACE_HANDLE_QMP_COMMAND)) { - QString *req_json =3D qobject_to_json(req); - trace_handle_qmp_command(mon, qstring_get_str(req_json)); - qobject_unref(req_json); - } - - if (qdict && qmp_is_oob(qdict)) { - /* OOB commands are executed immediately */ - trace_monitor_qmp_cmd_out_of_band(qobject_get_try_str(id) ?: ""); - monitor_qmp_dispatch(mon, req); - qobject_unref(req); - return; - } - - req_obj =3D g_new0(QMPRequest, 1); - req_obj->mon =3D mon; - req_obj->req =3D req; - req_obj->err =3D err; - - /* Protect qmp_requests and fetching its length. */ - qemu_mutex_lock(&mon->qmp_queue_lock); - - /* - * Suspend the monitor when we can't queue more requests after - * this one. Dequeuing in monitor_qmp_bh_dispatcher() will resume - * it. Note that when OOB is disabled, we queue at most one - * command, for backward compatibility. - */ - if (!qmp_oob_enabled(mon) || - mon->qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1) { - monitor_suspend(&mon->common); - } - - /* - * Put the request to the end of queue so that requests will be - * handled in time order. Ownership for req_obj, req, - * etc. will be delivered to the handler side. - */ - assert(mon->qmp_requests->length < QMP_REQ_QUEUE_LEN_MAX); - g_queue_push_tail(mon->qmp_requests, req_obj); - qemu_mutex_unlock(&mon->qmp_queue_lock); - - /* Kick the dispatcher routine */ - qemu_bh_schedule(qmp_dispatcher_bh); -} - -static void monitor_qmp_read(void *opaque, const uint8_t *buf, int size) -{ - MonitorQMP *mon =3D opaque; - - json_message_parser_feed(&mon->parser, (const char *) buf, size); -} - static void monitor_read(void *opaque, const uint8_t *buf, int size) { MonitorHMP *mon; @@ -4318,56 +4043,6 @@ void monitor_resume(Monitor *mon) trace_monitor_suspend(mon, -1); } =20 -static QDict *qmp_greeting(MonitorQMP *mon) -{ - QList *cap_list =3D qlist_new(); - QObject *ver =3D NULL; - QMPCapability cap; - - qmp_marshal_query_version(NULL, &ver, NULL); - - for (cap =3D 0; cap < QMP_CAPABILITY__MAX; cap++) { - if (mon->capab_offered[cap]) { - qlist_append_str(cap_list, QMPCapability_str(cap)); - } - } - - return qdict_from_jsonf_nofail( - "{'QMP': {'version': %p, 'capabilities': %p}}", - ver, cap_list); -} - -static void monitor_qmp_event(void *opaque, int event) -{ - QDict *data; - MonitorQMP *mon =3D opaque; - - switch (event) { - case CHR_EVENT_OPENED: - mon->commands =3D &qmp_cap_negotiation_commands; - monitor_qmp_caps_reset(mon); - data =3D qmp_greeting(mon); - qmp_send_response(mon, data); - qobject_unref(data); - mon_refcount++; - break; - case CHR_EVENT_CLOSED: - /* - * Note: this is only useful when the output of the chardev - * backend is still open. For example, when the backend is - * stdio, it's possible that stdout is still open when stdin - * is closed. - */ - monitor_qmp_cleanup_queues(mon); - json_message_parser_destroy(&mon->parser); - json_message_parser_init(&mon->parser, handle_qmp_command, - mon, NULL); - mon_refcount--; - monitor_fdsets_cleanup(); - break; - } -} - static void monitor_event(void *opaque, int event) { Monitor *mon =3D opaque; @@ -4503,7 +4178,7 @@ int error_vprintf_unless_qmp(const char *fmt, va_list= ap) return -1; } =20 -static void monitor_list_append(Monitor *mon) +void monitor_list_append(Monitor *mon) { qemu_mutex_lock(&monitor_lock); /* @@ -4523,57 +4198,6 @@ static void monitor_list_append(Monitor *mon) } } =20 -static void monitor_qmp_setup_handlers_bh(void *opaque) -{ - MonitorQMP *mon =3D opaque; - GMainContext *context; - - assert(mon->common.use_io_thread); - context =3D iothread_get_g_main_context(mon_iothread); - assert(context); - qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, - monitor_qmp_read, monitor_qmp_event, - NULL, &mon->common, context, true); - monitor_list_append(&mon->common); -} - -static void monitor_init_qmp(Chardev *chr, int flags) -{ - MonitorQMP *mon =3D g_malloc0(sizeof(*mon)); - - /* Note: we run QMP monitor in I/O thread when @chr supports that */ - monitor_data_init(&mon->common, flags, false, - qemu_chr_has_feature(chr, QEMU_CHAR_FEATURE_GCONTEXT= )); - - qemu_mutex_init(&mon->qmp_queue_lock); - mon->qmp_requests =3D g_queue_new(); - - qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); - qemu_chr_fe_set_echo(&mon->common.chr, true); - - json_message_parser_init(&mon->parser, handle_qmp_command, mon, NULL); - if (mon->common.use_io_thread) { - /* - * Make sure the old iowatch is gone. It's possible when - * e.g. the chardev is in client mode, with wait=3Don. - */ - remove_fd_in_watch(chr); - /* - * We can't call qemu_chr_fe_set_handlers() directly here - * since chardev might be running in the monitor I/O - * thread. Schedule a bottom half. - */ - aio_bh_schedule_oneshot(iothread_get_aio_context(mon_iothread), - monitor_qmp_setup_handlers_bh, mon); - /* The bottom half will add @mon to @mon_list */ - } else { - qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, - monitor_qmp_read, monitor_qmp_event, - NULL, &mon->common, NULL, true); - monitor_list_append(&mon->common); - } -} - static void monitor_init_hmp(Chardev *chr, int flags) { MonitorHMP *mon =3D g_malloc0(sizeof(*mon)); diff --git a/monitor/qmp.c b/monitor/qmp.c new file mode 100644 index 0000000000..74e72cf5b8 --- /dev/null +++ b/monitor/qmp.c @@ -0,0 +1,404 @@ +/* + * QEMU monitor + * + * Copyright (c) 2003-2004 Fabrice Bellard + * + * Permission is hereby granted, free of charge, to any person obtaining a= copy + * of this software and associated documentation files (the "Software"), t= o deal + * in the Software without restriction, including without limitation the r= ights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or se= ll + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included= in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS= OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OT= HER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING= FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS = IN + * THE SOFTWARE. + */ + +#include "qemu/osdep.h" +#include "monitor_int.h" + +#include "chardev/char-io.h" + +#include "qapi/error.h" +#include "qapi/qmp/qjson.h" +#include "qapi/qmp/qstring.h" +#include "qapi/qmp/qlist.h" +#include "qapi/qapi-commands-misc.h" + +#include "trace-root.h" + +struct QMPRequest { + /* Owner of the request */ + MonitorQMP *mon; + /* + * Request object to be handled or Error to be reported + * (exactly one of them is non-null) + */ + QObject *req; + Error *err; +}; +typedef struct QMPRequest QMPRequest; + +QmpCommandList qmp_commands, qmp_cap_negotiation_commands; + +static bool qmp_oob_enabled(MonitorQMP *mon) +{ + return mon->capab[QMP_CAPABILITY_OOB]; +} + +static void monitor_qmp_caps_reset(MonitorQMP *mon) +{ + memset(mon->capab_offered, 0, sizeof(mon->capab_offered)); + memset(mon->capab, 0, sizeof(mon->capab)); + mon->capab_offered[QMP_CAPABILITY_OOB] =3D mon->common.use_io_thread; +} + +static void qmp_request_free(QMPRequest *req) +{ + qobject_unref(req->req); + error_free(req->err); + g_free(req); +} + +/* Caller must hold mon->qmp.qmp_queue_lock */ +static void monitor_qmp_cleanup_req_queue_locked(MonitorQMP *mon) +{ + while (!g_queue_is_empty(mon->qmp_requests)) { + qmp_request_free(g_queue_pop_head(mon->qmp_requests)); + } +} + +static void monitor_qmp_cleanup_queues(MonitorQMP *mon) +{ + qemu_mutex_lock(&mon->qmp_queue_lock); + monitor_qmp_cleanup_req_queue_locked(mon); + qemu_mutex_unlock(&mon->qmp_queue_lock); +} + +void qmp_send_response(MonitorQMP *mon, const QDict *rsp) +{ + const QObject *data =3D QOBJECT(rsp); + QString *json; + + json =3D mon->common.flags & MONITOR_USE_PRETTY ? + qobject_to_json_pretty(data) : qobject_to_json(data); + assert(json !=3D NULL); + + qstring_append_chr(json, '\n'); + monitor_puts(&mon->common, qstring_get_str(json)); + + qobject_unref(json); +} + +/* + * Emit QMP response @rsp with ID @id to @mon. + * Null @rsp can only happen for commands with QCO_NO_SUCCESS_RESP. + * Nothing is emitted then. + */ +static void monitor_qmp_respond(MonitorQMP *mon, QDict *rsp) +{ + if (rsp) { + qmp_send_response(mon, rsp); + } +} + +static void monitor_qmp_dispatch(MonitorQMP *mon, QObject *req) +{ + Monitor *old_mon; + QDict *rsp; + QDict *error; + + old_mon =3D cur_mon; + cur_mon =3D &mon->common; + + rsp =3D qmp_dispatch(mon->commands, req, qmp_oob_enabled(mon)); + + cur_mon =3D old_mon; + + if (mon->commands =3D=3D &qmp_cap_negotiation_commands) { + error =3D qdict_get_qdict(rsp, "error"); + if (error + && !g_strcmp0(qdict_get_try_str(error, "class"), + QapiErrorClass_str(ERROR_CLASS_COMMAND_NOT_FOUND))) { + /* Provide a more useful error message */ + qdict_del(error, "desc"); + qdict_put_str(error, "desc", "Expecting capabilities negotiati= on" + " with 'qmp_capabilities'"); + } + } + + monitor_qmp_respond(mon, rsp); + qobject_unref(rsp); +} + +/* + * Pop a QMP request from a monitor request queue. + * Return the request, or NULL all request queues are empty. + * We are using round-robin fashion to pop the request, to avoid + * processing commands only on a very busy monitor. To achieve that, + * when we process one request on a specific monitor, we put that + * monitor to the end of mon_list queue. + * + * Note: if the function returned with non-NULL, then the caller will + * be with qmp_mon->qmp_queue_lock held, and the caller is responsible + * to release it. + */ +static QMPRequest *monitor_qmp_requests_pop_any_with_lock(void) +{ + QMPRequest *req_obj =3D NULL; + Monitor *mon; + MonitorQMP *qmp_mon; + + qemu_mutex_lock(&monitor_lock); + + QTAILQ_FOREACH(mon, &mon_list, entry) { + if (!monitor_is_qmp(mon)) { + continue; + } + + qmp_mon =3D container_of(mon, MonitorQMP, common); + qemu_mutex_lock(&qmp_mon->qmp_queue_lock); + req_obj =3D g_queue_pop_head(qmp_mon->qmp_requests); + if (req_obj) { + /* With the lock of corresponding queue held */ + break; + } + qemu_mutex_unlock(&qmp_mon->qmp_queue_lock); + } + + if (req_obj) { + /* + * We found one request on the monitor. Degrade this monitor's + * priority to lowest by re-inserting it to end of queue. + */ + QTAILQ_REMOVE(&mon_list, mon, entry); + QTAILQ_INSERT_TAIL(&mon_list, mon, entry); + } + + qemu_mutex_unlock(&monitor_lock); + + return req_obj; +} + +void monitor_qmp_bh_dispatcher(void *data) +{ + QMPRequest *req_obj =3D monitor_qmp_requests_pop_any_with_lock(); + QDict *rsp; + bool need_resume; + MonitorQMP *mon; + + if (!req_obj) { + return; + } + + mon =3D req_obj->mon; + /* qmp_oob_enabled() might change after "qmp_capabilities" */ + need_resume =3D !qmp_oob_enabled(mon) || + mon->qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1; + qemu_mutex_unlock(&mon->qmp_queue_lock); + if (req_obj->req) { + QDict *qdict =3D qobject_to(QDict, req_obj->req); + QObject *id =3D qdict ? qdict_get(qdict, "id") : NULL; + trace_monitor_qmp_cmd_in_band(qobject_get_try_str(id) ?: ""); + monitor_qmp_dispatch(mon, req_obj->req); + } else { + assert(req_obj->err); + rsp =3D qmp_error_response(req_obj->err); + req_obj->err =3D NULL; + monitor_qmp_respond(mon, rsp); + qobject_unref(rsp); + } + + if (need_resume) { + /* Pairs with the monitor_suspend() in handle_qmp_command() */ + monitor_resume(&mon->common); + } + qmp_request_free(req_obj); + + /* Reschedule instead of looping so the main loop stays responsive */ + qemu_bh_schedule(qmp_dispatcher_bh); +} + +static void handle_qmp_command(void *opaque, QObject *req, Error *err) +{ + MonitorQMP *mon =3D opaque; + QObject *id =3D NULL; + QDict *qdict; + QMPRequest *req_obj; + + assert(!req !=3D !err); + + qdict =3D qobject_to(QDict, req); + if (qdict) { + id =3D qdict_get(qdict, "id"); + } /* else will fail qmp_dispatch() */ + + if (req && trace_event_get_state_backends(TRACE_HANDLE_QMP_COMMAND)) { + QString *req_json =3D qobject_to_json(req); + trace_handle_qmp_command(mon, qstring_get_str(req_json)); + qobject_unref(req_json); + } + + if (qdict && qmp_is_oob(qdict)) { + /* OOB commands are executed immediately */ + trace_monitor_qmp_cmd_out_of_band(qobject_get_try_str(id) ?: ""); + monitor_qmp_dispatch(mon, req); + qobject_unref(req); + return; + } + + req_obj =3D g_new0(QMPRequest, 1); + req_obj->mon =3D mon; + req_obj->req =3D req; + req_obj->err =3D err; + + /* Protect qmp_requests and fetching its length. */ + qemu_mutex_lock(&mon->qmp_queue_lock); + + /* + * Suspend the monitor when we can't queue more requests after + * this one. Dequeuing in monitor_qmp_bh_dispatcher() will resume + * it. Note that when OOB is disabled, we queue at most one + * command, for backward compatibility. + */ + if (!qmp_oob_enabled(mon) || + mon->qmp_requests->length =3D=3D QMP_REQ_QUEUE_LEN_MAX - 1) { + monitor_suspend(&mon->common); + } + + /* + * Put the request to the end of queue so that requests will be + * handled in time order. Ownership for req_obj, req, + * etc. will be delivered to the handler side. + */ + assert(mon->qmp_requests->length < QMP_REQ_QUEUE_LEN_MAX); + g_queue_push_tail(mon->qmp_requests, req_obj); + qemu_mutex_unlock(&mon->qmp_queue_lock); + + /* Kick the dispatcher routine */ + qemu_bh_schedule(qmp_dispatcher_bh); +} + +static void monitor_qmp_read(void *opaque, const uint8_t *buf, int size) +{ + MonitorQMP *mon =3D opaque; + + json_message_parser_feed(&mon->parser, (const char *) buf, size); +} + +static QDict *qmp_greeting(MonitorQMP *mon) +{ + QList *cap_list =3D qlist_new(); + QObject *ver =3D NULL; + QMPCapability cap; + + qmp_marshal_query_version(NULL, &ver, NULL); + + for (cap =3D 0; cap < QMP_CAPABILITY__MAX; cap++) { + if (mon->capab_offered[cap]) { + qlist_append_str(cap_list, QMPCapability_str(cap)); + } + } + + return qdict_from_jsonf_nofail( + "{'QMP': {'version': %p, 'capabilities': %p}}", + ver, cap_list); +} + +static void monitor_qmp_event(void *opaque, int event) +{ + QDict *data; + MonitorQMP *mon =3D opaque; + + switch (event) { + case CHR_EVENT_OPENED: + mon->commands =3D &qmp_cap_negotiation_commands; + monitor_qmp_caps_reset(mon); + data =3D qmp_greeting(mon); + qmp_send_response(mon, data); + qobject_unref(data); + mon_refcount++; + break; + case CHR_EVENT_CLOSED: + /* + * Note: this is only useful when the output of the chardev + * backend is still open. For example, when the backend is + * stdio, it's possible that stdout is still open when stdin + * is closed. + */ + monitor_qmp_cleanup_queues(mon); + json_message_parser_destroy(&mon->parser); + json_message_parser_init(&mon->parser, handle_qmp_command, + mon, NULL); + mon_refcount--; + monitor_fdsets_cleanup(); + break; + } +} + +void monitor_data_destroy_qmp(MonitorQMP *mon) +{ + json_message_parser_destroy(&mon->parser); + qemu_mutex_destroy(&mon->qmp_queue_lock); + monitor_qmp_cleanup_req_queue_locked(mon); + g_queue_free(mon->qmp_requests); +} + +static void monitor_qmp_setup_handlers_bh(void *opaque) +{ + MonitorQMP *mon =3D opaque; + GMainContext *context; + + assert(mon->common.use_io_thread); + context =3D iothread_get_g_main_context(mon_iothread); + assert(context); + qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, + monitor_qmp_read, monitor_qmp_event, + NULL, &mon->common, context, true); + monitor_list_append(&mon->common); +} + +void monitor_init_qmp(Chardev *chr, int flags) +{ + MonitorQMP *mon =3D g_malloc0(sizeof(*mon)); + + /* Note: we run QMP monitor in I/O thread when @chr supports that */ + monitor_data_init(&mon->common, flags, false, + qemu_chr_has_feature(chr, QEMU_CHAR_FEATURE_GCONTEXT= )); + + qemu_mutex_init(&mon->qmp_queue_lock); + mon->qmp_requests =3D g_queue_new(); + + qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); + qemu_chr_fe_set_echo(&mon->common.chr, true); + + json_message_parser_init(&mon->parser, handle_qmp_command, mon, NULL); + if (mon->common.use_io_thread) { + /* + * Make sure the old iowatch is gone. It's possible when + * e.g. the chardev is in client mode, with wait=3Don. + */ + remove_fd_in_watch(chr); + /* + * We can't call qemu_chr_fe_set_handlers() directly here + * since chardev might be running in the monitor I/O + * thread. Schedule a bottom half. + */ + aio_bh_schedule_oneshot(iothread_get_aio_context(mon_iothread), + monitor_qmp_setup_handlers_bh, mon); + /* The bottom half will add @mon to @mon_list */ + } else { + qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, + monitor_qmp_read, monitor_qmp_event, + NULL, &mon->common, NULL, true); + monitor_list_append(&mon->common); + } +} diff --git a/Makefile.objs b/Makefile.objs index c8337fa34b..76c5b525f9 100644 --- a/Makefile.objs +++ b/Makefile.objs @@ -46,6 +46,7 @@ ifeq ($(CONFIG_SOFTMMU),y) common-obj-y =3D blockdev.o blockdev-nbd.o block/ common-obj-y +=3D bootdevice.o iothread.o common-obj-y +=3D job-qmp.o +common-obj-y +=3D monitor/ common-obj-y +=3D net/ common-obj-y +=3D qdev-monitor.o device-hotplug.o common-obj-$(CONFIG_WIN32) +=3D os-win32.o diff --git a/monitor/Makefile.objs b/monitor/Makefile.objs index e783b0616b..d04d58b583 100644 --- a/monitor/Makefile.objs +++ b/monitor/Makefile.objs @@ -1 +1,2 @@ obj-y +=3D misc.o +common-obj-y +=3D qmp.o --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559916489; cv=none; d=zoho.com; s=zohoarc; b=ldlxhK2s7oo02/7Z6woO//Ujj+K2hOoaVAIR2GgxJ/OpaIB1FtG9KHkzx+9NVkzOjjhV4eyw+zTVIpCc5EDGuSvJsa30GwN+gIVPUOFpko8LY0Qsw7yGvYThDSjvz2arQxhL8Autk4zgVPiZx52x10IDYMXWbbQL4gcsA6lc5eo= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559916489; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=0kofUs0tR2ySmp6hKic6d5RCaJ7SUpyjg3/vLJ4xHns=; b=l3vlrKH24gnVhOMHblo7jxTiphIuQ2FNdMhoVYLOBAFeVr2FV0g4yNbolYecujg2zbpHd8t7BYPuJLmt/0SG+Whq+bsFxuPZtX8EWE7NI3nM00ptMkFZC0C8T6XBCoBrzrE/iWYhw4QMIoiYI6t+jiflmCEuubFyBnmvZax2G6w= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.47]) by mx.zohomail.com with SMTPS id 15599164896931002.9970285230974; Fri, 7 Jun 2019 07:08:09 -0700 (PDT) Received: from localhost ([::1]:51492 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFXG-0003Mv-7K for importer@patchew.org; Fri, 07 Jun 2019 10:08:04 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45395) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFL9-0002DN-V3 for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:37 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFL1-0003vj-23 for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:30 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42136) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKk-0003OH-Ir; Fri, 07 Jun 2019 09:55:07 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id E3AB530832CE; Fri, 7 Jun 2019 13:54:55 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id 603711001B08; Fri, 7 Jun 2019 13:54:52 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:29 +0200 Message-Id: <20190607135430.22149-10-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.44]); Fri, 07 Jun 2019 13:55:00 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 09/10] monitor: Split out monitor/hmp.c X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Move HMP infrastructure from monitor/misc.c to monitor/hmp.c. This is code that can be shared for all targets, so compile it only once. The amount of function and particularly extern variables in monitor_int.h is probably a bit larger than it needs to be, but this way no non-trivial code modifications are needed. The interfaces between HMP and the monitor core can be cleaned up later. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- include/monitor/monitor.h | 1 + monitor/monitor_int.h | 31 + monitor/hmp.c | 1351 +++++++++++++++++++++++++++++++++++++ monitor/misc.c | 1338 +----------------------------------- monitor/Makefile.objs | 2 +- 5 files changed, 1390 insertions(+), 1333 deletions(-) create mode 100644 monitor/hmp.c diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h index 7bbab05320..8547529e49 100644 --- a/include/monitor/monitor.h +++ b/include/monitor/monitor.h @@ -22,6 +22,7 @@ bool monitor_cur_is_qmp(void); void monitor_init_globals(void); void monitor_init(Chardev *chr, int flags); void monitor_init_qmp(Chardev *chr, int flags); +void monitor_init_hmp(Chardev *chr, int flags); void monitor_cleanup(void); =20 int monitor_suspend(Monitor *mon); diff --git a/monitor/monitor_int.h b/monitor/monitor_int.h index 487618392f..8c5d95f942 100644 --- a/monitor/monitor_int.h +++ b/monitor/monitor_int.h @@ -27,6 +27,7 @@ =20 #include "qemu-common.h" #include "monitor/monitor.h" +#include "qemu/cutils.h" =20 #include "qapi/qmp/qdict.h" #include "qapi/qmp/json-parser.h" @@ -153,6 +154,29 @@ static inline bool monitor_is_qmp(const Monitor *mon) return (mon->flags & MONITOR_USE_CONTROL); } =20 +/** + * Is @name in the '|' separated list of names @list? + */ +static inline int compare_cmd(const char *name, const char *list) +{ + const char *p, *pstart; + int len; + len =3D strlen(name); + p =3D list; + for(;;) { + pstart =3D p; + p =3D qemu_strchrnul(p, '|'); + if ((p - pstart) =3D=3D len && !memcmp(pstart, name, len)) { + return 1; + } + if (*p =3D=3D '\0') { + break; + } + p++; + } + return 0; +} + typedef QTAILQ_HEAD(MonitorList, Monitor) MonitorList; extern IOThread *mon_iothread; extern QEMUBH *qmp_dispatcher_bh; @@ -161,6 +185,8 @@ extern QemuMutex monitor_lock; extern MonitorList mon_list; extern int mon_refcount; =20 +extern mon_cmd_t mon_cmds[]; + int monitor_puts(Monitor *mon, const char *str); void monitor_data_init(Monitor *mon, int flags, bool skip_flush, bool use_io_thread); @@ -172,4 +198,9 @@ void qmp_send_response(MonitorQMP *mon, const QDict *rs= p); void monitor_data_destroy_qmp(MonitorQMP *mon); void monitor_qmp_bh_dispatcher(void *data); =20 +void monitor_data_init_hmp(MonitorHMP *mon, int flags, bool skip_flush); +int get_monitor_def(int64_t *pval, const char *name); +void help_cmd(Monitor *mon, const char *name); +void handle_hmp_command(MonitorHMP *mon, const char *cmdline); + #endif diff --git a/monitor/hmp.c b/monitor/hmp.c new file mode 100644 index 0000000000..2bc464a7fc --- /dev/null +++ b/monitor/hmp.c @@ -0,0 +1,1351 @@ +/* + * QEMU monitor + * + * Copyright (c) 2003-2004 Fabrice Bellard + * + * Permission is hereby granted, free of charge, to any person obtaining a= copy + * of this software and associated documentation files (the "Software"), t= o deal + * in the Software without restriction, including without limitation the r= ights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or se= ll + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included= in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS= OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OT= HER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING= FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS = IN + * THE SOFTWARE. + */ + +#include "qemu/osdep.h" +#include "monitor_int.h" + +#include "qapi/error.h" +#include "qapi/qmp/qnum.h" + +#include "qemu/config-file.h" +#include "qemu/log.h" +#include "qemu/option.h" +#include "qemu/units.h" +#include "sysemu/block-backend.h" +#include "sysemu/sysemu.h" + +#include "trace-root.h" + +static int get_str(char *buf, int buf_size, const char **pp) +{ + const char *p; + char *q; + int c; + + q =3D buf; + p =3D *pp; + while (qemu_isspace(*p)) { + p++; + } + if (*p =3D=3D '\0') { + fail: + *q =3D '\0'; + *pp =3D p; + return -1; + } + if (*p =3D=3D '\"') { + p++; + while (*p !=3D '\0' && *p !=3D '\"') { + if (*p =3D=3D '\\') { + p++; + c =3D *p++; + switch (c) { + case 'n': + c =3D '\n'; + break; + case 'r': + c =3D '\r'; + break; + case '\\': + case '\'': + case '\"': + break; + default: + printf("unsupported escape code: '\\%c'\n", c); + goto fail; + } + if ((q - buf) < buf_size - 1) { + *q++ =3D c; + } + } else { + if ((q - buf) < buf_size - 1) { + *q++ =3D *p; + } + p++; + } + } + if (*p !=3D '\"') { + printf("unterminated string\n"); + goto fail; + } + p++; + } else { + while (*p !=3D '\0' && !qemu_isspace(*p)) { + if ((q - buf) < buf_size - 1) { + *q++ =3D *p; + } + p++; + } + } + *q =3D '\0'; + *pp =3D p; + return 0; +} + +#define MAX_ARGS 16 + +static void free_cmdline_args(char **args, int nb_args) +{ + int i; + + assert(nb_args <=3D MAX_ARGS); + + for (i =3D 0; i < nb_args; i++) { + g_free(args[i]); + } + +} + +/* + * Parse the command line to get valid args. + * @cmdline: command line to be parsed. + * @pnb_args: location to store the number of args, must NOT be NULL. + * @args: location to store the args, which should be freed by caller, must + * NOT be NULL. + * + * Returns 0 on success, negative on failure. + * + * NOTE: this parser is an approximate form of the real command parser. Nu= mber + * of args have a limit of MAX_ARGS. If cmdline contains more, it wi= ll + * return with failure. + */ +static int parse_cmdline(const char *cmdline, + int *pnb_args, char **args) +{ + const char *p; + int nb_args, ret; + char buf[1024]; + + p =3D cmdline; + nb_args =3D 0; + for (;;) { + while (qemu_isspace(*p)) { + p++; + } + if (*p =3D=3D '\0') { + break; + } + if (nb_args >=3D MAX_ARGS) { + goto fail; + } + ret =3D get_str(buf, sizeof(buf), &p); + if (ret < 0) { + goto fail; + } + args[nb_args] =3D g_strdup(buf); + nb_args++; + } + *pnb_args =3D nb_args; + return 0; + + fail: + free_cmdline_args(args, nb_args); + return -1; +} + +/* + * Can command @cmd be executed in preconfig state? + */ +static bool cmd_can_preconfig(const mon_cmd_t *cmd) +{ + if (!cmd->flags) { + return false; + } + + return strchr(cmd->flags, 'p'); +} + +static void help_cmd_dump_one(Monitor *mon, + const mon_cmd_t *cmd, + char **prefix_args, + int prefix_args_nb) +{ + int i; + + if (runstate_check(RUN_STATE_PRECONFIG) && !cmd_can_preconfig(cmd)) { + return; + } + + for (i =3D 0; i < prefix_args_nb; i++) { + monitor_printf(mon, "%s ", prefix_args[i]); + } + monitor_printf(mon, "%s %s -- %s\n", cmd->name, cmd->params, cmd->help= ); +} + +/* @args[@arg_index] is the valid command need to find in @cmds */ +static void help_cmd_dump(Monitor *mon, const mon_cmd_t *cmds, + char **args, int nb_args, int arg_index) +{ + const mon_cmd_t *cmd; + size_t i; + + /* No valid arg need to compare with, dump all in *cmds */ + if (arg_index >=3D nb_args) { + for (cmd =3D cmds; cmd->name !=3D NULL; cmd++) { + help_cmd_dump_one(mon, cmd, args, arg_index); + } + return; + } + + /* Find one entry to dump */ + for (cmd =3D cmds; cmd->name !=3D NULL; cmd++) { + if (compare_cmd(args[arg_index], cmd->name) && + ((!runstate_check(RUN_STATE_PRECONFIG) || + cmd_can_preconfig(cmd)))) { + if (cmd->sub_table) { + /* continue with next arg */ + help_cmd_dump(mon, cmd->sub_table, + args, nb_args, arg_index + 1); + } else { + help_cmd_dump_one(mon, cmd, args, arg_index); + } + return; + } + } + + /* Command not found */ + monitor_printf(mon, "unknown command: '"); + for (i =3D 0; i <=3D arg_index; i++) { + monitor_printf(mon, "%s%s", args[i], i =3D=3D arg_index ? "'\n" : = " "); + } +} + +void help_cmd(Monitor *mon, const char *name) +{ + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); + char *args[MAX_ARGS]; + int nb_args =3D 0; + + /* 1. parse user input */ + if (name) { + /* special case for log, directly dump and return */ + if (!strcmp(name, "log")) { + const QEMULogItem *item; + monitor_printf(mon, "Log items (comma separated):\n"); + monitor_printf(mon, "%-10s %s\n", "none", "remove all logs"); + for (item =3D qemu_log_items; item->mask !=3D 0; item++) { + monitor_printf(mon, "%-10s %s\n", item->name, item->help); + } + return; + } + + if (parse_cmdline(name, &nb_args, args) < 0) { + return; + } + } + + /* 2. dump the contents according to parsed args */ + help_cmd_dump(mon, hmp_mon->cmd_table, args, nb_args, 0); + + free_cmdline_args(args, nb_args); +} + +/*******************************************************************/ + +static const char *pch; +static sigjmp_buf expr_env; + +static void GCC_FMT_ATTR(2, 3) QEMU_NORETURN +expr_error(Monitor *mon, const char *fmt, ...) +{ + va_list ap; + va_start(ap, fmt); + monitor_vprintf(mon, fmt, ap); + monitor_printf(mon, "\n"); + va_end(ap); + siglongjmp(expr_env, 1); +} + +static void next(void) +{ + if (*pch !=3D '\0') { + pch++; + while (qemu_isspace(*pch)) + pch++; + } +} + +static int64_t expr_sum(Monitor *mon); + +static int64_t expr_unary(Monitor *mon) +{ + int64_t n; + char *p; + int ret; + + switch(*pch) { + case '+': + next(); + n =3D expr_unary(mon); + break; + case '-': + next(); + n =3D -expr_unary(mon); + break; + case '~': + next(); + n =3D ~expr_unary(mon); + break; + case '(': + next(); + n =3D expr_sum(mon); + if (*pch !=3D ')') { + expr_error(mon, "')' expected"); + } + next(); + break; + case '\'': + pch++; + if (*pch =3D=3D '\0') + expr_error(mon, "character constant expected"); + n =3D *pch; + pch++; + if (*pch !=3D '\'') + expr_error(mon, "missing terminating \' character"); + next(); + break; + case '$': + { + char buf[128], *q; + int64_t reg =3D 0; + + pch++; + q =3D buf; + while ((*pch >=3D 'a' && *pch <=3D 'z') || + (*pch >=3D 'A' && *pch <=3D 'Z') || + (*pch >=3D '0' && *pch <=3D '9') || + *pch =3D=3D '_' || *pch =3D=3D '.') { + if ((q - buf) < sizeof(buf) - 1) + *q++ =3D *pch; + pch++; + } + while (qemu_isspace(*pch)) + pch++; + *q =3D 0; + ret =3D get_monitor_def(®, buf); + if (ret < 0) + expr_error(mon, "unknown register"); + n =3D reg; + } + break; + case '\0': + expr_error(mon, "unexpected end of expression"); + n =3D 0; + break; + default: + errno =3D 0; + n =3D strtoull(pch, &p, 0); + if (errno =3D=3D ERANGE) { + expr_error(mon, "number too large"); + } + if (pch =3D=3D p) { + expr_error(mon, "invalid char '%c' in expression", *p); + } + pch =3D p; + while (qemu_isspace(*pch)) + pch++; + break; + } + return n; +} + +static int64_t expr_prod(Monitor *mon) +{ + int64_t val, val2; + int op; + + val =3D expr_unary(mon); + for(;;) { + op =3D *pch; + if (op !=3D '*' && op !=3D '/' && op !=3D '%') + break; + next(); + val2 =3D expr_unary(mon); + switch(op) { + default: + case '*': + val *=3D val2; + break; + case '/': + case '%': + if (val2 =3D=3D 0) + expr_error(mon, "division by zero"); + if (op =3D=3D '/') + val /=3D val2; + else + val %=3D val2; + break; + } + } + return val; +} + +static int64_t expr_logic(Monitor *mon) +{ + int64_t val, val2; + int op; + + val =3D expr_prod(mon); + for(;;) { + op =3D *pch; + if (op !=3D '&' && op !=3D '|' && op !=3D '^') + break; + next(); + val2 =3D expr_prod(mon); + switch(op) { + default: + case '&': + val &=3D val2; + break; + case '|': + val |=3D val2; + break; + case '^': + val ^=3D val2; + break; + } + } + return val; +} + +static int64_t expr_sum(Monitor *mon) +{ + int64_t val, val2; + int op; + + val =3D expr_logic(mon); + for(;;) { + op =3D *pch; + if (op !=3D '+' && op !=3D '-') + break; + next(); + val2 =3D expr_logic(mon); + if (op =3D=3D '+') + val +=3D val2; + else + val -=3D val2; + } + return val; +} + +static int get_expr(Monitor *mon, int64_t *pval, const char **pp) +{ + pch =3D *pp; + if (sigsetjmp(expr_env, 0)) { + *pp =3D pch; + return -1; + } + while (qemu_isspace(*pch)) + pch++; + *pval =3D expr_sum(mon); + *pp =3D pch; + return 0; +} + +static int get_double(Monitor *mon, double *pval, const char **pp) +{ + const char *p =3D *pp; + char *tailp; + double d; + + d =3D strtod(p, &tailp); + if (tailp =3D=3D p) { + monitor_printf(mon, "Number expected\n"); + return -1; + } + if (d !=3D d || d - d !=3D 0) { + /* NaN or infinity */ + monitor_printf(mon, "Bad number\n"); + return -1; + } + *pval =3D d; + *pp =3D tailp; + return 0; +} + +/* + * Store the command-name in cmdname, and return a pointer to + * the remaining of the command string. + */ +static const char *get_command_name(const char *cmdline, + char *cmdname, size_t nlen) +{ + size_t len; + const char *p, *pstart; + + p =3D cmdline; + while (qemu_isspace(*p)) + p++; + if (*p =3D=3D '\0') + return NULL; + pstart =3D p; + while (*p !=3D '\0' && *p !=3D '/' && !qemu_isspace(*p)) + p++; + len =3D p - pstart; + if (len > nlen - 1) + len =3D nlen - 1; + memcpy(cmdname, pstart, len); + cmdname[len] =3D '\0'; + return p; +} + +/** + * Read key of 'type' into 'key' and return the current + * 'type' pointer. + */ +static char *key_get_info(const char *type, char **key) +{ + size_t len; + char *p, *str; + + if (*type =3D=3D ',') + type++; + + p =3D strchr(type, ':'); + if (!p) { + *key =3D NULL; + return NULL; + } + len =3D p - type; + + str =3D g_malloc(len + 1); + memcpy(str, type, len); + str[len] =3D '\0'; + + *key =3D str; + return ++p; +} + +static int default_fmt_format =3D 'x'; +static int default_fmt_size =3D 4; + +static int is_valid_option(const char *c, const char *typestr) +{ + char option[3]; + + option[0] =3D '-'; + option[1] =3D *c; + option[2] =3D '\0'; + + typestr =3D strstr(typestr, option); + return (typestr !=3D NULL); +} + +static const mon_cmd_t *search_dispatch_table(const mon_cmd_t *disp_table, + const char *cmdname) +{ + const mon_cmd_t *cmd; + + for (cmd =3D disp_table; cmd->name !=3D NULL; cmd++) { + if (compare_cmd(cmdname, cmd->name)) { + return cmd; + } + } + + return NULL; +} + +/* + * Parse command name from @cmdp according to command table @table. + * If blank, return NULL. + * Else, if no valid command can be found, report to @mon, and return + * NULL. + * Else, change @cmdp to point right behind the name, and return its + * command table entry. + * Do not assume the return value points into @table! It doesn't when + * the command is found in a sub-command table. + */ +static const mon_cmd_t *monitor_parse_command(MonitorHMP *hmp_mon, + const char *cmdp_start, + const char **cmdp, + mon_cmd_t *table) +{ + Monitor *mon =3D &hmp_mon->common; + const char *p; + const mon_cmd_t *cmd; + char cmdname[256]; + + /* extract the command name */ + p =3D get_command_name(*cmdp, cmdname, sizeof(cmdname)); + if (!p) { + return NULL; + } + + cmd =3D search_dispatch_table(table, cmdname); + if (!cmd) { + monitor_printf(mon, "unknown command: '%.*s'\n", + (int)(p - cmdp_start), cmdp_start); + return NULL; + } + if (runstate_check(RUN_STATE_PRECONFIG) && !cmd_can_preconfig(cmd)) { + monitor_printf(mon, "Command '%.*s' not available with -preconfig " + "until after exit_preconfig.\n", + (int)(p - cmdp_start), cmdp_start); + return NULL; + } + + /* filter out following useless space */ + while (qemu_isspace(*p)) { + p++; + } + + *cmdp =3D p; + /* search sub command */ + if (cmd->sub_table !=3D NULL && *p !=3D '\0') { + return monitor_parse_command(hmp_mon, cmdp_start, cmdp, cmd->sub_t= able); + } + + return cmd; +} + +/* + * Parse arguments for @cmd. + * If it can't be parsed, report to @mon, and return NULL. + * Else, insert command arguments into a QDict, and return it. + * Note: On success, caller has to free the QDict structure. + */ +static QDict *monitor_parse_arguments(Monitor *mon, + const char **endp, + const mon_cmd_t *cmd) +{ + const char *typestr; + char *key; + int c; + const char *p =3D *endp; + char buf[1024]; + QDict *qdict =3D qdict_new(); + + /* parse the parameters */ + typestr =3D cmd->args_type; + for(;;) { + typestr =3D key_get_info(typestr, &key); + if (!typestr) { + break; + } + c =3D *typestr; + typestr++; + switch(c) { + case 'F': + case 'B': + case 's': + { + int ret; + + while (qemu_isspace(*p)) + p++; + if (*typestr =3D=3D '?') { + typestr++; + if (*p =3D=3D '\0') { + /* no optional string: NULL argument */ + break; + } + } + ret =3D get_str(buf, sizeof(buf), &p); + if (ret < 0) { + switch(c) { + case 'F': + monitor_printf(mon, "%s: filename expected\n", + cmd->name); + break; + case 'B': + monitor_printf(mon, "%s: block device name expecte= d\n", + cmd->name); + break; + default: + monitor_printf(mon, "%s: string expected\n", cmd->= name); + break; + } + goto fail; + } + qdict_put_str(qdict, key, buf); + } + break; + case 'O': + { + QemuOptsList *opts_list; + QemuOpts *opts; + + opts_list =3D qemu_find_opts(key); + if (!opts_list || opts_list->desc->name) { + goto bad_type; + } + while (qemu_isspace(*p)) { + p++; + } + if (!*p) { + break; + } + if (get_str(buf, sizeof(buf), &p) < 0) { + goto fail; + } + opts =3D qemu_opts_parse_noisily(opts_list, buf, true); + if (!opts) { + goto fail; + } + qemu_opts_to_qdict(opts, qdict); + qemu_opts_del(opts); + } + break; + case '/': + { + int count, format, size; + + while (qemu_isspace(*p)) + p++; + if (*p =3D=3D '/') { + /* format found */ + p++; + count =3D 1; + if (qemu_isdigit(*p)) { + count =3D 0; + while (qemu_isdigit(*p)) { + count =3D count * 10 + (*p - '0'); + p++; + } + } + size =3D -1; + format =3D -1; + for(;;) { + switch(*p) { + case 'o': + case 'd': + case 'u': + case 'x': + case 'i': + case 'c': + format =3D *p++; + break; + case 'b': + size =3D 1; + p++; + break; + case 'h': + size =3D 2; + p++; + break; + case 'w': + size =3D 4; + p++; + break; + case 'g': + case 'L': + size =3D 8; + p++; + break; + default: + goto next; + } + } + next: + if (*p !=3D '\0' && !qemu_isspace(*p)) { + monitor_printf(mon, "invalid char in format: '%c'\= n", + *p); + goto fail; + } + if (format < 0) { + format =3D default_fmt_format; + } + if (format !=3D 'i') { + /* for 'i', not specifying a size gives -1 as size= */ + if (size < 0) { + size =3D default_fmt_size; + } + default_fmt_size =3D size; + } + default_fmt_format =3D format; + } else { + count =3D 1; + format =3D default_fmt_format; + if (format !=3D 'i') { + size =3D default_fmt_size; + } else { + size =3D -1; + } + } + qdict_put_int(qdict, "count", count); + qdict_put_int(qdict, "format", format); + qdict_put_int(qdict, "size", size); + } + break; + case 'i': + case 'l': + case 'M': + { + int64_t val; + + while (qemu_isspace(*p)) + p++; + if (*typestr =3D=3D '?' || *typestr =3D=3D '.') { + if (*typestr =3D=3D '?') { + if (*p =3D=3D '\0') { + typestr++; + break; + } + } else { + if (*p =3D=3D '.') { + p++; + while (qemu_isspace(*p)) { + p++; + } + } else { + typestr++; + break; + } + } + typestr++; + } + if (get_expr(mon, &val, &p)) + goto fail; + /* Check if 'i' is greater than 32-bit */ + if ((c =3D=3D 'i') && ((val >> 32) & 0xffffffff)) { + monitor_printf(mon, "\'%s\' has failed: ", cmd->name); + monitor_printf(mon, "integer is for 32-bit values\n"); + goto fail; + } else if (c =3D=3D 'M') { + if (val < 0) { + monitor_printf(mon, "enter a positive value\n"); + goto fail; + } + val *=3D MiB; + } + qdict_put_int(qdict, key, val); + } + break; + case 'o': + { + int ret; + uint64_t val; + const char *end; + + while (qemu_isspace(*p)) { + p++; + } + if (*typestr =3D=3D '?') { + typestr++; + if (*p =3D=3D '\0') { + break; + } + } + ret =3D qemu_strtosz_MiB(p, &end, &val); + if (ret < 0 || val > INT64_MAX) { + monitor_printf(mon, "invalid size\n"); + goto fail; + } + qdict_put_int(qdict, key, val); + p =3D end; + } + break; + case 'T': + { + double val; + + while (qemu_isspace(*p)) { + p++; + } + if (*typestr =3D=3D '?') { + typestr++; + if (*p =3D=3D '\0') { + break; + } + } + if (get_double(mon, &val, &p) < 0) { + goto fail; + } + if (p[0] && p[1] =3D=3D 's') { + switch (*p) { + case 'm': + val /=3D 1e3; p +=3D 2; break; + case 'u': + val /=3D 1e6; p +=3D 2; break; + case 'n': + val /=3D 1e9; p +=3D 2; break; + } + } + if (*p && !qemu_isspace(*p)) { + monitor_printf(mon, "Unknown unit suffix\n"); + goto fail; + } + qdict_put(qdict, key, qnum_from_double(val)); + } + break; + case 'b': + { + const char *beg; + bool val; + + while (qemu_isspace(*p)) { + p++; + } + beg =3D p; + while (qemu_isgraph(*p)) { + p++; + } + if (p - beg =3D=3D 2 && !memcmp(beg, "on", p - beg)) { + val =3D true; + } else if (p - beg =3D=3D 3 && !memcmp(beg, "off", p - beg= )) { + val =3D false; + } else { + monitor_printf(mon, "Expected 'on' or 'off'\n"); + goto fail; + } + qdict_put_bool(qdict, key, val); + } + break; + case '-': + { + const char *tmp =3D p; + int skip_key =3D 0; + /* option */ + + c =3D *typestr++; + if (c =3D=3D '\0') + goto bad_type; + while (qemu_isspace(*p)) + p++; + if (*p =3D=3D '-') { + p++; + if(c !=3D *p) { + if(!is_valid_option(p, typestr)) { + monitor_printf(mon, "%s: unsupported option -%= c\n", + cmd->name, *p); + goto fail; + } else { + skip_key =3D 1; + } + } + if(skip_key) { + p =3D tmp; + } else { + /* has option */ + p++; + qdict_put_bool(qdict, key, true); + } + } + } + break; + case 'S': + { + /* package all remaining string */ + int len; + + while (qemu_isspace(*p)) { + p++; + } + if (*typestr =3D=3D '?') { + typestr++; + if (*p =3D=3D '\0') { + /* no remaining string: NULL argument */ + break; + } + } + len =3D strlen(p); + if (len <=3D 0) { + monitor_printf(mon, "%s: string expected\n", + cmd->name); + goto fail; + } + qdict_put_str(qdict, key, p); + p +=3D len; + } + break; + default: + bad_type: + monitor_printf(mon, "%s: unknown type '%c'\n", cmd->name, c); + goto fail; + } + g_free(key); + key =3D NULL; + } + /* check that all arguments were parsed */ + while (qemu_isspace(*p)) + p++; + if (*p !=3D '\0') { + monitor_printf(mon, "%s: extraneous characters at the end of line\= n", + cmd->name); + goto fail; + } + + return qdict; + +fail: + qobject_unref(qdict); + g_free(key); + return NULL; +} + +void handle_hmp_command(MonitorHMP *mon, const char *cmdline) +{ + QDict *qdict; + const mon_cmd_t *cmd; + const char *cmd_start =3D cmdline; + + trace_handle_hmp_command(mon, cmdline); + + cmd =3D monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table); + if (!cmd) { + return; + } + + qdict =3D monitor_parse_arguments(&mon->common, &cmdline, cmd); + if (!qdict) { + while (cmdline > cmd_start && qemu_isspace(cmdline[-1])) { + cmdline--; + } + monitor_printf(&mon->common, "Try \"help %.*s\" for more informati= on\n", + (int)(cmdline - cmd_start), cmd_start); + return; + } + + cmd->cmd(&mon->common, qdict); + qobject_unref(qdict); +} + +static void cmd_completion(MonitorHMP *mon, const char *name, const char *= list) +{ + const char *p, *pstart; + char cmd[128]; + int len; + + p =3D list; + for(;;) { + pstart =3D p; + p =3D qemu_strchrnul(p, '|'); + len =3D p - pstart; + if (len > sizeof(cmd) - 2) + len =3D sizeof(cmd) - 2; + memcpy(cmd, pstart, len); + cmd[len] =3D '\0'; + if (name[0] =3D=3D '\0' || !strncmp(name, cmd, strlen(name))) { + readline_add_completion(mon->rs, cmd); + } + if (*p =3D=3D '\0') + break; + p++; + } +} + +static void file_completion(MonitorHMP *mon, const char *input) +{ + DIR *ffs; + struct dirent *d; + char path[1024]; + char file[1024], file_prefix[1024]; + int input_path_len; + const char *p; + + p =3D strrchr(input, '/'); + if (!p) { + input_path_len =3D 0; + pstrcpy(file_prefix, sizeof(file_prefix), input); + pstrcpy(path, sizeof(path), "."); + } else { + input_path_len =3D p - input + 1; + memcpy(path, input, input_path_len); + if (input_path_len > sizeof(path) - 1) + input_path_len =3D sizeof(path) - 1; + path[input_path_len] =3D '\0'; + pstrcpy(file_prefix, sizeof(file_prefix), p + 1); + } + + ffs =3D opendir(path); + if (!ffs) + return; + for(;;) { + struct stat sb; + d =3D readdir(ffs); + if (!d) + break; + + if (strcmp(d->d_name, ".") =3D=3D 0 || strcmp(d->d_name, "..") =3D= =3D 0) { + continue; + } + + if (strstart(d->d_name, file_prefix, NULL)) { + memcpy(file, input, input_path_len); + if (input_path_len < sizeof(file)) + pstrcpy(file + input_path_len, sizeof(file) - input_path_l= en, + d->d_name); + /* stat the file to find out if it's a directory. + * In that case add a slash to speed up typing long paths + */ + if (stat(file, &sb) =3D=3D 0 && S_ISDIR(sb.st_mode)) { + pstrcat(file, sizeof(file), "/"); + } + readline_add_completion(mon->rs, file); + } + } + closedir(ffs); +} + +static const char *next_arg_type(const char *typestr) +{ + const char *p =3D strchr(typestr, ':'); + return (p !=3D NULL ? ++p : typestr); +} + +static void monitor_find_completion_by_table(MonitorHMP *mon, + const mon_cmd_t *cmd_table, + char **args, + int nb_args) +{ + const char *cmdname; + int i; + const char *ptype, *old_ptype, *str, *name; + const mon_cmd_t *cmd; + BlockBackend *blk =3D NULL; + + if (nb_args <=3D 1) { + /* command completion */ + if (nb_args =3D=3D 0) + cmdname =3D ""; + else + cmdname =3D args[0]; + readline_set_completion_index(mon->rs, strlen(cmdname)); + for (cmd =3D cmd_table; cmd->name !=3D NULL; cmd++) { + if (!runstate_check(RUN_STATE_PRECONFIG) || + cmd_can_preconfig(cmd)) { + cmd_completion(mon, cmdname, cmd->name); + } + } + } else { + /* find the command */ + for (cmd =3D cmd_table; cmd->name !=3D NULL; cmd++) { + if (compare_cmd(args[0], cmd->name) && + (!runstate_check(RUN_STATE_PRECONFIG) || + cmd_can_preconfig(cmd))) { + break; + } + } + if (!cmd->name) { + return; + } + + if (cmd->sub_table) { + /* do the job again */ + monitor_find_completion_by_table(mon, cmd->sub_table, + &args[1], nb_args - 1); + return; + } + if (cmd->command_completion) { + cmd->command_completion(mon->rs, nb_args, args[nb_args - 1]); + return; + } + + ptype =3D next_arg_type(cmd->args_type); + for(i =3D 0; i < nb_args - 2; i++) { + if (*ptype !=3D '\0') { + ptype =3D next_arg_type(ptype); + while (*ptype =3D=3D '?') + ptype =3D next_arg_type(ptype); + } + } + str =3D args[nb_args - 1]; + old_ptype =3D NULL; + while (*ptype =3D=3D '-' && old_ptype !=3D ptype) { + old_ptype =3D ptype; + ptype =3D next_arg_type(ptype); + } + switch(*ptype) { + case 'F': + /* file completion */ + readline_set_completion_index(mon->rs, strlen(str)); + file_completion(mon, str); + break; + case 'B': + /* block device name completion */ + readline_set_completion_index(mon->rs, strlen(str)); + while ((blk =3D blk_next(blk)) !=3D NULL) { + name =3D blk_name(blk); + if (str[0] =3D=3D '\0' || + !strncmp(name, str, strlen(str))) { + readline_add_completion(mon->rs, name); + } + } + break; + case 's': + case 'S': + if (!strcmp(cmd->name, "help|?")) { + monitor_find_completion_by_table(mon, cmd_table, + &args[1], nb_args - 1); + } + break; + default: + break; + } + } +} + +static void monitor_find_completion(void *opaque, + const char *cmdline) +{ + MonitorHMP *mon =3D opaque; + char *args[MAX_ARGS]; + int nb_args, len; + + /* 1. parse the cmdline */ + if (parse_cmdline(cmdline, &nb_args, args) < 0) { + return; + } + + /* if the line ends with a space, it means we want to complete the + next arg */ + len =3D strlen(cmdline); + if (len > 0 && qemu_isspace(cmdline[len - 1])) { + if (nb_args >=3D MAX_ARGS) { + goto cleanup; + } + args[nb_args++] =3D g_strdup(""); + } + + /* 2. auto complete according to args */ + monitor_find_completion_by_table(mon, mon->cmd_table, args, nb_args); + +cleanup: + free_cmdline_args(args, nb_args); +} + +static void monitor_read(void *opaque, const uint8_t *buf, int size) +{ + MonitorHMP *mon; + Monitor *old_mon =3D cur_mon; + int i; + + cur_mon =3D opaque; + mon =3D container_of(cur_mon, MonitorHMP, common); + + if (mon->rs) { + for (i =3D 0; i < size; i++) { + readline_handle_byte(mon->rs, buf[i]); + } + } else { + if (size =3D=3D 0 || buf[size - 1] !=3D 0) { + monitor_printf(cur_mon, "corrupted command\n"); + } else { + handle_hmp_command(mon, (char *)buf); + } + } + + cur_mon =3D old_mon; +} + +static void monitor_event(void *opaque, int event) +{ + Monitor *mon =3D opaque; + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); + + switch (event) { + case CHR_EVENT_MUX_IN: + qemu_mutex_lock(&mon->mon_lock); + mon->mux_out =3D 0; + qemu_mutex_unlock(&mon->mon_lock); + if (mon->reset_seen) { + readline_restart(hmp_mon->rs); + monitor_resume(mon); + monitor_flush(mon); + } else { + atomic_mb_set(&mon->suspend_cnt, 0); + } + break; + + case CHR_EVENT_MUX_OUT: + if (mon->reset_seen) { + if (atomic_mb_read(&mon->suspend_cnt) =3D=3D 0) { + monitor_printf(mon, "\n"); + } + monitor_flush(mon); + monitor_suspend(mon); + } else { + atomic_inc(&mon->suspend_cnt); + } + qemu_mutex_lock(&mon->mon_lock); + mon->mux_out =3D 1; + qemu_mutex_unlock(&mon->mon_lock); + break; + + case CHR_EVENT_OPENED: + monitor_printf(mon, "QEMU %s monitor - type 'help' for more " + "information\n", QEMU_VERSION); + if (!mon->mux_out) { + readline_restart(hmp_mon->rs); + readline_show_prompt(hmp_mon->rs); + } + mon->reset_seen =3D 1; + mon_refcount++; + break; + + case CHR_EVENT_CLOSED: + mon_refcount--; + monitor_fdsets_cleanup(); + break; + } +} + + +/* These functions just adapt the readline interface in a typesafe way. We + * could cast function pointers but that discards compiler checks. + */ +static void GCC_FMT_ATTR(2, 3) monitor_readline_printf(void *opaque, + const char *fmt, ..= .) +{ + MonitorHMP *mon =3D opaque; + va_list ap; + va_start(ap, fmt); + monitor_vprintf(&mon->common, fmt, ap); + va_end(ap); +} + +static void monitor_readline_flush(void *opaque) +{ + MonitorHMP *mon =3D opaque; + monitor_flush(&mon->common); +} + +void monitor_data_init_hmp(MonitorHMP *mon, int flags, bool skip_flush) +{ + monitor_data_init(&mon->common, flags, skip_flush, false); + + /* Use *mon_cmds by default. */ + mon->cmd_table =3D mon_cmds; +} + +void monitor_init_hmp(Chardev *chr, int flags) +{ + MonitorHMP *mon =3D g_malloc0(sizeof(*mon)); + bool use_readline =3D flags & MONITOR_USE_READLINE; + + monitor_data_init_hmp(mon, flags, false); + qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); + + if (use_readline) { + mon->rs =3D readline_init(monitor_readline_printf, + monitor_readline_flush, + mon, + monitor_find_completion); + monitor_read_command(mon, 0); + } + + qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, monitor_r= ead, + monitor_event, NULL, &mon->common, NULL, true= ); + monitor_list_append(&mon->common); +} diff --git a/monitor/misc.c b/monitor/misc.c index 6c67f0978c..408d11e1fe 100644 --- a/monitor/misc.c +++ b/monitor/misc.c @@ -24,7 +24,6 @@ =20 #include "qemu/osdep.h" #include "monitor_int.h" -#include "qemu/units.h" #include #include "cpu.h" #include "hw/hw.h" @@ -155,7 +154,6 @@ static QLIST_HEAD(, MonFdset) mon_fdsets; =20 int mon_refcount; =20 -static mon_cmd_t mon_cmds[]; static mon_cmd_t info_cmds[]; =20 __thread Monitor *cur_mon; @@ -540,8 +538,6 @@ static void monitor_qapi_event_init(void) qapi_event_throttle_equal); } =20 -static void handle_hmp_command(MonitorHMP *mon, const char *cmdline); - static void monitor_iothread_init(void); =20 void monitor_data_init(Monitor *mon, int flags, bool skip_flush, @@ -558,14 +554,6 @@ void monitor_data_init(Monitor *mon, int flags, bool s= kip_flush, mon->flags =3D flags; } =20 -static void monitor_data_init_hmp(MonitorHMP *mon, int flags, bool skip_fl= ush) -{ - monitor_data_init(&mon->common, flags, skip_flush, false); - - /* Use *mon_cmds by default. */ - mon->cmd_table =3D mon_cmds; -} - static void monitor_data_destroy(Monitor *mon) { g_free(mon->mon_cpu_path); @@ -619,248 +607,6 @@ out: return output; } =20 -static int compare_cmd(const char *name, const char *list) -{ - const char *p, *pstart; - int len; - len =3D strlen(name); - p =3D list; - for(;;) { - pstart =3D p; - p =3D qemu_strchrnul(p, '|'); - if ((p - pstart) =3D=3D len && !memcmp(pstart, name, len)) - return 1; - if (*p =3D=3D '\0') - break; - p++; - } - return 0; -} - -static int get_str(char *buf, int buf_size, const char **pp) -{ - const char *p; - char *q; - int c; - - q =3D buf; - p =3D *pp; - while (qemu_isspace(*p)) { - p++; - } - if (*p =3D=3D '\0') { - fail: - *q =3D '\0'; - *pp =3D p; - return -1; - } - if (*p =3D=3D '\"') { - p++; - while (*p !=3D '\0' && *p !=3D '\"') { - if (*p =3D=3D '\\') { - p++; - c =3D *p++; - switch (c) { - case 'n': - c =3D '\n'; - break; - case 'r': - c =3D '\r'; - break; - case '\\': - case '\'': - case '\"': - break; - default: - printf("unsupported escape code: '\\%c'\n", c); - goto fail; - } - if ((q - buf) < buf_size - 1) { - *q++ =3D c; - } - } else { - if ((q - buf) < buf_size - 1) { - *q++ =3D *p; - } - p++; - } - } - if (*p !=3D '\"') { - printf("unterminated string\n"); - goto fail; - } - p++; - } else { - while (*p !=3D '\0' && !qemu_isspace(*p)) { - if ((q - buf) < buf_size - 1) { - *q++ =3D *p; - } - p++; - } - } - *q =3D '\0'; - *pp =3D p; - return 0; -} - -#define MAX_ARGS 16 - -static void free_cmdline_args(char **args, int nb_args) -{ - int i; - - assert(nb_args <=3D MAX_ARGS); - - for (i =3D 0; i < nb_args; i++) { - g_free(args[i]); - } - -} - -/* - * Parse the command line to get valid args. - * @cmdline: command line to be parsed. - * @pnb_args: location to store the number of args, must NOT be NULL. - * @args: location to store the args, which should be freed by caller, must - * NOT be NULL. - * - * Returns 0 on success, negative on failure. - * - * NOTE: this parser is an approximate form of the real command parser. Nu= mber - * of args have a limit of MAX_ARGS. If cmdline contains more, it wi= ll - * return with failure. - */ -static int parse_cmdline(const char *cmdline, - int *pnb_args, char **args) -{ - const char *p; - int nb_args, ret; - char buf[1024]; - - p =3D cmdline; - nb_args =3D 0; - for (;;) { - while (qemu_isspace(*p)) { - p++; - } - if (*p =3D=3D '\0') { - break; - } - if (nb_args >=3D MAX_ARGS) { - goto fail; - } - ret =3D get_str(buf, sizeof(buf), &p); - if (ret < 0) { - goto fail; - } - args[nb_args] =3D g_strdup(buf); - nb_args++; - } - *pnb_args =3D nb_args; - return 0; - - fail: - free_cmdline_args(args, nb_args); - return -1; -} - -/* - * Can command @cmd be executed in preconfig state? - */ -static bool cmd_can_preconfig(const mon_cmd_t *cmd) -{ - if (!cmd->flags) { - return false; - } - - return strchr(cmd->flags, 'p'); -} - -static void help_cmd_dump_one(Monitor *mon, - const mon_cmd_t *cmd, - char **prefix_args, - int prefix_args_nb) -{ - int i; - - if (runstate_check(RUN_STATE_PRECONFIG) && !cmd_can_preconfig(cmd)) { - return; - } - - for (i =3D 0; i < prefix_args_nb; i++) { - monitor_printf(mon, "%s ", prefix_args[i]); - } - monitor_printf(mon, "%s %s -- %s\n", cmd->name, cmd->params, cmd->help= ); -} - -/* @args[@arg_index] is the valid command need to find in @cmds */ -static void help_cmd_dump(Monitor *mon, const mon_cmd_t *cmds, - char **args, int nb_args, int arg_index) -{ - const mon_cmd_t *cmd; - size_t i; - - /* No valid arg need to compare with, dump all in *cmds */ - if (arg_index >=3D nb_args) { - for (cmd =3D cmds; cmd->name !=3D NULL; cmd++) { - help_cmd_dump_one(mon, cmd, args, arg_index); - } - return; - } - - /* Find one entry to dump */ - for (cmd =3D cmds; cmd->name !=3D NULL; cmd++) { - if (compare_cmd(args[arg_index], cmd->name) && - ((!runstate_check(RUN_STATE_PRECONFIG) || - cmd_can_preconfig(cmd)))) { - if (cmd->sub_table) { - /* continue with next arg */ - help_cmd_dump(mon, cmd->sub_table, - args, nb_args, arg_index + 1); - } else { - help_cmd_dump_one(mon, cmd, args, arg_index); - } - return; - } - } - - /* Command not found */ - monitor_printf(mon, "unknown command: '"); - for (i =3D 0; i <=3D arg_index; i++) { - monitor_printf(mon, "%s%s", args[i], i =3D=3D arg_index ? "'\n" : = " "); - } -} - -static void help_cmd(Monitor *mon, const char *name) -{ - MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); - char *args[MAX_ARGS]; - int nb_args =3D 0; - - /* 1. parse user input */ - if (name) { - /* special case for log, directly dump and return */ - if (!strcmp(name, "log")) { - const QEMULogItem *item; - monitor_printf(mon, "Log items (comma separated):\n"); - monitor_printf(mon, "%-10s %s\n", "none", "remove all logs"); - for (item =3D qemu_log_items; item->mask !=3D 0; item++) { - monitor_printf(mon, "%-10s %s\n", item->name, item->help); - } - return; - } - - if (parse_cmdline(name, &nb_args, args) < 0) { - return; - } - } - - /* 2. dump the contents according to parsed args */ - help_cmd_dump(mon, hmp_mon->cmd_table, args, nb_args, 0); - - free_cmdline_args(args, nb_args); -} - static void do_help_cmd(Monitor *mon, const QDict *qdict) { help_cmd(mon, qdict_get_try_str(qdict, "name")); @@ -2510,30 +2256,16 @@ static mon_cmd_t info_cmds[] =3D { }; =20 /* mon_cmds and info_cmds would be sorted at runtime */ -static mon_cmd_t mon_cmds[] =3D { +mon_cmd_t mon_cmds[] =3D { #include "hmp-commands.h" { NULL, NULL, }, }; =20 -/*******************************************************************/ - -static const char *pch; -static sigjmp_buf expr_env; - - -static void GCC_FMT_ATTR(2, 3) QEMU_NORETURN -expr_error(Monitor *mon, const char *fmt, ...) -{ - va_list ap; - va_start(ap, fmt); - monitor_vprintf(mon, fmt, ap); - monitor_printf(mon, "\n"); - va_end(ap); - siglongjmp(expr_env, 1); -} - -/* return 0 if OK, -1 if not found */ -static int get_monitor_def(target_long *pval, const char *name) +/* + * Set @pval to the value in the register identified by @name. + * return 0 if OK, -1 if not found + */ +int get_monitor_def(int64_t *pval, const char *name) { const MonitorDef *md =3D target_monitor_defs(); CPUState *cs =3D mon_get_cpu(); @@ -2576,829 +2308,6 @@ static int get_monitor_def(target_long *pval, const= char *name) return ret; } =20 -static void next(void) -{ - if (*pch !=3D '\0') { - pch++; - while (qemu_isspace(*pch)) - pch++; - } -} - -static int64_t expr_sum(Monitor *mon); - -static int64_t expr_unary(Monitor *mon) -{ - int64_t n; - char *p; - int ret; - - switch(*pch) { - case '+': - next(); - n =3D expr_unary(mon); - break; - case '-': - next(); - n =3D -expr_unary(mon); - break; - case '~': - next(); - n =3D ~expr_unary(mon); - break; - case '(': - next(); - n =3D expr_sum(mon); - if (*pch !=3D ')') { - expr_error(mon, "')' expected"); - } - next(); - break; - case '\'': - pch++; - if (*pch =3D=3D '\0') - expr_error(mon, "character constant expected"); - n =3D *pch; - pch++; - if (*pch !=3D '\'') - expr_error(mon, "missing terminating \' character"); - next(); - break; - case '$': - { - char buf[128], *q; - target_long reg=3D0; - - pch++; - q =3D buf; - while ((*pch >=3D 'a' && *pch <=3D 'z') || - (*pch >=3D 'A' && *pch <=3D 'Z') || - (*pch >=3D '0' && *pch <=3D '9') || - *pch =3D=3D '_' || *pch =3D=3D '.') { - if ((q - buf) < sizeof(buf) - 1) - *q++ =3D *pch; - pch++; - } - while (qemu_isspace(*pch)) - pch++; - *q =3D 0; - ret =3D get_monitor_def(®, buf); - if (ret < 0) - expr_error(mon, "unknown register"); - n =3D reg; - } - break; - case '\0': - expr_error(mon, "unexpected end of expression"); - n =3D 0; - break; - default: - errno =3D 0; - n =3D strtoull(pch, &p, 0); - if (errno =3D=3D ERANGE) { - expr_error(mon, "number too large"); - } - if (pch =3D=3D p) { - expr_error(mon, "invalid char '%c' in expression", *p); - } - pch =3D p; - while (qemu_isspace(*pch)) - pch++; - break; - } - return n; -} - - -static int64_t expr_prod(Monitor *mon) -{ - int64_t val, val2; - int op; - - val =3D expr_unary(mon); - for(;;) { - op =3D *pch; - if (op !=3D '*' && op !=3D '/' && op !=3D '%') - break; - next(); - val2 =3D expr_unary(mon); - switch(op) { - default: - case '*': - val *=3D val2; - break; - case '/': - case '%': - if (val2 =3D=3D 0) - expr_error(mon, "division by zero"); - if (op =3D=3D '/') - val /=3D val2; - else - val %=3D val2; - break; - } - } - return val; -} - -static int64_t expr_logic(Monitor *mon) -{ - int64_t val, val2; - int op; - - val =3D expr_prod(mon); - for(;;) { - op =3D *pch; - if (op !=3D '&' && op !=3D '|' && op !=3D '^') - break; - next(); - val2 =3D expr_prod(mon); - switch(op) { - default: - case '&': - val &=3D val2; - break; - case '|': - val |=3D val2; - break; - case '^': - val ^=3D val2; - break; - } - } - return val; -} - -static int64_t expr_sum(Monitor *mon) -{ - int64_t val, val2; - int op; - - val =3D expr_logic(mon); - for(;;) { - op =3D *pch; - if (op !=3D '+' && op !=3D '-') - break; - next(); - val2 =3D expr_logic(mon); - if (op =3D=3D '+') - val +=3D val2; - else - val -=3D val2; - } - return val; -} - -static int get_expr(Monitor *mon, int64_t *pval, const char **pp) -{ - pch =3D *pp; - if (sigsetjmp(expr_env, 0)) { - *pp =3D pch; - return -1; - } - while (qemu_isspace(*pch)) - pch++; - *pval =3D expr_sum(mon); - *pp =3D pch; - return 0; -} - -static int get_double(Monitor *mon, double *pval, const char **pp) -{ - const char *p =3D *pp; - char *tailp; - double d; - - d =3D strtod(p, &tailp); - if (tailp =3D=3D p) { - monitor_printf(mon, "Number expected\n"); - return -1; - } - if (d !=3D d || d - d !=3D 0) { - /* NaN or infinity */ - monitor_printf(mon, "Bad number\n"); - return -1; - } - *pval =3D d; - *pp =3D tailp; - return 0; -} - -/* - * Store the command-name in cmdname, and return a pointer to - * the remaining of the command string. - */ -static const char *get_command_name(const char *cmdline, - char *cmdname, size_t nlen) -{ - size_t len; - const char *p, *pstart; - - p =3D cmdline; - while (qemu_isspace(*p)) - p++; - if (*p =3D=3D '\0') - return NULL; - pstart =3D p; - while (*p !=3D '\0' && *p !=3D '/' && !qemu_isspace(*p)) - p++; - len =3D p - pstart; - if (len > nlen - 1) - len =3D nlen - 1; - memcpy(cmdname, pstart, len); - cmdname[len] =3D '\0'; - return p; -} - -/** - * Read key of 'type' into 'key' and return the current - * 'type' pointer. - */ -static char *key_get_info(const char *type, char **key) -{ - size_t len; - char *p, *str; - - if (*type =3D=3D ',') - type++; - - p =3D strchr(type, ':'); - if (!p) { - *key =3D NULL; - return NULL; - } - len =3D p - type; - - str =3D g_malloc(len + 1); - memcpy(str, type, len); - str[len] =3D '\0'; - - *key =3D str; - return ++p; -} - -static int default_fmt_format =3D 'x'; -static int default_fmt_size =3D 4; - -static int is_valid_option(const char *c, const char *typestr) -{ - char option[3]; - =20 - option[0] =3D '-'; - option[1] =3D *c; - option[2] =3D '\0'; - =20 - typestr =3D strstr(typestr, option); - return (typestr !=3D NULL); -} - -static const mon_cmd_t *search_dispatch_table(const mon_cmd_t *disp_table, - const char *cmdname) -{ - const mon_cmd_t *cmd; - - for (cmd =3D disp_table; cmd->name !=3D NULL; cmd++) { - if (compare_cmd(cmdname, cmd->name)) { - return cmd; - } - } - - return NULL; -} - -/* - * Parse command name from @cmdp according to command table @table. - * If blank, return NULL. - * Else, if no valid command can be found, report to @mon, and return - * NULL. - * Else, change @cmdp to point right behind the name, and return its - * command table entry. - * Do not assume the return value points into @table! It doesn't when - * the command is found in a sub-command table. - */ -static const mon_cmd_t *monitor_parse_command(MonitorHMP *hmp_mon, - const char *cmdp_start, - const char **cmdp, - mon_cmd_t *table) -{ - Monitor *mon =3D &hmp_mon->common; - const char *p; - const mon_cmd_t *cmd; - char cmdname[256]; - - /* extract the command name */ - p =3D get_command_name(*cmdp, cmdname, sizeof(cmdname)); - if (!p) - return NULL; - - cmd =3D search_dispatch_table(table, cmdname); - if (!cmd) { - monitor_printf(mon, "unknown command: '%.*s'\n", - (int)(p - cmdp_start), cmdp_start); - return NULL; - } - if (runstate_check(RUN_STATE_PRECONFIG) && !cmd_can_preconfig(cmd)) { - monitor_printf(mon, "Command '%.*s' not available with -preconfig " - "until after exit_preconfig.\n", - (int)(p - cmdp_start), cmdp_start); - return NULL; - } - - /* filter out following useless space */ - while (qemu_isspace(*p)) { - p++; - } - - *cmdp =3D p; - /* search sub command */ - if (cmd->sub_table !=3D NULL && *p !=3D '\0') { - return monitor_parse_command(hmp_mon, cmdp_start, cmdp, cmd->sub_t= able); - } - - return cmd; -} - -/* - * Parse arguments for @cmd. - * If it can't be parsed, report to @mon, and return NULL. - * Else, insert command arguments into a QDict, and return it. - * Note: On success, caller has to free the QDict structure. - */ - -static QDict *monitor_parse_arguments(Monitor *mon, - const char **endp, - const mon_cmd_t *cmd) -{ - const char *typestr; - char *key; - int c; - const char *p =3D *endp; - char buf[1024]; - QDict *qdict =3D qdict_new(); - - /* parse the parameters */ - typestr =3D cmd->args_type; - for(;;) { - typestr =3D key_get_info(typestr, &key); - if (!typestr) - break; - c =3D *typestr; - typestr++; - switch(c) { - case 'F': - case 'B': - case 's': - { - int ret; - - while (qemu_isspace(*p)) - p++; - if (*typestr =3D=3D '?') { - typestr++; - if (*p =3D=3D '\0') { - /* no optional string: NULL argument */ - break; - } - } - ret =3D get_str(buf, sizeof(buf), &p); - if (ret < 0) { - switch(c) { - case 'F': - monitor_printf(mon, "%s: filename expected\n", - cmd->name); - break; - case 'B': - monitor_printf(mon, "%s: block device name expecte= d\n", - cmd->name); - break; - default: - monitor_printf(mon, "%s: string expected\n", cmd->= name); - break; - } - goto fail; - } - qdict_put_str(qdict, key, buf); - } - break; - case 'O': - { - QemuOptsList *opts_list; - QemuOpts *opts; - - opts_list =3D qemu_find_opts(key); - if (!opts_list || opts_list->desc->name) { - goto bad_type; - } - while (qemu_isspace(*p)) { - p++; - } - if (!*p) - break; - if (get_str(buf, sizeof(buf), &p) < 0) { - goto fail; - } - opts =3D qemu_opts_parse_noisily(opts_list, buf, true); - if (!opts) { - goto fail; - } - qemu_opts_to_qdict(opts, qdict); - qemu_opts_del(opts); - } - break; - case '/': - { - int count, format, size; - - while (qemu_isspace(*p)) - p++; - if (*p =3D=3D '/') { - /* format found */ - p++; - count =3D 1; - if (qemu_isdigit(*p)) { - count =3D 0; - while (qemu_isdigit(*p)) { - count =3D count * 10 + (*p - '0'); - p++; - } - } - size =3D -1; - format =3D -1; - for(;;) { - switch(*p) { - case 'o': - case 'd': - case 'u': - case 'x': - case 'i': - case 'c': - format =3D *p++; - break; - case 'b': - size =3D 1; - p++; - break; - case 'h': - size =3D 2; - p++; - break; - case 'w': - size =3D 4; - p++; - break; - case 'g': - case 'L': - size =3D 8; - p++; - break; - default: - goto next; - } - } - next: - if (*p !=3D '\0' && !qemu_isspace(*p)) { - monitor_printf(mon, "invalid char in format: '%c'\= n", - *p); - goto fail; - } - if (format < 0) - format =3D default_fmt_format; - if (format !=3D 'i') { - /* for 'i', not specifying a size gives -1 as size= */ - if (size < 0) - size =3D default_fmt_size; - default_fmt_size =3D size; - } - default_fmt_format =3D format; - } else { - count =3D 1; - format =3D default_fmt_format; - if (format !=3D 'i') { - size =3D default_fmt_size; - } else { - size =3D -1; - } - } - qdict_put_int(qdict, "count", count); - qdict_put_int(qdict, "format", format); - qdict_put_int(qdict, "size", size); - } - break; - case 'i': - case 'l': - case 'M': - { - int64_t val; - - while (qemu_isspace(*p)) - p++; - if (*typestr =3D=3D '?' || *typestr =3D=3D '.') { - if (*typestr =3D=3D '?') { - if (*p =3D=3D '\0') { - typestr++; - break; - } - } else { - if (*p =3D=3D '.') { - p++; - while (qemu_isspace(*p)) - p++; - } else { - typestr++; - break; - } - } - typestr++; - } - if (get_expr(mon, &val, &p)) - goto fail; - /* Check if 'i' is greater than 32-bit */ - if ((c =3D=3D 'i') && ((val >> 32) & 0xffffffff)) { - monitor_printf(mon, "\'%s\' has failed: ", cmd->name); - monitor_printf(mon, "integer is for 32-bit values\n"); - goto fail; - } else if (c =3D=3D 'M') { - if (val < 0) { - monitor_printf(mon, "enter a positive value\n"); - goto fail; - } - val *=3D MiB; - } - qdict_put_int(qdict, key, val); - } - break; - case 'o': - { - int ret; - uint64_t val; - const char *end; - - while (qemu_isspace(*p)) { - p++; - } - if (*typestr =3D=3D '?') { - typestr++; - if (*p =3D=3D '\0') { - break; - } - } - ret =3D qemu_strtosz_MiB(p, &end, &val); - if (ret < 0 || val > INT64_MAX) { - monitor_printf(mon, "invalid size\n"); - goto fail; - } - qdict_put_int(qdict, key, val); - p =3D end; - } - break; - case 'T': - { - double val; - - while (qemu_isspace(*p)) - p++; - if (*typestr =3D=3D '?') { - typestr++; - if (*p =3D=3D '\0') { - break; - } - } - if (get_double(mon, &val, &p) < 0) { - goto fail; - } - if (p[0] && p[1] =3D=3D 's') { - switch (*p) { - case 'm': - val /=3D 1e3; p +=3D 2; break; - case 'u': - val /=3D 1e6; p +=3D 2; break; - case 'n': - val /=3D 1e9; p +=3D 2; break; - } - } - if (*p && !qemu_isspace(*p)) { - monitor_printf(mon, "Unknown unit suffix\n"); - goto fail; - } - qdict_put(qdict, key, qnum_from_double(val)); - } - break; - case 'b': - { - const char *beg; - bool val; - - while (qemu_isspace(*p)) { - p++; - } - beg =3D p; - while (qemu_isgraph(*p)) { - p++; - } - if (p - beg =3D=3D 2 && !memcmp(beg, "on", p - beg)) { - val =3D true; - } else if (p - beg =3D=3D 3 && !memcmp(beg, "off", p - beg= )) { - val =3D false; - } else { - monitor_printf(mon, "Expected 'on' or 'off'\n"); - goto fail; - } - qdict_put_bool(qdict, key, val); - } - break; - case '-': - { - const char *tmp =3D p; - int skip_key =3D 0; - /* option */ - - c =3D *typestr++; - if (c =3D=3D '\0') - goto bad_type; - while (qemu_isspace(*p)) - p++; - if (*p =3D=3D '-') { - p++; - if(c !=3D *p) { - if(!is_valid_option(p, typestr)) { - =20 - monitor_printf(mon, "%s: unsupported option -%= c\n", - cmd->name, *p); - goto fail; - } else { - skip_key =3D 1; - } - } - if(skip_key) { - p =3D tmp; - } else { - /* has option */ - p++; - qdict_put_bool(qdict, key, true); - } - } - } - break; - case 'S': - { - /* package all remaining string */ - int len; - - while (qemu_isspace(*p)) { - p++; - } - if (*typestr =3D=3D '?') { - typestr++; - if (*p =3D=3D '\0') { - /* no remaining string: NULL argument */ - break; - } - } - len =3D strlen(p); - if (len <=3D 0) { - monitor_printf(mon, "%s: string expected\n", - cmd->name); - goto fail; - } - qdict_put_str(qdict, key, p); - p +=3D len; - } - break; - default: - bad_type: - monitor_printf(mon, "%s: unknown type '%c'\n", cmd->name, c); - goto fail; - } - g_free(key); - key =3D NULL; - } - /* check that all arguments were parsed */ - while (qemu_isspace(*p)) - p++; - if (*p !=3D '\0') { - monitor_printf(mon, "%s: extraneous characters at the end of line\= n", - cmd->name); - goto fail; - } - - return qdict; - -fail: - qobject_unref(qdict); - g_free(key); - return NULL; -} - -static void handle_hmp_command(MonitorHMP *mon, const char *cmdline) -{ - QDict *qdict; - const mon_cmd_t *cmd; - const char *cmd_start =3D cmdline; - - trace_handle_hmp_command(mon, cmdline); - - cmd =3D monitor_parse_command(mon, cmdline, &cmdline, mon->cmd_table); - if (!cmd) { - return; - } - - qdict =3D monitor_parse_arguments(&mon->common, &cmdline, cmd); - if (!qdict) { - while (cmdline > cmd_start && qemu_isspace(cmdline[-1])) { - cmdline--; - } - monitor_printf(&mon->common, "Try \"help %.*s\" for more informati= on\n", - (int)(cmdline - cmd_start), cmd_start); - return; - } - - cmd->cmd(&mon->common, qdict); - qobject_unref(qdict); -} - -static void cmd_completion(MonitorHMP *mon, const char *name, const char *= list) -{ - const char *p, *pstart; - char cmd[128]; - int len; - - p =3D list; - for(;;) { - pstart =3D p; - p =3D qemu_strchrnul(p, '|'); - len =3D p - pstart; - if (len > sizeof(cmd) - 2) - len =3D sizeof(cmd) - 2; - memcpy(cmd, pstart, len); - cmd[len] =3D '\0'; - if (name[0] =3D=3D '\0' || !strncmp(name, cmd, strlen(name))) { - readline_add_completion(mon->rs, cmd); - } - if (*p =3D=3D '\0') - break; - p++; - } -} - -static void file_completion(MonitorHMP *mon, const char *input) -{ - DIR *ffs; - struct dirent *d; - char path[1024]; - char file[1024], file_prefix[1024]; - int input_path_len; - const char *p; - - p =3D strrchr(input, '/'); - if (!p) { - input_path_len =3D 0; - pstrcpy(file_prefix, sizeof(file_prefix), input); - pstrcpy(path, sizeof(path), "."); - } else { - input_path_len =3D p - input + 1; - memcpy(path, input, input_path_len); - if (input_path_len > sizeof(path) - 1) - input_path_len =3D sizeof(path) - 1; - path[input_path_len] =3D '\0'; - pstrcpy(file_prefix, sizeof(file_prefix), p + 1); - } - - ffs =3D opendir(path); - if (!ffs) - return; - for(;;) { - struct stat sb; - d =3D readdir(ffs); - if (!d) - break; - - if (strcmp(d->d_name, ".") =3D=3D 0 || strcmp(d->d_name, "..") =3D= =3D 0) { - continue; - } - - if (strstart(d->d_name, file_prefix, NULL)) { - memcpy(file, input, input_path_len); - if (input_path_len < sizeof(file)) - pstrcpy(file + input_path_len, sizeof(file) - input_path_l= en, - d->d_name); - /* stat the file to find out if it's a directory. - * In that case add a slash to speed up typing long paths - */ - if (stat(file, &sb) =3D=3D 0 && S_ISDIR(sb.st_mode)) { - pstrcat(file, sizeof(file), "/"); - } - readline_add_completion(mon->rs, file); - } - } - closedir(ffs); -} - -static const char *next_arg_type(const char *typestr) -{ - const char *p =3D strchr(typestr, ':'); - return (p !=3D NULL ? ++p : typestr); -} - static void add_completion_option(ReadLineState *rs, const char *str, const char *option) { @@ -3829,127 +2738,6 @@ void loadvm_completion(ReadLineState *rs, int nb_ar= gs, const char *str) } } =20 -static void monitor_find_completion_by_table(MonitorHMP *mon, - const mon_cmd_t *cmd_table, - char **args, - int nb_args) -{ - const char *cmdname; - int i; - const char *ptype, *old_ptype, *str, *name; - const mon_cmd_t *cmd; - BlockBackend *blk =3D NULL; - - if (nb_args <=3D 1) { - /* command completion */ - if (nb_args =3D=3D 0) - cmdname =3D ""; - else - cmdname =3D args[0]; - readline_set_completion_index(mon->rs, strlen(cmdname)); - for (cmd =3D cmd_table; cmd->name !=3D NULL; cmd++) { - if (!runstate_check(RUN_STATE_PRECONFIG) || - cmd_can_preconfig(cmd)) { - cmd_completion(mon, cmdname, cmd->name); - } - } - } else { - /* find the command */ - for (cmd =3D cmd_table; cmd->name !=3D NULL; cmd++) { - if (compare_cmd(args[0], cmd->name) && - (!runstate_check(RUN_STATE_PRECONFIG) || - cmd_can_preconfig(cmd))) { - break; - } - } - if (!cmd->name) { - return; - } - - if (cmd->sub_table) { - /* do the job again */ - monitor_find_completion_by_table(mon, cmd->sub_table, - &args[1], nb_args - 1); - return; - } - if (cmd->command_completion) { - cmd->command_completion(mon->rs, nb_args, args[nb_args - 1]); - return; - } - - ptype =3D next_arg_type(cmd->args_type); - for(i =3D 0; i < nb_args - 2; i++) { - if (*ptype !=3D '\0') { - ptype =3D next_arg_type(ptype); - while (*ptype =3D=3D '?') - ptype =3D next_arg_type(ptype); - } - } - str =3D args[nb_args - 1]; - old_ptype =3D NULL; - while (*ptype =3D=3D '-' && old_ptype !=3D ptype) { - old_ptype =3D ptype; - ptype =3D next_arg_type(ptype); - } - switch(*ptype) { - case 'F': - /* file completion */ - readline_set_completion_index(mon->rs, strlen(str)); - file_completion(mon, str); - break; - case 'B': - /* block device name completion */ - readline_set_completion_index(mon->rs, strlen(str)); - while ((blk =3D blk_next(blk)) !=3D NULL) { - name =3D blk_name(blk); - if (str[0] =3D=3D '\0' || - !strncmp(name, str, strlen(str))) { - readline_add_completion(mon->rs, name); - } - } - break; - case 's': - case 'S': - if (!strcmp(cmd->name, "help|?")) { - monitor_find_completion_by_table(mon, cmd_table, - &args[1], nb_args - 1); - } - break; - default: - break; - } - } -} - -static void monitor_find_completion(void *opaque, - const char *cmdline) -{ - MonitorHMP *mon =3D opaque; - char *args[MAX_ARGS]; - int nb_args, len; - - /* 1. parse the cmdline */ - if (parse_cmdline(cmdline, &nb_args, args) < 0) { - return; - } - - /* if the line ends with a space, it means we want to complete the - next arg */ - len =3D strlen(cmdline); - if (len > 0 && qemu_isspace(cmdline[len - 1])) { - if (nb_args >=3D MAX_ARGS) { - goto cleanup; - } - args[nb_args++] =3D g_strdup(""); - } - - /* 2. auto complete according to args */ - monitor_find_completion_by_table(mon, mon->cmd_table, args, nb_args); - -cleanup: - free_cmdline_args(args, nb_args); -} - int monitor_can_read(void *opaque) { Monitor *mon =3D opaque; @@ -3957,28 +2745,6 @@ int monitor_can_read(void *opaque) return !atomic_mb_read(&mon->suspend_cnt); } =20 -static void monitor_read(void *opaque, const uint8_t *buf, int size) -{ - MonitorHMP *mon; - Monitor *old_mon =3D cur_mon; - int i; - - cur_mon =3D opaque; - mon =3D container_of(cur_mon, MonitorHMP, common); - - if (mon->rs) { - for (i =3D 0; i < size; i++) - readline_handle_byte(mon->rs, buf[i]); - } else { - if (size =3D=3D 0 || buf[size - 1] !=3D 0) - monitor_printf(cur_mon, "corrupted command\n"); - else - handle_hmp_command(mon, (char *)buf); - } - - cur_mon =3D old_mon; -} - static void monitor_command_cb(void *opaque, const char *cmdline, void *readline_opaque) { @@ -4043,58 +2809,6 @@ void monitor_resume(Monitor *mon) trace_monitor_suspend(mon, -1); } =20 -static void monitor_event(void *opaque, int event) -{ - Monitor *mon =3D opaque; - MonitorHMP *hmp_mon =3D container_of(cur_mon, MonitorHMP, common); - - switch (event) { - case CHR_EVENT_MUX_IN: - qemu_mutex_lock(&mon->mon_lock); - mon->mux_out =3D 0; - qemu_mutex_unlock(&mon->mon_lock); - if (mon->reset_seen) { - readline_restart(hmp_mon->rs); - monitor_resume(mon); - monitor_flush(mon); - } else { - atomic_mb_set(&mon->suspend_cnt, 0); - } - break; - - case CHR_EVENT_MUX_OUT: - if (mon->reset_seen) { - if (atomic_mb_read(&mon->suspend_cnt) =3D=3D 0) { - monitor_printf(mon, "\n"); - } - monitor_flush(mon); - monitor_suspend(mon); - } else { - atomic_inc(&mon->suspend_cnt); - } - qemu_mutex_lock(&mon->mon_lock); - mon->mux_out =3D 1; - qemu_mutex_unlock(&mon->mon_lock); - break; - - case CHR_EVENT_OPENED: - monitor_printf(mon, "QEMU %s monitor - type 'help' for more " - "information\n", QEMU_VERSION); - if (!mon->mux_out) { - readline_restart(hmp_mon->rs); - readline_show_prompt(hmp_mon->rs); - } - mon->reset_seen =3D 1; - mon_refcount++; - break; - - case CHR_EVENT_CLOSED: - mon_refcount--; - monitor_fdsets_cleanup(); - break; - } -} - static int compare_mon_cmd(const void *a, const void *b) { @@ -4137,25 +2851,6 @@ void monitor_init_globals(void) NULL); } =20 -/* These functions just adapt the readline interface in a typesafe way. We - * could cast function pointers but that discards compiler checks. - */ -static void GCC_FMT_ATTR(2, 3) monitor_readline_printf(void *opaque, - const char *fmt, ..= .) -{ - MonitorHMP *mon =3D opaque; - va_list ap; - va_start(ap, fmt); - monitor_vprintf(&mon->common, fmt, ap); - va_end(ap); -} - -static void monitor_readline_flush(void *opaque) -{ - MonitorHMP *mon =3D opaque; - monitor_flush(&mon->common); -} - /* * Print to current monitor if we have one, else to stderr. */ @@ -4198,27 +2893,6 @@ void monitor_list_append(Monitor *mon) } } =20 -static void monitor_init_hmp(Chardev *chr, int flags) -{ - MonitorHMP *mon =3D g_malloc0(sizeof(*mon)); - bool use_readline =3D flags & MONITOR_USE_READLINE; - - monitor_data_init_hmp(mon, flags, false); - qemu_chr_fe_init(&mon->common.chr, chr, &error_abort); - - if (use_readline) { - mon->rs =3D readline_init(monitor_readline_printf, - monitor_readline_flush, - mon, - monitor_find_completion); - monitor_read_command(mon, 0); - } - - qemu_chr_fe_set_handlers(&mon->common.chr, monitor_can_read, monitor_r= ead, - monitor_event, NULL, &mon->common, NULL, true= ); - monitor_list_append(&mon->common); -} - void monitor_init(Chardev *chr, int flags) { if (flags & MONITOR_USE_CONTROL) { diff --git a/monitor/Makefile.objs b/monitor/Makefile.objs index d04d58b583..48c73eed51 100644 --- a/monitor/Makefile.objs +++ b/monitor/Makefile.objs @@ -1,2 +1,2 @@ obj-y +=3D misc.o -common-obj-y +=3D qmp.o +common-obj-y +=3D qmp.o hmp.o --=20 2.20.1 From nobody Tue Sep 30 19:50:59 2025 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) client-ip=209.51.188.47; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=redhat.com ARC-Seal: i=1; a=rsa-sha256; t=1559916178; cv=none; d=zoho.com; s=zohoarc; b=MK17TdLVp1+TXFCL/PMUWVRuV48X1oH3TCDxeS5izj+3kzmmCZ+vhlpZiQv27VwFCUbNvtDLViNTbHGeWR7TW7WIBR44cVdaSH8P3dSt9dk4SLTI3msbYeY5ZJw2aQreG2L2z5CcNokbnJ2dsfm5eKTge+rvcpGz1LGSGwi8M3Y= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1559916178; h=Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To:ARC-Authentication-Results; bh=zQCkCf9YBP9S+9/qukdfvBw41jcrM/ENZND2eaG4k0g=; b=OIl3WikOQnc44cKJaoMFYC4aT69w24FrOO4NXX91kk0hpz+jN/l4wvFB8mVuFGM2ZouTZC3xy5SFEMBHnqRtfZHEydaZkmsJYPHGtj9SpDN5HCNC7cXy6R347f60/1gcLVaxoCxR/PDT1h5+GHFkrmwBZ10RrUM/qI3pM26DaxY= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.47 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (209.51.188.47 [209.51.188.47]) by mx.zohomail.com with SMTPS id 1559916178536450.86463022316457; Fri, 7 Jun 2019 07:02:58 -0700 (PDT) Received: from localhost ([::1]:51466 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFSC-0008Mg-Pi for importer@patchew.org; Fri, 07 Jun 2019 10:02:48 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:45335) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hZFKy-0001xr-Gr for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:24 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hZFKt-0003ee-T1 for qemu-devel@nongnu.org; Fri, 07 Jun 2019 09:55:20 -0400 Received: from mx1.redhat.com ([209.132.183.28]:40136) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hZFKf-0003FA-O6; Fri, 07 Jun 2019 09:55:02 -0400 Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 685D1308FF30; Fri, 7 Jun 2019 13:54:58 +0000 (UTC) Received: from dhcp-200-226.str.redhat.com (dhcp-200-226.str.redhat.com [10.33.200.226]) by smtp.corp.redhat.com (Postfix) with ESMTP id 307C01001B0C; Fri, 7 Jun 2019 13:54:56 +0000 (UTC) From: Kevin Wolf To: qemu-devel@nongnu.org Date: Fri, 7 Jun 2019 15:54:30 +0200 Message-Id: <20190607135430.22149-11-kwolf@redhat.com> In-Reply-To: <20190607135430.22149-1-kwolf@redhat.com> References: <20190607135430.22149-1-kwolf@redhat.com> MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.49]); Fri, 07 Jun 2019 13:54:58 +0000 (UTC) Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.132.183.28 Subject: [Qemu-devel] [RFC PATCH 10/10] monitor: Split out monitor/core.c X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kwolf@redhat.com, armbru@redhat.com, qemu-block@nongnu.org, dgilbert@redhat.com Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Type: text/plain; charset="utf-8" Move the monitor core infrastructure from monitor/misc.c to monitor/core.c. This is code that can be shared for all targets, so compile it only once. What remains in monitor/misc.c after this patch is mostly monitor command implementations and code that requires a system emulator or is even target-dependent. The amount of function and particularly extern variables in monitor_int.h is probably a bit larger than it needs to be, but this way no non-trivial code modifications are needed. The interfaces between all monitor parts can be cleaned up later. Signed-off-by: Kevin Wolf Reviewed-by: Dr. David Alan Gilbert --- include/monitor/monitor.h | 1 + monitor/monitor_int.h | 1 + monitor/core.c | 604 ++++++++++++++++++++++++++++++++++++++ monitor/misc.c | 567 +---------------------------------- monitor/Makefile.objs | 2 +- 5 files changed, 608 insertions(+), 567 deletions(-) create mode 100644 monitor/core.c diff --git a/include/monitor/monitor.h b/include/monitor/monitor.h index 8547529e49..b9f8d175ed 100644 --- a/include/monitor/monitor.h +++ b/include/monitor/monitor.h @@ -20,6 +20,7 @@ typedef struct MonitorHMP MonitorHMP; bool monitor_cur_is_qmp(void); =20 void monitor_init_globals(void); +void monitor_init_globals_core(void); void monitor_init(Chardev *chr, int flags); void monitor_init_qmp(Chardev *chr, int flags); void monitor_init_hmp(Chardev *chr, int flags); diff --git a/monitor/monitor_int.h b/monitor/monitor_int.h index 8c5d95f942..1a7af6a223 100644 --- a/monitor/monitor_int.h +++ b/monitor/monitor_int.h @@ -190,6 +190,7 @@ extern mon_cmd_t mon_cmds[]; int monitor_puts(Monitor *mon, const char *str); void monitor_data_init(Monitor *mon, int flags, bool skip_flush, bool use_io_thread); +void monitor_data_destroy(Monitor *mon); int monitor_can_read(void *opaque); void monitor_list_append(Monitor *mon); void monitor_fdsets_cleanup(void); diff --git a/monitor/core.c b/monitor/core.c new file mode 100644 index 0000000000..1cea56054b --- /dev/null +++ b/monitor/core.c @@ -0,0 +1,604 @@ +/* + * QEMU monitor + * + * Copyright (c) 2003-2004 Fabrice Bellard + * + * Permission is hereby granted, free of charge, to any person obtaining a= copy + * of this software and associated documentation files (the "Software"), t= o deal + * in the Software without restriction, including without limitation the r= ights + * to use, copy, modify, merge, publish, distribute, sublicense, and/or se= ll + * copies of the Software, and to permit persons to whom the Software is + * furnished to do so, subject to the following conditions: + * + * The above copyright notice and this permission notice shall be included= in + * all copies or substantial portions of the Software. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS= OR + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OT= HER + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING= FROM, + * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS = IN + * THE SOFTWARE. + */ + +#include "qemu/osdep.h" +#include "monitor_int.h" + +#include "qapi/error.h" +#include "qapi/qapi-emit-events.h" +#include "qapi/qmp/qstring.h" + +#include "qemu/option.h" +#include "sysemu/qtest.h" + +#include "trace-root.h" + +/* + * To prevent flooding clients, events can be throttled. The + * throttling is calculated globally, rather than per-Monitor + * instance. + */ +typedef struct MonitorQAPIEventState { + QAPIEvent event; /* Throttling state for this event type and... */ + QDict *data; /* ... data, see qapi_event_throttle_equal() */ + QEMUTimer *timer; /* Timer for handling delayed events */ + QDict *qdict; /* Delayed event (if any) */ +} MonitorQAPIEventState; + +typedef struct { + int64_t rate; /* Minimum time (in ns) between two events */ +} MonitorQAPIEventConf; + +/* Shared monitor I/O thread */ +IOThread *mon_iothread; + +/* Bottom half to dispatch the requests received from I/O thread */ +QEMUBH *qmp_dispatcher_bh; + +/* Protects mon_list, monitor_qapi_event_state, monitor_destroyed. */ +QemuMutex monitor_lock; +static GHashTable *monitor_qapi_event_state; + +MonitorList mon_list; +int mon_refcount; +static bool monitor_destroyed; + +/** + * Is @mon is using readline? + * Note: not all HMP monitors use readline, e.g., gdbserver has a + * non-interactive HMP monitor, so readline is not used there. + */ +static inline bool monitor_uses_readline(const Monitor *mon) +{ + return mon->flags & MONITOR_USE_READLINE; +} + +static inline bool monitor_is_hmp_non_interactive(const Monitor *mon) +{ + return !monitor_is_qmp(mon) && !monitor_uses_readline(mon); +} + +static void monitor_flush_locked(Monitor *mon); + +static gboolean monitor_unblocked(GIOChannel *chan, GIOCondition cond, + void *opaque) +{ + Monitor *mon =3D opaque; + + qemu_mutex_lock(&mon->mon_lock); + mon->out_watch =3D 0; + monitor_flush_locked(mon); + qemu_mutex_unlock(&mon->mon_lock); + return FALSE; +} + +/* Caller must hold mon->mon_lock */ +static void monitor_flush_locked(Monitor *mon) +{ + int rc; + size_t len; + const char *buf; + + if (mon->skip_flush) { + return; + } + + buf =3D qstring_get_str(mon->outbuf); + len =3D qstring_get_length(mon->outbuf); + + if (len && !mon->mux_out) { + rc =3D qemu_chr_fe_write(&mon->chr, (const uint8_t *) buf, len); + if ((rc < 0 && errno !=3D EAGAIN) || (rc =3D=3D len)) { + /* all flushed or error */ + qobject_unref(mon->outbuf); + mon->outbuf =3D qstring_new(); + return; + } + if (rc > 0) { + /* partial write */ + QString *tmp =3D qstring_from_str(buf + rc); + qobject_unref(mon->outbuf); + mon->outbuf =3D tmp; + } + if (mon->out_watch =3D=3D 0) { + mon->out_watch =3D + qemu_chr_fe_add_watch(&mon->chr, G_IO_OUT | G_IO_HUP, + monitor_unblocked, mon); + } + } +} + +void monitor_flush(Monitor *mon) +{ + qemu_mutex_lock(&mon->mon_lock); + monitor_flush_locked(mon); + qemu_mutex_unlock(&mon->mon_lock); +} + +/* flush at every end of line */ +int monitor_puts(Monitor *mon, const char *str) +{ + int i; + char c; + + qemu_mutex_lock(&mon->mon_lock); + for (i =3D 0; str[i]; i++) { + c =3D str[i]; + if (c =3D=3D '\n') { + qstring_append_chr(mon->outbuf, '\r'); + } + qstring_append_chr(mon->outbuf, c); + if (c =3D=3D '\n') { + monitor_flush_locked(mon); + } + } + qemu_mutex_unlock(&mon->mon_lock); + + return i; +} + +int monitor_vprintf(Monitor *mon, const char *fmt, va_list ap) +{ + char *buf; + int n; + + if (!mon) + return -1; + + if (monitor_is_qmp(mon)) { + return -1; + } + + buf =3D g_strdup_vprintf(fmt, ap); + n =3D monitor_puts(mon, buf); + g_free(buf); + return n; +} + +int monitor_printf(Monitor *mon, const char *fmt, ...) +{ + int ret; + + va_list ap; + va_start(ap, fmt); + ret =3D monitor_vprintf(mon, fmt, ap); + va_end(ap); + return ret; +} + + +static MonitorQAPIEventConf monitor_qapi_event_conf[QAPI_EVENT__MAX] =3D { + /* Limit guest-triggerable events to 1 per second */ + [QAPI_EVENT_RTC_CHANGE] =3D { 1000 * SCALE_MS }, + [QAPI_EVENT_WATCHDOG] =3D { 1000 * SCALE_MS }, + [QAPI_EVENT_BALLOON_CHANGE] =3D { 1000 * SCALE_MS }, + [QAPI_EVENT_QUORUM_REPORT_BAD] =3D { 1000 * SCALE_MS }, + [QAPI_EVENT_QUORUM_FAILURE] =3D { 1000 * SCALE_MS }, + [QAPI_EVENT_VSERPORT_CHANGE] =3D { 1000 * SCALE_MS }, +}; + +/* + * Return the clock to use for recording an event's time. + * It's QEMU_CLOCK_REALTIME, except for qtests it's + * QEMU_CLOCK_VIRTUAL, to support testing rate limits. + * Beware: result is invalid before configure_accelerator(). + */ +static inline QEMUClockType monitor_get_event_clock(void) +{ + return qtest_enabled() ? QEMU_CLOCK_VIRTUAL : QEMU_CLOCK_REALTIME; +} + +/* + * Broadcast an event to all monitors. + * @qdict is the event object. Its member "event" must match @event. + * Caller must hold monitor_lock. + */ +static void monitor_qapi_event_emit(QAPIEvent event, QDict *qdict) +{ + Monitor *mon; + MonitorQMP *qmp_mon; + + trace_monitor_protocol_event_emit(event, qdict); + QTAILQ_FOREACH(mon, &mon_list, entry) { + if (!monitor_is_qmp(mon)) { + continue; + } + + qmp_mon =3D container_of(mon, MonitorQMP, common); + if (qmp_mon->commands !=3D &qmp_cap_negotiation_commands) { + qmp_send_response(qmp_mon, qdict); + } + } +} + +static void monitor_qapi_event_handler(void *opaque); + +/* + * Queue a new event for emission to Monitor instances, + * applying any rate limiting if required. + */ +static void +monitor_qapi_event_queue_no_reenter(QAPIEvent event, QDict *qdict) +{ + MonitorQAPIEventConf *evconf; + MonitorQAPIEventState *evstate; + + assert(event < QAPI_EVENT__MAX); + evconf =3D &monitor_qapi_event_conf[event]; + trace_monitor_protocol_event_queue(event, qdict, evconf->rate); + + qemu_mutex_lock(&monitor_lock); + + if (!evconf->rate) { + /* Unthrottled event */ + monitor_qapi_event_emit(event, qdict); + } else { + QDict *data =3D qobject_to(QDict, qdict_get(qdict, "data")); + MonitorQAPIEventState key =3D { .event =3D event, .data =3D data }; + + evstate =3D g_hash_table_lookup(monitor_qapi_event_state, &key); + assert(!evstate || timer_pending(evstate->timer)); + + if (evstate) { + /* + * Timer is pending for (at least) evconf->rate ns after + * last send. Store event for sending when timer fires, + * replacing a prior stored event if any. + */ + qobject_unref(evstate->qdict); + evstate->qdict =3D qobject_ref(qdict); + } else { + /* + * Last send was (at least) evconf->rate ns ago. + * Send immediately, and arm the timer to call + * monitor_qapi_event_handler() in evconf->rate ns. Any + * events arriving before then will be delayed until then. + */ + int64_t now =3D qemu_clock_get_ns(monitor_get_event_clock()); + + monitor_qapi_event_emit(event, qdict); + + evstate =3D g_new(MonitorQAPIEventState, 1); + evstate->event =3D event; + evstate->data =3D qobject_ref(data); + evstate->qdict =3D NULL; + evstate->timer =3D timer_new_ns(monitor_get_event_clock(), + monitor_qapi_event_handler, + evstate); + g_hash_table_add(monitor_qapi_event_state, evstate); + timer_mod_ns(evstate->timer, now + evconf->rate); + } + } + + qemu_mutex_unlock(&monitor_lock); +} + +void qapi_event_emit(QAPIEvent event, QDict *qdict) +{ + /* + * monitor_qapi_event_queue_no_reenter() is not reentrant: it + * would deadlock on monitor_lock. Work around by queueing + * events in thread-local storage. + * TODO: remove this, make it re-enter safe. + */ + typedef struct MonitorQapiEvent { + QAPIEvent event; + QDict *qdict; + QSIMPLEQ_ENTRY(MonitorQapiEvent) entry; + } MonitorQapiEvent; + static __thread QSIMPLEQ_HEAD(, MonitorQapiEvent) event_queue; + static __thread bool reentered; + MonitorQapiEvent *ev; + + if (!reentered) { + QSIMPLEQ_INIT(&event_queue); + } + + ev =3D g_new(MonitorQapiEvent, 1); + ev->qdict =3D qobject_ref(qdict); + ev->event =3D event; + QSIMPLEQ_INSERT_TAIL(&event_queue, ev, entry); + if (reentered) { + return; + } + + reentered =3D true; + + while ((ev =3D QSIMPLEQ_FIRST(&event_queue)) !=3D NULL) { + QSIMPLEQ_REMOVE_HEAD(&event_queue, entry); + monitor_qapi_event_queue_no_reenter(ev->event, ev->qdict); + qobject_unref(ev->qdict); + g_free(ev); + } + + reentered =3D false; +} + +/* + * This function runs evconf->rate ns after sending a throttled + * event. + * If another event has since been stored, send it. + */ +static void monitor_qapi_event_handler(void *opaque) +{ + MonitorQAPIEventState *evstate =3D opaque; + MonitorQAPIEventConf *evconf =3D &monitor_qapi_event_conf[evstate->eve= nt]; + + trace_monitor_protocol_event_handler(evstate->event, evstate->qdict); + qemu_mutex_lock(&monitor_lock); + + if (evstate->qdict) { + int64_t now =3D qemu_clock_get_ns(monitor_get_event_clock()); + + monitor_qapi_event_emit(evstate->event, evstate->qdict); + qobject_unref(evstate->qdict); + evstate->qdict =3D NULL; + timer_mod_ns(evstate->timer, now + evconf->rate); + } else { + g_hash_table_remove(monitor_qapi_event_state, evstate); + qobject_unref(evstate->data); + timer_free(evstate->timer); + g_free(evstate); + } + + qemu_mutex_unlock(&monitor_lock); +} + +static unsigned int qapi_event_throttle_hash(const void *key) +{ + const MonitorQAPIEventState *evstate =3D key; + unsigned int hash =3D evstate->event * 255; + + if (evstate->event =3D=3D QAPI_EVENT_VSERPORT_CHANGE) { + hash +=3D g_str_hash(qdict_get_str(evstate->data, "id")); + } + + if (evstate->event =3D=3D QAPI_EVENT_QUORUM_REPORT_BAD) { + hash +=3D g_str_hash(qdict_get_str(evstate->data, "node-name")); + } + + return hash; +} + +static gboolean qapi_event_throttle_equal(const void *a, const void *b) +{ + const MonitorQAPIEventState *eva =3D a; + const MonitorQAPIEventState *evb =3D b; + + if (eva->event !=3D evb->event) { + return FALSE; + } + + if (eva->event =3D=3D QAPI_EVENT_VSERPORT_CHANGE) { + return !strcmp(qdict_get_str(eva->data, "id"), + qdict_get_str(evb->data, "id")); + } + + if (eva->event =3D=3D QAPI_EVENT_QUORUM_REPORT_BAD) { + return !strcmp(qdict_get_str(eva->data, "node-name"), + qdict_get_str(evb->data, "node-name")); + } + + return TRUE; +} + +int monitor_suspend(Monitor *mon) +{ + if (monitor_is_hmp_non_interactive(mon)) { + return -ENOTTY; + } + + atomic_inc(&mon->suspend_cnt); + + if (mon->use_io_thread) { + /* + * Kick I/O thread to make sure this takes effect. It'll be + * evaluated again in prepare() of the watch object. + */ + aio_notify(iothread_get_aio_context(mon_iothread)); + } + + trace_monitor_suspend(mon, 1); + return 0; +} + +static void monitor_accept_input(void *opaque) +{ + Monitor *mon =3D opaque; + + qemu_chr_fe_accept_input(&mon->chr); +} + +void monitor_resume(Monitor *mon) +{ + if (monitor_is_hmp_non_interactive(mon)) { + return; + } + + if (atomic_dec_fetch(&mon->suspend_cnt) =3D=3D 0) { + AioContext *ctx; + + if (mon->use_io_thread) { + ctx =3D iothread_get_aio_context(mon_iothread); + } else { + ctx =3D qemu_get_aio_context(); + } + + if (!monitor_is_qmp(mon)) { + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); + assert(hmp_mon->rs); + readline_show_prompt(hmp_mon->rs); + } + + aio_bh_schedule_oneshot(ctx, monitor_accept_input, mon); + } + + trace_monitor_suspend(mon, -1); +} + +int monitor_can_read(void *opaque) +{ + Monitor *mon =3D opaque; + + return !atomic_mb_read(&mon->suspend_cnt); +} + +void monitor_list_append(Monitor *mon) +{ + qemu_mutex_lock(&monitor_lock); + /* + * This prevents inserting new monitors during monitor_cleanup(). + * A cleaner solution would involve the main thread telling other + * threads to terminate, waiting for their termination. + */ + if (!monitor_destroyed) { + QTAILQ_INSERT_HEAD(&mon_list, mon, entry); + mon =3D NULL; + } + qemu_mutex_unlock(&monitor_lock); + + if (mon) { + monitor_data_destroy(mon); + g_free(mon); + } +} + +static void monitor_iothread_init(void) +{ + mon_iothread =3D iothread_create("mon_iothread", &error_abort); +} + +void monitor_data_init(Monitor *mon, int flags, bool skip_flush, + bool use_io_thread) +{ + if (use_io_thread && !mon_iothread) { + monitor_iothread_init(); + } + memset(mon, 0, sizeof(Monitor)); + qemu_mutex_init(&mon->mon_lock); + mon->outbuf =3D qstring_new(); + mon->skip_flush =3D skip_flush; + mon->use_io_thread =3D use_io_thread; + mon->flags =3D flags; +} + +void monitor_data_destroy(Monitor *mon) +{ + g_free(mon->mon_cpu_path); + qemu_chr_fe_deinit(&mon->chr, false); + if (monitor_is_qmp(mon)) { + MonitorQMP *qmp_mon =3D container_of(mon, MonitorQMP, common); + monitor_data_destroy_qmp(qmp_mon); + } else { + MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); + readline_free(hmp_mon->rs); + } + qobject_unref(mon->outbuf); + qemu_mutex_destroy(&mon->mon_lock); +} + +void monitor_init(Chardev *chr, int flags) +{ + if (flags & MONITOR_USE_CONTROL) { + monitor_init_qmp(chr, flags); + } else { + monitor_init_hmp(chr, flags); + } +} + +void monitor_cleanup(void) +{ + /* + * We need to explicitly stop the I/O thread (but not destroy it), + * clean up the monitor resources, then destroy the I/O thread since + * we need to unregister from chardev below in + * monitor_data_destroy(), and chardev is not thread-safe yet + */ + if (mon_iothread) { + iothread_stop(mon_iothread); + } + + /* Flush output buffers and destroy monitors */ + qemu_mutex_lock(&monitor_lock); + monitor_destroyed =3D true; + while (!QTAILQ_EMPTY(&mon_list)) { + Monitor *mon =3D QTAILQ_FIRST(&mon_list); + QTAILQ_REMOVE(&mon_list, mon, entry); + /* Permit QAPI event emission from character frontend release */ + qemu_mutex_unlock(&monitor_lock); + monitor_flush(mon); + monitor_data_destroy(mon); + qemu_mutex_lock(&monitor_lock); + g_free(mon); + } + qemu_mutex_unlock(&monitor_lock); + + /* QEMUBHs needs to be deleted before destroying the I/O thread */ + qemu_bh_delete(qmp_dispatcher_bh); + qmp_dispatcher_bh =3D NULL; + if (mon_iothread) { + iothread_destroy(mon_iothread); + mon_iothread =3D NULL; + } +} + +static void monitor_qapi_event_init(void) +{ + monitor_qapi_event_state =3D g_hash_table_new(qapi_event_throttle_hash, + qapi_event_throttle_equal); +} + +void monitor_init_globals_core(void) +{ + monitor_qapi_event_init(); + qemu_mutex_init(&monitor_lock); + + /* + * The dispatcher BH must run in the main loop thread, since we + * have commands assuming that context. It would be nice to get + * rid of those assumptions. + */ + qmp_dispatcher_bh =3D aio_bh_new(iohandler_get_aio_context(), + monitor_qmp_bh_dispatcher, + NULL); +} + +QemuOptsList qemu_mon_opts =3D { + .name =3D "mon", + .implied_opt_name =3D "chardev", + .head =3D QTAILQ_HEAD_INITIALIZER(qemu_mon_opts.head), + .desc =3D { + { + .name =3D "mode", + .type =3D QEMU_OPT_STRING, + },{ + .name =3D "chardev", + .type =3D QEMU_OPT_STRING, + },{ + .name =3D "pretty", + .type =3D QEMU_OPT_BOOL, + }, + { /* end of list */ } + }, +}; diff --git a/monitor/misc.c b/monitor/misc.c index 408d11e1fe..1f60f31c95 100644 --- a/monitor/misc.c +++ b/monitor/misc.c @@ -117,43 +117,13 @@ struct MonFdset { QLIST_ENTRY(MonFdset) next; }; =20 -/* - * To prevent flooding clients, events can be throttled. The - * throttling is calculated globally, rather than per-Monitor - * instance. - */ -typedef struct MonitorQAPIEventState { - QAPIEvent event; /* Throttling state for this event type and... */ - QDict *data; /* ... data, see qapi_event_throttle_equal() */ - QEMUTimer *timer; /* Timer for handling delayed events */ - QDict *qdict; /* Delayed event (if any) */ -} MonitorQAPIEventState; - -typedef struct { - int64_t rate; /* Minimum time (in ns) between two events */ -} MonitorQAPIEventConf; - -/* Shared monitor I/O thread */ -IOThread *mon_iothread; - -/* Bottom half to dispatch the requests received from I/O thread */ -QEMUBH *qmp_dispatcher_bh; - /* QMP checker flags */ #define QMP_ACCEPT_UNKNOWNS 1 =20 -/* Protects mon_list, monitor_qapi_event_state, monitor_destroyed. */ -QemuMutex monitor_lock; -static GHashTable *monitor_qapi_event_state; -MonitorList mon_list; -static bool monitor_destroyed; - /* Protects mon_fdsets */ static QemuMutex mon_fdsets_lock; static QLIST_HEAD(, MonFdset) mon_fdsets; =20 -int mon_refcount; - static mon_cmd_t info_cmds[]; =20 __thread Monitor *cur_mon; @@ -161,32 +131,6 @@ __thread Monitor *cur_mon; static void monitor_command_cb(void *opaque, const char *cmdline, void *readline_opaque); =20 -/** - * Is @mon is using readline? - * Note: not all HMP monitors use readline, e.g., gdbserver has a - * non-interactive HMP monitor, so readline is not used there. - */ -static inline bool monitor_uses_readline(const Monitor *mon) -{ - return mon->flags & MONITOR_USE_READLINE; -} - -static inline bool monitor_is_hmp_non_interactive(const Monitor *mon) -{ - return !monitor_is_qmp(mon) && !monitor_uses_readline(mon); -} - -/* - * Return the clock to use for recording an event's time. - * It's QEMU_CLOCK_REALTIME, except for qtests it's - * QEMU_CLOCK_VIRTUAL, to support testing rate limits. - * Beware: result is invalid before configure_accelerator(). - */ -static inline QEMUClockType monitor_get_event_clock(void) -{ - return qtest_enabled() ? QEMU_CLOCK_VIRTUAL : QEMU_CLOCK_REALTIME; -} - /** * Is the current monitor, if any, a QMP monitor? */ @@ -220,355 +164,6 @@ int monitor_read_password(MonitorHMP *mon, ReadLineFu= nc *readline_func, } =20 =20 -static void monitor_flush_locked(Monitor *mon); - -static gboolean monitor_unblocked(GIOChannel *chan, GIOCondition cond, - void *opaque) -{ - Monitor *mon =3D opaque; - - qemu_mutex_lock(&mon->mon_lock); - mon->out_watch =3D 0; - monitor_flush_locked(mon); - qemu_mutex_unlock(&mon->mon_lock); - return FALSE; -} - -/* Caller must hold mon->mon_lock */ -static void monitor_flush_locked(Monitor *mon) -{ - int rc; - size_t len; - const char *buf; - - if (mon->skip_flush) { - return; - } - - buf =3D qstring_get_str(mon->outbuf); - len =3D qstring_get_length(mon->outbuf); - - if (len && !mon->mux_out) { - rc =3D qemu_chr_fe_write(&mon->chr, (const uint8_t *) buf, len); - if ((rc < 0 && errno !=3D EAGAIN) || (rc =3D=3D len)) { - /* all flushed or error */ - qobject_unref(mon->outbuf); - mon->outbuf =3D qstring_new(); - return; - } - if (rc > 0) { - /* partial write */ - QString *tmp =3D qstring_from_str(buf + rc); - qobject_unref(mon->outbuf); - mon->outbuf =3D tmp; - } - if (mon->out_watch =3D=3D 0) { - mon->out_watch =3D - qemu_chr_fe_add_watch(&mon->chr, G_IO_OUT | G_IO_HUP, - monitor_unblocked, mon); - } - } -} - -void monitor_flush(Monitor *mon) -{ - qemu_mutex_lock(&mon->mon_lock); - monitor_flush_locked(mon); - qemu_mutex_unlock(&mon->mon_lock); -} - -/* flush at every end of line */ -int monitor_puts(Monitor *mon, const char *str) -{ - int i; - char c; - - qemu_mutex_lock(&mon->mon_lock); - for (i =3D 0; str[i]; i++) { - c =3D str[i]; - if (c =3D=3D '\n') { - qstring_append_chr(mon->outbuf, '\r'); - } - qstring_append_chr(mon->outbuf, c); - if (c =3D=3D '\n') { - monitor_flush_locked(mon); - } - } - qemu_mutex_unlock(&mon->mon_lock); - - return i; -} - -int monitor_vprintf(Monitor *mon, const char *fmt, va_list ap) -{ - char *buf; - int n; - - if (!mon) - return -1; - - if (monitor_is_qmp(mon)) { - return -1; - } - - buf =3D g_strdup_vprintf(fmt, ap); - n =3D monitor_puts(mon, buf); - g_free(buf); - return n; -} - -int monitor_printf(Monitor *mon, const char *fmt, ...) -{ - int ret; - - va_list ap; - va_start(ap, fmt); - ret =3D monitor_vprintf(mon, fmt, ap); - va_end(ap); - return ret; -} - -static MonitorQAPIEventConf monitor_qapi_event_conf[QAPI_EVENT__MAX] =3D { - /* Limit guest-triggerable events to 1 per second */ - [QAPI_EVENT_RTC_CHANGE] =3D { 1000 * SCALE_MS }, - [QAPI_EVENT_WATCHDOG] =3D { 1000 * SCALE_MS }, - [QAPI_EVENT_BALLOON_CHANGE] =3D { 1000 * SCALE_MS }, - [QAPI_EVENT_QUORUM_REPORT_BAD] =3D { 1000 * SCALE_MS }, - [QAPI_EVENT_QUORUM_FAILURE] =3D { 1000 * SCALE_MS }, - [QAPI_EVENT_VSERPORT_CHANGE] =3D { 1000 * SCALE_MS }, -}; - -/* - * Broadcast an event to all monitors. - * @qdict is the event object. Its member "event" must match @event. - * Caller must hold monitor_lock. - */ -static void monitor_qapi_event_emit(QAPIEvent event, QDict *qdict) -{ - Monitor *mon; - MonitorQMP *qmp_mon; - - trace_monitor_protocol_event_emit(event, qdict); - QTAILQ_FOREACH(mon, &mon_list, entry) { - if (!monitor_is_qmp(mon)) { - continue; - } - - qmp_mon =3D container_of(mon, MonitorQMP, common); - if (qmp_mon->commands !=3D &qmp_cap_negotiation_commands) { - qmp_send_response(qmp_mon, qdict); - } - } -} - -static void monitor_qapi_event_handler(void *opaque); - -/* - * Queue a new event for emission to Monitor instances, - * applying any rate limiting if required. - */ -static void -monitor_qapi_event_queue_no_reenter(QAPIEvent event, QDict *qdict) -{ - MonitorQAPIEventConf *evconf; - MonitorQAPIEventState *evstate; - - assert(event < QAPI_EVENT__MAX); - evconf =3D &monitor_qapi_event_conf[event]; - trace_monitor_protocol_event_queue(event, qdict, evconf->rate); - - qemu_mutex_lock(&monitor_lock); - - if (!evconf->rate) { - /* Unthrottled event */ - monitor_qapi_event_emit(event, qdict); - } else { - QDict *data =3D qobject_to(QDict, qdict_get(qdict, "data")); - MonitorQAPIEventState key =3D { .event =3D event, .data =3D data }; - - evstate =3D g_hash_table_lookup(monitor_qapi_event_state, &key); - assert(!evstate || timer_pending(evstate->timer)); - - if (evstate) { - /* - * Timer is pending for (at least) evconf->rate ns after - * last send. Store event for sending when timer fires, - * replacing a prior stored event if any. - */ - qobject_unref(evstate->qdict); - evstate->qdict =3D qobject_ref(qdict); - } else { - /* - * Last send was (at least) evconf->rate ns ago. - * Send immediately, and arm the timer to call - * monitor_qapi_event_handler() in evconf->rate ns. Any - * events arriving before then will be delayed until then. - */ - int64_t now =3D qemu_clock_get_ns(monitor_get_event_clock()); - - monitor_qapi_event_emit(event, qdict); - - evstate =3D g_new(MonitorQAPIEventState, 1); - evstate->event =3D event; - evstate->data =3D qobject_ref(data); - evstate->qdict =3D NULL; - evstate->timer =3D timer_new_ns(monitor_get_event_clock(), - monitor_qapi_event_handler, - evstate); - g_hash_table_add(monitor_qapi_event_state, evstate); - timer_mod_ns(evstate->timer, now + evconf->rate); - } - } - - qemu_mutex_unlock(&monitor_lock); -} - -void qapi_event_emit(QAPIEvent event, QDict *qdict) -{ - /* - * monitor_qapi_event_queue_no_reenter() is not reentrant: it - * would deadlock on monitor_lock. Work around by queueing - * events in thread-local storage. - * TODO: remove this, make it re-enter safe. - */ - typedef struct MonitorQapiEvent { - QAPIEvent event; - QDict *qdict; - QSIMPLEQ_ENTRY(MonitorQapiEvent) entry; - } MonitorQapiEvent; - static __thread QSIMPLEQ_HEAD(, MonitorQapiEvent) event_queue; - static __thread bool reentered; - MonitorQapiEvent *ev; - - if (!reentered) { - QSIMPLEQ_INIT(&event_queue); - } - - ev =3D g_new(MonitorQapiEvent, 1); - ev->qdict =3D qobject_ref(qdict); - ev->event =3D event; - QSIMPLEQ_INSERT_TAIL(&event_queue, ev, entry); - if (reentered) { - return; - } - - reentered =3D true; - - while ((ev =3D QSIMPLEQ_FIRST(&event_queue)) !=3D NULL) { - QSIMPLEQ_REMOVE_HEAD(&event_queue, entry); - monitor_qapi_event_queue_no_reenter(ev->event, ev->qdict); - qobject_unref(ev->qdict); - g_free(ev); - } - - reentered =3D false; -} - -/* - * This function runs evconf->rate ns after sending a throttled - * event. - * If another event has since been stored, send it. - */ -static void monitor_qapi_event_handler(void *opaque) -{ - MonitorQAPIEventState *evstate =3D opaque; - MonitorQAPIEventConf *evconf =3D &monitor_qapi_event_conf[evstate->eve= nt]; - - trace_monitor_protocol_event_handler(evstate->event, evstate->qdict); - qemu_mutex_lock(&monitor_lock); - - if (evstate->qdict) { - int64_t now =3D qemu_clock_get_ns(monitor_get_event_clock()); - - monitor_qapi_event_emit(evstate->event, evstate->qdict); - qobject_unref(evstate->qdict); - evstate->qdict =3D NULL; - timer_mod_ns(evstate->timer, now + evconf->rate); - } else { - g_hash_table_remove(monitor_qapi_event_state, evstate); - qobject_unref(evstate->data); - timer_free(evstate->timer); - g_free(evstate); - } - - qemu_mutex_unlock(&monitor_lock); -} - -static unsigned int qapi_event_throttle_hash(const void *key) -{ - const MonitorQAPIEventState *evstate =3D key; - unsigned int hash =3D evstate->event * 255; - - if (evstate->event =3D=3D QAPI_EVENT_VSERPORT_CHANGE) { - hash +=3D g_str_hash(qdict_get_str(evstate->data, "id")); - } - - if (evstate->event =3D=3D QAPI_EVENT_QUORUM_REPORT_BAD) { - hash +=3D g_str_hash(qdict_get_str(evstate->data, "node-name")); - } - - return hash; -} - -static gboolean qapi_event_throttle_equal(const void *a, const void *b) -{ - const MonitorQAPIEventState *eva =3D a; - const MonitorQAPIEventState *evb =3D b; - - if (eva->event !=3D evb->event) { - return FALSE; - } - - if (eva->event =3D=3D QAPI_EVENT_VSERPORT_CHANGE) { - return !strcmp(qdict_get_str(eva->data, "id"), - qdict_get_str(evb->data, "id")); - } - - if (eva->event =3D=3D QAPI_EVENT_QUORUM_REPORT_BAD) { - return !strcmp(qdict_get_str(eva->data, "node-name"), - qdict_get_str(evb->data, "node-name")); - } - - return TRUE; -} - -static void monitor_qapi_event_init(void) -{ - monitor_qapi_event_state =3D g_hash_table_new(qapi_event_throttle_hash, - qapi_event_throttle_equal); -} - -static void monitor_iothread_init(void); - -void monitor_data_init(Monitor *mon, int flags, bool skip_flush, - bool use_io_thread) -{ - if (use_io_thread && !mon_iothread) { - monitor_iothread_init(); - } - memset(mon, 0, sizeof(Monitor)); - qemu_mutex_init(&mon->mon_lock); - mon->outbuf =3D qstring_new(); - mon->skip_flush =3D skip_flush; - mon->use_io_thread =3D use_io_thread; - mon->flags =3D flags; -} - -static void monitor_data_destroy(Monitor *mon) -{ - g_free(mon->mon_cpu_path); - qemu_chr_fe_deinit(&mon->chr, false); - if (monitor_is_qmp(mon)) { - MonitorQMP *qmp_mon =3D container_of(mon, MonitorQMP, common); - monitor_data_destroy_qmp(qmp_mon); - } else { - MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); - readline_free(hmp_mon->rs); - } - qobject_unref(mon->outbuf); - qemu_mutex_destroy(&mon->mon_lock); -} - char *qmp_human_monitor_command(const char *command_line, bool has_cpu_ind= ex, int64_t cpu_index, Error **errp) { @@ -2738,13 +2333,6 @@ void loadvm_completion(ReadLineState *rs, int nb_arg= s, const char *str) } } =20 -int monitor_can_read(void *opaque) -{ - Monitor *mon =3D opaque; - - return !atomic_mb_read(&mon->suspend_cnt); -} - static void monitor_command_cb(void *opaque, const char *cmdline, void *readline_opaque) { @@ -2755,60 +2343,6 @@ static void monitor_command_cb(void *opaque, const c= har *cmdline, monitor_resume(&mon->common); } =20 -int monitor_suspend(Monitor *mon) -{ - if (monitor_is_hmp_non_interactive(mon)) { - return -ENOTTY; - } - - atomic_inc(&mon->suspend_cnt); - - if (mon->use_io_thread) { - /* - * Kick I/O thread to make sure this takes effect. It'll be - * evaluated again in prepare() of the watch object. - */ - aio_notify(iothread_get_aio_context(mon_iothread)); - } - - trace_monitor_suspend(mon, 1); - return 0; -} - -static void monitor_accept_input(void *opaque) -{ - Monitor *mon =3D opaque; - - qemu_chr_fe_accept_input(&mon->chr); -} - -void monitor_resume(Monitor *mon) -{ - if (monitor_is_hmp_non_interactive(mon)) { - return; - } - - if (atomic_dec_fetch(&mon->suspend_cnt) =3D=3D 0) { - AioContext *ctx; - - if (mon->use_io_thread) { - ctx =3D iothread_get_aio_context(mon_iothread); - } else { - ctx =3D qemu_get_aio_context(); - } - - if (!monitor_is_qmp(mon)) { - MonitorHMP *hmp_mon =3D container_of(mon, MonitorHMP, common); - assert(hmp_mon->rs); - readline_show_prompt(hmp_mon->rs); - } - - aio_bh_schedule_oneshot(ctx, monitor_accept_input, mon); - } - - trace_monitor_suspend(mon, -1); -} - static int compare_mon_cmd(const void *a, const void *b) { @@ -2828,27 +2362,12 @@ static void sortcmdlist(void) qsort((void *)info_cmds, array_num, elem_size, compare_mon_cmd); } =20 -static void monitor_iothread_init(void) -{ - mon_iothread =3D iothread_create("mon_iothread", &error_abort); -} - void monitor_init_globals(void) { + monitor_init_globals_core(); monitor_init_qmp_commands(); - monitor_qapi_event_init(); sortcmdlist(); - qemu_mutex_init(&monitor_lock); qemu_mutex_init(&mon_fdsets_lock); - - /* - * The dispatcher BH must run in the main loop thread, since we - * have commands assuming that context. It would be nice to get - * rid of those assumptions. - */ - qmp_dispatcher_bh =3D aio_bh_new(iohandler_get_aio_context(), - monitor_qmp_bh_dispatcher, - NULL); } =20 /* @@ -2873,90 +2392,6 @@ int error_vprintf_unless_qmp(const char *fmt, va_lis= t ap) return -1; } =20 -void monitor_list_append(Monitor *mon) -{ - qemu_mutex_lock(&monitor_lock); - /* - * This prevents inserting new monitors during monitor_cleanup(). - * A cleaner solution would involve the main thread telling other - * threads to terminate, waiting for their termination. - */ - if (!monitor_destroyed) { - QTAILQ_INSERT_HEAD(&mon_list, mon, entry); - mon =3D NULL; - } - qemu_mutex_unlock(&monitor_lock); - - if (mon) { - monitor_data_destroy(mon); - g_free(mon); - } -} - -void monitor_init(Chardev *chr, int flags) -{ - if (flags & MONITOR_USE_CONTROL) { - monitor_init_qmp(chr, flags); - } else { - monitor_init_hmp(chr, flags); - } -} - -void monitor_cleanup(void) -{ - /* - * We need to explicitly stop the I/O thread (but not destroy it), - * clean up the monitor resources, then destroy the I/O thread since - * we need to unregister from chardev below in - * monitor_data_destroy(), and chardev is not thread-safe yet - */ - if (mon_iothread) { - iothread_stop(mon_iothread); - } - - /* Flush output buffers and destroy monitors */ - qemu_mutex_lock(&monitor_lock); - monitor_destroyed =3D true; - while (!QTAILQ_EMPTY(&mon_list)) { - Monitor *mon =3D QTAILQ_FIRST(&mon_list); - QTAILQ_REMOVE(&mon_list, mon, entry); - /* Permit QAPI event emission from character frontend release */ - qemu_mutex_unlock(&monitor_lock); - monitor_flush(mon); - monitor_data_destroy(mon); - qemu_mutex_lock(&monitor_lock); - g_free(mon); - } - qemu_mutex_unlock(&monitor_lock); - - /* QEMUBHs needs to be deleted before destroying the I/O thread */ - qemu_bh_delete(qmp_dispatcher_bh); - qmp_dispatcher_bh =3D NULL; - if (mon_iothread) { - iothread_destroy(mon_iothread); - mon_iothread =3D NULL; - } -} - -QemuOptsList qemu_mon_opts =3D { - .name =3D "mon", - .implied_opt_name =3D "chardev", - .head =3D QTAILQ_HEAD_INITIALIZER(qemu_mon_opts.head), - .desc =3D { - { - .name =3D "mode", - .type =3D QEMU_OPT_STRING, - },{ - .name =3D "chardev", - .type =3D QEMU_OPT_STRING, - },{ - .name =3D "pretty", - .type =3D QEMU_OPT_BOOL, - }, - { /* end of list */ } - }, -}; - HotpluggableCPUList *qmp_query_hotpluggable_cpus(Error **errp) { MachineState *ms =3D MACHINE(qdev_get_machine()); diff --git a/monitor/Makefile.objs b/monitor/Makefile.objs index 48c73eed51..c8dff5e4b5 100644 --- a/monitor/Makefile.objs +++ b/monitor/Makefile.objs @@ -1,2 +1,2 @@ obj-y +=3D misc.o -common-obj-y +=3D qmp.o hmp.o +common-obj-y +=3D core.o qmp.o hmp.o --=20 2.20.1