Re: Nested paging in nested SVM setup

[Valentine Sinitsyn <valentine.sinitsyn@gmail.com>]

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,