Skip site navigation (1)Skip section navigation (2)
Date:      26 Feb 2003 15:19:07 +1030
From:      "Daniel O'Connor" <doconnor@gsoft.com.au>
To:        Scott Long <scottl@FreeBSD.ORG>
Cc:        src-committers@FreeBSD.ORG, cvs-src@FreeBSD.ORG, cvs-all@FreeBSD.ORG
Subject:   Re: cvs commit: src/sys/dev/aac aac.c aac_cam.c aac_disk.c aacvar.h
Message-ID:  <1046234946.45422.98.camel@chowder.gsoft.com.au>
In-Reply-To: <200302260446.h1Q4kLO6042833@repoman.freebsd.org>
References:  <200302260446.h1Q4kLO6042833@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, 2003-02-26 at 15:16, Scott Long wrote:
>   - the mutex aac_io_lock protects the main codepaths which handle queues and
>     hardware registers.  Only one acquire/release is done in the top-half and
>     the taskqueue.  This mutex also applies to the userland command path and
>     CAM data path.
>   - Move the taskqueue to the new Giant-free version.
>   - Register the disk device with DISKFLAG_NOGIANT so the top-half processing
>     runs without Giant.
>   - Move the dynamic command allocator to the worker thread to avoid locking
>     issues with bus_dmamem_alloc().
>   
>   This gives about 20% improvement in most of my benchmarks.

Are the tests on an SMP machine?

-- 
Daniel O'Connor software and network engineer
for Genesis Software - http://www.gsoft.com.au
"The nice thing about standards is that there
are so many of them to choose from."
  -- Andrew Tanenbaum
GPG Fingerprint - 9A8C 569F 685A D928 5140  AE4B 319B 41F4 5D17 FDD5


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




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?1046234946.45422.98.camel>