From owner-freebsd-sparc64@FreeBSD.ORG Tue Sep 6 15:21:35 2005 Return-Path: X-Original-To: freebsd-sparc64@freebsd.org Delivered-To: freebsd-sparc64@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id AEB5716A41F for ; Tue, 6 Sep 2005 15:21:35 +0000 (GMT) (envelope-from imura@ryu16.org) Received: from mail.ryu16.org (221x249x107x69.ap221.ftth.ucom.ne.jp [221.249.107.69]) by mx1.FreeBSD.org (Postfix) with ESMTP id D921B43D46 for ; Tue, 6 Sep 2005 15:21:33 +0000 (GMT) (envelope-from imura@ryu16.org) Received: from redeye.xt.ryu16.org (localhost [127.0.0.1]) by mail.ryu16.org (8.12.9p1/8.12.9) with ESMTP id j86FLLgr054354; Wed, 7 Sep 2005 00:21:21 +0900 (JST) (envelope-from imura@redeye.xt.ryu16.org) Received: (from imura@localhost) by redeye.xt.ryu16.org (8.12.9p1/8.12.9/Submit) id j86FLKuP054353; Wed, 7 Sep 2005 00:21:20 +0900 (JST) (envelope-from imura) Date: Wed, 7 Sep 2005 00:21:20 +0900 From: "R. Imura" To: Pyun YongHyeon Message-ID: <20050906152119.GA49672%imura@ryu16.org> References: <200508230911.18163.john@jnielsen.net> <20050824010147.GD4956@rndsoft.co.kr> <20050824143940.GA65078%imura@ryu16.org> <20050824191437.A36508@newtrinity.zeist.de> <20050827055841.GB83681%imura@ryu16.org> <20050827145503.G19225@newtrinity.zeist.de> <20050827132008.GA28954%imura@ryu16.org> <20050827153704.H19225@newtrinity.zeist.de> <20050827140525.GA29949%imura@ryu16.org> <20050829031058.GA871@rndsoft.co.kr> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20050829031058.GA871@rndsoft.co.kr> User-Agent: Mutt/1.4.1i-ja.1 Cc: John Nielsen , freebsd-sparc64@freebsd.org Subject: Re: "fast data access mmu miss" on kernels w/o "makeoptions DEBUG=-g" X-BeenThere: freebsd-sparc64@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Porting FreeBSD to the Sparc List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 06 Sep 2005 15:21:35 -0000 On Mon, Aug 29, 2005 at 12:10:58PM +0900, Pyun YongHyeon wrote: > Supporting smbfs on sparc64 is on my TODO list. But it's priority > is low due to a lack of time and other pending works. Based on > NetBSD's changes I managed to build smbfs. But it didn't work and it > needs more investigation. While reading the code I noticed there > is a bug in smbfs_hash() in sys/fs/smbfs/smbfs_node.c. NetBSD > switched to use hash32_strn() defined in . I think it > also has influence on our implementation. Would you please check > the function? > (http://cvsweb.netbsd.org/bsdweb.cgi/src/sys/fs/smbfs/smbfs_node.c, rev. 1.12) IMHO, we are not fully required to fix this problem at first because the mission of smbfs_hash() is to create unique i-node number. (however smbfs_hash can be replaced by any hash algorithm, though. Our smbfs_hash is called FNV hash according to google.) I hope you also look at Darwin's code, because I believe it may bring us some benefit easily if we keep differences small between Darwin's and ours, since Darwin has many improvement. For example, NetBSD had introduced SMBRQ_PUTLE16 macro but Darwin hadn't. I think this doesn't mean SMBRQ_PUTLE16 is an essential feature. BSD licensed Darwin's smbfs can be found at http://darwinsource.opendarwin.org/ . BTW, I found we can obtain macros from NetBSD in contrib/smbfs/include/netsmb/smb_lib.h. - R. Imura