test_vmptrld was actually invoking vmclear. make_vmcs_current is
the wrapper for vmptrld, not vmcs_clear!
Signed-off-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>
---
x86/vmx.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/x86/vmx.c b/x86/vmx.c
index cc2598f..ef9caca 100644
--- a/x86/vmx.c
+++ b/x86/vmx.c
@@ -702,18 +702,18 @@ static void test_vmptrld(void)
/* Unaligned page access */
tmp_root = (struct vmcs *)((intptr_t)vmcs + 1);
report("test vmptrld with unaligned vmcs",
- vmcs_clear(tmp_root) == 1);
+ make_vmcs_current(tmp_root) == 1);
/* gpa bits beyond physical address width are set*/
tmp_root = (struct vmcs *)((intptr_t)vmcs |
((u64)1 << (width+1)));
report("test vmptrld with vmcs address bits set beyond physical address width",
- vmcs_clear(tmp_root) == 1);
+ make_vmcs_current(tmp_root) == 1);
/* Pass VMXON region */
tmp_root = (struct vmcs *)vmxon_region;
report("test vmptrld with vmxon region",
- vmcs_clear(tmp_root) == 1);
+ make_vmcs_current(tmp_root) == 1);
report("test vmptrld with valid vmcs region", make_vmcs_current(vmcs) == 0);
}
--
1.8.3.1
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
