xfs-masters
[Top] [All Lists]

[xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (r

To: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Subject: [xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (rc5-mm2)
From: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
Date: Tue, 03 Apr 2007 17:18:25 +1000
Cc: lkml - Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>, linux-mm@xxxxxxxxx, xfs-masters@xxxxxxxxxxx, reiserfs-dev@xxxxxxxxxxx
In-reply-to: <20070402230954.27840721.akpm@xxxxxxxxxxxxxxxxxxxx>
References: <1175571885.12230.473.camel@xxxxxxxxxxxxxxxxxxxxx> <20070402205825.12190e52.akpm@xxxxxxxxxxxxxxxxxxxx> <1175575503.12230.484.camel@xxxxxxxxxxxxxxxxxxxxx> <20070402215702.6e3782a9.akpm@xxxxxxxxxxxxxxxxxxxx> <1175579225.12230.504.camel@xxxxxxxxxxxxxxxxxxxxx> <20070402230954.27840721.akpm@xxxxxxxxxxxxxxxxxxxx>
Reply-to: xfs-masters@xxxxxxxxxxx
Sender: xfs-masters-bounce@xxxxxxxxxxx
On Mon, 2007-04-02 at 23:09 -0700, Andrew Morton wrote:
> hm, well, six-of-one, VI of the other.  We save maybe four kmallocs across
> the entire uptime at the cost of exposing stuff kernel-side which doesn't
> need to be exposed.

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.

Sure, I'll apply on top of that patch.

Thanks!
Rusty.


<Prev in Thread] Current Thread [Next in Thread>