[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 7/6] qtest/arm-cpu-features: Use generic qtest_has_accel()
From: |
Andrew Jones |
Subject: |
Re: [PATCH v3 7/6] qtest/arm-cpu-features: Use generic qtest_has_accel() to check for KVM |
Date: |
Thu, 15 Apr 2021 15:23:17 +0200 |
On Thu, Apr 15, 2021 at 02:57:37PM +0200, Philippe Mathieu-Daudé wrote:
> Use the recently added generic qtest_has_accel() method to
> check if KVM is available.
>
> Suggested-by: Claudio Fontana <cfontana@suse.de>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> tests/qtest/arm-cpu-features.c | 25 +------------------------
> 1 file changed, 1 insertion(+), 24 deletions(-)
>
> diff --git a/tests/qtest/arm-cpu-features.c b/tests/qtest/arm-cpu-features.c
> index 8252b85bb85..7f4b2521277 100644
> --- a/tests/qtest/arm-cpu-features.c
> +++ b/tests/qtest/arm-cpu-features.c
> @@ -26,21 +26,6 @@
> " 'arguments': { 'type': 'full', "
> #define QUERY_TAIL "}}"
>
> -static bool kvm_enabled(QTestState *qts)
> -{
> - QDict *resp, *qdict;
> - bool enabled;
> -
> - resp = qtest_qmp(qts, "{ 'execute': 'query-kvm' }");
> - g_assert(qdict_haskey(resp, "return"));
> - qdict = qdict_get_qdict(resp, "return");
> - g_assert(qdict_haskey(qdict, "enabled"));
> - enabled = qdict_get_bool(qdict, "enabled");
> - qobject_unref(resp);
> -
> - return enabled;
> -}
> -
> static QDict *do_query_no_props(QTestState *qts, const char *cpu_type)
> {
> return qtest_qmp(qts, QUERY_HEAD "'model': { 'name': %s }"
> @@ -493,14 +478,6 @@ static void test_query_cpu_model_expansion_kvm(const
> void *data)
>
> qts = qtest_init(MACHINE_KVM "-cpu max");
>
> - /*
> - * These tests target the 'host' CPU type, so KVM must be enabled.
> - */
> - if (!kvm_enabled(qts)) {
> - qtest_quit(qts);
> - return;
> - }
> -
> /* Enabling and disabling kvm-no-adjvtime should always work. */
> assert_has_feature_disabled(qts, "host", "kvm-no-adjvtime");
> assert_set_feature(qts, "host", "kvm-no-adjvtime", true);
> @@ -624,7 +601,7 @@ int main(int argc, char **argv)
> * order avoid attempting to run an AArch32 QEMU with KVM on
> * AArch64 hosts. That won't work and isn't easy to detect.
> */
> - if (g_str_equal(qtest_get_arch(), "aarch64")) {
> + if (g_str_equal(qtest_get_arch(), "aarch64") && qtest_has_accel("kvm")) {
With this new guard in main(), we should be able to drop the tcg fallback
in MACHINE_KVM. But, to do that, we also need to guard the call to
sve_tests_sve_off_kvm().
> qtest_add_data_func("/arm/kvm/query-cpu-model-expansion",
> NULL, test_query_cpu_model_expansion_kvm);
> }
> --
> 2.26.3
>
And, shouldn't we also guard all the other tests, which require tcg, with
qtest_has_accel("tcg")?
Thanks,
drew
- [PATCH v3 1/6] accel: Introduce 'query-accels' QMP command, (continued)
- [PATCH v3 1/6] accel: Introduce 'query-accels' QMP command, Philippe Mathieu-Daudé, 2021/04/15
- [PATCH v3 2/6] tests/qtest: Add qtest_has_accel() method, Philippe Mathieu-Daudé, 2021/04/15
- [PATCH v3 3/6] qtest/bios-tables-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/15
- [PATCH v3 4/6] qtest/qmp-cmd-test: Make test build-independent from accelerator, Philippe Mathieu-Daudé, 2021/04/15
- [PATCH v3 5/6] tests/qtest: Do not restrict bios-tables-test to Aarch64 hosts anymore, Philippe Mathieu-Daudé, 2021/04/15
- [PATCH v3 6/6] tests/meson: Only build softfloat objects if TCG is selected, Philippe Mathieu-Daudé, 2021/04/15
- Re: [PATCH v3 0/6] qtests: Check accelerator available at runtime via QMP 'query-accels', Claudio Fontana, 2021/04/15
- [PATCH v3 7/6] qtest/arm-cpu-features: Use generic qtest_has_accel() to check for KVM, Philippe Mathieu-Daudé, 2021/04/15
- Re: [PATCH v3 7/6] qtest/arm-cpu-features: Use generic qtest_has_accel() to check for KVM,
Andrew Jones <=