xfs-masters
[Top] [All Lists]

[xfs-masters] Re: [PATCH] xfs: kill kmem_zone init

To: Nathan Scott <nathans@xxxxxxx>
Subject: [xfs-masters] Re: [PATCH] xfs: kill kmem_zone init
From: Pekka J Enberg <penberg@xxxxxxxxxxxxxx>
Date: Wed, 22 Mar 2006 09:37:33 +0200 (EET)
Cc: linux-kernel@xxxxxxxxxxxxxxx, xfs-masters@xxxxxxxxxxx
In-reply-to: <20060322085520.A664708@wobbly.melbourne.sgi.com>
References: <Pine.LNX.4.58.0603201501540.18684@sbz-30.cs.Helsinki.FI> <20060321082037.A653275@wobbly.melbourne.sgi.com> <Pine.LNX.4.58.0603210859450.14023@sbz-30.cs.Helsinki.FI> <20060322085520.A664708@wobbly.melbourne.sgi.com>
Reply-to: xfs-masters@xxxxxxxxxxx
Sender: xfs-masters-bounce@xxxxxxxxxxx
On Wed, 22 Mar 2006, Nathan Scott wrote:
> Regarding __GFP_NOFAIL, there are some situations where that could
> be used now, and others where it should not be - it'd take a very
> careful code audit and evidence of a level of low-memory-condition
> testing done, etc, before such a change would be merged.  Note that
> the -mm tree currently has a rework of the way incore extents are
> managed within XFS, which significantly changes (in a good way) the
> nature of the allocation requests we make (and hence I'm _really_,
> _really_ not interested in cosmetic patches in this area just now).

Yeah, just drop it then. After digging through your kmem_zone stuffs, I 
noticed you're using PF_FSTRANS in kmem_flags_convert() to detect GFP_NOFS 
cases which makes conversion to slab proper very difficult anyway. Would 
you accept patches to convert code under PF_FSTRANS to use KM_NOFS so we 
can kill the check in kmem_flags_convert()?

                                Pekka


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