qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: qemu-devel@nongnu.org
Cc: "Markus Armbruster" <armbru@redhat.com>,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Thomas Huth" <thuth@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Michael Roth" <michael.roth@amd.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>
Subject: [PATCH 0/6] qapi: generalize special features
Date: Thu,  1 Aug 2024 18:59:07 +0100	[thread overview]
Message-ID: <20240801175913.669013-1-berrange@redhat.com> (raw)

This series is a spin-off from

  https://lists.nongnu.org/archive/html/qemu-devel/2024-06/msg00807.html

That series introduced a pragma allowing a schema to declare extra
features that would be exposed to code.

Following Markus' suggestion:

  https://lists.nongnu.org/archive/html/qemu-devel/2024-07/msg03765.html

I've changed impl such that we expose all features to the code
regardless of whether they are special, and don't require any pragma.

I've split it from the QGA patches since it makes more sense to work
on this bit in isolation.

Daniel P. Berrangé (6):
  qapi: change 'unsigned special_features' to 'uint64_t features'
  scripts/qapi: rename 'special_features' to 'features'
  qapi: use "QAPI_FEATURE" as namespace for features
  qapi: cope with  feature names containing a '-'
  qapi: apply schema prefix to QAPI feature enum constants
  qapi: expose all schema features to code

 include/qapi/compat-policy.h  |   2 +-
 include/qapi/qmp/dispatch.h   |   4 +-
 include/qapi/util.h           |   7 +-
 include/qapi/visitor-impl.h   |   4 +-
 include/qapi/visitor.h        |  12 +--
 meson.build                   |   1 +
 qapi/qapi-forward-visitor.c   |   8 +-
 qapi/qapi-util.c              |   6 +-
 qapi/qapi-visit-core.c        |  12 +--
 qapi/qmp-dispatch.c           |   2 +-
 qapi/qmp-registry.c           |   4 +-
 qapi/qobject-input-visitor.c  |   4 +-
 qapi/qobject-output-visitor.c |   6 +-
 scripts/qapi/commands.py      |   9 ++-
 scripts/qapi/events.py        |   3 +-
 scripts/qapi/features.py      | 134 ++++++++++++++++++++++++++++++++++
 scripts/qapi/gen.py           |   9 ++-
 scripts/qapi/main.py          |   2 +
 scripts/qapi/schema.py        |   5 +-
 scripts/qapi/types.py         |  19 +++--
 scripts/qapi/visit.py         |  17 +++--
 21 files changed, 206 insertions(+), 64 deletions(-)
 create mode 100644 scripts/qapi/features.py

-- 
2.45.2



             reply	other threads:[~2024-08-01 18:00 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-01 17:59 Daniel P. Berrangé [this message]
2024-08-01 17:59 ` [PATCH 1/6] qapi: change 'unsigned special_features' to 'uint64_t features' Daniel P. Berrangé
2024-08-05 11:53   ` Markus Armbruster
2024-08-01 17:59 ` [PATCH 2/6] scripts/qapi: rename 'special_features' to 'features' Daniel P. Berrangé
2024-08-05 11:59   ` Markus Armbruster
2024-08-01 17:59 ` [PATCH 3/6] qapi: use "QAPI_FEATURE" as namespace for features Daniel P. Berrangé
2024-08-05 12:01   ` Markus Armbruster
2024-08-01 17:59 ` [PATCH 4/6] qapi: cope with feature names containing a '-' Daniel P. Berrangé
2024-08-05 12:10   ` Markus Armbruster
2024-08-01 17:59 ` [PATCH 5/6] qapi: apply schema prefix to QAPI feature enum constants Daniel P. Berrangé
2024-08-05 12:22   ` Markus Armbruster
2024-08-05 12:33     ` Daniel P. Berrangé
2024-08-05 13:11       ` Markus Armbruster
2024-08-05 13:24         ` Daniel P. Berrangé
2024-08-05 13:54           ` Markus Armbruster
2024-08-05 14:59             ` Markus Armbruster
2024-08-06 17:49               ` Complications due to having multiple QAPI schemas (was: [PATCH 5/6] qapi: apply schema prefix to QAPI feature enum constants) Markus Armbruster
2024-08-08 11:48   ` [PATCH 5/6] qapi: apply schema prefix to QAPI feature enum constants Markus Armbruster
2024-08-01 17:59 ` [PATCH 6/6] qapi: expose all schema features to code Daniel P. Berrangé
2024-08-02 13:50   ` Markus Armbruster
2024-08-02 15:43     ` Daniel P. Berrangé
2024-08-08 12:11   ` Markus Armbruster

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240801175913.669013-1-berrange@redhat.com \
    --to=berrange@redhat.com \
    --cc=armbru@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=michael.roth@amd.com \
    --cc=pbonzini@redhat.com \
    --cc=philmd@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=thuth@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).