[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC v8 15/27] cpu: Introduce TCGCpuOperations struct
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [RFC v8 15/27] cpu: Introduce TCGCpuOperations struct |
Date: |
Sat, 5 Dec 2020 17:33:03 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.4.0 |
On 12/5/20 5:15 PM, Claudio Fontana wrote:
> From: Eduardo Habkost <ehabkost@redhat.com>
>
> The TCG-specific CPU methods will be moved to a separate struct,
> to make it easier to move accel-specific code outside generic CPU
> code in the future. Start by moving tcg_initialize().
>
> The new CPUClass.tcg_opts field may eventually become a pointer,
> but keep it an embedded struct for now, to make code conversion
> easier.
>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
>
> [claudio: make the tcg code build for CONFIG_TCG only]
>
> Signed-off-by: Claudio Fontana <cfontana@suse.de>
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
> MAINTAINERS | 1 +
> cpu.c | 6 +++++-
> include/hw/core/cpu.h | 9 ++++++++-
> include/hw/core/tcg-cpu-ops.h | 25 +++++++++++++++++++++++++
> target/alpha/cpu.c | 2 +-
> target/arm/cpu.c | 2 +-
> target/avr/cpu.c | 2 +-
> target/cris/cpu.c | 12 ++++++------
> target/hppa/cpu.c | 2 +-
> target/i386/tcg-cpu.c | 2 +-
> target/lm32/cpu.c | 2 +-
> target/m68k/cpu.c | 2 +-
> target/microblaze/cpu.c | 2 +-
> target/mips/cpu.c | 2 +-
> target/moxie/cpu.c | 2 +-
> target/nios2/cpu.c | 2 +-
> target/openrisc/cpu.c | 2 +-
> target/ppc/translate_init.c.inc | 2 +-
> target/riscv/cpu.c | 2 +-
> target/rx/cpu.c | 2 +-
> target/s390x/cpu.c | 2 +-
> target/sh4/cpu.c | 2 +-
> target/sparc/cpu.c | 2 +-
> target/tilegx/cpu.c | 2 +-
> target/tricore/cpu.c | 2 +-
> target/unicore32/cpu.c | 2 +-
> target/xtensa/cpu.c | 2 +-
> 27 files changed, 67 insertions(+), 30 deletions(-)
> create mode 100644 include/hw/core/tcg-cpu-ops.h
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f53f2678d8..d876f504a6 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1535,6 +1535,7 @@ F: qapi/machine.json
> F: qapi/machine-target.json
> F: include/hw/boards.h
> F: include/hw/core/cpu.h
> +F: include/hw/core/tcg-cpu-ops.h
> F: include/hw/cpu/cluster.h
> F: include/sysemu/numa.h
> T: git https://github.com/ehabkost/qemu.git machine-next
> diff --git a/cpu.c b/cpu.c
> index 0be5dcb6f3..27ad096cc4 100644
> --- a/cpu.c
> +++ b/cpu.c
> @@ -174,14 +174,18 @@ void cpu_exec_initfn(CPUState *cpu)
> void cpu_exec_realizefn(CPUState *cpu, Error **errp)
> {
> CPUClass *cc = CPU_GET_CLASS(cpu);
> +#ifdef CONFIG_TCG
> static bool tcg_target_initialized;
> +#endif /* CONFIG_TCG */
Maybe worth extract as tcg_target_initialized() method.
>
> cpu_list_add(cpu);
>
> +#ifdef CONFIG_TCG
> if (tcg_enabled() && !tcg_target_initialized) {
> tcg_target_initialized = true;
> - cc->tcg_initialize();
> + cc->tcg_ops.initialize();
> }
> +#endif /* CONFIG_TCG */
> tlb_init(cpu);
[...]
- [RFC v8 02/27] accel/tcg: split tcg_start_vcpu_thread, (continued)
- [RFC v8 02/27] accel/tcg: split tcg_start_vcpu_thread, Claudio Fontana, 2020/12/05
- [RFC v8 05/27] i386: move whpx accel files into whpx/, Claudio Fontana, 2020/12/05
- [RFC v8 06/27] i386: move hax accel files into hax/, Claudio Fontana, 2020/12/05
- [RFC v8 07/27] i386: hvf: remove stale MAINTAINERS entry for old hvf stubs, Claudio Fontana, 2020/12/05
- [RFC v8 08/27] i386: move TCG accel files into tcg/, Claudio Fontana, 2020/12/05
- [RFC v8 01/27] accel/tcg: split CpusAccel into three TCG variants, Claudio Fontana, 2020/12/05
- [RFC v8 03/27] accel/tcg: rename tcg-cpus functions to match module name, Claudio Fontana, 2020/12/05
- [RFC v8 09/27] i386: move cpu dump out of helper.c into cpu-dump.c, Claudio Fontana, 2020/12/05
- [RFC v8 11/27] tcg: cpu_exec_{enter,exit} helpers, Claudio Fontana, 2020/12/05
- [RFC v8 15/27] cpu: Introduce TCGCpuOperations struct, Claudio Fontana, 2020/12/05
- Re: [RFC v8 15/27] cpu: Introduce TCGCpuOperations struct,
Philippe Mathieu-Daudé <=
- [RFC v8 17/27] cpu: Move synchronize_from_tb() to tcg_ops, Claudio Fontana, 2020/12/05
- [RFC v8 19/27] cpu: Move tlb_fill to tcg_ops, Claudio Fontana, 2020/12/05
- [RFC v8 12/27] tcg: make CPUClass.cpu_exec_* optional, Claudio Fontana, 2020/12/05
- [RFC v8 10/27] i386: move TCG cpu class initialization out of helper.c, Claudio Fontana, 2020/12/05
- [RFC v8 26/27] hw/core/cpu: call qemu_init_vcpu in cpu_common_realizefn, Claudio Fontana, 2020/12/05
- [RFC v8 23/27] accel: introduce AccelCPUClass extending CPUClass, Claudio Fontana, 2020/12/05
- [RFC v8 24/27] i386: split cpu accelerators from cpu.c, using AccelCPUClass, Claudio Fontana, 2020/12/05
- [RFC v8 27/27] cpu: introduce cpu_accel_instance_init, Claudio Fontana, 2020/12/05
- [RFC v8 14/27] cpu: Remove unnecessary noop methods, Claudio Fontana, 2020/12/05