Re: meson, NEED_CPU_H, CONFIG_TCG and tests/

[Thomas Huth <thuth@redhat.com>]

On 19/02/2021 14.04, Claudio Fontana wrote:
> Hi Paolo,
> 
> currently we have use of CONFIG_TCG in tests/,
> 
> but is that variable available at all in there?
> 
> I have to adapt some qemu/tests/qtest/* to work also without tcg for ARM,
> 
> but I think I am not seeing CONFIG_TCG filtering through, and I wonder whether all the checks in there are actually "wrong".
> 
> Looking at meson.build it would seem to me that only stuff in target/ would be able to see CONFIG_TCG,
> as a result of
> 
> foreach target : target_dirs
>    config_target = config_target_mak[target]
>    target_name = config_target['TARGET_NAME']
>    arch = config_target['TARGET_BASE_ARCH']
>    arch_srcs = [config_target_h[target]]
>    arch_deps = []
>    c_args = ['-DNEED_CPU_H',
>              '-DCONFIG_TARGET="@0@-config-target.h"'.format(target),
>              '-DCONFIG_DEVICES="@0@-config-devices.h"'.format(target)]
> 
> But how could tests see this?

The test are generic, not target-specific code, so they can not see things 
like NEED_CPU_H, of course.

But CONFIG_TCG seems to be defined in config-host.h, so you should be able 
to use that define, shouldn't you?

  Thomas