Date: Thu, 18 Jul 2002 13:48:40 -0400 (EDT) From: John Baldwin <jhb@FreeBSD.org> To: John Baldwin <jhb@FreeBSD.org> Cc: current@FreeBSD.org, dillon@FreeBSD.org Subject: RE: Questions about kern_descrip.c Message-ID: <XFMail.20020718134840.jhb@FreeBSD.org> In-Reply-To: <XFMail.20020718134618.jhb@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 18-Jul-2002 John Baldwin wrote: > In revision 1.94 of kern_descrip.c, in the dup2() syscall a goto > retry loop was added so that if we did a fdalloc() we always retested > everything. Since fd_nfiles can't shrink and new and old can't > change, I don't see why the loop is necessary. Neither dup() or > the F_DUPFD fcntl() were modified in this way either. Also, > calling fdalloc() in this case is somewhat bogus, because fdalloc() > is going to try and reserve an open slot and update variables such > as fd_lastfile, etc. appropriately. Perhaps we should have an > fdextend() function that both dup2() and fdalloc() call? Also, > in do_dup() you have commented out the call to munmapfd() on an > open file with UF_MAPPED and never turned it back on. Was that > intentional or just an accidental oversight? Nevermind about munmapfd(). -- John Baldwin <jhb@FreeBSD.org> <>< http://www.FreeBSD.org/~jhb/ "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?XFMail.20020718134840.jhb>