lkml.org 
[lkml]   [2002]   [Jul]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: BKL removal
On Sun, Jul 07, 2002 at 05:07:05PM -0700, Dave Hansen wrote:
>
> You are taking this example way too seriously. Thunder wanted an
> example and I grabbed the first one that I saw (I created it in the
> last hour). I showed you how I arrived at it, just a quick grepping.
> It wan't a real patch, only a quick little example snippet.

I know that, you're taking my response way too seriously :)
I just showed the typical response to one of your posts, you just picked
the wrong example, or maybe any USB example you could have picked would
have evicted much the same response :)

> > - even if you remove the BKL from this code, what have you
> > achieved? A faster kernel? A very tiny bit smaller kernel,
> > yes, but not any faster overall. This is not on _any_
> > critical path.
>
> How many times do I have to say it? We're going around in circles
> here. I _know_ that it isn't on a critical path, or saving a large
> quantity of program text. I just think that it is better than it was
> before.

So you agree with me? Good. I know you think the code is better than
it was before, but beauty is in the eye of the beholder, or in this
case, the eye of the people that fully understand the code :)

If nothing else, I hope you will think twice before sending off your
next BKL removel patch in a subsystem that you haven't fully tested or
understood. That's the point I keep trying to get across here.

thanks,

greg k-h
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:27    [W:0.172 / U:4.404 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site