Re: [PATCH 7/7] tests/qtest/migration: Allow using accelerators different of TCG / KVM

[Richard Henderson]

On 1/28/25 11:50, Fabiano Rosas wrote:
> Richard Henderson <richard.henderson@linaro.org> writes:
>
> > On 1/28/25 07:04, Fabiano Rosas wrote:
> > > Philippe Mathieu-Daudé <philmd@linaro.org> writes:
> > >
> > > > There is no particular reason to restrict all the framework
> > > > to TCG or KVM, since we can check on a per-test basis which
> > > > accelerator is available (via MigrationTestEnv::has_$ACCEL).
> > >
> > > The reason is:
> > >
> > > CONFIG_KVM=n
> > > CONFIG_TCG=n
> > >
> > > The check is about "there is no accelerator at all".
> >
> > We perform the no accelerator check in meson.build on a per-target basis.
> > So you'll never have a qemu-system-foo binary with zero accelerators.
>
> Ok, the issue was not zero accelerators, but Xen. On aarch64, this
> produces a binary with no TCG nor KVM:
>
> --target-list=x86_64-softmmu --disable-tcg --enable-xen

Right.  Phil's patch set specifically mentions HVF, but there's no reason Xen can't be 
better handled too.  Unless there's something explicit about Xen that cannot be handled 
here?  But even then, the if condition needs updating.


r~