Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 24 Oct 2010 11:39:43 +0200
From:      Fabian Keil <freebsd-listen@fabiankeil.de>
To:        Max Laier <max@laiers.net>
Cc:        freebsd-net <freebsd-net@freebsd.org>, freebsd-pf@freebsd.org
Subject:   Re: [PATCH] pf(4) patch from OpenBSD 4.5
Message-ID:  <20101024113943.3cc0d659@r500.local>
In-Reply-To: <4CC3C5C9.7040904@laiers.net>
References:  <AANLkTinXNRKSwjuOeQkDTANhSSbHYHZnf4SvaFHbEdrg@mail.gmail.com> <4CC3C5C9.7040904@laiers.net>

next in thread | previous in thread | raw e-mail | index | archive | help

[-- Attachment #1 --]
Max Laier <max@laiers.net> wrote:

> C'mon ... where are the testers at?
> 
> On 18.10.2010 11:10, Ermal Luçi wrote:
> > Feedback is very welcome.
> 
> Is there no-one testing Ermal's exciting patch?  Let's help getting this 
> tested ... before we put it into SVN!
> 
> fetch http://people.freebsd.org/~eri/pf45_1.diff
> patch -p1 < pf45_1.diff

I think that should be -p0.

> make buildworld buildkernel
> etc. ...
> 
> Let's go!!!  And do not forget to reply if it works just, too.

I got a bunch of rejections:

fk@r500 /usr/src $patch -C -p0 < /home/fk/test/freebsd/pf45_1.diff 2>&1 | grep .rej
1 out of 6 hunks failed--saving rejects to contrib/pf/pfctl/pfctl_table.c.rej
1 out of 114 hunks failed--saving rejects to contrib/pf/pfctl/parse.y.rej
1 out of 37 hunks failed--saving rejects to contrib/pf/pfctl/pfctl.c.rej
1 out of 4 hunks failed--saving rejects to contrib/pf/pfctl/pfctl_parser.h.rej
1 out of 15 hunks failed--saving rejects to contrib/pf/pfctl/pfctl.8.rej
1 out of 6 hunks failed--saving rejects to contrib/pf/pfctl/pfctl.h.rej
1 out of 56 hunks failed--saving rejects to sys/contrib/pf/net/pf_norm.c.rej
1 out of 16 hunks failed--saving rejects to sys/contrib/pf/net/if_pflog.c.rej
1 out of 15 hunks failed--saving rejects to sys/contrib/pf/net/pf_osfp.c.rej

They can be fixed by running the files through
perl -i'' -pe 's@\$FreeBSD[^\$]+\$@\$FreeBSD\$@' *
before applying the patch.

Afterwards the patch applies cleanly. However the build still fails for me:

/usr/obj/usr/src/make.amd64/make -V CFILES -V SYSTEM_CFILES -V GEN_CFILES |  MKDEP_CPP="cc -E" CC="cc" xargs mkdep -a -f .newdep -O2 -frename-registers -pipe -fno-strict-aliasing  -std=c99 -g -Wall -Wredundant-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign -fformat-extensions -nostdinc  -I. -I/usr/src/sys -I/usr/src/sys/contrib/altq -I/usr/src/sys/contrib/ipfilter -I/usr/src/sys/contrib/pf -I/usr/src/sys/dev/ath -I/usr/src/sys/dev/ath/ath_hal -I/usr/src/sys/contrib/ngatm -I/usr/src/sys/dev/twa -I/usr/src/sys/gnu/fs/xfs/FreeBSD -I/usr/src/sys/gnu/fs/xfs/FreeBSD/support -I/usr/src/sys/gnu/fs/xfs -I/usr/src/sys/contrib/opensolaris/compat -I/usr/src/sys/dev/cxgb -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common -finline-limit=8000 --param inline-unit-growth=100 --param large-function-growth=1000  -fno-omit-frame-pointer -mcmodel=kernel -mno-red-zone  -mfpmath=387 -mno-sse -mno-sse2 -mno-sse3 -mno-mmx -mno-3dnow  -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -fstack-protector
/usr/src/sys/contrib/pf/net/pf.c:149:26: error: net/if_pflow.h: No such file or directory
mkdep: compile failed
*** Error code 1

Stop in /usr/obj/usr/src/sys/ZOEY.
*** Error code 1

Stop in /usr/src.
*** Error code 1

Stop in /usr/src.

I'm using HEAD from yesterday and if_pflow.h indeed doesn't seem to exist.

Fabian

[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.16 (FreeBSD)

iEYEARECAAYFAkzD/uQACgkQBYqIVf93VJ3jLQCgqUP61qe9WF0zBD2XbiCbnF76
+zAAn1zPE43YCnXL1+M9tTp5fcyyoCw0
=jEYJ
-----END PGP SIGNATURE-----

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20101024113943.3cc0d659>