WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-changelog

[Xen-changelog] Turn down debug a bit, and generally tidy hings up a lit

To: xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-changelog] Turn down debug a bit, and generally tidy hings up a little.
From: Xen patchbot -unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>
Date: Mon, 30 Jan 2006 15:00:16 +0000
Delivery-date: Mon, 30 Jan 2006 15:24:03 +0000
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id: BK change log <xen-changelog.lists.xensource.com>
List-post: <mailto:xen-changelog@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to: xen-devel@xxxxxxxxxxxxxxxxxxx
Sender: xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx
# 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

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-changelog] Turn down debug a bit, and generally tidy hings up a little., Xen patchbot -unstable <=