[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v4 30/32] qapi: New QMP command query-schema
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH RFC v4 30/32] qapi: New QMP command query-schema for QMP schema introspection |
Date: |
Fri, 04 Sep 2015 09:19:16 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Michael Roth <address@hidden> writes:
> Quoting Eric Blake (2015-09-03 15:50:16)
>> On 09/03/2015 08:30 AM, Markus Armbruster wrote:
>> > qapi/introspect.json defines the introspection schema. It's designed
>> > for QMP introspection, but should do for similar uses, such as QGA.
>>
>> [review to follow in separate message; I'm using this message to focus
>> on one point for easier tracking of the sub-thread]
>>
>> >
>> > The introspection schema does not reflect all the rules and
>> > restrictions that apply to QAPI schemata. A valid QAPI schema has an
>> > introspection value conforming to the introspection schema, but the
>> > converse is not true.
>> >
>> > Introspection lowers away a number of schema details, and makes
>> > implicit things explicit:
>> >
>> > * The built-in types are declared with their JSON type.
>> >
>> > TODO Should we map all the integer types to just int?
>>
>> So, given the discussion on v3, I think we will have consensus if v5
>> does the following:
>>
>> - Merge 30 and 31 into a single patch, and drop this TODO statement (I
>> think we have agreement that exposing QMP [that is, JSON number] wire
>> form, with no additional constraints, as a single 'int', rather than the
>> underlying generated C types, is the way to go), provided that...
>> - Document in the qapi side that the schema intentionally lacks details
>> on ranges, and that consulting the qapi and/or C code may be required to
>> see what actual values are valid anywhere introspection merely says 'int',
Please review docs/qapi-code-gen.txt in this patch for this aspect.
>> - leave 32 as a separate patch, as it is complex enough, and still may
>> have debate on whether the types '123' and 'int' should have
>> corresponding array types '[123]' and '[int]' rather than the v3 patch
>> munging of '456' and '789' (I haven't yet reviewed whether v4 changed that).
>
> Agreed on all 3
Excellent.
>> Any further arguments on whether exposing just 'int' in the
>> introspection for all integral types, and/or whether patches 30 and 31
>> should be merged, are best made in response to this mail.
>
> Given the above, I don't think I have any further objections.
Thanks!
- Re: [Qemu-devel] [PATCH RFC v4 32/32] qapi-introspect: Hide type names, (continued)
- [Qemu-devel] [PATCH RFC v4 27/32] qom: Don't use 'gen': false for qom-get, qom-set, object-add, Markus Armbruster, 2015/09/03
- [Qemu-devel] [PATCH RFC v4 22/32] qapi: Rename qmp_marshal_input_FOO() to qmp_marshal_FOO(), Markus Armbruster, 2015/09/03
- [Qemu-devel] [PATCH RFC v4 29/32] qapi: Pseudo-type '**' is now unused, drop it, Markus Armbruster, 2015/09/03
- [Qemu-devel] [PATCH RFC v4 30/32] qapi: New QMP command query-schema for QMP schema introspection, Markus Armbruster, 2015/09/03
- Re: [Qemu-devel] [PATCH RFC v4 30/32] qapi: New QMP command query-schema for QMP schema introspection, Michael Roth, 2015/09/03
- Re: [Qemu-devel] [PATCH RFC v4 30/32] qapi: New QMP command query-schema for QMP schema introspection, Eric Blake, 2015/09/03
[Qemu-devel] [PATCH RFC v4 19/32] qapi: Clean up after recent conversions to QAPISchemaVisitor, Markus Armbruster, 2015/09/03
Re: [Qemu-devel] [PATCH RFC v4 00/32] qapi: QMP introspection, Marc-André Lureau, 2015/09/04