[openib-general][kdapl]: vmalloc instead of kmalloc

Muli Ben-Yehuda mulix at mulix.org
Tue Aug 2 08:40:19 PDT 2005


On Tue, Aug 02, 2005 at 06:24:49PM +0300, Guy German wrote:

> There are some places where kmalloc might not be enough :
> in dapl_evd_event_alloc there is an allocation:
> 
> 	event = kmalloc(evd->qlen * sizeof *event);
> 
> whereas evd->qlen can be 128k (depends on max_cqe of the hca) 
> and kmalloc would fail.
> 
> The same goes to dapl_rbuf_alloc.
> 
> Is it legit to replace those kmallocs with vmallocs ?

Why do we need such a large allocation? 
To answer your question, vmalloc has a performance overhead and can
and will fail when vmalloc-space is exhausted (as can kmalloc, for
different reasons). Can this allocation be cut down so that it becomes
a non-issue?

Cheers,
Muli
-- 
Muli Ben-Yehuda
http://www.mulix.org | http://mulix.livejournal.com/




More information about the general mailing list