From owner-cvs-all Fri Jul 13 13:50:51 2001 Delivered-To: cvs-all@freebsd.org Received: from freefall.freebsd.org (freefall.freebsd.org [216.136.204.21]) by hub.freebsd.org (Postfix) with ESMTP id 7842737B42F; Fri, 13 Jul 2001 13:50:38 -0700 (PDT) (envelope-from iedowse@FreeBSD.org) Received: (from iedowse@localhost) by freefall.freebsd.org (8.11.4/8.11.4) id f6DKocp62372; Fri, 13 Jul 2001 13:50:38 -0700 (PDT) (envelope-from iedowse) Message-Id: <200107132050.f6DKocp62372@freefall.freebsd.org> From: Ian Dowse Date: Fri, 13 Jul 2001 13:50:38 -0700 (PDT) To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c ufs_lookup.c X-FreeBSD-CVS-Branch: HEAD 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 iedowse 2001/07/13 13:50:38 PDT Modified files: sys/ufs/ufs dirhash.h ufs_dirhash.c ufs_lookup.c Log: Return a locked struct buf from ufsdirhash_lookup() to avoid one extra getblk/brelse sequence for each lookup. We already had this buf in ufsdirhash_lookup(), so there was no point in brelse'ing it only to have the caller immediately reaquire the same buffer. This should make the case of sequential lookups marginally faster; in my tests, sequential lookups with dirhash enabled are now only around 1% slower than without dirhash. Revision Changes Path 1.2 +3 -2 src/sys/ufs/ufs/dirhash.h 1.2 +5 -4 src/sys/ufs/ufs/ufs_dirhash.c 1.51 +4 -6 src/sys/ufs/ufs/ufs_lookup.c To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message