cvs commit: src/sys/kern imgact_elf.c init_main.c kern_checkpoint.c kern_descrip.c kern_event.c sys_generic.c sys_pipe.c uipc_syscalls.c uipc_usrreq.c vfs_aio.c vfs_syscalls.c src/sys/sys filedesc.h src/sys/dev/misc/streams streams.c ...

Hiten Pandya hmp at backplane.com
Wed Jun 22 09:35:55 PDT 2005


Joerg Sonnenberger wrote:
On Wed, Jun 22, 2005 at 01:13:01PM +0200, Simon 'corecode' Schubert wrote:

Lately Matthew Dillon <dillon at xxxxxxxxxxxxxxxxxxxxxxx> said:

 Log:
 File descriptor cleanup stage 2, remove the separate arrays for file
 pointers, fileflags, and allocation counts and replace the mess with a
 single structural array.  Also revamp the code that checks whether the
 file descriptor array is built-in or allocated.
can anybody enlighten me why this was done in the first place?
this way it's much clearer anyways


But it is also slower because the amount of data which has to be copied when
the array is extended is now doubled. (pointer vs. struct fdnode)
Joerg
	Can't we not just make them all allocated?  I am assuming here that
	allocator descriptor arrays is "pointer" and the other is "fdnode",
	according to your interpretation, right?
					Hiten Pandya





More information about the Commits mailing list