From owner-freebsd-pf@FreeBSD.ORG Wed Mar 8 15:24:12 2006 Return-Path: X-Original-To: freebsd-pf@freebsd.org Delivered-To: freebsd-pf@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 71E9816A420 for ; Wed, 8 Mar 2006 15:24:12 +0000 (GMT) (envelope-from max@love2party.net) Received: from moutng.kundenserver.de (moutng.kundenserver.de [212.227.126.186]) by mx1.FreeBSD.org (Postfix) with ESMTP id 6E5AD43D4C for ; Wed, 8 Mar 2006 15:24:11 +0000 (GMT) (envelope-from max@love2party.net) Received: from [84.163.239.198] (helo=amd64.laiers.local) by mrelayeu.kundenserver.de (node=mrelayeu0) with ESMTP (Nemesis), id 0MKwh2-1FH0WH3VHz-0008Gg; Wed, 08 Mar 2006 16:24:10 +0100 From: Max Laier Organization: FreeBSD To: Yar Tikhiy Date: Wed, 8 Mar 2006 16:21:44 +0100 User-Agent: KMail/1.9.1 References: <200603061610.k26GAJKR071335@repoman.freebsd.org> <20060308071210.GA99290@comp.chem.msu.su> In-Reply-To: <20060308071210.GA99290@comp.chem.msu.su> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart12712004.I3z3mcTqQp"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <200603081621.50675.max@love2party.net> X-Provags-ID: kundenserver.de abuse@kundenserver.de login:61c499deaeeba3ba5be80f48ecc83056 Cc: freebsd-pf@freebsd.org Subject: Re: cvs commit: src/etc/rc.d pflog src/sys/contrib/pf/net if_pflog.c if_pflog.h pf_ioctl.c src/sys/modules Makefile src/sys/modules/pf Makefile src/sys/modules/pflog Makefile X-BeenThere: freebsd-pf@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Technical discussion and general questions about packet filter \(pf\)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 08 Mar 2006 15:24:12 -0000 --nextPart12712004.I3z3mcTqQp Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Wednesday 08 March 2006 08:12, Yar Tikhiy wrote: > [moving this thread to freebsd-pf] > > On Mon, Mar 06, 2006 at 04:10:19PM +0000, Max Laier wrote: > > mlaier 2006-03-06 16:10:19 UTC > > > > FreeBSD src repository > > > > Modified files: (Branch: RELENG_6) > > etc/rc.d pflog > > sys/contrib/pf/net if_pflog.c if_pflog.h pf_ioctl.c > > sys/modules Makefile > > sys/modules/pf Makefile > > Added files: (Branch: RELENG_6) > > sys/modules/pflog Makefile > > Log: > > MFC: > > Make pflog a seperate module. As a result pflog_packet() becomes a > > function pointer that is declared in pf_ioctl.c > > > > Requested by: yar (as part of the module build reorg) > > > > Approved by: re (scottl) > > > > Revision Changes Path > > 1.5.2.2 +3 -3 src/etc/rc.d/pflog > > 1.13.2.2 +8 -1 src/sys/contrib/pf/net/if_pflog.c > > 1.6.2.1 +14 -0 src/sys/contrib/pf/net/if_pflog.h > > 1.20.2.2 +5 -0 src/sys/contrib/pf/net/pf_ioctl.c > > 1.450.2.12 +2 -0 src/sys/modules/Makefile > > 1.7.2.1 +0 -3 src/sys/modules/pf/Makefile > > 1.4.2.1 +27 -0 src/sys/modules/pflog/Makefile (new) > > Thanks a lot! > > BTW, are the DEV_PF and DEV_PFLOG #defines really called for in the > Makefiles? DEV_PF is not used explicitly in the whole /sys while > the explicit use of DEV_PFLOG has been eliminated from our kernel > by this change. When the kernel is built w/o static pf or pflog, > they will be undefined. I'd rather remove the #defines from the > Makefiles when doing my part of the job, integrating KERNBUILDDIR > stuff from HEAD to RELENG_6 in pf/Makefile. Just tested whether > the status of DEV_PF or DEV_PFLOG would affect the pf.ko or pflog.ko > binaries -- the result was negative. > > I also fail to recall if I proposed the attached patch to get rid > of the inclusion of from if_pflog.h. Both pf.ko and > pflog.ko build well with this patch applied, and it's a well-known > approach in general. Right, please go ahead. =2D-=20 /"\ Best regards, | mlaier@freebsd.org \ / Max Laier | ICQ #67774661 X http://pf4freebsd.love2party.net/ | mlaier@EFnet / \ ASCII Ribbon Campaign | Against HTML Mail and News --nextPart12712004.I3z3mcTqQp Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (FreeBSD) iD8DBQBEDvaOXyyEoT62BG0RAmK3AJ9uAQ2Qf8sxqTmYAJ5O+V/SGv0+dQCdFaaZ 985yRaTbwFDsM+glm0Z6jCY= =/2Na -----END PGP SIGNATURE----- --nextPart12712004.I3z3mcTqQp--