[PATCH 1/8] qemu: capabilities: Explain that QEMU_CAPS_CHARDEV_JSON will be used in tests only

Peter Krempa posted 8 patches 1 year, 4 months ago
[PATCH 1/8] qemu: capabilities: Explain that QEMU_CAPS_CHARDEV_JSON will be used in tests only
Posted by Peter Krempa 1 year, 4 months ago
I've added that capability a long time ago when I was converting various
stuff to use JSON but the support in '-chardev' didn't yet materialize.

Fix the comment to make that clear and also that it'll be used in tests
for the upcoming refactor of the chardev code (so that we can validate
generator against the schema even if that doesn't yet work).

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 src/qemu/qemu_capabilities.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/qemu/qemu_capabilities.h b/src/qemu/qemu_capabilities.h
index 5036d49aab..736d34179e 100644
--- a/src/qemu/qemu_capabilities.h
+++ b/src/qemu/qemu_capabilities.h
@@ -623,7 +623,7 @@ typedef enum { /* virQEMUCapsFlags grouping marker for syntax-check */
     QEMU_CAPS_MEMORY_BACKEND_RESERVE, /* -object memory-backend-*.reserve= */
     QEMU_CAPS_PIIX4_ACPI_ROOT_PCI_HOTPLUG, /* -M pc PIIX4_PM.acpi-root-pci-hotplug */
     QEMU_CAPS_NETDEV_JSON, /* -netdev accepts JSON */
-    QEMU_CAPS_CHARDEV_JSON, /* -chardev accepts JSON */
+    QEMU_CAPS_CHARDEV_JSON, /* Reserved for '-chardev' JSON support. For now used only in tests. */

     /* 415 */
     X_QEMU_CAPS_DEVICE_JSON_BROKEN_HOTPLUG, /* -device accepts JSON (must not be used - users are filtering the capbility) */
-- 
2.46.0
Re: [PATCH 1/8] qemu: capabilities: Explain that QEMU_CAPS_CHARDEV_JSON will be used in tests only
Posted by Ján Tomko 1 year, 4 months ago
On a Wednesday in 2024, Peter Krempa wrote:
>I've added that capability a long time ago when I was converting various
>stuff to use JSON but the support in '-chardev' didn't yet materialize.
>
>Fix the comment to make that clear and also that it'll be used in tests
>for the upcoming refactor of the chardev code (so that we can validate
>generator against the schema even if that doesn't yet work).
>
>Signed-off-by: Peter Krempa <pkrempa@redhat.com>
>---
> src/qemu/qemu_capabilities.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano