Re: [RFC PATCH 6/8] tests/qtest/libqtest: Allow checking for HVF accelerator

[Thomas Huth]

On 19/01/2023 12.24, Thomas Huth wrote:
> On 19/01/2023 11.05, Philippe Mathieu-Daudé wrote:
> > Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> > ---
> > RFC: CONFIG_HVF is poisoned.
> >
> > We could pass host config definitions to qtest using:
> >
> >    diff --git a/meson.build b/meson.build
> >    @@ -2547,6 +2547,7 @@ foreach target : target_dirs
> >
> >       accel_kconfig = []
> >       foreach sym: accelerators
> >    +    config_host_data.set(sym + '_QTEST', '')
> >         if sym == 'CONFIG_TCG' or target in accelerator_targets.get(sym, [])
> >           config_target += { sym: 'y' }
> >           config_all += { sym: 'y' }
> >
> > Then test for CONFIG_HVF_QTEST ...
>
> I don't think that would really work well. The qtests are build once for all 
> targets, and HVF is only available in the target that matches the host 
> architecture. It's poisoned on purpose.
>
> The TCG accelerator is special, since we have it in either none or in all 
> targets, that's why we can use CONFIG_TCG there.
>
> The kvm part is also rather a hack... we should maybe rather additionally 
> use the "query-kvm" QAPI command to check whether it is really available...?

Scratch that ... I forgot that you already have to run with "-accel kvm" to 
see whether the accelerator is working with "query-kvm" ... so that would 
not work here for probing whether "-accel kvm" should be used or not ;-)

 Thomas