[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v16 06/99] qtest/arm-cpu-features: Remove TCG fallback to KVM
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH v16 06/99] qtest/arm-cpu-features: Remove TCG fallback to KVM specific tests |
Date: |
Tue, 8 Jun 2021 10:23:47 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
On 6/7/21 3:28 PM, Thomas Huth wrote:
> On 04/06/2021 17.51, Alex Bennée wrote:
>> From: Philippe Mathieu-Daudé <philmd@redhat.com>
>>
>> sve_tests_sve_off_kvm() and test_query_cpu_model_expansion_kvm()
>> tests are now only being run if KVM is available. Drop the TCG
>> fallback.
>>
>> Suggested-by: Andrew Jones <drjones@redhat.com>
>> Reviewed-by: Andrew Jones <drjones@redhat.com>
>> Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> Message-Id: <20210505125806.1263441-7-philmd@redhat.com>
>> ---
>> tests/qtest/arm-cpu-features.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/tests/qtest/arm-cpu-features.c
>> b/tests/qtest/arm-cpu-features.c
>> index 66300c3bc2..b1d406542f 100644
>> --- a/tests/qtest/arm-cpu-features.c
>> +++ b/tests/qtest/arm-cpu-features.c
>> @@ -21,7 +21,7 @@
>> #define SVE_MAX_VQ 16
>> #define MACHINE "-machine virt,gic-version=max -accel tcg "
>> -#define MACHINE_KVM "-machine virt,gic-version=max -accel kvm -accel
>> tcg "
>> +#define MACHINE_KVM "-machine virt,gic-version=max -accel kvm "
>
> Same comment as with patch 04/99: I think this is wrong. You're mixing
> up whether an accelerator has been built into the binary with the fact
> whether an accelerator is available and usable. There are plenty of
> cases where e.g. kvm is built into the binary but not usable during
> runtime, e.g. because:
> 1) The kernel does not support it
> 2) The current host CPU does not support it
> 3) There are problems with the permission to /dev/kvm
> etc.
Yes.
> I think we either need the fallback mechanism to tcg
No, this is precisely what we want to test.
> or you need to
> properly check whether KVM is usable, too.
Yes.
- [PATCH v16 08/99] qtest/migration-test: Skip tests if KVM not builtin on s390x/ppc64, (continued)
- [PATCH v16 08/99] qtest/migration-test: Skip tests if KVM not builtin on s390x/ppc64, Alex Bennée, 2021/06/04
- [PATCH v16 07/99] qtest/arm-cpu-features: Use generic qtest_has_accel() to check for TCG, Alex Bennée, 2021/06/04
- [PATCH v16 06/99] qtest/arm-cpu-features: Remove TCG fallback to KVM specific tests, Alex Bennée, 2021/06/04
- [PATCH v16 09/99] qtest/bios-tables-test: Rename tests not TCG specific, Alex Bennée, 2021/06/04
- [PATCH v16 11/99] qtest/bios-tables-test: Make test build-independent from accelerator, Alex Bennée, 2021/06/04
- [PATCH v16 12/99] qtest: Do not restrict bios-tables-test to Aarch64 hosts anymore, Alex Bennée, 2021/06/04
- [PATCH v16 13/99] meson: add target_user_arch, Alex Bennée, 2021/06/04