All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Yonggang Luo <luoyonggang@gmail.com>
Cc: qemu-devel@nongnu.org, "Eric Blake" <eblake@redhat.com>,
	qemu-riscv@nongnu.org, "Daniel P. Berrangé" <berrange@redhat.com>,
	"Alexander Bulekov" <alxndr@bu.edu>,
	"Laurent Vivier" <lvivier@redhat.com>,
	"Paolo Bonzini" <pbonzini@redhat.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	qemu-s390x@nongnu.org, "Song Gao" <17746591750@163.com>,
	"Matthew Rosato" <mjrosato@linux.ibm.com>,
	"Alistair Francis" <alistair.francis@wdc.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	"Fabiano Rosas" <farosas@suse.de>,
	"Eric Farman" <farman@linux.ibm.com>,
	"Brian Cain" <brian.cain@oss.qualcomm.com>,
	"Stefan Hajnoczi" <stefanha@redhat.com>,
	"Darren Kenny" <darren.kenny@oracle.com>,
	"Ani Sinha" <anisinha@redhat.com>,
	"Peter Maydell" <peter.maydell@linaro.org>,
	"Marc-André Lureau" <marcandre.lureau@redhat.com>,
	"Richard Henderson" <richard.henderson@linaro.org>,
	"Stafford Horne" <shorne@gmail.com>,
	"Cornelia Huck" <cohuck@redhat.com>,
	"Zhao Liu" <zhao1.liu@intel.com>,
	"Weiwei Li" <liwei1518@gmail.com>,
	"David Hildenbrand" <david@kernel.org>,
	"Laurent Vivier" <laurent@vivier.eu>,
	"Pierrick Bouvier" <pierrick.bouvier@oss.qualcomm.com>,
	"Fam Zheng" <fam@euphon.net>,
	"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
	"Philippe Mathieu-Daudé" <philmd@oss.qualcomm.com>,
	"Daniel Henrique Barboza" <daniel.barboza@oss.qualcomm.com>,
	"Palmer Dabbelt" <palmer@dabbelt.com>,
	qemu-arm@nongnu.org, "Chao Liu" <chao.liu@processmission.com>,
	"Philippe Mathieu-Daude" <philmd@mailo.com>,
	"Jiaxun Yang" <jiaxun.yang@flygoat.com>,
	"Max Filippov" <jcmvbkbc@gmail.com>,
	"Stefano Garzarella" <sgarzare@redhat.com>
Subject: Re: [PATCH 04/34] qapi: rename SysEmuTarget to EmuTarget
Date: Wed, 14 Oct 2026 13:24:12 +0200	[thread overview]
Message-ID: <87ecdscyoz.fsf@pond.sub.org> (raw)
In-Reply-To: <20261014093152.834-5-luoyonggang@gmail.com> (Yonggang Luo's message of "Wed, 14 Oct 2026 17:31:14 +0800")

Yonggang Luo <luoyonggang@gmail.com> writes:

> As now it's also expressing user-mode targets.
>
> This changes the type name exposed by query-qmp-schema; enum values and
> serialized target values remain unchanged.

Are you sure?  query-qmp-schema masks the type names.

> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>

[...]

> diff --git a/qapi/machine.json b/qapi/machine.json
> index 9b145c635ba..07fac4f0764 100644
> --- a/qapi/machine.json
> +++ b/qapi/machine.json
> @@ -14,7 +14,7 @@
>  { 'include': 'machine-common.json' }
>  
>  ##
> -# @SysEmuTarget:
> +# @EmuTarget:
>  #     The comprehensive enumeration of QEMU system emulation
>  #     ("softmmu") targets.  Run "./configure --help" in the project
>  #     root directory, and look for the \*-softmmu targets near the
   #     "--target-list" option.  The individual target constants are not
   #     documented here, for the time being.

Is this still correct?

> @@ -35,7 +35,7 @@
>  #
>  # Since: 3.0
>  ##
> -{ 'enum' : 'SysEmuTarget',
> +{ 'enum' : 'EmuTarget',
>    'data' : [ 'aarch64', 'alpha', 'arm', 'avr', 'hexagon', 'hppa', 'i386',
>               'loongarch64', 'm68k', 'microblaze', 'mips', 'mips64',
>               'mips64el', 'mipsel', 'or1k', 'ppc',
> @@ -95,7 +95,7 @@
>                        'qom-path'     : 'str',
>                        'thread-id'    : 'int',
>                        '*props'       : 'CpuInstanceProperties',
> -                      'target'       : 'SysEmuTarget' },
> +                      'target'       : 'EmuTarget' },
>    'discriminator' : 'target',
>    'data'          : { 's390x'        : 'CpuInfoS390' } }
>  
> @@ -281,7 +281,7 @@
>  # Since: 1.2
>  ##
>  { 'struct': 'QemuTargetInfo',
> -  'data': { 'arch': 'SysEmuTarget' } }
> +  'data': { 'arch': 'EmuTarget' } }
>  
>  ##
>  # @query-target:
> diff --git a/qapi/pragma.json b/qapi/pragma.json
> index 342cedc42e7..93a710ee937 100644
> --- a/qapi/pragma.json
> +++ b/qapi/pragma.json
> @@ -72,7 +72,7 @@
>          'SsidSizeMode',
>          'String',
>          'StringWrapper',
> -        'SysEmuTarget',
> +        'EmuTarget',
>          'ThrottleGroupProperties',
>          'VncPrimaryAuth',
>          'VncVencryptSubAuth',
> @@ -109,7 +109,7 @@
>          'QKeyCode',                 # send-key, input-sent-event
>          'QapiErrorClass',           # QMP error replies
>          'SshHostKeyCheckMode',      # blockdev-add, -blockdev
> -        'SysEmuTarget',             # query-cpu-fast, query-target
> +        'EmuTarget',                # query-cpu-fast, query-target
>          'UuidInfo',                 # query-uuid
>          'VncClientInfo',            # query-vnc, query-vnc-servers, ...
>          'X86CPURegister32'          # qom-get of x86 CPU properties

