QAPI patches for 2019-01-24
-----BEGIN PGP SIGNATURE-----
iQIcBAABAgAGBQJcScrPAAoJEDhwtADrkYZTONEP/jbAXZGv1N4HM/0oTtVrLWCo
tNshxN6GOmSLov6DXeJoaPQTVmDSBdFQ5tprCdOjeJ3YxCVZtWea5l+IB5UMG/IL
/wfiOH7ajJV7OxwoA3Ts6T4gX6ypgRm0lss0xQff7/RIbmltounYwvHDHH410K+o
UVed5sSzTDmmSvpin5GaJBdVhRPIW7iyiuTIVypF2u4On5VqCttx8P5b2AKd5yON
JbftwsFCjsXq/rzPA7itGqYe/yPhaoMI1mcLQxSKqfnwilpXZriTwm/YE9a1rwYU
pwNwX/Db9GwOxQ78PZWeP/m+jpe6KHltyZ+H0KZQN5HsIfdqKHpuQ7xibT36hxPk
ikDKIptbp66nAGKk8PL3i5ANEumLP3OgNTc8aoakqIb5pAu2wnf8lFS340KNmzj3
xxzv6SEPqYt/ycDCW6HRCkunXaYIU9dsWi8HDNVij+y14IENi0LwzhsrbDlfljTG
thXXAylwfY5HkFKUWVpVb7sAPRCYyMUNBYbUiwJkwLzF8gkaT0vXsNTTJ+mzYUFk
SqHHAmUml9Vkz3KCTiDJGrsUMdLxJKHrLjTabk6ACFJ5iur+Cqv8h/nmJjK6JmpU
9OrTc0CYQnTTPpDVSwDc3VqYKzKRHK/DUkLTTvA05avVuFiUDCJMMfrLJCrtel88
f3648CQG/utzIr8NruLV
=Oima
-----END PGP SIGNATURE-----
Merge remote-tracking branch 'remotes/armbru/tags/pull-qapi-2019-01-24' into staging
QAPI patches for 2019-01-24
# gpg: Signature made Thu 24 Jan 2019 14:25:19 GMT
# gpg: using RSA key 3870B400EB918653
# gpg: Good signature from "Markus Armbruster <armbru@redhat.com>"
# gpg: aka "Markus Armbruster <armbru@pond.sub.org>"
# Primary key fingerprint: 354B C8B3 D7EB 2A6B 6867 4E5F 3870 B400 EB91 8653
* remotes/armbru/tags/pull-qapi-2019-01-24:
json: Fix % handling when not interpolating
qmp: Add examples to qom list, get, and set commands
qapi: Eliminate indirection through qmp_event_get_func_emit()
qapi: Belatedly update docs for commit 9c2f56e9f9
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
Peter Maydell
commit
9dd0d8111f
@ -1369,8 +1369,8 @@ Example:
|
||||
void qapi_event_send_my_event(void);
|
||||
|
||||
typedef enum example_QAPIEvent {
|
||||
EXAMPLE_QAPI_EVENT_MY_EVENT = 0,
|
||||
EXAMPLE_QAPI_EVENT__MAX = 1,
|
||||
EXAMPLE_QAPI_EVENT_MY_EVENT,
|
||||
EXAMPLE_QAPI_EVENT__MAX,
|
||||
} example_QAPIEvent;
|
||||
|
||||
#define example_QAPIEvent_str(val) \
|
||||
@ -1385,16 +1385,10 @@ Example:
|
||||
void qapi_event_send_my_event(void)
|
||||
{
|
||||
QDict *qmp;
|
||||
QMPEventFuncEmit emit;
|
||||
|
||||
emit = qmp_event_get_func_emit();
|
||||
if (!emit) {
|
||||
return;
|
||||
}
|
||||
|
||||
qmp = qmp_event_build_dict("MY_EVENT");
|
||||
|
||||
emit(EXAMPLE_QAPI_EVENT_MY_EVENT, qmp);
|
||||
example_qapi_event_emit(EXAMPLE_QAPI_EVENT_MY_EVENT, qmp);
|
||||
|
||||
qobject_unref(qmp);
|
||||
}
|
||||
|
||||
@ -14,11 +14,5 @@
|
||||
#ifndef QMP_EVENT_H
|
||||
#define QMP_EVENT_H
|
||||
|
||||
typedef void (*QMPEventFuncEmit)(unsigned event, QDict *dict);
|
||||
|
||||
void qmp_event_set_func_emit(QMPEventFuncEmit emit);
|
||||
|
||||
QMPEventFuncEmit qmp_event_get_func_emit(void);
|
||||
|
||||
QDict *qmp_event_build_dict(const char *event_name);
|
||||
#endif
|
||||
|
||||
@ -590,8 +590,7 @@ monitor_qapi_event_queue_no_reenter(QAPIEvent event, QDict *qdict)
|
||||
qemu_mutex_unlock(&monitor_lock);
|
||||
}
|
||||
|
||||
static void
|
||||
monitor_qapi_event_queue(QAPIEvent event, QDict *qdict)
|
||||
void qapi_event_emit(QAPIEvent event, QDict *qdict)
|
||||
{
|
||||
/*
|
||||
* monitor_qapi_event_queue_no_reenter() is not reentrant: it
|
||||
@ -704,7 +703,6 @@ static void monitor_qapi_event_init(void)
|
||||
{
|
||||
monitor_qapi_event_state = g_hash_table_new(qapi_event_throttle_hash,
|
||||
qapi_event_throttle_equal);
|
||||
qmp_event_set_func_emit(monitor_qapi_event_queue);
|
||||
}
|
||||
|
||||
static void handle_hmp_command(Monitor *mon, const char *cmdline);
|
||||
|
||||
@ -1380,6 +1380,16 @@
|
||||
# object.
|
||||
#
|
||||
# Since: 1.2
|
||||
#
|
||||
# Example:
|
||||
#
|
||||
# -> { "execute": "qom-list",
|
||||
# "arguments": { "path": "/chardevs" } }
|
||||
# <- { "return": [ { "name": "type", "type": "string" },
|
||||
# { "name": "parallel0", "type": "child<chardev-vc>" },
|
||||
# { "name": "serial0", "type": "child<chardev-vc>" },
|
||||
# { "name": "mon0", "type": "child<chardev-stdio>" } ] }
|
||||
#
|
||||
##
|
||||
{ 'command': 'qom-list',
|
||||
'data': { 'path': 'str' },
|
||||
@ -1417,6 +1427,23 @@
|
||||
# returned as #int.
|
||||
#
|
||||
# Since: 1.2
|
||||
#
|
||||
# Example:
|
||||
#
|
||||
# 1. Use absolute path
|
||||
#
|
||||
# -> { "execute": "qom-get",
|
||||
# "arguments": { "path": "/machine/unattached/device[0]",
|
||||
# "property": "hotplugged" } }
|
||||
# <- { "return": false }
|
||||
#
|
||||
# 2. Use partial path
|
||||
#
|
||||
# -> { "execute": "qom-get",
|
||||
# "arguments": { "path": "unattached/sysbus",
|
||||
# "property": "type" } }
|
||||
# <- { "return": "System" }
|
||||
#
|
||||
##
|
||||
{ 'command': 'qom-get',
|
||||
'data': { 'path': 'str', 'property': 'str' },
|
||||
@ -1436,6 +1463,15 @@
|
||||
# for a description of type mapping.
|
||||
#
|
||||
# Since: 1.2
|
||||
#
|
||||
# Example:
|
||||
#
|
||||
# -> { "execute": "qom-set",
|
||||
# "arguments": { "path": "/machine",
|
||||
# "property": "graphics",
|
||||
# "value": false } }
|
||||
# <- { "return": {} }
|
||||
#
|
||||
##
|
||||
{ 'command': 'qom-set',
|
||||
'data': { 'path': 'str', 'property': 'str', 'value': 'any' },
|
||||
|
||||
@ -19,18 +19,6 @@
|
||||
#include "qapi/qmp/qdict.h"
|
||||
#include "qapi/qmp/qjson.h"
|
||||
|
||||
static QMPEventFuncEmit qmp_emit;
|
||||
|
||||
void qmp_event_set_func_emit(QMPEventFuncEmit emit)
|
||||
{
|
||||
qmp_emit = emit;
|
||||
}
|
||||
|
||||
QMPEventFuncEmit qmp_event_get_func_emit(void)
|
||||
{
|
||||
return qmp_emit;
|
||||
}
|
||||
|
||||
static void timestamp_put(QDict *qdict)
|
||||
{
|
||||
int err;
|
||||
|
||||
@ -208,11 +208,13 @@ static QString *parse_string(JSONParserContext *ctxt, JSONToken *token)
|
||||
}
|
||||
break;
|
||||
case '%':
|
||||
if (ctxt->ap && ptr[1] != '%') {
|
||||
parse_error(ctxt, token, "can't interpolate into string");
|
||||
goto out;
|
||||
if (ctxt->ap) {
|
||||
if (ptr[1] != '%') {
|
||||
parse_error(ctxt, token, "can't interpolate into string");
|
||||
goto out;
|
||||
}
|
||||
ptr++;
|
||||
}
|
||||
ptr++;
|
||||
/* fall through */
|
||||
default:
|
||||
cp = mod_utf8_codepoint(ptr, 6, &end);
|
||||
|
||||
@ -58,7 +58,7 @@ def gen_param_var(typ):
|
||||
return ret
|
||||
|
||||
|
||||
def gen_event_send(name, arg_type, boxed, event_enum_name):
|
||||
def gen_event_send(name, arg_type, boxed, event_enum_name, event_emit):
|
||||
# FIXME: Our declaration of local variables (and of 'errp' in the
|
||||
# parameter list) can collide with exploded members of the event's
|
||||
# data type passed in as parameters. If this collision ever hits in
|
||||
@ -70,7 +70,6 @@ def gen_event_send(name, arg_type, boxed, event_enum_name):
|
||||
%(proto)s
|
||||
{
|
||||
QDict *qmp;
|
||||
QMPEventFuncEmit emit;
|
||||
''',
|
||||
proto=build_event_send_proto(name, arg_type, boxed))
|
||||
|
||||
@ -86,11 +85,6 @@ def gen_event_send(name, arg_type, boxed, event_enum_name):
|
||||
|
||||
ret += mcgen('''
|
||||
|
||||
emit = qmp_event_get_func_emit();
|
||||
if (!emit) {
|
||||
return;
|
||||
}
|
||||
|
||||
qmp = qmp_event_build_dict("%(name)s");
|
||||
|
||||
''',
|
||||
@ -121,9 +115,10 @@ def gen_event_send(name, arg_type, boxed, event_enum_name):
|
||||
''')
|
||||
|
||||
ret += mcgen('''
|
||||
emit(%(c_enum)s, qmp);
|
||||
%(event_emit)s(%(c_enum)s, qmp);
|
||||
|
||||
''',
|
||||
event_emit=event_emit,
|
||||
c_enum=c_enum_const(event_enum_name, name))
|
||||
|
||||
if arg_type and not arg_type.is_empty():
|
||||
@ -145,6 +140,7 @@ class QAPISchemaGenEventVisitor(QAPISchemaModularCVisitor):
|
||||
' * Schema-defined QAPI/QMP events', __doc__)
|
||||
self._event_enum_name = c_name(prefix + 'QAPIEvent', protect=False)
|
||||
self._event_enum_members = []
|
||||
self._event_emit_name = c_name(prefix + 'qapi_event_emit')
|
||||
|
||||
def _begin_module(self, name):
|
||||
types = self._module_basename('qapi-types', name)
|
||||
@ -170,15 +166,23 @@ class QAPISchemaGenEventVisitor(QAPISchemaModularCVisitor):
|
||||
|
||||
def visit_end(self):
|
||||
(genc, genh) = self._module[self._main_module]
|
||||
genh.add(gen_enum(self._event_enum_name, self._event_enum_members))
|
||||
genh.add(gen_enum(self._event_enum_name,
|
||||
self._event_enum_members))
|
||||
genc.add(gen_enum_lookup(self._event_enum_name,
|
||||
self._event_enum_members))
|
||||
genh.add(mcgen('''
|
||||
|
||||
void %(event_emit)s(%(event_enum)s event, QDict *qdict);
|
||||
''',
|
||||
event_emit=self._event_emit_name,
|
||||
event_enum=self._event_enum_name))
|
||||
|
||||
def visit_event(self, name, info, ifcond, arg_type, boxed):
|
||||
with ifcontext(ifcond, self._genh, self._genc):
|
||||
self._genh.add(gen_event_send_decl(name, arg_type, boxed))
|
||||
self._genc.add(gen_event_send(name, arg_type, boxed,
|
||||
self._event_enum_name))
|
||||
self._event_enum_name,
|
||||
self._event_emit_name))
|
||||
self._event_enum_members.append(QAPISchemaMember(name, ifcond))
|
||||
|
||||
|
||||
|
||||
@ -1,5 +1,6 @@
|
||||
#include "qemu/osdep.h"
|
||||
#include "qapi/error.h"
|
||||
#include "qapi/qapi-events.h"
|
||||
#include "qemu-common.h"
|
||||
#include "monitor/monitor.h"
|
||||
|
||||
@ -14,3 +15,7 @@ int monitor_get_fd(Monitor *mon, const char *name, Error **errp)
|
||||
void monitor_init(Chardev *chr, int flags)
|
||||
{
|
||||
}
|
||||
|
||||
void qapi_event_emit(QAPIEvent event, QDict *qdict)
|
||||
{
|
||||
}
|
||||
|
||||
@ -529,7 +529,7 @@ QEMU_CFLAGS += -I$(SRC_PATH)/tests
|
||||
test-util-obj-y = libqemuutil.a
|
||||
test-qom-obj-y = $(qom-obj-y) $(test-util-obj-y)
|
||||
test-qapi-obj-y = tests/test-qapi-visit.o tests/test-qapi-types.o \
|
||||
tests/test-qapi-events.o tests/test-qapi-introspect.o \
|
||||
tests/test-qapi-introspect.o \
|
||||
$(test-qom-obj-y)
|
||||
benchmark-crypto-obj-y = $(crypto-obj-y) $(test-qom-obj-y)
|
||||
test-crypto-obj-y = $(crypto-obj-y) $(test-qom-obj-y)
|
||||
@ -621,7 +621,7 @@ tests/qapi-schema/doc-good.test.texi: $(SRC_PATH)/tests/qapi-schema/doc-good.jso
|
||||
|
||||
tests/test-string-output-visitor$(EXESUF): tests/test-string-output-visitor.o $(test-qapi-obj-y)
|
||||
tests/test-string-input-visitor$(EXESUF): tests/test-string-input-visitor.o $(test-qapi-obj-y)
|
||||
tests/test-qmp-event$(EXESUF): tests/test-qmp-event.o $(test-qapi-obj-y)
|
||||
tests/test-qmp-event$(EXESUF): tests/test-qmp-event.o $(test-qapi-obj-y) tests/test-qapi-events.o
|
||||
tests/test-qobject-output-visitor$(EXESUF): tests/test-qobject-output-visitor.o $(test-qapi-obj-y)
|
||||
tests/test-clone-visitor$(EXESUF): tests/test-clone-visitor.o $(test-qapi-obj-y)
|
||||
tests/test-qobject-input-visitor$(EXESUF): tests/test-qobject-input-visitor.o $(test-qapi-obj-y)
|
||||
|
||||
@ -175,6 +175,11 @@ static void utf8_string(void)
|
||||
"\xCE\xBA\xE1\xBD\xB9\xCF\x83\xCE\xBC\xCE\xB5",
|
||||
"\xCE\xBA\xE1\xBD\xB9\xCF\x83\xCE\xBC\xCE\xB5",
|
||||
"\\u03BA\\u1F79\\u03C3\\u03BC\\u03B5",
|
||||
},
|
||||
/* '%' character when not interpolating */
|
||||
{
|
||||
"100%",
|
||||
"100%",
|
||||
},
|
||||
/* 2 Boundary condition test cases */
|
||||
/* 2.1 First possible sequence of a certain length */
|
||||
|
||||
@ -93,9 +93,7 @@ static bool qdict_cmp_simple(QDict *a, QDict *b)
|
||||
return d.result;
|
||||
}
|
||||
|
||||
/* This function is hooked as final emit function, which can verify the
|
||||
correctness. */
|
||||
static void event_test_emit(test_QAPIEvent event, QDict *d)
|
||||
void test_qapi_event_emit(test_QAPIEvent event, QDict *d)
|
||||
{
|
||||
QDict *t;
|
||||
int64_t s, ms;
|
||||
@ -241,8 +239,6 @@ static void test_event_d(TestEventData *data,
|
||||
|
||||
int main(int argc, char **argv)
|
||||
{
|
||||
qmp_event_set_func_emit(event_test_emit);
|
||||
|
||||
g_test_init(&argc, &argv, NULL);
|
||||
|
||||
event_test_add("/event/event_a", test_event_a);
|
||||
|
||||
Loading…
Reference in New Issue
Block a user