mbox series

[0/3] accel: Factor tcg_cpu_reset_hold() out of cpu-common.c

Message ID 20230918104153.24433-1-philmd@linaro.org
Headers show
Series accel: Factor tcg_cpu_reset_hold() out of cpu-common.c | expand

Message

Philippe Mathieu-Daudé Sept. 18, 2023, 10:41 a.m. UTC 
Hi,

We want to have exec/ code agnostic to accelerators.
still we use various call to TCG and KVM. This series
factor the TCG code from cpu_common_reset_hold() to an
accel-specific handler within AccelOpsClass.

Based-on: <20230915190009.68404-1-philmd@linaro.org>

Philippe Mathieu-Daudé (3):
  accel/tcg: Declare tcg_flush_jmp_cache() in 'exec/tb-flush.h'
  accel: Introduce cpu_exec_reset_hold()
  accel/tcg: Factor tcg_cpu_reset_hold() out

 include/exec/cpu-common.h  | 3 ---
 include/exec/tb-flush.h    | 2 ++
 include/hw/core/cpu.h      | 1 +
 include/sysemu/accel-ops.h | 1 +
 accel/stubs/tcg-stub.c     | 4 ----
 accel/tcg/cputlb.c         | 1 +
 accel/tcg/tcg-accel-ops.c  | 9 +++++++++
 accel/tcg/translate-all.c  | 8 --------
 accel/tcg/user-exec-stub.c | 4 ++++
 hw/core/cpu-common.c       | 5 +----
 plugins/core.c             | 1 -
 softmmu/cpus.c             | 7 +++++++
 12 files changed, 26 insertions(+), 20 deletions(-)

Comments

Anton Johansson Sept. 18, 2023, 11:42 a.m. UTC | #1 
On 18/09/23, Philippe Mathieu-Daudé wrote:
> Hi,
> 
> We want to have exec/ code agnostic to accelerators.
> still we use various call to TCG and KVM. This series
> factor the TCG code from cpu_common_reset_hold() to an
> accel-specific handler within AccelOpsClass.
> 
> Based-on: <20230915190009.68404-1-philmd@linaro.org>
> 
> Philippe Mathieu-Daudé (3):
>   accel/tcg: Declare tcg_flush_jmp_cache() in 'exec/tb-flush.h'
>   accel: Introduce cpu_exec_reset_hold()
>   accel/tcg: Factor tcg_cpu_reset_hold() out
> 
>  include/exec/cpu-common.h  | 3 ---
>  include/exec/tb-flush.h    | 2 ++
>  include/hw/core/cpu.h      | 1 +
>  include/sysemu/accel-ops.h | 1 +
>  accel/stubs/tcg-stub.c     | 4 ----
>  accel/tcg/cputlb.c         | 1 +
>  accel/tcg/tcg-accel-ops.c  | 9 +++++++++
>  accel/tcg/translate-all.c  | 8 --------
>  accel/tcg/user-exec-stub.c | 4 ++++
>  hw/core/cpu-common.c       | 5 +----
>  plugins/core.c             | 1 -
>  softmmu/cpus.c             | 7 +++++++
>  12 files changed, 26 insertions(+), 20 deletions(-)
> 
> -- 
> 2.41.0
> 
Series:
Reviewed-by: Anton Johansson <anjo@rev.ng>
Richard Henderson Sept. 19, 2023, 2:47 p.m. UTC | #2 
On 9/18/23 12:41, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (3):
>    accel/tcg: Declare tcg_flush_jmp_cache() in 'exec/tb-flush.h'
>    accel: Introduce cpu_exec_reset_hold()
>    accel/tcg: Factor tcg_cpu_reset_hold() out

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~