[Xen-devel] [PATCH 1 of 6] mm: guest_remove_page() should not populate or unshare

[Tim Deegan <tim@xxxxxxx>]

# HG changeset patch
# User Tim Deegan <tim@xxxxxxx>
# Date 1330013729 0
# Node ID e165cfc82565addf9881487ee1aa829de1ee1402
# Parent  0c3d19f40ab145d101de84051c3e00eef17fa1cb
mm: guest_remove_page() should not populate or unshare.

guest_remove_page() ought to use get_gfn_query() to look up the
current state of the gfn.  Otherwise it might populate or unshare
the gfn just before dropping it.

Signed-off-by: Tim Deegan <tim@xxxxxxx>

diff -r 0c3d19f40ab1 -r e165cfc82565 xen/common/memory.c
--- a/xen/common/memory.c       Mon Feb 20 22:16:32 2012 +0100
+++ b/xen/common/memory.c       Thu Feb 23 16:15:29 2012 +0000
@@ -162,7 +162,7 @@ int guest_remove_page(struct domain *d, 
     unsigned long mfn;
 
 #ifdef CONFIG_X86
-    mfn = mfn_x(get_gfn(d, gmfn, &p2mt)); 
+    mfn = mfn_x(get_gfn_query(d, gmfn, &p2mt)); 
     if ( unlikely(p2m_is_paging(p2mt)) )
     {
         guest_physmap_remove_page(d, gmfn, mfn, 0);

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel