linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Rolf Eike Beer" <eike-kernel@sf-tec.de>
To: "Al Viro" <viro@ZenIV.linux.org.uk>
Cc: "Linus Torvalds" <torvalds@linux-foundation.org>,
	"Stefan Richter" <stefanr@s5r6.in-berlin.de>,
	"Nicholas A. Bellinger" <nab@linux-iscsi.org>,
	"linux-kernel" <linux-kernel@vger.kernel.org>,
	"linux-scsi" <linux-scsi@vger.kernel.org>,
	"linux-fsdevel" <linux-fsdevel@vger.kernel.org>,
	"Joel Becker" <jlbec@evilplan.org>,
	"Randy Dunlap" <randy.dunlap@oracle.com>,
	"James Bottomley" <james.bottomley@suse.de>
Subject: Re: [PATCH] configfs: change depends -> select SYSFS
Date: Mon, 17 Jan 2011 08:11:17 +0100	[thread overview]
Message-ID: <48f447ae8623cdca64e23f05fc46ba7f.squirrel@webmail.sf-mail.de> (raw)
In-Reply-To: <20110117000558.GG22723@ZenIV.linux.org.uk>

> On Sun, Jan 16, 2011 at 03:45:35PM -0800, Linus Torvalds wrote:
>
>> And now ask yourself, do you really expect some random user to say "I
>> want to enable SYSFS and CONFIGFS, because I am going to use ocfs2 on
>> my system"?
>>
>> Really?
>>
>> Or do you expect a high-quality implementation of a configuration
>> script to allow the user to just say "I want ocfs2", and then figure
>> out the dependencies and solve them for you?
>>
>> I'd say that the latter case is OBVIOUSLY the quality implementation,
>> while the former one is just stupid.
>
> It's a bit more complicated than that...
> 	* nitpick: config option for a driver foo should be seen only after
> foo got enabled, so that's another case where we need depends on
> 	* nitpick 2: some switchable things really shouldn't be selectable,
> or you'll get users wondering which of the drivers they'd enabled had
> lead to 32bit kernel on their amd64 boxen ;-)
> 	* all nitpicks aside, there ought to be a way to say "no, I _REALLY_
> don't want that shit doing #define if(x) <horrible pile of crap> and if it
> means that I can't have oprofile, so be it".  Without chasing tons of
> select
> chains to see which FPOS do I need to disable to get rid of the damn
> thing.
> Right now the propagation of select chains makes it more and more
> unpleasant.
>
> IOW, it would be very nice to be able to pin a select target down, turning
> selects leading to it into hard dependencies.
>
> Another obviously missing thing is that dependencies should propagate back
> through select chains.  I.e. if A selects B, then hard dependencies of B
> should append to those of A.
>
> It gets really lousy when you have shitloads of drivers spread over
> several
> Kconfig menus, all selecting an option that has a hard dependency.  As it
> is, if you want e.g. allmodconfig to produce something that builds you
> have
> to chase them all down and add dependencies to each of those drivers.  We
> have such piles in e.g. drivers/media.

This sounds _exactly_ like doing a software package selection on distro
install. "No, I don't ever want mono because even if I am then missing
$foo". And it will likely need the same handling, e.g. as zypper does it
already: you don't want to install mono, but $foo depends on it. What to
do? 1) keep mono, 2) don't install $foo. The third option zypper usually
gives (ignore those dependency) usually doesn't make any sense for kernel
stuff so we can omit that.

For menuconfig this will end up that we have something beyond y-n-m, we
need also a forced-no as we currently have auto-m and auto-y.

Eike

  reply	other threads:[~2011-01-17  7:11 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-01-15 21:10 [PATCH] configfs: change depends -> select SYSFS Nicholas A. Bellinger
2011-01-16 13:11 ` Stefan Richter
2011-01-16 21:53   ` Nicholas A. Bellinger
2011-01-16 23:06     ` Stefan Richter
2011-01-16 23:11       ` Linus Torvalds
2011-01-16 23:19         ` Al Viro
2011-01-16 23:45           ` Linus Torvalds
2011-01-17  0:05             ` Al Viro
2011-01-17  7:11               ` Rolf Eike Beer [this message]
2011-01-17  0:52             ` Stefan Richter
2011-01-17  1:07               ` Randy Dunlap
2011-01-17 11:16               ` Bernd Petrovitsch
2011-01-16 23:14       ` Nicholas A. Bellinger
2011-01-17  0:22       ` James Bottomley
2011-01-17  6:05         ` Nicholas A. Bellinger
2011-01-17 18:24           ` Randy Dunlap
2011-01-17 22:10             ` Nicholas A. Bellinger
2011-01-18  0:13               ` Randy Dunlap
2011-01-18  0:46                 ` Nicholas A. Bellinger
2011-01-17 18:11         ` Randy Dunlap
2011-01-16 17:54 ` Linus Torvalds
2011-01-16 21:44   ` Nicholas A. Bellinger
  -- strict thread matches above, loose matches on Subject: below --
2011-01-10 19:45 Nicholas A. Bellinger
2011-01-10 23:46 ` Joel Becker

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=48f447ae8623cdca64e23f05fc46ba7f.squirrel@webmail.sf-mail.de \
    --to=eike-kernel@sf-tec.de \
    --cc=james.bottomley@suse.de \
    --cc=jlbec@evilplan.org \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=nab@linux-iscsi.org \
    --cc=randy.dunlap@oracle.com \
    --cc=stefanr@s5r6.in-berlin.de \
    --cc=torvalds@linux-foundation.org \
    --cc=viro@ZenIV.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).