[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v11 06/13] tests/qtest: Adjust and document query-cpu-model-e
From: |
Fabiano Rosas |
Subject: |
Re: [PATCH v11 06/13] tests/qtest: Adjust and document query-cpu-model-expansion test for arm |
Date: |
Thu, 27 Apr 2023 10:16:48 -0300 |
Richard Henderson <richard.henderson@linaro.org> writes:
> On 4/26/23 19:00, Fabiano Rosas wrote:
>> We're about to move the 32-bit CPUs under CONFIG_TCG, so adjust the
>> query-cpu-model-expansion test to check against the cortex-a7, which
>> is already under CONFIG_TCG. That allows the next patch to contain
>> only code movement.
>>
>> While here add comments clarifying what we're testing.
>>
>> Signed-off-by: Fabiano Rosas<farosas@suse.de>
>> Suggested-by: Philippe Mathieu-Daudé<philmd@linaro.org>
>> ---
>> tests/qtest/arm-cpu-features.c | 20 +++++++++++++++++---
>> 1 file changed, 17 insertions(+), 3 deletions(-)
>
> I don't see why you're changing the cpu model here.
> Neither cpu will work, of course, but why change?
>
Because there's already a patch in master that puts the cortex-a7 under
CONFIG_TCG, so I can have the whole if/else in this patch.
If I keep the cortex-a15, this change needs to go into the next patch
("move cpu_tcg to tcg/cpu32.c") which moves the rest of the 32bit cpus,
which was supposed to be only code movement.
- [PATCH v11 02/13] target/arm: Remove dead code from cpu_max_set_sve_max_vq, (continued)
- [PATCH v11 02/13] target/arm: Remove dead code from cpu_max_set_sve_max_vq, Fabiano Rosas, 2023/04/26
- [PATCH v11 01/13] target/arm: Move cortex sysregs into a separate file, Fabiano Rosas, 2023/04/26
- [PATCH v11 04/13] target/arm: Do not expose all -cpu max features to qtests, Fabiano Rosas, 2023/04/26
- [PATCH v11 03/13] target/arm: Extract TCG -cpu max code into a function, Fabiano Rosas, 2023/04/26
- [PATCH v11 06/13] tests/qtest: Adjust and document query-cpu-model-expansion test for arm, Fabiano Rosas, 2023/04/26
[PATCH v11 05/13] target/arm: Move 64-bit TCG CPUs into tcg/, Fabiano Rosas, 2023/04/26
[PATCH v11 07/13] target/arm: move cpu_tcg to tcg/cpu32.c, Fabiano Rosas, 2023/04/26
[PATCH v11 09/13] tests/avocado: Pass parameters to migration test, Fabiano Rosas, 2023/04/26
[PATCH v11 08/13] tests/qtest: Fix tests when no KVM or TCG are present, Fabiano Rosas, 2023/04/26