Re: [PATCH] Summit support for 2.5 [0/4]

From: Greg KH (greg@kroah.com)
Date: Sun Oct 13 2002 - 21:25:15 EST


On Sun, Oct 13, 2002 at 07:02:18PM -0700, Martin J. Bligh wrote:
>
> PS. This distros want Summit to autodetect for their install kernels,
> which is what the x86_summit switch is for.

Why? Can't summit boot just fine on a i386 UP kernel? Then they can
look at the chipset id and determine that they should install a
summit-built kernel, right?

> diff -urpN -X /home/fletch/.diff.exclude virgin/arch/i386/Config.help subarch-1/arch/i386/Config.help
> --- virgin/arch/i386/Config.help Fri Oct 11 21:21:31 2002
> +++ subarch-1/arch/i386/Config.help Sun Oct 13 18:40:30 2002
> @@ -73,6 +73,12 @@ CONFIG_X86_CYCLONE
> If you are suffering from time skew using a multi-CEC system, say YES.
> Otherwise it is safe to say NO.
>
> +CONFIG_X86_SUMMIT

Can this just be CONFIG_SUMMIT? I think most of these fixes need to be
around for the ia64 version too :(

> /home/fletch/.diff.exclude virgin/arch/i386/Makefile subarch-1/arch/i386/Makefile
> --- virgin/arch/i386/Makefile Fri Oct 11 21:21:39 2002
> +++ subarch-1/arch/i386/Makefile Sun Oct 13 17:54:32 2002
> @@ -46,7 +46,11 @@ CFLAGS += $(cflags-y)
> ifdef CONFIG_VISWS
> MACHINE := mach-visws
> else
> -MACHINE := mach-generic
> + ifdef CONFIG_X86_SUMMIT
> + MACHINE := mach-summit
> + else
> + MACHINE := mach-generic
> + endif
> endif

As we're going to end up with a mess of a ifdef nest over time with new
archs added, how about something like this (completly untested):

--- 1.12/arch/i386/Makefile Fri Oct 11 14:22:55 2002
+++ edited/Makefile Sun Oct 13 19:26:18 2002
@@ -43,10 +43,14 @@
 
 CFLAGS += $(cflags-y)
 
-ifdef CONFIG_VISWS
-MACHINE := mach-visws
-else
-MACHINE := mach-generic
+MACHINE = mach-generic
+
+ifeq ($(CONFIG_VISWS),y)
+MACHINE = mach-visws
+endif
+
+ifeq ($(CONFIG_SUMMIT),y)
+MACHINE = mach-summit
 endif
 
 HEAD := arch/i386/kernel/head.o arch/i386/kernel/init_task.o

Can make handle reassigning a variable?

Other than that, looks like a good start to me (oh your email client is
wrapping lines of the patch...)

thanks,

greg k-h
-
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 : Tue Oct 15 2002 - 22:00:48 EST