[Xen-ia64-devel] clean up extraneous verbose output

[Aron Griffis <aron@xxxxxx>]

Fedora builds the HV with verbose=y.  On the other arches this seems
to be fine, but ia64 is generating some apparent debug output in this
mode.  See http://free.linux.hp.com/~agriffis/2.6.17-1.2566.fc6xen.txt

One of the culprits looks okay to remove completely.  The other seems
more useful so I hid it behind #ifndef NDEBUG, so we can get it back
when debug=y is repaired.

# HG changeset patch
# User agriffis@xxxxxxxxxxxxxxx
# Node ID c9fd75cefbc150994bd8638147ed3b3396bd8466
# Parent  91169603a8e8dded9eba6cb6c3421b5d58a85a97
Remove extraneous verbose output to clean up Fedora boot.

Signed-off-by: Aron Griffis <aron@xxxxxx>

diff -r 91169603a8e8 -r c9fd75cefbc1 xen/arch/ia64/xen/mm.c
--- a/xen/arch/ia64/xen/mm.c    Tue Aug 22 14:45:49 2006 -0600
+++ b/xen/arch/ia64/xen/mm.c    Wed Aug 23 00:04:47 2006 -0400
@@ -949,8 +949,6 @@ efi_mmio(unsigned long physaddr, unsigne
                 return 1;
             }
 
-            DPRINTK("%s:%d physaddr 0x%lx size = 0x%lx\n",
-                    __func__, __LINE__, physaddr, size);
             return 0;
         }
 
@@ -971,8 +969,10 @@ assign_domain_mmio_page(struct domain *d
                 __func__, d, mpaddr, size);
     }
     if (!efi_mmio(mpaddr, size)) {
+#ifndef NDEBUG
         DPRINTK("%s:%d domain %p mpaddr 0x%lx size = 0x%lx\n",
                 __func__, __LINE__, d, mpaddr, size);
+#endif
         return -EINVAL;
     }
     assign_domain_same_page(d, mpaddr, size, ASSIGN_writable | ASSIGN_nocache);

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel