[RFC PATCH] slab: __GFP_NOWARN not being propagated from mempool_alloc()

[RFC PATCH] slab: __GFP_NOWARN not being propagated from mempool_alloc()

[Miklos Szeredi]

We see page allocation failure warnings on the mempool_alloc() path.
See this lkml posting for example:

http://lkml.org/lkml/2008/10/27/100

The cause is that on NUMA, alloc_slabmgmt() clears __GFP_NOWARN,
together with __GFP_THISNODE and __GFP_NORETRY.  But AFAICS it really
only wants to clear __GFP_THISNODE.

Does this patch looks good?

Warning: it's completely untested.

Miklos
---
 mm/slab.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Index: linux-2.6/mm/slab.c
===================================================================
--- linux-2.6.orig/mm/slab.c	2008-10-24 12:40:34.000000000 +0200
+++ linux-2.6/mm/slab.c	2008-11-24 22:17:04.000000000 +0100
@@ -2609,7 +2609,8 @@ static struct slab *alloc_slabmgmt(struc
 	if (OFF_SLAB(cachep)) {
 		/* Slab management obj is off-slab. */
 		slabp = kmem_cache_alloc_node(cachep->slabp_cache,
-					      local_flags & ~GFP_THISNODE, nodeid);
+					      local_flags & ~__GFP_THISNODE,
+					      nodeid);
 		if (!slabp)
 			return NULL;
 	} else {

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [RFC PATCH] slab: __GFP_NOWARN not being propagated from mempool_alloc()

[Pekka J Enberg]

On Mon, 24 Nov 2008, Miklos Szeredi wrote:
> We see page allocation failure warnings on the mempool_alloc() path.
> See this lkml posting for example:
> 
> http://lkml.org/lkml/2008/10/27/100
> 
> The cause is that on NUMA, alloc_slabmgmt() clears __GFP_NOWARN,
> together with __GFP_THISNODE and __GFP_NORETRY.  But AFAICS it really
> only wants to clear __GFP_THISNODE.
> 
> Does this patch looks good?

Yes, it does but looking at mm/slab.c history I think we want something 
like the following instead. Christoph?

P.S. First one to test it gets a fabulous prize of a Tested-by tag in the 
patch description! How cool is that?

		Pekka

Re: [RFC PATCH] slab: __GFP_NOWARN not being propagated from mempool_alloc()

[Christoph Lameter]

On Tue, 25 Nov 2008, Pekka J Enberg wrote:

> Yes, it does but looking at mm/slab.c history I think we want something
> like the following instead. Christoph?

Right.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [RFC PATCH] slab: __GFP_NOWARN not being propagated from mempool_alloc()

[Pekka Enberg]

On Tue, 25 Nov 2008, Pekka J Enberg wrote:
> > Yes, it does but looking at mm/slab.c history I think we want something
> > like the following instead. Christoph?

i>>?On Tue, 2008-11-25 at 12:58 -0600, Christoph Lameter wrote:
> Right.

OK, even though no tester showed up, I went ahead and merged my version
of the patch as it's a pretty obvious one-liner fix.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [RFC PATCH] slab: __GFP_NOWARN not being propagated from mempool_alloc()

[Miklos Szeredi]

On Wed, 26 Nov 2008, Pekka Enberg wrote:
> On Tue, 25 Nov 2008, Pekka J Enberg wrote:
> > > Yes, it does but looking at mm/slab.c history I think we want something
> > > like the following instead. Christoph?
> 
> i>>?On Tue, 2008-11-25 at 12:58 -0600, Christoph Lameter wrote:
> > Right.
> 
> OK, even though no tester showed up, I went ahead and merged my version
> of the patch as it's a pretty obvious one-liner fix.

Thanks.  I merged your version in the suse tree as well, so there'll
be lots of testers :)

Miklos

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>