problems with altq
Divacky Roman
xdivac02 at stud.fit.vutbr.cz
Thu Aug 5 04:16:42 PDT 2004
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
More information about the freebsd-current
mailing list