[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v6 0/8] Unified CPU type check
From: |
Gavin Shan |
Subject: |
[PATCH v6 0/8] Unified CPU type check |
Date: |
Mon, 20 Nov 2023 10:27:16 +1000 |
This series bases on Phil's repository because the prepatory commits
have been queued to the branch.
https://gitlab.com/philmd/qemu.git (branch: cpus-next)
There are two places where the user specified CPU type is checked to see
if it's supported or allowed by the board: machine_run_board_init() and
mc->init(). We don't have to maintain two duplicate sets of logic. This
series intends to move the check to machine_run_board_init() so that we
have unified CPU type check.
PATCH[1-3] refactors the logic to validate CPU type in machine_run_board_init()
PATCH[4-8] validates the CPU type in machine_run_board_init() for the
individual boards
v1: https://lists.nongnu.org/archive/html/qemu-arm/2023-07/msg00302.html
v2: https://lists.nongnu.org/archive/html/qemu-arm/2023-07/msg00528.html
v3: https://lists.nongnu.org/archive/html/qemu-arm/2023-09/msg00157.html
v4: https://lists.nongnu.org/archive/html/qemu-arm/2023-11/msg00005.html
v5: https://lists.nongnu.org/archive/html/qemu-arm/2023-11/msg00611.html
Testing
=======
With the following command lines, the output messages are varied before
and after the series is applied.
./build/qemu-system-aarch64 \
-accel tcg -machine virt,gic-version=3 \
-cpu cortex-a8 -smp maxcpus=2,cpus=1
Before the series is applied:
qemu-system-aarch64: mach-virt: CPU type cortex-a8-arm-cpu not supported
After the series is applied:
qemu-system-aarch64: Invalid CPU type: cortex-a8-arm-cpu
The valid models are: cortex-a7, cortex-a15, cortex-a35, cortex-a55,
cortex-a72, cortex-a76, a64fx, neoverse-n1,
neoverse-v1, cortex-a53, cortex-a57, max
Changelog
=========
v6:
* Drop PATCH[v5 01-23], queued by Phil (Phil)
* Clearer hint if only one CPU type is supported and have
'const MachineState *' in is_cpu_type_supported() (Phil)
* Move valid_cpu_types[] to board's class_init() function (Phil)
v5:
* PATCH[v5 01] to remove CPU class 'ev67' for alpha (Ricard/Igor)
* PATCH[v5 02] to remove object_class_is_abstract() for hppa (Gavin)
* Don't move cpu_class_by_name() (Richard)
* PATCH[v5 04] to remove 'oc == NULL' since the check has
been covered in object_class_dynamic_cast() (Igor)
* Introduce generic cpu_list(), shared by most of the targets (Richard)
* Use g_str_has_suffix and g_auto_free (Richard)
* Collect r-bs from Igor and Richard (Gavin)
v4:
* Integrate Philippe's patches where cpu_class_by_name()
is consolidated and my duplicate code is dropped (Philippe)
* Simplified changelog and improvements (Thomas)
* g_assert() on the return value from cpu_model_from_type()
in is_cpu_type_supported() (Philippe)
* Collected r-bs from Philippe Mathieu-Daudé, Leif Lindholm,
Bastian Koppelmann, Daniel Henrique Barboza, Cédric Le Goater,
Gavin Shan (Gavin)
v3:
* Generic helper cpu_model_from_type() (Igor)
* Apply cpu_model_from_type() to the individual targets (Igor)
* Implement cpu_list() for the missed targets (Gavin)
* Remove mc->valid_cpu_models (Richard)
* Separate patch to constify mc->validate_cpu_types (Gavin)
v2:
* Constify mc->valid_cpu_types (Richard)
* Print the supported CPU models, instead of typenames (Peter)
* Misc improvements for the hleper to do the check (Igor)
* More patches to move the check (Marcin)
Gavin Shan (8):
machine: Use error handling when CPU type is checked
machine: Introduce helper is_cpu_type_supported()
machine: Print CPU model name instead of CPU type
hw/arm/virt: Hide host CPU model for tcg
hw/arm/virt: Check CPU type in machine_run_board_init()
hw/arm/sbsa-ref: Check CPU type in machine_run_board_init()
hw/arm: Check CPU type in machine_run_board_init()
hw/riscv/shakti_c: Check CPU type in machine_run_board_init()
hw/arm/bananapi_m2u.c | 12 ++---
hw/arm/cubieboard.c | 12 ++---
hw/arm/mps2-tz.c | 26 ++++++++---
hw/arm/mps2.c | 26 ++++++++---
hw/arm/msf2-som.c | 12 ++---
hw/arm/musca.c | 12 +++--
hw/arm/npcm7xx_boards.c | 12 +++--
hw/arm/orangepi.c | 12 ++---
hw/arm/sbsa-ref.c | 36 +++++----------
hw/arm/virt.c | 60 ++++++++++---------------
hw/core/machine.c | 97 +++++++++++++++++++++++++----------------
hw/riscv/shakti_c.c | 13 +++---
12 files changed, 173 insertions(+), 157 deletions(-)
--
2.41.0
- [PATCH v6 0/8] Unified CPU type check,
Gavin Shan <=
- [PATCH v6 1/8] machine: Use error handling when CPU type is checked, Gavin Shan, 2023/11/19
- [PATCH v6 2/8] machine: Introduce helper is_cpu_type_supported(), Gavin Shan, 2023/11/19
- [PATCH v6 3/8] machine: Print CPU model name instead of CPU type, Gavin Shan, 2023/11/19
- [PATCH v6 4/8] hw/arm/virt: Hide host CPU model for tcg, Gavin Shan, 2023/11/19
- [PATCH v6 5/8] hw/arm/virt: Check CPU type in machine_run_board_init(), Gavin Shan, 2023/11/19
- [PATCH v6 6/8] hw/arm/sbsa-ref: Check CPU type in machine_run_board_init(), Gavin Shan, 2023/11/19
- [PATCH v6 7/8] hw/arm: Check CPU type in machine_run_board_init(), Gavin Shan, 2023/11/19
- [PATCH v6 8/8] hw/riscv/shakti_c: Check CPU type in machine_run_board_init(), Gavin Shan, 2023/11/19
- Re: [PATCH v6 0/8] Unified CPU type check, Marcin Juszkiewicz, 2023/11/20