On 10/07/20 09:32, Tianjia Zhang wrote: > Hi Paolo, > > Any opinion on this series patches? Can I help with this patchset ? I was hoping to have some Tested-by, for now I'm queuing patches 1 and 2. Thanks, Paolo > 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(-) >> >
