On 3/29/21 4:03 PM, Richard Henderson wrote:
> On 3/29/21 1:02 AM, Claudio Fontana wrote:
>> On 3/28/21 7:21 PM, Richard Henderson wrote:
>>> On 3/26/21 1:36 PM, Claudio Fontana wrote:
>>>> test is TCG-only.
>>>>
>>>> Signed-off-by: Claudio Fontana <cfont...@suse.de>
>>>> Cc: Philippe Mathieu-Daudé <f4...@amsat.org>
>>>> ---
>>>> tests/qtest/bios-tables-test.c | 7 +++++++
>>>> 1 file changed, 7 insertions(+)
>>>>
>>>> diff --git a/tests/qtest/bios-tables-test.c
>>>> b/tests/qtest/bios-tables-test.c
>>>> index e020c83d2a..bd7b85909c 100644
>>>> --- a/tests/qtest/bios-tables-test.c
>>>> +++ b/tests/qtest/bios-tables-test.c
>>>> @@ -1488,6 +1488,13 @@ static void test_acpi_oem_fields_virt(void)
>>>> };
>>>> char *args;
>>>>
>>>> +#ifndef CONFIG_TCG
>>>> + if (data.tcg_only) {
>>>> + g_test_skip("TCG disabled, skipping ACPI tcg_only test");
>>>> + return;
>>>> + }
>>>> +#endif /* CONFIG_TCG */
>>>
>>> Why is this an ifdef and not tcg_enabled()?
>>
>> We want to launch this whenever TCG is compiled in.
>>
>>> We're using kvm_enabled() elsewhere...
>>
>> where?
>
> Whoops, there's a local re-definition of the function at the top of
> tests/qtest/arm-cpu-features.c. Hmm.
>
> r~
>
Indeed, an unfortunate name clash.
