2003-09-04 03:32:34

by Trivial Patch Monkey

[permalink] [raw]
Subject: [TRIVIAL] [resend patch] CONFIG_X86_GENERIC description fixup

From: Stewart Smith <[email protected]>

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 :)



--- 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.

#
# Define implied options from the CPU selection here
--
What is this? http://www.kernel.org/pub/linux/kernel/people/rusty/trivial/
Don't blame me: the Monkey is driving
File: Stewart Smith <[email protected]>: [resend patch] CONFIG_X86_GENERIC description fixup


2003-09-04 12:21:44

by Dave Jones

[permalink] [raw]
Subject: Re: [TRIVIAL] [resend patch] CONFIG_X86_GENERIC description fixup

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

2003-09-05 05:22:14

by Rusty Russell

[permalink] [raw]
Subject: Re: [TRIVIAL] [resend patch] CONFIG_X86_GENERIC description fixup

In message <[email protected]> you write:
> 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 ?

Whatever it says will go out of date too quickly 8(. I think this
patch will be fine.

Rusty.
--
Anyone who quotes me in their sig is an idiot. -- Rusty Russell.