Hi,
On 1/25/06, Matthew Dobson <[email protected]> wrote:
> -static void *kmem_getpages(kmem_cache_t *cachep, gfp_t flags, int nodeid)
> +static void *kmem_getpages(kmem_cache_t *cachep, gfp_t flags, int nodeid,
> + mempool_t *pool)
> {
> struct page *page;
> void *addr;
> int i;
>
> flags |= cachep->gfpflags;
> - page = alloc_pages_node(nodeid, flags, cachep->gfporder);
> + /*
> + * If this allocation request isn't backed by a memory pool, or if that
> + * memory pool's gfporder is not the same as the cache's gfporder, fall
> + * back to alloc_pages_node().
> + */
> + if (!pool || cachep->gfporder != (int)pool->pool_data)
> + page = alloc_pages_node(nodeid, flags, cachep->gfporder);
> + else
> + page = mempool_alloc_node(pool, flags, nodeid);
You're not returning any pages to the pool, so the it will run out
pages at some point, no? Also, there's no guarantee the slab allocator
will give back the critical page any time soon either because it will
use it for non-critical allocations as well as soon as it becomes part
of the object cache slab lists.
Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
