From owner-cvs-all  Wed Nov 14  7: 8:15 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 63E2737B405; Wed, 14 Nov 2001 07:08:08 -0800 (PST)
Received: (from iedowse@localhost)
	by freefall.freebsd.org (8.11.4/8.11.4) id fAEF88X98138;
	Wed, 14 Nov 2001 07:08:08 -0800 (PST)
	(envelope-from iedowse)
Message-Id: <200111141508.fAEF88X98138@freefall.freebsd.org>
From: Ian Dowse <iedowse@FreeBSD.org>
Date: Wed, 14 Nov 2001 07:08:08 -0800 (PST)
To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject: cvs commit: src/sys/ufs/ufs ufs_dirhash.c
X-FreeBSD-CVS-Branch: HEAD
Sender: owner-cvs-all@FreeBSD.ORG
Precedence: bulk
List-ID: <cvs-all.FreeBSD.ORG>
List-Archive: <http://docs.freebsd.org/mail/> (Web Archive)
List-Help: <mailto:majordomo@FreeBSD.ORG?subject=help> (List Instructions)
List-Subscribe: <mailto:majordomo@FreeBSD.ORG?subject=subscribe%20cvs-all>
List-Unsubscribe: <mailto:majordomo@FreeBSD.ORG?subject=unsubscribe%20cvs-all>
X-Loop: FreeBSD.ORG

iedowse     2001/11/14 07:08:07 PST

  Modified files:
    sys/ufs/ufs          ufs_dirhash.c 
  Log:
  Oops, when trying the dirhash sequential-access optimisation,
  compare the slot offset against the predicted offset, not a boolean
  flag. This typo effectively disabled the sequential optimisation,
  but was otherwise harmless.
  
  Not surprisingly, fixing this improves performance in the sequential
  access case. I am seeing a 7% speedup on one machine here; using
  dirhash when sequentially looking up directory entries is now about
  5% faster instead of 2% slower than the non-dirhash case.
  
  Submitted by:	KOIE Hidetaka <koie@suri.co.jp>
  MFC after:	1 week
  
  Revision  Changes    Path
  1.7       +1 -1      src/sys/ufs/ufs/ufs_dirhash.c

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message