From owner-cvs-all Sun Jun 23 19:23:46 2002 Delivered-To: cvs-all@freebsd.org Received: from overcee.wemm.org (12-232-114-102.client.attbi.com [12.232.114.102]) by hub.freebsd.org (Postfix) with ESMTP id E690737B403; Sun, 23 Jun 2002 19:23:40 -0700 (PDT) Received: from wemm.org (localhost [127.0.0.1]) by overcee.wemm.org (Postfix) with ESMTP id 3FDDB380E; Sun, 23 Jun 2002 19:23:43 -0700 (PDT) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Ian Dowse Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/powerpc/include types.h src/sys/arm/include types.h src/sys/i386/include types.h In-Reply-To: <200206232157.g5NLvKF31769@freefall.freebsd.org> Date: Sun, 23 Jun 2002 19:23:43 -0700 From: Peter Wemm Message-Id: <20020624022343.3FDDB380E@overcee.wemm.org> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Ian Dowse wrote: > iedowse 2002/06/23 14:57:19 PDT > > Modified files: > sys/powerpc/include types.h > sys/arm/include types.h > sys/i386/include types.h > Log: > Make vm_pindex_t 64-bit on all platforms. This is necessary to avoid > overflows with the large file sizes that UFS2 permits. Ouch, is this really necessary? The whole index::offset thing was explicitly to avoid using 64 bit arithmetic here due to the performance hit. Dyson posted some numbers somewhere in the distant past, they can probably be found on the archives. If this really is necessary, then we may as well strip out the pindex stuff entirely and just use 64 bit byte offsets instead of "64bit pindex::32bit offset". ie: why carry a 96 bit synthetic value when we only need a 64 bit value? Cheers, -Peter -- Peter Wemm - peter@wemm.org; peter@FreeBSD.org; peter@yahoo-inc.com "All of this is for nothing if we don't go to the stars" - JMS/B5 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message