network performance

[Kris Kennaway]

Stefan Lambrev wrote:
> 
> 
> Kris Kennaway wrote:
>> Stefan Lambrev wrote:
>>> Greetings,
>>>
>>> Kris Kennaway wrote:
>>>> http://www.freebsd.org/~kris/p4-net.tbz is a sys/ tarball from my p4 
>>>> branch, which includes these and other optimizations.
>>> I have some problems with compiling new kernel:
>>>
>>> cc -c -O2 -frename-registers -pipe -fno-strict-aliasing -march=nocona 
>>> -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 -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-mmx -mno-3dnow  
>>> -msoft-float -fno-asynchronous-unwind-tables -ffreestanding -Werror  
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c: In function 'ufs_lookup':
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c:171: error: 'td' undeclared (first 
>>> use in this function)
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c:171: error: (Each undeclared 
>>> identifier is reported only once
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c:171: error: for each function it 
>>> appears in.)
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c:173:45: error: macro "VOP_LOCK" 
>>> passed 3 arguments, but takes just 2
>>> /usr/src/sys/ufs/ufs/ufs_lookup.c:173: error: 'VOP_LOCK' undeclared 
>>> (first use in this function)
>>> *** Error code 1
>>>
>>>
>>
>> Sorry, forgot to check in a fix.  Apply this patch:
>>
>>   http://www.freebsd.org/~kris/ufs_lookup.c.diff
>>
>> (you'll need to specify the path by hand)
>>
>> Kris
> Now this compiles, but :
> 
> cc -O2 -fno-strict-aliasing -pipe -march=nocona -Werror -D_KERNEL 
> -DKLD_MODULE -std=c99 -nostdinc   -DHAVE_KERNEL_OPTION_HEADERS -include 
> /usr/obj/usr/src/sys/CORE/opt_global.h -I. -I@ -I@/contrib/altq 
> -finline-limit=8000 --param inline-unit-growth=100 --param 
> large-function-growth=1000 -fno-common -g -fno-omit-frame-pointer 
> -I/usr/obj/usr/src/sys/CORE -mcmodel=kernel -mno-red-zone  -mfpmath=387 
> -mno-sse -mno-sse2 -mno-mmx -mno-3dnow  -msoft-float 
> -fno-asynchronous-unwind-tables -ffreestanding -Wall -Wredundant-decls 
> -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes 
> -Wpointer-arith -Winline -Wcast-qual  -Wundef -Wno-pointer-sign 
> -fformat-extensions -c 
> /usr/src/sys/modules/mac_lomac/../../security/mac_lomac/mac_lomac.c
> cc1: warnings being treated as errors
> /usr/src/sys/modules/mac_lomac/../../security/mac_lomac/mac_lomac.c: In 
> function 'lomac_thread_userret':
> /usr/src/sys/modules/mac_lomac/../../security/mac_lomac/mac_lomac.c:2172: 
> warning: implicit declaration of function 'PROC_LOCK'
> /usr/src/sys/modules/mac_lomac/../../security/mac_lomac/mac_lomac.c:2172: 
> warning: nested extern declaration of 'PROC_LOCK'
> /usr/src/sys/modules/mac_lomac/../../security/mac_lomac/mac_lomac.c:2190: 
> warning: implicit declaration of function 'PROC_UNLOCK'
> /usr/src/sys/modules/mac_lomac/../../security/mac_lomac/mac_lomac.c:2190: 
> warning: nested extern declaration of 'PROC_UNLOCK'
> *** Error code 1
> 
> Stop in /usr/src/sys/modules/mac_lomac.
> 
> where -Werror is defined - in the Makefile? :)
> Is it safe to just remove it for security/ and continue with build?
> 
> BTW I removed "options      ADAPTIVE_GIANT" as it is unknown with your 
> sources.

Yes, it is gone with 8.0.  Disable the module builds because some of 
them like this one probably need compile fixes.  If you need a subset of 
modules use MODULES_OVERRIDE=list (in /etc/make.conf)

Kris