From: Manfred Spraul <manfred@colorfullife.com>
To: Badari Pulavarty <pbadari@us.ibm.com>
Cc: Andrew Morton <akpm@osdl.org>, linux-mm@kvack.org
Subject: Re: slab fragmentation ?
Date: Tue, 05 Oct 2004 20:49:17 +0200 [thread overview]
Message-ID: <4162ECAD.8090403@colorfullife.com> (raw)
In-Reply-To: <1097000846.12861.143.camel@dyn318077bld.beaverton.ibm.com>
Badari Pulavarty wrote:
>>The fix would be simple: kmem_cache_alloc_node must walk through the
>>list of partial slabs and check if it finds a slab from the correct
>>node. If it does, then just use that slab instead of allocating a new
>>one. And statistics must be added to kmem_cache_alloc_node - I forgot
>>that when I wrote the function.
>>
>>
>
>I will add more debug to find out if this is happening or not.
>
>What stats you want me to update in kmem_cache_alloc_node() ?
>
>
>
I would just add a printk to confirm our suspicion.
"kmem_cache_alloc_node called" + dump_stack(). I always use that
approach, thus I forgot to add proper statistics.
For the final fix:
Add a field to struct kmem_cache_s. Something like "unsigned long
alloc_node" or so. Add a STATS_INC_ALLOC_NODE macro and use it. Export
the field value in the globalstat block (see s_show() near the end of
slab.c) and increase the version number to 2.1 (in s_start()).
--
Manfred
--
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:"aart@kvack.org"> aart@kvack.org </a>
next prev parent reply other threads:[~2004-10-05 18:49 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-09-29 23:36 slab fragmentation ? Badari Pulavarty
2004-09-30 3:41 ` Andrew Morton
2004-09-30 4:52 ` badari
2004-09-30 14:49 ` Martin J. Bligh
2004-09-30 14:48 ` Badari Pulavarty
2004-10-03 6:04 ` Manfred Spraul
2004-10-04 15:51 ` Badari Pulavarty
2004-10-04 16:08 ` Manfred Spraul
2004-10-04 17:37 ` Badari Pulavarty
2004-10-05 14:46 ` Badari Pulavarty
2004-10-05 17:58 ` Manfred Spraul
2004-10-05 18:27 ` Badari Pulavarty
2004-10-05 18:49 ` Manfred Spraul [this message]
2004-10-05 18:47 ` Badari Pulavarty
2004-10-05 21:13 ` Badari Pulavarty
2004-10-05 22:11 ` Chen, Kenneth W
2004-10-05 22:18 ` Chen, Kenneth W
2004-10-06 14:58 ` Badari Pulavarty
2004-10-09 14:28 ` Manfred Spraul
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4162ECAD.8090403@colorfullife.com \
--to=manfred@colorfullife.com \
--cc=akpm@osdl.org \
--cc=linux-mm@kvack.org \
--cc=pbadari@us.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.