qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 09/22] qga: conditionalize schema for commands unsupported


From: Philippe Mathieu-Daudé
Subject: Re: [PATCH v2 09/22] qga: conditionalize schema for commands unsupported on non-Linux POSIX
Date: Wed, 3 Jul 2024 10:31:35 +0200
User-agent: Mozilla Thunderbird

On 13/6/24 17:43, Daniel P. Berrangé wrote:
Rather than creating stubs for every command that just return
QERR_UNSUPPORTED, use 'if' conditions in the QAPI schema to
fully exclude generation of the commands on non-Linux POSIX
platforms

The command will be rejected at QMP dispatch time instead,
avoiding reimplementing rejection by blocking the stub commands.
This changes the error message for affected commands from

     {"class": "CommandNotFound", "desc": "Command FOO has been disabled"}

to

     {"class": "CommandNotFound", "desc": "The command FOO has not been found"}

This has the additional benefit that the QGA protocol reference
now documents what conditions enable use of the command.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
  qga/commands-posix.c | 66 --------------------------------------------
  qga/qapi-schema.json | 30 +++++++++++---------
  2 files changed, 17 insertions(+), 79 deletions(-)

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>




reply via email to

[Prev in Thread] Current Thread [Next in Thread]