[PATCH 0/2] qapi: Doc & MAINTAINERS tweaks

[PATCH 0/2] qapi: Doc & MAINTAINERS tweaks

[Markus Armbruster]

Markus Armbruster (2):
  MAINTAINERS: Add QAPI schema modules to their subsystems
  qapi: Fix missing headers in QMP Reference Manual

 qapi/audio.json | 4 ++++
 qapi/authz.json | 6 ++++--
 MAINTAINERS     | 6 ++++++
 3 files changed, 14 insertions(+), 2 deletions(-)

-- 
2.26.2

[PATCH 1/2] MAINTAINERS: Add QAPI schema modules to their subsystems

[Markus Armbruster]

Add the relevant QAPI schema modules to section Audio, QMP, Tracing,
Cryptography.

Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Stefan Hajnoczi <stefanha@redhat.com>
Cc: Daniel P. Berrange <berrange@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 MAINTAINERS | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8c744a9bdf..b4b6b36542 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1907,6 +1907,7 @@ Rocker
 M: Jiri Pirko <jiri@resnulli.us>
 S: Maintained
 F: hw/net/rocker/
+F: qapi/rocker.json
 F: tests/rocker/
 F: docs/specs/rocker.txt
 
@@ -2109,6 +2110,7 @@ S: Maintained
 F: audio/
 F: hw/audio/
 F: include/hw/audio/
+F: qapi/audio.json
 F: tests/qtest/ac97-test.c
 F: tests/qtest/es1370-test.c
 F: tests/qtest/intel-hda-test.c
@@ -2488,7 +2490,9 @@ F: monitor/monitor-internal.h
 F: monitor/qmp*
 F: monitor/misc.c
 F: monitor/monitor.c
+F: qapi/control.json
 F: qapi/error.json
+F: qapi/introspect.json
 F: docs/devel/*qmp-*
 F: docs/interop/*qmp-*
 F: scripts/qmp/
@@ -2549,6 +2553,7 @@ S: Maintained
 F: trace/
 F: trace-events
 F: docs/qemu-option-trace.rst.inc
+F: qapi/trace.json
 F: scripts/tracetool.py
 F: scripts/tracetool/
 F: scripts/qemu-trace-stap*
@@ -2608,6 +2613,7 @@ M: Daniel P. Berrange <berrange@redhat.com>
 S: Maintained
 F: crypto/
 F: include/crypto/
+F: qapi/crypto.json
 F: tests/test-crypto-*
 F: tests/benchmark-crypto-*
 F: tests/crypto-tls-*
-- 
2.26.2

Re: [PATCH 1/2] MAINTAINERS: Add QAPI schema modules to their subsystems

[Philippe Mathieu-Daudé]

On 11/2/20 9:15 AM, Markus Armbruster wrote:
> Add the relevant QAPI schema modules to section Audio, QMP, Tracing,
> Cryptography.
> 
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Stefan Hajnoczi <stefanha@redhat.com>
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  MAINTAINERS | 6 ++++++
>  1 file changed, 6 insertions(+)

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Re: [PATCH 1/2] MAINTAINERS: Add QAPI schema modules to their subsystems

[Stefan Hajnoczi]

[-- Attachment #1: Type: text/plain, Size: 475 bytes --]

On Mon, Nov 02, 2020 at 09:15:49AM +0100, Markus Armbruster wrote:
> Add the relevant QAPI schema modules to section Audio, QMP, Tracing,
> Cryptography.
> 
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Stefan Hajnoczi <stefanha@redhat.com>
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  MAINTAINERS | 6 ++++++
>  1 file changed, 6 insertions(+)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH 1/2] MAINTAINERS: Add QAPI schema modules to their subsystems

[Daniel P. Berrangé]

On Mon, Nov 02, 2020 at 09:15:49AM +0100, Markus Armbruster wrote:
> Add the relevant QAPI schema modules to section Audio, QMP, Tracing,
> Cryptography.
> 
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Stefan Hajnoczi <stefanha@redhat.com>
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  MAINTAINERS | 6 ++++++
>  1 file changed, 6 insertions(+)

Acked-by: Daniel P. Berrangé <berrange@redhat.com>


Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

[PATCH 2/2] qapi: Fix missing headers in QMP Reference Manual

[Markus Armbruster]

Audio stuff is under "Miscellanea", and authorization stuff is under
"Input".  Add suitable header doc comments to correct that.

Cc: Gerd Hoffmann <kraxel@redhat.com>
Cc: Daniel P. Berrange <berrange@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
---
 qapi/audio.json | 4 ++++
 qapi/authz.json | 6 ++++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/qapi/audio.json b/qapi/audio.json
index 3b843878d2..072ed79def 100644
--- a/qapi/audio.json
+++ b/qapi/audio.json
@@ -5,6 +5,10 @@
 # This work is licensed under the terms of the GNU GPL, version 2 or later.
 # See the COPYING file in the top-level directory.
 
+##
+# = Audio
+##
+
 ##
 # @AudiodevPerDirectionOptions:
 #
diff --git a/qapi/authz.json b/qapi/authz.json
index f3e9745426..42afe752d1 100644
--- a/qapi/authz.json
+++ b/qapi/authz.json
@@ -1,7 +1,9 @@
 # -*- Mode: Python -*-
 # vim: filetype=python
-#
-# QAPI authz definitions
+
+##
+# = User authorization
+##
 
 ##
 # @QAuthZListPolicy:
-- 
2.26.2

Re: [PATCH 2/2] qapi: Fix missing headers in QMP Reference Manual

[Daniel P. Berrangé]

On Mon, Nov 02, 2020 at 09:15:50AM +0100, Markus Armbruster wrote:
> Audio stuff is under "Miscellanea", and authorization stuff is under
> "Input".  Add suitable header doc comments to correct that.
> 
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Cc: Daniel P. Berrange <berrange@redhat.com>
> Signed-off-by: Markus Armbruster <armbru@redhat.com>
> ---
>  qapi/audio.json | 4 ++++
>  qapi/authz.json | 6 ++++--
>  2 files changed, 8 insertions(+), 2 deletions(-)

Acked-by: Daniel P. Berrangé <berrange@redhat.com>

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|