Xen devel - Oct 2009 - x86: Fix ept and vt-d co-existence issue.

x86: Fix ept and vt-d co-existence issue. 
 
For vt-d''s mmio address ranges, once ept enables, they should
be added to ept page tables with p2m lock held, and then guest can 
access these ranges like conventional ram, but to change the ept entries
, it should take the p2m lock first.
 
Signed-off-by: Xiantao Zhang <xiantao.zhang@intel.com>
 
diff -r 18758847bf31 xen/arch/x86/mm/hap/p2m-ept.c
--- a/xen/arch/x86/mm/hap/p2m-ept.c Wed Oct 14 09:09:23 2009 +0100
+++ b/xen/arch/x86/mm/hap/p2m-ept.c Thu Oct 15 19:25:42 2009 +0800
@@ -524,6 +524,7 @@ void ept_change_entry_emt_with_range(str
     mfn_t mfn;
     int order = 0;
 
+    p2m_lock(d->arch.p2m);
     for ( gfn = start_gfn; gfn <= end_gfn; gfn++ )
     {
         e = ept_get_entry_content(d, gfn);
@@ -561,6 +562,7 @@ void ept_change_entry_emt_with_range(str
                 ept_set_entry(d, gfn, mfn, order, e.avail1);
         }
     }
+    p2m_unlock(d->arch.p2m);
 }
 
 /* 
diff -r 18758847bf31 xen/arch/x86/mm/p2m.c
--- a/xen/arch/x86/mm/p2m.c Wed Oct 14 09:09:23 2009 +0100
+++ b/xen/arch/x86/mm/p2m.c Thu Oct 15 17:42:40 2009 +0800
@@ -2237,7 +2237,9 @@ set_mmio_p2m_entry(struct domain *d, uns
     }
 
     P2M_DEBUG("set mmio %lx %lx\n", gfn, mfn_x(mfn));
+    p2m_lock(d->arch.p2m);
     rc = set_p2m_entry(d, gfn, mfn, 0, p2m_mmio_direct);
+    p2m_unlock(d->arch.p2m);
     if ( 0 == rc )
         gdprintk(XENLOG_ERR,
             "set_mmio_p2m_entry: set_p2m_entry failed! mfn=%08lx\n",
@@ -2261,7 +2263,9 @@ clear_mmio_p2m_entry(struct domain *d, u
             "clear_mmio_p2m_entry: gfn_to_mfn failed! gfn=%08lx\n",
gfn);
         return 0;
     }
+    p2m_lock(d->arch.p2m);
     rc = set_p2m_entry(d, gfn, _mfn(INVALID_MFN), 0, 0);
+    p2m_unlock(d->arch.p2m);
 
     return rc;
 }



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel