Re: [PATCH v6 0/5] clean up redundant 'kvm_run' parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Paolo,

Any opinion on this series patches? Can I help with this patchset ?

Thanks and best,
Tianjia

On 2020/6/23 21:14, Tianjia Zhang wrote:
In the current kvm version, 'kvm_run' has been included in the 'kvm_vcpu'
structure. For historical reasons, many kvm-related function parameters
retain the 'kvm_run' and 'kvm_vcpu' parameters at the same time. This
patch does a unified cleanup of these remaining redundant parameters.

This series of patches has completely cleaned the architecture of
arm64, mips, ppc, and s390 (no such redundant code on x86). Due to
the large number of modified codes, a separate patch is made for each
platform. On the ppc platform, there is also a redundant structure
pointer of 'kvm_run' in 'vcpu_arch', which has also been cleaned
separately.

---
v6 changes:
   Rearrange patch sets, only keep the unmerged patch.
   rebase on mainline.

v5 change:
   ppc: fix for review.

v4 change:
   mips: fixes two errors in entry.c.

v3 change:
   Keep the existing `vcpu->run` in the function body unchanged.

v2 change:
   s390 retains the original variable name and minimizes modification.

Tianjia Zhang (5):
   KVM: s390: clean up redundant 'kvm_run' parameters
   KVM: arm64: clean up redundant 'kvm_run' parameters
   KVM: PPC: clean up redundant kvm_run parameters in assembly
   KVM: MIPS: clean up redundant 'kvm_run' parameters
   KVM: MIPS: clean up redundant kvm_run parameters in assembly

  arch/arm64/include/asm/kvm_coproc.h   |  12 +--
  arch/arm64/include/asm/kvm_host.h     |  11 +--
  arch/arm64/include/asm/kvm_mmu.h      |   2 +-
  arch/arm64/kvm/arm.c                  |   6 +-
  arch/arm64/kvm/handle_exit.c          |  36 ++++----
  arch/arm64/kvm/mmio.c                 |  11 +--
  arch/arm64/kvm/mmu.c                  |   5 +-
  arch/arm64/kvm/sys_regs.c             |  13 ++-
  arch/mips/include/asm/kvm_host.h      |  32 ++------
  arch/mips/kvm/emulate.c               |  59 +++++--------
  arch/mips/kvm/entry.c                 |  21 ++---
  arch/mips/kvm/mips.c                  |  14 ++--
  arch/mips/kvm/trap_emul.c             | 114 +++++++++++---------------
  arch/mips/kvm/vz.c                    |  26 +++---
  arch/powerpc/include/asm/kvm_ppc.h    |   2 +-
  arch/powerpc/kvm/book3s_interrupts.S  |  22 +++--
  arch/powerpc/kvm/book3s_pr.c          |   9 +-
  arch/powerpc/kvm/booke.c              |   9 +-
  arch/powerpc/kvm/booke_interrupts.S   |   9 +-
  arch/powerpc/kvm/bookehv_interrupts.S |  10 +--
  arch/s390/kvm/kvm-s390.c              |  23 ++++--
  21 files changed, 188 insertions(+), 258 deletions(-)




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux