Re: [PATCH v1 0/9] KVM selftests for s390x
From: Christian Borntraeger
Date: Tue Jun 04 2019 - 13:41:34 EST
On 04.06.19 19:19, Paolo Bonzini wrote:
> On 23/05/19 18:43, Thomas Huth wrote:
>> This patch series enables the KVM selftests for s390x. As a first
>> test, the sync_regs from x86 has been adapted to s390x, and after
>> a fix for KVM_CAP_MAX_VCPU_ID on s390x, the kvm_create_max_vcpus
>> is now enabled here, too.
>>
>> Please note that the ucall() interface is not used yet - since
>> s390x neither has PIO nor MMIO, this needs some more work first
>> before it becomes usable (we likely should use a DIAG hypercall
>> here, which is what the sync_reg test is currently using, too...
>> I started working on that topic, but did not finish that work
>> yet, so I decided to not include it yet).
>
> Christian, please include this in your tree (rebasing on top of kvm/next
> as soon as I push it). Note that Thomas is away for about a month.
Will do. Right now it is part of my next tree on top of rc3.
>
> Paolo
>
>> RFC -> v1:
>> - Rebase, needed to add the first patch for vcpu_nested_state_get/set
>> - Added patch to introduce VM_MODE_DEFAULT macro
>> - Improved/cleaned up the code in processor.c
>> - Added patch to fix KVM_CAP_MAX_VCPU_ID on s390x
>> - Added patch to enable the kvm_create_max_vcpus on s390x and aarch64
>>
>> Andrew Jones (1):
>> kvm: selftests: aarch64: fix default vm mode
>>
>> Thomas Huth (8):
>> KVM: selftests: Wrap vcpu_nested_state_get/set functions with x86
>> guard
>> KVM: selftests: Guard struct kvm_vcpu_events with
>> __KVM_HAVE_VCPU_EVENTS
>> KVM: selftests: Introduce a VM_MODE_DEFAULT macro for the default bits
>> KVM: selftests: Align memory region addresses to 1M on s390x
>> KVM: selftests: Add processor code for s390x
>> KVM: selftests: Add the sync_regs test for s390x
>> KVM: s390: Do not report unusabled IDs via KVM_CAP_MAX_VCPU_ID
>> KVM: selftests: Move kvm_create_max_vcpus test to generic code
>>
>> MAINTAINERS | 2 +
>> arch/mips/kvm/mips.c | 3 +
>> arch/powerpc/kvm/powerpc.c | 3 +
>> arch/s390/kvm/kvm-s390.c | 1 +
>> arch/x86/kvm/x86.c | 3 +
>> tools/testing/selftests/kvm/Makefile | 7 +-
>> .../testing/selftests/kvm/include/kvm_util.h | 10 +
>> .../selftests/kvm/include/s390x/processor.h | 22 ++
>> .../kvm/{x86_64 => }/kvm_create_max_vcpus.c | 3 +-
>> .../selftests/kvm/lib/aarch64/processor.c | 2 +-
>> tools/testing/selftests/kvm/lib/kvm_util.c | 25 +-
>> .../selftests/kvm/lib/s390x/processor.c | 286 ++++++++++++++++++
>> .../selftests/kvm/lib/x86_64/processor.c | 2 +-
>> .../selftests/kvm/s390x/sync_regs_test.c | 151 +++++++++
>> virt/kvm/arm/arm.c | 3 +
>> virt/kvm/kvm_main.c | 2 -
>> 16 files changed, 514 insertions(+), 11 deletions(-)
>> create mode 100644 tools/testing/selftests/kvm/include/s390x/processor.h
>> rename tools/testing/selftests/kvm/{x86_64 => }/kvm_create_max_vcpus.c (93%)
>> create mode 100644 tools/testing/selftests/kvm/lib/s390x/processor.c
>> create mode 100644 tools/testing/selftests/kvm/s390x/sync_regs_test.c
>>
>