Re: CONFIG_TINY

Tom Rini (trini@kernel.crashing.org)
Tue, 5 Nov 2002 19:05:03 -0700


On Tue, Nov 05, 2002 at 05:55:56PM +0000, Rob Landley wrote:
> On Tuesday 05 November 2002 19:56, Tom Rini wrote:
> > On Tue, Nov 05, 2002 at 02:26:08PM -0500, Bill Davidsen wrote:
> > > On Mon, 4 Nov 2002, Tom Rini wrote:
> > > > On Mon, Nov 04, 2002 at 02:13:48AM +0000, Rob Landley wrote:
> > > > > I've used -Os. I've compiled dozens and dozens of packages with -Os.
> > > > > It has always saved at least a few bytes, I have yet to see it make
> > > > > something larger. And in the benchmarks I've done, the smaller code
> > > > > actually runs slightly faster. More of it fits in cache, you know.
> > > >
> > > > Then we don't we always use -Os?
> >
> > [snip 6 good reasons]

Okay, let's just call it 5 then...

>
> Reasons 1 and 2 were that you can't be sure it works on all compiler versions
> and all platforms until you'e tried it, which you could say about anything.

Not every kernel person wants to, knows how to, or should have to debug
the compiler as well.

> Reason 3, 5, and 6 were about performance gains, when the point of CONFIG_TINY
> is, in fact, size.

CONFIG_TINY is in fact about asking questions which should reduce the
size, and allowing the user to determine space / speed tradeoffs.

> > So why do we want to force it on for CONFIG_TINY?
>
> 1) The point of CONFIG_TINY is size?

The point of CONFIG_TINY, as in the current patches is to offer a bunch
of tunables. From what I read of the patch, nothing actually uses
CONFIG_TINY, except for config.in bits. Therefore the point of
CONFIG_TINY is not about size, it's about asking questions. So ask
another question.

> The setting in question is a default value. CONFIG_TINY sets a lot of
> defaults at once, and gives you something grep for if you don't like them. I
> realise this isn't what you want, but objecting to patches because they're
> completely unrelated to what you want is kind of silly.

CONFIG_TINY sets no default values right now. Go look at
http://www.jaquet.dk/kernel/config_tiny/2.5.44-allinone and tell me
where CONFIG_TINY sets defaults.

Anyhow, I hope to have enough time to finish up a rough cut of my
template work this week and post something for comments. I've just got
to figure out how to get TWEAK_XXX to work in the Makefiles like
CONFIG_XXX does now.

-- 
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
-
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/