|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] [xen-unstable] x86/mm: remove 0x55 debug pattern from M2P table
# HG changeset patch
# User Tim Deegan <tim@xxxxxxx>
# Date 1322834872 28800
# Node ID 109b99239b21275ee2249873dcdb9a413741142d
# Parent 771e2a303753666c5bc23557264b7a6df96e6b89
x86/mm: remove 0x55 debug pattern from M2P table
It's not really any more useful than explicitly setting new M2P
entries to the invalid value.
Signed-off-by: Tim Deegan <tim@xxxxxxx>
Committed-by: Keir Fraser <keir@xxxxxxx>
---
diff -r 771e2a303753 -r 109b99239b21 tools/libxc/xc_domain.c
--- a/tools/libxc/xc_domain.c Fri Dec 02 14:22:47 2011 +0100
+++ b/tools/libxc/xc_domain.c Fri Dec 02 06:07:52 2011 -0800
@@ -1474,8 +1474,7 @@
int xc_domain_p2m_audit(xc_interface *xch,
uint32_t domid,
- uint64_t *orphans_debug,
- uint64_t *orphans_invalid,
+ uint64_t *orphans,
uint64_t *m2p_bad,
uint64_t *p2m_bad)
{
@@ -1486,10 +1485,9 @@
domctl.domain = domid;
rc = do_domctl(xch, &domctl);
- *orphans_debug = domctl.u.audit_p2m.orphans_debug;
- *orphans_invalid = domctl.u.audit_p2m.orphans_invalid;
- *m2p_bad = domctl.u.audit_p2m.m2p_bad;
- *p2m_bad = domctl.u.audit_p2m.p2m_bad;
+ *orphans = domctl.u.audit_p2m.orphans;
+ *m2p_bad = domctl.u.audit_p2m.m2p_bad;
+ *p2m_bad = domctl.u.audit_p2m.p2m_bad;
return rc;
}
diff -r 771e2a303753 -r 109b99239b21 tools/libxc/xenctrl.h
--- a/tools/libxc/xenctrl.h Fri Dec 02 14:22:47 2011 +0100
+++ b/tools/libxc/xenctrl.h Fri Dec 02 06:07:52 2011 -0800
@@ -718,9 +718,7 @@
* @parm xch a handle to an open hypervisor interface
* @parm domid the domain id whose top level p2m we
* want to audit
- * @parm orphans_debug count of m2p entries for valid
- * domain pages containing a debug value
- * @parm orphans_invalid count of m2p entries for valid
+ * @parm orphans count of m2p entries for valid
* domain pages containing an invalid value
* @parm m2p_bad count of m2p entries mismatching the
* associated p2m entry for this domain
@@ -732,9 +730,8 @@
* -EFAULT: could not copy results back to guest
*/
int xc_domain_p2m_audit(xc_interface *xch,
- uint32_t domid,
- uint64_t *orphans_debug,
- uint64_t *orphans_invalid,
+ uint32_t domid,
+ uint64_t *orphans,
uint64_t *m2p_bad,
uint64_t *p2m_bad);
diff -r 771e2a303753 -r 109b99239b21 xen/arch/x86/domctl.c
--- a/xen/arch/x86/domctl.c Fri Dec 02 14:22:47 2011 +0100
+++ b/xen/arch/x86/domctl.c Fri Dec 02 06:07:52 2011 -0800
@@ -1459,8 +1459,7 @@
break;
audit_p2m(d,
- &domctl->u.audit_p2m.orphans_debug,
- &domctl->u.audit_p2m.orphans_invalid,
+ &domctl->u.audit_p2m.orphans,
&domctl->u.audit_p2m.m2p_bad,
&domctl->u.audit_p2m.p2m_bad);
rcu_unlock_domain(d);
diff -r 771e2a303753 -r 109b99239b21 xen/arch/x86/mm/p2m.c
--- a/xen/arch/x86/mm/p2m.c Fri Dec 02 14:22:47 2011 +0100
+++ b/xen/arch/x86/mm/p2m.c Fri Dec 02 06:07:52 2011 -0800
@@ -307,13 +307,7 @@
/* Pages should not be shared that early */
ASSERT(gfn != SHARED_M2P_ENTRY);
page_count++;
- if (
-#ifdef __x86_64__
- (gfn != 0x5555555555555555L)
-#else
- (gfn != 0x55555555L)
-#endif
- && gfn != INVALID_M2P_ENTRY
+ if ( gfn != INVALID_M2P_ENTRY
&& !set_p2m_entry(p2m, gfn, mfn, PAGE_ORDER_4K, p2m_ram_rw,
p2m->default_access) )
goto error_unlock;
}
@@ -513,14 +507,7 @@
if ( page_get_owner(mfn_to_page(_mfn(mfn + i))) != d )
continue;
ogfn = mfn_to_gfn(d, _mfn(mfn+i));
- if (
-#ifdef __x86_64__
- (ogfn != 0x5555555555555555L)
-#else
- (ogfn != 0x55555555L)
-#endif
- && (ogfn != INVALID_M2P_ENTRY)
- && (ogfn != gfn + i) )
+ if ( (ogfn != INVALID_M2P_ENTRY) && (ogfn != gfn + i) )
{
/* This machine frame is already mapped at another physical
* address */
@@ -1447,8 +1434,7 @@
#if P2M_AUDIT
void audit_p2m(struct domain *d,
- uint64_t *orphans_debug,
- uint64_t *orphans_invalid,
+ uint64_t *orphans,
uint64_t *m2p_bad,
uint64_t *p2m_bad)
{
@@ -1456,7 +1442,7 @@
struct domain *od;
unsigned long mfn, gfn;
mfn_t p2mfn;
- unsigned long orphans_d = 0, orphans_i = 0, mpbad = 0, pmbad = 0;
+ unsigned long orphans_count = 0, mpbad = 0, pmbad = 0;
p2m_access_t p2ma;
p2m_type_t type;
struct p2m_domain *p2m = p2m_get_hostp2m(d);
@@ -1492,20 +1478,12 @@
gfn = get_gpfn_from_mfn(mfn);
if ( gfn == INVALID_M2P_ENTRY )
{
- orphans_i++;
+ orphans_count++;
P2M_PRINTK("orphaned guest page: mfn=%#lx has invalid gfn\n",
mfn);
continue;
}
- if ( gfn == 0x55555555 || gfn == 0x5555555555555555 )
- {
- orphans_d++;
- P2M_PRINTK("orphaned guest page: mfn=%#lx has debug gfn\n",
- mfn);
- continue;
- }
-
if ( gfn == SHARED_M2P_ENTRY )
{
P2M_PRINTK("shared mfn (%lx) on domain page list!\n",
@@ -1538,9 +1516,8 @@
p2m_unlock(p2m);
P2M_PRINTK("p2m audit complete\n");
- if ( orphans_i | orphans_d | mpbad | pmbad )
- P2M_PRINTK("p2m audit found %lu orphans (%lu inval %lu debug)\n",
- orphans_i + orphans_d, orphans_i, orphans_d);
+ if ( orphans_count | mpbad | pmbad )
+ P2M_PRINTK("p2m audit found %lu orphans\n", orphans);
if ( mpbad | pmbad )
{
P2M_PRINTK("p2m audit found %lu odd p2m, %lu bad m2p entries\n",
@@ -1549,10 +1526,9 @@
}
out_p2m_audit:
- *orphans_debug = (uint64_t) orphans_d;
- *orphans_invalid = (uint64_t) orphans_i;
- *m2p_bad = (uint64_t) mpbad;
- *p2m_bad = (uint64_t) pmbad;
+ *orphans = (uint64_t) orphans_count;
+ *m2p_bad = (uint64_t) mpbad;
+ *p2m_bad = (uint64_t) pmbad;
}
#endif /* P2M_AUDIT */
diff -r 771e2a303753 -r 109b99239b21 xen/arch/x86/x86_32/mm.c
--- a/xen/arch/x86/x86_32/mm.c Fri Dec 02 14:22:47 2011 +0100
+++ b/xen/arch/x86/x86_32/mm.c Fri Dec 02 06:07:52 2011 -0800
@@ -114,8 +114,8 @@
l2e_write(&idle_pg_table_l2[l2_linear_offset(RO_MPT_VIRT_START) + i],
l2e_from_page(
pg, (__PAGE_HYPERVISOR | _PAGE_PSE) & ~_PAGE_RW));
- /* Fill with an obvious debug pattern. */
- memset((void *)(RDWR_MPT_VIRT_START + (i << L2_PAGETABLE_SHIFT)), 0x55,
+ /* Fill with INVALID_M2P_ENTRY. */
+ memset((void *)(RDWR_MPT_VIRT_START + (i << L2_PAGETABLE_SHIFT)), 0xFF,
1UL << L2_PAGETABLE_SHIFT);
}
#undef CNT
diff -r 771e2a303753 -r 109b99239b21 xen/arch/x86/x86_64/mm.c
--- a/xen/arch/x86/x86_64/mm.c Fri Dec 02 14:22:47 2011 +0100
+++ b/xen/arch/x86/x86_64/mm.c Fri Dec 02 06:07:52 2011 -0800
@@ -495,7 +495,8 @@
PAGE_HYPERVISOR);
if ( err )
break;
- memset((void *)rwva, 0x55, 1UL << L2_PAGETABLE_SHIFT);
+ /* Fill with INVALID_M2P_ENTRY. */
+ memset((void *)rwva, 0xFF, 1UL << L2_PAGETABLE_SHIFT);
/* NB. Cannot be GLOBAL as the ptes get copied into per-VM space. */
l2e_write(&l2_ro_mpt[l2_table_offset(va)], l2e_from_page(l1_pg,
_PAGE_PSE|_PAGE_PRESENT));
}
@@ -569,8 +570,9 @@
PAGE_HYPERVISOR);
if ( ret )
goto error;
+ /* Fill with INVALID_M2P_ENTRY. */
memset((void *)(RDWR_MPT_VIRT_START + i * sizeof(unsigned long)),
- 0x55, 1UL << L2_PAGETABLE_SHIFT);
+ 0xFF, 1UL << L2_PAGETABLE_SHIFT);
ASSERT(!(l3e_get_flags(l3_ro_mpt[l3_table_offset(va)]) &
_PAGE_PSE));
@@ -727,8 +729,9 @@
page_to_mfn(l1_pg),
1UL << PAGETABLE_ORDER,
PAGE_HYPERVISOR);
+ /* Fill with INVALID_M2P_ENTRY. */
memset((void *)(RDWR_MPT_VIRT_START + (i << L2_PAGETABLE_SHIFT)),
- 0x55, 1UL << L2_PAGETABLE_SHIFT);
+ 0xFF, 1UL << L2_PAGETABLE_SHIFT);
}
if ( !((unsigned long)l2_ro_mpt & ~PAGE_MASK) )
{
diff -r 771e2a303753 -r 109b99239b21 xen/include/asm-x86/p2m.h
--- a/xen/include/asm-x86/p2m.h Fri Dec 02 14:22:47 2011 +0100
+++ b/xen/include/asm-x86/p2m.h Fri Dec 02 06:07:52 2011 -0800
@@ -566,10 +566,9 @@
#if P2M_AUDIT
extern void audit_p2m(struct domain *d,
- uint64_t *orphans_debug,
- uint64_t *orphans_invalid,
- uint64_t *m2p_bad,
- uint64_t *p2m_bad);
+ uint64_t *orphans,
+ uint64_t *m2p_bad,
+ uint64_t *p2m_bad);
#endif /* P2M_AUDIT */
/* Printouts */
diff -r 771e2a303753 -r 109b99239b21 xen/include/public/domctl.h
--- a/xen/include/public/domctl.h Fri Dec 02 14:22:47 2011 +0100
+++ b/xen/include/public/domctl.h Fri Dec 02 06:07:52 2011 -0800
@@ -806,8 +806,7 @@
struct xen_domctl_audit_p2m {
/* OUT error counts */
- uint64_t orphans_debug;
- uint64_t orphans_invalid;
+ uint64_t orphans;
uint64_t m2p_bad;
uint64_t p2m_bad;
};
_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |