Re: [Qemu-devel] [PATCH v2 12/32] qmp: Redo how the client requests out-of-band execution

[Eric Blake]

On 07/03/2018 03:53 AM, Markus Armbruster wrote:
> Commit cf869d53172 "qmp: support out-of-band (oob) execution" added a
> general mechanism for command-independent arguments just for an
> out-of-band flag:
>
>      The "control" key is introduced to store this extra flag.  "control"
>      field is used to store arguments that are shared by all the commands,
>      rather than command specific arguments.  Let "run-oob" be the first.
>
> However, it failed to reject unknown members of "control".  For
> instance, in QMP command
>
>      {"execute": "query-name", "id": 42, "control": {"crap": true}}
>
> "crap" gets silently ignored.
>
> Instead of fixing this, revert the general "control" mechanism
> (because YAGNI), and do it the way I initially proposed, with key
> "exec-oob".  Simpler code, simper interface.

s/simper/simpler/

> An out-of-band command
>
>      {"execute": "migrate-pause", "id": 42, "control": {"run-oob": true}}
>
> becomes
>
>      {"exec-oob": "migrate-pause", "id": 42}
>
> Signed-off-by: Markus Armbruster <address@hidden>
> Reviewed-by: Eric Blake <address@hidden>
> ---

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org