lkml.org 
[lkml]   [2005]   [Nov]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] make miniconfig (take 2)
Hi,

On Sun, 27 Nov 2005, Rob Landley wrote:

> The current miniconfig.sh is based on a very simple (and slow) procedure, and
> even then I still haven't figured out why miniconfig.sh run on a straight
> allnoconfig insists that CONFIG_PM should be set. (It correctly eliminates
> everything else...)

It seems because ACPI takes itself too important and sets the default to y
and this selects PM and prevents it from setting to n, until ACPI is
reset. default and select is a really bad combination. :-(

> > I think it can even be done in a single pass over all the symbols, where
> > boolean/tristate symbols are checked if they are already at the minimum
> > value and string/hex/int values are compared with their default values.
>
> Minimum value?

This is actually documented. :)
n - 0, m - 1, y - 2

> > Next step could be to add a variation of allnoconfig with better error
> > checking (e.g. checking that all requested symbols have been set),
>
> Um, I thought my patch did that. If any unrecognized symbols were
> encountered, my miniconfig patch would report it and exit with an error by
> the simple expedient of making the warning count a global and checking it
> afterwards. (I did a sort of -Werror for kconfig.) If it attempts to set an
> unrecognized symbol, it would already generate a warning, and if the warning
> count is nonzero it bails out with an error at the end. Seemed to work quite
> well, for me anyway...
>
> What cases would that not catch?

Symbols can be hidden by new dependencies.

> Good point, but the existing format is 90% of the gain for 10% of the effort.
> Going from .config to miniconfig for my laptop's kernel, for example, goes
> from 1370 lines to 138 lines, almost exactly a 10x reduction. And that can
> be done (admittedly badly) today, with the patch I posted.
>
> Dropping that 138 down to 120, or even to 100, is a polishing step in
> comparison. Do you think there are another 30 lines that could be trimmed
> out of that 138? (Attached.)

Yes, some symbols are hidden behind a lot of dependencies, if a user wants
to enable a new option, he only adds one new option and kconfig can try to
figure out the missing options.

bye, Roman
-
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/

\
 
 \ /
  Last update: 2005-11-29 02:03    [W:0.179 / U:1.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site