Received: with ECARTIS (v1.0.0; list xfs); Tue, 03 Oct 2006 14:51:08 -0700 (PDT) Received: from cuda.sgi.com (cuda1.sgi.com [192.48.168.28]) by oss.sgi.com (8.12.10/8.12.10/SuSE Linux 0.7) with ESMTP id k93LowaG011326 for ; Tue, 3 Oct 2006 14:50:59 -0700 X-ASG-Debug-ID: 1159912217-5975-765-0 X-Barracuda-URL: http://cuda.sgi.com:80/cgi-bin/mark.cgi Received: from smtp113.sbc.mail.mud.yahoo.com (smtp113.sbc.mail.mud.yahoo.com [68.142.198.212]) by cuda.sgi.com (Spam Firewall) with SMTP id 0F4C146803E for ; Tue, 3 Oct 2006 14:50:17 -0700 (PDT) Received: (qmail 83577 invoked from network); 3 Oct 2006 21:23:37 -0000 Received: from unknown (HELO stupidest.org) (cwedgwood@sbcglobal.net@24.5.75.45 with login) by smtp113.sbc.mail.mud.yahoo.com with SMTP; 3 Oct 2006 21:23:37 -0000 Received: by tuatara.stupidest.org (Postfix, from userid 10000) id 7ED381824260; Tue, 3 Oct 2006 14:23:35 -0700 (PDT) Date: Tue, 3 Oct 2006 14:23:35 -0700 From: Chris Wedgwood To: David Chinner Cc: xfs-dev@sgi.com, xfs@oss.sgi.com, dhowells@redhat.com, LKML X-ASG-Orig-Subj: Re: [RFC 0/3] Convert XFS inode hashes to radix trees Subject: Re: [RFC 0/3] Convert XFS inode hashes to radix trees Message-ID: <20061003212335.GA13120@tuatara.stupidest.org> References: <20061003060610.GV3024@melbourne.sgi.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20061003060610.GV3024@melbourne.sgi.com> X-Barracuda-Spam-Score: 0.00 X-Barracuda-Spam-Status: No, SCORE=0.00 using per-user scores of TAG_LEVEL=3.5 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=9.0 tests= X-Barracuda-Spam-Report: Code version 3.02, rules version 3.0.22497 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- X-archive-position: 9160 X-ecartis-version: Ecartis v1.0.0 Sender: xfs-bounce@oss.sgi.com Errors-to: xfs-bounce@oss.sgi.com X-original-sender: cw@f00f.org Precedence: bulk X-list: xfs On Tue, Oct 03, 2006 at 04:06:10PM +1000, David Chinner wrote: > Overall, the patchset removes more than 200 lines of code from the > xfs inode caching and lookup code and provides more consistent > scalability for large numbers of cached inodes. The only down side > is that it limits us to 32 bit inode numbers of 32 bit platforms due > to the way the radix tree uses unsigned longs for it's indexes commit afefdbb28a0a2af689926c30b94a14aea6036719 tree 6ee500575cac928cd90045bcf5b691cf2b8daa09 parent 1d32849b14bc8792e6f35ab27dd990d74b16126c author David Howells 1159863226 -0700 committer Linus Torvalds 1159887820 -0700 [PATCH] VFS: Make filldir_t and struct kstat deal in 64-bit inode numbers These patches make the kernel pass 64-bit inode numbers internally when communicating to userspace, even on a 32-bit system. They are required because some filesystems have intrinsic 64-bit inode numbers: NFS3+ and XFS for example. The 64-bit inode numbers are then propagated to userspace automatically where the arch supports it. [...] Doing this will mean XFS won't be able to support 32-bit inodes on 32-bit platforms the above (merged) patch --- though given that cheap 64-bit systems are now abundant does anyone really care?