Re: [Qemu-devel] [PATCH 56/56] docs/interop/qmp-spec: How to force known good parser state

2018-08-17 Thread Eric Blake
On 08/17/2018 03:37 AM, Markus Armbruster wrote: Perhaps worth documenting is the range of JSON numbers produced by qemu (maybe as a separate patch). Libvirt just hit a bug with the jansson library making it extremely difficult to parse JSON containing numbers larger than INT64_MAX, when compare

Re: [Qemu-devel] [PATCH 56/56] docs/interop/qmp-spec: How to force known good parser state

2018-08-17 Thread Eric Blake
On 08/17/2018 06:16 AM, Markus Armbruster wrote: +2.7 QGA Synchronization --- When using QGA, an additional synchronization feature is built into -the protocol. If the Client sends a raw 0xFF sentinel byte (not valid -JSON), then the Server will reset its state and d

Re: [Qemu-devel] [PATCH 56/56] docs/interop/qmp-spec: How to force known good parser state

2018-08-17 Thread Markus Armbruster
Eric Blake writes: > On 08/08/2018 07:03 AM, Markus Armbruster wrote: >> Section "QGA Synchronization" specifies that sending "a raw 0xFF >> sentinel byte" makes the server "reset its state and discard all >> pending data prior to the sentinel." What actually happens there is a >> lexical error,

Re: [Qemu-devel] [PATCH 56/56] docs/interop/qmp-spec: How to force known good parser state

2018-08-17 Thread Markus Armbruster
Eric Blake writes: > On 08/08/2018 07:03 AM, Markus Armbruster wrote: >> Section "QGA Synchronization" specifies that sending "a raw 0xFF >> sentinel byte" makes the server "reset its state and discard all >> pending data prior to the sentinel." What actually happens there is a >> lexical error,

Re: [Qemu-devel] [PATCH 56/56] docs/interop/qmp-spec: How to force known good parser state

2018-08-10 Thread Eric Blake
On 08/08/2018 07:03 AM, Markus Armbruster wrote: Section "QGA Synchronization" specifies that sending "a raw 0xFF sentinel byte" makes the server "reset its state and discard all pending data prior to the sentinel." What actually happens there is a lexical error, which will produce one ore more

[Qemu-devel] [PATCH 56/56] docs/interop/qmp-spec: How to force known good parser state

2018-08-08 Thread Markus Armbruster
Section "QGA Synchronization" specifies that sending "a raw 0xFF sentinel byte" makes the server "reset its state and discard all pending data prior to the sentinel." What actually happens there is a lexical error, which will produce one ore more error responses. Moreover, it's not specific to QGA