linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt
@ 2012-08-27 18:22 Steven Rostedt
  2012-08-29  1:28 ` Cong Wang
  0 siblings, 1 reply; 6+ messages in thread
From: Steven Rostedt @ 2012-08-27 18:22 UTC (permalink / raw)
  To: linux-kernel; +Cc: Linus Torvalds, Andrew Morton

The file feature-removal-schedule.txt is ignored by most people except
for people that add to it. It's more of a global TODO list for
developers than being anything useful by anyone.

Add a feature removal of removing the feature-removal-schedule.txt.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>

---
 Documentation/feature-removal-schedule.txt |   11 +++++++++++
 1 files changed, 11 insertions(+), 0 deletions(-)

diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index afaff31..dcbb573 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -6,6 +6,17 @@ be removed from this file.  The suggested deprecation period is 3 releases.
 
 ---------------------------
 
+What:	Documentation/feature-removal-schedule.txt
+When:	v3.8
+Why:	Developers add their TODO list here, and no one else reads it.
+	It causes a lot of silly conflicts that annoy Linus.
+	There are better ways to remove features than adding your crap here.
+	Add a printk() or 'CONFIG_FOO default n' and see who complains.
+	This file has been proven to be absolutely useless.
+Who:	Steven Rostedt <rostedt@goodmis.org>
+
+---------------------------
+
 What:	ddebug_query="query" boot cmdline param
 When:	v3.8
 Why:	obsoleted by dyndbg="query" and module.dyndbg="query"



^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt
  2012-08-27 18:22 [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt Steven Rostedt
@ 2012-08-29  1:28 ` Cong Wang
  2012-09-04 15:39   ` Steven Rostedt
  0 siblings, 1 reply; 6+ messages in thread
From: Cong Wang @ 2012-08-29  1:28 UTC (permalink / raw)
  To: Steven Rostedt; +Cc: linux-kernel, Linus Torvalds, Andrew Morton

On Tue, Aug 28, 2012 at 2:22 AM, Steven Rostedt <rostedt@goodmis.org> wrote:
> The file feature-removal-schedule.txt is ignored by most people except
> for people that add to it. It's more of a global TODO list for
> developers than being anything useful by anyone.
>
> Add a feature removal of removing the feature-removal-schedule.txt.
>

Hi, Steven,

I sometimes remind people to remove features scheduled in this file,
by sending a proposed patch. ;)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt
  2012-08-29  1:28 ` Cong Wang
@ 2012-09-04 15:39   ` Steven Rostedt
  2012-09-06 14:29     ` Cong Wang
  0 siblings, 1 reply; 6+ messages in thread
From: Steven Rostedt @ 2012-09-04 15:39 UTC (permalink / raw)
  To: Cong Wang; +Cc: linux-kernel, Linus Torvalds, Andrew Morton

On Wed, 2012-08-29 at 09:28 +0800, Cong Wang wrote:
> On Tue, Aug 28, 2012 at 2:22 AM, Steven Rostedt <rostedt@goodmis.org> wrote:
> > The file feature-removal-schedule.txt is ignored by most people except
> > for people that add to it. It's more of a global TODO list for
> > developers than being anything useful by anyone.
> >
> > Add a feature removal of removing the feature-removal-schedule.txt.
> >
> 
> Hi, Steven,
> 
> I sometimes remind people to remove features scheduled in this file,
> by sending a proposed patch. ;)

At Kernel Summit, Linus went on a rant about this file. He said he hates
it and is totally useless. Basically, it is only used by us developers,
and is not used by normal users. As it is a dumping ground for all
developers, this file becomes the biggest conflict during merges that
Linus does.

There are better ways to tell users something is about to go away. My
preference is a nasty WARN_ON_ONCE() in the code when the feature is
used. People tend to report call traces, and you can either tell the
person, "don't use this" or you know that it is something that is
actually used and you can't remove it.

-- Steve



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt
  2012-09-04 15:39   ` Steven Rostedt
@ 2012-09-06 14:29     ` Cong Wang
  2012-09-06 14:42       ` Steven Rostedt
  0 siblings, 1 reply; 6+ messages in thread
From: Cong Wang @ 2012-09-06 14:29 UTC (permalink / raw)
  To: Steven Rostedt; +Cc: linux-kernel, Linus Torvalds, Andrew Morton

On Tue, Sep 4, 2012 at 11:39 PM, Steven Rostedt <rostedt@goodmis.org> wrote:
> On Wed, 2012-08-29 at 09:28 +0800, Cong Wang wrote:
>> On Tue, Aug 28, 2012 at 2:22 AM, Steven Rostedt <rostedt@goodmis.org> wrote:
>> > The file feature-removal-schedule.txt is ignored by most people except
>> > for people that add to it. It's more of a global TODO list for
>> > developers than being anything useful by anyone.
>> >
>> > Add a feature removal of removing the feature-removal-schedule.txt.
>> >
>>
>> Hi, Steven,
>>
>> I sometimes remind people to remove features scheduled in this file,
>> by sending a proposed patch. ;)
>
> At Kernel Summit, Linus went on a rant about this file. He said he hates
> it and is totally useless. Basically, it is only used by us developers,
> and is not used by normal users. As it is a dumping ground for all
> developers, this file becomes the biggest conflict during merges that
> Linus does.
>
> There are better ways to tell users something is about to go away. My
> preference is a nasty WARN_ON_ONCE() in the code when the feature is
> used. People tend to report call traces, and you can either tell the
> person, "don't use this" or you know that it is something that is
> actually used and you can't remove it.
>

I don't think feature-removal-schedule.txt is for kernel users, it
is a kind of a TODO list for kernel developers.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt
  2012-09-06 14:29     ` Cong Wang
@ 2012-09-06 14:42       ` Steven Rostedt
  2012-09-06 15:42         ` Randy Dunlap
  0 siblings, 1 reply; 6+ messages in thread
From: Steven Rostedt @ 2012-09-06 14:42 UTC (permalink / raw)
  To: Cong Wang; +Cc: linux-kernel, Linus Torvalds, Andrew Morton

On Thu, 2012-09-06 at 22:29 +0800, Cong Wang wrote:

> I don't think feature-removal-schedule.txt is for kernel users, it
> is a kind of a TODO list for kernel developers.

And this is exactly what Linus was complaining about. It shouldn't be
used as a TODO list for developers.

-- Steve



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt
  2012-09-06 14:42       ` Steven Rostedt
@ 2012-09-06 15:42         ` Randy Dunlap
  0 siblings, 0 replies; 6+ messages in thread
From: Randy Dunlap @ 2012-09-06 15:42 UTC (permalink / raw)
  To: Steven Rostedt; +Cc: Cong Wang, linux-kernel, Linus Torvalds, Andrew Morton

On 09/06/2012 07:42 AM, Steven Rostedt wrote:

> On Thu, 2012-09-06 at 22:29 +0800, Cong Wang wrote:
> 
>> I don't think feature-removal-schedule.txt is for kernel users, it
>> is a kind of a TODO list for kernel developers.
> 
> And this is exactly what Linus was complaining about. It shouldn't be
> used as a TODO list for developers.


Were there some better alternatives discussed?

-- 
~Randy

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2012-09-06 16:26 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-27 18:22 [PATCH] Add feature-removal-schedule.txt removal to feature-removal-schedule.txt Steven Rostedt
2012-08-29  1:28 ` Cong Wang
2012-09-04 15:39   ` Steven Rostedt
2012-09-06 14:29     ` Cong Wang
2012-09-06 14:42       ` Steven Rostedt
2012-09-06 15:42         ` Randy Dunlap

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).