Re: [TRIVIAL] [resend patch] CONFIG_X86_GENERIC description fixup

From: Dave Jones
Date: Thu Sep 04 2003 - 07:22:49 EST


On Thu, Sep 04, 2003 at 01:26:41PM +1000, Rusty Trivial Russell wrote:

> as per thread on lkml a little while ago, a better explanation
> of the X86_GENERIC config option follows. The person who questioned
> it originally seemed to like this improved version, so that's one point :)

How about explaining _exactly_ what it does? This is still a somewhat
mysterious description. "generic optimisations" what is that ?

> --- trivial-2.6.0-test4-bk5/arch/i386/Kconfig.orig 2003-09-04 13:02:02.000000000 +1000
> +++ trivial-2.6.0-test4-bk5/arch/i386/Kconfig 2003-09-04 13:02:02.000000000 +1000
> @@ -303,9 +303,13 @@
> config X86_GENERIC
> bool "Generic x86 support"
> help
> - Including some tuning for non selected x86 CPUs too.
> - when it has moderate overhead. This is intended for generic
> - distributions kernels.
> + Instead of just including optimizations for the selected
> + x86 variant (e.g. PII, Crusoe or Athlon), include some more
> + generic optimizations as well. This will make the kernel
> + perform better on x86 CPUs other than that selected.
> +
> + This is really intended for distributors who need more
> + generic optimizations.

All that it seems to do right now is set the cacheline size to the
same as it would if the kernel was compiled for P4, regardless of
what target CPU is selected, so that for eg, an i686 kernel won't
perform any worse on a P4, saving vendors shipping seperate P4 kernels.
it should also note what performance impact (if any) users of this
option will see if they run such a kernel on a box with a smaller
cacheline size.

If this option ever does anything else, that too should get documented here.

Dave

--
Dave Jones http://www.codemonkey.org.uk
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/