From owner-cvs-src@FreeBSD.ORG Thu Jan 15 03:11:55 2004 Return-Path: Delivered-To: cvs-src@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6005E16A4CE; Thu, 15 Jan 2004 03:11:55 -0800 (PST) Received: from smtp02.syd.iprimus.net.au (smtp02.syd.iprimus.net.au [210.50.76.52]) by mx1.FreeBSD.org (Postfix) with ESMTP id 8E9CA43D66; Thu, 15 Jan 2004 03:11:53 -0800 (PST) (envelope-from tim@robbins.dropbear.id.au) Received: from robbins.dropbear.id.au (210.50.32.85) by smtp02.syd.iprimus.net.au (7.0.020) id 3F8F522A01A3C609; Thu, 15 Jan 2004 22:11:51 +1100 Received: by robbins.dropbear.id.au (Postfix, from userid 1000) id 4CECF4152; Thu, 15 Jan 2004 22:12:19 +1100 (EST) Date: Thu, 15 Jan 2004 22:12:19 +1100 From: Tim Robbins To: Dag-Erling Smorgrav Message-ID: <20040115111219.GA20914@cat.robbins.dropbear.id.au> References: <200401151015.i0FAF49u009868@repoman.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200401151015.i0FAF49u009868@repoman.freebsd.org> User-Agent: Mutt/1.4.1i cc: cvs-src@FreeBSD.org cc: src-committers@FreeBSD.org cc: cvs-all@FreeBSD.org Subject: Re: 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 X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 15 Jan 2004 11:11:55 -0000 On Thu, Jan 15, 2004 at 02:15:04AM -0800, Dag-Erling Smorgrav wrote: > des 2004/01/15 02:15:04 PST > > FreeBSD src repository > > Modified files: > sys/kern init_main.c kern_descrip.c sys_pipe.c > uipc_syscalls.c uipc_usrreq.c > vfs_syscalls.c > sys/sys filedesc.h > Log: > New file descriptor allocation code, derived from similar code introduced > in OpenBSD by Niels Provos. The patch introduces a bitmap of allocated > file descriptors which is used to locate available descriptors when a new > one is needed. It also moves the task of growing the file descriptor table > out of fdalloc(), reducing complexity in both fdalloc() and do_dup(). Thanks for committing this. I'm interested, though, what was the reason you changed it to use a single bitmap instead of two levels? Tim