From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from az33egw02.freescale.net (az33egw02.freescale.net [192.88.158.103]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "az33egw02.freescale.net", Issuer "Thawte Premium Server CA" (verified OK)) by ozlabs.org (Postfix) with ESMTPS id 089CADDE11 for ; Tue, 28 Apr 2009 06:47:09 +1000 (EST) Message-ID: <49F619C4.20100@freescale.com> Date: Mon, 27 Apr 2009 15:47:00 -0500 From: Timur Tabi MIME-Version: 1.0 To: Ira Snyder Subject: Re: [PATCH] fsldma: use PCI Read Multiple command References: <20090424183517.GB23140@ovro.caltech.edu> <49F608B7.9080409@ovro.caltech.edu> <49F60A3A.4060402@freescale.com> <49F61416.8040501@ovro.caltech.edu> <49F614FC.70000@freescale.com> <20090427204213.GA4960@ovro.caltech.edu> In-Reply-To: <20090427204213.GA4960@ovro.caltech.edu> Content-Type: text/plain; charset=ISO-8859-1 Cc: David Hawkins , Liu Dave-R63238 , linux-kernel@vger.kernel.org, linuxppc-dev@ozlabs.org, Dan Williams List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Adding Kumar to the CC: list, since he might pick up the patch. Ira Snyder wrote: > From 73e42fa58c93de8d4d429ba8e069b60c42037b58 Mon Sep 17 00:00:00 2001 > From: Ira W. Snyder > Date: Thu, 23 Apr 2009 16:17:54 -0700 > Subject: [PATCH] fsldma: use PCI Read Multiple command > > By default, the Freescale 83xx DMA controller uses the PCI Read Line > command when reading data over the PCI bus. Setting the controller to use > the PCI Read Multiple command instead allows the controller to read much > larger bursts of data, which provides a drastic speed increase. > > The slowdown due to using PCI Read Line was only observed when a PCI-to-PCI > bridge was between the devices trying to communicate. > > A simple test driver showed an increase from 4MB/sec to 116MB/sec when > performing DMA over the PCI bus. Using DMA to transfer between blocks of > local SDRAM showed no change in performance with this patch. The dmatest > driver was also used to verify the correctness of the transfers, and showed > no errors. > > Signed-off-by: Ira W. Snyder Acked-by: Timur Tabi -- Timur Tabi Linux kernel developer at Freescale