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?
>
>
> r~
>
