On Wed, Aug 03, 2005 at 04:01:14PM -0700, Venkatesh Pallipadi wrote:
> 
> Below is the updated patch.

Looks good. Only minor nit.

> +++ linux-2.6.13-rc3-auto/arch/i386/kernel/setup.c    2005-08-03 
> 14:33:34.450182216 -0700
> @@ -1593,8 +1594,13 @@ void __init setup_arch(char **cmdline_p)
>        */
>       acpi_boot_table_init();
>       acpi_boot_init();
> -#endif
>  
> +#ifdef CONFIG_X86_PC
> +     if (def_to_bigsmp) {
> +             printk(KERN_WARNING "More than 8 CPUs detected and 
> CONFIG_X86_PC cannot handle it. Use CONFIG_X86_GENERICARCH or 
> CONFIG_X86_BIGSMP.\n");

Isn't that printk line longer than 80 characters?  Will it even fit 
on the screen? 

-Andi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to