From owner-cvs-src@FreeBSD.ORG Fri Dec 16 08:25:36 2005 Return-Path: X-Original-To: cvs-src@FreeBSD.org Delivered-To: cvs-src@FreeBSD.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id B757516A41F; Fri, 16 Dec 2005 08:25:36 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.FreeBSD.org (Postfix) with ESMTP id CA3FB43D5A; Fri, 16 Dec 2005 08:25:35 +0000 (GMT) (envelope-from scottl@samsco.org) Received: from [192.168.254.11] (junior.samsco.home [192.168.254.11]) (authenticated bits=0) by pooker.samsco.org (8.13.4/8.13.4) with ESMTP id jBG8PYJH016566; Fri, 16 Dec 2005 01:25:34 -0700 (MST) (envelope-from scottl@samsco.org) Message-ID: <43A279FD.5070705@samsco.org> Date: Fri, 16 Dec 2005 01:25:33 -0700 From: Scott Long User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.8) Gecko/20050615 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Martin Nilsson References: <200512140326.jBE3QnUT010666@repoman.freebsd.org> <70e8236f0512151455g1231cb7oa74aa7d54cfd5b18@mail.gmail.com> <43A26D4A.9080403@samsco.org> <43A27742.2030102@gneto.com> In-Reply-To: <43A27742.2030102@gneto.com> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-1.4 required=3.8 tests=ALL_TRUSTED autolearn=failed version=3.1.0 X-Spam-Checker-Version: SpamAssassin 3.1.0 (2005-09-13) on pooker.samsco.org Cc: cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/amr amr.c amr_cam.c amr_disk.c amr_pci.c amrio.h amrreg.h amrvar.h X-BeenThere: cvs-src@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 16 Dec 2005 08:25:36 -0000 Martin Nilsson wrote: > Scott Long wrote: > >>>> Items 3 and 4 significantly increase the performance of the >>>> driver. On an >>>> LSI 320-2X card, transactions per second went from 13,000 to 31,000 >>>> in my >>> >>> A 238% improvement is at the very least impressive! >> >> >> Yeah, I was shocked at first too. The PCIe controllers can likely do >> even better than that, but my only PCIe test system had other hardware >> related instabilities, so I didn't do much testing with it. > > > How did you test the transaction rate? I Have a 320-2e and 7*10K disks > in RAID5 running 6.0/amd64 at present. I could boot from a SATA disk > with current and see if I get any improvement. As it is now i can > read/write sequential to the array with about 200MB/s (with 256MB write > back cache on the card) but on real world random transactions I'm > limited by the seven 10K disks. > I have a simple test program that spawns a variable number of threads, with each thread trying to read/write variable sized blocks as fast as it can. Kind of a worthless test, but good at saturating the system with I/O, and thus drawing comparisons for driver efficiency. All of my testing is with RAID-0 and caches enabled, since again I just want to measure how well the driver can deliver commands to the card, not how well the card can handle them. But, to put in real-world persective, Kris could untar a full ports tree (80,000+ inodes) in around 30 seconds with the new driver. That's kinda impressive =-) >> I'm a bit worried that old controllers (especially ones that predate the >> move to i960 CPUs) might be broken with this. > > > You mean the old cards with i960 CPUs? The adapters before the Elite > 1600 that uses the old disk model? Sorry I don't have any of those left > anymore but you can always get them on ebay for $20 or so if anyone > still cares about them... > I'm not terribly familiar with the history of MegaRAID, only that there seemed to be an architectural change early on that involved the driver having to use different registers in different ways. Some of the changes in this patch touch that code. I'm also curious if older Elite cards rely on less efficient timing from the driver. Scott