lkml.org 
[lkml]   [2007]   [Apr]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (rc5-mm2)
On Tue, 3 Apr 2007 22:37:06 +1000 David Chinner <dgc@sgi.com> wrote:

> On Tue, Apr 03, 2007 at 05:18:25PM +1000, Rusty Russell wrote:
> > On Mon, 2007-04-02 at 23:09 -0700, Andrew Morton wrote:
> > This is not about efficiency. When have I *ever* posted optimization
> > patches?
> >
> > This is about clarity. We have a standard convention for
> > register/unregister. And they can't fail. Either of these would be
> > sufficient to justify a change.
> >
> > Too many people doing cool new things in the kernel, not enough
> > polishing of the crap that's already there 8(
> >
> > > But I think we need to weed that crappiness out of XFS first.
>
> Can anyone else see the contradiction in these statements?
>
> XFS's "crappiness" is a register/unregister interface. The only
> reason it's being removed is because it's getting replaced with a
> nearly identical register/unregister interface.

Nope. XFS is introducing two new typedefs, one of which is identical to
one which we already have and it has wrapper functions which do little more
than add new names for existing stuff.

What Rusty is doing is changing the API so that the caller registers a
caller-owned struct rather than registering a caller-provided function.
For some reason.

-
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: 2007-04-03 19:39    [W:0.346 / U:0.144 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site