Date: Thu, 5 Aug 2004 13:16:37 +0200 From: Divacky Roman <xdivac02@stud.fit.vutbr.cz> To: Max Laier <max@love2party.net> Cc: Dan Cojocar <dan.cojocar@gmail.com> Subject: Re: problems with altq Message-ID: <20040805111637.GA29010@stud.fit.vutbr.cz> In-Reply-To: <20040805105048.GA28144@stud.fit.vutbr.cz> References: <20040804125640.GA78633@stud.fit.vutbr.cz> <20040804130530.GA78973@stud.fit.vutbr.cz> <200408042003.07356.max@love2party.net> <20040805105048.GA28144@stud.fit.vutbr.cz>
next in thread | previous in thread | raw e-mail | index | archive | help
On Thu, Aug 05, 2004 at 12:50:48PM +0200, Divacky Roman wrote: > On Wed, Aug 04, 2004 at 08:03:00PM +0200, Max Laier wrote: > > On Wednesday 04 August 2004 15:05, Divacky Roman wrote: > > > struct pf_altq *altq, *a; > > > > > > if (pa->ticket != ticket_altqs_inactive) { > > > error = EBUSY; > > > break; > > > } > > > altq = pool_get(&pf_altq_pl, PR_NOWAIT); > > > ^^^^^^^^^^ > > > shouldnt this be initalized to NULL? at least its not nice to have it > > > uninitalized... > > > > hum? > > $ cd /usr/src/sys/contrib/pf/net && grep pf_altq_pl * > > <...> > > pf_ioctl.c: pf_state_pl = pf_altq_pl = pf_pooladdr_pl = NULL; > > pf_ioctl.c: UMA_DESTROY(pf_altq_pl); > > pf_ioctl.c: UMA_CREATE(pf_altq_pl, struct pf_altq, "pfaltqpl"); > > <...> > > > > Or what are you thinking about? > > pf/net/pf_ioctl.c:2032 and below > > hm... I got it wrong. I looked at it again and I changed type with variable > name ;( > > hysteria.sk/~neologism/NEOLOGISM is my kernel config > hysteria.sk/~neologism/pf.conf is my whole pf config > > I am compiling kernel with -Os (I know its not supported) I'll try with plain > -O > > btw: why does the malloc has to be NOWAIT? kernel compiled with -O and M_WAITOK doesnt help either
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20040805111637.GA29010>