Date: Thu, 19 Apr 2007 15:49:07 +0200 From: Max Laier <max@love2party.net> To: Marko Zec <zec@freebsd.org> Cc: Perforce Change Reviews <perforce@freebsd.org> Subject: Re: PERFORCE change 118399 for review Message-ID: <200704191549.13955.max@love2party.net> In-Reply-To: <200704190739.l3J7dFng023292@repoman.freebsd.org>
index | next in thread | previous in thread | raw e-mail
[-- Attachment #1 --] On Thursday 19 April 2007 09:39, Marko Zec wrote: > http://perforce.freebsd.org/chv.cgi?CH=118399 > > Change 118399 by zec@zec_tca51 on 2007/04/19 07:38:43 > > Fix to allow compilation with nooptions VIMAGE. > > Affected files ... > > .. //depot/projects/vimage/src/sys/contrib/pf/net/pf_ioctl.c#5 edit > > Differences ... > > ==== //depot/projects/vimage/src/sys/contrib/pf/net/pf_ioctl.c#5 > (text+ko) ==== > > @@ -401,10 +401,14 @@ > callout_reset(&V_pf_expire_to, my_timeout[PFTM_INTERVAL] * hz, > pf_purge_timeout, curvnet); > > -if (curvnet == &vnet_0) { > +#ifdef VIMAGE > + if (curvnet == &vnet_0) { > +#endif > pf_normalize_init(); > pf_pfil_hooked = 0; > -} > +#ifdef VIMAGE > + } > +#endif I don't quite understand this one. I'd believe that pf_pfil_hooked should be virtualized as well. Otherwise you can only enable/hook all instances at once. > bzero(&V_pf_status, sizeof(V_pf_status)); > /* XXX do our best to avoid a conflict */ -- /"\ Best regards, | mlaier@freebsd.org \ / Max Laier | ICQ #67774661 X http://pf4freebsd.love2party.net/ | mlaier@EFnet / \ ASCII Ribbon Campaign | Against HTML Mail and News [-- Attachment #2 --] -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.6 (FreeBSD) iD8DBQBGJ3NZXyyEoT62BG0RAohuAJ0VnhsPm20LFxQXESZEWZBMFWEPZwCeMMVH a1bts9w7/jrRpq90KIYnyvc= =sAWa -----END PGP SIGNATURE-----home | help
Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200704191549.13955.max>