lkml.org 
[lkml]   [2006]   [Mar]   [21]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateTue, 21 Mar 2006 09:05:14 +0200 (EET)
FromPekka J Enberg <>
SubjectRe: [PATCH] xfs: kill kmem_zone init
On Tue, 21 Mar 2006, Nathan Scott wrote:
> Sorry, but thats just silly.  Did you even look at the code
> around what you're changing (it has to do more than just wrap
> up slab calls)?  So, NACK on this patch - it leaves the code
> very confused (half zoney, half slaby), and is just unhelpful
> code churn at the end of the day.

You're already using kmem_cache_destroy() mixed with the zone stuff so I 
don't see your point. I would really prefer to feed small bits at a time 
so is there any way I can sweet-talk you into merging the patch?

On Tue, 21 Mar 2006, Nathan Scott wrote:
> For your zalloc patch, you will need to duplicate the logic
> in kmem_zone_alloc into kmem_zone_zalloc in order to use that
> new zalloc interface you're introducing - which should be fine.

I am planning to kill the slab wrappers completely. The logic you're 
referring to looks awful lot like GFP_NOFAIL with limiter. Any 
reason we can't just use GFP_NOFAIL for those cases?

				Pekka 
-
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: 2006-03-21 07:09    [from the cache]
©2003-2008