cvs commit: src/sys/kern init_main.c kern_descrip.c sys_pipe.c uipc_syscalls.c uipc_usrreq.c vfs_syscalls.c src/sys/sys filedesc.h

Nate Lawson nate at root.org
Thu Jan 15 10:11:55 PST 2004


On Thu, 15 Jan 2004, Dag-Erling [iso-8859-1] Smørgrav wrote:
> Tim Robbins <tjr at freebsd.org> writes:
> > Thanks for committing this. I'm interested, though, what was the reason you
> > changed it to use a single bitmap instead of two levels?
>
> The increased complexity didn't actually translate into a performance
> improvement.  There was no reason to use two levels instead of one.
> The theoretical advantage is logarithmic rather than linear early
> growth, but in practice it wasn't noticeable.  The cost of the logic
> required to maintain and use the high-level bitmap outweighs the gain
> (which is to save a handful of comparisons against zero in the
> low-level bitmap).

A related question, why wasn't sys/bitstring.h used?

-Nate


More information about the cvs-src mailing list