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~
