[v2] accel: Preparatory cleanups for split-accel

[RFC PATCH v2 26/48] accel/system: Document cpu_synchronize_state_post_init/reset()

Posted by Philippe Mathieu-Daudé 4 months, 3 weeks ago

Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
---
 include/system/accel-ops.h | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/include/system/accel-ops.h b/include/system/accel-ops.h
index f40098c1c92..b1b9dce27d0 100644
--- a/include/system/accel-ops.h
+++ b/include/system/accel-ops.h
@@ -43,6 +43,14 @@ struct AccelOpsClass {
     void (*kick_vcpu_thread)(CPUState *cpu);
     bool (*cpu_thread_is_idle)(CPUState *cpu);
 
+    /**
+     * synchronize_post_reset:
+     * synchronize_post_init:
+     * @cpu: The vCPU to synchronize.
+     *
+     * Request to synchronize QEMU vCPU registers to the hardware accelerator
+     * (QEMU is the reference).
+     */
     void (*synchronize_post_reset)(CPUState *cpu);
     void (*synchronize_post_init)(CPUState *cpu);
     /**
-- 
2.49.0

Re: [RFC PATCH v2 26/48] accel/system: Document cpu_synchronize_state_post_init/reset()

Posted by Richard Henderson 4 months, 3 weeks ago

On 6/20/25 10:13, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---
>   include/system/accel-ops.h | 8 ++++++++
>   1 file changed, 8 insertions(+)
> 
> diff --git a/include/system/accel-ops.h b/include/system/accel-ops.h
> index f40098c1c92..b1b9dce27d0 100644
> --- a/include/system/accel-ops.h
> +++ b/include/system/accel-ops.h
> @@ -43,6 +43,14 @@ struct AccelOpsClass {
>       void (*kick_vcpu_thread)(CPUState *cpu);
>       bool (*cpu_thread_is_idle)(CPUState *cpu);
>   
> +    /**
> +     * synchronize_post_reset:
> +     * synchronize_post_init:
> +     * @cpu: The vCPU to synchronize.
> +     *
> +     * Request to synchronize QEMU vCPU registers to the hardware accelerator
> +     * (QEMU is the reference).
> +     */
>       void (*synchronize_post_reset)(CPUState *cpu);
>       void (*synchronize_post_init)(CPUState *cpu);
>       /**

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

r~