| 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. |
| Previous by Date: | [xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (rc5-mm2), David Chinner |
|---|---|
| Next by Date: | [xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (rc5-mm2), Andi Kleen |
| Previous by Thread: | [xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (rc5-mm2), Andrew Morton |
| Next by Thread: | [xfs-masters] Re: [PATCH] Cleanup and kernelify shrinker registration (rc5-mm2), David Chinner |
| Indexes: | [Date] [Thread] [Top] [All Lists] |