LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: David Newall <davidn@davidnewall.com>
To: Adrian Bunk <bunk@kernel.org>
Cc: Ingo Molnar <mingo@elte.hu>, Andi Kleen <andi@firstfloor.org>,
	akpm@osdl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH for mm] Remove iBCS support
Date: Fri, 25 Jan 2008 12:44:05 +1030	[thread overview]
Message-ID: <479945ED.7040007@davidnewall.com> (raw)
In-Reply-To: <20080124181442.GE4476@does.not.exist>

Adrian Bunk wrote:
> On Fri, Jan 25, 2008 at 04:25:24AM +1030, David Newall wrote:
>   
>> The performance benefit is trivial, and the improvement to
>> maintainability is even less.
>>     
>
> The effects become bigger when you realize that there are many such 
> places in the kernel.
>
> And the benefit of keeping it is zero.
>   

The benefit is not zero.  Repeating myself: While the code is there, it
encourages either removal or repair.  If the option to remove is taken
off the table then it will eventually be repaired.

> What you are doing is not contributing but wasting other people's time.
>   
You want to remove the code so you attack me.  Sadly for you, your
personal taste is irrelevant to the benefit that I bring.  What kind of
a person considers robust debate to be a waste of time?  A  bit
pathetic, sadly.

> The only thing you could ever achieve with this kind of "contribution" 
> is to end up in some killfiles.
>   

I'm comfortable with that.  I'm also comfortable that consensus might go
against me.  This childish threat of kill-files is not going to stop me.

>>>> At one stage iBCS2 support DID work.  Now it doesn't.  Now there's an
>>>> argument that the remaining infrastructure should be removed.  This is
>>>> the wrong direction to take.
>>>>     
>>>>         
>>> When did iBCS2 support work in a plain ftp.kernel.org kernel?
>>>   
>>>       
>> I don't know when.  Are you disputing that it ever did?  I think it's a
>> given that once it worked.
>>     
>
> AFAIK the kernel never shipped with iBCS2 support.
>   

Are you claiming that it never did?  Is that even important?  Clearly
there was support for it in the mainline kernel.  Anecdotally the
support worked.

...
> The point is that ideas do not turn themselves into code.
>   
This discussion is about removing code.  That's a bit like tearing down
the pergola because the vine has shrivelled.  Easy to do, but
counter-productive.  LIkewise, removing iBCS2 code would be
unproductive.  It would achieve no benefit, whilst simultaneously
leading Linux in the wrong direction.  This is a point you have
consistently failed to address.

> And there are far too many people who want to see their great ideas 
> implemented without implementing it themselves.
>   
This is not about a great idea.  It's about a pointless idea.  Even
allowing what you say to be true, and it probably is, there is nothing
wrong with somebody having a great idea and leaving it to others to
implement.  If the only people allowed to have great ideas were those
who could implement them then the world would be a much poorer place. 
You demonstrate a twisted view of value.

> Talking about a feature without having anyone willing to implement it 
> simply has no value.

Who said nobody is willing to implement it?  We've all recently learned
that there is a patch.  From there to implementation is much closer than
you or I thought last week.  So already this discussion has prompted
tangible benefit.

  reply	other threads:[~2008-01-25  2:14 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-19  4:59 Andi Kleen
2008-01-20  2:27 ` David Newall
2008-01-20  3:11   ` Andi Kleen
2008-01-20  4:46     ` David Newall
2008-01-20  5:18       ` Andi Kleen
2008-01-20  5:33         ` David Newall
2008-01-20  5:55           ` Andi Kleen
2008-01-20  6:23             ` David Newall
2008-01-20  7:29               ` Andi Kleen
2008-01-21  1:37                 ` David Newall
2008-01-22 10:24                   ` Karl Kiniger
2008-01-22 15:06                     ` David Newall
2008-01-22 15:52                       ` Adrian Bunk
2008-01-23  8:48                     ` Andi Kleen
2008-01-23 14:12                       ` Karl Kiniger
2008-01-23 14:24                         ` Andi Kleen
2008-01-24 17:06                           ` David Newall
2008-01-22 11:12                   ` Ingo Molnar
2008-01-22 12:42                     ` Karl Kiniger
2008-01-22 15:13                     ` David Newall
2008-01-22 15:49                       ` Ingo Molnar
2008-01-24 17:01                         ` David Newall
2008-01-22 16:01                       ` Adrian Bunk
2008-01-24 17:04                         ` David Newall
2008-01-24 17:24                           ` Adrian Bunk
2008-01-24 17:55                             ` David Newall
2008-01-24 18:14                               ` Adrian Bunk
2008-01-25  2:14                                 ` David Newall [this message]
2008-01-25  5:12                                   ` Valdis.Kletnieks
2008-01-25 16:40                                   ` Alan Cox
2008-01-24 19:51                               ` Pavel Machek
2008-01-25  2:17                                 ` David Newall
2008-01-24 20:37                               ` Andi Kleen
2008-01-25  2:16                                 ` David Newall
2008-01-22 16:50                       ` Alan Cox
2008-01-24 17:08                         ` David Newall
2008-01-22 13:20                 ` Giulio
2008-01-20 13:06           ` Alan Cox
2008-01-20 13:43             ` David Newall
2008-01-20 13:51               ` Alan Cox
2008-01-25 12:17 ` Ingo Molnar

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=479945ED.7040007@davidnewall.com \
    --to=davidn@davidnewall.com \
    --cc=akpm@osdl.org \
    --cc=andi@firstfloor.org \
    --cc=bunk@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --subject='Re: [PATCH for mm] Remove iBCS support' \
    /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

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