Re: [PATCH v5 00/31] Unified CPU type check

[Philippe Mathieu-Daudé]

Hi Gavin,

On 15/11/23 00:55, Gavin Shan wrote:

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.

Gavin Shan (30):
   target/alpha: Remove 'ev67' CPU class
   target/hppa: Remove object_class_is_abstract()
   target: Remove 'oc == NULL' check
   cpu: Add helper cpu_model_from_type()
   cpu: Add generic cpu_list()
   target/alpha: Use generic cpu_list()
   target/arm: Use generic cpu_list()
   target/avr: Use generic cpu_list()
   target/cris: Use generic cpu_list()
   target/hexagon: Use generic cpu_list()
   target/hppa: Use generic cpu_list()
   target/loongarch: Use generic cpu_list()
   target/m68k: Use generic cpu_list()
   target/mips: Use generic cpu_list()
   target/openrisc: Use generic cpu_list()
   target/riscv: Use generic cpu_list()
   target/rx: Use generic cpu_list()
   target/sh4: Use generic cpu_list()
   target/tricore: Use generic cpu_list()
   target/xtensa: Use generic cpu_list()
   target: Use generic cpu_model_from_type()
   machine: Constify MachineClass::valid_cpu_types[i]

I'm queuing patches 1-3 & 5-23 to my cpus-next tree. No need to
repost them, please base them on my tree. I'll follow up with the
branch link when I finish my testing and push it.

   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 name
   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()