All of lore.kernel.org
 help / color / mirror / Atom feed
From: Markus Armbruster <armbru@redhat.com>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: "Pierrick Bouvier" <pierrick.bouvier@linaro.org>,
	qemu-devel@nongnu.org,
	"Philippe Mathieu-Daudé" <philmd@linaro.org>,
	"Brian Cain" <brian.cain@oss.qualcomm.com>,
	"Daniel P. Berrangé" <berrange@redhat.com>
Subject: Renaming targets (was: [PATCH] target-info: set target_arch statically)
Date: Fri, 06 Feb 2026 07:58:42 +0100	[thread overview]
Message-ID: <878qd6xsrh.fsf_-_@pond.sub.org> (raw)
In-Reply-To: <6ddad25a-192c-4b5a-80fc-b006436810a2@linaro.org> (Richard Henderson's message of "Fri, 6 Feb 2026 08:36:14 +1000")

Richard Henderson <richard.henderson@linaro.org> writes:

> On 2/5/26 19:52, Markus Armbruster wrote:
>>> I assume the qapi string is the one that should take precedence; everything else appears to be merely qemu source level strings.  Marcus, can you confirm?
>> 
>> What exactly would you like me to confirm?
>> 
>> I *guess* it's about the messy part of the patch you posted upthread.
>> There, you have to normalize TARGET_ARCH value 'openrisc' to 'or1k'.
>
> Yes.  I wanted you to confirm that changing the string in qapi is a non-starter, therefore 
> 'or1k' is the string we should standardize on.

Yes, changing enum value @or1k is problematic.

Enum SysEmuTarget is visible in QMP: query-target return member @arch,
query-cpus-fast return member @target.  These are stable interfaces.

I can see two ways to change the enum value to @openrisc:

1. We create a new qemu-system-openrisc, identical to qemu-system-or1k
   except for the target name.  We deprecate qemu-system-or1k and
   SysEmuTarget member @or1k, and remove them after the grace period.

   I doubt it's worth the bother.

2. We cheat: we rename with the excuse that the target is bottom support
   tier, and our compatibility promise doesn't apply there.

   I don't think that's a good idea.  If we want to qualify our
   compatibility promise with tiers, we better define the tiers and what
   they mean for the promise *first*.  Related:

    From: Daniel P. Berrangé <berrange@redhat.com>
    Subject: [PATCH] docs/about: propose OS platform/arch support tiers
    Date: Thu, 15 Jan 2026 18:01:23 +0000
    Message-ID: <20260115180123.848640-1-berrange@redhat.com>
    https://lore.kernel.org/qemu-devel/20260115180123.848640-1-berrange@redhat.com/

>> I figure this is about adding 'hexagon' to SysEmuTarget even though it's
>> not actually a system emulator target now.
>> 
>> The fact that adding it there helps indicates SysEmuTarget has leaked
>> into user emulators, and its name has become misleading.  Is this true?
>
> Yes.  The target_info structure is used for both user and system, and one of these fields 
> is the SysEmuTarget entry.

Then SysEmuTarget has become misleading.

We *can* fix that: QAPI type names are not part of the external
interface by design.



  reply	other threads:[~2026-02-06  6:59 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-03 19:41 [PATCH] target-info: set target_arch statically Pierrick Bouvier
2026-02-04  1:36 ` Richard Henderson
2026-02-04 16:36   ` Pierrick Bouvier
2026-02-04 22:22     ` Richard Henderson
2026-02-05  9:52       ` Markus Armbruster
2026-02-05 22:36         ` Richard Henderson
2026-02-06  6:58           ` Markus Armbruster [this message]
2026-02-06  7:21             ` Renaming targets Pierrick Bouvier
2026-02-06  7:38               ` Markus Armbruster
2026-02-06  7:52                 ` Pierrick Bouvier
2026-02-04  9:10 ` [PATCH] target-info: set target_arch statically Daniel P. Berrangé
2026-02-04 16:38   ` Pierrick Bouvier
2026-02-04 17:06 ` Anton Johansson via qemu development
2026-02-05  9:29 ` Markus Armbruster
2026-02-05 19:02   ` Pierrick Bouvier

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=878qd6xsrh.fsf_-_@pond.sub.org \
    --to=armbru@redhat.com \
    --cc=berrange@redhat.com \
    --cc=brian.cain@oss.qualcomm.com \
    --cc=philmd@linaro.org \
    --cc=pierrick.bouvier@linaro.org \
    --cc=qemu-devel@nongnu.org \
    --cc=richard.henderson@linaro.org \
    /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.