KVM: selftests: Fix size of memslots created by Xen tests
authorSean Christopherson <seanjc@google.com>
Wed, 10 Feb 2021 18:26:06 +0000 (10:26 -0800)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 11 Feb 2021 13:02:09 +0000 (08:02 -0500)
For better or worse, the memslot APIs take the number of pages, not the
size in bytes.  The Xen tests need 2 pages, not 8192 pages.

Fixes: 8d4e7e80838f ("KVM: x86: declare Xen HVM shared info capability and add test case")
Cc: David Woodhouse <dwmw@amazon.co.uk>
Signed-off-by: Sean Christopherson <seanjc@google.com>
Message-Id: <20210210182609.435200-3-seanjc@google.com>
Reviewed-by: David Woodhouse <dwmw@amazon.co.uk>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c
tools/testing/selftests/kvm/x86_64/xen_vmcall_test.c

index bdb3feb..cb39639 100644 (file)
@@ -79,8 +79,7 @@ int main(int argc, char *argv[])
 
        /* Map a region for the shared_info page */
        vm_userspace_mem_region_add(vm, VM_MEM_SRC_ANONYMOUS,
-                                    SHINFO_REGION_GPA, SHINFO_REGION_SLOT,
-                                   2 * getpagesize(), 0);
+                                   SHINFO_REGION_GPA, SHINFO_REGION_SLOT, 2, 0);
        virt_map(vm, SHINFO_REGION_GPA, SHINFO_REGION_GPA, 2, 0);
 
        struct kvm_xen_hvm_config hvmc = {
index 8665336..8389e0b 100644 (file)
@@ -102,8 +102,7 @@ int main(int argc, char *argv[])
 
        /* Map a region for the hypercall pages */
        vm_userspace_mem_region_add(vm, VM_MEM_SRC_ANONYMOUS,
-                                    HCALL_REGION_GPA, HCALL_REGION_SLOT,
-                                   2 * getpagesize(), 0);
+                                   HCALL_REGION_GPA, HCALL_REGION_SLOT, 2, 0);
        virt_map(vm, HCALL_REGION_GPA, HCALL_REGION_GPA, 2, 0);
 
        for (;;) {