lkml.org 
[lkml]   [2006]   [Jan]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [patch 2/6] Create and Use common mempool allocators
From
Date
Hi,

On Fri, 2006-01-27 at 16:19 -0800, Matthew Dobson wrote:
> - cc->page_pool = mempool_create(MIN_POOL_PAGES, mempool_alloc_page,
> - mempool_free_page, NULL);
> + cc->page_pool = mempool_create(MIN_POOL_PAGES, mempool_alloc_pages,
> + mempool_free_pages, 0);

You need to cast that zero to a void pointer to avoid compliation
warning (found in various other places as well). Would probably make
sense to implement helper functions so the casting doesn't spread all
over the place. Other than that, looks good to me.

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-01-28 11:10    [W:0.052 / U:0.196 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site