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>