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

[Xen-changelog] [xen-unstable] x86: Add more specific logging for get_page_from_l1e failures



# HG changeset patch
# User George Dunlap <george.dunlap@xxxxxxxxxxxxx>
# Date 1331198169 0
# Node ID 02216ba702b0a8f58871e1d9292b243f9b645b0b
# Parent  569ce192e3dd3beb45e311783cca1c633c28a7cc
x86: Add more specific logging for get_page_from_l1e failures

This patch should clarify which of the two possible reasons cause the
messages
of the form "(XEN) mm.c:908:d1 Error getting mfn 29b2b (pfn
5555555555555555)
from L1 entry 0000000029b2b067 for l1e_owner=1, pg_owner=1".

Signed-off-by: George Dunlap <george.dunlap@xxxxxxxxxxxxx>
Committed-by: Keir Fraser <keir@xxxxxxx>
---


diff -r 569ce192e3dd -r 02216ba702b0 xen/arch/x86/mm.c
--- a/xen/arch/x86/mm.c Thu Mar 08 09:15:26 2012 +0000
+++ b/xen/arch/x86/mm.c Thu Mar 08 09:16:09 2012 +0000
@@ -889,7 +889,12 @@
          */
         if ( (real_pg_owner == NULL) || (pg_owner == l1e_owner) ||
              !IS_PRIV_FOR(pg_owner, real_pg_owner) )
+        {
+            MEM_LOG("pg_owner %d l1e_owner %d, but real_pg_owner %d",
+                    pg_owner->domain_id, l1e_owner->domain_id,
+                    real_pg_owner?real_pg_owner->domain_id:-1);
             goto could_not_pin;
+        }
         pg_owner = real_pg_owner;
     }
 
@@ -905,7 +910,10 @@
     write = (l1f & _PAGE_RW) &&
             ((l1e_owner == pg_owner) || !paging_mode_external(pg_owner));
     if ( write && !get_page_type(page, PGT_writable_page) )
+    {
+        MEM_LOG("Could not get page type PGT_writable_page");
         goto could_not_pin;
+    }
 
     if ( pte_flags_to_cacheattr(l1f) !=
          ((page->count_info & PGC_cacheattr_mask) >> PGC_cacheattr_base) )

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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.