Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 01 Jun 2009 10:42:04 -0700
From:      Julian Elischer <julian@elischer.org>
To:        "Bjoern A. Zeeb" <bz@FreeBSD.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r193232 - in head: . sys/net sys/netinet sys/netinet6 sys/nfsclient sys/sys usr.bin/netstat
Message-ID:  <4A2412EC.6020009@elischer.org>
In-Reply-To: <20090601155120.D12292@maildrop.int.zabbadoz.net>
References:  <200906011549.n51FngRA083299@svn.freebsd.org> <20090601155120.D12292@maildrop.int.zabbadoz.net>

next in thread | previous in thread | raw e-mail | index | archive | help
Bjoern A. Zeeb wrote:
> On Mon, 1 Jun 2009, Bjoern A. Zeeb wrote:
> 
>> Author: bz
>> Date: Mon Jun  1 15:49:42 2009
>> New Revision: 193232
>> URL: http://svn.freebsd.org/changeset/base/193232
>>
>> Log:
>>  Convert the two dimensional array to be malloced and introduce
>>  an accessor function to get the correct rnh pointer back.
>>
>>  Update netstat to get the correct pointer using kvm_read()
>>  as well.
>>
>>  This not only fixes the ABI problem depending on the kernel
>>  option but also permits the tunable to overwrite the kernel
>>  option at boot time up to MAXFIBS, enlarging the number of
>>  FIBs without having to recompile. So people could just use
>>  GENERIC now.
>>
>>  Reviewed by:    julian, rwatson, zec
>>  X-MFC:        not possible
> 
> The solution is not ideal but will help FreeBSD 8.x.

The previous code was needed for 7.0 and 6.0 compatibility.
8 can break that ABI so this is an excellent step in the right direction.

we could look at making multiple fibs not need any compile option
but always be enabled and control it purely through NUMFIB -> 1


> 
> Julian has suggested a cleaner way but I considered that to be to
> late and intrusive for 8.x; this way we will have enough time for
> 9.x to convert this to per AF/domain handler routines.
> 
> 
> Note: I will garbage collect a lot of opt_route.h dependencies by the
> end of the week as that option no longer has to be visible to most of
> the tree.
> 
> 
> PS: I cannot spell 'routing' correctly usually so whoever touches
> UPDATING next please fix the spelling;-)
> 




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