[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-changelog] Turn down debug a bit, and generally tidy hings up a little.
# HG changeset patch # User sos22@xxxxxxxxxxxxxxxxxxxx # Node ID a47b7a464f09648401ce43a6b24b2f42cb1a7e37 # Parent e9770d41e5bd85afc3a8bace375e529b83cdaa92 Turn down debug a bit, and generally tidy hings up a little. Signed-off-by: Steven Smith, sos22@xxxxxxxxx diff -r e9770d41e5bd -r a47b7a464f09 linux-2.6-xen-sparse/arch/xen/i386/kernel/setup.c --- a/linux-2.6-xen-sparse/arch/xen/i386/kernel/setup.c Fri Jan 27 20:38:55 2006 +++ b/linux-2.6-xen-sparse/arch/xen/i386/kernel/setup.c Fri Jan 27 20:57:07 2006 @@ -1605,7 +1605,6 @@ copy_edd(); - /* Make the PFNs in the Xen hole reserved. */ if (!MOUNT_ROOT_RDONLY) root_mountflags &= ~MS_RDONLY; init_mm.start_code = (unsigned long) _text; @@ -1768,8 +1767,6 @@ console_use_vt = 0; #endif } - - } static int diff -r e9770d41e5bd -r a47b7a464f09 linux-2.6-xen-sparse/arch/xen/i386/mm/init.c --- a/linux-2.6-xen-sparse/arch/xen/i386/mm/init.c Fri Jan 27 20:38:55 2006 +++ b/linux-2.6-xen-sparse/arch/xen/i386/mm/init.c Fri Jan 27 20:57:07 2006 @@ -597,6 +597,7 @@ int tmp; int bad_ppro; unsigned long pfn; + unsigned long hole_start, hole_size; contiguous_bitmap = alloc_bootmem_low_pages( (max_low_pfn + 2*BITS_PER_LONG) >> 3); @@ -645,15 +646,10 @@ } /* Make the Xen hole reserved. */ - unsigned long hole_start, hole_size; hole_size = xen_pfn_hole_size(); hole_start = xen_pfn_hole_start(); - for (pfn = hole_start; pfn < hole_start + hole_size; pfn++) { - printk("<0>Reserve %lx for hole.\n", - pfn); + for (pfn = hole_start; pfn < hole_start + hole_size; pfn++) SetPageReserved(pfn_to_page(pfn)); - BUG_ON(!PageReserved(pfn_to_page(pfn))); - } reservedpages = 0; for (tmp = 0; tmp < max_low_pfn; tmp++) diff -r e9770d41e5bd -r a47b7a464f09 linux-2.6-xen-sparse/arch/xen/i386/mm/pgtable.c --- a/linux-2.6-xen-sparse/arch/xen/i386/mm/pgtable.c Fri Jan 27 20:38:55 2006 +++ b/linux-2.6-xen-sparse/arch/xen/i386/mm/pgtable.c Fri Jan 27 20:57:07 2006 @@ -317,7 +317,6 @@ pgd_test_and_unpin(pgd); - memset(pgd, 0, 10); if (PTRS_PER_PMD == 1 || !pgd) return pgd; diff -r e9770d41e5bd -r a47b7a464f09 linux-2.6-xen-sparse/drivers/xen/balloon/balloon.c --- a/linux-2.6-xen-sparse/drivers/xen/balloon/balloon.c Fri Jan 27 20:38:55 2006 +++ b/linux-2.6-xen-sparse/drivers/xen/balloon/balloon.c Fri Jan 27 20:57:07 2006 @@ -113,8 +113,6 @@ if (hole_len == -1) { hole_start = xen_pfn_hole_start(); hole_len = xen_pfn_hole_size(); - printk("<0>Xen hole at [%lx,%lx).\n", hole_start, - hole_start + hole_len); } return pfn >= hole_start && pfn < hole_start + hole_len; } @@ -122,12 +120,9 @@ /* balloon_append: add the given page to the balloon. */ static void balloon_append(struct page *page) { - BUG_ON(PageReserved(page)); - if (page_is_xen_hole(page_to_pfn(page))) { - printk("<0>Attempt to add reserved pfn %lx to balloon.\n", + if (page_is_xen_hole(page_to_pfn(page))) + panic("Attempt to add Xen-reserved pfn %lx to balloon.\n", page_to_pfn(page)); - BUG(); - } /* Lowmem is re-populated first, so highmem pages go at list tail. */ if (PageHighMem(page)) { list_add_tail(PAGE_TO_LIST(page), &ballooned_pages); @@ -243,25 +238,14 @@ xen_machphys_update(frame_list[i], pfn); #endif - printk("<0>Balloon allocated %lx.\n", pfn); /* Link back into the page tables if not highmem. */ if (pfn < max_low_pfn) { int ret; - pgd_t *pgd = pgd_offset_k((unsigned long)__va(pfn << PAGE_SHIFT)); - printk("pgd is %lx.\n", *(unsigned long *)pgd); - (void)copy_from_user(&ret, - (unsigned long *)__va(pfn << PAGE_SHIFT), - 4); ret = HYPERVISOR_update_va_mapping( (unsigned long)__va(pfn << PAGE_SHIFT), pfn_pte_ma(frame_list[i], PAGE_KERNEL), 0); BUG_ON(ret); - printk("<0>Rehooked va; pte now %lx.\n", - *(unsigned long *)virt_to_ptep(__va(pfn << PAGE_SHIFT))); - *(unsigned long *)__va(pfn << PAGE_SHIFT) = - 0xf001; - printk("<0>Touched va.\n"); } /* Relinquish the page back to the allocator. */ diff -r e9770d41e5bd -r a47b7a464f09 linux-2.6-xen-sparse/drivers/xen/netfront/netfront.c --- a/linux-2.6-xen-sparse/drivers/xen/netfront/netfront.c Fri Jan 27 20:38:55 2006 +++ b/linux-2.6-xen-sparse/drivers/xen/netfront/netfront.c Fri Jan 27 20:57:07 2006 @@ -614,7 +614,7 @@ reservation.domid = DOMID_SELF; /* Tell the ballon driver what is going on. */ -//SOS22 balloon_update_driver_allowance(i); + balloon_update_driver_allowance(i); /* Zap PTEs and give away pages in one big multicall. */ (void)HYPERVISOR_multicall(rx_mcl, i+1); @@ -818,7 +818,7 @@ } /* Some pages are no longer absent... */ -//SOS22 balloon_update_driver_allowance(-work_done); + balloon_update_driver_allowance(-work_done); /* Do all the remapping work, and M2P updates, in one big hypercall. */ if (likely((mcl - rx_mcl) != 0)) { diff -r e9770d41e5bd -r a47b7a464f09 linux-2.6-xen-sparse/mm/memory.c --- a/linux-2.6-xen-sparse/mm/memory.c Fri Jan 27 20:38:55 2006 +++ b/linux-2.6-xen-sparse/mm/memory.c Fri Jan 27 20:57:07 2006 @@ -291,7 +291,6 @@ spin_lock(&mm->page_table_lock); if (!new) return NULL; - /* * Because we dropped the lock, we should re-check the * entry, as somebody else could have populated it.. diff -r e9770d41e5bd -r a47b7a464f09 tools/libxc/xc_linux_build.c --- a/tools/libxc/xc_linux_build.c Fri Jan 27 20:38:55 2006 +++ b/tools/libxc/xc_linux_build.c Fri Jan 27 20:57:07 2006 @@ -32,8 +32,6 @@ #define L3_PROT (_PAGE_PRESENT|_PAGE_RW|_PAGE_ACCESSED|_PAGE_DIRTY|_PAGE_USER) #define L4_PROT (_PAGE_PRESENT|_PAGE_RW|_PAGE_ACCESSED|_PAGE_DIRTY|_PAGE_USER) #endif - -#define NR_GRANT_FRAMES 4 #ifdef __ia64__ #define get_tot_pages xc_get_max_pages diff -r e9770d41e5bd -r a47b7a464f09 xen/arch/x86/mm.c --- a/xen/arch/x86/mm.c Fri Jan 27 20:38:55 2006 +++ b/xen/arch/x86/mm.c Fri Jan 27 20:57:07 2006 @@ -1041,7 +1041,7 @@ if ( unlikely(cmpxchg_user(pl1e, o, n) != 0) || unlikely(o != l1e_get_intpte(ol1e)) ) { - printf("Failed to update %" PRIpte " -> %" PRIpte + MEM_LOG("Failed to update %" PRIpte " -> %" PRIpte ": saw %" PRIpte, l1e_get_intpte(ol1e), l1e_get_intpte(nl1e), @@ -1058,16 +1058,11 @@ l1_pgentry_t ol1e; struct domain *d = current->domain; - shadow_sync_all(d); - if ( unlikely(__copy_from_user(&ol1e, pl1e, sizeof(ol1e)) != 0) ) { - printf("copy_from_user1 failed %p, l2 %lx.\n", pl1e, - *(unsigned long *)&__linear_l2_table[l2_table_offset((unsigned long)pl1e)]); + if ( unlikely(__copy_from_user(&ol1e, pl1e, sizeof(ol1e)) != 0) ) return 0; - } - - if ( unlikely(shadow_mode_refcounts(d)) ) { + + if ( unlikely(shadow_mode_refcounts(d)) ) return update_l1e(pl1e, ol1e, nl1e); - } if ( l1e_get_flags(nl1e) & _PAGE_PRESENT ) { @@ -2545,10 +2540,8 @@ perfc_incrc(calls_to_update_va); - if ( unlikely(!__addr_ok(va) && !shadow_mode_external(d)) ) { - printf("Bad update_va_mapping.\n"); + if ( unlikely(!__addr_ok(va) && !shadow_mode_external(d)) ) return -EINVAL; - } LOCK_BIGLOCK(d); @@ -2557,13 +2550,9 @@ if ( unlikely(shadow_mode_enabled(d)) ) check_pagetable(v, "pre-va"); /* debug */ - shadow_sync_all(d); - if ( unlikely(!mod_l1_entry(&linear_pg_table[l1_linear_offset(va)], - val)) ) { - printf("mod_l1_entry failed.\n"); + val)) ) rc = -EINVAL; - } if ( likely(rc == 0) && unlikely(shadow_mode_enabled(d)) ) { @@ -2580,8 +2569,7 @@ } rc = shadow_do_update_va_mapping(va, val, v); - if (rc) - printf("shadow_do_update_va_mapping says %d.\n", rc); + check_pagetable(v, "post-va"); /* debug */ } diff -r e9770d41e5bd -r a47b7a464f09 xen/arch/x86/shadow32.c --- a/xen/arch/x86/shadow32.c Fri Jan 27 20:38:55 2006 +++ b/xen/arch/x86/shadow32.c Fri Jan 27 20:57:07 2006 @@ -2502,8 +2502,6 @@ l2_pgentry_t *snapshot2 = snapshot; l1_pgentry_t *shadow2 = shadow; - printf("Update hl2 shadow.\n"); - ASSERT(shadow_mode_write_all(d) || shadow_mode_wr_pt_pte(d)); BUG_ON(!shadow_mode_refcounts(d)); // not yet implemented diff -r e9770d41e5bd -r a47b7a464f09 xen/common/grant_table.c --- a/xen/common/grant_table.c Fri Jan 27 20:38:55 2006 +++ b/xen/common/grant_table.c Fri Jan 27 20:57:07 2006 @@ -522,9 +522,7 @@ ASSERT(d->grant_table != NULL); (void)put_user(GNTST_okay, &uop->status); for ( i = 0; i < op.nr_frames; i++ ) { - mfn = gnttab_shared_mfn(d, d->grant_table, i); - if (shadow_mode_translate(d)) - mfn = __mfn_to_gpfn(d, mfn); + mfn = __mfn_to_gpfn(d, gnttab_shared_mfn(d, d->grant_table, i)); (void)put_user(mfn, &op.frame_list[i]); } } diff -r e9770d41e5bd -r a47b7a464f09 xen/common/memory.c --- a/xen/common/memory.c Fri Jan 27 20:38:55 2006 +++ b/xen/common/memory.c Fri Jan 27 20:57:07 2006 @@ -115,8 +115,6 @@ goto out; for ( j = 0; j < (1 << extent_order); j++ ) { - printf("Populating %lx with %lx.\n", - pfn + j, mfn + j); if (shadow_mode_translate(d)) set_p2m_entry(d, pfn + j, mfn + j, &cache1, &cache2); set_pfn_from_mfn(mfn + j, pfn + j); diff -r e9770d41e5bd -r a47b7a464f09 xen/include/asm-x86/shadow.h --- a/xen/include/asm-x86/shadow.h Fri Jan 27 20:38:55 2006 +++ b/xen/include/asm-x86/shadow.h Fri Jan 27 20:57:07 2006 @@ -326,7 +326,7 @@ #define SHADOW_SNAPSHOT_ELSEWHERE (-1L) /************************************************************************/ -#define SHADOW_DEBUG 1 +#define SHADOW_DEBUG 0 #define SHADOW_VERBOSE_DEBUG 0 #define SHADOW_VVERBOSE_DEBUG 0 #define SHADOW_VVVERBOSE_DEBUG 0 diff -r e9770d41e5bd -r a47b7a464f09 xen/include/public/dom0_ops.h --- a/xen/include/public/dom0_ops.h Fri Jan 27 20:38:55 2006 +++ b/xen/include/public/dom0_ops.h Fri Jan 27 20:57:07 2006 @@ -430,7 +430,7 @@ domid_t domain; /* domain to be affected */ unsigned long mfn; /* machine frame to be initialised */ } dom0_hypercall_init_t; - + typedef struct { uint32_t cmd; uint32_t interface_version; /* DOM0_INTERFACE_VERSION */ _______________________________________________ Xen-changelog mailing list Xen-changelog@xxxxxxxxxxxxxxxxxxx http://lists.xensource.com/xen-changelog
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |