Hi On Fri, Mar 13, 2020 at 7:42 PM Philippe Mathieu-Daudé <phi...@redhat.com> wrote: > > Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com>
Without looking at the rest of the series, I fail to see the improvement, quite the opposite. A bit of context? > --- > qapi/char.json | 32 ++++++++++++++++++++++++++++++++ > qapi/misc.json | 32 -------------------------------- > monitor/qmp-cmds.c | 1 + > 3 files changed, 33 insertions(+), 32 deletions(-) > > diff --git a/qapi/char.json b/qapi/char.json > index 6907b2bfdb..8b7baf11eb 100644 > --- a/qapi/char.json > +++ b/qapi/char.json > @@ -572,3 +572,35 @@ > { 'event': 'VSERPORT_CHANGE', > 'data': { 'id': 'str', > 'open': 'bool' } } > + > +## > +# @add_client: > +# > +# Allow client connections for VNC, Spice and socket based > +# character devices to be passed in to QEMU via SCM_RIGHTS. > +# > +# @protocol: protocol name. Valid names are "vnc", "spice" or the > +# name of a character device (eg. from -chardev id=XXXX) > +# > +# @fdname: file descriptor name previously passed via 'getfd' command > +# > +# @skipauth: whether to skip authentication. Only applies > +# to "vnc" and "spice" protocols > +# > +# @tls: whether to perform TLS. Only applies to the "spice" > +# protocol > +# > +# Returns: nothing on success. > +# > +# Since: 0.14.0 > +# > +# Example: > +# > +# -> { "execute": "add_client", "arguments": { "protocol": "vnc", > +# "fdname": "myclient" } } > +# <- { "return": {} } > +# > +## > +{ 'command': 'add_client', > + 'data': { 'protocol': 'str', 'fdname': 'str', '*skipauth': 'bool', > + '*tls': 'bool' } } > diff --git a/qapi/misc.json b/qapi/misc.json > index c18fe681fb..e84e6823e9 100644 > --- a/qapi/misc.json > +++ b/qapi/misc.json > @@ -39,38 +39,6 @@ > { 'enum': 'LostTickPolicy', > 'data': ['discard', 'delay', 'slew' ] } > > -## > -# @add_client: > -# > -# Allow client connections for VNC, Spice and socket based > -# character devices to be passed in to QEMU via SCM_RIGHTS. > -# > -# @protocol: protocol name. Valid names are "vnc", "spice" or the > -# name of a character device (eg. from -chardev id=XXXX) > -# > -# @fdname: file descriptor name previously passed via 'getfd' command > -# > -# @skipauth: whether to skip authentication. Only applies > -# to "vnc" and "spice" protocols > -# > -# @tls: whether to perform TLS. Only applies to the "spice" > -# protocol > -# > -# Returns: nothing on success. > -# > -# Since: 0.14.0 > -# > -# Example: > -# > -# -> { "execute": "add_client", "arguments": { "protocol": "vnc", > -# "fdname": "myclient" } } > -# <- { "return": {} } > -# > -## > -{ 'command': 'add_client', > - 'data': { 'protocol': 'str', 'fdname': 'str', '*skipauth': 'bool', > - '*tls': 'bool' } } > - > ## > # @NameInfo: > # > diff --git a/monitor/qmp-cmds.c b/monitor/qmp-cmds.c > index 864cbfa32e..67d95b4af7 100644 > --- a/monitor/qmp-cmds.c > +++ b/monitor/qmp-cmds.c > @@ -31,6 +31,7 @@ > #include "sysemu/block-backend.h" > #include "qapi/error.h" > #include "qapi/qapi-commands-block.h" > +#include "qapi/qapi-commands-char.h" > #include "qapi/qapi-commands-control.h" > #include "qapi/qapi-commands-machine.h" > #include "qapi/qapi-commands-misc.h" > -- > 2.21.1 >