Re: Wierd /proc/cpuinfo with 2.4.11-pre4

From: Brian Gerst (bgerst@didntduck.org)
Date: Fri Oct 05 2001 - 18:17:17 EST


"Martin J. Bligh" wrote:
>
> >> Wow! That's pretty impressive, a new kernel build gives me an
> >> additional _7_ CPUs!
>
> Sorry. Mea culpa
>
> --- setup.c.old Fri Oct 5 14:20:29 2001
> +++ setup.c Fri Oct 5 14:28:51 2001
> @@ -2420,7 +2420,7 @@
> * WARNING - nasty evil hack ... if we print > 8, it overflows the
> * page buffer and corrupts memory - this needs fixing properly
> */
> - for (n = 0; n < 8; n++, c++) {
> + for (n = 0; n < (clustered_apic_mode ? 8 : NR_CPUS); n++, c++) {
> /* for (n = 0; n < NR_CPUS; n++, c++) { */
> int fpu_exception;
> #ifdef CONFIG_SMP
>
> M.
>
> PS. I just tested this since my last post. It seems to work.

It would be more appropriate to do:
        for (n = 0; n < min(8, NR_CPUS); n++, c++) {

-- 

Brian Gerst - 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/



This archive was generated by hypermail 2b29 : Sun Oct 07 2001 - 21:00:40 EST