Build scatterlist covering a process' text segment?

Jonathan Neuschäfer j.neuschaefer at gmx.net
Fri Jun 17 14:46:35 EDT 2011


On Fri, Jun 17, 2011 at 06:18:06PM +0300, Metin KAYA wrote:
> Jonathan, it's kmalloc(), not kcalloc().

No, I was trying to suggest using kcalloc(), which behaves roughly like
the userspace calloc, and implements overflow checking, which you may
need and zeroing, which you don't seem to need.

(see include/linux/slab.h, line 223 for reference)

> 
> On 17 Haz 2011, at 18:10, Jonathan Neuschäfer <j.neuschaefer at gmx.net> wrote:
> 
> > On Fri, Jun 17, 2011 at 02:47:25AM +0200, Arvid Brodin wrote:
> > [...]
> >>    /**** User page code ****/
> >> 
> >>    pages = kmalloc(nr_pages * sizeof(*pages), GFP_KERNEL);
> > 
> > kcalloc?
> > 
> > You don't need the zeroing but the overflow check might be useful.
> > 
> >>    if (!pages) {
> >>        retval = -ENOMEM;
> >>        goto err_pages;
> >>    }
> > 
> > Greetings,
> >    Jonathan Neuschäfer

HTH,
	Jonathan Neuschäfer



More information about the Kernelnewbies mailing list