From: Pierrick Bouvier <pierrick.bouvier@oss.qualcomm.com>
To: Richard Henderson <richard.henderson@linaro.org>, qemu-devel@nongnu.org
Cc: philmd@linaro.org
Subject: Re: [PATCH 0/5] target_info optimizations
Date: Mon, 11 May 2026 19:14:51 -0700 [thread overview]
Message-ID: <a31c87d2-8c57-4353-af45-01bdaa02fdc0@oss.qualcomm.com> (raw)
In-Reply-To: <20260511190634.472174-1-richard.henderson@linaro.org>
On 5/11/2026 12:06 PM, Richard Henderson wrote:
> Based-on: 20260509005453.3984184-1-pierrick.bouvier@oss.qualcomm.com
> ("[PATCH v5 0/6] single-binary: deduplicate target_info()")
>
> Allow TargetInfo accesses to resolve to a direct structure access.
> Arrange for the structure to appear const to the majority of qemu.
> For user-only, the structure actually is const; for system, the
> structure is initialized early.
>
It would be nice, if possible, to see those optimizations driven by a
concrete benchmark. We had to deal with several premature optimizations
along our road on the single-binary, and it would be great to not repeat
the same thing again on this new component.
>
> r~
>
>
> Richard Henderson (5):
> meson: Build target-info-qom.c with -fno-lto
> target-info: Expose target_info as const data
> target-info: Merge target-info-impl.h into target-info.h
> target-info: Merge target-info-qapi.h into target-info.h
> target-info: Inline accessors
>
> include/qemu/target-info-impl.h | 44 -----------
> include/qemu/target-info-init.h | 45 ++++++++---
> include/qemu/target-info-qapi.h | 29 -------
> include/qemu/target-info-qom.h | 2 +-
> include/qemu/target-info.h | 123 +++++++++++++++++++++++++++---
> configs/targets/aarch64-softmmu.c | 18 +----
> configs/targets/arm-softmmu.c | 18 +----
> hw/core/machine-qmp-cmds.c | 1 -
> hw/uefi/ovmf-log.c | 2 +-
> hw/virtio/virtio-mem.c | 2 +-
> page-vary-common.c | 10 +--
> page-vary-system.c | 8 +-
> system/arch_init.c | 2 +-
> target-info-qom.c | 13 ++--
> target-info-stub.c | 27 +------
> target-info.c | 95 -----------------------
> meson.build | 13 ++--
> 17 files changed, 179 insertions(+), 273 deletions(-)
> delete mode 100644 include/qemu/target-info-impl.h
> delete mode 100644 include/qemu/target-info-qapi.h
> delete mode 100644 target-info.c
>
prev parent reply other threads:[~2026-05-12 2:15 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-11 19:06 [PATCH 0/5] target_info optimizations Richard Henderson
2026-05-11 19:06 ` [PATCH 1/5] meson: Build target-info-qom.c with -fno-lto Richard Henderson
2026-05-11 19:38 ` Pierrick Bouvier
2026-05-11 20:08 ` Richard Henderson
2026-05-11 19:06 ` [PATCH 2/5] target-info: Expose target_info as const data Richard Henderson
2026-05-12 2:22 ` Pierrick Bouvier
2026-05-11 19:06 ` [PATCH 3/5] target-info: Merge target-info-impl.h into target-info.h Richard Henderson
2026-05-12 2:25 ` Pierrick Bouvier
2026-05-12 3:20 ` Philippe Mathieu-Daudé
2026-05-11 19:06 ` [PATCH 4/5] target-info: Merge target-info-qapi.h " Richard Henderson
2026-05-12 2:25 ` Pierrick Bouvier
2026-05-11 19:06 ` [PATCH 5/5] target-info: Inline accessors Richard Henderson
2026-05-12 2:26 ` Pierrick Bouvier
2026-05-12 2:14 ` Pierrick Bouvier [this message]
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=a31c87d2-8c57-4353-af45-01bdaa02fdc0@oss.qualcomm.com \
--to=pierrick.bouvier@oss.qualcomm.com \
--cc=philmd@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.