* [PATCH] IP_VS should depend on EXPERIMENTAL ? @ 2007-07-20 11:44 Gabriel C 2007-07-20 11:59 ` Robert P. J. Day 2007-07-20 12:11 ` Robert P. J. Day 0 siblings, 2 replies; 7+ messages in thread From: Gabriel C @ 2007-07-20 11:44 UTC (permalink / raw) To: Linux Kernel Mailing List; +Cc: netdev Hi, IP_VS has : .. tristate "IP virtual server support (EXPERIMENTAL)" .. but it does not depend on EXPERIMENTAL. Signed-off-by: Gabriel Craciunescu <nix.or.die@googlemail.com> --- net/ipv4/ipvs/Kconfig | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/ipv4/ipvs/Kconfig b/net/ipv4/ipvs/Kconfig index 09d0c3f..3c594ec 100644 --- a/net/ipv4/ipvs/Kconfig +++ b/net/ipv4/ipvs/Kconfig @@ -3,7 +3,7 @@ # menuconfig IP_VS tristate "IP virtual server support (EXPERIMENTAL)" - depends on NETFILTER + depends on NETFILTER && EXPERIMENTAL ---help--- IP Virtual Server support will let you build a high-performance virtual server based on cluster of two or more real servers. This ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH] IP_VS should depend on EXPERIMENTAL ? 2007-07-20 11:44 [PATCH] IP_VS should depend on EXPERIMENTAL ? Gabriel C @ 2007-07-20 11:59 ` Robert P. J. Day 2007-07-20 12:17 ` Gabriel C 2007-07-20 12:11 ` Robert P. J. Day 1 sibling, 1 reply; 7+ messages in thread From: Robert P. J. Day @ 2007-07-20 11:59 UTC (permalink / raw) To: Gabriel C; +Cc: Linux Kernel Mailing List, netdev On Fri, 20 Jul 2007, Gabriel C wrote: > Hi, > > IP_VS has : > > .. > > tristate "IP virtual server support (EXPERIMENTAL)" > > .. > > but it does not depend on EXPERIMENTAL. > > > Signed-off-by: Gabriel Craciunescu <nix.or.die@googlemail.com> > > --- > > net/ipv4/ipvs/Kconfig | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/net/ipv4/ipvs/Kconfig b/net/ipv4/ipvs/Kconfig > index 09d0c3f..3c594ec 100644 > --- a/net/ipv4/ipvs/Kconfig > +++ b/net/ipv4/ipvs/Kconfig > @@ -3,7 +3,7 @@ > # > menuconfig IP_VS > tristate "IP virtual server support (EXPERIMENTAL)" > - depends on NETFILTER > + depends on NETFILTER && EXPERIMENTAL > ---help--- > IP Virtual Server support will let you build a high-performance > virtual server based on cluster of two or more real servers. This there's maturity-level inconsistency like that in a few places, like when stuff is tagged as EXPERIMENTAL, but labelled as OBSOLETE: ./net/ipv6/netfilter/Kconfig-config IP6_NF_QUEUE ./net/ipv6/netfilter/Kconfig- tristate "IP6 Userspace queueing via NETLINK (OBSOLETE)" ./net/ipv6/netfilter/Kconfig: depends on INET && IPV6 && NETFILTER && EXPERIMENTAL and other examples: ./net/ipv6/netfilter/Kconfig-config IP6_NF_IPTABLES ./net/ipv6/netfilter/Kconfig- tristate "IP6 tables support (required for filtering)" ./net/ipv6/netfilter/Kconfig: depends on INET && IPV6 && EXPERIMENTAL ... ./net/ipv6/Kconfig-config IPV6_MULTIPLE_TABLES ./net/ipv6/Kconfig- bool "IPv6: Multiple Routing Tables" ./net/ipv6/Kconfig: depends on IPV6 && EXPERIMENTAL ... ./net/ipv4/Kconfig-config TCP_CONG_HSTCP ./net/ipv4/Kconfig- tristate "High Speed TCP" ./net/ipv4/Kconfig: depends on EXPERIMENTAL ... ./net/rxrpc/Kconfig-config AF_RXRPC ./net/rxrpc/Kconfig- tristate "RxRPC session sockets" ./net/rxrpc/Kconfig: depends on INET && EXPERIMENTAL ... it might be a better investment in time to look through the Kconfig files, and decide which entries currently marked as "EXPERIMENTAL" really *shouldn't* be, and remove those silly dependencies. rday -- ======================================================================== Robert P. J. Day Linux Consulting, Training and Annoying Kernel Pedantry Waterloo, Ontario, CANADA http://fsdev.net/wiki/index.php?title=Main_Page ======================================================================== ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] IP_VS should depend on EXPERIMENTAL ? 2007-07-20 11:59 ` Robert P. J. Day @ 2007-07-20 12:17 ` Gabriel C 2007-07-20 12:21 ` Robert P. J. Day 0 siblings, 1 reply; 7+ messages in thread From: Gabriel C @ 2007-07-20 12:17 UTC (permalink / raw) To: Robert P. J. Day; +Cc: Linux Kernel Mailing List, netdev Robert P. J. Day wrote: > On Fri, 20 Jul 2007, Gabriel C wrote: > >> Hi, >> >> IP_VS has : >> >> .. >> >> tristate "IP virtual server support (EXPERIMENTAL)" >> >> .. >> >> but it does not depend on EXPERIMENTAL. >> >> >> Signed-off-by: Gabriel Craciunescu <nix.or.die@googlemail.com> >> >> --- >> >> net/ipv4/ipvs/Kconfig | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/net/ipv4/ipvs/Kconfig b/net/ipv4/ipvs/Kconfig >> index 09d0c3f..3c594ec 100644 >> --- a/net/ipv4/ipvs/Kconfig >> +++ b/net/ipv4/ipvs/Kconfig >> @@ -3,7 +3,7 @@ >> # >> menuconfig IP_VS >> tristate "IP virtual server support (EXPERIMENTAL)" >> - depends on NETFILTER >> + depends on NETFILTER && EXPERIMENTAL >> ---help--- >> IP Virtual Server support will let you build a high-performance >> virtual server based on cluster of two or more real servers. This > > there's maturity-level inconsistency like that in a few places, like > when stuff is tagged as EXPERIMENTAL, but labelled as OBSOLETE: > [ a lot examples ] I know that and there are a lot more things depending on 'EXPERIMENTAL' and not having EXPERIMENTAL visible all over the tree but that patch I've made for the _net_ part got NACK'ed while your maturity idea and I rm -rf'ed all the other. This one has a missing depends on EXPERIMENTAL while saying it is. So *could* we please stop this maturity stuff for now ? I don't see it in .23 nor .24 if at all. And even you get this kind changes for .24 almost a sed line will cleanup the whole tree from : ... "FOO blah (EXPERIMENTAL)" ... so what is your problem ? > rday Gabriel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] IP_VS should depend on EXPERIMENTAL ? 2007-07-20 12:17 ` Gabriel C @ 2007-07-20 12:21 ` Robert P. J. Day 2007-07-20 12:32 ` Gabriel C 0 siblings, 1 reply; 7+ messages in thread From: Robert P. J. Day @ 2007-07-20 12:21 UTC (permalink / raw) To: Gabriel C; +Cc: Linux Kernel Mailing List, netdev On Fri, 20 Jul 2007, Gabriel C wrote: > Robert P. J. Day wrote: > > On Fri, 20 Jul 2007, Gabriel C wrote: > > > >> Hi, > >> > >> IP_VS has : > >> > >> .. > >> > >> tristate "IP virtual server support (EXPERIMENTAL)" > >> > >> .. > >> > >> but it does not depend on EXPERIMENTAL. > >> > >> > >> Signed-off-by: Gabriel Craciunescu <nix.or.die@googlemail.com> > >> > >> --- > >> > >> net/ipv4/ipvs/Kconfig | 2 +- > >> 1 files changed, 1 insertions(+), 1 deletions(-) > >> > >> diff --git a/net/ipv4/ipvs/Kconfig b/net/ipv4/ipvs/Kconfig > >> index 09d0c3f..3c594ec 100644 > >> --- a/net/ipv4/ipvs/Kconfig > >> +++ b/net/ipv4/ipvs/Kconfig > >> @@ -3,7 +3,7 @@ > >> # > >> menuconfig IP_VS > >> tristate "IP virtual server support (EXPERIMENTAL)" > >> - depends on NETFILTER > >> + depends on NETFILTER && EXPERIMENTAL > >> ---help--- > >> IP Virtual Server support will let you build a high-performance > >> virtual server based on cluster of two or more real servers. This > > > > there's maturity-level inconsistency like that in a few places, like > > when stuff is tagged as EXPERIMENTAL, but labelled as OBSOLETE: > > > [ a lot examples ] > > I know that and there are a lot more things depending on > 'EXPERIMENTAL' and not having EXPERIMENTAL visible all over the tree > but that patch I've made for the _net_ part got NACK'ed while your > maturity idea and I rm -rf'ed all the other. > > This one has a missing depends on EXPERIMENTAL while saying it is. > > So *could* we please stop this maturity stuff for now ? I don't see > it in .23 nor .24 if at all. this has *nothing* to do with the aforementioned maturity levels. i understand entirely the inconsistency above. what i'm suggesting is that it might very well be more appropriate to *drop the dependency* rather than munge the prompt to add the qualifier. i think it's safe to say that there's *piles* of stuff in the Kconfig files that is still saddled with an EXPERIMENTAL dependency that's been around for years and has stabilized nicely. i mean, seriously, is IP virtual server support still "experimental" in any way? rday -- ======================================================================== Robert P. J. Day Linux Consulting, Training and Annoying Kernel Pedantry Waterloo, Ontario, CANADA http://fsdev.net/wiki/index.php?title=Main_Page ======================================================================== ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] IP_VS should depend on EXPERIMENTAL ? 2007-07-20 12:21 ` Robert P. J. Day @ 2007-07-20 12:32 ` Gabriel C 2007-07-20 13:04 ` Robert P. J. Day 0 siblings, 1 reply; 7+ messages in thread From: Gabriel C @ 2007-07-20 12:32 UTC (permalink / raw) To: Robert P. J. Day; +Cc: Linux Kernel Mailing List, netdev Robert P. J. Day wrote: > On Fri, 20 Jul 2007, Gabriel C wrote: > >> Robert P. J. Day wrote: >>> On Fri, 20 Jul 2007, Gabriel C wrote: >>> >>>> Hi, >>>> >>>> IP_VS has : >>>> >>>> .. >>>> >>>> tristate "IP virtual server support (EXPERIMENTAL)" >>>> >>>> .. >>>> >>>> but it does not depend on EXPERIMENTAL. >>>> >>>> >>>> Signed-off-by: Gabriel Craciunescu <nix.or.die@googlemail.com> >>>> >>>> --- >>>> >>>> net/ipv4/ipvs/Kconfig | 2 +- >>>> 1 files changed, 1 insertions(+), 1 deletions(-) >>>> >>>> diff --git a/net/ipv4/ipvs/Kconfig b/net/ipv4/ipvs/Kconfig >>>> index 09d0c3f..3c594ec 100644 >>>> --- a/net/ipv4/ipvs/Kconfig >>>> +++ b/net/ipv4/ipvs/Kconfig >>>> @@ -3,7 +3,7 @@ >>>> # >>>> menuconfig IP_VS >>>> tristate "IP virtual server support (EXPERIMENTAL)" >>>> - depends on NETFILTER >>>> + depends on NETFILTER && EXPERIMENTAL >>>> ---help--- >>>> IP Virtual Server support will let you build a high-performance >>>> virtual server based on cluster of two or more real servers. This >>> there's maturity-level inconsistency like that in a few places, like >>> when stuff is tagged as EXPERIMENTAL, but labelled as OBSOLETE: >>> >> [ a lot examples ] >> >> I know that and there are a lot more things depending on >> 'EXPERIMENTAL' and not having EXPERIMENTAL visible all over the tree >> but that patch I've made for the _net_ part got NACK'ed while your >> maturity idea and I rm -rf'ed all the other. >> >> This one has a missing depends on EXPERIMENTAL while saying it is. >> >> So *could* we please stop this maturity stuff for now ? I don't see >> it in .23 nor .24 if at all. > > this has *nothing* to do with the aforementioned maturity levels. i > understand entirely the inconsistency above. what i'm suggesting is > that it might very well be more appropriate to *drop the dependency* > rather than munge the prompt to add the qualifier. This is a thing the author/maintainer/subsystem maintainer should and need do. They know when something is not EXPERIMENTAL anymore. > > i think it's safe to say that there's *piles* of stuff in the Kconfig > files that is still saddled with an EXPERIMENTAL dependency that's > been around for years and has stabilized nicely. I agree with you , there may be a lot things are marked EXPERIMENTAL but aren't anymore. >i mean, seriously, > is IP virtual server support still "experimental" in any way? > > rday Gabriel ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] IP_VS should depend on EXPERIMENTAL ? 2007-07-20 12:32 ` Gabriel C @ 2007-07-20 13:04 ` Robert P. J. Day 0 siblings, 0 replies; 7+ messages in thread From: Robert P. J. Day @ 2007-07-20 13:04 UTC (permalink / raw) To: Gabriel C; +Cc: Linux Kernel Mailing List, netdev On Fri, 20 Jul 2007, Gabriel C wrote: > Robert P. J. Day wrote: > > this has *nothing* to do with the aforementioned maturity levels. > > i understand entirely the inconsistency above. what i'm > > suggesting is that it might very well be more appropriate to *drop > > the dependency* rather than munge the prompt to add the qualifier. > > This is a thing the author/maintainer/subsystem maintainer should and need do. > > They know when something is not EXPERIMENTAL anymore. agreed (sort of). all i'm saying (and after this, i'll shut the heck up about it) is that, if you find a Kconfig entry of the form: config FUBAR prompt "whatever" depends on ... && EXPERIMENTAL and you want to make that selection consistent, you have two possibilities: 1) add "(EXPERIMENTAL)" to the prompt to match the dependency, or 2) drop the dependency on EXPERIMENTAL to match the prompt i contend that, *for now*, it's a better investment in time to find entries like that that are *clearly* not experimental any more, and drop the dependency. not only will that make it consistent, but it's unlikely that you'll ever have to *revert* that decision. OTOH, it's quite possible that, after you add the prompt suffix of "(EXPERIMENTAL)" for consistency, the feature maintainer might come by tomorrow and say something like, "nah, that feature's been around for years, it's as stable as it gets," and will undo the patch you just made, wasting your time and effort. lastly, i'm not convinced that it's *only* the feature maintainer that can make that decision. surely, there's enough clever people here who can look at any given feature and say, "yeah, i've been using that for years, it's rock solid, it's stupid to keep that dependent on EXPERIMENTAL." all i'm saying is, if you want to put some time in here, it's better invested in *removing* what are clearly ridiculous dependencies on EXPERIMENTAL, rather than *adding* even more of that labelling to the tree. rday p.s. even if a given Kconfig entry *is* consistent, as in: config SNAFU bool "... (EXPERIMENTAL)" depends on ... && EXPERIMENTAL ... it's clear that there's a pile of *that* stuff for which all references to EXPERIMENTAL can be dropped. all in all, given the possible cleanup here, i'm thinking that going in and *adding* more EXPERIMENTAL clutter to the Kconfig files is going in exactly the wrong direction. -- ======================================================================== Robert P. J. Day Linux Consulting, Training and Annoying Kernel Pedantry Waterloo, Ontario, CANADA http://fsdev.net/wiki/index.php?title=Main_Page ======================================================================== ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] IP_VS should depend on EXPERIMENTAL ? 2007-07-20 11:44 [PATCH] IP_VS should depend on EXPERIMENTAL ? Gabriel C 2007-07-20 11:59 ` Robert P. J. Day @ 2007-07-20 12:11 ` Robert P. J. Day 1 sibling, 0 replies; 7+ messages in thread From: Robert P. J. Day @ 2007-07-20 12:11 UTC (permalink / raw) To: Gabriel C; +Cc: Linux Kernel Mailing List, netdev as a short followup to my previous post, at the *very least*, a cleanup that could be done now is to find all entries which have an actual dependency on "EXPERIMENTAL" but don't advertise themselves as such: ... config MA600_DONGLE tristate "Mobile Action MA600 dongle" depends on IRTTY_SIR && DONGLE && IRDA && EXPERIMENTAL ... and determine which of those should have the dependency on "EXPERIMENTAL" removed. no, don't go adding "(EXPERIMENTAL)" to the prompt -- just deal with *only* those for which the dependency is clearly out of date and not relevant anymore, and delete it. that would be, at least, a place to start. and, yes, there's piles of potential (net/ipv4/Kconfig): config TCP_CONG_HSTCP tristate "High Speed TCP" depends on EXPERIMENTAL ... config TCP_CONG_HYBLA tristate "TCP-Hybla congestion control algorithm" depends on EXPERIMENTAL ... config TCP_CONG_VEGAS tristate "TCP Vegas" depends on EXPERIMENTAL ... ... etc etc ... rday -- ======================================================================== Robert P. J. Day Linux Consulting, Training and Annoying Kernel Pedantry Waterloo, Ontario, CANADA http://fsdev.net/wiki/index.php?title=Main_Page ======================================================================== ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2007-07-20 13:04 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2007-07-20 11:44 [PATCH] IP_VS should depend on EXPERIMENTAL ? Gabriel C 2007-07-20 11:59 ` Robert P. J. Day 2007-07-20 12:17 ` Gabriel C 2007-07-20 12:21 ` Robert P. J. Day 2007-07-20 12:32 ` Gabriel C 2007-07-20 13:04 ` Robert P. J. Day 2007-07-20 12:11 ` Robert P. J. Day
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).