Date: Fri, 13 Jul 2001 13:50:38 -0700 (PDT) From: Ian Dowse <iedowse@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/sys/ufs/ufs dirhash.h ufs_dirhash.c ufs_lookup.c Message-ID: <200107132050.f6DKocp62372@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200107132050.f6DKocp62372>