* Yinghai Lu <yhlu.kernel@xxxxxxxxx> wrote: > On Fri, Nov 13, 2009 at 4:33 PM, tip-bot for Dave Jones > <davej@xxxxxxxxxx> wrote: > > Commit-ID: ?0388423dba2217b4e5b6c61690b0506d13b25a49 > > Gitweb: ? ? http://git.kernel.org/tip/0388423dba2217b4e5b6c61690b0506d13b25a49 > > Author: ? ? Dave Jones <davej@xxxxxxxxxx> > > AuthorDate: Fri, 13 Nov 2009 15:30:00 -0500 > > Committer: ?Ingo Molnar <mingo@xxxxxxx> > > CommitDate: Sat, 14 Nov 2009 01:18:05 +0100 > > > > x86: Minimise printk spew from per-vendor init code > > > > In the default case where the kernel supports all CPU vendors, > > we currently print out a bunch of not useful messages on every > > system. > > > > 32-bit: > > KERNEL supported cpus: > > ?Intel GenuineIntel > > ?AMD AuthenticAMD > > ?NSC Geode by NSC > > ?Cyrix CyrixInstead > > ?Centaur CentaurHauls > > ?Transmeta GenuineTMx86 > > ?Transmeta TransmetaCPU > > ?UMC UMC UMC UMC > > > > 64-bit: > > KERNEL supported cpus: > > ?Intel GenuineIntel > > ?AMD AuthenticAMD > > ?Centaur CentaurHauls > > > > Given that "what CPUs does the kernel support" isn't useful for > > the "support everything" case, we can suppress these printk's. > > > > Signed-off-by: Dave Jones <davej@xxxxxxxxxx> > > LKML-Reference: <20091113203000.GA19160@xxxxxxxxxx> > > Signed-off-by: Ingo Molnar <mingo@xxxxxxx> > > --- > > ?arch/x86/kernel/cpu/common.c | ? ?3 ++- > > ?1 files changed, 2 insertions(+), 1 deletions(-) > > > > diff --git a/arch/x86/kernel/cpu/common.c b/arch/x86/kernel/cpu/common.c > > index cc25c2b..617a29f 100644 > > --- a/arch/x86/kernel/cpu/common.c > > +++ b/arch/x86/kernel/cpu/common.c > > @@ -656,6 +656,7 @@ static void __init early_identify_cpu(struct cpuinfo_x86 *c) > > > > ?void __init early_cpu_init(void) > > ?{ > > +#ifdef PROCESSOR_SELECT > > ? ? ? ?const struct cpu_dev *const *cdev; > > ? ? ? ?int count = 0; > > > > @@ -676,7 +677,7 @@ void __init early_cpu_init(void) > > ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?cpudev->c_ident[j]); > > ? ? ? ? ? ? ? ?} > > ? ? ? ?} > > - > > +#endif > > ? ? ? ?early_identify_cpu(&boot_cpu_data); > > ?} > > NACK. > > the function is initializing cpu_devs array. Good point. I fixed this. Ingo -- To unsubscribe from this list: send the line "unsubscribe linux-tip-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html