[patch] ram scrub: only print dot for actual memory

[patch] ram scrub: only print dot for actual memory

[Jes Sorensen]

[-- Attachment #1: Type: text/plain, Size: 292 bytes --]

Hi Keir,

I would like to propose we reorganize the code in scrub_heap_pages()
slightly so it only prints a dot for pages that are actually present.

For those of us who have hugely sparse physical memory maps, this saves
printing a couple of hundred lines of dots at boot time.

Cheers,
Jes

[-- Attachment #2: xen-ram-scrub.diff --]
[-- Type: text/plain, Size: 1015 bytes --]

# HG changeset patch
# User jes@eye.emea.sgi.com
# Date 1166187377 -3600
# Node ID 38bae9f2448d6d694eaea626db0bb63b75fdd1b2
# Parent  fd3e080a4f54d1fbb4a1518a05508262b7ab3fce
free-mem-scrubbing: only print dots for actual memory

Signed-off-by: Jes Sorensen <jes@sgi.com>

diff -r fd3e080a4f54 -r 38bae9f2448d xen/common/page_alloc.c
--- a/xen/common/page_alloc.c	Fri Dec 15 13:50:24 2006 +0100
+++ b/xen/common/page_alloc.c	Fri Dec 15 13:56:17 2006 +0100
@@ -493,15 +493,15 @@ void scrub_heap_pages(void)
 
     for ( pfn = first_pg; pfn < max_page; pfn++ )
     {
+        process_pending_timers();
+
+        /* Quick lock-free check. */
+        if ( allocated_in_map(pfn) )
+            continue;
+
         /* Every 100MB, print a progress dot. */
         if ( (pfn % ((100*1024*1024)/PAGE_SIZE)) == 0 )
             printk(".");
-
-        process_pending_timers();
-
-        /* Quick lock-free check. */
-        if ( allocated_in_map(pfn) )
-            continue;
 
         spin_lock_irq(&heap_lock);
 

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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