[...]



  reply	other threads:[~2026-10-14 11:24 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-14  9:31 [PATCH 00/34] single-binary: Select the softmmu target through -machine <type> Yonggang Luo
2026-10-14  9:31 ` [PATCH 01/34] qemu/base-arch-defs: extract QEMU_ARCH_BIT for QEMU_ARCH_* enums Yonggang Luo
2026-10-14  9:31 ` [PATCH 02/34] qemu: Use SYS_EMU_TARGET__MAX to define QEMU_ARCH_ALL Yonggang Luo
2026-10-14  9:31 ` [PATCH 03/34] qemu: increase width of QEMU_ARCH_* masks enums to be 64 bits Yonggang Luo
2026-10-14  9:31 ` [PATCH 04/34] qapi: rename SysEmuTarget to EmuTarget Yonggang Luo
2026-10-14 11:24   ` Markus Armbruster [this message]
2026-10-14 12:18     ` Yonggang Luo
2026-10-14  9:31 ` [PATCH 05/34] qapi: add missing Linux user-mode targets " Yonggang Luo
2026-10-14 10:24   ` Philippe Mathieu-Daudé
2026-10-14 10:30     ` Daniel P. Berrangé
2026-10-14 10:45       ` Philippe Mathieu-Daudé
2026-10-14 10:48         ` Yonggang Luo
2026-10-14 10:46     ` Yonggang Luo
2026-10-14 17:26   ` Pierrick Bouvier
2026-10-14  9:31 ` [PATCH 06/34] meson: define TARGET_ARCH_VARIANT to target_name only Yonggang Luo
2026-10-14  9:31 ` [PATCH 07/34] target-info: prepare the QOM target list before module_call_init(MODULE_INIT_QOM) Yonggang Luo
2026-10-14  9:31 ` [PATCH 08/34] hw: add MachineClass::targets with a usage and test Yonggang Luo
2026-10-14  9:31 ` [PATCH 09/34] accel: register CPU parent for each target Yonggang Luo
2026-10-14  9:31 ` [PATCH 10/34] meson: collect Rust crates for combined system target Yonggang Luo
2026-10-14  9:31 ` [PATCH 11/34] module: Add module_allow_all_arch for allow all modules be loaded Yonggang Luo
2026-10-14  9:31 ` [PATCH 12/34] target/arm: defer max-v8 initializer selection until CPU creation Yonggang Luo
2026-10-14  9:31 ` [PATCH 13/34] hw/microblaze: defer PetaLogix endianness default until init Yonggang Luo
2026-10-14  9:31 ` [PATCH 14/34] hw/pci: Avoid guard loadparm with target_s390x() for s390x in class_init Yonggang Luo
2026-10-14  9:31 ` [PATCH 15/34] hw/scsi: " Yonggang Luo
2026-10-14 17:35 ` [PATCH 00/34] single-binary: Select the softmmu target through -machine <type> Pierrick Bouvier
2026-10-15  2:27   ` Yonggang Luo

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=87ecdscyoz.fsf@pond.sub.org \
    --to=armbru@redhat.com \
    --cc=17746591750@163.com \
    --cc=alistair.francis@wdc.com \
    --cc=alxndr@bu.edu \
    --cc=anisinha@redhat.com \
    --cc=berrange@redhat.com \
    --cc=brian.cain@oss.qualcomm.com \
    --cc=chao.liu@processmission.com \
    --cc=cohuck@redhat.com \
    --cc=daniel.barboza@oss.qualcomm.com \
    --cc=darren.kenny@oracle.com \
    --cc=david@kernel.org \
    --cc=eblake@redhat.com \
    --cc=edgar.iglesias@gmail.com \
    --cc=fam@euphon.net \
    --cc=farman@linux.ibm.com \
    --cc=farosas@suse.de \
    --cc=jcmvbkbc@gmail.com \
    --cc=jiaxun.yang@flygoat.com \
    --cc=kraxel@redhat.com \
    --cc=laurent@vivier.eu \
    --cc=liwei1518@gmail.com \
    --cc=luoyonggang@gmail.com \
    --cc=lvivier@redhat.com \
    --cc=marcandre.lureau@redhat.com \
    --cc=mjrosato@linux.ibm.com \
    --cc=mst@redhat.com \
    --cc=palmer@dabbelt.com \
    --cc=pbonzini@redhat.com \
    --cc=peter.maydell@linaro.org \
    --cc=philmd@mailo.com \
    --cc=philmd@oss.qualcomm.com \
    --cc=pierrick.bouvier@oss.qualcomm.com \
    --cc=qemu-arm@nongnu.org \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-riscv@nongnu.org \
    --cc=qemu-s390x@nongnu.org \
    --cc=richard.henderson@linaro.org \
    --cc=sgarzare@redhat.com \
    --cc=shorne@gmail.com \
    --cc=stefanha@redhat.com \
    --cc=zhao1.liu@intel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.