On 6/7/21 3:39 PM, Thomas Huth wrote: > On 04/06/2021 17.51, Alex Bennée wrote: >> From: Philippe Mathieu-Daudé <phi...@redhat.com> >> >> Various tests don't require TCG, but have '_tcg' in their name. >> As this is misleading, remove 'tcg' from their name. >> >> Reported-by: Igor Mammedov <imamm...@redhat.com> >> Reviewed-by: Igor Mammedov <imamm...@redhat.com> >> Signed-off-by: Philippe Mathieu-Daudé <phi...@redhat.com> >> Signed-off-by: Alex Bennée <alex.ben...@linaro.org> >> Message-Id: <20210505125806.1263441-10-phi...@redhat.com> >> --- >> tests/qtest/bios-tables-test.c | 142 ++++++++++++++++----------------- >> 1 file changed, 71 insertions(+), 71 deletions(-) > [...] >> @@ -1255,7 +1255,7 @@ static void test_acpi_microvm_rtc_tcg(void) >> free_test_data(&data); >> } >> -static void test_acpi_microvm_pcie_tcg(void) >> +static void test_acpi_microvm_pcie(void) >> { >> test_data data; > > This change is wrong: test_acpi_microvm_pcie_tcg() uses data.tcg_only = > true, so the _tcg suffix indeed makes sense here.
I supposed I messed while rebasing...