From: Konrad Rzeszutek Wilk Date: Fri, 30 Mar 2012 19:37:07 +0000 (-0400) Subject: xen/setup: Only print "Freeing XXX-YYY pfn range: Z pages freed" if Z > 0 X-Git-Tag: v2.6.39-400.9.0~543^2~9^2~12 X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=36a2ed4e384a4d7e15d54ed71c5117043978e583;p=users%2Fjedix%2Flinux-maple.git xen/setup: Only print "Freeing XXX-YYY pfn range: Z pages freed" if Z > 0 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" [upstream git commit ca1182387e57470460294ce1e39e2d5518809811] Acked-by: David Vrabel [v1: Extended to the summary printks] Signed-off-by: Konrad Rzeszutek Wilk --- diff --git a/arch/x86/xen/setup.c b/arch/x86/xen/setup.c index 6338655e2c95..f1b95ed74c75 100644 --- a/arch/x86/xen/setup.c +++ b/arch/x86/xen/setup.c @@ -109,8 +109,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; } @@ -157,8 +158,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; }