Skip to content

Commit 0bd2d3f

Browse files
Maxim Levitskybonzini
authored andcommitted
kvm: selftests: add svm nested shutdown test
Add test that tests that on SVM if L1 doesn't intercept SHUTDOWN, then L2 crashes L1 and doesn't crash L2 Signed-off-by: Maxim Levitsky <mlevitsk@redhat.com> Message-Id: <20221103141351.50662-7-mlevitsk@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
1 parent fc6392d commit 0bd2d3f

File tree

3 files changed

+69
-0
lines changed

3 files changed

+69
-0
lines changed

tools/testing/selftests/kvm/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
/x86_64/svm_vmcall_test
4242
/x86_64/svm_int_ctl_test
4343
/x86_64/svm_nested_soft_inject_test
44+
/x86_64/svm_nested_shutdown_test
4445
/x86_64/sync_regs_test
4546
/x86_64/tsc_msrs_test
4647
/x86_64/tsc_scaling_sync

tools/testing/selftests/kvm/Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,6 +101,7 @@ TEST_GEN_PROGS_x86_64 += x86_64/state_test
101101
TEST_GEN_PROGS_x86_64 += x86_64/vmx_preemption_timer_test
102102
TEST_GEN_PROGS_x86_64 += x86_64/svm_vmcall_test
103103
TEST_GEN_PROGS_x86_64 += x86_64/svm_int_ctl_test
104+
TEST_GEN_PROGS_x86_64 += x86_64/svm_nested_shutdown_test
104105
TEST_GEN_PROGS_x86_64 += x86_64/svm_nested_soft_inject_test
105106
TEST_GEN_PROGS_x86_64 += x86_64/tsc_scaling_sync
106107
TEST_GEN_PROGS_x86_64 += x86_64/sync_regs_test
Lines changed: 67 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,67 @@
1+
// SPDX-License-Identifier: GPL-2.0-only
2+
/*
3+
* svm_nested_shutdown_test
4+
*
5+
* Copyright (C) 2022, Red Hat, Inc.
6+
*
7+
* Nested SVM testing: test that unintercepted shutdown in L2 doesn't crash the host
8+
*/
9+
10+
#include "test_util.h"
11+
#include "kvm_util.h"
12+
#include "processor.h"
13+
#include "svm_util.h"
14+
15+
static void l2_guest_code(struct svm_test_data *svm)
16+
{
17+
__asm__ __volatile__("ud2");
18+
}
19+
20+
static void l1_guest_code(struct svm_test_data *svm, struct idt_entry *idt)
21+
{
22+
#define L2_GUEST_STACK_SIZE 64
23+
unsigned long l2_guest_stack[L2_GUEST_STACK_SIZE];
24+
struct vmcb *vmcb = svm->vmcb;
25+
26+
generic_svm_setup(svm, l2_guest_code,
27+
&l2_guest_stack[L2_GUEST_STACK_SIZE]);
28+
29+
vmcb->control.intercept &= ~(BIT(INTERCEPT_SHUTDOWN));
30+
31+
idt[6].p = 0; // #UD is intercepted but its injection will cause #NP
32+
idt[11].p = 0; // #NP is not intercepted and will cause another
33+
// #NP that will be converted to #DF
34+
idt[8].p = 0; // #DF will cause #NP which will cause SHUTDOWN
35+
36+
run_guest(vmcb, svm->vmcb_gpa);
37+
38+
/* should not reach here */
39+
GUEST_ASSERT(0);
40+
}
41+
42+
int main(int argc, char *argv[])
43+
{
44+
struct kvm_vcpu *vcpu;
45+
struct kvm_run *run;
46+
vm_vaddr_t svm_gva;
47+
struct kvm_vm *vm;
48+
49+
TEST_REQUIRE(kvm_cpu_has(X86_FEATURE_SVM));
50+
51+
vm = vm_create_with_one_vcpu(&vcpu, l1_guest_code);
52+
vm_init_descriptor_tables(vm);
53+
vcpu_init_descriptor_tables(vcpu);
54+
55+
vcpu_alloc_svm(vm, &svm_gva);
56+
57+
vcpu_args_set(vcpu, 2, svm_gva, vm->idt);
58+
run = vcpu->run;
59+
60+
vcpu_run(vcpu);
61+
TEST_ASSERT(run->exit_reason == KVM_EXIT_SHUTDOWN,
62+
"Got exit_reason other than KVM_EXIT_SHUTDOWN: %u (%s)\n",
63+
run->exit_reason,
64+
exit_reason_str(run->exit_reason));
65+
66+
kvm_vm_free(vm);
67+
}

0 commit comments

Comments
 (0)