[PATCH] kvm-unit-tests: VMX: Comments on the framework and writing test cases

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Add some comments on the framework of nested VMX testing, and guides of
how to write new test cases.

Signed-off-by: Arthur Chunqi Li <yzt356@xxxxxxxxx>
---
 x86/vmx.c       |   25 +++++++++++++++++++++++++
 x86/vmx_tests.c |   13 +++++++++++++
 2 files changed, 38 insertions(+)

diff --git a/x86/vmx.c b/x86/vmx.c
index 9db4ef4..3aa8600 100644
--- a/x86/vmx.c
+++ b/x86/vmx.c
@@ -1,3 +1,28 @@
+/*
+ * x86/vmx.c : Framework for testing nested virtualization
+ *	This is a framework to test nested VMX for KVM, which is
+ * 	a project of GSoC 2013. All test cases are located in
+ *	"vmx_tests", which is defined in x86/vmx_tests.c. All test
+ *	cases should be located in x86/vmx_tests.c and framework
+ *	related functions should be in this file.
+ *
+ * How to write test suite?
+ *	Add functions of test suite in variant "vmx_tests". You can
+ *	write:
+ *		init function used for initializing test suite
+ *		main function for codes running in L2 guest, 
+ *		exit_handler to handle vmexit of L2 to L1 (framework)
+ *		syscall handler to handle L2 syscall vmexit
+ *		vmenter fail handler to handle direct failure of vmenter
+ *		init registers used to store register value in initialization
+ *	If no special function is needed for a test suite, you can use
+ *	basic_* series of functions. More handlers can be added to
+ *	"vmx_tests", see details of "struct vmx_test" and function
+ *	test_run().
+ *
+ * Author : Arthur Chunqi Li <yzt356@xxxxxxxxx>
+ */
+
 #include "libcflat.h"
 #include "processor.h"
 #include "vm.h"
diff --git a/x86/vmx_tests.c b/x86/vmx_tests.c
index 0759e10..5fc16a3 100644
--- a/x86/vmx_tests.c
+++ b/x86/vmx_tests.c
@@ -1,3 +1,8 @@
+/*
+ * All test cases of nested virtualization should be in this file
+ *
+ * Author : Arthur Chunqi Li <yzt356@xxxxxxxxx>
+ */
 #include "vmx.h"
 #include "msr.h"
 #include "processor.h"
@@ -782,6 +787,14 @@ struct insn_table {
 	u32 test_field;
 };
 
+/*
+ * Add more test cases of instruction intercept here. Elements in this
+ * table is:
+ *	name/control flag/insn function/type/exit reason/exit qulification/
+ *	instruction info/field to test
+ * The last field defines which fields (exit_qual and insn_info) need to be
+ * tested in exit handler. If set to 0, only "reason" is checked.
+ */
 static struct insn_table insn_table[] = {
 	// Flags for Primary Processor-Based VM-Execution Controls
 	{"HLT",  CPU_HLT, insn_hlt, INSN_CPU0, 12, 0, 0, 0},
-- 
1.7.9.5

--
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




[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux