[v2] accel/kvm: Cleanups around kvm_arch_put_registers()

[PATCH v2 0/3] accel/kvm: Cleanups around kvm_arch_put_registers()

Philippe Mathieu-Daudé posted 3 patches 1 month, 1 week ago

Diff against v1
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20251008040715.81513-1-philmd@linaro.org

Maintainers: Paolo Bonzini <pbonzini@redhat.com>, Marcelo Tosatti <mtosatti@redhat.com>, Song Gao <gaosong@loongson.cn>, Huacai Chen <chenhuacai@kernel.org>, "Philippe Mathieu-Daudé" <philmd@linaro.org>, Aurelien Jarno <aurelien@aurel32.net>, Jiaxun Yang <jiaxun.yang@flygoat.com>, Aleksandar Rikalo <arikalo@gmail.com>, Nicholas Piggin <npiggin@gmail.com>, Harsh Prateek Bora <harshpb@linux.ibm.com>, Chinmay Rath <rathc@linux.ibm.com>, Palmer Dabbelt <palmer@dabbelt.com>, Alistair Francis <alistair.francis@wdc.com>, Weiwei Li <liwei1518@gmail.com>, Daniel Henrique Barboza <dbarboza@ventanamicro.com>, Liu Zhiwei <zhiwei_liu@linux.alibaba.com>, Halil Pasic <pasic@linux.ibm.com>, Christian Borntraeger <borntraeger@linux.ibm.com>, Eric Farman <farman@linux.ibm.com>, Matthew Rosato <mjrosato@linux.ibm.com>, Richard Henderson <richard.henderson@linaro.org>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Thomas Huth <thuth@redhat.com>

include/system/kvm.h       | 16 +++++++------
accel/kvm/kvm-all.c        | 47 +++++++++++++++-----------------------
target/i386/kvm/kvm.c      |  6 ++---
target/loongarch/kvm/kvm.c |  8 +++----
target/mips/kvm.c          |  6 ++---
target/ppc/kvm.c           |  2 +-
target/riscv/kvm/kvm-cpu.c |  2 +-
target/s390x/kvm/kvm.c     |  2 +-
8 files changed, 41 insertions(+), 48 deletions(-)

Expand all Fold all

[PATCH v2 0/3] accel/kvm: Cleanups around kvm_arch_put_registers()

Posted by Philippe Mathieu-Daudé 1 month, 1 week ago

Extracted from a bigger series aiming to make accelerator
synchronization of vcpu state slightly clearer. Here KVM
patches around kvm_arch_put_registers():
- Move KVM_PUT_[RESET|RUNTIME|FULL]_STATE to an enum
- Factor common code out of kvm_cpu_synchronize_post_*()

Philippe Mathieu-Daudé (3):
  accel/kvm: Do not expect more then KVM_PUT_FULL_STATE
  accel/kvm: Introduce KvmPutState enum
  accel/kvm: Factor kvm_cpu_synchronize_put() out

 include/system/kvm.h       | 16 +++++++------
 accel/kvm/kvm-all.c        | 47 +++++++++++++++-----------------------
 target/i386/kvm/kvm.c      |  6 ++---
 target/loongarch/kvm/kvm.c |  8 +++----
 target/mips/kvm.c          |  6 ++---
 target/ppc/kvm.c           |  2 +-
 target/riscv/kvm/kvm-cpu.c |  2 +-
 target/s390x/kvm/kvm.c     |  2 +-
 8 files changed, 41 insertions(+), 48 deletions(-)

-- 
2.51.0

Re: [PATCH v2 0/3] accel/kvm: Cleanups around kvm_arch_put_registers()

Posted by Paolo Bonzini 1 month ago

Queued patches 2-3, thanks.

For patch 1, I think it's better to have consistency in the use of >=.

Paolo

Re: [PATCH v2 0/3] accel/kvm: Cleanups around kvm_arch_put_registers()

Posted by Andrew Jones 1 month, 1 week ago

On Wed, Oct 08, 2025 at 06:07:11AM +0200, Philippe Mathieu-Daudé wrote:
> Extracted from a bigger series aiming to make accelerator
> synchronization of vcpu state slightly clearer. Here KVM
> patches around kvm_arch_put_registers():
> - Move KVM_PUT_[RESET|RUNTIME|FULL]_STATE to an enum
> - Factor common code out of kvm_cpu_synchronize_post_*()
> 
> Philippe Mathieu-Daudé (3):
>   accel/kvm: Do not expect more then KVM_PUT_FULL_STATE
>   accel/kvm: Introduce KvmPutState enum
>   accel/kvm: Factor kvm_cpu_synchronize_put() out
> 
>  include/system/kvm.h       | 16 +++++++------
>  accel/kvm/kvm-all.c        | 47 +++++++++++++++-----------------------
>  target/i386/kvm/kvm.c      |  6 ++---
>  target/loongarch/kvm/kvm.c |  8 +++----
>  target/mips/kvm.c          |  6 ++---
>  target/ppc/kvm.c           |  2 +-
>  target/riscv/kvm/kvm-cpu.c |  2 +-
>  target/s390x/kvm/kvm.c     |  2 +-
>  8 files changed, 41 insertions(+), 48 deletions(-)
> 
> -- 
> 2.51.0
> 
>

For the series,

Reviewed-by: Andrew Jones <ajones@ventanamicro.com>