[Chad David copied for last comment in message.]

On Sun, 30 Dec 2001 15:28:23 +0300, Maxim Konovalov wrote:

> > sys/dev/ccd/ccd.c:ccdinit() has a couple of very large items on
> > the stack.
> >
> > Rewrite ccdinit() to allocate them with MALLOC(9) instead.
> 
> tmppath is a rather big one but I can't find the second item. What
> about this patch:

I think the others are the partinfo and ccdgeom structures.

Note that you don't need to (and shouldn't as per style(9)) initialize
tmppath to NULL.

Also, your bzero() is unnecessary if you use the M_ZERO flag to
MALLOC(9).

As an aside, what's the undocumented M_DEVBUF flag for?

Ciao,
Sheldon.

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-current" in the body of the message

Reply via email to