Re: [PATCH] cpus: Drop declaration of cpu_remove()

[Laurent Vivier]

Le 14/10/2020 à 15:40, Greg Kurz a écrit :
> cpu_remove() got superseded by cpu_remove_sync() in commit dbadee4ff4a0,
> but its prototype wasn't removed. We could possibly keep the shorter
> cpu_remove() naming but it seems better to highligth that this blocks
> until the CPU thread is joined.
> 
> Fixes: dbadee4ff4a0 ("cpus: join thread when removing a vCPU")
> Cc: pbonzini@redhat.com
> Signed-off-by: Greg Kurz <groug@kaod.org>
> ---
>  include/hw/core/cpu.h |    8 --------
>  1 file changed, 8 deletions(-)
> 
> diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h
> index 4879f2502659..ace443e88297 100644
> --- a/include/hw/core/cpu.h
> +++ b/include/hw/core/cpu.h
> @@ -924,14 +924,6 @@ void cpu_exit(CPUState *cpu);
>  void cpu_resume(CPUState *cpu);
>  
>  /**
> - * cpu_remove:
> - * @cpu: The CPU to remove.
> - *
> - * Requests the CPU to be removed.
> - */
> -void cpu_remove(CPUState *cpu);
> -
> - /**
>   * cpu_remove_sync:
>   * @cpu: The CPU to remove.
>   *
> 
> 
> 
Applied to my trivial-patches branch.

Thanks,
Laurent