Hi,
This has bothered me for a long time, and it just seems to be getting
worse. Can people please STOP defaulting non-essential stuff to 'y'?
Grrr.
diff --git a/drivers/macintosh/Kconfig b/drivers/macintosh/Kconfig
index 58926da..adbb5ca 100644
--- a/drivers/macintosh/Kconfig
+++ b/drivers/macintosh/Kconfig
@@ -2,7 +2,6 @@
menuconfig MACINTOSH_DRIVERS
bool "Macintosh device drivers"
depends on PPC || MAC || X86
- default y
if MACINTOSH_DRIVERS
diff --git a/drivers/media/Kconfig b/drivers/media/Kconfig
index 624b21c..986dcbc 100644
--- a/drivers/media/Kconfig
+++ b/drivers/media/Kconfig
@@ -89,7 +89,6 @@ config VIDEO_TVEEPROM
config DAB
boolean "DAB adapters"
- default y
---help---
Allow selecting support for for Digital Audio Broadcasting (DAB)
Receiver adapters.
--
Jens Axboe
On May 12 2007 20:23, Jens Axboe wrote:
>Hi,
>
>This has bothered me for a long time, and it just seems to be getting
>worse. Can people please STOP defaulting non-essential stuff to 'y'?
>Grrr.
http://lkml.org/lkml/2007/5/8/76
Jan
--
On Sat, May 12 2007, Jan Engelhardt wrote:
>
> On May 12 2007 20:23, Jens Axboe wrote:
> >Hi,
> >
> >This has bothered me for a long time, and it just seems to be getting
> >worse. Can people please STOP defaulting non-essential stuff to 'y'?
> >Grrr.
>
> http://lkml.org/lkml/2007/5/8/76
Sorry, I don't buy that reason at all - it's a short term advantage,
causing long term pain. It's not what we have done in the past, don't
start doing crap like that now.
--
Jens Axboe
On May 12 2007 21:27, Jens Axboe wrote:
>On Sat, May 12 2007, Jan Engelhardt wrote:
>>
>> On May 12 2007 20:23, Jens Axboe wrote:
>> >Hi,
>> >
>> >This has bothered me for a long time, and it just seems to be getting
>> >worse. Can people please STOP defaulting non-essential stuff to 'y'?
>> >Grrr.
>>
>> http://lkml.org/lkml/2007/5/8/76
>
>Sorry, I don't buy that reason at all - it's a short term advantage,
>causing long term pain. It's not what we have done in the past, don't
>start doing crap like that now.
Right, I shall create some patches to revert that and send them off
to Linus ASAP.
Jan
--
On Sat, May 12 2007, Jan Engelhardt wrote:
>
> On May 12 2007 21:27, Jens Axboe wrote:
> >On Sat, May 12 2007, Jan Engelhardt wrote:
> >>
> >> On May 12 2007 20:23, Jens Axboe wrote:
> >> >Hi,
> >> >
> >> >This has bothered me for a long time, and it just seems to be getting
> >> >worse. Can people please STOP defaulting non-essential stuff to 'y'?
> >> >Grrr.
> >>
> >> http://lkml.org/lkml/2007/5/8/76
> >
> >Sorry, I don't buy that reason at all - it's a short term advantage,
> >causing long term pain. It's not what we have done in the past, don't
> >start doing crap like that now.
>
> Right, I shall create some patches to revert that and send them off
> to Linus ASAP.
Thanks!
--
Jens Axboe
On 12/05/07 19:23, Jens Axboe wrote:
> Hi,
>
> This has bothered me for a long time, and it just seems to be getting
> worse. Can people please STOP defaulting non-essential stuff to 'y'?
> Grrr.
Is there a reason why various 10/100/1000Mbit network cards are 'y' too?
There's even a default SCSI 'm' that seems to be completely hidden from
the menu too (CONFIG_SCSI_WAIT_SCAN). It depends on SCSI but I can't
disable SCSI...
--
Simon Arlott
> Sorry, I don't buy that reason at all - it's a short term advantage,
> causing long term pain. It's not what we have done in the past, don't
> start doing crap like that now.
It doesn't really matter what it defaults too
cp .config somewheresafe
Install new kernel
cp it back
make oldconfig
Alan
On Sat, May 12 2007, Simon Arlott wrote:
> On 12/05/07 19:23, Jens Axboe wrote:
> >Hi,
> >
> >This has bothered me for a long time, and it just seems to be getting
> >worse. Can people please STOP defaulting non-essential stuff to 'y'?
> >Grrr.
>
> Is there a reason why various 10/100/1000Mbit network cards are 'y' too?
> There's even a default SCSI 'm' that seems to be completely hidden from
> the menu too (CONFIG_SCSI_WAIT_SCAN). It depends on SCSI but I can't
> disable SCSI...
For the exact same (wrong) reason that the other menuconfig changes did
it, I suppose. Need fixing, too.
--
Jens Axboe
On Sat, May 12 2007, Alan Cox wrote:
> > Sorry, I don't buy that reason at all - it's a short term advantage,
> > causing long term pain. It's not what we have done in the past, don't
> > start doing crap like that now.
>
> It doesn't really matter what it defaults too
It does matter what it defaults to, it should only default to y if it's
really needed. If it didn't matter, just default every driver to y?
Thanks, but no thanks.
> cp .config somewheresafe
>
> Install new kernel
>
> cp it back
>
> make oldconfig
Ehm yes, kernel updating 101. What's your point? Do that with your
2.6.21 config and get met with lots of new questions on ps2, ethernet,
macintosh drivers, dab, etc. All because of some menuconfig shuffling.
--
Jens Axboe
Hi Linus,
>> >> On May 12 2007 20:23, Jens Axboe wrote:
>> >> >Hi,
>> >> >
>> >> >This has bothered me for a long time, and it just seems to be getting
>> >> >worse. Can people please STOP defaulting non-essential stuff to 'y'?
>> >> >Grrr.
>> >>
>> >> http://lkml.org/lkml/2007/5/8/76
>> >
>> >Sorry, I don't buy that reason at all - it's a short term advantage,
>> >causing long term pain. It's not what we have done in the past, don't
>> >start doing crap like that now.
>>
>> Right, I shall create some patches to revert that and send them off
>> to Linus ASAP.
>
>Thanks!
Revert "default y" in Kconfig files that were introduced as part of my
menuconfig objects series (unfortunately, more patches may find their
way into mainline as subsystem maintainers merge). Apply this for now,
please.
Signed-off-by: Jan Engelhardt <[email protected]>
drivers/atm/Kconfig | 1 -
drivers/macintosh/Kconfig | 1 -
drivers/media/radio/Kconfig | 1 -
drivers/media/video/Kconfig | 2 --
drivers/net/Kconfig | 1 -
5 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/drivers/atm/Kconfig b/drivers/atm/Kconfig
index f5a47a4..b466a5c 100644
--- a/drivers/atm/Kconfig
+++ b/drivers/atm/Kconfig
@@ -5,7 +5,6 @@
menuconfig ATM_DRIVERS
bool "ATM drivers"
depends on NETDEVICES && ATM
- default y
if ATM_DRIVERS
diff --git a/drivers/macintosh/Kconfig b/drivers/macintosh/Kconfig
index 58926da..adbb5ca 100644
--- a/drivers/macintosh/Kconfig
+++ b/drivers/macintosh/Kconfig
@@ -2,7 +2,6 @@
menuconfig MACINTOSH_DRIVERS
bool "Macintosh device drivers"
depends on PPC || MAC || X86
- default y
if MACINTOSH_DRIVERS
diff --git a/drivers/media/radio/Kconfig b/drivers/media/radio/Kconfig
index a6ac82a..0486780 100644
--- a/drivers/media/radio/Kconfig
+++ b/drivers/media/radio/Kconfig
@@ -5,7 +5,6 @@
menuconfig RADIO_ADAPTERS
bool "Radio Adapters"
depends on VIDEO_DEV
- default y
---help---
Say Y here to enable selecting AM/FM radio adapters.
diff --git a/drivers/media/video/Kconfig b/drivers/media/video/Kconfig
index 5cb3f54..ca33d55 100644
--- a/drivers/media/video/Kconfig
+++ b/drivers/media/video/Kconfig
@@ -5,7 +5,6 @@
menuconfig VIDEO_CAPTURE_DRIVERS
bool "Video capture adapters"
depends on VIDEO_DEV
- default y
---help---
Say Y here to enable selecting the video adapters for
webcams, analog TV, and hybrid analog/digital TV.
@@ -689,7 +688,6 @@ config VIDEO_CAFE_CCIC
menuconfig V4L_USB_DRIVERS
bool "V4L USB devices"
depends on USB
- default y
if V4L_USB_DRIVERS
diff --git a/drivers/net/Kconfig b/drivers/net/Kconfig
index fb99cd4..80f5413 100644
--- a/drivers/net/Kconfig
+++ b/drivers/net/Kconfig
@@ -1901,7 +1901,6 @@ endmenu
menuconfig NETDEV_1000
bool "Ethernet (1000 Mbit)"
depends on !UML
- default y
if NETDEV_1000
#<EOF>
Jan
--
On Sat, 2007-05-12 at 20:58 +0100, Simon Arlott wrote:
> On 12/05/07 19:23, Jens Axboe wrote:
> > Hi,
> >
> > This has bothered me for a long time, and it just seems to be getting
> > worse. Can people please STOP defaulting non-essential stuff to 'y'?
> > Grrr.
>
> Is there a reason why various 10/100/1000Mbit network cards are 'y' too?
> There's even a default SCSI 'm' that seems to be completely hidden from
> the menu too (CONFIG_SCSI_WAIT_SCAN). It depends on SCSI but I can't
> disable SCSI...
That might not necessarily be our fault ... there's no default 'm' for
SCSI in drivers/scsi/Kconfig. On the other hand, the select directive
can also be the cause of these problems, for instance
drivers/ata/Kconfig has a select SCSI at the top, so if you selected
modular ATA, you'll be forced to have modular SCSI.
James
On 5/13/07, Simon Arlott <[email protected]> wrote:
> On 12/05/07 19:23, Jens Axboe wrote:
> > Hi,
> >
> > This has bothered me for a long time, and it just seems to be getting
> > worse. Can people please STOP defaulting non-essential stuff to 'y'?
> > Grrr.
>
> Is there a reason why various 10/100/1000Mbit network cards are 'y' too?
> There's even a default SCSI 'm' that seems to be completely hidden from
> the menu too (CONFIG_SCSI_WAIT_SCAN). It depends on SCSI but I can't
> disable SCSI...
Yeah, I've seen a lot of unnecessary default y's too, seems to be a
contagious disease.
And there are another kind of (harmless) default y's too, that don't really
force unnecessary compilation and linking of stuff into the kernel at all,
ones like (21-mm2's) init/Kconfig: SLUB_DEBUG = default y (it's
completely contained in mm/slub.c and not exposed to mm/Makefile),
but it still seems funny to find a spurious CONFIG_SLUB_DEBUG=y in
your .config when you also have # CONFIG_SLUB is not set.
Do we care enough to fix such cases too? This was a one-liner, so
including a patch anyway, CC'ing to Andrew and Christoph.
---
Prevent SLUB_DEBUG from being set if SLUB is not. SLUB_DEBUG
without SLUB does not cause anything to be compiled or linked in
anyway, so this is not critical, but simply removes a spurious
CONFIG_SLUB_DEBUG=y from creeping into the .config even when
CONFIG_SLUB is not set.
Signed-off-by: Satyam Sharma <[email protected]>
---
diff -ruNp a-mm2/init/Kconfig b-mm2/init/Kconfig
--- a-mm2/init/Kconfig 2007-05-10 23:19:36.000000000 +0530
+++ b-mm2/init/Kconfig 2007-05-13 02:33:21.000000000 +0530
@@ -567,7 +567,7 @@ config VM_EVENT_COUNTERS
if VM event counters are disabled.
config SLUB_DEBUG
- default y
+ default y if SLUB
bool "Enable SLUB debugging support" if EMBEDDED
help
SLUB has extensive debug support features. Disabling these can
Hi Jan,
All the submenus bellow are dependent of VIDEO_DEV (Video4Linux core).
If someone wants V4L, it is very likely that he will select a radio or a
video adapter, since the subsystem is useless without the drivers (*).
(*) Except if you are using an out-of-tree driver.
> diff --git a/drivers/media/radio/Kconfig b/drivers/media/radio/Kconfig
> index a6ac82a..0486780 100644
> --- a/drivers/media/radio/Kconfig
> +++ b/drivers/media/radio/Kconfig
> @@ -5,7 +5,6 @@
> menuconfig RADIO_ADAPTERS
> bool "Radio Adapters"
> depends on VIDEO_DEV
> - default y
> ---help---
> Say Y here to enable selecting AM/FM radio adapters.
>
> diff --git a/drivers/media/video/Kconfig b/drivers/media/video/Kconfig
> index 5cb3f54..ca33d55 100644
> --- a/drivers/media/video/Kconfig
> +++ b/drivers/media/video/Kconfig
> @@ -5,7 +5,6 @@
> menuconfig VIDEO_CAPTURE_DRIVERS
> bool "Video capture adapters"
> depends on VIDEO_DEV
> - default y
> ---help---
> Say Y here to enable selecting the video adapters for
> webcams, analog TV, and hybrid analog/digital TV.
> @@ -689,7 +688,6 @@ config VIDEO_CAFE_CCIC
> menuconfig V4L_USB_DRIVERS
> bool "V4L USB devices"
> depends on USB
> - default y
It should be noticed that selecting the "Drivers" Kconfig items won't
produce any code. No drivers are marked with default.
Those items are just used to better organize Multimedia menu. The result
of having 'y' is just to have the submenus open by default. For me, it
makes sense to keep default y, especially if we consider the arguments
at:
http://lkml.org/lkml/2007/5/8/76
This is also the same case for DVB drivers submenu:
menuconfig DVB_CAPTURE_DRIVERS
bool "DVB/ATSC adapters"
depends on DVB_CORE
default y
---help---
Say Y to select Digital TV adapters
Just to keep the entire menu coherent, "default y" were added for DAB:
config DAB
boolean "DAB adapters"
default y
---help---
Allow selecting support for for Digital Audio Broadcasting (DAB)
Receiver adapters.
if DAB
config USB_DABUSB
tristate "DABUSB driver"
depends on USB
---help---
A Digital Audio Broadcasting (DAB) Receiver for USB and Linux
brought to you by the DAB-Team
<http://wwwbode.cs.tum.edu/Par/arch/dab/>. This driver can be taken
as an example for URB-based bulk, control, and isochronous
transactions. URB's are explained in
<Documentation/usb/URB.txt>.
To compile this driver as a module, choose M here: the
module will be called dabusb.
endif # DAB
It is fine for me to remove "default y" for DAB, although i would prefer
to have it marked as 'y' just for coherence.
Cheers,
Mauro
James Bottomley <[email protected]> writes:
> drivers/ata/Kconfig has a select SCSI at the top, so if you selected
> modular ATA, you'll be forced to have modular SCSI.
Or built-in.
--
Krzysztof Halasa