[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


 


Rackspace

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