Re: [PATCH v7 00/49] single-binary: compile target/arm twice

[Peter Maydell <peter.maydell@linaro.org>]

On Mon, 12 May 2025 at 17:53, Pierrick Bouvier
<pierrick.bouvier@linaro.org> wrote:
>
> Hi Peter,
>
> On 5/11/25 6:40 AM, Peter Maydell wrote:
> > On Thu, 8 May 2025 at 00:42, Pierrick Bouvier
> > <pierrick.bouvier@linaro.org> wrote:
> >>
> >> More work toward single-binary.
> >>
> >> Some files have external dependencies for the single-binary:
> >> - target/arm/gdbstub.c: gdbhelpers
> >> - target/arm/arm-qmp-cmds.c: qapi
> >> - target/arm/tcg/translate*: need deep cleanup in include/tcg
> >> - target/arm/tcg/cpu*: need TargetInfo implemented for arm/aarch64
> >> - target/arm/tcg/*-helper*: need deeper split between aarch64 and arm code
> >> They will not be ported in this series.
> >>
> >> Built on {linux, windows, macos} x {x86_64, aarch64}
> >> Fully tested on linux x {x86_64, aarch64}
> >>
> >> Series is now tested and fully reviewed. Thanks for pulling it.
> >
> > Do you/Philippe have a plan for how you want this to go into
> > the tree? I know Philippe has been taking a lot of the
> > single-binary related patches. Let me know if you want me
> > to pick it up via target-arm.
> >
>
> During the release code freeze, we mostly used tcg-next.
> However, now everything is back to normal, we simply work upstream, with
> a simple "first pulled, first in" strategy, fixing the occasional
> conflicts on our respective sides.
>
> So if you could pull this, that would be appreciated.

I had a go, but it seems to depend on some other patch
or series that isn't in upstream git yet. Specifically,
the changes to include/system/hvf.h assume it has an
include of "cpu.h": you can see it in the context in
patch 2:
https://lore.kernel.org/qemu-devel/20250507234241.957746-3-pierrick.bouvier@linaro.org/
but that include isn't there in upstream git yet, so the
patches touching that file eventually fail to apply cleanly.
I assume that's "accel/hvf: Include missing 'hw/core/cpu.h' header",
but is there anything else this series was based on?

thanks
-- PMM