From owner-freebsd-current@FreeBSD.ORG Fri May 7 03:02:24 2004 Return-Path: Delivered-To: freebsd-current@www.freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 6816416A4CE for ; Fri, 7 May 2004 03:02:24 -0700 (PDT) Received: from smtp01.syd.iprimus.net.au (smtp01.syd.iprimus.net.au [210.50.30.52]) by mx1.FreeBSD.org (Postfix) with ESMTP id 984F743D62 for ; Fri, 7 May 2004 03:02:23 -0700 (PDT) (envelope-from tim@robbins.dropbear.id.au) Received: from robbins.dropbear.id.au (210.50.201.139) by smtp01.syd.iprimus.net.au (7.0.024) id 409956B4000CFFA5; Fri, 7 May 2004 20:01:44 +1000 Received: by robbins.dropbear.id.au (Postfix, from userid 1000) id 884E241E5; Fri, 7 May 2004 20:01:19 +1000 (EST) Date: Fri, 7 May 2004 20:01:19 +1000 From: Tim Robbins To: Marc Olzheim Message-ID: <20040507100119.GA15782@cat.robbins.dropbear.id.au> References: <20040507092235.GA61837@stack.nl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20040507092235.GA61837@stack.nl> User-Agent: Mutt/1.4.1i cc: Bruce M Simpson cc: Poul-Henning Kamp cc: freebsd-current@www.freebsd.org Subject: Re: Unified getcwd() implementation X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 07 May 2004 10:02:24 -0000 On Fri, May 07, 2004 at 11:22:35AM +0200, Marc Olzheim wrote: > Hi, > > (Re: http://lists.freebsd.org/pipermail/freebsd-arch/2003-August/001152.html) > > > Yes, it's quite an old patch, and much has happened since it was written. > > Mostly some fine-grained locking was introduced. > > I hope I got everything covered. Here Bruce's patch reworked, that works > for me. (even over NFS ;-)) > > Anyone care to share their view on it ? Why is this necessary? Emulation of the Linux getcwd() syscall belongs in the Linux emulator. There is also some fairly blatantly plagiarised code in this patch. Tim