Re: [GIT] HID patches for 2.6.28

From: Ingo Molnar
Date: Wed Oct 15 2008 - 06:04:21 EST



* Jiri Kosina <jkosina@xxxxxxx> wrote:

> On Wed, 15 Oct 2008, Ingo Molnar wrote:
>
> > this or the I2C updates cause widespread build failures in latest -git:
>
> For the usbhid_set_leds(), Adrian has already sent me a fix [1].
> Still, I would like to understand whether this failure is bug or
> feature in select/depends handling in Kconfig, because it seems to
> behave differently from what I was expecting.

yes, it's a kconfig bug: Kconfig must either not allow complex selects
(during its initial parsing pass of the Kconfig dependnecy tree) or
handle the dependencies correctly.

The current "allow them silently and pepper the kernel with build
errors" approach is obviously silly.

We already have over 100 bugfixes in the tree from the past 3 years due
to kconfig's inability to figure out the correct dependencies. (count:
rising steadily)

Changes to the Kconfig tool to address this design bug did not get
picked up. See for example this quick patch from Vegard half a year ago:

http://lkml.org/lkml/2008/5/4/13

that patch was not adopted and the config options in the warning log
were later on frequent participants in build failures and subsequent
stream of trivial fixes.

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