On 21.08.2014 17:24, Paolo Bonzini wrote:
It seems to work for VMX (see the testcase I just sent). For SVM, can you
check if this test works for you, so that we can work on a simple testcase?
I was able to reproduce the bug with your testcase when I changed APIC
register access size (see below). Please check if it fails on VMX as
well now.
On a side note, npt_rsvd also seem to be broken as I mentioned previously.
HTH,
Valentine
diff --git a/x86/svm.c b/x86/svm.c
index a9b29b1..d0ddff7 100644
--- a/x86/svm.c
+++ b/x86/svm.c
@@ -797,6 +797,27 @@ static bool npt_pfwalk_check(struct test *test)
&& (test->vmcb->control.exit_info_2 == read_cr3());
}
+static void npt_l1mmio_prepare(struct test *test)
+{
+ vmcb_ident(test->vmcb);
+}
+
+u32 nested_apic_version;
+
+static void npt_l1mmio_test(struct test *test)
+{
+ u32 *data = (u32*)(0xfee00030UL);
+
+ nested_apic_version = *data;
+}
+
+static bool npt_l1mmio_check(struct test *test)
+{
+ u32 *data = (u32*)(0xfee00030);
+
+ return (nested_apic_version == *data);
+}
+
static void latency_prepare(struct test *test)
{
default_prepare(test);
@@ -962,6 +983,8 @@ static struct test tests[] = {
default_finished, npt_rw_check },
{ "npt_pfwalk", npt_supported, npt_pfwalk_prepare, null_test,
default_finished, npt_pfwalk_check },
+ { "npt_l1mmio", npt_supported, npt_l1mmio_prepare, npt_l1mmio_test,
+ default_finished, npt_l1mmio_check },
{ "latency_run_exit", default_supported, latency_prepare,
latency_test,
latency_finished, latency_check },
{ "latency_svm_insn", default_supported, lat_svm_insn_prepare,
null_test,
--
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
