Assert that VMXON succeeds in the INIT/SIPI tests, _vmx_on() doesn't check the result, i.e. doesn't guarantee success. Signed-off-by: Sean Christopherson <seanjc@xxxxxxxxxx> --- x86/vmx_tests.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c index 1b277cfb..4c963b96 100644 --- a/x86/vmx_tests.c +++ b/x86/vmx_tests.c @@ -9695,7 +9695,7 @@ static void init_signal_test_thread(void *data) u64 *ap_vmxon_region = alloc_page(); enable_vmx(); init_vmx(ap_vmxon_region); - _vmx_on(ap_vmxon_region); + TEST_ASSERT(!_vmx_on(ap_vmxon_region)); /* Signal CPU have entered VMX operation */ vmx_set_test_stage(1); @@ -9743,7 +9743,7 @@ static void init_signal_test_thread(void *data) while (vmx_get_test_stage() != 8) ; /* Enter VMX operation (i.e. exec VMXON) */ - _vmx_on(ap_vmxon_region); + TEST_ASSERT(!_vmx_on(ap_vmxon_region)); /* Signal to BSP we are in VMX operation */ vmx_set_test_stage(9); @@ -9920,7 +9920,7 @@ static void sipi_test_ap_thread(void *data) ap_vmxon_region = alloc_page(); enable_vmx(); init_vmx(ap_vmxon_region); - _vmx_on(ap_vmxon_region); + TEST_ASSERT(!_vmx_on(ap_vmxon_region)); init_vmcs(&ap_vmcs); make_vmcs_current(ap_vmcs); -- 2.36.1.255.ge46751e96f-goog
