Re: [PATCH] 2.5.69 Changes to Kconfig and i386 Makefile to include support for various K7 optimizations

Thomas Horsten (thomas@horsten.com)
Wed, 14 May 2003 19:40:10 +0100


On Wednesday 14 May 2003 15:04, Dave Jones wrote:

> > I made this patch to support the various K7 model-specific
> > optimizations that the later GCC compilers can use.
> >
> > Please have a look, and pass me any comments.
>
> I don't think this is worth the extra complication. The potential wins
> (if any) outweigh the confusion to users who might have no clue as to
> what core they have. Additionally, some gcc's got these options wrong.
> The athlon4 option was completely wrong for a while for eg.

I think when GCC supports the different cores, it should be supported by the
kernel scripts, the differences between the cores are real enough to have
potential optimizations at least in theory (as far as I could see the only
difference in GCC 3.2 is whether to use SSE, but that could change in the
future).

Maybe it should be renamed "Advanced CPU model selection", or something like
that, instead of "K7 Model Selection", but the help IMHO makes it clear
enough that you should go with the default if you are not sure and that it
will work on any core.

I think it's a fairly simple patch that doesn't break anything, and there are
certainly options lurking around in Kconfig that are much more obscure than
this :)

> > I also have the same patch for 2.4, if you are interested it's
> > available on my Linux page on http://www.infowares.com/linux
>
> For 2.4, it's an even worse idea IMO.

I completely agree that this shouldn't go into the stock 2.4 kernel, that's
also why I didn't post it here. It's just on my site as a convenience for
those who might want to use the XP specific options etc. in an easy way (like
I use it myself).

// Thomas
-
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/