[Xen-devel] [PATCH 5/8] xen/setup: Only print "Freeing XXX-YYY pfn range: Z pages freed" if Z > 0

[Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>]

Otherwise we can get these meaningless:
Freeing  bad80-badf4 pfn range: 0 pages freed

We also can do this for the summary ones - no point of printing
"Set 0 page(s) to 1-1 mapping"

[v1: Extended to the summary printks]
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
---
 arch/x86/xen/setup.c |   11 +++++++----
 1 files changed, 7 insertions(+), 4 deletions(-)

diff --git a/arch/x86/xen/setup.c b/arch/x86/xen/setup.c
index 1ba8dff..7b0ab77 100644
--- a/arch/x86/xen/setup.c
+++ b/arch/x86/xen/setup.c
@@ -114,8 +114,9 @@ static unsigned long __init xen_release_chunk(unsigned long 
start,
                        len++;
                }
        }
-       printk(KERN_INFO "Freeing  %lx-%lx pfn range: %lu pages freed\n",
-              start, end, len);
+       if (len)
+               printk(KERN_INFO "Freeing  %lx-%lx pfn range: %lu pages 
freed\n",
+                      start, end, len);
 
        return len;
 }
@@ -162,8 +163,10 @@ static unsigned long __init xen_set_identity_and_release(
                }
        }
 
-       printk(KERN_INFO "Released %lu pages of unused memory\n", released);
-       printk(KERN_INFO "Set %ld page(s) to 1-1 mapping\n", identity);
+       if (released)
+               printk(KERN_INFO "Released %lu pages of unused memory\n", 
released);
+       if (identity)
+               printk(KERN_INFO "Set %ld page(s) to 1-1 mapping\n", identity);
 
        return released;
 }
-- 
1.7.7.5


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel