lkml.org 
[lkml]   [2003]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[TRIVIAL] Re: Remove chatty printk on CPU bringup.
Date
[ Only having one processor is not an error.  It still happens to some
people: let's not deprive them of their bogomips printk as well --RR ]


From: Nick Piggin <piggin@cyberone.com.au>


Rusty Russell wrote:

>In message <3F092E1F.7060406@cyberone.com.au> you write:
>
>>Hi Rusty,
>>That reminds me of this minor irritation. Maybe you
>>could get it applied. _Or_, do the else block
>>unconditionally - which would get a more consistent
>>output. Either way I fail to see how its an error.
>>
>
>Hmm, this means an SMP configuration is found, but there's only one
>CPU. Does this actually happen for you?
>

I test with SMP kernels on that computer I use at IBM. That
is how I came across the error.

>
>If so, agreed. My crash box is SMP: can you eliminate that branch
>entirely and do a test boot for me?
>
I don't have a UP here to test with, and the test doesn't
trigger with nosmp. Anyway the following compiles and boots
on SMP.


--- trivial-2.5.75-bk3/arch/i386/kernel/smpboot.c.orig 2003-07-21 21:22:56.000000000 +1000
+++ trivial-2.5.75-bk3/arch/i386/kernel/smpboot.c 2003-07-21 21:22:56.000000000 +1000
@@ -937,6 +937,7 @@
static void __init smp_boot_cpus(unsigned int max_cpus)
{
int apicid, cpu, bit, kicked;
+ unsigned long bogosum = 0;

/*
* Setup boot CPU information
@@ -1050,24 +1051,23 @@
*/

Dprintk("Before bogomips.\n");
- if (!cpucount) {
- printk(KERN_ERR "Error: only one processor found.\n");
- } else {
- unsigned long bogosum = 0;
- for (cpu = 0; cpu < NR_CPUS; cpu++)
- if (cpu_callout_map & (1<<cpu))
- bogosum += cpu_data[cpu].loops_per_jiffy;
- printk(KERN_INFO "Total of %d processors activated (%lu.%02lu BogoMIPS).\n",
- cpucount+1,
- bogosum/(500000/HZ),
- (bogosum/(5000/HZ))%100);
- Dprintk("Before bogocount - setting activated=1.\n");
- }
+ for (cpu = 0; cpu < NR_CPUS; cpu++)
+ if (cpu_callout_map & (1<<cpu))
+ bogosum += cpu_data[cpu].loops_per_jiffy;
+ printk(KERN_INFO
+ "Total of %d processors activated (%lu.%02lu BogoMIPS).\n",
+ cpucount+1,
+ bogosum/(500000/HZ),
+ (bogosum/(5000/HZ))%100);
+ Dprintk("Before bogocount - setting activated=1.\n");

if (smp_b_stepping)
printk(KERN_WARNING "WARNING: SMP operation may be unreliable with B stepping processors.\n");

- /* Don't taint if we are running SMP kernel on a single non-MP approved Athlon */
+ /*
+ * Don't taint if we are running SMP kernel on a single non-MP
+ * approved Athlon
+ */
if (tainted & TAINT_UNSAFE_SMP) {
if (cpucount)
printk (KERN_INFO "WARNING: This combination of AMD processors is not suitable for SMP.\n");
--
What is this? http://www.kernel.org/pub/linux/kernel/people/rusty/trivial/
Don't blame me: the Monkey is driving
File: Nick Piggin <piggin@cyberone.com.au>: Re: [TRIVIAL] Remove chatty printk on CPU bringup.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
\
 
 \ /
  Last update: 2005-03-22 13:47    [W:0.023 / U:0.944 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site