On Sat, Dec 31, 2016 at 05:03:18PM -0800, Nick Gonella wrote:
> Hey Misc,
> As I read through the code, especially in the boot code, there
> seem to be quite a few blocks of code of the style:
> #if 0
> /* some code here */
> On example of this is in sys/arch/amd64/amd64/pmap.c:2326
> #if 0
> Obviously, this code needs to be modified if we want it to be
> included, but my question is, is this old, dead code, or is
> there some reason it's still in the source?
Considering cvs history says it's been disabled since 2007 (the 2014
change by sf@ was whitespace removal), I'd say it's old and dead at
best, with very little probability of ever being used or useful again.
Maybe this will summon The Tedu to go on a stampede in there to finish
it off. ;-)
$ cvs log -r1.30 pmap.c | sed '1,/---/d'
date: 2007/12/09 00:24:04; author: tedu; state: Exp; lines: +10 -9;
big patch to simplify pool code.
remove pool_cache code. it was barely used, and quite complex. it's
silly to have both a "fast" and "faster" allocation interface. provide
a ctor/dtor interface, and convert the few cache users to use it. no
caching at this time.
use mutexes to protect pools. they should be initialized with pool_setipl
if the pool may be used in an interrupt context, without existing spl
ok art deraadt thib