[PATCH] quiet down SMP boot messages

From: Jes Sorensen <jes@trained-monkey.org>

I'd like to propose the following for 2.6.1-mm/2.6.2. On systems with a
large number of CPUs the number of printk's flowing by for each CPU
booting starts becoming a real console hog.

The following patch eliminates a couple of them (already sent a patch to
David for the ia64 specific ones) as well as changes the
"Building zonelist : X" in "Built Y zonelists". IMHO it doesn't make any
sense to print for each zonelist since it's run in a for loop running
from 0 to Y-1 anyway.

The patch nukes a few new printk's that were introduced with the
scheduler changes to the NUMA code in -mm3, if these are still needed
then I won't fight for that part of the patch.

diff --git a/init/main.c b/init/main.c
index bf123c794be1d72b34946259251fad1b9e0a4b87..3ac9aef54f31799c1ff75cdb8a4967bd64b6d0db 100644 (file)
--- a/init/main.c
+++ b/init/main.c
@@ -339,21 +339,21 @@ static void __init setup_per_cpu_areas(void)
 /* Called by boot processor to activate the rest. */
 static void __init smp_init(void)
 {
-       unsigned int i, j=0;
+       unsigned int i;
+       unsigned j = 0;
 
        /* FIXME: This should be done in userspace --RR */
        for (i = 0; i < NR_CPUS; i++) {
                if (num_online_cpus() >= max_cpus)
                        break;
                if (cpu_possible(i) && !cpu_online(i)) {
-                       printk("Bringing up %i\n", i);
                        cpu_up(i);
                        j++;
                }
        }
 
        /* Any cleanup work */
-       printk("CPUS done %u\n", j);
+       printk("Brought up %u CPUs\n", j);
        smp_cpus_done(max_cpus);
 #if 0
        /* Get other processors into their bootup holding patterns. */

diff --git a/kernel/cpu.c b/kernel/cpu.c
index 0652d1871f2fb948d62efee71dcaa637795fa2aa..027e1997f81c8fda8fd9a46dbdb12daf6f3a9bef 100644 (file)
--- a/kernel/cpu.c
+++ b/kernel/cpu.c
@@ -55,7 +55,6 @@ int __devinit cpu_up(unsigned int cpu)
                BUG();
 
        /* Now call notifier in preparation. */
-       printk("CPU %u IS NOW UP!\n", cpu);
        notifier_call_chain(&cpu_chain, CPU_ONLINE, hcpu);
 
 out_notify:

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index b2a185f246d45e3a4e1743ec42a8b6268d00e0a2..21cdc12dff7f73cad0e664386bb62fd773931a2b 100644 (file)
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -1080,7 +1080,6 @@ static void __init build_zonelists(pg_data_t *pgdat)
        int i, j, k, node, local_node;
 
        local_node = pgdat->node_id;
-       printk("Building zonelist for node : %d\n", local_node);
        for (i = 0; i < MAX_NR_ZONES; i++) {
                struct zonelist *zonelist;
 
@@ -1118,6 +1117,7 @@ void __init build_all_zonelists(void)
 
        for(i = 0 ; i < numnodes ; i++)
                build_zonelists(NODE_DATA(i));
+       printk("Built %i zonelists\n", numnodes);
 }
 
 /*