[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC v6 11/27] qmp: introduce QMPCapability
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [RFC v6 11/27] qmp: introduce QMPCapability |
Date: |
Thu, 21 Dec 2017 17:56:37 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Tue, 12/19 16:45, Peter Xu wrote:
> diff --git a/monitor.c b/monitor.c
> index 4b2bee773f..81fb0a42b4 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -3943,12 +3943,18 @@ void monitor_resume(Monitor *mon)
>
> static QObject *get_qmp_greeting(void)
> {
> + QList *cap_list = qlist_new();
> QObject *ver = NULL;
> + QMPCapability cap;
>
> qmp_marshal_query_version(NULL, &ver, NULL);
>
> - return qobject_from_jsonf("{'QMP': {'version': %p, 'capabilities': []}}",
> - ver);
> + for (cap = 0; cap < QMP_CAPABILITY__MAX; cap++) {
> + qlist_append(cap_list, qstring_from_str(QMPCapability_str(cap)));
Did we want to not include "oob" if current monitor doesn't have use_io_thr?
Fam
- [Qemu-devel] [RFC v6 06/27] monitor: move the cur_mon hack deeper for QMP, (continued)
- [Qemu-devel] [RFC v6 06/27] monitor: move the cur_mon hack deeper for QMP, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 07/27] monitor: unify global init, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 08/27] monitor: let mon_list be tail queue, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 09/27] monitor: create monitor dedicate iothread, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 10/27] monitor: allow to use IO thread for parsing, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 11/27] qmp: introduce QMPCapability, Peter Xu, 2017/12/19
- Re: [Qemu-devel] [RFC v6 11/27] qmp: introduce QMPCapability,
Fam Zheng <=
- [Qemu-devel] [RFC v6 12/27] qmp: negotiate QMP capabilities, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 13/27] monitor: introduce monitor_qmp_respond(), Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 14/27] monitor: let suspend_cnt be thread safe, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 15/27] monitor: let suspend/resume work even with QMPs, Peter Xu, 2017/12/19
- [Qemu-devel] [RFC v6 16/27] monitor: separate QMP parser and dispatcher, Peter Xu, 2017/12/19