Messages in this thread | | | Subject | Re: [PATCH] x86: cpu: use printk_once | From | Joe Perches <> | Date | Sat, 16 Jun 2012 14:07:45 -0700 |
| |
On Sat, 2012-06-16 at 21:45 +0200, Davidlohr Bueso wrote: > Use printk_once() instead of extra variables.
This is _very_ slightly larger. The only thing I'd change is to use true instead of 1; Well, I might use pr_info too...
> diff --git a/arch/x86/kernel/cpu/common.c b/arch/x86/kernel/cpu/common.c [] > @@ -457,7 +457,6 @@ void __cpuinit detect_ht(struct cpuinfo_x86 *c) > #ifdef CONFIG_X86_HT > u32 eax, ebx, ecx, edx; > int index_msb, core_bits; > - static bool printed; > > if (!cpu_has(c, X86_FEATURE_HT)) > return; > @@ -493,12 +492,11 @@ void __cpuinit detect_ht(struct cpuinfo_x86 *c) > ((1 << core_bits) - 1); > > out: > - if (!printed && (c->x86_max_cores * smp_num_siblings) > 1) { > - printk(KERN_INFO "CPU: Physical Processor ID: %d\n", > - c->phys_proc_id); > - printk(KERN_INFO "CPU: Processor Core ID: %d\n", > - c->cpu_core_id); > - printed = 1; > + if ((c->x86_max_cores * smp_num_siblings) > 1) { > + printk_once(KERN_INFO "CPU: Physical Processor ID: %d\n", > + c->phys_proc_id); > + printk_once(KERN_INFO "CPU: Processor Core ID: %d\n", > + c->cpu_core_id); > } > #endif > } > diff --git a/arch/x86/kernel/cpu/topology.c b/arch/x86/kernel/cpu/topology.c > index 4397e98..2f01ae0 100644 > --- a/arch/x86/kernel/cpu/topology.c > +++ b/arch/x86/kernel/cpu/topology.c > @@ -32,7 +32,6 @@ void __cpuinit detect_extended_topology(struct cpuinfo_x86 *c) > unsigned int eax, ebx, ecx, edx, sub_index; > unsigned int ht_mask_width, core_plus_mask_width; > unsigned int core_select_mask, core_level_siblings; > - static bool printed; > > if (c->cpuid_level < 0xb) > return; > @@ -86,14 +85,9 @@ void __cpuinit detect_extended_topology(struct cpuinfo_x86 *c) > > c->x86_max_cores = (core_level_siblings / smp_num_siblings); > > - if (!printed) { > - printk(KERN_INFO "CPU: Physical Processor ID: %d\n", > - c->phys_proc_id); > - if (c->x86_max_cores > 1) > - printk(KERN_INFO "CPU: Processor Core ID: %d\n", > - c->cpu_core_id); > - printed = 1; > - } > + printk_once(KERN_INFO "CPU: Physical Processor ID: %d\n", c->phys_proc_id); > + if (c->x86_max_cores > 1) > + printk_once(KERN_INFO "CPU: Processor Core ID: %d\n", c->cpu_core_id); > return; > #endif > }
| |